Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: agent install instructions #2292

Merged
merged 2 commits into from
Nov 11, 2024
Merged

fix: agent install instructions #2292

merged 2 commits into from
Nov 11, 2024

Conversation

moshloop
Copy link
Member

No description provided.

Copy link

vercel bot commented Sep 17, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
aws-preview ✅ Ready (Inspect) Visit Preview Nov 1, 2024 0:34am
flanksource-ui ✅ Ready (Inspect) Visit Preview Nov 1, 2024 0:34am

Copy link

netlify bot commented Sep 17, 2024

Deploy Preview for goofy-euclid-75956c ready!

Name Link
🔨 Latest commit 57591e4
🔍 Latest deploy log https://app.netlify.com/sites/goofy-euclid-75956c/deploys/6724c8598d12120008bea6d0
😎 Deploy Preview https://deploy-preview-2292--goofy-euclid-75956c.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Sep 17, 2024

Deploy Preview for clerk-saas-ui ready!

Name Link
🔨 Latest commit 57591e4
🔍 Latest deploy log https://app.netlify.com/sites/clerk-saas-ui/deploys/6724c859c603e70008cb4a23
😎 Deploy Preview https://deploy-preview-2292--clerk-saas-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Sep 17, 2024

Deploy Preview for flanksource-demo-stable ready!

Name Link
🔨 Latest commit 57591e4
🔍 Latest deploy log https://app.netlify.com/sites/flanksource-demo-stable/deploys/6724c85912249600086a5f86
😎 Deploy Preview https://deploy-preview-2292--flanksource-demo-stable.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@moshloop
Copy link
Member Author

@mainawycliffe - the schedule is not pulling through, can we just remove it ?

@moshloop
Copy link
Member Author

@mainawycliffe there are sill differences in the values -

Can we create a Helm Component like https://github.com/flanksource/docs/blob/main/common/src/components/Helm.jsx - the values are dynamic and single-sourced

@mainawycliffe
Copy link
Contributor

@moshloop The current component is dynamic; I had something similar originally, but I found the template syntax to lead to more issues compared to Handlesbar for both the Yaml and CLI (you would end up need handle all nullable options, which meant a small change could make the template incorrect), they take in the same data object, but return a string based on Yaml handlebar template or the command to run for the CLI, I think the handlebar template for the Yaml and CLI command passing in data object for replacement is far much easier compared to using the template syntax. Other than that, both components work similarly.

Update FluxInstallAgent.tsx

Update CLIInstallAgent.tsx

Update CLIInstallAgent.unit.test.tsx

Update FluxInstallAgent.unit.test.tsx

Update CLIInstallAgent.tsx

Update CLIInstallAgent.unit.test.tsx

refactor: lift up the data composition to parent component

chore: fix failing tests

chore: fix issue after running npm i

fix: make the values being passed to helm & flux template consistent and external

fix: fix  issues
…nstructions

Update src/ui/HelmSnippet/CLISnippet.tsx

Co-authored-by: Moshe Immerman <[email protected]>

chore: update manifests

chore: fix failing tests
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.

2 participants