Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

Use wing + puppet to bootstrap vault/consul instances #494

Merged
merged 15 commits into from
Jan 3, 2019
Merged

Use wing + puppet to bootstrap vault/consul instances #494

merged 15 commits into from
Jan 3, 2019

Conversation

JoshVanL
Copy link
Contributor

@JoshVanL JoshVanL commented Sep 6, 2018 •
edited

What this PR does / why we need it:
Uses wing and puppet to bootstrap vault/consul instances. Tarmak will now wait for convergence through wing for vault instances.

fixes #93

Vault instances are now bootstrapped and converged through wing + puppet

@jetstack-bot jetstack-bot added release-note Denotes a PR that will be considered when it comes time to generate release notes. dco-signoff: yes Indicates that all commits in the pull request have the valid DCO sign-off message. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Sep 6, 2018
@jetstack-bot jetstack-bot added dco-signoff: no Indicates that at least one commit in this pull request is missing the DCO sign-off message. and removed dco-signoff: yes Indicates that all commits in the pull request have the valid DCO sign-off message. labels Sep 6, 2018
@JoshVanL JoshVanL changed the title Use wing + puppet to bootstrap vault/consul instances WIP: Use wing + puppet to bootstrap vault/consul instances Sep 6, 2018
@jetstack-bot jetstack-bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 6, 2018
@JoshVanL
Copy link
Contributor Author

JoshVanL commented Sep 6, 2018

/assign

@jetstack-bot jetstack-bot added dco-signoff: yes Indicates that all commits in the pull request have the valid DCO sign-off message. and removed dco-signoff: no Indicates that at least one commit in this pull request is missing the DCO sign-off message. labels Sep 6, 2018
@jetstack-bot jetstack-bot added dco-signoff: no Indicates that at least one commit in this pull request is missing the DCO sign-off message. and removed dco-signoff: yes Indicates that all commits in the pull request have the valid DCO sign-off message. labels Sep 7, 2018
@jetstack-bot jetstack-bot added dco-signoff: yes Indicates that all commits in the pull request have the valid DCO sign-off message. and removed dco-signoff: no Indicates that at least one commit in this pull request is missing the DCO sign-off message. labels Sep 7, 2018
@JoshVanL JoshVanL changed the title WIP: Use wing + puppet to bootstrap vault/consul instances Use wing + puppet to bootstrap vault/consul instances Sep 7, 2018
@jetstack-bot jetstack-bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 7, 2018
@JoshVanL
Copy link
Contributor Author

JoshVanL commented Sep 7, 2018

/hold

@jetstack-bot jetstack-bot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. dco-signoff: no Indicates that at least one commit in this pull request is missing the DCO sign-off message. and removed dco-signoff: yes Indicates that all commits in the pull request have the valid DCO sign-off message. labels Sep 7, 2018
@jetstack-bot jetstack-bot added dco-signoff: yes Indicates that all commits in the pull request have the valid DCO sign-off message. and removed dco-signoff: no Indicates that at least one commit in this pull request is missing the DCO sign-off message. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Sep 7, 2018
@JoshVanL
Fix consul acceptance test
f2af59e 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Adds review changes to reduce duplication and keep legacy naming
4abbd8a 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Fix terraform fmt
66a133e 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL JoshVanL force-pushed the 93-use-wing-puppet branch 5 times, most recently from 7ff192a to baf2fc3 Compare December 3, 2018 19:26
@JoshVanL
Use legacy file to ensure compatibility upgrading
ff6437c 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Require docker class for kubelet
09b344d 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Adds site_module to tarmak and Kubernetes fixtures.yml
e8a059a 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Copy link
Contributor Author

JoshVanL commented Dec 4, 2018

/test puppet-vault_server-acceptance-centos

@JoshVanL
Copy link
Contributor Author

JoshVanL commented Dec 4, 2018

/test puppet-consul-acceptance-single-node

@JoshVanL
Copy link
Contributor Author

JoshVanL commented Dec 4, 2018

/test puppet-kubernetes-acceptance

@JoshVanL
Moves docker class to higher hieradata
1254ece 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Copy link
Contributor Author

JoshVanL commented Dec 4, 2018

/test puppet-vault_server-acceptance-centos

@JoshVanL
Copy link
Contributor Author

JoshVanL commented Dec 4, 2018

/test verify quick

@JoshVanL
Copy link
Contributor Author

JoshVanL commented Dec 4, 2018

/test puppet-vault_server-acceptance-centos

1 similar comment
@JoshVanL
Copy link
Contributor Author

JoshVanL commented Dec 4, 2018

/test puppet-vault_server-acceptance-centos

@JoshVanL
Copy link
Contributor Author

JoshVanL commented Dec 4, 2018

/unassign
/assign @simonswine

@jetstack-bot jetstack-bot assigned simonswine and unassigned JoshVanL Dec 4, 2018
@JoshVanL JoshVanL mentioned this pull request Dec 19, 2018
Merged
@simonswine
Copy link
Contributor

Perfect that works fine for me

/approve
/lgtm

@jetstack-bot jetstack-bot added the lgtm Indicates that a PR is ready to be merged. label Jan 3, 2019
@jetstack-bot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: simonswine

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@jetstack-bot jetstack-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 3, 2019
@jetstack-bot jetstack-bot merged commit c4be874 into jetstack:master Jan 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@simonswine simonswine simonswine approved these changes

@kragniz kragniz Awaiting requested review from kragniz

Assignees

@simonswine simonswine

Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. dco-signoff: yes Indicates that all commits in the pull request have the valid DCO sign-off message. lgtm Indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.
Projects
No open projects
release-0.6
  
Needs review
Milestone
v0.6
Development

Successfully merging this pull request may close these issues.

Use puppet for the Consul/Vault instances
4 participants
@JoshVanL @jetstack-bot @simonswine @seanturner83