Skip to content

fix(hidi): refresh container base images during publishing - #934

Open
Treicy Sanchez (Treicysg) wants to merge 5 commits into
mainfrom
fix/hidi-runtime-8.0.33
Open

Treicy Sanchez (Treicysg) wants to merge 5 commits into
mainfrom
fix/hidi-runtime-8.0.33

Conversation

@Treicysg

@Treicysg Treicy Sanchez (Treicysg) commented Nov 4, 2026 •

Copy link
Copy Markdown

Pull current base images when building Hidi containers.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@Treicysg
Treicy Sanchez (Treicysg) requested a review from a team as a code owner November 4, 2026 19:44
@Treicysg
Treicy Sanchez (Treicysg) marked this pull request as draft November 4, 2026 19:47
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@Treicysg Treicy Sanchez (Treicysg) changed the title fix(hidi): update container runtime to .NET 8.0.33 fix(hidi): refresh container base images during publishing Nov 4, 2026
@Treicysg
Treicy Sanchez (Treicysg) marked this pull request as ready for review November 4, 2026 20:59
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Comment thread .azure-pipelines/hidi-release.yml Outdated
--tag "msgraphprodregistry.azurecr.io/public/openapi/hidi:$(HidiReleaseVersion)" `
--tag "msgraphprodregistry.azurecr.io/public/openapi/hidi:latest" `
--push .
docker buildx build --pull --platform linux/amd64,linux/arm64/v8 --secret "id=nuget_config,src=$nugetConfigPath" --tag "msgraphprodregistry.azurecr.io/public/openapi/hidi:$(HidiReleaseVersion)" --tag "msgraphprodregistry.azurecr.io/public/openapi/hidi:latest" --push .

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

can we please retain the multi-line style?

The run-on line is a pain to parse in an editor, yeah, I still like things to be human readable

@Treicysg Treicy Sanchez (Treicysg) Nov 5, 2026 •

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Should we mark SonarCloud warnings as false positives? or use multi-line argument arrays?

Treicy Sanchez Gutierrez (from Dev Box) and others added 2 commits November 5, 2026 12:06
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@sonarqubecloud

sonarqubecloud Bot commented Nov 5, 2026

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
D Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

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