| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-01-11 | |||
| 06:52:49 | openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Create specs directory for Train https://review.openstack.org/630037 | |
| 09:15:49 | kashyap | Can I get a second reviewer who can +W this doc patch? -- https://review.openstack.org/#/c/629627/ ("docs: Secure live migration with QEMU-native TLS") | |
| 09:15:55 | kashyap | Happy to trade a few reviews in exchange :-) | |
| 09:17:40 | kashyap | IMHO, the last couple of -1 nits from Takashi are just too trivial to deserve a respin. | |
| 09:20:56 | kashyap | sean-k-mooney: Thanks for the review, and the 'recheck' while I was asleep :-) | |
| 09:29:49 | openstackgerrit | 98k proposed openstack/gantt master: Updatae home-page https://review.openstack.org/630096 | |
| 10:20:46 | stephenfin | What the heck if gantt and why is the openstackgerrit bot logging it here? | |
| 10:21:13 | stephenfin | ah, dead project | |
| 10:44:10 | openstackgerrit | Stephen Finucane proposed openstack/gantt master: Retire gantt https://review.openstack.org/630138 | |
| 11:19:09 | openstackgerrit | Stephen Finucane proposed openstack/python-ganttclient master: Retire python-ganttclient https://review.openstack.org/630154 | |
| 11:32:07 | stephenfin | melwitt: When you're about, are you OK with me sending out a retirement email for these two projects? http://github.com/openstack/gantt http://github.com/openstack/python-ganttclient | |
| 11:34:26 | stephenfin | melwitt: Alternatively, if you've more context, feel free to do it yourself. I have the patches already proposed against the various repos https://review.openstack.org/630162 | |
| 11:42:57 | openstackgerrit | Merged openstack/nova master: libvirt: Support native TLS for migration and disks over NBD https://review.openstack.org/625216 | |
| 11:53:43 | kashyap | stephenfin: gibi: Since the above is merged, can anyone give +W for the accompanying: https://review.openstack.org/#/c/629627/ | |
| 11:54:13 | stephenfin | kashyap: Needs another +2 before I can do that, I'm afraid | |
| 11:54:23 | kashyap | Yeah, that's why I was asking Gibi :-) | |
| 11:54:26 | stephenfin | ack | |
| 11:54:26 | kashyap | Thanks for your review, already! | |
| 11:56:40 | gibi | kashyap: it is on my list :) | |
| 11:56:50 | kashyap | Most excellent; thanks for your time. | |
| 12:09:10 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: docs: Secure live migration with QEMU-native TLS https://review.openstack.org/629627 | |
| 12:09:11 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: docs: Address minor comments for the QEMU-native TLS document https://review.openstack.org/630183 | |
| 12:10:40 | kashyap | stephenfin: When you can, does the last patch look OK to you? | |
| 12:10:56 | kashyap | Which addresses the Takashi's remark | |
| 12:11:06 | stephenfin | kashyap: are the TripleO docs versioned? | |
| 12:11:20 | kashyap | Oh, I don't know | |
| 12:11:26 | kashyap | You said "this comment is invalid" | |
| 12:11:31 | stephenfin | yeah, I left comments | |
| 12:11:40 | kashyap | Who's comment is that? Takashi's or my response to him? | |
| 12:11:47 | kashyap | Just read your comment | |
| 12:11:47 | stephenfin | Takashi's | |
| 12:11:51 | stephenfin | Build it locally and see if the docs link through | |
| 12:11:56 | stephenfin | I suspect they won't. | |
| 12:12:02 | stephenfin | If they do, it's all good | |
| 12:12:22 | kashyap | I built it locally; and it works | |
| 12:12:41 | kashyap | As in, it renders the linked TLS TripleO doc correctly | |
| 12:12:52 | stephenfin | ah yeah, that's because it links to latest | |
| 12:12:59 | stephenfin | but there's no e.g. rocky docs, I think | |
| 12:13:00 | stephenfin | https://docs.openstack.org/tripleo-docs/latest/ | |
| 12:13:08 | kashyap | Ah-ha | |
| 12:13:13 | stephenfin | https://docs.openstack.org/nova/rocky/ is valid | |
| 12:13:23 | stephenfin | https://docs.openstack.org/tripleo-docs/rocky/ is not | |
| 12:13:52 | stephenfin | So maybe drop that bit and add a NOTE in place of the conf.py change noting that TripleO docs aren't versioned? | |
| 12:13:54 | kashyap | Okay, then I'll remove that bit of change, good that I did a follow-up, instead of messing with the current one | |
| 12:14:00 | stephenfin | Just to prevent confusion in the future | |
| 12:14:21 | kashyap | Yeah, I'll drop that bit | |
| 12:14:31 | kashyap | stephenfin: Where would you want me to add the note? In the conf.py? | |
| 12:14:38 | stephenfin | ideally, yes | |
| 12:14:46 | stephenfin | just above that openstack_projects option | |
| 12:15:08 | kashyap | Yep, doing that | |
| 12:22:43 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: docs: Address minor comments for the QEMU-native TLS document https://review.openstack.org/630183 | |
| 12:24:30 | kashyap | Alright, I've addressed all things pointed out by Takashi in the follow-up. | |
| 12:27:24 | stephenfin | lyarwood: Think you could take a look at this today? https://review.openstack.org/#/c/626218/ | |
| 12:27:43 | lyarwood | stephenfin: ack sure thing | |
| 13:07:43 | jaypipes | stephenfin, bauzas, sean-k-mooney: I'm fed up with the cpu resource tracking spec. It's just analysis paralysis at this point, and I'm tired of retreading the same points about CONF.vcpu_pin_set over and over. Since at this point, folks from Intel and RH are really the only ones that seem to understand how any of the NUMA and CPU topology stuff "should" work, it's probably best to have an Intel or RH employee try to get this over the finish | |
| 13:07:43 | jaypipes | line. | |
| 13:08:10 | jaypipes | I've essentially given up trying to refactor any of the poo in Nova any more. | |
| 13:08:21 | jaypipes | just seems like a completely wasted effort on my part. | |
| 13:11:54 | cdent | it does seem a bit paralyzed | |
| 13:12:16 | kashyap | jaypipes: Mind posting the spec link? Your comment makes me curious | |
| 13:14:01 | cdent | kashyap: https://review.openstack.org/#/c/555081/ | |
| 13:15:45 | kashyap | Thanks | |
| 13:16:40 | kashyap | Oh dear, look at that scrollbar. | |
| 14:11:40 | stephenfin | jaypipes: Yeah, that's fair. The thought that maybe we'd be better of debating this stuff during code did cross my mind yesterday, but I'd been hoping to agree that on the spec | |
| 14:12:25 | stephenfin | jaypipes: As-is, I'm not sure how to make it any easier. It is a load of poo, as you rightly point out, and the nest of dependencies is somewhat ridiculous | |
| 14:13:04 | openstackgerrit | Paul-Emile Element proposed openstack/os-traits master: Add new traits for TPM support https://review.openstack.org/629299 | |
| 14:14:17 | mnaser | good morning everyone. i need to rename a bunch of hostnames in a cloud, which can go very bad in the past | |
| 14:14:29 | mnaser | do we have a documented procedure or should i just evacuate hosts and reboot them with a new host and call it a day | |
| 14:25:42 | jaypipes | mnaser: no idea what the best practice is but will eagerly consume others' suggestions. | |
| 14:27:39 | mnaser | because the hypervisor host seems to be engrained in so many places like instances table, placement db | |
| 14:27:54 | mnaser | and i know before if the hostname changed and nova-compute restarted it would self destruct :) | |
| 14:29:33 | cdent | mnaser: I think unless you do some hand holding, placement will with orphaned inventory, throwing things out of wack. I don't think that has changed. But it is hard to keep track | |
| 14:30:08 | mnaser | i think i'll go the evacuate host/change hostname/reboot route | |
| 14:30:13 | mnaser | slower but probably reliable i guess | |
| 14:35:26 | openstackgerrit | Merged openstack/os-traits master: Add new traits for TPM support https://review.openstack.org/629299 | |
| 14:44:36 | temka | stephenfin, docmaster, an easy review for you: https://review.openstack.org/#/c/629904/ | |
| 14:45:39 | finucannot | temka: thanks for the reminder | |
| 14:46:23 | finucannot | temka: done | |
| 14:46:31 | temka | finucannot, cheers! | |
| 14:48:29 | finucannot | jaypipes: If you're not totally burnt by NUMA'y stuff, I'd appreciate some eyes on this bugfix https://review.openstack.org/#/c/629281/ | |
| 14:53:56 | cdent | dansmith, jaypipes, gibi, melwitt: does wednesday at 5utc seems a reasonable time to have the placement check in meeting? Or should we wait until monday and mriedem being back to pick a real time | |
| 14:56:14 | dansmith | cdent: do you mean 1700UTC? | |
| 14:56:38 | cdent | dansmith: yes | |
| 14:56:45 | cdent | sorry for being ambiguos | |
| 14:57:42 | dansmith | works for me time wise, but obviously not sure what matt might have on his calendar | |
| 14:59:12 | cdent | i'll suggest wednesday in the pupdate, and we can firm up the actual time on monday | |
| 14:59:13 | gibi | cdent: works for me | |
| 14:59:40 | cdent | gibi: the bandwith thing is on tuesday, yeah? | |
| 14:59:46 | gibi | cdent: the bandwidth hangouts is currently scheduled for tuesday 17:00 utc, but I got feedback from rubasov that it does not work for him, so I thinking about waiting for mriedem's input before move the bandwidth hangouts again. | |
| 14:59:57 | cdent | ah | |
| 15:00:10 | gibi | the joy of finding a good timeslot :) | |
| 15:20:14 | melwitt | finucannot: yeah, wow, I had no idea gantt was still around in gerrit etc. feel free to email | |
| 15:22:09 | melwitt | cdent: sounds good, thanks | |
| 16:11:45 | devfaz | Hi, is there a easy way to (re)create the tap-devices of running instances if the openvswitch.db got cleared? I already restarted openvswitch-agent and this restored all other interfaces, just the tap-devices for nova are missing? | |
| 16:23:21 | devfaz | so found a way "ip link del qvb-XXX" and restart openvswitch-agent. Is this a "good" workaround or is there a better way to restore the network if ovs lost its settings? | |
| 16:43:32 | openstackgerrit | Merged openstack/nova master: Doc: rebuild can result in SHUTOFF VM state https://review.openstack.org/629904 | |
| 16:47:07 | finucannot | melwitt: Sweet, done. Guess we need bauzas or johnthetubaguy to unilaterally merge these patches since they're the only active folks in gantt-core https://review.openstack.org/#/c/630154/ https://review.openstack.org/#/c/630138/ | |
| 16:47:45 | melwitt | ack, thanks | |
| 16:56:54 | gibi | kashyap: I have some nits in https://review.openstack.org/#/c/629627 If you ack that you can do them in the followup then can +W that patch | |
| 16:59:40 | openstackgerrit | Jack Ding proposed openstack/nova master: [WIP] Flavor extra spec and image properties validation https://review.openstack.org/620706 | |