Change createAlertCmdRun parsing to include namespace #1278
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Previously I couldn't create the following yaml in the docs using the flux cli
flux2/docs/guides/notifications.md
Lines 209 to 221 in 65d5cad
Running the create alert command with the appropriate kind/name.namespace resulted in name: .
$ flux create alert podinfo \ --provider-ref flux-system \ --event-severity info \ --event-source Kustomization/podinfo.flux-system \ --namespace flux-system \ --export --- apiVersion: notification.toolkit.fluxcd.io/v1beta1 kind: Alert metadata: name: podinfo namespace: flux-system spec: eventSeverity: info eventSources: - kind: Kustomization name: podinfo.flux-system providerRef: name: flux-system
This fix changes it so the output of the command, matches what
s in the docs, and the alert api spec.
Signed-off-by: Alison Dowdney alison@alisondowdney.com