-
Notifications
You must be signed in to change notification settings - Fork 545
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
Ansible playbook #356
Ansible playbook #356
Conversation
ecordell
commented
May 22, 2018
•
edited
Loading
edited
- Ansible playbook generated from existing deploy artifacts
- Vendored (submodule) openshift-ansible libs so that the playbook can be run on its own (for testing). If this doesn't seem valuable we can remove the libs.
- There is a corresponding PR in openshift-ansible, we can sync releases from this repo to that one
158b2ea
to
2b3147f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
make ver=0.5.0 release
fails.
Makefile
Outdated
# copy manifest files into release | ||
./scripts/package-release.sh $(ver) deploy/aos-olm/$(ver)/files deploy/aos-olm/values.yaml | ||
# generate install/remove tasks based on manifest files | ||
./scripts/k8s_yaml_to_ansible_install.sh deploy/aos-olm/$(ver)/files deploy/aos-olm/$(ver)/tasks/install.yaml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing /role/
in the path, I think
# generate install/remove tasks based on manifest files
./scripts/k8s_yaml_to_ansible_install.sh deploy/aos-olm/0.5.0/files deploy/aos-olm/0.5.0/tasks/install.yaml
./scripts/k8s_yaml_to_ansible_install.sh: line 18: deploy/aos-olm/0.5.0/tasks/install.yaml: No such file or directory
make: *** [Makefile:178: ansible-release] Error 1
lib_openshift and lib_utils
for eventual inclusion into openshift. didn't add to our CI now since we know we won't need this just yet
19c5418
to
142a612
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Success!
Ansible playbook
Ansible playbook