| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-27 | |||
| 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 | |
| 11:47:20 | sdague | that's solvable | |
| 11:50:53 | sdague | vdrok: ok, lets try again | |
| 11:51:32 | vdrok | yup :) | |
| 11:54:02 | openstackgerrit | Yaroslav proposed openstack/nova master: Added Overbooking DB quota driver https://review.openstack.org/484359 | |
| 12:03:24 | sean-k-mooney | o/ would people object to adding a numa aware varient of reserved_host_memory_mb in queens. we have been seeing OOM errors on our ci systems pereodically for some time even when the system has several GB of free ram. | |
| 12:04:39 | sean-k-mooney | turns out that the OOM killer kicks in when any numa node is out of memory and hte kernel need to alloacte memory there not when the system is out of memory globally | |
| 12:14:18 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/477091 | |
| 12:21:28 | openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | |
| 12:24:08 | catintheroof | Hi, does anyone has a good guide on how to configure live migration on ocata ? doing the same that worked as of mitaka, doesnt anymore | |
| 12:27:46 | mriedem | oh hi everybody | |
| 12:28:09 | vdrok | sdague: gah, same thing, compute does not get the build instance request http://logs.openstack.org/58/487458/3/check/gate-tempest-dsvm-ironic-ipa-partition-redfish-tinyipa-ubuntu-xenial/7aa067c/logs/screen-n-cpu.txt.gz | |
| 12:28:15 | vdrok | morning mriedem | |
| 12:29:10 | openstackgerrit | LIU Yulong proposed openstack/nova master: Enable reset keypair while rebuilding instance https://review.openstack.org/379128 | |
| 12:29:39 | mriedem | vdrok: did my attempts at dumping the rabbitmqctl report ever work? | |
| 12:30:14 | vdrok | mriedem: seems like that file is not there http://logs.openstack.org/65/487665/2/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-ubuntu-xenial/8cbf580/logs/rabbitmq/ | |
| 12:30:26 | mriedem | yeah...i was getting permission denied errors | |