-
Notifications
You must be signed in to change notification settings - Fork 146
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
Full Support of fog-core version 2 #421
Comments
@gildub I'm very sorry but I'm not quite sure why 2.1.0 -> 2.1.2 upgrade is required. Can you help me understand this? You are saying 2+, yet we already are on 2.1.0 AFAIK you can use the new addresses (like Fog::Google::Storage, instead of Fog::Storage::Google) in 2.1.0 without issues - the only difference is that 2.1.2 has deprecation errors. If I'm mistaken however - please, let me know and I'll do my best to come up with something that works. |
@Temikus, that's right fog-google is already supporting up to fog-core 2.1.0 but not 2.1.2 yet. The need comes from having The reason for specifically 2.1.2 is because fog-openstack needs it to be able to use 2+. So if any other project lock to anything prior to the latter version will create a dependency blockage. Although the deprecation message was added later in fog-core 2.1.1, the namespace change is necessary to keep up and also make sense. I've updated PR#422 accordingly so fog-google is up to date with latest fog-core. |
@gildub Very sorry but I still do not understand the technical reason behind this. Can you help me here a bit? I'm asking since I'm hesitant exposing users to #417 and overhauling all tests, references, etc. without a major version bump for which we are not ready for yet. There should be no major API differences between minor versions (i.e. 2.1.0 and 2.1.2). I took a look into the diff and don't see anything particularly major either: fog/fog-core@v2.1.0...master So looking through that, fog-openstack should run just fine on fog-core 2.1.0. Or was there some major bug that I'm missing? |
|
Latest Net::SSH versions are not currently compatible with fog-core(2.1.0) @Temikus, any updates on this? |
@icco - Can you take a look at this by any chance? This is connected to #422 (comment) |
This issue has been marked inactive and will be closed if no further activity occurs. |
This issue is still needed as EPEL on EL8 includes a newer Net::SSH gem, and pinning the Net::SSH version causes other packages to be held at older versions as well. Specifically for me, this issue is holding back my deployment of Foreman/Katello on EL8. |
This issue has been marked inactive and will be closed if no further activity occurs. |
bumping again. Still needed. |
Hi @Temikus I am now using ruby --v '2.4.6' and rails '5.1.2' |
This issue has been marked inactive and will be closed if no further activity occurs. |
bumping again. Still needed. |
Is there a label to permanently ignore this one so we aren't spammed on a regular interval. |
Fog-core is already supported up to version 2.1.0.
ManageIQ/manageiq#17997
The text was updated successfully, but these errors were encountered: