Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-01-02
13:38:19 kashyap Where you see a couple of: "TODO / Need Opinions" sections
13:38:19 jaypipes kashyap: yep, saw that in my email this morning... will look later today when I'm properly caffeinated.
13:38:50 kashyap jaypipes: Sure, caffine, very important. Or else, it might irrirate the delicate neurons.
13:38:53 kashyap Thanks!
13:39:01 jaypipes np :)
13:40:03 openstackgerrit Stephen Finucane proposed openstack/nova master: Add an online migration for PciDevice.uuid https://review.openstack.org/530487
13:47:13 efried Good morning nova
13:48:46 cdent welcome back efried. apologies but I utterly failed to do much with regard to the stuff I said I would do while you were away, sickness and other stuff got in the way. I believe stephenfin manage to keep some of your stuff up to date, though
13:50:17 efried cdent No worries, hope you're feeling better. I saw some rebasing and minor updates by stephenfin and others, but actually haven't logged in at all since last Sunday - so I have some catching up to do.
13:50:51 cdent a) good on you for staying away, b) any urgency is false, c) still coughing, but not in bed
13:51:05 efried Wow, sounds like a doozy
13:51:06 efried Any idea who's still going to be out this week?
13:52:06 jaypipes efried: welcome back! :)
13:52:15 efried jaypipes Thanks!
13:53:15 openstackgerrit Stephen Finucane proposed openstack/nova stable/pike: Fix disk size during live migration with disk over-commit https://review.openstack.org/530743
13:53:30 openstackgerrit Stephen Finucane proposed openstack/nova stable/ocata: Fix disk size during live migration with disk over-commit https://review.openstack.org/530744
13:54:26 efried Lovely, 17 of my open patches are in merge conflict.
13:54:46 jaypipes efried, cdent: I'm going to grab some breakfast here and will be back in around an hour, but I'd love to have a google hangout with you guys (and edleafe/alex_xu and anyone else placement-focused) when I return.
13:55:01 efried jaypipes Sounds good to me.
13:55:26 cdent jaypipes: that sounds like a great idea, but I'm in public coffee land, so a bit later in the day would be better for me
13:55:38 jaypipes cdent: gotcha, np
14:23:51 stephenfin kashyap: I assume you fixed the problem with https://review.openstack.org/#/c/506720/ ? I can't see anything wrong with it now
14:24:01 kashyap stephenfin: Is it the "Upgrade Impact" thing?
14:24:09 kashyap _Just_ looking at the error
14:24:12 kashyap http://logs.openstack.org/20/506720/4/check/openstack-tox-pep8/1f31318/job-output.txt.gz
14:24:29 kashyap 2018-01-02 13:45:19.289008 | ubuntu-xenial | AssertionError: While checking 'specs/rocky/approved/ovmf-with-secure-boot-and-smm.rst':
14:24:32 kashyap 2018-01-02 13:45:19.289047 | ubuntu-xenial | Section 'Proposed change' is missing subsections: [u'Upgrade impact']
14:24:44 stephenfin Oh, the pep8 job. Yeah, I guess so
14:25:06 stephenfin But the grey blocks you were on about earlier: I can't see anything wrong in that regard
14:25:37 kashyap Hmm
14:25:48 kashyap I'll just leave it at that for now.
14:29:01 jroll jaypipes: I may or may not be here (have an appt in an hour), but would like a ping for that hangout if it happens
14:32:49 openstackgerrit Kashyap Chamarthy proposed openstack/nova-specs master: Add UEFI Secure Boot support for QEMU/KVM guests, using OVMF https://review.openstack.org/506720
14:35:16 alex_xu jaypipes: yea, i still wake up \o/
14:36:16 cdent jroll, jaypipes, alex_xu if it needs to happen sooner than later so as to include more folk, I'm happy to listen in and speak rarely
14:36:28 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Use volume shared_targets to lock during attach/detach https://review.openstack.org/529695
14:36:28 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Add support for getting volume details with a specified microversion https://review.openstack.org/529656
14:36:54 alex_xu cdent: jaypipes, no, just pick the time good for you guys, i'm really close to sleep time :)
14:38:37 jroll cdent: you're more involved than me, I'm still trying to catch back up on things. I should be back by 1600ish anyway.
15:15:53 jaypipes alex_xu: ok, no prob. will fill you in tomorrow if we have a hangout.
15:24:44 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: Add support for the 2.57 microversion https://review.openstack.org/528128
15:24:45 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: Remove incorrect legacy QuotaSet.id property https://review.openstack.org/530757
15:26:21 jaypipes kashyap: tox -epep8, sir. ;)
15:26:32 jaypipes well, good morning mriedem.
15:26:48 mriedem hi
15:27:41 kashyap jaypipes: Yeah, I normally run - `tox -epep8 -- -HEAD`; seems it got commented out in my shell :P
15:28:01 mriedem tox -e fast8
15:32:20 edleafe jaypipes: just saw your message. I'm currently consumed with my getting set up in my Austin office, after being forced by IBM to commute here instead of, you know, being productive
15:32:35 jaypipes edleafe: :(
15:32:40 jaypipes edleafe: that sux.
15:34:47 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: Microversion 2.58 - Instance actions list pagination https://review.openstack.org/528601
15:35:00 edleafe jaypipes: this will also mean that we will need someone to run the scheduler subteam meetings, as I'll be on the road at that hour
15:35:15 jaypipes edleafe: ack.
15:35:34 jaypipes edleafe: or you could tell your management chain to stuff it and let you be productive.
15:42:33 edleafe jaypipes: I'd love to (and sort of have already), but it's dictated by someone clueless several levels above me.
15:43:16 edleafe jaypipes: I've been trying to land a new gig, but OpenStack isn't the hiring hotness it was a few short years ago
15:43:24 dhellmann hey folks, is this patch to add whereto for testing your doc redirects of any interest? or should I drop it? https://review.openstack.org/516385
15:45:29 cdent (i'm excluding myself because that plus the friday report would kill me)
15:45:35 stephenfin dhellmann: sdague would normally be your man but he's not around. Maybe mriedem
15:46:22 dhellmann stephenfin : yeah, I added sdague to the review since he's the one that threw the fit that drove us to add support for this in the first place.
15:46:26 efried cdent I wouldn't necessarily have a problem doing that; but IMO we should move the meeting to a time where edleafe can at least attend, if not continue to run it.
15:46:42 cdent also true
15:47:04 cdent I think part of the problem is that 4 hours of edleafe's day is being eaten by stupidity
15:47:11 cdent so scheduling around him may be challenging
15:47:14 edleafe cdent: nailed it
15:47:43 mdbooth Is the gate unhappy today?
15:47:52 jaypipes edleafe: there's always k8s.
15:48:14 mdbooth jaypipes: Thanks for all those reviews on my patch series, btw.
15:48:27 jaypipes mdbooth: have they merged yet?
15:48:32 mdbooth 2 landed, currently trying to corral 3 more.
15:48:34 edleafe jaypipes: heh, I spent a lot of my holiday time improving my golang skills
15:48:37 jaypipes mdbooth: k
15:48:52 jaypipes edleafe: a good idea.
15:49:17 mdbooth edleafe: Thoughts on golang?
15:50:34 edleafe mdbooth: it's a perfectly fine language. The main issue isn't syntax, of course, but learning the proper way of doing things. 15 years of Pythonic thinking is a bit difficult to overcome. :)
15:50:47 mriedem mdbooth: might have a problem here https://review.openstack.org/#/c/525599/
15:51:32 mdbooth mriedem: I'll regret pointing this out, but that's in the gate right now.
15:51:41 mriedem mdbooth: put a fix on top
15:51:49 mdbooth mriedem: Thanks :)
15:51:52 mriedem should be easy to adjust your test as well to use a deleted bdm
15:53:44 mdbooth mriedem: Ah, actually I think that was intentional because otherwise we can't but a not null constraint on the column subsequently.
15:54:13 mriedem so we can load a deleted record and update it?
15:54:24 mriedem you should at least have a wrinkle in your test showing this is ok
15:54:43 mozart89 Hello. Is it necessary to use Numa Topology ?
15:54:43 mdbooth mriedem: That is an excellent point. That should at least be tested.
15:54:43 mriedem also, another reason we don't have constraints on these things at first
15:54:49 mdbooth And I wouldn't be surprised to find it's broken.
15:54:51 mriedem mozart89: no
15:55:01 kashyap Oops, the 'i' in "Performance impact" _have_ to be capital; or else pep8 fails. /me corrected it out of OCD, and realized the template itself was like htat
15:55:43 mdbooth There's no constraint on it right now. The goal is to put a constraint on it in the first release where we positively assert that this migration has already run.
15:56:01 mdbooth i.e. if it fails, it's the operator's fault.
15:56:41 mriedem i was thinking of the unique constraint https://review.openstack.org/#/c/242602/28/nova/db/sqlalchemy/migrate_repo/versions/374_bdm_uuid.py
15:56:45 mriedem but i see uuid is nullable=True
15:56:51 mriedem which is the constraint you're talking about
15:57:21 mriedem the BlockDeviceMapping.uuid is not nullable though
15:57:24 mriedem which is probably good enough
15:58:26 mdbooth mriedem: Ah, actually this is still ok because it never loads the object. This is purely a db operation. I'll still add a test for it, if for no other reason than to make it obvious it's intentional.
15:59:32 mozart89 mriedem: I'm not using any NUMA Topology ( CPU pinning and e.t.c). But when i'm triggering "live migration" it stops with message "Require both a host and instance NUMA topology to fit instance on host".
16:00:24 mriedem mozart89: that's just a debug message isn't it?
16:00:55 mriedem mozart89: that's just a debug message that is output when the NUMATopologyFilter runs

Earlier   Later