Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
Brett Thurber
@bthurber
cool
thanks!
Yadnyawalkya Tale
@Yadnyawalkya
@fdupont-redhat about https://gitter.im/ManageIQ/v2v?at=5bbdde4ac08b8b3067f8b1ea
so I can see comment on BZ https://bugzilla.redhat.com/show_bug.cgi?id=1634834#c4 
The backport of https://github.com/ManageIQ/manageiq-content/pull/417 is reverted.

This will now target 5.10 only as per Eng/PM decision. If we decide to fix this in 5.9.z in the future, please re-open this BZ.
so are we not going to fix this issue for 5.9.z? how migration will work if we don't add this changes in 5.9?
from the last issue we found on 5.9.5.1, we were missing this changes and hence we wanted it for 5.9.z too
@fdupont-redhat please confirm, and let me also know if I need create new BZ for it
Dennis Metzger
@dmetzger57
@Yadnyawalkya Are we now saying migrations are completely broken in 5.9.5 or is it still just “righrt size” changes (an RFE) are not in 5.9.5?
Yadnyawalkya Tale
@Yadnyawalkya
migration was completed broken last time, and as per what fabien said it was due to we forgot to backport 418 for 5.9
Dennis Metzger
@dmetzger57
@bthurber @fdupont-redhat are migrations completely broken without this RFE 5.9.5 BZ? Today is the 5.9.5 GA build and this is the first time I’m hearing 5.9.5 migrations are broken without this BZ
Yadnyawalkya Tale
@Yadnyawalkya
are we saying-> right size changes are in 417 and 418, so instead of adding 418 we revert 417 so migration should work as earlier at least for 5.9?
Brett Thurber
@bthurber
@Yadnyawalkya @dmetzger57 ManageIQ/manageiq-content#417 was unbackported for 5.9.5 and the agreement was this feature would land in 5.10.
Fabien Dupont
@fdupont-redhat
@bthurber @Yadnyawalkya @dmetzger57 The right-size feature implementation relies on ManageIQ/manageiq-content#418, which has been merged but the associated BZ (1594196) is targeted for 5.10. So it was decided to align ManageIQ/manageiq-content#417 on the same target, i.e. Hammer, and so it was un-backported for consistency.
Dennis Metzger
@dmetzger57
For my clarity, is known that migrations work with the “un-merge” changes, or will we not know until 5.9.5 is built and in QE?
Brett Thurber
@bthurber
@dmetzger57 we will need to QE it
Yadnyawalkya Tale
@Yadnyawalkya
Will check this thing in next 5.9 build and let you folks know if that un-backported thing works or not
@bthurber @fdupont-redhat ack, thanks for clearing this.
Yadnyawalkya Tale
@Yadnyawalkya
Update: running migrations on 5.10.0.19 (latest 5.10.z) and migration works very well with right size changes. @fdupont-redhat +1
Fabien Dupont
@fdupont-redhat
Thanks @Yadnyawalkya
Yadnyawalkya Tale
@Yadnyawalkya
Update: running migration on 5.9.5.2 and migration works well without right size changes
that means un-backporting worked, thanks @simaishi @fdupont-redhat +1
smallamp
@smallamp
@Yadnyawalkya that’s a relief.. thanks for the update
Aparna Karve
@AparnaKarve
Screen Shot 2018-10-17 at 7.41.31 AM.png
Kedar Kulkarni
@kedark3
@AparnaKarve @michaelkro @vconzola UI Question
why was the "Infrastructure Mapping Wizard" renamed to "Create Infrastructure Mapping"
?
just curios
Aparna Karve
@AparnaKarve
@kedark3 we did that to be able to distingush if the Wizard is in Create mode or Edit mode
in the Edit mode, you should see the title as “Edit Infrastructure Mapping"
Kedar Kulkarni
@kedark3
got it
thanks
will those changes be backported to 5.9 ? any idea?
cause I need to update automation to fit that
If not backported, I need to pick correct title based on version of appliance
Aparna Karve
@AparnaKarve
they wont be because we are not going to support Edit in 5.9
Michael Ro
@michaelkro
While we have your attention, as a heads up, we just merged a PR that adds the new Infra Mappings section
Kedar Kulkarni
@kedark3
got it
which section Michael
Michael Ro
@michaelkro
we moved the Infra Mappings ListView from the Overview page to its own page. There will be a new vert nav item
Kedar Kulkarni
@kedark3
it will be in next 5.10 build right?
Michael Ro
@michaelkro
correct
Kedar Kulkarni
@kedark3
got it thanks
Michael Ro
@michaelkro
well, it's not backported yet
but it will be there soon :)
Kedar Kulkarni
@kedark3
QE is right now swamped with testing, but will try to find time to update the code ... depending on how major/minor refactoring is needed
Michael Ro
@michaelkro
yes, I can imagine you guys are busy :) I just wanted to give you a heads up that the change is incoming
Kedar Kulkarni
@kedark3
that does help
thanks :)
Brett Thurber
@bthurber
@roliveri this PR wil need review and merge before the next alpha build tomorrow. ManageIQ/manageiq-content#441 Would you mind helping to locate a reviewer?