Skip to content

Commit dcf66ab

Browse files
committed
linting
1 parent 6320bd5 commit dcf66ab

File tree

6 files changed

+6
-18
lines changed

6 files changed

+6
-18
lines changed

migrations/v1.0/standardize-labels.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
#!/bin/bash
22

3-
# We switched to the standard labels recommend in Helm's "Best Practices" doc.
3+
# We switched to the standard labels recommend in Helm's "Best Practices" doc.
44
# https://helm.sh/docs/chart_best_practices/labels/#standard-labels
55
#
66
# This script adds those labels to all the resources in an existing release,

tests/unit/ingress_test.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ tests:
88
- it: Ingress not present if disabled
99
set:
1010
ingress:
11-
enabled: false
11+
enabled: false
1212
asserts:
1313
- hasDocuments:
1414
count: 0

tests/unit/overrides_test.yaml

Lines changed: 1 addition & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -43,28 +43,23 @@ tests:
4343
asserts:
4444
- hasDocuments:
4545
count: 5
46-
4746
- contains: &overrides_volume
4847
path: spec.template.spec.volumes
4948
content:
5049
name: st2-overrides-vol
5150
configMap:
5251
name: st2ha-st2-overrides-configs
5352
documentIndex: 3 # register_content
54-
55-
5653
- contains: &overrides_mnt
5754
path: spec.template.spec.containers[0].volumeMounts
5855
content:
5956
name: st2-overrides-vol
6057
mountPath: /opt/stackstorm/overrides
6158
documentIndex: 3 # register_content
62-
6359
- contains: *overrides_mnt
6460
documentIndex: 4 #Extra_jobs
6561
- contains: *overrides_volume
6662
documentIndex: 4 #extra_jobs
67-
6863
- notContains: *overrides_volume
6964
documentIndex: 0
7065
- notContains: *overrides_mnt
@@ -78,7 +73,7 @@ tests:
7873
- notContains: *overrides_mnt
7974
documentIndex: 2
8075

81-
76+
8277
- it: Deployments with overrides
8378
template: deployments.yaml
8479
set:
@@ -102,14 +97,10 @@ tests:
10297
asserts:
10398
- hasDocuments:
10499
count: 14
105-
106-
107100
- contains: *overrides_volume # always included
108101
documentIndex: 12 # st2client
109102
- contains: *overrides_mnt # always included
110103
documentIndex: 12 # st2client
111-
112-
113104
- notContains: *overrides_volume
114105
documentIndex: 1
115106
- notContains: *overrides_mnt

tests/unit/post_start_script_test.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -267,4 +267,3 @@ tests:
267267
- equal: *assert_lifecycle
268268
- contains: *assert_volume_mount
269269
- contains: *assert_volume
270-

tests/unit/service_account_test.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -198,4 +198,3 @@ tests:
198198
documentIndex: 11
199199
- equal: *assert_sa_custom
200200
documentIndex: 13
201-

tests/unit/services_test.yaml

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,13 +8,13 @@ tests:
88
- it: st2web, st2auth, st2api, st2stream should work without externalName
99
set:
1010
st2chatops:
11-
enabled: false
11+
enabled: false
1212
asserts:
1313
- hasDocuments:
1414
count: 4
1515
- isNull:
1616
path: spec.externalName
17-
17+
1818
- it: st2web, st2auth, st2api, st2stream should work with externalName if type is ExternalName
1919
set:
2020
st2web:
@@ -34,7 +34,7 @@ tests:
3434
hostname: some-host-name
3535
type: ExternalName
3636
st2chatops:
37-
enabled: false
37+
enabled: false
3838
asserts:
3939
- hasDocuments:
4040
count: 4
@@ -44,4 +44,3 @@ tests:
4444
- equal:
4545
path: spec.externalName
4646
value: some-host-name
47-

0 commit comments

Comments
 (0)