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
toolbox dump: include AWS route table objects #16073
Conversation
This "can't" be the fix, but #16072 passed with this included. I think the cloud-init change is more likely to be the cause (though that doesn't make a lot of sense), but this can serve as our "control" /test pull-kops-e2e-cni-calico-ipv6 |
/wip so that it doesn't merge, I don't think it's particularly time-critical |
/wip |
e02562a
to
cffd473
Compare
This turned out not to be related to the IPv6 test failures, but I think we should be dumping all these objects. Debating whether we should just default the Dump function, but as we still have to set object I though a bit of repetition here wasn't the end of the world. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: hakman 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 |
No description provided.