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

[Bug] Remove mentions of JMX in Credential Database #573

Closed
tthvo opened this issue May 1, 2023 · 2 comments
Closed

[Bug] Remove mentions of JMX in Credential Database #573

tthvo opened this issue May 1, 2023 · 2 comments
Labels
bug Something isn't working

Comments

@tthvo
Copy link
Member

tthvo commented May 1, 2023

Current Behavior

Currently, we are still mentioning JMX in Database Configirations

{
...omitted-output...
	// Options to configure the Cryostat application's JMX credentials database.
	// +optional
	// +operator-sdk:csv:customresourcedefinitions:type=spec,displayName="JMX Credentials Database Options"
	JmxCredentialsDatabaseOptions *JmxCredentialsDatabaseOptions `json:"jmxCredentialsDatabaseOptions,omitempty"`
}

...omitted-output...

// JmxCredentialsDatabaseOptions provides configuration options to the Cryostat application's JMX credentials database.
type JmxCredentialsDatabaseOptions struct {
	// Name of the secret containing the password to encrypt JMX credentials database.
	// +optional
	// +operator-sdk:csv:customresourcedefinitions:type=spec,xDescriptors={"urn:alm:descriptor:io.kubernetes:Secret"}
	DatabaseSecretName *string `json:"databaseSecretName,omitempty"`
}

Expected Behavior

Credential Database is not only used for JMX credentials but also for Http-based Agent's.

cryostatio/cryostat-web#900

@tthvo tthvo added the bug Something isn't working label May 1, 2023
@tthvo
Copy link
Member Author

tthvo commented May 1, 2023

For Upgrading from previous version to 2.3, I am just concerned about:

const jmxSecretNameSuffix = "-jmx-auth"

Updating this (also its keys too) will cause creation of new secret, right? Can we somehow migrate old secrets content?

Also, we will need a different field name for the Cryostat CRs. This needs some sort of "deprecated alias" for the old JMX- and keep both the old and new fields?

@ebaron
Copy link
Member

ebaron commented May 8, 2024

I think this is fixed with #808

@ebaron ebaron closed this as completed May 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
No open projects
Status: Todo
Development

No branches or pull requests

2 participants