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

Slim down E2E artifacts #6097

Merged
merged 2 commits into from
Jun 6, 2024
Merged

Slim down E2E artifacts #6097

merged 2 commits into from
Jun 6, 2024

Conversation

dereknola
Copy link
Member

@dereknola dereknola commented Jun 3, 2024

Signed-off-by: Derek Nola [email protected]

Proposed Changes

  • For E2E CI, don't use and copy rke2-runtime.tar the install.sh script doesn't handle it and rke2-images.linux-amd64.tar.zst is actually just rke2-runtime compressed. Saves about 100MB in artifact uploads/downloads

Types of Changes

  • E2E optimization

Verification

CI still Green

Testing

Linked Issues

N/A

User-Facing Change


Further Comments

@dereknola dereknola requested a review from a team as a code owner June 3, 2024 21:21
@dereknola dereknola changed the title [WIP] Slim E2E Slim down E2E artifacts Jun 4, 2024
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 9.82%. Comparing base (c727b36) to head (9e94f8f).
Report is 9 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #6097       +/-   ##
==========================================
- Coverage   26.10%   9.82%   -16.29%     
==========================================
  Files          32      32               
  Lines        2697    2698        +1     
==========================================
- Hits          704     265      -439     
- Misses       1947    2411      +464     
+ Partials       46      22       -24     
Flag Coverage Δ
inttests 9.82% <ø> (-0.01%) ⬇️
unittests ?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@dereknola dereknola merged commit eb6ad1e into rancher:master Jun 6, 2024
5 checks passed
dereknola added a commit to dereknola/rke2 that referenced this pull request Jun 7, 2024
* Don't use rke2-runtime.tar in e2e

Signed-off-by: Derek Nola <[email protected]>

* Don't upload rke2-runtime.tar

Signed-off-by: Derek Nola <[email protected]>

---------

Signed-off-by: Derek Nola <[email protected]>
dereknola added a commit to dereknola/rke2 that referenced this pull request Jun 7, 2024
* Don't use rke2-runtime.tar in e2e

Signed-off-by: Derek Nola <[email protected]>

* Don't upload rke2-runtime.tar

Signed-off-by: Derek Nola <[email protected]>

---------

Signed-off-by: Derek Nola <[email protected]>
dereknola added a commit to dereknola/rke2 that referenced this pull request Jun 7, 2024
* Don't use rke2-runtime.tar in e2e

Signed-off-by: Derek Nola <[email protected]>

* Don't upload rke2-runtime.tar

Signed-off-by: Derek Nola <[email protected]>

---------

Signed-off-by: Derek Nola <[email protected]>
dereknola added a commit that referenced this pull request Jun 10, 2024
* Fix loadManifests function (#6058)

Signed-off-by: Derek Nola <[email protected]>

* Slim down E2E artifacts (#6097)

* Don't use rke2-runtime.tar in e2e

Signed-off-by: Derek Nola <[email protected]>

* Don't upload rke2-runtime.tar

Signed-off-by: Derek Nola <[email protected]>

---------

Signed-off-by: Derek Nola <[email protected]>

* Add custom golang setup action for better caching (#6144)

Signed-off-by: Derek Nola <[email protected]>

* Support MixedOS E2E local testing (#6137)

* Consolidate CreateWindowsCluster test function

Signed-off-by: Derek Nola <[email protected]>

* Support local windows builds in mixedOS e2e test

Signed-off-by: Derek Nola <[email protected]>

* Add mixedos to E2E CI

Signed-off-by: Derek Nola <[email protected]>

* Simplify local cluster provisioning with libvirt native parallel bringup

Signed-off-by: Derek Nola <[email protected]>

* Don't run mixedOS on E2E

Signed-off-by: Derek Nola <[email protected]>

---------

Signed-off-by: Derek Nola <[email protected]>

---------

Signed-off-by: Derek Nola <[email protected]>
dereknola added a commit that referenced this pull request Jun 10, 2024
* Fix loadManifests function (#6058)

Signed-off-by: Derek Nola <[email protected]>

* Slim down E2E artifacts (#6097)

* Don't use rke2-runtime.tar in e2e

Signed-off-by: Derek Nola <[email protected]>

* Don't upload rke2-runtime.tar

Signed-off-by: Derek Nola <[email protected]>

---------

Signed-off-by: Derek Nola <[email protected]>

* Add custom golang setup action for better caching (#6144)

Signed-off-by: Derek Nola <[email protected]>

* Support MixedOS E2E local testing (#6137)

* Consolidate CreateWindowsCluster test function

Signed-off-by: Derek Nola <[email protected]>

* Support local windows builds in mixedOS e2e test

Signed-off-by: Derek Nola <[email protected]>

* Add mixedos to E2E CI

Signed-off-by: Derek Nola <[email protected]>

* Simplify local cluster provisioning with libvirt native parallel bringup

Signed-off-by: Derek Nola <[email protected]>

* Don't run mixedOS on E2E

Signed-off-by: Derek Nola <[email protected]>

---------

Signed-off-by: Derek Nola <[email protected]>

---------

Signed-off-by: Derek Nola <[email protected]>
dereknola added a commit that referenced this pull request Jun 10, 2024
* Fix loadManifests function (#6058)

Signed-off-by: Derek Nola <[email protected]>

* Slim down E2E artifacts (#6097)

* Don't use rke2-runtime.tar in e2e

Signed-off-by: Derek Nola <[email protected]>

* Don't upload rke2-runtime.tar

Signed-off-by: Derek Nola <[email protected]>

---------

Signed-off-by: Derek Nola <[email protected]>

* Add custom golang setup action for better caching (#6144)

Signed-off-by: Derek Nola <[email protected]>

* Support MixedOS E2E local testing (#6137)

* Consolidate CreateWindowsCluster test function

Signed-off-by: Derek Nola <[email protected]>

* Support local windows builds in mixedOS e2e test

Signed-off-by: Derek Nola <[email protected]>

* Add mixedos to E2E CI

Signed-off-by: Derek Nola <[email protected]>

* Simplify local cluster provisioning with libvirt native parallel bringup

Signed-off-by: Derek Nola <[email protected]>

* Don't run mixedOS on E2E

Signed-off-by: Derek Nola <[email protected]>

---------

Signed-off-by: Derek Nola <[email protected]>

---------

Signed-off-by: Derek Nola <[email protected]>
@dereknola dereknola deleted the slim_package branch June 13, 2024 16:05
iamsarthakk pushed a commit to iamsarthakk/rke2 that referenced this pull request Aug 19, 2024
* Don't use rke2-runtime.tar in e2e

Signed-off-by: Derek Nola <[email protected]>

* Don't upload rke2-runtime.tar

Signed-off-by: Derek Nola <[email protected]>

---------

Signed-off-by: Derek Nola <[email protected]>
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