| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-02-03 | |||
| 00:32:16 | dansmith | I'm projecting a little of our OSP backport rules here, but still | |
| 00:32:16 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add release note for filter/sort whitelist https://review.openstack.org/428285 | |
| 00:32:34 | jroll | edleafe: I'm investigating that big comment to be sure, but have a few there for you | |
| 00:32:35 | edleafe | jroll: heh, but I'm not :) | |
| 00:32:39 | mriedem | laura kicked my ass into eating dinner with her so i'll be back in a bit | |
| 00:32:45 | mriedem | and all questions will be answered | |
| 00:33:40 | melwitt | dansmith: yeah. it takes me awhile to understand the backport implications to upgrades | |
| 00:33:53 | dims_ | mriedem : better listen to her :) | |
| 00:34:15 | dansmith | melwitt: wish you had brought this up earlier, but the actual patch for this request to the scheduler took like four weeks longer than expected, so :? | |
| 00:34:18 | dansmith | er -> :/ | |
| 00:34:58 | edleafe | jroll: doh! Yeah, the way that the merge conflicts were marked in git didn't help | |
| 00:35:08 | edleafe | cleaning that up now | |
| 00:35:15 | jroll | edleafe: yeah no worries | |
| 00:35:26 | jroll | the main problem is the get_by_host thing | |
| 00:35:38 | jroll | I think it should likely be get_by_host_and_node | |
| 00:36:03 | melwitt | dansmith: yeah, I should have been paying more attention to that. I didn't know whether the bfv thing was planned to be handled this cycle. and cells v2 madness occurred, yadayada | |
| 00:39:36 | dansmith | melwitt: yeah | |
| 00:40:19 | edleafe | jroll: the get_by_host_and_node makes sense. I was using what jaypipes suggested, but I think he typed that off the top of his head. | |
| 00:41:05 | jroll | edleafe: the host/node thing is weird when you aren't an ironic person :) | |
| 00:41:45 | edleafe | jroll: and I thought it was always weird | |
| 00:41:54 | dansmith | edleafe: and you would be right | |
| 00:42:28 | jroll | edleafe: true that | |
| 00:42:36 | jroll | I gotta bounce for the night, I'll look again tomorrow | |
| 00:43:19 | edleafe | jroll: thanks | |
| 00:57:29 | mriedem | dansmith: melwitt: let's put the scheduler BFV patch or whatever at the bottom of the etherpad in the nice to haves list | |
| 00:58:39 | dansmith | I still have 21 failures without getting into unshelve | |
| 00:58:50 | dansmith | and I'm about to drop off for the evening | |
| 00:58:51 | dansmith | but, sure. | |
| 00:59:30 | mriedem | ultimately it's going to depend on if we can backport it | |
| 01:02:51 | mriedem | gmann: ok i think https://review.openstack.org/#/c/428285/ is good to go | |
| 01:02:53 | mriedem | oomichi: ^ | |
| 01:03:46 | mriedem | http://docs-draft.openstack.org/85/428285/3/check/gate-nova-releasenotes/aa1d391//releasenotes/build/html/unreleased.html#upgrade-notes | |
| 01:04:32 | oomichi | mriedem: terminated_at is valid, not terminate_at :) | |
| 01:04:53 | mriedem | ? | |
| 01:05:05 | mriedem | terminate_at isn't in there | |
| 01:05:10 | mriedem | i already fixed that bug yesterday | |
| 01:05:32 | oomichi | mriedem: yeah, by you. LGTM for the reno | |
| 01:05:55 | mriedem | just sign with your +W | |
| 01:07:04 | oomichi | yep, done | |
| 01:07:19 | mriedem | great, thanks | |
| 01:13:17 | gmann | mriedem: +1, all grep and extra check good now | |
| 01:13:33 | edleafe | jroll: thanks | |
| 01:13:37 | edleafe | doh | |
| 01:13:51 | mriedem | i'm going to tag the highest for ocata microversions in the docs too, for compute and placement | |
| 01:15:07 | gmann | +1 | |
| 01:15:49 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Mark compute/placement REST API max microversions for Ocata https://review.openstack.org/428528 | |
| 01:16:07 | mriedem | oomichi: ^ | |
| 01:17:31 | oomichi | mriedem: so will you -2 patches which increase microversion in this cycle? | |
| 01:17:42 | oomichi | if so, +2 on the patch | |
| 01:18:14 | gmann | after that branch going to be made right | |
| 01:18:18 | melwitt | dansmith, mriedem: what are the 21 failures? I don't see that on the jenkins run | |
| 01:18:23 | mriedem | we won't have any other microversions in ocata | |
| 01:18:29 | gmann | yea | |
| 01:18:48 | oomichi | ok, +2 | |
| 01:19:00 | dansmith | melwitt: 21 failures after I've started making the other functions do the same | |
| 01:19:38 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP Request zero root disk for boot-from-volume instances https://review.openstack.org/428481 | |
| 01:19:38 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP Add functional test for boot-from-volume instances https://review.openstack.org/428505 | |
| 01:19:42 | dansmith | melwitt: that ^ | |
| 01:19:46 | dansmith | and .. I'm out | |
| 01:20:25 | melwitt | oh, gotcya | |
| 01:20:39 | melwitt | that means ... gotcha and cya at the same time | |
| 01:22:10 | mriedem | we need more depends-ons https://review.openstack.org/#/c/428531/ | |
| 01:25:02 | openstackgerrit | Maciej Szankin proposed openstack/nova master: conf: Resolve TODOs in 'database' https://review.openstack.org/393695 | |
| 01:29:33 | mriedem | melwitt: want to send this in https://review.openstack.org/#/c/428528/ ? | |
| 01:30:12 | openstackgerrit | Ed Leafe proposed openstack/nova master: Delete a compute node's resource provider when node is deleted https://review.openstack.org/428375 | |
| 01:30:51 | melwitt | mriedem: k | |
| 01:31:20 | melwitt | +W. gotta run for now | |
| 01:31:24 | edleafe | mriedem: jroll: ^^ fixed jroll's comments | |
| 01:55:56 | openstackgerrit | Zhenguo Niu proposed openstack/nova master: Fix breaking out of nested loops https://review.openstack.org/424931 | |
| 03:18:17 | mriedem | edleafe: reviewed your patch, some issues, mostly minor, should be easy to deal with in the morning | |
| 03:48:09 | mriedem | EmilienM: in a tripleo experimental run on nova, which job should i be looking at? gate-tripleo-ci-centos-7-ovb-ha-tempest? | |
| 03:48:34 | mriedem | i can't make heads or tails of the failures in gate-tripleo-ci-centos-7-ovb-nonha | |
| 03:49:15 | mriedem | i guess the 2 hour 13 minute tempest one is the one i want http://logs.openstack.org/60/428360/2/experimental-tripleo/gate-tripleo-ci-centos-7-ovb-ha-tempest/4eff29f/console.html#_2017-02-03_00_42_04_028094 | |
| 03:49:21 | EmilienM | mriedem: no, gate-tripleo-ci-centos-7-nonha-multinode-nv | |
| 03:49:35 | EmilienM | it's the fastest we have | |
| 03:49:37 | mriedem | EmilienM: that doesn't run on nova's experimental queue | |
| 03:49:45 | mriedem | looking at https://review.openstack.org/#/c/428360/ | |
| 03:50:11 | EmilienM | it did, gate-tripleo-ci-centos-7-nonha-multinode | |
| 03:50:19 | EmilienM | the -nv is when the job will move in check pipeline | |
| 03:52:12 | EmilienM | mriedem: do you need anything else? otherwise I'm heading to sleep | |
| 03:52:24 | mriedem | oh i was looking in the group of tripleo jobs | |
| 03:52:39 | mriedem | no i'll poke in the morning i guess | |
| 03:54:09 | mriedem | i'm gonna have a hell of a time figuring out why this job failed | |
| 03:54:35 | EmilienM | mriedem: I can release my morning and debug with you on the tripleo job | |
| 03:54:49 | EmilienM | I've patched our CI tonight to add more debug | |
| 03:55:11 | EmilienM | if you run "check experimental" again, you should have debug = true on nova.conf | |
| 03:55:11 | mriedem | ok | |
| 03:55:20 | mriedem | i also see the heat stack failed on instance not found http://logs.openstack.org/60/428360/2/experimental/gate-tripleo-ci-centos-7-nonha-multinode/6ca334c/console.html#_2017-02-02_20_42_31_044252 | |
| 03:55:22 | mriedem | so that killed it | |
| 03:55:27 | mriedem | anyway, talk to you tomorrow | |
| 03:55:35 | EmilienM | so this is a pingtest | |
| 03:55:42 | EmilienM | which means we weren't able to spawn a server | |
| 03:56:05 | EmilienM | http://logs.openstack.org/60/428360/2/experimental/gate-tripleo-ci-centos-7-nonha-multinode/6ca334c/logs/subnode-2/var/log/nova/nova-api.txt.gz#_2017-02-02_20_42_17_097 | |
| 03:56:26 | EmilienM | recheck the job (and check experimental), so you'll have DEBUG | |
| 03:56:30 | EmilienM | mriedem: good night | |
| 04:25:22 | mriedem | rc1 is basically ready to go i think https://review.openstack.org/#/c/428531/ | |
| 07:17:37 | openstackgerrit | Jiri Suchomel proposed openstack/nova master: Retry few times when creating the initial connection to libvirtd. https://review.openstack.org/416928 | |
| 07:28:43 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Should raise correct error instead of class exist in Placement API https://review.openstack.org/428612 | |
| 07:31:02 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Check destination_type when booting with bdm provided https://review.openstack.org/402372 | |
| 07:31:17 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Don't delete neutron port when attach failed https://review.openstack.org/403568 | |