Skip to content

Restore Github Actions #36

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

Merged
merged 9 commits into from
Jun 10, 2025
Merged

Restore Github Actions #36

merged 9 commits into from
Jun 10, 2025

Conversation

puneetsarna
Copy link
Contributor

No description provided.

@puneetsarna
Copy link
Contributor Author

NOTE: This does not have the swagger updates yet. Will add those as a follow up.

@puneetsarna puneetsarna force-pushed the ps-dev-actions-test branch from a23c0e8 to ee270e3 Compare June 10, 2025 16:38
@kuthiala
Copy link

Please add a flag for adding --debug to az deployment group create and so that we don't display the base64 tarball output by default. That is sensitive

@puneetsarna puneetsarna force-pushed the ps-dev-actions-test branch 4 times, most recently from 3140756 to 8bfcd30 Compare June 10, 2025 21:14
This is the new deployment that will now be used
to test the Github action.
We don't have the test infrastructure fully set up
to handle some of this yet. For now, we can just
proxy pass to an upstream on localhost and update
it down the road.
Now that we are using localhost as an upstream we
need to make sure that the test to pass traffic
checks for the correct headers based on the new
config.
These are newer and recommended for use,
especially for OIDC.
Azure CLI does not support wget from within the
container for its action so swapping the
invocations entirely with curl instead.
This flag, turned off by default, can be enabled
by the customer to debug their Azure CLI
invocations.
@puneetsarna puneetsarna force-pushed the ps-dev-actions-test branch from 8bfcd30 to 835b0f9 Compare June 10, 2025 21:21
@puneetsarna puneetsarna marked this pull request as ready for review June 10, 2025 21:22
@puneetsarna
Copy link
Contributor Author

README and mention RBAC

- Update the referenced nginx actions to 0.3.1 as
  cert actions do not work for 0.3.0.
- Add an excerpt around Azure RBAC describing the
  access that the service principal will need in
  order for the action to work.
@puneetsarna puneetsarna merged commit 5ad0d6a into main Jun 10, 2025
2 checks passed
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