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

Copilot CI/CD pipeline #15

Merged
merged 13 commits into from
Jul 22, 2023
Merged

Copilot CI/CD pipeline #15

merged 13 commits into from
Jul 22, 2023

Conversation

alliscode
Copy link
Member

@alliscode alliscode commented Jul 21, 2023

Motivation and Context

Continuous deployment and verification will help us maintain the quality of Chat Copilot and ensure that it remains functional and deployable to Azure.

Description

  • Created workflow to deploy to two Azure hosted environments. The first deployment to int happens automatically after a merge to the main branch where scenario tests will be run. The second deployment to stable will happen when approved by the team and after all testing and verifications have successfully passed in int.
  • Updates deployment scripts to allow the Client Authority to be specified which allows us to restrict authentication in the app to members of a specific tenant, Microsoft in the case of this pipeline.
  • Several fixes in the various deployment scripts, .ps and .sh.

Contribution Checklist

@alliscode alliscode added this pull request to the merge queue Jul 22, 2023
Merged via the queue into main with commit 6bab3e2 Jul 22, 2023
7 checks passed
@alliscode alliscode deleted the copilot-pipeline-win branch July 22, 2023 00:06
pnashmsft pushed a commit to pnashmsft/chat-copilot that referenced this pull request May 27, 2024
Fixed issue with updating User Settings that caused a 404 to be thrown
teamleader-dev pushed a commit to vlink-group/chat-copilot that referenced this pull request Oct 7, 2024
### Motivation and Context

Continuous deployment and verification will help us maintain the quality
of Chat Copilot and ensure that it remains functional and deployable to
Azure.

### Description

- Created workflow to deploy to two Azure hosted environments. The first
deployment to `int` happens automatically after a merge to the main
branch where scenario tests will be run. The second deployment to
`stable` will happen when approved by the team and after all testing and
verifications have successfully passed in `int`.
- Updates deployment scripts to allow the Client Authority to be
specified which allows us to restrict authentication in the app to
members of a specific tenant, Microsoft in the case of this pipeline.
- Several fixes in the various deployment scripts, .ps and .sh.

### Contribution Checklist

<!-- Before submitting this PR, please make sure: -->

- [x] The code builds clean without any errors or warnings
- [x] The PR follows the [Contribution
Guidelines](https:/microsoft/copilot-chat/blob/main/CONTRIBUTING.md)
and the [pre-submission formatting
script](https:/microsoft/copilot-chat/blob/main/CONTRIBUTING.md#dev-scripts)
raises no violations
- [x] All unit tests pass, and I have added new tests where possible
- [x] I didn't break anyone 😄

---------

Co-authored-by: Ben Thomas <[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.

2 participants