| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-10-27 | |||
| 10:00:41 | openstackgerrit | Ameed Ashour proposed openstack/nova master: change instance_system_metadata column type https://review.opendev.org/526900 | |
| 13:08:10 | openstackgerrit | Ameed Ashour proposed openstack/nova master: change instance_system_metadata column type https://review.opendev.org/526900 | |
| #openstack-nova - 2019-10-28 | |||
| 01:26:31 | openstackgerrit | Merged openstack/os-traits master: Stop testing with py2 and switch to ussuri jobs https://review.opendev.org/690997 | |
| 01:28:07 | openstackgerrit | Merged openstack/os-resource-classes master: Stop testing with py2 https://review.opendev.org/690996 | |
| 08:32:21 | bauzas | good morning Nov | |
| 08:32:23 | bauzas | Nova* | |
| 08:37:58 | openstackgerrit | sunjiyun-inspur proposed openstack/nova master: On branch vm-query-notsameproject https://review.opendev.org/691629 | |
| 09:35:55 | openstackgerrit | Kobi Samoray proposed openstack/nova master: Avoid fetching metadata when no subnets found https://review.opendev.org/679247 | |
| 09:50:31 | dpawlik | Good morning. If you have some free time, pls check https://review.opendev.org/#/c/690482/ . Thanks! | |
| 10:07:36 | openstackgerrit | Elod Illes proposed openstack/nova stable/queens: cleanup evacuated instances not on hypervisor https://review.opendev.org/687873 | |
| 10:08:31 | openstackgerrit | Silvan Kaiser proposed openstack/nova master: [WIP] Move Nova Quobyte driver to LibvirtMountedFileSystemVolumeDriver https://review.opendev.org/687066 | |
| 10:10:31 | openstackgerrit | Qiu Fossen proposed openstack/nova-specs master: support instance tag fuzzy query https://review.opendev.org/691651 | |
| 10:13:59 | kaisers | mdbooth: I've a change in flight to move the Quobyte driver to LibvirtMountedFileSystemVolumeDriver (https://review.opendev.org/#/c/687066/). It's still wip but I't be great if you could have a look if the approach for the systemd related mount in there looks sane to you. | |
| 10:35:56 | mdbooth | kaisers: Sure. Will try to look later today. | |
| 11:15:59 | openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Support re-configure deleted_on_termination in server https://review.opendev.org/580336 | |
| 11:16:41 | openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Add flavor group https://review.opendev.org/663563 | |
| 11:17:19 | openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Allow specify user to reset password https://review.opendev.org/682302 | |
| 11:37:26 | brinzhang | Hi, Shanghai expo center in "1500 Shi Bo Da Dao (世博大道1500号) Pudong Shanghai Shanghai"? The submmit venue | |
| 11:38:10 | brinzhang | anyone one can tell me the accurate location? thanks | |
| 12:30:14 | openstackgerrit | Elod Illes proposed openstack/nova stable/queens: Add functional test for resize crash compute restart revert https://review.opendev.org/687874 | |
| 12:52:26 | openstackgerrit | Elod Illes proposed openstack/nova stable/queens: Prevent init_host test to interfere with other tests https://review.opendev.org/687876 | |
| 13:09:09 | openstackgerrit | Elod Illes proposed openstack/nova stable/queens: Functional reproduce for bug 1833581 https://review.opendev.org/687877 | |
| 13:09:09 | openstack | bug 1833581 in OpenStack Compute (nova) queens "instance stuck in BUILD state if nova-compute is restarted" [Low,In progress] https://launchpad.net/bugs/1833581 - Assigned to Balazs Gibizer (balazs-gibizer) | |
| 13:24:35 | openstackgerrit | Elod Illes proposed openstack/nova stable/queens: Error out interrupted builds https://review.opendev.org/687878 | |
| 13:26:46 | mriedem | lyarwood: can you hit this train backport and then i'll queue up a train release https://review.opendev.org/#/c/691402/ | |
| 13:28:37 | lyarwood | looking | |
| 13:29:47 | lyarwood | done | |
| 13:29:49 | lyarwood | LGTM | |
| 13:30:16 | mriedem | thanks | |
| 13:55:49 | mriedem | huh, apparently our api db in functional tests leaks across tests | |
| 13:55:59 | mriedem | tox -e functional-py36 -- "test_list_deleted_servers_with_marker|test_concurrent_request_server_group_members_over_quota" | |
| 13:56:13 | mriedem | err, tox -e functional-py36 -- --serial "test_concurrent_request_server_group_members_over_quota|test_list_deleted_servers_with_marker" | |
| 14:15:59 | openstackgerrit | Elod Illes proposed openstack/nova stable/queens: cleanup evacuated instances not on hypervisor https://review.opendev.org/687873 | |
| 14:17:18 | openstackgerrit | Elod Illes proposed openstack/nova stable/queens: Add functional test for resize crash compute restart revert https://review.opendev.org/687874 | |
| 14:17:19 | openstackgerrit | Elod Illes proposed openstack/nova stable/queens: Prevent init_host test to interfere with other tests https://review.opendev.org/687876 | |
| 14:19:52 | openstackgerrit | Elod Illes proposed openstack/nova stable/queens: Functional reproduce for bug 1833581 https://review.opendev.org/687877 | |
| 14:19:52 | openstack | bug 1833581 in OpenStack Compute (nova) queens "instance stuck in BUILD state if nova-compute is restarted" [Low,In progress] https://launchpad.net/bugs/1833581 - Assigned to Elod Illes (elod-illes) | |
| 14:21:08 | openstackgerrit | Elod Illes proposed openstack/nova stable/queens: Error out interrupted builds https://review.opendev.org/687878 | |
| 14:34:19 | openstackgerrit | Dan Smith proposed openstack/nova master: Add Aggregate image caching progress notifications https://review.opendev.org/691390 | |
| 14:49:56 | gibi | dansmith: quick comment in https://review.opendev.org/#/c/691390/4/nova/tests/functional/compute/test_cache_image.py@77 | |
| 14:50:10 | dansmith | cripes | |
| 14:50:26 | dansmith | it's still passing because of what I mentioned before, but... yep, oops :D | |
| 14:50:43 | openstackgerrit | Dan Smith proposed openstack/nova master: Add Aggregate image caching progress notifications https://review.opendev.org/691390 | |
| 14:50:43 | gibi | yeah it is passing, you proved your point :) | |
| 14:50:47 | dansmith | haha | |
| 14:50:57 | dansmith | gibi: thanks | |
| 14:51:25 | gibi | dansmith: I'm +2 now | |
| 14:51:34 | dansmith | thanks! | |
| 14:51:39 | gibi | dansmith: thank you for not waiting for me to write these patches :) | |
| 14:53:19 | dansmith | heh, well, this was definitely a learning moment for me.. I (like many) have just depended on you to do all of this in the past, | |
| 14:53:26 | dansmith | so "glad" to have the opportunity | |
| 14:53:51 | gibi | I'm glad that I'm not alone in this boat any more :) | |
| 14:54:20 | mriedem | gmann: i noticed that the tempest-integrated-compute job is running volume API tests - https://openstack.fortnebula.com:13808/v1/AUTH_e8fd161dc34c421a979a9e6421f823e9/zuul_opendev_logs_3dc/691129/2/gate/tempest-integrated-compute/3dcc0e0/testr_results.html.gz - is there a reason for that? those are cinder-only so i'm not sure why they are part of that job | |
| 14:55:19 | mriedem | it's also running glance and neutron api tests | |
| 15:01:43 | gibi | mriedem: I cannot reproduce your functional test api db leak locally | |
| 15:02:58 | gibi | mriedem: both test case order passes to me | |
| 15:03:23 | mriedem | gibi: yeah same on master, train and stein but 100% reproducible on rocky with https://review.opendev.org/#/c/690720/ | |
| 15:03:28 | mriedem | i'm not sure what changed in stein | |
| 15:03:39 | gibi | mriedem: ohh, then I will check on rocky | |
| 15:05:12 | mriedem | it looks like the leaked instances are coming from cell0 | |
| 15:05:13 | mriedem | which is odd | |
| 15:36:50 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Add functional regression test for bug 1849409 https://review.opendev.org/690720 | |
| 15:36:50 | openstack | bug 1849409 in OpenStack Compute (nova) rocky "openstack server list --deleted --limit -1 hangs" [High,In progress] https://launchpad.net/bugs/1849409 - Assigned to Matt Riedemann (mriedem) | |
| 15:36:50 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Fix listing deleted servers with a marker https://review.opendev.org/690721 | |
| 15:37:20 | mriedem | i just worked around it, can't figure it out, tried a bunch of different thigns | |
| 15:37:22 | mriedem | *things | |
| 15:38:35 | openstackgerrit | Merged openstack/nova master: Added openssh-client into bindep https://review.opendev.org/690482 | |
| 15:40:09 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add new policy rule for viewing host status UNKNOWN https://review.opendev.org/679181 | |
| 15:40:09 | smcginnis | mriedem: Did you have a list of which of the outstanding patches were needed before switching nova stable/queens to EM? | |
| 15:41:38 | mriedem | i think i'm just trying to get https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:stable/queens+topic:bug/183358 , https://review.opendev.org/#/c/687058/, https://review.opendev.org/#/c/688090/ , | |
| 15:41:51 | mriedem | https://review.opendev.org/#/c/691283/ would be nice but has to be updated and land in stein first | |
| 15:42:01 | smcginnis | Dang, that's a lot. | |
| 15:42:27 | mriedem | gibi: how much do you care that https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:stable/queens+topic:bug/183358 gets released in the final queens release? | |
| 15:42:39 | mriedem | you're really targeting pike right? so maybe you don't care that those get into the final queens release | |
| 15:42:45 | gibi | mriedem, smcginnis: from my perspective https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:stable/queens+topic:bug/183358 is not super important in the last release as I need it in pike as well where there is no more releases | |
| 15:42:58 | mriedem | ok yeah so let's ignore those for queens final release | |
| 15:43:02 | smcginnis | Can still get merged, but just not able to be in an official release after the cutover. | |
| 15:43:04 | gibi | mriedem: agree | |
| 15:43:12 | smcginnis | Thanks gibi. | |
| 15:43:16 | gibi | smcginnis: sure | |
| 15:43:36 | mriedem | so https://review.opendev.org/#/c/688090/ just needs a +W | |
| 15:43:54 | smcginnis | Just reviewed it. | |
| 15:43:55 | mriedem | as does https://review.opendev.org/#/c/691282/ (stein) and then i can re-cherry-pick that to queens and approve it | |
| 15:43:57 | dansmith | mriedem: you need me to hit somequeens things? | |
| 15:44:06 | mriedem | sorting it out | |
| 15:46:04 | mriedem | lyarwood: do you want to take a pass at this queens one again? https://review.opendev.org/#/c/676285/ - i addressed the comment | |
| 15:47:01 | mriedem | beyond that i say we just punt on everything else for the final queens release | |
| 15:49:02 | mriedem | smcginnis: so i guess just these three and then we're ready https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:stable/queens+label:Code-Review=2 | |
| 15:49:43 | smcginnis | Awesome, thanks! | |
| 16:09:53 | gmann | mriedem: reason to run them was, current API tests are mixed with single API operation and multiple service API operation like - https://github.com/openstack/tempest/blob/master/tempest/api/volume/test_volumes_actions.py#L39 | |
| 16:10:21 | gmann | mriedem: so i thought of running them to avoid any coverage miss. glance API tests we can remove I think. | |
| 16:10:53 | mriedem | i would assume volume attach is covered in an existing compute api test | |
| 16:14:20 | gmann | this is example, either we have to go with case by case and take care that new tests etc are added in scenario only but that is difficult. | |
| 16:14:31 | mriedem | yeah i understand | |
| 16:14:58 | gmann | i can try to run with related tests where 'compute' service is added for non-compute folder tests (hoping we tag them correctly ) | |
| 16:15:42 | gmann | that way we can run only compute related test only | |
| 16:38:14 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Add functional regression test for bug 1849409 https://review.opendev.org/690720 | |
| 16:38:14 | openstack | bug 1849409 in OpenStack Compute (nova) rocky "openstack server list --deleted --limit -1 hangs" [High,In progress] https://launchpad.net/bugs/1849409 - Assigned to Matt Riedemann (mriedem) | |
| 16:38:14 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Fix listing deleted servers with a marker https://review.opendev.org/690721 | |
| 18:16:56 | openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: add troubleshooting guide for cleaning up orphaned allocations https://review.opendev.org/691427 | |