| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-27 | |||
| 03:16:56 | openstackgerrit | Merged openstack/nova master: add a retry on DBDeadlock to _set_allocations() https://review.openstack.org/487483 | |
| 04:22:48 | openstackgerrit | yuanyue proposed openstack/nova master: Add a periodic task to destroy ReqSpecs of deleted instances https://review.openstack.org/484694 | |
| 04:55:39 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | |
| 04:58:05 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | |
| 04:58:31 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | |
| 04:58:42 | openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.54 - Enable cold migration with target host https://review.openstack.org/406707 | |
| 04:59:05 | openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.54 - List/Show all server migration types https://review.openstack.org/430839 | |
| 06:11:51 | openstackgerrit | NidhiMittalHada proposed openstack/python-novaclient master: Help text for "--matching" is not clear. https://review.openstack.org/487708 | |
| 06:29:05 | openstackgerrit | jichenjc proposed openstack/nova master: add cli to support matrix https://review.openstack.org/481498 | |
| 06:30:24 | openstackgerrit | jichenjc proposed openstack/nova master: Enhance support matrix document https://review.openstack.org/482013 | |
| 06:31:09 | openstackgerrit | jichenjc proposed openstack/nova master: Enhance support matrix document https://review.openstack.org/482020 | |
| 06:31:55 | openstackgerrit | jichenjc proposed openstack/nova master: Add quiesce and unquiesce in support matrix https://review.openstack.org/482390 | |
| 06:33:05 | openstackgerrit | jichenjc proposed openstack/nova master: Add device tag support info in support matrix https://review.openstack.org/481478 | |
| 06:33:05 | openstackgerrit | jichenjc proposed openstack/nova master: Add description for tag support in attach https://review.openstack.org/481479 | |
| 06:40:01 | openstackgerrit | jichenjc proposed openstack/nova master: WIP:add cert when send keystone request https://review.openstack.org/485121 | |
| 06:48:08 | openstackgerrit | jichenjc proposed openstack/nova master: refactory some code on get_password https://review.openstack.org/449458 | |
| 06:52:45 | openstackgerrit | LIU Yulong proposed openstack/nova master: Enable reset keypair while rebuilding instance https://review.openstack.org/379128 | |
| 07:13:56 | openstackgerrit | Merged openstack/python-novaclient master: Removed extra word 'method' from the NOTE https://review.openstack.org/480988 | |
| 07:40:17 | gibi | bauzas: hi! good morning! There is a bugfix that only need a second +2 https://review.openstack.org/#/c/475276/ Could you check it if your time allows? | |
| 08:06:38 | bauzas | gibi: roger, roger. | |
| 08:15:54 | gibi | bauzas: thanks | |
| 08:25:58 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Send soft_delete from context manager https://review.openstack.org/476459 | |
| 08:25:59 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: use context mgr in instance.delete https://review.openstack.org/443764 | |
| 08:26:01 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform missing delete notifications https://review.openstack.org/410297 | |
| 08:43:13 | openstackgerrit | jichenjc proposed openstack/nova master: refactory some code on get_password https://review.openstack.org/449458 | |
| 08:44:56 | mamandle | alex_xu: bauzas: sfinucan: mriedem: please take a look at https://review.openstack.org/#/c/483911/4, comments addressed. | |
| 08:51:16 | bauzas | mamandle: given it's a bugfix and since today is pike-3 milestone day (so FeatureFreeze), I'll prioritize blueprints | |
| 08:51:35 | bauzas | mamandle: that said, just ping me next week and I'll look at it, for sure :) | |
| 08:51:56 | mamandle | bauzas: sure, thanks :) | |
| 09:03:44 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: stabilize test_create_delete_server functional test https://review.openstack.org/487772 | |
| 09:15:58 | vdrok | good morning folks! | |
| 09:24:16 | takashin | bauzas: Please review https://review.openstack.org/#/c/408955/. Your comments were addressed. | |
| 09:27:35 | bauzas | takashin: yup, I saw your update | |
| 09:28:23 | openstackgerrit | jichenjc proposed openstack/nova master: add cert when send keystone request https://review.openstack.org/485121 | |
| 09:28:28 | bauzas | takashin: I knew you already provided your spec since 2 cycles | |
| 09:28:41 | bauzas | takashin: unfortunately, I didn't have time to review it until last week | |
| 09:28:48 | bauzas | so it's my bad | |
| 09:29:34 | bauzas | takashin: but now we are very close to the FF, would you accept that we could fast-merging that spec for Queens and ask me to be my main priority review for Q ? | |
| 09:30:05 | bauzas | takashin: I really think you worked fine, and adding a new 'forced_destination' seems good with me | |
| 09:30:27 | takashin | bauzas: I want to get it merged by Pike release. | |
| 09:30:41 | bauzas | tbh, if we would even add it for the beginning of Queens, I could add another spec for using it for the nova boot call | |
| 09:31:09 | bauzas | takashin: the problem is that we have 3 changes, one internal, one for the REST API and the last for the python novaclient | |
| 09:31:20 | bauzas | and we only have like 6 hours for that | |
| 09:31:55 | bauzas | I totally understand you would be sad if we would push it to Queens, but the main problem we had was about not a lot of reviews | |
| 09:32:15 | bauzas | given we'll open Queens in two weeks, I do wonder if we could just push it for 2 weeks | |
| 09:32:28 | bauzas | the main problem is about having the feature not in Pike | |
| 10:02:46 | sdague | vdrok: how is the ironic patch going? | |
| 10:03:25 | vdrok | sdague: I have not yet had a chance to look, will do in ~half an hour | |
| 10:03:32 | alex_xu | bauzas: why we don't have API for delete the aggregates in the placement? I remember there are some discussion, but I forget the detail... | |
| 10:06:12 | sdague | johnthetubaguy: https://review.openstack.org/#/c/357726 - this patch is ready to go if we reno it | |
| 10:06:18 | sdague | can you rev for that? | |
| 10:30:28 | kiennt | sdague: Hi, i am working on Heat gate job for rolling upgrade testing. I already setup multinode grenade gate job but it doesn't have any tests for testing mixed version. | |
| 10:31:02 | kiennt | I find out that Nova already had rolling-upgrade tag, so I want to ask about the way Nova do the tests | |
| 10:31:08 | kiennt | thank you | |
| 10:32:53 | openstackgerrit | Sean Dague proposed openstack/nova master: Assume neutron port_binding extensions enabled https://review.openstack.org/357726 | |
| 10:32:54 | openstackgerrit | Sean Dague proposed openstack/nova master: Use admin context to create neutron ports https://review.openstack.org/357540 | |
| 10:33:44 | sdague | kiennt: it's all about what makes sense for heat and it's collection of services. How would you expect heat would be upgraded across a set of nodes during upgrade? | |
| 10:34:13 | sdague | johnthetubaguy - ok, reno added for - https://review.openstack.org/#/c/357726/ | |
| 10:35:38 | andreaf | sdague: around? is there a flag in d-g or grenade that decides which host is upgraded in a multinode setup | |
| 10:36:05 | sdague | andreaf: no, it's baked in logic | |
| 10:36:13 | sdague | the primary node is what is upgraded | |
| 10:36:20 | sdague | subnodes are not | |
| 10:37:07 | andreaf | sdague: ok that was my impression looking at devstack-vm-gate.sh, thanks | |
| 10:37:39 | andreaf | kiennt: ^^^ | |
| 10:37:58 | andreaf | sdague: so everything is setup on old first and then primary is upgraded to new right? | |
| 10:38:01 | vdrok | http://logs.openstack.org/65/487665/2/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-ubuntu-xenial/8cbf580/logs/screen-n-super-cond.txt.gz#_Jul_27_04_27_53_252817, which happens after select_destinations), but in the compute log there is no build_and_run_instance, don't see the 'Starting instnace...' | |
| 10:38:01 | vdrok | sdague: so as mriedem said, in that job I see that the conductor called into scheduler which selected a host http://logs.openstack.org/65/487665/2/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-ubuntu-xenial/8cbf580/logs/screen-n-sch.txt.gz#_Jul_27_04_27_52_998156, then conductor proceeded (as I see | |
| 10:39:37 | kiennt | sdague, andreaf: So, after upgrade, if I run some tests (ex.tempest), it will run in services (2 versions). That will test these mixed services could talk to each other or not. Is this right? | |
| 10:39:58 | andreaf | kiennt: yeah that's the idea | |
| 10:40:35 | openstackgerrit | Chris Dent proposed openstack/nova master: placement: account for move operations in claim https://review.openstack.org/487589 | |
| 10:42:06 | kiennt | andreaf, sdague: Thank you, guys. This is very important information with me. I have stucked at this for a long time, because i'm completely new with all these things. | |
| 10:42:53 | kiennt | Seem like I only need some tempest tests after upgrade now because I already have multinode gate. | |
| 10:43:26 | openstackgerrit | Artem Tiumentcev proposed openstack/python-novaclient master: Add 'local_gb' disk quota support https://review.openstack.org/484391 | |
| 10:43:46 | openstackgerrit | Chris Dent proposed openstack/nova master: claim resources in placement API during schedule() https://review.openstack.org/483566 | |
| 10:48:39 | sdague | andreaf: yes, primary node is installed, subnode is installed. Tests are run. Then primary node is upgraded. | |
| 10:49:43 | andreaf | sdague: oh I see, so we don't run any test after upgrade? on check resources? | |
| 10:49:47 | andreaf | s/on/only | |
| 10:50:02 | sdague | andreaf: sorry, no we run tests again at the end | |
| 10:50:25 | andreaf | sdague: ok thanks | |
| 10:50:41 | sdague | so, because this is an intergration between devstack-gate and grenade, I don't know that it's ever gotten documented very well | |
| 10:50:58 | sdague | I'd be happy to write up a webpage with what it is, if you figure out where you'd like it to live :) | |
| 10:51:28 | sdague | do we currently have overarching qa docs outside of tempest / grenade / devstack ones? | |
| 10:54:43 | vdrok | http://logs.openstack.org/65/487665/2/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-ubuntu-xenial/8cbf580/logs/etc/nova/nova.conf.txt.gz | |
| 10:54:43 | vdrok | sdague: just guessing, in the failing job I see transport_url with cell1 at http://logs.openstack.org/65/487665/2/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-ubuntu-xenial/8cbf580/logs/etc/nova/nova_cell1.conf.txt.gz but without cell1 at | |
| 10:56:29 | sdague | ok, we'll need dansmith I think. Exactly where the mismatches are here are beyond me. | |
| 10:59:41 | vdrok | same here :( | |
| 11:10:57 | sdague | vdrok: actually, I might have figured it out | |
| 11:11:03 | sdague | we were missing one last conditional | |
| 11:11:11 | sdague | https://review.openstack.org/487809 | |
| 11:12:36 | sdague | I changed the ironic patch to depend on that one | |
| 11:20:00 | takashin | oomichi: Are you around? | |
| 11:23:04 | vdrok | sdague: cool, thank you! | |
| 11:23:29 | sdague | vdrok: fingers crossed | |
| 11:24:09 | Ritesh | Hi, | |
| 11:24:15 | Ritesh | Can anyone please review this patch | |
| 11:24:28 | Ritesh | review pending since long time | |
| 11:24:30 | Ritesh | https://review.openstack.org/#/c/457886/ | |
| 11:32:48 | openstackgerrit | LIU Yulong proposed openstack/nova master: Enable reset keypair while rebuilding instance https://review.openstack.org/379128 | |
| 11:45:17 | vdrok | sdague: ouch http://logs.openstack.org/58/487458/3/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_snmp-tinyipa-ubuntu-xenial-nv/e4c3322/logs/screen-n-cpu.txt.gz#_Jul_27_11_32_29_018076 | |
| 11:47:10 | sdague | vdrok: ok, the conductor didn't get the right config file - http://logs.openstack.org/58/487458/3/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_snmp-tinyipa-ubuntu-xenial-nv/e4c3322/logs/screen-n-cond.txt.gz#_Jul_27_11_29_55_156482 | |