Skip to content

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented May 21, 2024

Backport of #92958 to release/8.0

/cc @jkoritzinsky

Infrastructure fix for publishing PGO assets. When backporting other work for the 1ES Pipelines move, this follow-up PR was missed, which broke publishing assets from the PGO legs.

Customer Impact

  • Customer reported
  • Found internally

[Select one or both of the boxes. Describe how this issue impacts customers, citing the expected and actual behaviors and scope of the issue. If customer-reported, provide the issue number.]

Regression

  • Yes
  • No

[If yes, specify when the regression was introduced. Provide the PR or commit if known.]

Regressed with #102097

Testing

[How was the fix verified? How was the issue missed previously? What tests were added?]
We have high confidence in this fix as the same PR provided a fix for the corresponding change in main.

Risk

[High/Medium/Low. Justify the indication by mentioning how risks were measured and addressed.]

Low risk

@ghost ghost added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label May 21, 2024
@jkoritzinsky jkoritzinsky requested a review from eduardo-vp May 21, 2024 21:52
@jkoritzinsky jkoritzinsky added area-Infrastructure and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels May 21, 2024
@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

@carlossanlop carlossanlop added the NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) label May 21, 2024
@carlossanlop
Copy link
Contributor

It was decided not to merge this at the moment to avoid resetting the runtime build.

@jkoritzinsky jkoritzinsky changed the base branch from release/8.0 to release/8.0-staging May 21, 2024 22:26
@jkoritzinsky jkoritzinsky force-pushed the backport/pr-92958-to-release/8.0 branch from 117896c to 987ce19 Compare May 21, 2024 22:27
@jkoritzinsky jkoritzinsky changed the title [release/8.0] Fix publishing PGO assets [release/8.0-staging] Fix publishing PGO assets May 21, 2024
@jkoritzinsky
Copy link
Member

Retargeted to staging so as to not require resetting the build (as it will only get into the next servicing build).

@jkoritzinsky jkoritzinsky added Servicing-consider Issue for next servicing release review and removed NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) labels May 21, 2024
@carlossanlop
Copy link
Contributor

I see it's just a yaml change. Do we still need this to go through Tactics? Does this modify the shipped bits?

@jkoritzinsky
Copy link
Member

This does not modify shipping bits. It only modifies non-shipping PGO bits.

@carlossanlop carlossanlop added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels May 21, 2024
Copy link
Contributor

@carlossanlop carlossanlop left a comment

Choose a reason for hiding this comment

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

We can treat is as tell-mode now that it is targeting staging. Feel free to merge if the CI looks good.

@jkoritzinsky jkoritzinsky merged commit abbf9c6 into release/8.0-staging May 22, 2024
@jkoritzinsky jkoritzinsky deleted the backport/pr-92958-to-release/8.0 branch May 22, 2024 02:02
@github-actions github-actions bot locked and limited conversation to collaborators Jun 21, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-Infrastructure Servicing-approved Approved for servicing release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants