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

Remove mysql root user setting #14477

Merged
merged 1 commit into from
Mar 27, 2021
Merged

Conversation

deepu105
Copy link
Member

@deepu105 deepu105 commented Mar 27, 2021

Setting MYSQL_USER to root seems to fail with latest mySQL version being used when deployed as container

There are two more places where this is set for openshift but to non-root values so I guess that should be fine

here is the error I got

kubectl logs product-mysql-55fbb8fdd8-6dbwc -n jhipster         
2021-03-27 13:52:40+00:00 [Note] [Entrypoint]: Entrypoint script for MySQL Server 8.0.23-1debian10 started.
2021-03-27 13:52:40+00:00 [Note] [Entrypoint]: Switching to dedicated user 'mysql'
2021-03-27 13:52:40+00:00 [Note] [Entrypoint]: Entrypoint script for MySQL Server 8.0.23-1debian10 started.
2021-03-27 13:52:40+00:00 [ERROR] [Entrypoint]: MYSQL_USER="root", MYSQL_USER and MYSQL_PASSWORD are for configuring a regular user and cannot be used for the root user
    Remove MYSQL_USER="root" and use one of the following to control the root user password:
    - MYSQL_ROOT_PASSWORD
    - MYSQL_ALLOW_EMPTY_PASSWORD
    - MYSQL_RANDOM_ROOT_PASSWORD

@jhipster/developers I also did the same for MariaDB but not usre about that can someone take a second look as well

Please make sure the below checklist is followed for Pull Requests.

When you are still working on the PR, consider converting it to Draft (bellow reviewers) and adding skip-ci label, you can still see CI build result at your branch.

@DanielFran
Copy link
Member

DanielFran commented Mar 27, 2021

Related with this fix: 22c630d

docker-library/mysql@6a9edeb

@DanielFran DanielFran merged commit 554b48a into jhipster:main Mar 27, 2021
@deepu105
Copy link
Member Author

deepu105 commented Mar 27, 2021 via email

@DanielFran
Copy link
Member

@deepu105 did not found same changes for Mariadb

@deepu105
Copy link
Member Author

deepu105 commented Mar 28, 2021 via email

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.

3 participants