Skip to content

Migrate github-release pipeline to 1ES governed template#178

Merged
shishirb-MSFT merged 19 commits intodevelopfrom
user/shishirb/active
Nov 18, 2023
Merged

Migrate github-release pipeline to 1ES governed template#178
shishirb-MSFT merged 19 commits intodevelopfrom
user/shishirb/active

Conversation

@shishirb-MSFT
Copy link
Collaborator

No description provided.

@shishirb-MSFT shishirb-MSFT requested a review from a team as a code owner November 15, 2023 19:40
imageVersion: ${{parameters.imageVersion}}
config: debug
- publish: $(Build.ArtifactStagingDirectory)
artifact: 'doclient-drop-${{parameters.targetOsArch}}-debug'
Copy link
Collaborator

@Jerry-GeGarcia Jerry-GeGarcia Nov 17, 2023

Choose a reason for hiding this comment

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

drop

is this naming convention new? #Closed

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes. The 1ES tasks also publish artifacts/drop, so I'm making ours distinguishable.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I see. What is different from the artifacts that the 1ES task produces and ours?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Copy link
Collaborator

Choose a reason for hiding this comment

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

thank you, that helps. The doclient convention makes sense; wasn't too sure about the "drop", but I guess its consistent.

TargetFolder: '$(Build.ArtifactStagingDirectory)/agent-${{parameters.targetOsArch}}-${{parameters.config}}'
displayName: 'Populate debug symbols to staging dir'

- task: PublishBuildArtifacts@1
Copy link
Collaborator

@Jerry-GeGarcia Jerry-GeGarcia Nov 17, 2023

Choose a reason for hiding this comment

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

PublishBuildArtifacts

this was removed across all three. I assume the "artifact" part in the do-docker-jobs.yml does the equivalent now? #Closed

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes. And the same part in release-docker-build-steps.yml as well.

It was the migration tool that moved these lines from here the parent yaml, so I kept it that way.

changeLogCompareToRelease: 'lastNonDraftReleaseByTag'
changeLogCompareToReleaseTag: '$(Release.PreviousVersion)'
changeLogType: 'commitBased'
- name: SkipPublishing
Copy link
Collaborator

@Jerry-GeGarcia Jerry-GeGarcia Nov 17, 2023

Choose a reason for hiding this comment

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

SkipPublishing

is this where we will integrate the PMC publishing flow? #Closed

Copy link
Collaborator Author

@shishirb-MSFT shishirb-MSFT Nov 17, 2023

Choose a reason for hiding this comment

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

We are currently attempting to create that as its own separate pipeline (release pipeline) but hitting some issues. We'll know more next week.

If we decide to do it in this pipeline, then yes we will integrate it after the GithubRelease task at the bottom.

@Jerry-GeGarcia
Copy link
Collaborator

@shiyi-peng is added to the review. #Closed

# Example:
# docker run -ti --rm --entrypoint=/bin/bash -v D:\do-client-lite:/code -v D:\temp\build_client_lite\linux-debug:/build ubuntu1804-x64

FROM mcr.microsoft.com/mirror/docker/library/ubuntu:18.04@sha256:152dc042452c496007f07ca9127571cb9c29697f42acbfad72324b2bb2e43c98
Copy link
Collaborator

@Jerry-GeGarcia Jerry-GeGarcia Nov 17, 2023

Choose a reason for hiding this comment

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

152dc042452c496007f07ca9127571cb9c29697f42acbfad72324b2bb2e43c98

hash of the container? worth adding a comment? #Resolved

Copy link
Collaborator

@Jerry-GeGarcia Jerry-GeGarcia left a comment

Choose a reason for hiding this comment

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

:shipit:

@shishirb-MSFT shishirb-MSFT merged commit 0d93af4 into develop Nov 18, 2023
@shishirb-MSFT shishirb-MSFT deleted the user/shishirb/active branch November 18, 2023 00:52
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