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

Correcting the name of the Operator Console TLS Secret #1802

Closed
wants to merge 1 commit into from

Conversation

cniackz
Copy link
Contributor

@cniackz cniackz commented Oct 7, 2023

In Deployment console, correcting the name of the Operator Console TLS Secret as it got changed in https://github.com/minio/operator/pull/1712/files:

Screenshot 2023-10-07 at 12 28 11 AM

and everytime is tested, manual modification is required in the deployment to use proper secret

@cniackz cniackz requested a review from pjuarezd October 7, 2023 04:28
@cniackz cniackz self-assigned this Oct 7, 2023
@cniackz cniackz added the consistency When we are changing something to be consistent label Oct 7, 2023
Copy link
Contributor

@shtripat shtripat left a comment

Choose a reason for hiding this comment

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

lgtm

name: operator-console-tls
name: console-tls
Copy link
Member

Choose a reason for hiding this comment

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

is this a breaking change? how does it work now?

Copy link
Member

@pjuarezd pjuarezd left a comment

Choose a reason for hiding this comment

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

Actually, I introduced the name of the secreted by default as console-tls in kubectl-minio on PR #1348

f.BoolVar(&o.operatorOpts.ConsoleTLS, "console-tls", false, "enable tls for Operator console")

And wrongly set the constant to operator-console-tls on the same PR
https://github.com/minio/operator/pull/1348/files#diff-0ee76ed6676e7fcf9179ac75e1d5eeb7a6e25b7bc489bb8dec243fd80e247574R40.

I fixed that discrepancy and set the expected secret name as console-tls in the PR https://github.com/minio/operator/pull/1712/files

Which test is failing? we should fix the broken test instead

@cniackz
Copy link
Contributor Author

cniackz commented Nov 14, 2023

Ok, if no longer an issue, then we should simply close it.

@cniackz cniackz closed this Nov 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
consistency When we are changing something to be consistent
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants