stsstack-bundles

Merge ~sombrafam/stsstack-bundles:master into stsstack-bundles:master

  1. Git
  2. lp:~sombrafam/stsstack-bundles
  3. master
  4. Merge into master
Proposed by Erlon R. Cruz
Status: Merged
Merged at revision: a38b8963132d6843fd3d0faf983ed9688d40cc8e
Proposed branch: ~sombrafam/stsstack-bundles:master
Merge into: stsstack-bundles:master
Diff against target: 12 lines (+2/-1)
1 file modified
kubernetes/configure (+2/-1)
Reviewer Review Type Date Requested Status
Edward Hope-Morley Approve
Review via email: mp+385768@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Edward Hope-Morley (hopem) wrote :

This feels a bit nicer:

kubemaster_unit=`juju status| egrep 'kubernetes-master/[[:digit:]]+'| cut -f 1 -d ' '`

Revision history for this message
Erlon R. Cruz (sombrafam) wrote :

> This feels a bit nicer:
>
> kubemaster_unit=`juju status| egrep 'kubernetes-master/[[:digit:]]+'| cut -f 1
> -d ' '`

Sure, the backticks right? :)

Revision history for this message
Edward Hope-Morley (hopem) wrote :

lgtm

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/kubernetes/configure b/kubernetes/configure
2index 90a8198..c69f1c6 100755
3--- a/kubernetes/configure
4+++ b/kubernetes/configure
5@@ -1,5 +1,6 @@
6 #!/bin/bash -x
7 sudo snap install kubectl --classic
8 mkdir -p ~/.kube
9-juju scp kubernetes-master/0:config ~/.kube/config
10+kubemaster_unit=$(juju status| egrep 'kubernetes-master/[[:digit:]]+'| cut -f 1 -d ' ')
11+juju scp ${kubemaster_unit}:config ~/.kube/config
12 kubectl --namespace kube-system get pods

Subscribers

People subscribed via source and target branches