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
AGENT-171: Update machine-api-operator and related dependencies #3871
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: lranjbar The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Codecov Report
@@ Coverage Diff @@
## master #3871 +/- ##
==========================================
- Coverage 66.22% 66.20% -0.02%
==========================================
Files 179 179
Lines 24872 24887 +15
==========================================
+ Hits 16472 16477 +5
- Misses 6877 6883 +6
- Partials 1523 1527 +4
|
It's complaining because that version of MAO used a replace directive for CAPAWS, with the version not listed in the openshift/installer#5970 works around this, but it certainly wouldn't hurt to update to more recent dependencies. (I assume it's safe to remove the replace directive for the AWS and maybe Azure CAPI providers now too.) |
@lranjbar: PR needs rebase. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
What are our plans to solve this vendoring requirement? |
We'll work to move out of vendoring by using cachito in the CI/release pipelines |
@lranjbar: The following tests failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
Working on updating the agent-installer branch in openshift/installer. I'm getting some dependency error centered around the old version of MAO that assisted-service uses.
Error:
Commands run:
Built locally using
skipper make validate
.List all the issues related to this PR
What environments does this code impact?
How was this code tested?
Assignees
/cc @pawanpinjarkar
/cc @andfasano
Checklist
docs
, README, etc)Reviewers Checklist