Skip to content

[Packaging] Add CI job to build Mariner RPM - #20584

Merged
Jiashuo Li (jiasli) merged 6 commits into
Azure:devfrom
mfrw:mfrw/create-mariner-rpm
Dec 6, 2021
Merged

Jiashuo Li (jiasli) merged 6 commits into
Azure:devfrom
mfrw:mfrw/create-mariner-rpm

Conversation

@mfrw

Copy link
Copy Markdown
Member

Signed-off-by: Muhammad Falak R Wani falakreyaz@gmail.com

Description

Adds a new build step in the CI to create Mariner RPM.

Testing Guide

Tested the script locally.

History Notes

Add Build job in CI for Mariner Rpm.


This checklist is used to make sure that common guidelines for a pull request are followed.

This PR Depends on:

@jiasli

Jiashuo Li (jiasli) commented Dec 2, 2021

Copy link
Copy Markdown
Contributor

The artifact won't be built because Build.Reason doesn't include PullRequest.

@mfrw

Muhammad Falak R Wani (mfrw) commented Dec 2, 2021

Copy link
Copy Markdown
Member Author

The artifact won't be built because Build.Reason doesn't include PullRequest.

Sure .. Updating :) also this wont work as we are blocked on:

Signed-off-by: Muhammad Falak R Wani <falakreyaz@gmail.com>
Signed-off-by: Muhammad Falak R Wani <falakreyaz@gmail.com>
Comment thread azure-pipelines.yml Outdated
Comment thread azure-pipelines.yml
Comment on lines +682 to +696
- job: BuildYumPackageMariner
displayName: Build Yum Package Mariner
pool:
vmImage: 'ubuntu-20.04'
steps:
- task: Bash@3
displayName: 'Build Rpm Package: Mariner'
inputs:
targetType: 'filePath'
filePath: scripts/release/rpm/pipeline_mariner.sh
- task: PublishPipelineArtifact@0
displayName: 'Publish Artifact: yum-mariner'
inputs:
TargetPath: $(Build.ArtifactStagingDirectory)
ArtifactName: yum-mariner

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.

To be honest, I dislike using yum and rpm interchangeably (as it already is). This makes it difficult to understand for newcomers.

@jiasli
Jiashuo Li (jiasli) merged commit a7ca2bf into Azure:dev Dec 6, 2021
Jiashuo Li (jiasli) pushed a commit to jiasli/azure-cli that referenced this pull request Dec 6, 2021
@mfrw
Muhammad Falak R Wani (mfrw) deleted the mfrw/create-mariner-rpm branch December 6, 2021 06:16
@mfrw

Copy link
Copy Markdown
Member Author

Thank you Jiashuo Li (@jiasli)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants