Skip to content

CI steps for Chocolatey package - take 2#4536

Merged
alpeb merged 2 commits intomasterfrom
alpeb/chocolatey-ci-2
Jun 15, 2020
Merged

CI steps for Chocolatey package - take 2#4536
alpeb merged 2 commits intomasterfrom
alpeb/chocolatey-ci-2

Conversation

@alpeb
Copy link
Member

@alpeb alpeb commented Jun 2, 2020

Followup to #4205, supersedes #4400

This adds:

  • A new job psscript-analyzer into the statics_checks.yml
    workflow for linting the Chocolatey Powershell script.
  • A new choco_pack job in the release.yml workflow for
    updating the Chocolatey spec file and generating the
    package. This is only triggered for stable releases. It requires
    a windows runner in order to run the choco tooling (in theory
    it should have worked on a linux runner but in practice it
    didn't).
  • The Create release step was updated to upload the generated package,
    if present.
  • The source file path in bin/win/linkerd.nuspec was updated
    to make this work.

Followup to #4205, supersedes #4205

This adds:

- A new job psscript-analyzer into the `statics_checks.yml`
workflow for linting the Chocolatey Powershell script.
- A new `choco_pack` job in the `release.yml` workflow for
updating the Chocolatey spec file and generating the
package. This is only triggered for stable releases. It requires
a windows runner in order to run the choco tooling (in theory
it should have worked on a linux runner but in practice it
didn't).
- The `Create release` step was updated to upload the generated package,
if present.
- The source file path in `bin/win/linkerd.nuspec` was updated
to make this work.
@alpeb alpeb requested a review from kleimkuhler as a code owner June 2, 2020 14:22
@alpeb
Copy link
Member Author

alpeb commented Jun 2, 2020

This tested fine in my fork: action run
Which produced this file in the list of the released assets.

To test you can download that file into Windows, and do choco install linkerd -s .

image

Copy link
Contributor

@kleimkuhler kleimkuhler left a comment

Choose a reason for hiding this comment

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

Looks great!

@grampelberg
Copy link
Contributor

@alpeb any reason this hasn't merged yet?

@alpeb alpeb merged commit d10ed2a into master Jun 15, 2020
@alpeb alpeb deleted the alpeb/chocolatey-ci-2 branch June 15, 2020 21:42
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.

4 participants