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

OpenSUSE support #604

Closed
wants to merge 0 commits into from
Closed

OpenSUSE support #604

wants to merge 0 commits into from

Conversation

CaptainSifff
Copy link

Hi all,
here's opensuse support for this role. Sadly the tests currently fail since
the pull-request is not yet merged:
dev-sec/ansible-role-mysql#7
is missing.

@rndmh3ro
Copy link
Member

Im erged the other PR, however the tests are still failing. You might want to look into it.

@CaptainSifff
Copy link
Author

Thanks! I will look into the failing tests.

@rndmh3ro
Copy link
Member

You can test this locally, too:

pip install molecule[docker] 
export MOLECULE_DISTRO=opensuse_tumbleweed
molecule test -s mysql_hardening

This way you don't have to wait for the ci tests every time.

On Monday I can take a look, too, if you want.

@CaptainSifff
Copy link
Author

Thanks for that offer!
I would very much appreciate, if you could have a look. The failure comes now from the mysql role again:
https://github.com/CaptainSifff/ansible-role-mysql/actions/runs/3553964438/jobs/5969775889

where it says, that the credentials of the mysql root user are not available.
All previous steps in that pipeline, where the credentials could be given are skipped.
I guess that means that there are anonymous users on OpenSUSE and not on the other distros....

@rndmh3ro
Copy link
Member

If the PR in our baseline is merged (or the problem fixed), we can merge this. :)

@CaptainSifff
Copy link
Author

Cool! Seems like the other minor changes over here are not required anymore in the base mysql role?

molecule/mysql_hardening/converge.yml Outdated Show resolved Hide resolved
CHANGELOG.md Outdated
@@ -1,6 +1,6 @@
# Changelog
Copy link
Contributor

Choose a reason for hiding this comment

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

I really dislike seeing the CHANGELOG.md in a PR :D

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.

None yet

3 participants