Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-12-04
16:44:39 dansmith so doesn't affect curl
16:44:57 efried dansmith cdent I have solutions that *always* or *never* strip the ANSI colors; what I'd like to get to eventually is a way for you to curl download with a querystring that allows you to choose.
16:45:02 dansmith I'm not a hard -2 on the ansi logs, but I'd prefer it if I can get it
16:45:14 dansmith efried: that'd be cool
16:45:40 dansmith efried: just put me a curl link on the top somewhere so I can copy that, kinda like the download links in gerrit
16:46:07 efried dansmith The theming is actually an HTML filter that sits on the apache server and kicks in when the user agent is a browser (or something). Then there's a separate filter that you get when it's a download, that doesn't do the HTML conversion at all.
16:46:11 efried dansmith ++ good call
16:46:29 dansmith efried: ah, okay I dunno how that works, but cool
16:46:34 efried dansmith I usually right-click Save-As the file from the directory view, but a copyable link would be neat.
16:46:49 efried dansmith In case you're interested, the theme-ification patch: https://review.openstack.org/#/c/524744/
16:47:04 dansmith efried: yeah, copyable would be nice, because I wget them to my dev box.. right-click-save doesn't do what I want because it'd be on my local machine instead
16:47:19 efried dansmith I gotcha.
16:48:32 dansmith but yeah, aside from my complaints, looks awesome
16:48:52 dansmith sometimes being able to just switch the theme based on what you're looking for can be nice, and/or what you've been staring at for hours
16:56:28 mriedem jaypipes: easy refactor https://review.openstack.org/#/c/523492/
16:56:34 mriedem in the alt hosts series
16:56:36 artom What happened to nova-specs?
16:56:43 mriedem artom: ?
16:56:45 artom https://review.openstack.org/#/c/490824/ merged October 3rd
16:57:00 artom Latest commit in https://github.com/openstack/nova-specs/commits/master/specs is september 21
16:57:10 mriedem efried: ^ didn't you ask infra about this a week or two ago?
16:57:36 mriedem http://git.openstack.org/cgit/openstack/nova-specs/log/
16:57:45 mriedem the github mirror repo is definitely stale
16:57:55 artom Yeah
16:58:09 artom And maybe bring it up in #infra?
16:58:14 efried mriedem After spec freeze I noticed that they weren't getting published to specs.o.o -- sounds like that's a different issue.
16:58:35 efried ...which was resolved, at least at the time.
16:58:47 clarkb mriedem: artom efried the github mirror being stale is a known issue, the repo is corrupt according to github and won't accept pushes
16:59:01 clarkb if we take a gerrit downtime we can surgery the repo but we haven't had opportunity for that yet
16:59:08 artom clarkb, aha, thanks!
16:59:13 mriedem oh yes
16:59:13 efried artom FWIW, that spec is published: https://specs.openstack.org/openstack/nova-specs/specs/queens/approved/libvirt-qemu-native-luks.html
16:59:17 clarkb its been a low priority because we do host working mirrors elsewhere and github is just barely a thing we support
16:59:33 artom efried, doh, of course!
16:59:33 cfriesen has anyone ever considered an option to "tox -e pep8" that would run it only on the files in the top N commits?
16:59:43 efried cfriesen -- -HEAD
16:59:47 mriedem cfriesen: tox -e fast8
16:59:50 artom clarkb, yep, fair enough, thanks for, err, being aware?
17:00:05 mriedem fast8 is different
17:00:10 mriedem fast8 is pep8 on just the files changed
17:00:13 artom mriedem, I thought we're only at Fast and Furious 6 or 7
17:00:18 efried Ah, so same as -- -HEAD
17:00:18 clarkb with the holidays coming up, things tend to slow down and we can probably more easily sneak in an extended gerrit downtime
17:00:26 mriedem yes
17:00:30 clarkb I'll add it to tomorrows meeting agenda
17:00:37 mriedem because typing 'tox -e pep8 -- -HEAD' was too tedious
17:00:49 openstackgerrit Ed Leafe proposed openstack/nova master: Add Selection objects https://review.openstack.org/499239
17:00:50 cfriesen mriedem: efried: sweet...guess I should check the ini file first. :)
17:00:50 openstackgerrit Ed Leafe proposed openstack/nova master: Return Selection objects from the scheduler driver https://review.openstack.org/495854
17:00:50 openstackgerrit Ed Leafe proposed openstack/nova master: Refactor the code to check for sufficient hosts https://review.openstack.org/520242
17:00:51 openstackgerrit Ed Leafe proposed openstack/nova master: Modify select_destinations() to return objects and alts https://review.openstack.org/510159
17:00:51 openstackgerrit Ed Leafe proposed openstack/nova master: Move the to_dict() method to the Selection object https://review.openstack.org/523492
17:00:52 openstackgerrit Ed Leafe proposed openstack/nova master: Move the claim_resources method to scheduler utils https://review.openstack.org/511357
17:00:52 openstackgerrit Ed Leafe proposed openstack/nova master: Change RPC for select_destinations() https://review.openstack.org/516707
17:00:53 openstackgerrit Ed Leafe proposed openstack/nova master: Make conductor pass and use host_lists https://review.openstack.org/511358
17:00:58 edleafe mriedem: ^^ there ya go
17:01:01 mriedem thanks
17:01:36 efried cfriesen The other thing I do when I'm working on a series is: source .tox/pep8/bin/activate to get into the pep8 virtualenv, and leave that window open. Then from within that window I'll do: git diff --name-only HEAD~1 | grep '\.py$' | xargs flake8
17:01:51 efried cfriesen HEAD~{whatever-number-here} to go back that many patches.
17:02:39 emccormick Hey, if anyone has a sec for a possibly dumb question: Is there a reason that it's required to have the instance type in the domain xml match the instance type in nova.conf?
17:03:25 emccormick I ask because I'm working on migrating backends from local to rbd, and while running instances continue to function after the change to nova.conf, and can also be migrated, they cannot be rebooted.
17:03:44 emccormick just wondering if that's something that could be changed or if it would break things horribly
17:04:24 efried cfriesen The only caveat to the permanent-venv deal is if you need to rebuild the venv for whatever reason; sometimes running via tox will figure that bit out for you without needing to use -r; but sitting in the venv you get nuffink.
17:05:03 edleafe efried: I use aliases to do roughly the same thing
17:05:06 cfriesen efried: that's a useful trick and a good caveat.
17:05:34 efried edleafe Aliases for what, running flake8 in the venv?
17:06:02 openstackgerrit Chris Friesen proposed openstack/nova master: unreserve volumes after scheduling has failed https://review.openstack.org/525253
17:06:17 edleafe efried: like this:
17:06:18 edleafe alias ptt='source .tox/py27/bin/activate; python -m testtools.run'
17:06:28 cfriesen ^ we ran into this while testing Pike, looks like it's still an issue in master
17:06:39 edleafe I have different aliases for functional, py3, etc.
17:06:41 efried edleafe Ah, gotcha.
17:08:34 openstackgerrit Stephen Finucane proposed openstack/nova master: Resolve review comments from I41b03a347 https://review.openstack.org/525255
17:08:34 openstackgerrit Stephen Finucane proposed openstack/nova master: zuul: Move legacy jobs to project https://review.openstack.org/514309
17:08:48 cfriesen The issue that the patch above ^^^ fixes is that if you do boot-from-vol and then fail scheduling, currently the volume remains reserved and is stuck in the "attaching" state.
17:09:50 efried mriedem The _ensure_resource_provider error handling cherry-pick for pike is looking green now (https://review.openstack.org/#/c/524618/) -- will proceed to backport to ocata unless some reason not to?
17:11:50 mriedem go ahead
17:12:26 openstackgerrit Surya Seetharaman proposed openstack/nova master: update_cell allows more than once cell to have the same db/transport url https://review.openstack.org/518556
17:16:23 efried urgh, merge conflict (which I guess isn't surprising). Will hit it after "lunch".
17:26:46 dmsimard kashyap: o/ mriedem said you could help with a bug we're trying to nail down. Have you gotten familiar with https://bugs.launchpad.net/nova/+bug/1735823 ?
17:26:48 openstack Launchpad bug 1735823 in OpenStack Compute (nova) "Nova can hang when creating a VM with disk injection" [Medium,In progress] - Assigned to Matt Riedemann (mriedem)
17:27:35 dmsimard tl;dr, we thought we ran libguestfs with force_tcg for disk injection but we didn't, so in some environments (such as OVH nodepool cloud regions) with broken nested virtualization, this failed horribly
17:28:19 dmsimard We believe the fix is here: https://review.openstack.org/#/c/524748/ -- however, when testing it here: https://review.openstack.org/#/c/524750/, we're seeing increased failures.. and outside of OVH too.
17:29:14 dmsimard Before the fix (at OVH) http://logs.openstack.org/46/523646/1/check/legacy-tempest-dsvm-neutron-full-centos-7/5bf092c/job-output.txt#_2017-11-29_03_02_38_031436 -- After the fix (at RAX): http://logs.openstack.org/50/524750/1/check/legacy-tempest-dsvm-neutron-full-centos-7/a7f051e/job-output.txt.gz#_2017-12-04_15_25_13_211317
17:29:45 dmsimard I did a recheck in https://review.openstack.org/#/c/524750/ to make sure it wasn't a flap and it does look like it breaks things.
17:32:07 stephenfin efried: Interesting little way to speed up tox - 'touch nova.egg-ingo'
17:32:41 stephenfin I read the tox source a while back to figure out why, but it skips the "check if any dependencies are out-of-date" step, which half of what takes so long
17:32:50 cdent mriedem: I’m thinking that https://bugs.launchpad.net/nova/+bug/1734491 may have back port potential?
17:32:51 openstack Launchpad bug 1734491 in OpenStack Compute (nova) "placement keystonemiddleware_authtoken ignores OS_PLACEMENT_CONFIG_DIR" [Undecided,In progress] - Assigned to Chris Dent (cdent)
17:32:57 stephenfin Probably a bug in tox but I haven't figured out why yet
17:36:18 jaypipes efried: let's go with get_provider_uuids() then.
17:38:16 ericyoung Any idea when the patch for ScaleIO ephemeral/image volume support will bubble to the top of the review list? https://review.openstack.org/#/c/407440/
17:40:58 cdent jaypipes: the point on the jsonschema comment was: webob is passing a string to jsonschema, one that looks like an int, but is not an int
17:41:08 cdent just int will _not_ work
17:41:58 jaypipes cdent: if that's the case, then we shouldn't have "integer" in there at all, since we'll never be supplying actual numbers.
17:42:21 cdent unless we change webob out from under it (this has been discussed further up the review stack)
17:42:47 cdent I can make it just string if you like, but the way it is now is to be as flexible yet accurate as possible
17:43:03 cdent it is also what the nova api does
17:43:06 cdent so I copied it
17:44:15 jaypipes cdent: meh...

Earlier   Later