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

Ignore Windows artifacts when creating Linux bundle #2928

Conversation

jossemargt-3pillar
Copy link
Contributor

@jossemargt-3pillar jossemargt-3pillar commented May 16, 2022

As reported in #2883, currently Windows binaries are being bundled together with Linux ones, and as result polluting the users environment when installing RKE2.

Proposed Changes

Update packaging scripts to ignore Windows artifacts when creating the RKE2 Linux bundle (tar.gz)

Types of Changes

RKE2 packaging

Verification

  • The Windows artifacts (.exe and .ps1) shouldn't be present in RKE2 linux CI and release tar.gz files
  • No regressions detected on windows bundle
$ export RKE2_COMMIT=98f78ada09b6f356e7b1d4bdc2db22e03228a957
# Expected change for Linux
$ curl -sL https://storage.googleapis.com/rke2-ci-builds/rke2.linux-amd64-${RKE2_COMMIT}.tar.gz | tar -zt | grep bin
bin/
bin/rke2-uninstall.sh
bin/rke2
bin/rke2-killall.sh
# Check regressions for Windows
$ curl -sL https://storage.googleapis.com/rke2-ci-builds/rke2.windows-amd64-${RKE2_COMMIT}.tar.gz | tar -zt | grep -E '.(exe|ps1)'
bin/rke2-uninstall.ps1
bin/rke2.exe

Linked Issues

#2883

Further Comments

Problem: As reported in rancher#2883, currently Windows binaries are being
bundled together with Linux ones, and as result polluting the users
environment when installing RKE2

Solution: Ignore Windows scripts and binaries when creating the RKE2
Linux bundle.

Signed-off-by: Jonnatan Jossemar Cordero <[email protected]>
@jossemargt-3pillar jossemargt-3pillar self-assigned this May 16, 2022
@jossemargt-3pillar jossemargt-3pillar changed the title Ignore Windows binaries when creating Linux bundle Ignore Windows artifacts when creating Linux bundle May 16, 2022
@jossemargt-3pillar jossemargt-3pillar marked this pull request as ready for review May 17, 2022 15:35
@jossemargt-3pillar jossemargt-3pillar requested a review from a team as a code owner May 17, 2022 15:35
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.

3 participants