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

Continuous deployment to azure #34

Merged
merged 15 commits into from
Jul 24, 2024

Conversation

CalPinSW
Copy link
Owner

No description provided.

Copy link
Contributor

@JackMeadDev JackMeadDev left a comment

Choose a reason for hiding this comment

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

Nice work, I can see the app running happily in Azure with the favicons, and the pipeline reporting successful deployment - go ahead and merge when you're ready!

@@ -76,5 +76,5 @@ The frontend and backend applications are deployed to two Azure Web Applications
1. Building new Docker images with (from the relevant frontend/backend directory) `docker build --target production --tag calpin/playlist-manager-"frontend/backend":prod .`
2. Pushing the new image to Docker Hub `docker push calpin/playlist-manager-"frontend/backend":prod`
3. Sending a curl request to the appropriate Webhook url:
- Frontend: `curl -dH -X POST "https://\$PlayMan:d17p8pbD0t6xpCdep4tPdetcbsTlZiomZ6kwAwuJB9XaXwjE5hmmgSpBTYqQ@playman.scm.azurewebsites.net/api/registry/webhook"`
- Backend: `curl -dH -X POST "https://\$PlayManBackend:aBPc2NHMrYLEHQtmPB1JhQKgtsonxezPjliwxptyx2142STiv91aRxmnmKe6@playmanbackend.scm.azurewebsites.net/api/registry/webhook"`
- Frontend: `curl -dH -X POST "Frontend Webhook from Azure"`
Copy link
Contributor

Choose a reason for hiding this comment

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

Ah, caught! But I think not rotated in Azure still!

Copy link
Owner Author

Choose a reason for hiding this comment

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

Should now be rotated 😬

Copy link
Owner Author

Choose a reason for hiding this comment

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

Let's merge and see what happens 😀

@CalPinSW CalPinSW merged commit 27c482f into host-on-azure Jul 24, 2024
8 checks passed
@CalPinSW CalPinSW deleted the continuous-deployment-to-azure branch July 24, 2024 15:28
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