Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-06-25
14:53:54 sean-k-mooney my devstack vms are centos
14:53:55 dansmith *typo, hah
14:54:32 mriedem did you know a cashew is not a nut?
14:54:36 dansmith artom|gmtplus3: like a fancy brazilian walnut laptop, or the *really* fancy "ubunut" laptop
14:54:48 sean-k-mooney :)
14:55:50 sean-k-mooney mriedem: that was most based on the fact that centos finally started to wrok with nested vert although canonica broke it in 18.04 which made me sad
14:55:51 mriedem lyarwood: just so you're aware, stable/stein and rocky are essentially blocked until these devstack changes land https://review.opendev.org/#/q/I0d7f9f2baaee2836a719f199939156bd4f53f778
14:56:24 artom|gmtplus3 Oh, not the whole laptop, just the CPU: Hazwellnut architecture :D
14:56:58 sean-k-mooney haha i would totally by that
14:57:44 lyarwood mriedem: was that behind tempest-slow etc timing out due to the network tests taking ages?
14:58:35 lyarwood ah nvm it doesn't look like it is
15:00:28 mriedem the agent on the tempest-slow jobs blows up
15:01:19 lyarwood ah I see
15:02:46 lyarwood well thanks for the heads up, I've had a tab open to look at that all day, I'll get on with some reviews once that has started to land in stable.
15:04:59 sean-k-mooney dansmith: im always amazed by how many grammer and spelling issue my brain just completely ignores
15:05:46 dansmith sean-k-mooney: and they make it really difficult for me to concentrate on the content.. diff'rent strokes.
15:07:28 sean-k-mooney ya, it just show how differently we read. not that you have pointed them out they are obvious but without your commets my brain just fills in the gaps/corrects them on the fly unless i force myself to read word by word really slowly
15:08:01 artom|gmtplus3 Yeah, there's a "severity" boundary where it switches from "distracting but harmless" to "active sabotage"
15:10:14 jangutter artom|gmtplus3: the late, great Iain M Banks had one of the most frustratingly good stories that tripped that balance in "Feersum Endjinn"
15:10:43 mriedem yay everything i had in the gate last night failed with RETRY_LIMIT and now i get to recheck everything
15:10:53 artom|gmtplus3 jangutter, will read
15:12:16 sean-k-mooney jangutter: im kind of tempted to read that
15:14:45 jangutter I think he's much more known for the Culture work, but FE was a good diversion. Just be warned that some chapters are physically hard to read, intentionally.
15:15:14 sean-k-mooney i have no issue reading "Woak up. Got dresd. Had brekfast. Spoke wif Ergates thi ant who sed itz juss been wurk wurk wurk 4 u lately master Bascule, Y dont u ½ a holiday? & I agreed & that woz how we decided we otter go 2 c Mr Zoliparia in thi I-ball ov thi gargoyle Rosbrith. "
15:19:32 efried mriedem: would you please give this a quick look for sanity https://review.opendev.org/655985 before I invest in test?
15:23:09 artom|gmtplus3 sean-k-mooney, your brain is weird, I can barely parse that
15:29:08 sean-k-mooney artom|gmtplus3: "Woke up. Got dressed. Had breakfast. spoke with Ergates the ant, who said, it's just been work work work for you lately master Bascule, why dont you have a holiday? And i aggred, an that was how we decied we ought to go to see Mr Zoliparia in the eye ball of the gargoyle Rosbrith." :)
15:29:49 artom|gmtplus3 sean-k-mooney, aha, thi = the didn't register at all
15:29:59 artom|gmtplus3 I was thinking "thy", the old English possessive
15:30:39 sean-k-mooney o/
15:34:36 mriedem efried: comments inline
15:42:52 openstackgerrit Matt Riedemann proposed openstack/nova master: (Re-)enable vnc console tests in nova-multi-cell job https://review.opendev.org/667388
15:51:11 openstackgerrit Kashyap Chamarthy proposed openstack/nova stable/stein: libvirt: Rework 'EBUSY' (SIGKILL) error handling code path https://review.opendev.org/667389
16:09:29 openstackgerrit Balazs Gibizer proposed openstack/nova master: Functional reproduce for bug 1833581 https://review.opendev.org/667396
16:09:30 openstack bug 1833581 in OpenStack Compute (nova) "instance stuck in BUILD state if nova-compute is restarted" [Low,In progress] https://launchpad.net/bugs/1833581 - Assigned to Balazs Gibizer (balazs-gibizer)
16:09:30 openstackgerrit Balazs Gibizer proposed openstack/nova master: Error out interrupted builds https://review.opendev.org/666857
16:09:30 openstackgerrit Balazs Gibizer proposed openstack/nova master: Functional reproduce for bug 1833581 https://review.opendev.org/667397
16:11:03 openstackgerrit Balazs Gibizer proposed openstack/nova master: Enhance service restart in functional env https://review.opendev.org/512552
16:11:24 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add functional test coverage for bug 1724172 https://review.opendev.org/512553
16:11:25 openstack bug 1724172 in OpenStack Compute (nova) rocky "Allocation of an evacuated instance is not cleaned on the source host if instance is not defined on the hypervisor" [Medium,Confirmed] https://launchpad.net/bugs/1724172
16:12:06 openstackgerrit Balazs Gibizer proposed openstack/nova master: Functional reproduce for bug 1833581 https://review.opendev.org/667396
16:12:07 openstack bug 1833581 in OpenStack Compute (nova) "instance stuck in BUILD state if nova-compute is restarted" [Low,In progress] https://launchpad.net/bugs/1833581 - Assigned to Balazs Gibizer (balazs-gibizer)
16:13:56 openstackgerrit Stephen Finucane proposed openstack/nova master: doc: Correct issues with 'openstack quota set' commands https://review.opendev.org/667164
16:13:56 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Rewrite quotas documentation https://review.opendev.org/667165
16:14:20 openstackgerrit Balazs Gibizer proposed openstack/nova master: Error out interrupted builds https://review.opendev.org/666857
16:16:31 stephenfin dansmith: Want to take another look through this? https://review.opendev.org/#/c/666629/
16:29:53 dansmith stephenfin: will in a bit
16:30:00 stephenfin no rush
16:31:00 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove no longer required "inner" methods. https://review.opendev.org/655282
16:31:00 openstackgerrit Stephen Finucane proposed openstack/nova master: Privsepify ipv4 forwarding enablement. https://review.opendev.org/635431
16:31:01 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove unused FP device creation and deletion methods. https://review.opendev.org/635433
16:31:01 openstackgerrit Stephen Finucane proposed openstack/nova master: Privsep the ebtables modification code. https://review.opendev.org/635435
16:31:02 openstackgerrit Stephen Finucane proposed openstack/nova master: Move adding vlans to interfaces to privsep. https://review.opendev.org/635436
16:31:02 openstackgerrit Stephen Finucane proposed openstack/nova master: Move iptables rule fetching and setting to privsep. https://review.opendev.org/636508
16:31:03 openstackgerrit Stephen Finucane proposed openstack/nova master: Move dnsmasq restarts to privsep. https://review.opendev.org/639280
16:31:03 openstackgerrit Stephen Finucane proposed openstack/nova master: Move router advertisement daemon restarts to privsep. https://review.opendev.org/639281
16:31:04 openstackgerrit Stephen Finucane proposed openstack/nova master: Move calls to ovs-vsctl to privsep. https://review.opendev.org/639282
16:31:05 openstackgerrit Stephen Finucane proposed openstack/nova master: Move setting of device trust to privsep. https://review.opendev.org/639283
16:31:05 openstackgerrit Stephen Finucane proposed openstack/nova master: Move final bridge commands to privsep. https://review.opendev.org/639580
16:31:05 openstackgerrit Stephen Finucane proposed openstack/nova master: Cleanup the _execute shim in nova/network. https://review.opendev.org/639581
16:31:06 openstackgerrit Stephen Finucane proposed openstack/nova master: We no longer need rootwrap. https://review.opendev.org/554438
16:31:06 openstackgerrit Stephen Finucane proposed openstack/nova master: Cleanup no longer required filters and add a release note. https://review.opendev.org/639826
16:33:21 efried thanks mriedem
16:41:35 mriedem efried: how do you feel about having a mini personal goal of making https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L985-L997 obsolete for in-tree drivers so we could deprecate or remove it in Train - out of tree drivers could be hit so if we wanted to be nice we'd just log a deprecation warning in T and remove in U
16:41:47 mriedem the only in-tree drivers that would hit those paths are xenapi and hyperv,
16:41:53 mriedem the former implements get_inventory,
16:41:59 mriedem the latter implements neither get_inventory or upt
16:43:27 mriedem oh looky here the hyperv out of tree driver has upt implemented https://opendev.org/openstack/compute-hyperv/src/branch/master/compute_hyperv/nova/driver.py#L464
16:44:49 efried mriedem: I'm big in favor of this. At some point I looked into doing it myself, but it (possibly hyperv) was going to be hairier than I was ready to deal with at that time.
16:46:17 efried mriedem: by "personal" you mean me or you? :P
16:46:45 efried stephenfin: I should wait to re-look at rootwrap series, yes?
16:47:14 mriedem efried: as in something we call each other about late on a friday night to talk about yes
16:47:16 efried stephenfin: The 'done' on the ipv4 patch is still forthcoming?
16:47:17 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix update_provider_tree signature in reference docs https://review.opendev.org/667408
16:47:21 stephenfin efried: 2 secs
16:47:31 efried mriedem: ack, I like it.
16:48:08 stephenfin Actually, I want to test this more so I'll leave it til the morning
16:48:46 stephenfin efried: Sort of. I'm going to drop the first patch and then address the open TODOs in the remainder. If you want to take a look at any you haven't reviewed yet though, that could be helpful
16:49:38 efried something weird just happened
16:50:04 stephenfin Ohh, that's interesting
16:50:18 stephenfin I Ctrl + C'd git-review during the upload once I realized I'd more work to do
16:50:40 efried bot didn't notify, and when I look at the review it seems... corrupted.
16:51:23 efried the series isn't showing up on the right
16:51:32 stephenfin Feck it, I'll push it again just to fix that
16:51:46 openstackgerrit Stephen Finucane proposed openstack/nova master: Privsepify ipv4 forwarding enablement. https://review.opendev.org/635431
16:51:47 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove unused FP device creation and deletion methods. https://review.opendev.org/635433
16:51:47 openstackgerrit Stephen Finucane proposed openstack/nova master: Privsep the ebtables modification code. https://review.opendev.org/635435
16:51:48 openstackgerrit Stephen Finucane proposed openstack/nova master: Move adding vlans to interfaces to privsep. https://review.opendev.org/635436
16:51:48 openstackgerrit Stephen Finucane proposed openstack/nova master: Move iptables rule fetching and setting to privsep. https://review.opendev.org/636508
16:51:49 openstackgerrit Stephen Finucane proposed openstack/nova master: Move dnsmasq restarts to privsep. https://review.opendev.org/639280
16:51:49 openstackgerrit Stephen Finucane proposed openstack/nova master: Move router advertisement daemon restarts to privsep. https://review.opendev.org/639281
16:51:50 openstackgerrit Stephen Finucane proposed openstack/nova master: Move calls to ovs-vsctl to privsep. https://review.opendev.org/639282
16:51:50 openstackgerrit Stephen Finucane proposed openstack/nova master: Move setting of device trust to privsep. https://review.opendev.org/639283
16:51:51 openstackgerrit Stephen Finucane proposed openstack/nova master: Move final bridge commands to privsep. https://review.opendev.org/639580
16:51:51 openstackgerrit Stephen Finucane proposed openstack/nova master: Cleanup the _execute shim in nova/network. https://review.opendev.org/639581
16:51:52 openstackgerrit Stephen Finucane proposed openstack/nova master: We no longer need rootwrap. https://review.opendev.org/554438
16:51:52 openstackgerrit Stephen Finucane proposed openstack/nova master: Cleanup no longer required filters and add a release note. https://review.opendev.org/639826
16:53:06 efried that seems to have worked.
16:53:10 stephenfin Indeed
16:53:12 stephenfin and with that

Earlier   Later