Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

fix: resolve container-image label and Strimzi API index warnings #917

Merged
merged 1 commit into from
Jul 19, 2024

Conversation

MikeEdgar
Copy link
Member

@MikeEdgar MikeEdgar commented Jul 18, 2024

Resolve errors in the API logs about unknown quarkus.container-image.labels.* properties. These properties apply at build time only, so this PR uses a build profile to apply them only at that time. Additionally, this resolves a warning about the Strimzi API classes not being present in the Jandex index.

Warning for unknown runtime properties:

2024-07-18 14:53:42,089 WARN  [io.qua.config] (Quarkus Main Thread) Unrecognized configuration key "quarkus.container-image.labels."org.opencontainers.image.revision"" was provided; it will be ignored; verify that the dependency extension for this configuration is set or that you did not make a typo
2024-07-18 14:53:42,089 WARN  [io.qua.config] (Quarkus Main Thread) Unrecognized configuration key "quarkus.container-image.labels."org.opencontainers.image.version"" was provided; it will be ignored; verify that the dependency extension for this configuration is set or that you did not make a typo

Warning for unindexed classes:

2024-07-18 15:05:09,473 WARN  [io.qua.kub.cli.dep.KubernetesClientProcessor] (build-30) Unable to lookup class: io.strimzi.api.kafka.model.topic.KafkaTopic
2024-07-18 15:05:09,473 WARN  [io.qua.kub.cli.dep.KubernetesClientProcessor] (build-30) Unable to lookup class: io.strimzi.api.kafka.model.kafka.Kafka

@MikeEdgar MikeEdgar added this to the 0.2.1 milestone Jul 18, 2024
@MikeEdgar MikeEdgar requested a review from a team July 18, 2024 19:23
@MikeEdgar MikeEdgar added the ready for review Pull request is ready for a static review label Jul 18, 2024
@MikeEdgar MikeEdgar enabled auto-merge (squash) July 19, 2024 11:38
@MikeEdgar MikeEdgar added ready for merge Pull request was reviewed, approved and is ready to be merged by maintainers. and removed ready for review Pull request is ready for a static review labels Jul 19, 2024
Copy link

@MikeEdgar MikeEdgar merged commit 29d9b40 into main Jul 19, 2024
10 checks passed
@MikeEdgar MikeEdgar deleted the fix-warnings branch July 19, 2024 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for merge Pull request was reviewed, approved and is ready to be merged by maintainers.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants