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

WIP: tarmak snapshot consul/etcd save/restore #618

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

WIP: tarmak snapshot consul/etcd save/restore #618

wants to merge 15 commits into from

Conversation

JoshVanL
Copy link
Contributor

@JoshVanL JoshVanL commented Nov 7, 2018

fixes #613

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

/assign

@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. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Nov 7, 2018
@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 the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Nov 8, 2018
@JoshVanL JoshVanL force-pushed the 613-vault-consul-save-restore branch from bb9194d to 81bb832 Compare November 9, 2018 17:24
@simonswine simonswine added this to the release-0.6 milestone Nov 14, 2018
@JoshVanL
SSH passthrough uses ssh package
4f1dc62 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Move Execute to use pkg ssh
9b41a20 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Create ssh tunnel with ssh package
9fa953d 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Uses in package ssh for tunnelling
a1b5d01 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Use same local port when daemoning tunnel
410617f 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Set tunnel tries to 5 and exit immediately
e2f597f 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Update vault tunnel test
b98cdb7 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Update Gopkg.lock and cmd docs
3d389be 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL JoshVanL force-pushed the 613-vault-consul-save-restore branch from 81bb832 to d38a948 Compare November 27, 2018 09:46
@jetstack-bot jetstack-bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Nov 27, 2018
@JoshVanL
Adds snapshot save and restore for consul
f981b61 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Adds tarmak cluster snapshot etcd save
2551d5e 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Attempt to tar both received and sent snapshots
f218661 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Gzip files
ec962f2 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Use ssh package to save backups
9f83889 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL JoshVanL force-pushed the 613-vault-consul-save-restore branch from d38a948 to 9f83889 Compare November 27, 2018 16:37
@jetstack-bot jetstack-bot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Nov 27, 2018
@JoshVanL
Restore consul cluster by snapshot
3994a9f 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@JoshVanL
Adds snapshot etcd restore
51f235e 
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
@jetstack-bot jetstack-bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Nov 28, 2018
@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
Copy link
Collaborator

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

Test name Commit Details Rerun command
tarmak-puppet-roles-quick-verify 51f235e link /test puppet-roles-verify
tarmak-puppet-module-tarmak-quick-verify 51f235e link /test puppet-tarmak-verify
tarmak-quick-verify 51f235e link /test verify quick
tarmak-docs-verify 51f235e link /test verify docs
tarmak-puppet-module-etcd-quick-verify 51f235e link /test puppet-etcd-verify
tarmak-puppet-module-etcd-acceptance-single-node 51f235e link /test puppet-etcd-acceptance-single-node
tarmak-puppet-module-tarmak-acceptance-1-11-centos 51f235e link /test puppet-tarmak-acceptance-centos v1.11
tarmak-puppet-module-tarmak-acceptance-1-14-centos 51f235e link /test puppet-tarmak-acceptance-centos v1.14
tarmak-puppet-module-tarmak-acceptance-1-15-centos 51f235e link /test puppet-tarmak-acceptance-centos v1.15
tarmak-puppet-module-tarmak-acceptance-1-13-ubuntu 51f235e 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

@opeyemi33171 opeyemi33171 Awaiting requested review from opeyemi33171

Assignees

@JoshVanL JoshVanL

Labels
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. 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.6
Development

Successfully merging this pull request may close these issues.

tarmak snapshot consul/etcd save/restore
3 participants
@JoshVanL @jetstack-bot @simonswine