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

[azure-docs] refine deployment with ACR docs #26397

Merged
merged 1 commit into from
Dec 12, 2024

Conversation

mlarose
Copy link
Contributor

@mlarose mlarose commented Dec 11, 2024

Summary & Motivation

Updates the Azure Container Registry (ACR) user code documentation with missing instructions for configuring GitHub Actions and Dagster Cloud. Adds detailed steps for setting up environment variables, updating other step of the CI/CD workflow, and configuring the dagster_cloud.yaml file to work with ACR.

How I Tested These Changes

  • All changes are resulting from reproducing and fixing omissions in the previous version.

Changelog

  • NOCHANGELOG

Copy link
Contributor Author

mlarose commented Dec 11, 2024

@mlarose mlarose requested a review from dpeng817 December 11, 2024 02:11
@mlarose mlarose force-pushed the mlarose/dogfood-azure-docs branch from deca5e4 to 6875473 Compare December 11, 2024 02:18
@mlarose mlarose marked this pull request as ready for review December 11, 2024 12:37
@mlarose mlarose requested a review from neverett as a code owner December 11, 2024 12:37
Copy link
Contributor

@dpeng817 dpeng817 left a comment

Choose a reason for hiding this comment

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

Thanks for the improvements! A few comments for further discussion but I think this overall lgtm

@mlarose mlarose force-pushed the mlarose/dogfood-azure-docs branch from d6e677f to ba9f895 Compare December 11, 2024 16:45
@neverett neverett added the docs-to-migrate Docs to migrate to new docs site label Dec 12, 2024
Copy link
Contributor

@neverett neverett left a comment

Choose a reason for hiding this comment

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

Left a few suggestions, otherwise this looks great to me.

@mlarose mlarose force-pushed the mlarose/dogfood-azure-docs branch from ba9f895 to b8cb7dd Compare December 12, 2024 19:09
Copy link
Contributor Author

mlarose commented Dec 12, 2024

Merge activity

  • Dec 12, 2:10 PM EST: A user started a stack merge that includes this pull request via Graphite.
  • Dec 12, 2:11 PM EST: Graphite rebased this pull request as part of a merge.
  • Dec 12, 2:13 PM EST: A user merged this pull request with Graphite.

Additions and modifications noted from dogfooding the guide
@mlarose mlarose force-pushed the mlarose/dogfood-azure-docs branch from b8cb7dd to 22311e0 Compare December 12, 2024 19:11
@mlarose mlarose merged commit 3a406e2 into master Dec 12, 2024
1 of 2 checks passed
@mlarose mlarose deleted the mlarose/dogfood-azure-docs branch December 12, 2024 19:13
pskinnerthyme pushed a commit to pskinnerthyme/dagster that referenced this pull request Dec 16, 2024
## Summary & Motivation
Updates the Azure Container Registry (ACR) user code documentation with missing instructions for configuring GitHub Actions and Dagster Cloud. Adds detailed steps for setting up environment variables, updating other step of the CI/CD workflow, and configuring the `dagster_cloud.yaml` file to work with ACR.

## How I Tested These Changes
- All changes are resulting from reproducing and fixing omissions in the previous version.

## Changelog
- NOCHANGELOG
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs-to-migrate Docs to migrate to new docs site
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants