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

[cloud images] drop jmx and tools packages #524

Merged
merged 1 commit into from
May 7, 2024

Conversation

yaronkaikov
Copy link
Collaborator

Following the work done in scylladb/scylladb@b8634fb, we no longer installing jmx and tools-java packages

Removing it from the machine image as well

This should be merged only after scylladb/scylladb#18494

@yaronkaikov yaronkaikov requested review from a team and syuu1228 May 2, 2024 10:07
@mykaul
Copy link

mykaul commented May 2, 2024

I'm not sure - did we completely deprecate them, or left them as optional?

@yaronkaikov
Copy link
Collaborator Author

I'm not sure - did we completely deprecate them, or left them as optional?

As optional, but we don't install them. If someone want to install it manually it is still part of the repo

@fruch
Copy link
Collaborator

fruch commented May 2, 2024

@yaronkaikov

this is gonna break all SCT artifact tests, since they c-s is available...

@yaronkaikov
Copy link
Collaborator Author

c-s

The packages will not be installed, since Scylla doesn't automatically install them now. I think SCT should be fixed regardless

@yaronkaikov
Copy link
Collaborator Author

Waiting for scylladb/scylla-cluster-tests#7397

Following the work done in
scylladb/scylladb@b8634fb,
we no longer installing jmx and tools-java packages

Removing it from machine image as well
@yaronkaikov
Copy link
Collaborator Author

@yaronkaikov yaronkaikov merged commit 4644f6a into scylladb:next May 7, 2024
1 check passed
@yaronkaikov yaronkaikov deleted the drop-jmx-tools branch May 7, 2024 06:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants