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

Root db access #68

Merged
merged 9 commits into from
Nov 10, 2016
Merged

Root db access #68

merged 9 commits into from
Nov 10, 2016

Conversation

martbhell
Copy link
Contributor

Configure a mysql root user.
It is possible to disable it by setting a variable: slurm_manage_mysql_security to False
I would like another eye on this before merging and testing in our test cluster.
Based on travis testing it seems to work when installing from scratch, maybe there are some caveats for when doing it on a running system?

@A1ve5
Copy link
Contributor

A1ve5 commented Nov 2, 2016

Seems legit to me. Must be tested in test system.

@martbhell
Copy link
Contributor Author

Force pushed because of a rebase with master.

Before the rebase:

  • travis shows that installing with this role from scratch still works (and sets mysql root user password)
  • on an openstack CentOS7 VM and first using master branch of this role slurm still works, if I then change to this branch then it also still works - and mysql root user has a password.

After rebase:

  • travis install also works
  • in an openstack VM first configuring from master also works
  • in an openstack VM configuringt then the same VM from this branch also works.

Merging this and testing on io.

@martbhell martbhell merged commit 4ab0539 into master Nov 10, 2016
@martbhell martbhell removed the review label Nov 10, 2016
martbhell added a commit to fgci-org/fgci-ansible that referenced this pull request Nov 10, 2016
@martbhell martbhell deleted the rootDBaccess branch March 19, 2018 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants