| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-04-25 | |||
| 11:42:22 | sean-k-mooney | we will see but the room was pretty split at the last ptg on that topic | |
| 11:43:18 | jaypipes | ack | |
| 12:05:48 | openstackgerrit | Merged openstack/nova stable/queens: Fix regression in glance client call https://review.opendev.org/655186 | |
| 12:10:29 | openstackgerrit | Merged openstack/nova stable/queens: libvirt: disconnect volume when encryption fails https://review.opendev.org/653033 | |
| 12:23:37 | fungi | aspiers: stephenfin: sean-k-mooney: yes, current plan i think is going straight from 2.13 to 2.16 we just need to re-test the upgrade process on our staging server. the transition from mysql to notedb is going to be the biggest challenge as it will likely require a day-long service outage to complete | |
| 12:24:13 | fungi | we didn't want to conflate that work with the opendev switch for gerrit/git, but this ptg will be the time to make plans | |
| 12:24:47 | sean-k-mooney | is there an infra meetup at the ptg or just people will be about so it will come up | |
| 12:25:15 | fungi | infra is meeting thursday and friday all day (we share a room with qa) | |
| 12:25:25 | sean-k-mooney | ah ok | |
| 12:26:37 | sean-k-mooney | i kind of want to get a bit more involved with the infra side of things but on the other hand was not sure how to go about that so i was just going to say hi | |
| 12:54:05 | kashyap | fungi: What is "notedb"? /me looks up | |
| 12:54:42 | kashyap | Okay, it is this: "NoteDb is the next generation of Gerrit storage backend, which replaces the traditional SQL backend ..." | |
| 12:54:45 | kashyap | https://gerrit-review.googlesource.com/Documentation/note-db.html | |
| 12:55:40 | fungi | kashyap: yep, basically we'll be etl'ing gigabytes of data from an rdbms into git repositories | |
| 12:55:50 | fungi | not a speedy process | |
| 12:57:39 | fungi | at one point the upgrade process was necessary to be performed stepwise first moving some of the data into an h2 database and then moving it from there into git in a subsequent gerrit version, but we hear they've added the ability to migrate directly from sql to notedb now so it may go a lot more quickly, we just need to test | |
| 13:05:00 | stephenfin | fungi: Thanks for the context. That's all very good to hear :) | |
| 13:07:15 | kashyap | fungi: Yeah, noted | |
| 13:07:38 | fungi | also part of the upgrade process involved moving to a new server on a newer ubuntu release so we would have a new enough version of java to work with newer gerrit. we got that done last year, which was a nontrivial undertaking | |
| 13:08:20 | sean-k-mooney | fungi: out of interst shicne the data will now be sotred in git | |
| 13:08:37 | sean-k-mooney | could we use the notedb data in tools like gertty | |
| 13:08:54 | sean-k-mooney | or will that content not be downloadable by others | |
| 13:09:38 | fungi | well, gertty already has access to the relevant data (modulo access controls allowing your account to access it) through gerrit's rest api | |
| 13:09:42 | sean-k-mooney | gerrty effectivly buildis its own sqlite db replicting the gerrit one if that was not required it could be a nice improvemnt | |
| 13:10:19 | fungi | oh, i see what you're asking... i think the data in notedb is aggregated in ways which wouldn't make that useful for per-user data tracking | |
| 13:11:09 | fungi | what it does enable though is distribution of state, an there's a plugin recently contributed upstream to gerrit by the gerrithub folks which does multi-master replication between notedbs | |
| 13:11:36 | fungi | so we could finally have a decent cross-provider high availability solution for it | |
| 13:11:50 | sean-k-mooney | ya that would be cool | |
| 13:12:28 | fungi | i know the wmf folks have just upgraded to 2.16 and are looking at leveraging that plugin soon | |
| 13:12:49 | sean-k-mooney | wmf? | |
| 13:12:59 | sean-k-mooney | wiki media foundaiton? | |
| 13:15:13 | sean-k-mooney | actully looking at https://gitenterprise.me/tag/notedb/ it looks like it will allow you to work offline | |
| 13:15:26 | stephenfin | ffs https://review.opendev.org/651297 | |
| 13:16:14 | stephenfin | there is definitely something buggy with one of our lower-constraint packages to say I keep seeing these StringException errors http://logs.openstack.org/97/651297/4/gate/openstack-tox-lower-constraints/758a74d/testr_results.html.gz | |
| 13:16:17 | stephenfin | But who knows what | |
| 13:19:25 | efried | stephenfin: That's bug 1825435 | |
| 13:19:26 | openstack | bug 1825435 in OpenStack Compute (nova) "TestRPC unit tests intermittently fail with "'>' not supported between instances of 'NoneType' and 'datetime.datetime'" - maybe due to "Fatal Python error: Cannot recover from stack overflow."" [High,Confirmed] https://launchpad.net/bugs/1825435 | |
| 13:20:00 | sean-k-mooney | did we find where that was intoduced yet | |
| 13:20:12 | efried | it seems almost exclusively isolated to py36, so it hits lower-constraints (which runs on py36) and the py36 UT. | |
| 13:20:40 | efried | sean-k-mooney: No. mriedem has been trying to debug it (https://review.opendev.org/#/c/654468/ and its dep) but we can't get it to reproduce with those patches |
|
| 13:20:41 | sean-k-mooney | apparently it started to show up first in stephenfin cells v1 removal patches but that could also be a coincidnces | |
| 13:21:12 | efried | sean-k-mooney: If you can update the video models bp description as requested by kashyap, I'll approve it. | |
| 13:21:34 | sean-k-mooney | am sure ill go do that now. | |
| 13:21:35 | efried | jaypipes: I was specifically asking if those two topics were ones you wanted to be around for. If so, I can try to shuffle things around. | |
| 13:21:39 | efried | thanks sean-k-mooney | |
| 13:21:42 | kashyap | sean-k-mooney: Thank you! | |
| 13:22:25 | efried | aspiers: If you're around and want to fix that one typo in https://review.opendev.org/#/c/655673/ I'll fast approve. | |
| 13:22:36 | sean-k-mooney | i have it more or less in the relse note https://review.opendev.org/#/c/647733/3/releasenotes/notes/extend-libvirt-video-model-support-d630b99ef5039f51.yaml | |
| 13:22:39 | efried | If you're not around, I'll fix it. | |
| 13:22:52 | efried | sean-k-mooney: copy paste, baby | |
| 13:23:05 | efried | kashyap: You okay with the wording there --^ ? | |
| 13:23:35 | sean-k-mooney | i need to change it slightly but to make sense in a blueprint | |
| 13:23:46 | sean-k-mooney | mainly the tense but ill use it as teh basis | |
| 13:23:48 | stephenfin | efried: I don't think it is, actually | |
| 13:23:51 | mriedem | lyarwood: seems like an easy bug to fix, but you should look https://bugs.launchpad.net/nova/+bug/1825882 | |
| 13:23:52 | stephenfin | http://logs.openstack.org/97/651297/4/gate/openstack-tox-lower-constraints/758a74d/job-output.txt.gz | |
| 13:23:52 | openstack | Launchpad bug 1825882 in OpenStack Compute (nova) "Virsh disk attach errors silently ignored" [Undecided,New] | |
| 13:24:16 | kashyap | efried: I'll add a comment there. But the 'none' model should be described. You can copy/paste the phrase I posted earlier | |
| 13:24:28 | stephenfin | It looks like the test never finishes. If you grep for 'test_get_transport_url' the status is inprogress | |
| 13:24:36 | stephenfin | though I could be misreading that | |
| 13:25:31 | efried | mriedem: Is that not the same bug? http://logs.openstack.org/97/651297/4/gate/openstack-tox-lower-constraints/758a74d/job-output.txt.gz#_2019-04-25_07_44_46_919295 | |
| 13:25:40 | stephenfin | I also missed that you'd already rechecked. Oops | |
| 13:26:10 | efried | no takesie backsies | |
| 13:26:13 | kashyap | sean-k-mooney: efried I added the phrasing in the change. | |
| 13:26:38 | mriedem | efried: yes http://logs.openstack.org/97/651297/4/gate/openstack-tox-lower-constraints/758a74d/job-output.txt.gz#_2019-04-25_07_42_59_709756 | |
| 13:27:04 | efried | stephenfin: ^ | |
| 13:27:41 | stephenfin | jaypipes: If you've time, mind glancing over the cpu-resources spec to make sure I haven't cannibalized it too much? https://review.opendev.org/#/c/555081 | |
| 13:28:06 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova-specs master: Add "CPU selection with hypervisor consideration" spec https://review.opendev.org/645814 | |
| 13:28:25 | openstackgerrit | Eric Fried proposed openstack/os-traits master: Document policy of never removing traits https://review.opendev.org/655673 | |
| 13:29:01 | efried | aspiers: FTFY and fast approved ^ | |
| 13:29:15 | stephenfin | efried, mriedem: I saw that but I didn't see the link to the failing tests. I'd have expected to have seen the name of the failing test reported before stacktrace | |
| 13:29:25 | kashyap | efried: Meanwhile, in your copious free time ^ As best as I can addressed remarks from mriedem, sean-k-mooney et al. If you want to fast-approve. | |
| 13:29:42 | efried | stephenfin: It varies which test fails | |
| 13:30:04 | kashyap | (Also I will add / update the spec as I learn other important details, docs while implenenting.) | |
| 13:30:16 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Microversion 2.73: Support adding the reason behind a server lock https://review.opendev.org/648662 | |
| 13:30:25 | efried | jeebuz, with that big a delta, I doubt I'll be able to fast approve | |
| 13:30:39 | efried | but thanks for updating. | |
| 13:30:59 | kashyap | efried: Nothing fundamental has changed. I largely added clarifying text | |
| 13:31:28 | kashyap | efried: Or removed unnecessary bit. The main work mentioned in the Work Items is still in tact | |
| 13:31:48 | openstackgerrit | Balazs Gibizer proposed openstack/nova-specs master: Server move operations with ports having resource request https://review.opendev.org/652608 | |
| 13:32:13 | lyarwood | mriedem: ack I'll take a look now | |
| 13:33:05 | mriedem | efried: want to continue the backport train? https://review.opendev.org/#/c/655429/ | |
| 13:33:08 | jaypipes | stephenfin: ack. will do that right now. | |
| 13:33:22 | efried | kashyap: Okay, you've said nova will validate, but I'm still not seeing the part where you say: "...and will cause {spawn to fail | nova-compute to refuse to start}, with a helpful log message indicating which combination of thingies was bad" | |
| 13:33:27 | jaypipes | efried: no, not a huge prio for me. | |
| 13:33:29 | stephenfin | ta | |
| 13:33:32 | efried | jaypipes: ack, thx | |
| 13:33:41 | efried | mriedem: Yes, need to do it manually, will get to it asap. | |
| 13:33:55 | kashyap | efried: Okay, I used the word "validate" too loseely then. Let me add that phrase in. | |
| 13:34:11 | efried | right, I (as the reader) don't know what that means. | |
| 13:34:20 | kashyap | (The word "validate" was supposed to capture that. And I know I will uncover more details as I mull on the implementation.) | |
| 13:35:03 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif stable/rocky: Prevent "qbr" Linux Bridge from replying to ARP messages https://review.opendev.org/655692 | |
| 13:35:42 | gibi | efried: I saw the discussion about the lenght of the bandwith PTG slot | |
| 13:35:55 | gibi | efried: I think we can shrink that to 15-20 minutes | |
| 13:35:56 | efried | gibi: ack, I saw your response, will shrink. | |
| 13:35:59 | efried | thanks. | |
| 13:36:01 | gibi | efried: thanks | |
| 13:36:41 | gibi | efried: I also saw that you prepared the commit for the onboarding session. Is there anything I can still help with? | |
| 13:38:57 | openstackgerrit | Eric Fried proposed openstack/nova stable/rocky: Fix {min|max}_version in ironic Adapter setup https://review.opendev.org/655693 | |
| 13:39:10 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif stable/queens: Prevent "qbr" Linux Bridge from replying to ARP messages https://review.opendev.org/655694 | |