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

ArcBox - refactor GitOps script for updated repository URL #2758

Closed

Conversation

janegilring
Copy link
Contributor

This pull request includes changes to the azure_jumpstart_arcbox/artifacts/gitops_scripts/K3sGitOps.ps1 file, primarily focusing on updating repository URLs and file paths.

Repository URL update:

  • Changed the repository URL from https://github.com/$Env:githubUser/azure-arc-jumpstart-apps to https://github.com/$Env:githubUser/jumpstart-apps to reflect the new repository location. ([azure_jumpstart_arcbox/artifacts/gitops_scripts/K3sGitOps.ps1L15-R15](https://github.com/microsoft/azure_arc/pull/2758/files#diff-770b4da0148c2b5f9fe459fa68ef7a33b2ed56a43e42557f24d3c235cfa0e59eL15-R15))

File path update:

  • Modified the kustomization path from ./hello-arc/yaml to ./arcbox/hello_arc/yaml in the az k8s-configuration flux create command to match the updated directory structure. ([azure_jumpstart_arcbox/artifacts/gitops_scripts/K3sGitOps.ps1L59-R59](https://github.com/microsoft/azure_arc/pull/2758/files#diff-770b4da0148c2b5f9fe459fa68ef7a33b2ed56a43e42557f24d3c235cfa0e59eL59-R59))

@janegilring janegilring added the ArcBox Jumpstart ArcBox related label Oct 13, 2024
@janegilring janegilring added this to the October 2024 milestone Oct 13, 2024
@janegilring janegilring self-assigned this Oct 13, 2024

Hi janegilring! Thank you for opening this Pull Request. Someone will review it soon. Thank you for committing to making the Arc Jumpstart better.

Copy link
Contributor

@fcabrera23 fcabrera23 left a comment

Choose a reason for hiding this comment

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

LGTM

@janegilring
Copy link
Contributor Author

Superseeded by #2768

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ArcBox Jumpstart ArcBox related
Projects
Development

Successfully merging this pull request may close these issues.

2 participants