| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-27 | |||
| 15:55:04 | bauzas | so a backport looks good to me | |
| 15:55:36 | s-dean | ok well that table has data inside it, and references my compute node | |
| 15:57:20 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Allow tuple as for nics value https://review.openstack.org/475816 | |
| 15:58:15 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: doc: Remove Makefile https://review.openstack.org/480096 | |
| 16:01:44 | openstackgerrit | Ed Leafe proposed openstack/nova master: Handle hash ring rebalancing in ironic flavor migration https://review.openstack.org/487925 | |
| 16:02:05 | s-dean | does setting cell type in the nova.conf screw things up ? | |
| 16:02:39 | s-dean | is that option only for V1 | |
| 16:02:40 | mriedem | s-dean: cell type is only for cells v1 | |
| 16:02:45 | s-dean | right ok | |
| 16:02:56 | ildikov | mriedem: I know it's code freeze week, but can you come to the Cinder-Nova meeting? | |
| 16:03:04 | s-dean | thats my issue | |
| 16:03:44 | mriedem | s-dean: you shouldn't touch anything in the [cells] group in nova.conf | |
| 16:04:10 | s-dean | ok, man have cell disabled makes it sound like you should set it to enabled | |
| 16:04:21 | s-dean | but that activates cells v1 right | |
| 16:04:23 | mriedem | the naming is unfortunate and confusing yes | |
| 16:04:30 | s-dean | omg | |
| 16:04:33 | s-dean | alright | |
| 16:04:34 | mriedem | [cells]/enable=True is not what you want | |
| 16:04:43 | s-dean | hahha | |
| 16:04:59 | s-dean | ok | |
| 16:05:18 | s-dean | cheers for your help ill let you get back to your development | |
| 16:05:28 | s-dean | and go and reinstall again | |
| 16:05:31 | mriedem | s-dean: note https://github.com/openstack/nova/blob/master/nova/conf/cells.py#L24-L27 | |
| 16:05:31 | s-dean | for the 10th time | |
| 16:06:11 | mriedem | the config options and their help is also here https://docs.openstack.org/nova/ocata/sample_config.html | |
| 16:06:40 | s-dean | that link is for the Dev docs | |
| 16:06:42 | s-dean | right | |
| 16:08:11 | mriedem | jaypipes: have you checked the CI results on the claims in scheduler change yet? | |
| 16:08:16 | mriedem | how are we looking? | |
| 16:08:23 | mriedem | i'm nervous there is no chatter in here about that patch | |
| 16:09:11 | mriedem | vdrok: grenade failed http://logs.openstack.org/58/487458/5/check/gate-grenade-dsvm-ironic-ubuntu-xenial/f1fd08b/logs/grenade.sh.txt.gz#_2017-07-27_15_59_36_717 | |
| 16:10:14 | vdrok | yup, just commented on the patch. apart from the definition in upgrade/settings I also see this http://logs.openstack.org/58/487458/5/check/gate-grenade-dsvm-ironic-ubuntu-xenial/f1fd08b/logs/grenade.sh.txt.gz#_2017-07-27_15_58_24_805. guess I have to define it once again in the upgrade.sh? | |
| 16:13:58 | openstackgerrit | Doug Hellmann proposed openstack/nova master: WIP: add a redirect for the old cells landing page https://review.openstack.org/487932 | |
| 16:36:26 | openstackgerrit | Jackie Truong proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949 | |
| 16:41:10 | jaypipes | mriedem: sorry, was at lunch | |
| 16:41:16 | jaypipes | mriedem: checking now. | |
| 16:42:07 | jaypipes | mriedem: bottom patch has an unrelated failure: http://logs.openstack.org/89/487589/6/check/gate-nova-python35/ce8b77c/testr_results.html.gz | |
| 16:42:16 | jaypipes | mriedem: top patch (the claims one) has no failures. | |
| 16:42:42 | mriedem | yeah i've seen that weird rpc timeout one | |
| 16:49:00 | openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: add FAQ entry for cells v1 config options https://review.openstack.org/487938 | |
| 16:49:10 | mriedem | s-dean: ^ | |
| 16:50:29 | dansmith | mriedem: I was thinking of expanding the warning on my new cells doc to explain not only it not applying to cellsv1, but also that anything about the nova-cells service, the [cells] section of the config, etc | |
| 16:50:45 | dansmith | since "cellsv1" is not a neat box everyone understands | |
| 16:52:06 | mriedem | yeah there is a warning here https://docs.openstack.org/ocata/config-reference/compute/cells.html | |
| 16:52:13 | mriedem | but doesn't mention the config options, | |
| 16:52:18 | mriedem | or nova-cells servie | |
| 16:52:21 | mriedem | which are part of the confusion | |
| 16:53:08 | dansmith | aye | |
| 16:53:46 | sdague | vdrok: ok https://review.openstack.org/#/c/487809/ is +A | |
| 16:54:27 | sdague | the ironic patch is looking pretty good | |
| 16:54:40 | vdrok | sdague: thanks! | |
| 16:54:43 | sdague | vdrok: you should start lining up +2s on that one | |
| 16:55:06 | vdrok | will do as soon as I have a pass :) | |
| 16:58:34 | cfriesen_ | mriedem: thanks for fixing those nits on the unshelve. I'm a couple hours behind you. :) | |
| 16:59:18 | mriedem | np, thanks for fixing | |
| 17:02:19 | openstackgerrit | Merged openstack/nova master: Add 'updated_at' field to InstancePayload in notifications https://review.openstack.org/475276 | |
| 17:03:31 | mriedem | this fixes another one of our random functional test failures https://review.openstack.org/#/c/487772/ | |
| 17:07:14 | sdague | mriedem nice | |
| 17:11:22 | openstackgerrit | Merged openstack/python-novaclient master: Add 'Forced down' column in serivce-list https://review.openstack.org/487263 | |
| 17:17:28 | openstackgerrit | Merged openstack/python-novaclient master: Be clear about hypevisors.search used in a few CLIs https://review.openstack.org/487513 | |
| 17:19:27 | openstackgerrit | Ed Leafe proposed openstack/nova master: Handle hash ring rebalancing in ironic flavor migration https://review.openstack.org/487954 | |
| 17:19:33 | openstackgerrit | Doug Hellmann proposed openstack/nova master: WIP: add a redirect for the old cells landing page https://review.openstack.org/487932 | |
| 17:19:34 | openstackgerrit | Matt Riedemann proposed openstack/nova master: hardware offload support for openvswitch https://review.openstack.org/398265 | |
| 17:19:55 | openstackgerrit | Ed Leafe proposed openstack/nova master: Handle ironicclient failures in Ironic driver https://review.openstack.org/487925 | |
| 17:20:33 | mriedem | jaypipes: +2 on moshe's hw offload thingamajig https://review.openstack.org/#/c/398265/ | |
| 17:33:05 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test resize with placement api https://review.openstack.org/487958 | |
| 17:33:28 | gibi | mriedem: here is my first stab at the resize tests with placement api ^^ | |
| 17:34:10 | gibi | mriedem: it is pretty unstable at the current form and ther are some asserts commented out that are failing right now | |
| 17:34:20 | mriedem | gibi: cool, thanks | |
| 17:34:30 | openstackgerrit | Ed Leafe proposed openstack/nova master: Handle hash ring rebalancing in ironic flavor migration https://review.openstack.org/487954 | |
| 17:34:44 | gibi | mriedem: can be a real bug or can be some race in the test | |
| 17:34:52 | edleafe | dansmith: ^^ updated to address your comments | |
| 17:34:58 | gibi | mriedem: I have to leave for today but check back tomorrow morning | |
| 17:35:34 | dansmith | edleafe: cool, lets get ironic eyes on it and then I think we're good | |
| 17:35:46 | openstackgerrit | Sean Dague proposed openstack/nova master: always show urls in list_cells https://review.openstack.org/487860 | |
| 17:35:55 | cdent | gibi++ | |
| 17:36:17 | openstackgerrit | Merged openstack/python-novaclient master: Help text for "--matching" is not clear. https://review.openstack.org/487708 | |
| 17:36:55 | sdague | gibi: thanks for solving that test race | |
| 17:37:40 | sdague | mriedem: the sqla_url functions don't really provide the same reassembly that you get from urlparse | |
| 17:37:47 | sdague | which is why it was not used in https://review.openstack.org/#/c/487860/ | |
| 17:49:10 | mriedem | "Alex Xu 2:33 AM oh..damn it." | |
| 17:49:18 | mriedem | someone is going to have to have a talk with alex_xu | |
| 17:52:07 | mriedem | bauzas: so your -1 on this is just stale at this point https://review.openstack.org/#/c/483566/ | |
| 17:52:38 | mriedem | bauzas: can you remove the -1 and re-review? if you're ok, but don't want to +W until i go through it too that's fine | |
| 17:53:22 | jaypipes | mriedem: yeah, I'm on it. | |
| 17:53:57 | mriedem | you know, this low sodium vegetarian lentil soup that laura buys could really use some pork | |
| 17:54:11 | jaypipes | omg lmao | |
| 17:54:58 | openstackgerrit | Doug Hellmann proposed openstack/nova master: WIP: add a redirect for the old cells landing page https://review.openstack.org/487932 | |
| 17:55:30 | bauzas | mriedem: sure , just at dinner atm so will do in the next hour | |
| 17:58:00 | smcginnis | mriedem: Some of the vegetarian pork like they tried to give dims? | |
| 17:58:17 | dims | LOL | |
| 17:58:36 | dansmith | just don't give him quinoa whatever you do | |
| 17:58:42 | dims | amen! | |
| 17:58:48 | dansmith | pork = funny joke | |
| 17:58:53 | dansmith | quinoa = unfunny disaster | |
| 17:58:57 | sdague | right, all the quinoa goes to dansmith | |
| 17:59:09 | mriedem | like a gremlin eating after midnight | |
| 17:59:13 | dansmith | sdague: joke's on you homeboy, I like quinoa | |
| 18:00:52 | sdague | I am confused by dansmith liking something that's not a hamburger | |
| 18:01:19 | dansmith | http://www.bonappetit.com/recipe/stellar-quinoa-burger | |