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

Add dev plan #1

Merged
merged 5 commits into from Sep 28, 2017
Merged

Conversation

djzager
Copy link
Contributor

@djzager djzager commented Sep 27, 2017

No description provided.

@djzager
Copy link
Contributor Author

djzager commented Sep 27, 2017

I wouldn't merge this just yet, still testing. Will report back when I've finished.

test: false
triggers:
- type: ConfigChange
triggers:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Unindented this because I am pretty sure it doesn't belong under volumes.

Dockerfile Outdated
@@ -3,32 +3,36 @@ MAINTAINER Ansible Playbook Bundle Community

LABEL "com.redhat.apb.version"="0.1.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

Make this "1.0"

k8s_v1_service:
name: "{{ service_name }}"
namespace: '{{ namespace }}'
state: present
Copy link
Contributor

Choose a reason for hiding this comment

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

This will never get deprovisioned. Need "{{ state }}"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

😎

@djzager
Copy link
Contributor Author

djzager commented Sep 27, 2017

So I tested this by pushing to my org and provisioning both plans. I'm guessing the next thing I should do is bind this to something.

@djzager
Copy link
Contributor Author

djzager commented Sep 27, 2017

Started an origin 3.6 cluster with catasb and:

  • Provision mariadb dev plan
  • Provision mediawiki
  • Bound mediawiki -> mariadb
  • Unbound mediawiki from mariadb
  • Deprovision mariadb
  • Provision mariadb prod plan
  • Provision mediawiki
  • Bound mediawiki -> mariadb
  • Unbound mediawiki from mariadb
  • Deprovision mariadb

Copy link
Contributor

@dymurray dymurray left a comment

Choose a reason for hiding this comment

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

ACK, looks good.

@dymurray dymurray merged commit e94dcd6 into ansibleplaybookbundle:master Sep 28, 2017
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

2 participants