| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-12-16 | |||
| 15:40:15 | efried | stephenfin: that's a trivial fup you requested, probably fast-approvable ^ | |
| 15:44:45 | gibi | efried: this is my last day in the office | |
| 15:45:34 | gibi | efried: I will try to still look at my qos patches during this week | |
| 15:45:54 | efried | gibi: okay, cool. If you have time, I think you might find https://review.opendev.org/#/c/699050/ interesting. But no big deal if you don't get to it. (Uses placement 1.35 to do root_required, and puts a request-level params object next to RequestSpec.requested_resources) | |
| 15:46:12 | gibi | efried: ack, added to the queue | |
| 15:52:49 | KeithMnemonic | mriedem: do you mean a test here? https://github.com/openstack/python-openstackclient/tree/master/openstackclient/tests/functional/compute/v2 or unit? or both? this is new to me | |
| 15:54:27 | mriedem | KeithMnemonic: functional would be ideal since it's easy to have false positive unit tests in osc | |
| 15:55:05 | KeithMnemonic | thank you | |
| 15:55:37 | mriedem | i.e. a functional test that does something like, (1) creates a server and then deletes it, (2) tries to list using the marker being the server name with --deleted and assert it fails with server not found, and then (3) lists servers with marker being the server.id + --deleted and show it works | |
| 16:25:18 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add negative test to delete server during cross-cell resize claim https://review.opendev.org/688832 | |
| 16:25:18 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix accumulated non-docs nits for cross-cell-resize series https://review.opendev.org/698935 | |
| 16:25:19 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Plumb graceful_exit through to EventReporter https://review.opendev.org/699237 | |
| 16:25:19 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Implement reschedule logic for cross-cell resize/migrate https://review.opendev.org/696213 | |
| 16:25:20 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Use graceful_exit=True in ComputeTaskManager.revert_snapshot_based_resize https://review.opendev.org/699238 | |
| 16:29:13 | mriedem | #success cross-cell resize/cold migrate is on its way to the gate | |
| 16:29:15 | openstackstatus | mriedem: Added success to Success page (https://wiki.openstack.org/wiki/Successes) | |
| 16:29:43 | mriedem | gibi: stephenfin: efried_pto: dansmith: et al ^ thanks for making it happen | |
| 16:34:38 | gibi | mriedem: wooot! | |
| 16:38:59 | gibi | mriedem: thanks for pushing the feauter forward | |
| 17:09:36 | gibi_pto | mriedem: I hope your ghost will linger around on IRC still in next year. But if not then good luck and hapy hacking! | |
| 17:09:55 | gibi_pto | and thanks for all the fish | |
| 17:14:09 | mriedem | gibi_pto: i'm sure i'll do some haunting, have a good break | |
| 17:53:26 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add negative test to delete server during cross-cell resize claim https://review.opendev.org/688832 | |
| 17:53:27 | openstackgerrit | Matt Riedemann proposed openstack/nova master: FUP for docs nits in cross-cell-resize series https://review.opendev.org/699259 | |
| 17:53:27 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Implement reschedule logic for cross-cell resize/migrate https://review.opendev.org/696213 | |
| 18:48:40 | openstackgerrit | Merged openstack/nova master: DRY: Build ImageMetaPropsPayload from ImageMetaProps https://review.opendev.org/698803 | |
| 19:36:13 | KeithMnemonic | mriedem ok i addressed your comments please let me know what you think https://review.opendev.org/#/c/691045/ | |
| 19:36:29 | KeithMnemonic | melwitt: if you care to review as well that would be great | |
| 21:18:44 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Ensure source compute is up when confirming a resize https://review.opendev.org/699291 | |
| 21:18:44 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add recreate test for bug 1855927 https://review.opendev.org/699045 | |
| 21:18:45 | openstack | bug 1855927 in OpenStack Compute (nova) "_poll_unconfirmed_resizes may not retry later if confirm_resize fails in API" [Low,Confirmed] https://launchpad.net/bugs/1855927 | |
| 22:01:08 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Ensure source compute is up when confirming a resize https://review.opendev.org/699291 | |
| 22:01:08 | openstack | bug 1855927 in OpenStack Compute (nova) "_poll_unconfirmed_resizes may not retry later if confirm_resize fails in API" [Low,In progress] https://launchpad.net/bugs/1855927 - Assigned to Matt Riedemann (mriedem) | |
| 22:01:08 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add recreate test for bug 1855927 https://review.opendev.org/699045 | |
| 22:03:49 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Ensure source compute is up when confirming a resize https://review.opendev.org/699291 | |
| 22:03:49 | openstack | bug 1855927 in OpenStack Compute (nova) "_poll_unconfirmed_resizes may not retry later if confirm_resize fails in API" [Low,In progress] https://launchpad.net/bugs/1855927 - Assigned to Matt Riedemann (mriedem) | |
| 22:03:49 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add recreate test for bug 1855927 https://review.opendev.org/699045 | |
| #openstack-nova - 2019-12-17 | |||
| 07:11:43 | openstack | bug 1550919 in OpenStack Compute (nova) "[Libvirt]Evacuate fail may cause disk image be deleted" [Medium,In progress] https://launchpad.net/bugs/1550919 - Assigned to Matthew Booth (mbooth-9) | |
| 07:11:43 | openstackgerrit | Ning Yao proposed openstack/nova master: Add functional test for bug 1550919 https://review.opendev.org/631294 | |
| 08:09:00 | openstackgerrit | Boxiang Zhu proposed openstack/nova master: Make evacuation respects anti-affinity rule https://review.opendev.org/649963 | |
| 08:29:05 | openstackgerrit | Boxiang Zhu proposed openstack/nova master: Fix live migration break group policy simultaneously https://review.opendev.org/651969 | |
| 14:28:12 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Ensure source compute is up when confirming a resize https://review.opendev.org/699291 | |
| 15:12:20 | mriedem | KeithMnemonic: a few things in your osc patch https://review.opendev.org/#/c/691045/ | |
| 15:20:11 | KeithMnemonic | thanks mriedem, did you want that full context added to the commit message | |
| 15:21:28 | mriedem | up to you | |
| 15:22:16 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Use graceful_exit=True in ComputeTaskManager.revert_snapshot_based_resize https://review.opendev.org/699238 | |
| 15:22:16 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Plumb graceful_exit through to EventReporter https://review.opendev.org/699237 | |
| 15:22:17 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add negative test to delete server during cross-cell resize claim https://review.opendev.org/688832 | |
| 15:22:17 | openstackgerrit | Matt Riedemann proposed openstack/nova master: FUP for docs nits in cross-cell-resize series https://review.opendev.org/699259 | |
| 15:22:18 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Implement reschedule logic for cross-cell resize/migrate https://review.opendev.org/696213 | |
| 15:29:46 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Use COMPUTE_SAME_HOST_COLD_MIGRATE trait during migrate https://review.opendev.org/695220 | |
| 15:37:53 | KeithMnemonic | mriedem, i will make all of those. the only one i can not is line 81 in the test, it is as you said due to the order of how the instances are launched, the first one actually is the second one in the list once both are ACTIVE | |
| 15:38:14 | KeithMnemonic | i saw that in testing when i first tried id1 | |
| 15:44:18 | mriedem | i wasn't asking for a change in the test, just making a note to myself | |
| 15:59:07 | mriedem | melwitt: i dumped thoughts in https://review.opendev.org/#/c/678100/ about splitting that up | |
| 16:11:38 | stephenfin | mriedem: You've probably seen this already, but it looks like grenade is properly broken. Ansible version mismatches | |
| 16:13:57 | mriedem | ERROR: ara 0.16.6 has requirement ansible>=2.6.0, but you'll have ansible 2.5.7 which is incompatible. | |
| 16:13:57 | mriedem | nope i hadn't noticed but now i see | |
| 16:15:17 | stephenfin | Yup. I'm not sure where it's coming from. Clearly not us | |
| 16:15:50 | mriedem | fungi: known issue ^? | |
| 16:16:26 | fungi | weird. ansible 2.5 is well past eol and i didn't think we had any jobs using it | |
| 16:16:44 | mriedem | seeing it here https://zuul.opendev.org/t/openstack/build/ce17ff84afb04b339cb98917c3625ced/log/job-output.txt#2225 | |
| 16:16:51 | fungi | zuul is about to completely drop support for ansible 2.5 as well (just saw the change for that approved) | |
| 16:17:28 | mriedem | hmm, grenade jobs are not happy: | |
| 16:17:31 | mriedem | http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22ERROR%3A%20ara%200.16.6%20has%20requirement%20ansible%3E%3D2.6.0%2C%20but%20you'll%20have%20ansible%202.5.7%20which%20is%20incompatible.%5C%22%20AND%20tags%3A%5C%22console%5C%22&from=7d | |
| 16:18:02 | fungi | https://zuul.opendev.org/t/openstack/build/ce17ff84afb04b339cb98917c3625ced/log/job-output.txt#2052 | |
| 16:18:15 | fungi | "Collecting ansible==2.5.7" | |
| 16:22:03 | stephenfin | jackpot | |
| 16:22:09 | stephenfin | devstack-vm-gate-wrap.sh | |
| 16:22:14 | stephenfin | whoops | |
| 16:22:16 | stephenfin | 31:ANSIBLE_VERSION=${ANSIBLE_VERSION:-2.5. | |
| 16:22:16 | stephenfin | devstack-vm-gate-wrap.sh | |
| 16:22:16 | stephenfin | $ ag ANSIBLE_VERSION | |
| 16:22:22 | stephenfin | 7} | |
| 16:22:47 | stephenfin | fungi: I wonder what should that be set to now? | |
| 16:25:01 | fungi | oof | |
| 16:25:21 | mriedem | 2.8.7 is what tempest-integrated-compute is using | |
| 16:25:24 | fungi | it's too bad our cycle goal to move off devstack-gate isn't already done | |
| 16:25:27 | mriedem | https://zuul.opendev.org/t/openstack/build/e0aa96296eea4930977c80cf599bc6c5/log/job-output.txt#33 | |
| 16:25:57 | mriedem | devstack-gate is branchless so i'm not sure if that will cause issues, but we could control that variable via the grenade jobs maybe | |
| 16:26:07 | fungi | note that devstack-gate is branchless, but maybe grenade can override $ANSIBLE_VERSION | |
| 16:26:13 | mriedem | jinx | |
| 16:26:32 | fungi | indeed ;) | |
| 16:26:41 | mriedem | i'll push a patch to play with it | |
| 16:26:44 | fungi | worth bringing up with the qa team too | |
| 16:26:54 | stephenfin | mriedem: I was just about to ask :) | |
| 16:27:15 | fungi | infra has been pushing to drop devstack-gate for a couple years already, so really doesn't want to have to make new changes to it | |
| 16:28:59 | openstack | Launchpad bug 1856732 in grenade "ERROR: ara 0.16.6 has requirement ansible>=2.6.0, but you'll have ansible 2.5.7 which is incompatible." [Undecided,New] | |
| 16:28:59 | mriedem | ok starting with https://bugs.launchpad.net/grenade/+bug/1856732 to at least track it | |
| 17:02:50 | mriedem | stephenfin: i don't think that's a blocker, it's a red herring, | |
| 17:02:57 | mriedem | https://zuul.opendev.org/t/openstack/build/ce17ff84afb04b339cb98917c3625ced/log/logs/devstack-gate-setup-host.txt#1847 | |
| 17:02:57 | mriedem | in the case above the job failed because of a network issue | |
| 17:03:25 | mriedem | stephenfin: which explains why this isn't a 100% failure and ara 0.16.6 was released in november - we would have noticed sooner if it was a blocker | |
| 17:03:37 | stephenfin | gdi | |
| 17:03:49 | stephenfin | makes sense | |
| 17:05:19 | mriedem | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_ed7/638047/56/check/nova-grenade-multinode/ed76ba4/logs/ara/index.html.gz | |
| 17:05:19 | mriedem | doesn't look like it | |
| 17:05:40 | mriedem | fungi: i thought ara was replaced with the zuul dashboard console stuff? | |
| 17:06:12 | mriedem | yeah the tempest jobs don't even have ara it looks like | |
| 17:07:00 | stephenfin | damn :( | |