-
Notifications
You must be signed in to change notification settings - Fork 282
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
[META] Update all instances of admin:admin across different repositories #3624
Comments
[Triage] This is a meta tracking issue for replacing default credential usage in the cluster. |
Relevant line in opensearch-build repo: https://github.com/opensearch-project/opensearch-build/blob/main/scripts/default/integtest.sh#L92 |
CI references to admin:admin :https://github.com/search?q=org%3Aopensearch-project+admin%3Aadmin&type=code&p=1 |
What version of OpenSearch will have the new password? Data Prepper actually has quite a few occurrences of the password |
@dlvenable it will be for version 2.12 and above |
Most references in this repo: seem to disable install demo config: https://github.com/opensearch-project/opensearch-benchmark/blob/main/osbenchmark/resources/docker-compose.yml.j2#L18, I don't think we need to do anything for this repo, but maybe we should make an issue and tag maintainers. |
https://github.com/opensearch-project/opensearch-php/actions/runs/7250326156/job/19750248987 - opensearch-php is running without security it seems like, so no changes necessary |
opensearch-project/opensearch-build#4302 - this means that all repos which still need to maintain their own integtest.sh files should be switching to admin:myStrongPassword123! as new default creds |
[Update: 01/24/'24]
|
[Update: 01/25/'24]
|
[Update: 01/26/'24]
|
[Update: 01/29/'24]
|
[Update: 01/30/'24]
|
[Update: 01/31/'24]
|
[Update: 02/01/'24]
|
[Update: 02/02/'24]
|
[Update: 02/05/'24]
|
[Update: 02/06/'24]
|
[Update: 02/07/'24]
|
[Update: 02/08/'24]
|
Next update will be post 2.12 release. This task is no longer a blocker for 2.12 |
[Update: 02/22/'24]
This is the last update on this issue. The maintainers of PRs with ⏳ status have been nudged to bring this across the finish line. Closing this as the feature has been delivered in 2.12. |
Description
There are many instances of
admin:admin
credentials usage across different repositories. They will be affected by the change to replace hard-coded admin password with a dynamic value. This issue requests fixing all those instances of admin credentials to address this breaking change.Acceptance Criteria
Tasks
Search and update the references to
admin:admin
, passwordadmin
and demo install script in following repos, as required:(to track work done, tag the PR next to the repo name)
(Legend: ❌ - no updates, 🕐 - in progress, ⏳ - Waiting 2.12 release, ✅ - complete)
The text was updated successfully, but these errors were encountered: