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 installer and permissions so that private links work #62

Merged
merged 1 commit into from
Sep 12, 2024

Conversation

asmith030
Copy link
Contributor

No description provided.

Copy link

github-actions bot commented Aug 15, 2024

Pull Request Review Status

  • 🖌 Terraform Format and Style: success
  • 🔍 Terraform Linting: success
  • 👮 Terraform Security Check: success
  • 🔘 Terraform Tests: success
  • 🔧 Terraform Initialisation: success
  • 🤖 Terraform Validation: success
  • 🤖 Terraform Example Validation: success
  • 📖 Terraform Documentation: success
  • 🔖 Commitlint: success

Working Directory: .
Pusher: @asmith030, Action: pull_request
Workflow Run Link: https://github.com/appvia/terraform-azurerm-wayfinder/actions/runs/10797812835

Copy link

github-actions bot commented Aug 15, 2024

Pull Request Review Status

  • 🖌 Terraform Format and Style: success
  • 🔍 Terraform Linting: success
  • 👮 Terraform Security Check: success
  • 🔘 Terraform Tests: success
  • 🔧 Terraform Initialisation: success
  • 🤖 Terraform Validation: success
  • 🤖 Terraform Example Validation: success
  • 📖 Terraform Documentation: success
  • 🔖 Commitlint: success

Working Directory: modules/cloudaccess
Pusher: @asmith030, Action: pull_request
Workflow Run Link: https://github.com/appvia/terraform-azurerm-wayfinder/actions/runs/10797812835

@asmith030 asmith030 force-pushed the private-link-adam branch 2 times, most recently from bafb0de to 76c3155 Compare August 15, 2024 11:59
Copy link
Member

@mrsheepuk mrsheepuk left a comment

Choose a reason for hiding this comment

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

One thought, other than that, this looks fine. As it is off by default I'm comfortable with this going in.

@asmith030 asmith030 merged commit 1d549ff into vnext Sep 12, 2024
22 checks passed
@KashifSaadat KashifSaadat deleted the private-link-adam branch September 12, 2024 09:05
KashifSaadat pushed a commit that referenced this pull request Sep 16, 2024
KashifSaadat added a commit that referenced this pull request Sep 16, 2024
* chore: update Wayfinder to version 2.8.1 (#61)

* chore: update Wayfinder to version 2.8.1
* chore: update kube version to 1.30

* feat: add permissions to installer for private links (#62)

Co-authored-by: Adam Smith <[email protected]>

* refactor: switch to supported kubectl provider (#67)

* chore: update Wayfinder to version 2.8.1 (#61)

* chore: update Wayfinder to version 2.8.1
* chore: update kube version to 1.30

* refactor: switch to supported kubectl provider

---------

Co-authored-by: Lewis Marshall <[email protected]>

---------

Co-authored-by: Lewis Marshall <[email protected]>
Co-authored-by: Adam Smith <[email protected]>
Co-authored-by: Adam Smith <[email protected]>
KashifSaadat added a commit that referenced this pull request Sep 16, 2024
* chore: update Wayfinder to version 2.8.1 (#61)

* chore: update Wayfinder to version 2.8.1
* chore: update kube version to 1.30

* feat: add permissions to installer for private links (#62)

Co-authored-by: Adam Smith <[email protected]>

* refactor: switch to supported kubectl provider (#67)

* chore: update Wayfinder to version 2.8.1 (#61)

* chore: update Wayfinder to version 2.8.1
* chore: update kube version to 1.30

* refactor: switch to supported kubectl provider

---------

Co-authored-by: Lewis Marshall <[email protected]>

* feat: bump AKS nodepool version to 1.30 (#69)

---------

Co-authored-by: Lewis Marshall <[email protected]>
Co-authored-by: Adam Smith <[email protected]>
Co-authored-by: Adam Smith <[email protected]>
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