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

WIP: Consul and Etcd save+restore #696

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

WIP: Consul and Etcd save+restore #696

wants to merge 36 commits into from

Conversation

JoshVanL
Copy link
Contributor 

Adds `tarmak cluster snapshot` to save or restore consul/etcd snapshots

rebased on top of #477
/assign

@JoshVanL
Adds key verification to lambda function
055af3c 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Finishes up verification tests
f41dd76 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Adds test for create tags
4c2969d 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Complete lambda logic
6b29b32 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Updates naming and Makefile
0a9c0f7 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Adds tagging_control terraform module
bbfd1e6 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Adds start of lambda request client to wing
5397261 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Adds tagging request to wing and allows bastion wing dev mode
b628635 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Set up permissions and networking for instances to lambda function
3242678 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Strictly enforce and maintain known hosts file
0599093 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Adds ignore changes to tags for instances
1080f41 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Further restricts instance lambda invoking privileges
49de727 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Updates vendor
150d770 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Share tagging control policy across roles
c2c649a 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Adds wing tag command, used by Jenkins
62af15f 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Updates docs and formats terraform code
7f4fe24 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Removes unused Jenkins user data file
2d0cc22 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Adds environment to lambda name with fixes
6991e5c 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Adds in tree zip pkg to zip tagging_control during generate
88a99e2 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Uses cobra for tagging_control function
3efa646 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Remove lambda function ingress SG
24f34a0 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Share wing binary read policy across roles in terraform
d79852c 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Adds a fallback for instances that have no public key tags
f74b18d 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Use iam_policy_role_attachment for shared wing binary read policy
42ad0ff 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Use option ask for strict host checking when in interactive mode
06f23fe 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Disable cmd auto gen tags for tagging_control
97c52c8 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Move important instance tags to awstag_ec2_tag
086fb9c 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Remove quotes from WING_ENVIRONMENT
423a13e 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Removes duplicate variable declarations in terraform
db18013 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Updates terraform aws provider to prevent early timeouts on lambda
ac70dff 
function uploads

Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Updates depreciated terraform vpc_id
962cd33 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Remove trailing period from private_zone name
66152d1 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Use in package ssh for all programmatic use cases
b1d7eb7 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Adds host key call back function
791d421 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Updates docs, Gopkg.lock and fixes tunnel port string
bbeaa49 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Adds consul and etcd save and consul restore
884d7da 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@jetstack-bot jetstack-bot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. 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. labels Jan 21, 2019
@jetstack-bot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: joshvanl

If they are not already assigned, you can assign the PR to them by writing /assign @joshvanl in a comment when ready.

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 size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. area/puppet Indicates a PR is affecting Puppet manifests area/terraform Indicates a PR is affecting Terraform configuration kind/documentation Categorizes issue or PR as related to documentation. labels Jan 21, 2019
@simonswine simonswine added this to the v0.7 milestone Jan 23, 2019
@jetstack-bot
Copy link
Collaborator

@JoshVanL: PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@jetstack-bot jetstack-bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jan 25, 2019
@jetstack-bot
Copy link
Collaborator

@JoshVanL: The following tests failed, say /retest to rerun them all:

Test name Commit Details Rerun command
tarmak-quick-verify 884d7da link /test verify quick
tarmak-puppet-module-etcd-acceptance-three-node 884d7da link /test puppet-etcd-acceptance-three-node
tarmak-puppet-module-tarmak-acceptance-1-14-centos 884d7da link /test puppet-tarmak-acceptance-centos v1.14
tarmak-puppet-module-tarmak-acceptance-1-15-centos 884d7da link /test puppet-tarmak-acceptance-centos v1.15
tarmak-puppet-module-tarmak-acceptance-1-13-ubuntu 884d7da link /test puppet-tarmak-acceptance-ubuntu v1.13

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@dippynark dippynark Awaiting requested review from dippynark

@simonswine simonswine Awaiting requested review from simonswine

Assignees

@JoshVanL JoshVanL

Labels
area/puppet Indicates a PR is affecting Puppet manifests area/terraform Indicates a PR is affecting Terraform configuration dco-signoff: yes Indicates that all commits in the pull request have the valid DCO sign-off message. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. kind/documentation Categorizes issue or PR as related to documentation. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. release-note Denotes a PR that will be considered when it comes time to generate release notes. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Milestone
v0.7
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@JoshVanL @jetstack-bot @simonswine