| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-09-07 | |||
| 17:09:12 | jaypipes | cburgess: ugh. well, at least there's no prop damage. | |
| 17:10:08 | cburgess | jaypipes yeah hope you fare as well | |
| 17:31:40 | openstackgerrit | Michael Still proposed openstack/nova master: Move nbd commands to privsep. https://review.openstack.org/500351 | |
| 17:31:41 | openstackgerrit | Michael Still proposed openstack/nova master: Move lvm handling to privsep. https://review.openstack.org/495516 | |
| 17:31:42 | openstackgerrit | Michael Still proposed openstack/nova master: Move xend existence probes to privsep. https://review.openstack.org/495538 | |
| 17:31:43 | openstackgerrit | Michael Still proposed openstack/nova master: Move shred to privsep. https://review.openstack.org/495537 | |
| 17:31:44 | openstackgerrit | Michael Still proposed openstack/nova master: Cleanup mount / umount and associated rmdir calls https://review.openstack.org/494423 | |
| 17:31:46 | openstackgerrit | Michael Still proposed openstack/nova master: Move loopback setup and removal to privsep. https://review.openstack.org/495664 | |
| 17:31:46 | openstackgerrit | Michael Still proposed openstack/nova master: WIP / Aspirational: we don't need rootwrap any more. https://review.openstack.org/495542 | |
| 17:31:51 | openstackgerrit | Michael Still proposed openstack/nova master: Move libvirt usages of chown to privsep. https://review.openstack.org/471972 | |
| 17:31:54 | openstackgerrit | Michael Still proposed openstack/nova master: Don't shell out to mkdir, use ensure_tree() https://review.openstack.org/492326 | |
| 17:31:56 | openstackgerrit | Michael Still proposed openstack/nova master: Move ploop commands to privsep. https://review.openstack.org/492325 | |
| 17:32:01 | openstackgerrit | Michael Still proposed openstack/nova master: Read from console ptys using privsep. https://review.openstack.org/489486 | |
| 17:32:02 | openstackgerrit | Michael Still proposed openstack/nova master: Move the idmapshift binary into privsep. https://review.openstack.org/495541 | |
| 17:32:03 | openstackgerrit | Michael Still proposed openstack/nova master: Move kpartx calls to privsep. https://review.openstack.org/500354 | |
| 17:32:06 | openstackgerrit | Michael Still proposed openstack/nova master: Move execs of touch to privsep. https://review.openstack.org/489190 | |
| 17:32:08 | openstackgerrit | Michael Still proposed openstack/nova master: Move libvirts dmcrypt support to privsep. https://review.openstack.org/490737 | |
| 17:32:09 | openstackgerrit | Michael Still proposed openstack/nova master: Move blkid calls to privsep. https://review.openstack.org/500398 | |
| 17:32:23 | openstackgerrit | Michael Still proposed openstack/nova master: Move execs of tee to privsep. https://review.openstack.org/489438 | |
| 17:32:43 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Allow setting up multiple cells in the base TestCase https://review.openstack.org/501818 | |
| 17:32:44 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Track which cell each instance is created in and use it consistently https://review.openstack.org/501820 | |
| 17:32:44 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Make ConductorTaskTestCase run with 2 cells https://review.openstack.org/501819 | |
| 17:32:45 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Ensure instance mapping is updated in case of quota recheck fails https://review.openstack.org/501821 | |
| 17:32:50 | mikal | sdague: this round of gate smashing is all updates of reno to document adds and removes | |
| 18:05:19 | mikal | mriedem: you seem to have forgotten to schedule a three hours session on privsep for first thing Wednesday. I can fix the etherpad if you want. | |
| 18:06:46 | mriedem | you can have 4pm on friday | |
| 18:13:15 | sean-k-mooney | stephenfin: by the way when are you flying out | |
| 18:13:52 | mikal | mriedem: don't make me break the gate more | |
| 18:14:06 | sean-k-mooney | stephenfin: i get in late saturday night will you be around deven on sunday/monday. | |
| 18:15:32 | sean-k-mooney | stephenfin: if you are interested in the nova neutron negociation work i can bring you up to speed on where we are and how i want to evolve it to support minium band with gurrentess,neutron provided traits and several other usecases. | |
| 18:16:21 | sean-k-mooney | stephenfin: got to go buy a suitcase so ttyl | |
| 18:24:14 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Transform instance.resize.error notifications https://review.openstack.org/417882 | |
| 18:40:15 | mtreinish | sdague: hmm, the libvirt tests all fail: http://logs.openstack.org/76/501776/1/check/gate-cross-nova-python27-ubuntu-xenial/a161f12/testr_results.html.gz | |
| 18:40:25 | mtreinish | I guess it's not the group regex like I though | |
| 18:41:24 | Nisha_Agarwal | jaypipes, hi | |
| 18:43:53 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Transform keypair.delete notification https://review.openstack.org/463946 | |
| 18:44:05 | mriedem | Nisha_Agarwal: probably not a good time for jay, he's trying to avoid a hurricane | |
| 18:44:19 | Nisha_Agarwal | mriedem, ohk. | |
| 18:44:51 | Nisha_Agarwal | mriedem, i was proposing https://review.openstack.org/#/c/500429/ for ironic. Wanted discussion on that aspect from nova side. | |
| 18:45:36 | Nisha_Agarwal | mriedem, Are u fine to discuss? or anyone else i should ask? | |
| 18:46:03 | openstackgerrit | Jackie Truong proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949 | |
| 18:46:04 | mriedem | we plan on having some time at the ptg to talk about nova/ironic things on wednesday afternoon, | |
| 18:46:06 | mriedem | are you going to be at the ptg? | |
| 18:46:29 | Nisha_Agarwal | mriedem, no :( thats why i want to leave a spec before PTG starts | |
| 18:46:34 | mtreinish | mriedem: it the intent of: https://github.com/openstack/nova/blob/master/tox.ini#L30 to only run that 1 test module? | |
| 18:46:40 | mtreinish | s/it/is/g | |
| 18:46:56 | mriedem | mtreinish: idk | |
| 18:47:03 | mriedem | it was added with the osprofiler stuff | |
| 18:47:31 | mtreinish | ok, because that test path doesn't do anything, and we've been running unit tests twice... | |
| 18:47:34 | mriedem | Nisha_Agarwal: it looks like dtantsur|afk has reviewed the spec at least | |
| 18:47:40 | Nisha_Agarwal | mriedem, yes | |
| 18:48:11 | mriedem | Nisha_Agarwal: does it fall under anything around L90 here? https://etherpad.openstack.org/p/nova-ptg-queens | |
| 18:48:14 | Nisha_Agarwal | mriedem, dtantsur|afk, wants that first we should solve the issue on the nova side before we fix on ironic side | |
| 18:48:53 | Nisha_Agarwal | mriedem, yes and no both | |
| 18:49:38 | Nisha_Agarwal | mriedem, yes in the sense that the topic is to avoid such ugly hack, but not with how nova supports the ironci node today | |
| 18:50:52 | mriedem | Nisha_Agarwal: ok can you add something under that "Ironic problems" section then and link in your spec? | |
| 18:51:12 | Nisha_Agarwal | mriedem, i was looking forward from the aspect that nova supports the 1. Multiple resource classes for ironic node. 2. Atleast the basic flavor inventroy should get scheduled from nova for ironic node, how it used to work before pike | |
| 18:51:47 | Nisha_Agarwal | mriedem, yes :) | |
| 18:52:15 | Nisha_Agarwal | mriedem, so on nova side both above points can be covered in single blueprint or u see them as two blueprints? | |
| 18:52:33 | Nisha_Agarwal | and hence one/two specs? | |
| 18:52:50 | mriedem | i don't understand the problem yet since i haven't dug into the spec, so i can't really say | |
| 18:53:22 | Nisha_Agarwal | mriedem, currently ironic node just gets scheduled based on custom resource class | |
| 18:53:59 | Nisha_Agarwal | and even basic inventory like ram, disk and cpus are not used for the ironic node scheduling | |
| 18:55:00 | Nisha_Agarwal | since nova now treats each of these resources as different resource class, hence ironic shud also support multiple resource class so that it can be used for scheduling by nova | |
| 18:55:42 | mriedem | right, be the instance in nova consumes the entire ironic node | |
| 18:55:59 | mriedem | that's why the min/max units are 1 | |
| 18:56:17 | Nisha_Agarwal | yes, i understand the resourceclass structure | |
| 18:56:35 | mriedem | sorry, i should say, the max unit is the full amount of whatever resource class on the node | |
| 18:56:50 | Nisha_Agarwal | thats what ironic should have for any resource it wants to use for scheduling IMO | |
| 18:57:21 | Nisha_Agarwal | yes, min is 1, max= total | |
| 18:57:34 | Nisha_Agarwal | and step-size shud also be as what is total | |
| 18:57:52 | Nisha_Agarwal | allocation ratio also 1:0 | |
| 18:58:30 | Nisha_Agarwal | if we have this kind of structure for ironic resource, then ironic resources can be claimed fully at the placement | |
| 18:59:40 | mriedem | right this https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L801 | |
| 18:59:52 | mriedem | min/max/step/total/allocation are all 1 | |
| 19:00:21 | Nisha_Agarwal | mriedem, total and max should be the value of the resource | |
| 19:00:29 | Nisha_Agarwal | rest all shud be 1 | |
| 19:00:58 | Nisha_Agarwal | and if we schedule it from placement it can be fully claimed | |
| 19:01:01 | mriedem | ^ is for the custom resource class | |
| 19:01:17 | mriedem | total and max being the value of the resource is only for vcpus/ram/disk | |
| 19:01:28 | mriedem | https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L769-L794 | |
| 19:01:52 | Nisha_Agarwal | mriedem, yes. i was looking for scheduling to happen based on vcpus/ram/disk | |
| 19:02:15 | mriedem | yeah i see the argument in the spec about not using the Exact filters | |
| 19:02:39 | Nisha_Agarwal | we have customers who in their production(not on master branch), use non-exact filter | |
| 19:02:50 | Nisha_Agarwal | for ironic node scheduling | |
| 19:03:41 | Nisha_Agarwal | with the custom resource class that scheduling doesnt happen | |
| 19:04:56 | Nisha_Agarwal | mriedem, thats the main reason we want atleast scheduling to happen for these resources. Eventually we would want to leverage the same resource class structure in ironic for any other quantitative resources | |
| 19:05:32 | Nisha_Agarwal | which cannot be handled by capabilities whenever they are deprecated and taken over by traits | |
| 19:06:43 | openstackgerrit | Merged openstack/nova master: Refactor out claim_resources_on_destination into a utility https://review.openstack.org/499718 | |
| 19:07:50 | mriedem | Nisha_Agarwal: so you're aware of these release notes in pike right? http://docs-draft.openstack.org/77/501477/1/check/gate-nova-releasenotes/1dc7513//releasenotes/build/html/unreleased.html#id2 | |
| 19:08:14 | Nisha_Agarwal | mriedem, yes. | |
| 19:08:48 | Nisha_Agarwal | but the same filter_scheduler could be used for scheduling ironic nodes as well, isnt it? | |
| 19:09:25 | openstackgerrit | Matthew Treinish proposed openstack/nova master: Set regex flag on ostestr command for osprofiler tests https://review.openstack.org/501842 | |
| 19:09:32 | Nisha_Agarwal | mriedem, s/filter_scheduler/enabled_filters | |
| 19:13:17 | mriedem | i think that depends on if you're using the IronicHostManager doesn't it? | |
| 19:14:10 | mtreinish | mriedem: it also looks like that second command breaks openstack-health. We're only capturing the subunit for the last run | |
| 19:14:20 | mtreinish | so we only ever track that one test: http://status.openstack.org/openstack-health/#/job/gate-nova-python27-ubuntu-xenial | |
| 19:14:54 | mriedem | guh | |
| 19:15:00 | mriedem | idk, what are other projects doing for the osprofiler test? | |
| 19:15:03 | mriedem | running it in a separate job? | |