Skip to content

Improve 🧂 SonarQube default admin password#159

Merged
ckavili merged 3 commits into
rhoai-mlops:mainfrom
kareiva:sonarqube
Dec 1, 2025
Merged

Improve 🧂 SonarQube default admin password#159
ckavili merged 3 commits into
rhoai-mlops:mainfrom
kareiva:sonarqube

Conversation

@kareiva

@kareiva kareiva commented Dec 1, 2025

Copy link
Copy Markdown
Contributor

Let the SonarQube out-of-date warning go away.

This should be safe to merge, tested locally, and docsify renders it nicely.

Then we can bump the sonarqube version here or somewhere in between: redhat-cop/helm-charts#659

Tested ✅ on cluster-dqn7k.dqn7k.sandbox3357

The password needs to fulfill the following requirements: at least 12 characters and contain at least one uppercase character, one lowercase character, one digit and one special character.

So I have picked _ as the safest-possible choice (+ and # do not work as they are not URL-safe)

Hope it makes sense. 🤞

@kareiva kareiva changed the title Improve 🧂 SonarQube default admin password WIP: Improve 🧂 SonarQube default admin password Dec 1, 2025
@kareiva kareiva closed this Dec 1, 2025
@kareiva kareiva changed the title WIP: Improve 🧂 SonarQube default admin password Improve 🧂 SonarQube default admin password Dec 1, 2025
This makes the sonarqube API usage more future-proof.
@kareiva kareiva reopened this Dec 1, 2025
@ckavili

ckavili commented Dec 1, 2025

Copy link
Copy Markdown
Member

I just left a comment at the helm chart repo, @kareiva. And is there any screenshot changes needed in the chapter https://rhoai-mlops.github.io/lab-instructions/#/8-the-supporting-acts/4-code-analysis? thanks!

@kareiva

kareiva commented Dec 1, 2025

Copy link
Copy Markdown
Contributor Author

@ckavili
This can be merged independently from the helm chart repo. And until then, it actually requires no screenshot updates. I have tested it with the current version and it's backwards-compatible.

@ckavili ckavili merged commit c18314b into rhoai-mlops:main Dec 1, 2025
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.

2 participants