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

[ci]: Download artifact instead of using nfs storage #6570

Merged
merged 4 commits into from
Jan 28, 2021

Conversation

qiluo-msft
Copy link
Collaborator

- Why I did it
I notice that I rerun a failed job (not the stages), the nfs store is already cleaned by previous failed jobs.

- How I did it
Use artifacts instead.

- How to verify it

- Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

@qiluo-msft qiluo-msft requested a review from lguohan January 27, 2021 01:00
@@ -149,9 +146,13 @@ stages:
displayName: "kvmtest"
timeoutInMinutes: 240
steps:
- task: DownloadPipelineArtifact@2
inputs:
artifact: sonic-buildimage.kvm
Copy link
Collaborator

Choose a reason for hiding this comment

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

i am not sure it will download the artifact from the current job?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I learned on https://docs.microsoft.com/en-us/azure/devops/pipelines/artifacts/pipeline-artifacts?view=azure-devops&tabs=yaml

Pipeline artifacts provide a way to share files between stages in a pipeline or between different pipelines.

In this case, we share files between stages.

@qiluo-msft qiluo-msft marked this pull request as draft January 27, 2021 05:25
@qiluo-msft qiluo-msft marked this pull request as ready for review January 27, 2021 18:40
@lguohan
Copy link
Collaborator

lguohan commented Jan 28, 2021

kvmtest passed.

@lguohan lguohan merged commit 69c5832 into sonic-net:master Jan 28, 2021
@qiluo-msft qiluo-msft deleted the qiluo/ap_artifact branch January 28, 2021 04:15
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