containerized-data-importer icon indicating copy to clipboard operation
containerized-data-importer copied to clipboard

Using fork of openshift/library-go

Open mhenriks opened this issue 5 years ago • 12 comments

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:

/kind bug

/kind enhancement

What happened:

#1091 references a fork of openshift/library-go

Once the following PR is merged, CDI should go back the mainline:

https://github.com/openshift/library-go/pull/540

What you expected to happen:

How to reproduce it (as minimally and precisely as possible):

Anything else we need to know?:

Environment:

  • CDI version (use kubectl get deployments cdi-deployment -o yaml):
  • Kubernetes version (use kubectl version):
  • Cloud provider or hardware configuration:
  • Install tools:
  • Others:

mhenriks avatar Feb 05 '20 19:02 mhenriks

Issues go stale after 90d of inactivity. Mark the issue as fresh with /remove-lifecycle stale. Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

/lifecycle stale

kubevirt-bot avatar May 05 '20 19:05 kubevirt-bot

Stale issues rot after 30d of inactivity. Mark the issue as fresh with /remove-lifecycle rotten. Rotten issues close after an additional 30d of inactivity.

If this issue is safe to close now please do so with /close.

/lifecycle rotten

kubevirt-bot avatar Jun 04 '20 20:06 kubevirt-bot

/lifecycle frozen

awels avatar Jun 04 '20 20:06 awels

This still applies right @mhenriks

awels avatar Jun 04 '20 20:06 awels

@mhenriks please take a look and see if we can make the needed methods public in openshift/library-go#540

aglitke avatar Feb 13 '23 13:02 aglitke

created https://github.com/openshift/library-go/pull/1486

mhenriks avatar Mar 10 '23 15:03 mhenriks

created openshift/library-go#1486

Just lgtmed, hope we can merge it soon.

alromeros avatar Aug 02 '23 15:08 alromeros

Closing this since it's been addressed by https://github.com/openshift/library-go/commit/434fa21f5e457d51f436c4b3804c7803db1da42d. Feel free to reopen if necessary.

alromeros avatar Jan 29 '24 13:01 alromeros

/reopen so we remember to get rid of the fork

akalenyu avatar May 02 '24 14:05 akalenyu

@akalenyu: Reopened this issue.

In response to this:

/reopen so we remember to get rid of the fork

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.

kubevirt-bot avatar May 02 '24 14:05 kubevirt-bot

thanks for the reminder @akalenyu

mhenriks avatar May 03 '24 13:05 mhenriks

Unfortunately have to wait until we move to k8s 1.29 (release-4.16+ branch in library-go) to do this

mhenriks avatar May 03 '24 14:05 mhenriks