scylladb / scylla-machine-image

Apache License 2.0
18 stars 25 forks source link

[cloud images] drop jmx and tools packages #524

Closed yaronkaikov closed 1 month ago

yaronkaikov commented 2 months ago

Following the work done in https://github.com/scylladb/scylladb/commit/b8634fb244ab08c2ffa66700745b06e21ee57f30, we no longer installing jmx and tools-java packages

Removing it from the machine image as well

This should be merged only after https://github.com/scylladb/scylladb/pull/18494

mykaul commented 2 months ago

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

yaronkaikov commented 2 months ago

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 commented 2 months ago

@yaronkaikov

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

yaronkaikov commented 2 months ago

c-s

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

yaronkaikov commented 1 month ago

Waiting for https://github.com/scylladb/scylla-cluster-tests/issues/7397

yaronkaikov commented 1 month ago

Waiting for scylladb/scylla-cluster-tests#7397

was merged , verified with https://jenkins.scylladb.com/job/scylla-master/job/releng-testing/job/next-machine-image/321/

@benipeled @Annamikhlin please review