Skip to content

Conversation

@stenwt
Copy link
Contributor

@stenwt stenwt commented Jun 13, 2023

No description provided.

@stenwt stenwt requested a review from brathina-spectro June 22, 2023 21:42
# Declare variables to be passed into your templates.

charts:
applier:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Chart name should be stormforge-applier

"charts/stormforge-applier-2.0.0-beta.9.tgz"
],
"addonType": "monitoring",
"annotations": {}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These packs should have additional annotations to differentiate community vs spectrocloud managed.

"charts/stormforge-agent-2.0.3.tgz"
],
"addonType": "monitoring",
"annotations": {}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These packs should have additional annotations to differentiate community vs spectrocloud managed.

@@ -0,0 +1,7 @@
charts.stormforge-agent.authorization.clientID:
schema: '{{ required }}'
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A hint will be useful in all these schema for novice users. Check doc on hints usage. Will give a tooltip in the UI for users to understand what these fields are.

@brathina-spectro
Copy link
Collaborator

@stenwt Apologies for the delay in getting this reviewed. Since there were no updates during the initial days after my last comment, i assumed we're not actively working on this pack.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants