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

Update path to flux.yaml in kustomization.yaml #235

Merged
merged 1 commit into from
Aug 12, 2021

Conversation

svadali2
Copy link
Contributor

@svadali2 svadali2 commented Aug 3, 2021

A reference to the caf-terraform-starter project had to be updated. The file in question had moved due to a change in the folder structure.

234

PR Checklist


  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • My code follows the code style of this project.
  • I ran lint checks locally prior to submission.
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Description

The path to flux.yaml is outdated now that the caf-terraform-starter project has had some new changes to folder structure. Thus, this change will refer to the file in its new location.

Does this introduce a breaking change

  • YES
  • NO

Testing

  1. Open up Kustomization.yaml on your IDE.
  2. Open the link in line 5 and ensure it points to a valid URL.

the old path does not work now that the caf-terraform-starter project has had some new changes to folder structure and content merged to the starter branch
@LaurentLesle LaurentLesle changed the base branch from master to patch.5.4.2 August 11, 2021 11:14
@arnaudlh arnaudlh added this to the 2108 milestone Aug 12, 2021
@arnaudlh arnaudlh linked an issue Aug 12, 2021 that may be closed by this pull request
@arnaudlh
Copy link
Member

merging to integration. thanks for your contribution @svadali2

@arnaudlh arnaudlh merged commit 5c5fcc7 into Azure:patch.5.4.2 Aug 12, 2021
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.

Kustomization path is outdated
3 participants