Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

terrascan 1.19.6 #184483

Closed
wants to merge 1 commit into from
Closed

terrascan 1.19.6 #184483

wants to merge 1 commit into from

Conversation

BrewTestBot
Copy link
Member

Created by brew bump


Created with brew bump-formula-pr.

@github-actions github-actions bot added go Go use is a significant feature of the PR or issue bump-formula-pr PR was created using `brew bump-formula-pr` labels Sep 10, 2024
@chenrui333
Copy link
Member

chenrui333 commented Sep 10, 2024

==> /opt/homebrew/Cellar/terrascan/1.19.6/bin/terrascan scan -f /private/tmp/terrascan-test-20240910-33760-vlpqhj/ami.tf -t aws
panic: proto: file "extensions/extension.proto" is already registered
		previously from: "github.com/google/gnostic-models/extensions"
		currently from:  "github.com/googleapis/gnostic/extensions"
	See https://protobuf.dev/reference/go/faq#namespace-conflict


goroutine 1 [running]:
google.golang.org/protobuf/reflect/protoregistry.init.func1({0x107151700?, 0x10570f640?}, {0x1057b3720, 0x140001b2320})
	google.golang.org/protobuf@v1.31.0/reflect/protoregistry/registry.go:56 +0x1f8
google.golang.org/protobuf/reflect/protoregistry.(*Files).RegisterFile(0x140002224f8, {0x1057f13f8, 0x140001836c0})
	google.golang.org/protobuf@v1.31.0/reflect/protoregistry/registry.go:130 +0x9b0
google.golang.org/protobuf/internal/filedesc.Builder.Build({{0x104caf585, 0x28}, {0x1071b87e0, 0x282, 0x282}, 0x0, 0x4, 0x0, 0x0, {0x1057c1a98, ...}, ...})
	google.golang.org/protobuf@v1.31.0/internal/filedesc/build.go:112 +0x1a0
google.golang.org/protobuf/internal/filetype.Builder.Build({{{0x104caf585, 0x28}, {0x1071b87e0, 0x282, 0x282}, 0x0, 0x4, 0x0, 0x0, {0x0, ...}, ...}, ...})
	google.golang.org/protobuf@v1.31.0/internal/filetype/build.go:138 +0x158
github.com/googleapis/gnostic/extensions.file_extensions_extension_proto_init()
	github.com/googleapis/gnostic@v0.5.5/extensions/extension.pb.go:456 +0x144
github.com/googleapis/gnostic/extensions.init.0()
	github.com/googleapis/gnostic@v0.5.5/extensions/extension.pb.go:388 +0x1c

/opt/homebrew/Library/Homebrew/ignorable.rb:27:in `block in raise'
Minitest::Assertion: Expected: 0
  Actual: 2


@chenrui333 chenrui333 added upstream issue An upstream issue report is needed test failure CI fails while running the test-do block labels Sep 10, 2024
terrascan: add ldflag config for protobuf

Signed-off-by: Rui Chen <rui@chenrui.dev>
@chenrui333 chenrui333 force-pushed the bump-terrascan-1.19.6 branch from 8f17148 to 19874a3 Compare September 10, 2024 14:09
@chenrui333 chenrui333 added ready to merge PR can be merged once CI is green pre-release Artifact is pre-release and removed upstream issue An upstream issue report is needed test failure CI fails while running the test-do block ready to merge PR can be merged once CI is green labels Sep 10, 2024
@chenrui333
Copy link
Member

image

@chenrui333 chenrui333 closed this Sep 10, 2024
@github-actions github-actions bot deleted the bump-terrascan-1.19.6 branch September 10, 2024 17:23
@stefanb stefanb mentioned this pull request Sep 18, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bump-formula-pr PR was created using `brew bump-formula-pr` go Go use is a significant feature of the PR or issue pre-release Artifact is pre-release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants