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

changed to yaml syntax, fixed debian bug #251

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

tiggi
Copy link
Contributor

@tiggi tiggi commented Jul 10, 2020

Hello!
Back to loan some old ansible roles.

Switched to fully yaml syntax, instead of the old foo=bar baz=bar
The Debian part now actually does what the intent was, using {{ net_path }} instead of hard coding the path.
Removed the commented out old code.

Greetings from Ericsson

@tiggi
Copy link
Contributor Author

tiggi commented Jul 10, 2020

Has travis taken a break?

@tiggi
Copy link
Contributor Author

tiggi commented Jul 14, 2020

The role has bugs preventing it's use by centos8, working on fixing them.

@VilleS1
Copy link
Contributor

VilleS1 commented Oct 1, 2020

Greetings, this PR is against master branch. Please do PRs against devel branch.

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