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

Onboard ci-mgmt #1596

Open
wants to merge 22 commits into
base: master
Choose a base branch
from
Open

Onboard ci-mgmt #1596

wants to merge 22 commits into from

Conversation

rquitales
Copy link
Member

@rquitales rquitales commented Jan 24, 2025

Proposed changes

This PR revives #1474 by rebasing and modifying to fit the latest changes to pulumi-eks and ci-mgmt.

Changes made:

  • Restructure the go modules to align with bridged providers for now
  • Generated ci-mgmt workflows
  • Added kubectl installation as a setup step
  • Runs all tests in parallel across 25 shards (including upgrade tests)
  • Fixed the ignore-sclaing-changes test code to be valid

Passing alpha release: https://github.com/pulumi/pulumi-eks/actions/runs/12961981890

Related issues (optional)

Copy link

Does the PR have any schema changes?

Looking good! No breaking changes found.
No new resources/functions.

Maintainer note: consult the runbook for dealing with any breaking changes.

Copy link

codecov bot commented Jan 24, 2025

Codecov Report

Attention: Patch coverage is 0% with 84 lines in your changes missing coverage. Please review.

Please upload report for BASE (master@5c42f19). Learn more about missing BASE report.

Files with missing lines Patch % Lines
provider/cmd/pulumi-gen-eks/main.go 0.00% 84 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##             master   #1596   +/-   ##
========================================
  Coverage          ?   0.00%           
========================================
  Files             ?       1           
  Lines             ?    2806           
  Branches          ?       0           
========================================
  Hits              ?       0           
  Misses            ?    2806           
  Partials          ?       0           

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

@rquitales rquitales changed the title [wip] Onboard ci-mgmt Jan 24, 2025
@rquitales rquitales marked this pull request as ready for review January 24, 2025 23:12
@rquitales rquitales requested review from t0yv0 and flostadler January 24, 2025 23:52
@rquitales rquitales self-assigned this Jan 24, 2025
@rquitales rquitales mentioned this pull request Jan 25, 2025
4 tasks
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.

1 participant