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

Update fog-kubevirt to newer version #160

Merged
merged 1 commit into from Apr 20, 2020

Conversation

agrare
Copy link
Member

@agrare agrare commented Apr 20, 2020

Allow for the use of a newer version of kubeclient by bumping fog-kubevirt

@masayag
Copy link
Contributor

masayag commented Apr 20, 2020

Hi @agrare

I think that further changes are required: The namespace was changed as well when we fog-kubevirt was bumped to support fog-code > 2.0

fog/fog-kubevirt#23

@agrare
Copy link
Member Author

agrare commented Apr 20, 2020

👍 thanks yeah just saw the test failures

Trying to get on a newer version of kubeclient which is what prompted this

@miq-bot
Copy link
Member

miq-bot commented Apr 20, 2020

Checked commit agrare@7801837 with ruby 2.5.7, rubocop 0.69.0, haml-lint 0.28.0, and yamllint
2 files checked, 1 offense detected

app/models/manageiq/providers/kubevirt/infra_manager/connection.rb

@masayag masayag merged commit 91ae48c into ManageIQ:master Apr 20, 2020
@agrare
Copy link
Member Author

agrare commented Apr 20, 2020

Thanks @masayag !

@agrare agrare deleted the bump_fog_kubevirt branch April 20, 2020 16:19
@agrare
Copy link
Member Author

agrare commented Jun 9, 2020

Backport depends on ManageIQ/manageiq-providers-kubernetes#368

simaishi pushed a commit that referenced this pull request Jun 10, 2020
Update fog-kubevirt to newer version

(cherry picked from commit 91ae48c)
@simaishi
Copy link

Jansa backport details:

$ git log -1
commit 45d4b13928e9c9b21ab315e38a74058a8c8f3f3b
Author: Moti Asayag <masayag@redhat.com>
Date:   Mon Apr 20 19:19:22 2020 +0300

    Merge pull request #160 from agrare/bump_fog_kubevirt

    Update fog-kubevirt to newer version

    (cherry picked from commit 91ae48caf10794864ba79642c888642e3310a581)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants