OCPBUGS-18906: Remove dependency on k8s.io/kubernetes packages#3913
Conversation
|
@djoshy: This pull request references Jira Issue OCPBUGS-18906, which is invalid:
Comment The bug has been updated to refer to the pull request using the external bug tracker. DetailsIn response to this:
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. |
|
Skipping CI for Draft Pull Request. |
|
/test unit |
|
Nice; I wonder if there's actually some public equivalent of this in the github.com/containers namespace? Using a search leads to the podman copy; another copy here looks like? https://github.com/containers/image/blob/255f36875ce5ca16bf941c18d4a2af2663daf952/pkg/docker/config/config.go#L26 May be worth trying to factor that out into a public type. (But obviously, this is way better than vendoring kube, so please don't block on it, just a drive-by thought) |
|
/test e2e-gcp-op /jira refresh |
|
@djoshy: This pull request references Jira Issue OCPBUGS-18906, which is valid. The bug has been moved to the POST state. 3 validation(s) were run on this bug
Requesting review from QA contact: DetailsIn response to this:
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. |
|
/test all |
|
/test okd-scos-images |
|
Verified using IPI on GCP All critical test cases passed These test cases were executed and passed:
We can add the qe-approved label /label qe-approved |
|
/retest-required |
|
@djoshy: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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. |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: djoshy, sinnykumari The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
@djoshy: Jira Issue OCPBUGS-18906: All pull requests linked via external trackers have merged: Jira Issue OCPBUGS-18906 has been moved to the MODIFIED state. DetailsIn response to this:
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. |
|
/cherrypick release-4.14 |
|
@djoshy: new pull request created: #3940 DetailsIn response to this:
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 I did
Created internal objects since we were not using any functionality of the package anyway. We were just using it as a vehicle to marshal and unmarshal.
- How to verify it
Secrets merging should work as before.
- Description for the changelog
operator: removed k8s pkg dependancy