Skip to content

Commit 930d616

Browse files
committed
fix(operator): missed changes related to changing min value for priority
1 parent 1fc5528 commit 930d616

File tree

3 files changed

+2
-3
lines changed

3 files changed

+2
-3
lines changed

chart/templates/skyhook-crd.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -476,7 +476,7 @@ spec:
476476
priority:
477477
description: Priority determines the order in which skyhooks are applied. Lower values are applied first.
478478
type: integer
479-
minimum: 0
479+
minimum: 1
480480
default: 200
481481
serial:
482482
default: false

docs/ordering_of_skyhooks.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# Ordering of Skyhooks
22
## What
3-
With v0.8.0 Skyhooks now always get applied in a repeatable and specific order. This also means that all Skyhooks will now be sequential, though packages within a Skyhook can be parallel. Each custom resource now supports a `priority` field which is a non-zero positive integer. Skyhooks will be processed in order starting from 0, any Skyhooks with the same `priority` will be processed by sorting them by their `metadata.name` field.
3+
With v0.8.0 Skyhooks now always get applied in a repeatable and specific order. This also means that all Skyhooks will now be sequential, though packages within a Skyhook can be parallel. Each custom resource now supports a `priority` field which is a non-zero positive integer. Skyhooks will be processed in order starting from 1, any Skyhooks with the same `priority` will be processed by sorting them by their `metadata.name` field.
44

55
**NOTE**: Any Skyhook which does NOT provide a `priority` field will be assigned a priority value of 200.
66

k8s-tests/chainsaw/skyhook/simple-skyhook/chainsaw-test.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@ kind: Test
2424
metadata:
2525
name: simple-skyhook
2626
spec:
27-
# skip: false ## this test doesn't seem to be useful, just slows things down, leaving it for, should delete at some point if still skipped
2827
timeouts:
2928
assert: 240s
3029
steps:

0 commit comments

Comments
 (0)