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-openstack to v1.0 #734

Merged
merged 10 commits into from
May 8, 2023
Merged

Conversation

agrare
Copy link
Member

@agrare agrare commented Sep 28, 2021

@agrare
Copy link
Member Author

agrare commented Sep 28, 2021

@miq-bot cross-repo-tests ManageIQ/manageiq-gems-pending#525

@agrare
Copy link
Member Author

agrare commented Sep 28, 2021

Getting some VCR refresher errors that I'll need to work out the root cause of

@kbrock
Copy link
Member

kbrock commented May 5, 2022

kicking

@kbrock kbrock closed this May 5, 2022
@kbrock kbrock reopened this May 5, 2022
@agrare agrare force-pushed the update_fog_openstack_v1 branch 2 times, most recently from 8ce0e0d to 5418158 Compare August 29, 2022 16:52
@agrare
Copy link
Member Author

agrare commented Aug 29, 2022

Only failing on the ironic/infra_manager specs because microstack doesn't have that service enabled

@miq-bot
Copy link
Member

miq-bot commented Oct 12, 2022

This pull request is not mergeable. Please rebase and repush.

@miq-bot
Copy link
Member

miq-bot commented Feb 27, 2023

This pull request has been automatically closed because it has not been updated for at least 3 months.

Feel free to reopen this pull request if these changes are still valid.

Thank you for all your contributions! More information about the ManageIQ triage process can be found in the triage process documentation.

@miq-bot miq-bot closed this Feb 27, 2023
@agrare agrare reopened this May 2, 2023
@agrare
Copy link
Member Author

agrare commented May 2, 2023

@miq-bot cross-repo-tests ManageIQ/manageiq-gems-pending#558

@agrare
Copy link
Member Author

agrare commented May 4, 2023

Cross-repo tests are green

@nasark
Copy link
Member

nasark commented May 4, 2023

Just wanted to add that fog v1.0 will allow us to support cinder V3 which is used in upstream Openstack Wallaby> releases

@agrare agrare changed the title [WIP] Update fog-openstack to v1.0 Update fog-openstack to v1.0 May 4, 2023
@agrare
Copy link
Member Author

agrare commented May 5, 2023

Hey @kbrock can you take a look?

@kbrock
Copy link
Member

kbrock commented May 8, 2023

Are we concerned with:

Not sure if Fog::{Compute,Network,Identity} still works the same or if we need to add OpenStack in there as well.

@kbrock
Copy link
Member

kbrock commented May 8, 2023

Also, could you update this comment with 2 references to the 2 classes?
Not sure if we want to verify/update that comment now that we are bumping fog by so much.

https://github.com/ManageIQ/manageiq-providers-openstack/blob/master/app/models/manageiq/providers/openstack/inventory/parser/cloud_manager.rb#L162

@agrare
Copy link
Member Author

agrare commented May 8, 2023

Not sure if Fog::{Compute,Network,Identity} still works the same or if we need to add OpenStack in there as well.

These do still work but can update for consistency

@agrare
Copy link
Member Author

agrare commented May 8, 2023

Also, could you update this comment with 2 references to the 2 classes?

Done

@miq-bot
Copy link
Member

miq-bot commented May 8, 2023

Some comments on commits agrare/manageiq-providers-openstack@d066587~...94b6b2b

spec/models/manageiq/providers/openstack/cloud_manager/refresh_spec_common.rb

  • ⚠️ - 32 - Detected allow_any_instance_of. This RSpec method is highly discouraged, please only use when absolutely necessary.
  • ⚠️ - 33 - Detected allow_any_instance_of. This RSpec method is highly discouraged, please only use when absolutely necessary.
  • ⚠️ - 34 - Detected allow_any_instance_of. This RSpec method is highly discouraged, please only use when absolutely necessary.
  • ⚠️ - 35 - Detected allow_any_instance_of. This RSpec method is highly discouraged, please only use when absolutely necessary.
  • ⚠️ - 38 - Detected allow_any_instance_of. This RSpec method is highly discouraged, please only use when absolutely necessary.
  • ⚠️ - 41 - Detected allow_any_instance_of. This RSpec method is highly discouraged, please only use when absolutely necessary.
  • ⚠️ - 42 - Detected allow_any_instance_of. This RSpec method is highly discouraged, please only use when absolutely necessary.
  • ⚠️ - 43 - Detected allow_any_instance_of. This RSpec method is highly discouraged, please only use when absolutely necessary.
  • ⚠️ - 44 - Detected allow_any_instance_of. This RSpec method is highly discouraged, please only use when absolutely necessary.
  • ⚠️ - 46 - Detected allow_any_instance_of. This RSpec method is highly discouraged, please only use when absolutely necessary.
  • ⚠️ - 47 - Detected allow_any_instance_of. This RSpec method is highly discouraged, please only use when absolutely necessary.
  • ⚠️ - 48 - Detected allow_any_instance_of. This RSpec method is highly discouraged, please only use when absolutely necessary.
  • ⚠️ - 51 - Detected allow_any_instance_of. This RSpec method is highly discouraged, please only use when absolutely necessary.
  • ⚠️ - 52 - Detected allow_any_instance_of. This RSpec method is highly discouraged, please only use when absolutely necessary.

@miq-bot
Copy link
Member

miq-bot commented May 8, 2023

Checked commits agrare/manageiq-providers-openstack@d066587~...94b6b2b with ruby 2.6.10, rubocop 1.28.2, haml-lint 0.35.0, and yamllint
24 files checked, 3 offenses detected

spec/models/manageiq/providers/openstack/infra_manager/refresher_rhos_juno_spec.rb

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.

4 participants