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: cve fix for kafka and jmx-javaagent #145

Merged
merged 10 commits into from
Sep 18, 2024
Merged

Conversation

SandhyaRavi2403
Copy link
Contributor

@SandhyaRavi2403 SandhyaRavi2403 self-assigned this Sep 13, 2024
@SandhyaRavi2403 SandhyaRavi2403 added the ok-to-test Trigger all configured tests label Sep 13, 2024
@SandhyaRavi2403 SandhyaRavi2403 force-pushed the sandhya/kafka-cve-fix branch 2 times, most recently from 2ed08d2 to eea9b3d Compare September 14, 2024 19:04
Copy link
Contributor

@mhrabovcin mhrabovcin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@SandhyaRavi2403 we need keep the kafka 0.25.1 version in the /services directory and add the 0.25.2 with image bumps and changes.

@SandhyaRavi2403
Copy link
Contributor Author

@SandhyaRavi2403 we need keep the kafka 0.25.1 version in the /services directory and add the 0.25.2 with image bumps and changes.

@mhrabovcin the version is kept as is version: 0.25.1, wherein the bump is updated. Do you mean not bump, get back to previous one /services/0.25.1

@mhrabovcin
Copy link
Contributor

@SandhyaRavi2403 we will end up with 2 directories in the services dir. 0.25.1 and 0.25.2. We need to keep 0.25.1 as it is and add 0.25.2 with image bumps.

Copy link
Contributor

@msdolbey msdolbey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍
it is though

@mhrabovcin
Copy link
Contributor

@SandhyaRavi2403 the last commit title needs to be fixed here.

@mhrabovcin mhrabovcin merged commit f70f780 into main Sep 18, 2024
3 checks passed
@mhrabovcin mhrabovcin deleted the sandhya/kafka-cve-fix branch September 18, 2024 09:00
SandhyaRavi2403 added a commit that referenced this pull request Sep 18, 2024
* fix: cve fix for kafka and jmx-javaagent

* fix: cve fix for kafka and jmx-javaagent

* fix: cve fix for kafka and jmx-javaagent

* fix: cve fix for kafka and jmx-javaagent

* fix: cve fix for kafka and jmx-javaagent

* fix: cve fix for kafka and jmx-javaagent

* fix: cve fix for kafka and jmx-javaagent

* fix: cve fix for kafka and jmx-javaagent

* fix: cve fix for kafka and jmx-javaagent

* fix: Update services/kafka-operator/0.25.2/kafka-operator.yaml

Co-authored-by: Martin Hrabovcin <[email protected]>

---------

Co-authored-by: Martin Hrabovcin <[email protected]>
SandhyaRavi2403 added a commit that referenced this pull request Sep 18, 2024
* fix: cve fix for kafka and jmx-javaagent

* fix: cve fix for kafka and jmx-javaagent

* fix: cve fix for kafka and jmx-javaagent

* fix: cve fix for kafka and jmx-javaagent

* fix: cve fix for kafka and jmx-javaagent

* fix: cve fix for kafka and jmx-javaagent

* fix: cve fix for kafka and jmx-javaagent

* fix: cve fix for kafka and jmx-javaagent

* fix: cve fix for kafka and jmx-javaagent

* fix: Update services/kafka-operator/0.25.2/kafka-operator.yaml

Co-authored-by: Martin Hrabovcin <[email protected]>

---------

Co-authored-by: Martin Hrabovcin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ok-to-test Trigger all configured tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants