Skip to content

Commit f684fc7

Browse files
authored
Merge pull request #887 from oracle/voyager-fix
OWLS-71184 REDUNDANT WORD VOYAGER,APACHE OR IN LBER SERVICE NAME
2 parents b850c8e + 2dbb736 commit f684fc7

File tree

2 files changed

+32
-8
lines changed

2 files changed

+32
-8
lines changed

kubernetes/samples/charts/util/setup.sh

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,8 @@
55
# This script is to create or delete Ingress controllers. We support two ingress controllers: traefik and voyager.
66

77
MYDIR="$(dirname "$(readlink -f "$0")")"
8+
VNAME=voyager-operator # release name of Voyager
9+
TNAME=traefik-operator # release name of Traefik
810

911
function createVoyager() {
1012
echo "Creating Voyager operator on namespace 'voyager'."
@@ -19,10 +21,10 @@ function createVoyager() {
1921
fi
2022
echo
2123

22-
if [ "$(helm list | grep voyager-operator | wc -l)" = 0 ]; then
24+
if [ "$(helm list | grep $VNAME | wc -l)" = 0 ]; then
2325
echo "Ihstall voyager operator."
2426

25-
helm install appscode/voyager --name voyager-operator --version 7.4.0 \
27+
helm install appscode/voyager --name $VNAME --version 7.4.0 \
2628
--namespace voyager \
2729
--set cloudProvider=baremetal \
2830
--set apiserver.enableValidatingWebhook=false
@@ -52,9 +54,9 @@ function createTraefik() {
5254
echo "Creating Traefik operator on namespace 'traefik'."
5355
echo
5456

55-
if [ "$(helm list | grep traefik-operator | wc -l)" = 0 ]; then
57+
if [ "$(helm list | grep $TNAME | wc -l)" = 0 ]; then
5658
echo "Install Traefik Operator."
57-
helm install --name traefik-operator --namespace traefik --values ${MYDIR}/../traefik/values.yaml stable/traefik
59+
helm install --name $TNAME --namespace traefik --values ${MYDIR}/../traefik/values.yaml stable/traefik
5860
else
5961
echo "Traefik Operator is already installed."
6062
fi
@@ -108,9 +110,9 @@ function purgeCRDs() {
108110
}
109111

110112
function deleteVoyager() {
111-
if [ "$(helm list | grep voyager-operator | wc -l)" = 1 ]; then
113+
if [ "$(helm list | grep $VNAME | wc -l)" = 1 ]; then
112114
echo "Delete Voyager Operator. "
113-
helm delete --purge voyager-operator
115+
helm delete --purge $VNAME
114116
kubectl delete ns voyager
115117
purgeCRDs
116118
else
@@ -127,9 +129,9 @@ function deleteVoyager() {
127129
}
128130

129131
function deleteTraefik() {
130-
if [ "$(helm list | grep traefik-operator | wc -l)" = 1 ]; then
132+
if [ "$(helm list | grep $TNAME | wc -l)" = 1 ]; then
131133
echo "Delete Traefik operator."
132-
helm delete --purge traefik-operator
134+
helm delete --purge $TNAME
133135
kubectl delete ns traefik
134136
else
135137
echo "Traefik operator has already been deleted."

kubernetes/samples/charts/voyager/README.md

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,12 @@ As a demonstration, the following are the detailed steps to install the Voyager
99
```
1010
$ helm repo add appscode https://charts.appscode.com/stable/
1111
$ helm repo update
12+
```
13+
Verify that the chart repository has been added.
14+
```
1215
$ helm search appscode/voyager
16+
NAME CHART VERSION APP VERSION DESCRIPTION
17+
appscode/voyager 8.0.1 8.0.1 Voyager by AppsCode - Secure HAProxy Ingress Co...
1318
```
1419

1520
### 2. Install the Voyager operator
@@ -21,6 +26,23 @@ $ helm install appscode/voyager --name voyager-operator --version 7.4.0 \
2126
--set cloudProvider=baremetal \
2227
--set apiserver.enableValidatingWebhook=false
2328
```
29+
Wait until the Voyager Operator is running.
30+
```
31+
$ kubectl -n voyager get all
32+
NAME READY STATUS RESTARTS AGE
33+
pod/voyager-voyager-operator-77cbfdcb86-gqwgt 1/1 Running 0 46m
34+
35+
NAME TYPE CLUSTER-IP EXTERNAL-IP PORT(S) AGE
36+
service/voyager-voyager-operator ClusterIP 10.105.254.144 <none> 443/TCP,56791/TCP 46m
37+
38+
NAME DESIRED CURRENT UP-TO-DATE AVAILABLE AGE
39+
deployment.apps/voyager-voyager-operator 1 1 1 1 46m
40+
41+
NAME DESIRED CURRENT READY AGE
42+
replicaset.apps/voyager-voyager-operator-77cbfdcb86 1 1 1 46m
43+
```
44+
> **NOTE**: All the generated Kubernetes resources of the Voyager operator have names with the pattern `voyager-<releaseName>XXX`. This logic is controlled by the Voyager Helm chart. In our case, we use `releaseName` `voyager-operator`, so all the generated resources have names like `voyager-voyager-operatorXXX`.
45+
2446
## Optionally, download the Voyager Helm chart
2547
If you want, you can download the Voyager Helm chart and untar it into a local folder:
2648
```

0 commit comments

Comments
 (0)