| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-01 | |||
| 15:21:29 | mriedem | sdague: there is conditional logic in devstack to know if it's a subnode or not based on n-api running | |
| 15:21:30 | mriedem | is my point | |
| 15:22:29 | sdague | mriedem: yeh | |
| 15:23:04 | sdague | dfisher: anyway, if you have suggestions on how to make this more clear and straightforward to setup on the devstack side, let me know | |
| 15:23:26 | dfisher | sdague: just mention that the transport_url on the compute node needs to match the cell1 transport_url | |
| 15:23:28 | dfisher | that's all you need. | |
| 15:24:26 | dfisher | and, again, thank you sdague, mriedem, and dansmith for putting up with dumb questions from the peanut gallery | |
| 15:24:43 | gibi | jaypipes, dansmith: I have one more question at https://review.openstack.org/#/c/487958/21/nova/tests/functional/test_servers.py@1370 | |
| 15:25:20 | sdague | dfisher: any idea where that should be mentioned that it would be more obvious? | |
| 15:25:54 | dfisher | sdague: | |
| 15:25:54 | dfisher | 1.Configure and start your compute hosts. Before step 7, make sure you have compute hosts in the database by running nova service-list --binary nova-compute. | |
| 15:25:59 | dfisher | gah. that should be #6 | |
| 15:27:00 | dfisher | maybe something like: 'Configure your compute node's transport_url to point to the cell1 transport_url (visible with nova-manage cell_v2 list_cells —verbose) and start the nova-compute service' | |
| 15:27:39 | jaypipes | gibi: that's concerning to me. I can't explain why that would be. :( | |
| 15:29:54 | gibi | jaypipes: sorry I pointed to the wrong assert it fails the assert 'self.assertEqual(0, len(allocations))' | |
| 15:29:55 | mriedem | dfisher: step 5 says, "The transport url is taken from the [DEFAULT]/transport_url setting in the Nova configuration file if not specified on the command line." - which would assume that if you created cell1 with a transport_url from the config file, you'd be using that same config file on the compute service if you have a single MQ | |
| 15:30:19 | mriedem | dfisher: devstack is a bit different now in that by default there are 2 conductors | |
| 15:30:50 | mriedem | what you suggest isn't a bad reminder though | |
| 15:32:03 | mriedem | dfisher: did you want to push a change to adjust that part of the docs? | |
| 15:32:10 | bauzas | oh man, my computer crashed | |
| 15:32:27 | mriedem | bauzas: it didn't crash, it's on french government mandated PTO | |
| 15:32:32 | bauzas | (says the one who uses AndChat) | |
| 15:32:35 | mriedem | hence the smoke | |
| 15:32:39 | jaypipes | gibi: I believe there should be a *separate* assertion above line 1370 that shows we have the correct allocation after CONFIRM_RESIZE but after the periodics run, that allocation is messed up. I think that's what dansmith was going for there. | |
| 15:32:45 | dfisher | I can. Not sure how, however. That and me no word good. | |
| 15:33:02 | bauzas | mriedem: it's a trap | |
| 15:33:12 | mriedem | dfisher: have you ever pushed a change to review.openstack.org? | |
| 15:33:16 | dfisher | yep. | |
| 15:33:26 | dansmith | isn't dfisher from oracle? | |
| 15:33:30 | dfisher | that's me | |
| 15:33:33 | dansmith | he might burst into flame if he pushes a patch | |
| 15:33:36 | dansmith | so we should probably do it for him | |
| 15:33:37 | dfisher | :( | |
| 15:33:57 | mriedem | dfisher: http://git.openstack.org/cgit/openstack/nova/tree/doc/source/user/cells.rst#n391 is the line you want | |
| 15:34:06 | dfisher | on it. | |
| 15:34:16 | gibi | jaypipes: make sense | |
| 15:34:36 | jaypipes | gibi: I'll add that assertion. | |
| 15:34:42 | gibi | jaypipes: thanks | |
| 15:35:46 | dfisher | dansmith: i'll let you know if you if any parts of me (fleshy or ethereal) start to smoke as I work on this | |
| 15:36:17 | dansmith | dfisher: I'll stand by with a fire extinguisher | |
| 15:36:30 | dfisher | even though I'm an Oracle employee? :) | |
| 15:36:39 | mriedem | sdague: redirect works https://docs.openstack.org/nova/latest/cells.html | |
| 15:36:40 | dansmith | dfisher: you're still a human... | |
| 15:36:45 | dansmith | dfisher: um, right? | |
| 15:36:58 | gibi | jaypipes: would it make sense if I start building some more funcitonal tests top of the your bug fix to see further ahead? | |
| 15:37:05 | dansmith | heh | |
| 15:37:14 | mriedem | gibi: yes we want a single node functional set of tests for resize confirm/revert | |
| 15:37:18 | mriedem | resize to same host | |
| 15:37:23 | openstackgerrit | Matthew Booth proposed openstack/nova master: fake_notifier: allow waiting on either notification type https://review.openstack.org/489637 | |
| 15:37:38 | dfisher | as a Denver resident I can suggest places to eat (and not to eat) for the PTG | |
| 15:37:54 | mriedem | gibi: that's going to be needed for this bug https://bugs.launchpad.net/nova/+bug/1707252 | |
| 15:37:55 | jaypipes | gibi: not at this time, I don't think so. | |
| 15:37:55 | openstack | Launchpad bug 1707252 in OpenStack Compute (nova) "Claims in the scheduler does not account for doubling allocations on resize to same host" [Medium,Confirmed] | |
| 15:37:55 | dfisher | i can trade that for fire-putting-outting, maybe | |
| 15:38:29 | mdbooth | mriedem: Incidentally, I now have a patch on top of gibi's patch, but because it also has another dependency I can't push it | |
| 15:38:58 | mdbooth | Any chance we could keep the scope of gibi's patch small so it can merge soon? | |
| 15:39:04 | mdbooth | And add more later. | |
| 15:39:40 | gibi | mdbooth: If you are refering to https://review.openstack.org/#/c/487958/ then I think that patch is pretty close to land | |
| 15:39:47 | mriedem | mdbooth: i was assuming any new tests would be built on top | |
| 15:40:01 | mdbooth | gibi: Yep, that's the one. | |
| 15:40:08 | mdbooth | mriedem: Excellent, just checking. | |
| 15:40:08 | mriedem | mdbooth: believe me, i'd like to see that test patch land asap | |
| 15:40:17 | gibi | mdbooth: I think that patch will not grow further | |
| 15:40:37 | mdbooth | gibi: Now could you land it in the next 10 minutes? ;) | |
| 15:40:51 | gibi | mdbooth: I have no power over it ;) | |
| 15:41:30 | mdbooth | I have your patch stashed in the middle of my own series, so I need to be really careful not to submit it! | |
| 15:43:01 | gibi | mdbooth: to be clear it is not my patch, it is shared at least between jaypipes, dansmith, cdent and myself. Just to make it easy ;) | |
| 15:43:58 | gibi | mriedem: so I'm not committing myself to start the resize to same host today, but if nobody starts that until I wake up tomorrow then I will stat | |
| 15:44:17 | gibi | mriedem: s/stat/start/ | |
| 15:44:32 | mriedem | gibi: ok i might start that today | |
| 15:44:37 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add redirect for api-microversion-history doc https://review.openstack.org/489641 | |
| 15:45:05 | gibi | mriedem: ping me when you start to avoid double work | |
| 15:45:31 | mriedem | aye aye | |
| 15:46:45 | mriedem | fyi, also sent to the ML https://etherpad.openstack.org/p/nova-pike-release-candidate-todo | |
| 15:48:38 | openstackgerrit | Drew Fisher proposed openstack/nova master: Add language for compute node configuration https://review.openstack.org/489643 | |
| 15:49:43 | openstackgerrit | Jay Pipes proposed openstack/nova master: Test resize with placement api https://review.openstack.org/487958 | |
| 15:49:43 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: remove existing allocs when set allocs https://review.openstack.org/489273 | |
| 15:49:44 | openstackgerrit | Jay Pipes proposed openstack/nova master: remove source provider allocs in confirm_resize() https://review.openstack.org/488510 | |
| 15:50:00 | jaypipes | gibi: done. | |
| 15:51:06 | gibi | jaypipes: looking | |
| 15:54:34 | dansmith | dfisher-cinder: sorry, I was distracted while you were exploding | |
| 15:55:11 | sdague | mriedem: woot, now we just need to figure out what other redirects are needed | |
| 15:55:29 | mriedem | sdague: basically everything | |
| 15:55:36 | mriedem | placement, vendordata, everything has changed | |
| 15:55:41 | mriedem | api microversion history | |
| 15:55:45 | mriedem | i'm in -doc | |
| 16:00:56 | gibi | jaypipes: looks good to me | |
| 16:06:56 | bauzas | mriedem: have time for a quick question about https://bugs.launchpad.net/nova/+bug/1702454 ? | |
| 16:06:56 | openstack | Launchpad bug 1702454 in OpenStack Compute (nova) "Transforming the RequestSpec object into legacy dicts doesn't support the requested_destination field" [High,In progress] - Assigned to Sylvain Bauza (sylvain-bauza) | |
| 16:07:38 | bauzas | mriedem: given it works for Pike but not Newton and Ocata, the regression test would be different between stable changes and the master change | |
| 16:07:52 | bauzas | mriedem: doesn't that look a bit weird ? | |
| 16:08:54 | bauzas | mriedem: tbc, I'll be testing that if you have 2 hosts (A, B) with A as a target for the instance, it would provide a NoValidHost if A is not accepting it | |
| 16:09:15 | bauzas | mriedem: that would be for the fix | |
| 16:09:32 | bauzas | mriedem: and without the fix (in Ocata and N) it would just provide B as instance.host | |
| 16:09:43 | bauzas | or at least accepting to evacuate | |
| 16:09:57 | bauzas | see my problem ? | |
| 16:10:24 | bauzas | the cherry-pick change would be different | |
| 16:10:31 | openstackgerrit | Dan Smith proposed openstack/nova master: Add some more cellsv2 doc goodness https://review.openstack.org/487183 | |
| 16:10:31 | mriedem | bauzas: do we have a fix to make in pike at all? | |
| 16:10:56 | bauzas | mriedem: so, only a stable change ? | |