| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-05-18 | |||
| 15:42:27 | melwitt | and, the forumtopics.o.o site is broken, I don't know what happened there | |
| 15:42:41 | mriedem | they dropped it once the sessions were put into the main schedule | |
| 15:42:53 | melwitt | okay, I missed that memo | |
| 15:42:59 | mriedem | i don't think there was one | |
| 15:43:13 | melwitt | oh | |
| 15:44:55 | mriedem | https://www.openstack.org/summit/vancouver-2018/summit-schedule/events/21787/pre-emptible-instances-the-way-forward should also be on the list | |
| 15:45:20 | melwitt | ack. let me know if you notice any others I missed | |
| 15:45:33 | mriedem | https://www.openstack.org/summit/vancouver-2018/summit-schedule/events/21749/missing-features-in-openstack-for-public-clouds | |
| 15:45:40 | mriedem | ^ always overlaps with nova requirements | |
| 15:45:55 | melwitt | yup, good one | |
| 15:45:58 | mriedem | https://www.openstack.org/summit/vancouver-2018/summit-schedule/events/21881/api-debt-cleanup is the new one | |
| 15:46:22 | mriedem | extended maintenance and FFU are not directly nova related, but impact us in some way | |
| 15:46:25 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova stable/pike: libvirt: Lift the restriction of choices for `cpu_model_extra_flags` https://review.openstack.org/569449 | |
| 15:46:26 | mriedem | for those interested | |
| 15:46:32 | melwitt | thx | |
| 15:46:43 | mriedem | S release goals | |
| 15:46:49 | mriedem | https://www.openstack.org/summit/vancouver-2018/summit-schedule/events/21738/s-release-goals | |
| 15:46:58 | mriedem | "re-introduce mox!" | |
| 15:47:12 | melwitt | lol | |
| 15:47:28 | mriedem | there are some keystone ones that we should likely have presence (unified limits and default scopes) | |
| 15:47:30 | superdan | kashyap: why are you backporting those restriction-lifting patches? | |
| 15:48:01 | superdan | kashyap: the whole point was to make the original one restrict to just pcid and then un-restrict in master only yeah? | |
| 15:48:11 | melwitt | mriedem: yes, agreed | |
| 15:49:17 | cfriesen | what about the "nova onboarding" one, or python 2 deprecation? | |
| 15:49:35 | mriedem | nova onboarding is implied | |
| 15:49:45 | mriedem | it's marked as a presentation | |
| 15:50:00 | jgwentworth | yeah, and nova project update. but doesn't hurt to add them to the list, I'll make sections for stuff | |
| 16:06:14 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Skip ServerShowV254Test.test_rebuild_server in cells v1 job https://review.openstack.org/569454 | |
| 16:07:03 | openstackgerrit | Dan Smith proposed openstack/nova master: Fix interpretation of max_attempts for scheduling alternates https://review.openstack.org/569127 | |
| 16:08:57 | mriedem | superdan: i'm not sure, but https://review.openstack.org/#/c/569267/ might be failing the cells v1 job because of this: http://logs.openstack.org/67/569267/2/check/nova-cells-v1/ce85331/logs/screen-n-cell-child.txt.gz?level=TRACE#_May_18_14_02_29_212925 | |
| 16:09:39 | superdan | mriedem: ah interesting | |
| 16:09:51 | superdan | I bet there's some cells code that looks for objectactionerror instead | |
| 16:11:02 | superdan | or, hmm, maybe refresh | |
| 16:11:06 | mriedem | yeah | |
| 16:12:38 | jgwentworth | huh, there's not one, but two fast-forward upgrade sessions | |
| 16:12:59 | mriedem | jgwentworth: i think there always is, | |
| 16:13:00 | superdan | jgwentworth: yeah, there's the one that is talking about the actual progress being made, | |
| 16:13:08 | mriedem | because the entire first session is recapping everything already said before | |
| 16:13:08 | superdan | and then another about ... not that | |
| 16:14:14 | superdan | mriedem: that's a nice way of putting it | |
| 16:14:32 | jgwentworth | superdan: are you referring to the fast-forward upgrades talk? because that is a THIRD thing | |
| 16:14:32 | superdan | and the first one has two parts, IIRC | |
| 16:14:47 | superdan | jgwentworth: oh you were just commenting that there are two parts to the first one? | |
| 16:15:04 | mriedem | I and Ii | |
| 16:15:14 | jgwentworth | oh, yes. I didn't think to think of it as one | |
| 16:15:15 | mriedem | FFU IV is when FFU fights the russians | |
| 16:15:26 | superdan | based on previous history, I think we need two hours because we'll spend a bunch of time talking about what FFU isn't actually doing but the moderator wants to be doing | |
| 16:15:54 | jgwentworth | :) | |
| 16:16:11 | superdan | or rather, what the moderator wants other people to do | |
| 16:16:27 | mriedem | upgrade fastly with no downtime? | |
| 16:17:03 | leakypipes | mriedem: he will crush you. | |
| 16:17:16 | mriedem | FFU is like iron | |
| 16:17:43 | jgwentworth | debating whether to put the two-part FFU session on this mail | |
| 16:19:07 | jgwentworth | maybe not, it's not too nova-related last time I've gone | |
| 16:20:10 | superdan | agree | |
| 16:20:29 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Send resource allocations in the port binding https://review.openstack.org/569459 | |
| 16:26:54 | leakypipes | gibi: safe travels mate. | |
| 16:28:28 | openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Testing a thing https://review.openstack.org/569461 | |
| 16:28:33 | gibi | leakypipes: thanks | |
| 16:28:45 | openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova master: Remove the remaining of the removed option https://review.openstack.org/569462 | |
| 16:29:17 | openstackgerrit | Merged openstack/nova master: Update the deprecate os_region_name option https://review.openstack.org/567829 | |
| 16:30:52 | openstackgerrit | Merged openstack/nova master: Fix inconsistency in docs https://review.openstack.org/569348 | |
| 16:34:09 | openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: Localdisk https://review.openstack.org/549300 | |
| 16:54:42 | superdan | mriedem: so, I don't think my instance change actually caused that error | |
| 16:54:54 | superdan | mriedem: here's another example from a different unrelated patch: http://logs.openstack.org/38/564838/6/gate/nova-cells-v1/5fc0493/logs/screen-n-cell-child.txt.gz?level=TRACE#_May_18_14_09_53_542120 | |
| 16:56:08 | superdan | mriedem: my DNM testing patch above should help uncover what it's actually choking on | |
| 17:38:32 | superdan | mriedem: that cells failure is trying to load keypairs | |
| 17:39:06 | superdan | mriedem: which is probably because the instance that came out of the cell database can't have loaded that from the api db down there or some such | |
| 17:40:30 | jgwentworth | I wonder why the cells v1 job is otherwise on the fritz today | |
| 17:40:48 | jgwentworth | seeing a lot of fails and rechecks in my gerrit notifications | |
| 17:40:59 | superdan | jgwentworth: what I just said :) | |
| 17:41:17 | jgwentworth | oh, that change already landed? I had thought it was stuck on the cells v1 job | |
| 17:41:30 | jgwentworth | if so, nevermind | |
| 17:41:35 | superdan | no | |
| 17:41:40 | superdan | it's unrelated to that change | |
| 17:42:16 | jgwentworth | oh. I thought when you mentioned failure to load keypairs it was in the context of the "load everything" patch | |
| 17:42:38 | jgwentworth | so weird that it's suddenly acting up if not for that | |
| 17:42:56 | superdan | we thought it was caused by that patch but it's not | |
| 17:43:06 | superdan | yeah I dunno why it just started coming up, | |
| 17:43:10 | superdan | but it's clearly racy so .. | |
| 17:43:16 | jgwentworth | yeah ++ | |
| 17:44:55 | superdan | I'm going to put it underneath the bottom one though just so it will squash the issue and make it easier to land that set | |
| 17:47:01 | superdan | heh we actually have a test that ensures that refresh does this exactly | |
| 17:47:36 | superdan | but I don't think the cells code can do anything about this, so probably better to just alter the refresh() behavior | |
| 17:51:58 | jgwentworth | guh, I gotta pack | |
| 18:12:59 | openstackgerrit | Dan Smith proposed openstack/nova master: Make instance.refresh() avoid recursion better https://review.openstack.org/569461 | |
| 18:13:00 | openstackgerrit | Dan Smith proposed openstack/nova master: Make instance able to lazy-load almost everything https://review.openstack.org/569267 | |
| 18:13:01 | openstackgerrit | Dan Smith proposed openstack/nova master: Expose instance_get_all_uuids_by_host() from DB API and use it https://review.openstack.org/569268 | |
| 18:13:18 | superdan | first one there ^ should fix the cellsv1 thing | |
| 18:18:10 | mriedem | superdan: ok i was wondering how your change would have caused that | |
| 18:18:18 | openstackgerrit | Merged openstack/nova master: Fix typo in NUMATopologyFilter docs https://review.openstack.org/543505 | |
| 18:26:19 | mriedem | jgwentworth: superdan: this is why we're seeing the spike in cells v1 job failures https://review.openstack.org/#/c/563851/ | |
| 18:26:21 | mriedem | the test is new | |
| 18:26:33 | superdan | ah | |
| 18:26:51 | superdan | how fail-y is it? | |
| 18:27:19 | mriedem | http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22ServerShowV254Test%5C%22%20AND%20message%3A%5C%22rebuild_instance%5C%22%20AND%20message%3A%5C%22OrphanedObjectError%5C%22%20AND%20tags%3A%5C%22screen-n-cell-child.txt%5C%22&from=7d | |
| 18:27:44 | mriedem | it's a new rebuild test + key_name change | |
| 18:27:51 | mriedem | that doesn't really need to live in tempest... | |
| 18:27:53 | mriedem | gmann: ^ | |
| 18:28:47 | superdan | mriedem: that didn't load anything for me | |