| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-05-24 | |||
| 08:38:50 | mdbooth | lyarwood: See: file, http, rbd, swift, sheepdog, cinder, vmware | |
| 08:42:07 | openstackgerrit | Boxiang Zhu proposed openstack/nova master: Add valid compute nodes with host and/or node in api layer https://review.opendev.org/661237 | |
| 08:53:00 | lyarwood | mdbooth: right, I just don't think we need yet another lib for this | |
| 10:08:34 | openstackgerrit | Silvan Kaiser proposed openstack/nova stable/stein: Added mount fstype based validation of Quobyte mounts https://review.opendev.org/660706 | |
| 10:23:32 | openstackgerrit | Merged openstack/nova stable/stein: Fix target used in nova.policy.check_is_admin https://review.opendev.org/660330 | |
| 10:23:45 | openstackgerrit | Merged openstack/nova stable/queens: Define irrelevant-files for tempest-full-py3 job https://review.opendev.org/650460 | |
| 10:23:52 | openstackgerrit | Merged openstack/nova stable/queens: Error out migration when confirm_resize fails https://review.opendev.org/652150 | |
| 10:32:37 | openstackgerrit | Merged openstack/nova stable/queens: Teardown networking when rolling back live migration even if shared disk https://review.opendev.org/658149 | |
| 10:32:47 | openstackgerrit | Merged openstack/nova master: Link versioned notification talk into docs https://review.opendev.org/661115 | |
| 10:57:51 | openstackgerrit | Adam Spiers proposed openstack/nova master: Move patch_exists() to nova.test.TestCase for reuse https://review.opendev.org/660500 | |
| 11:07:42 | openstackgerrit | Adam Spiers proposed openstack/nova master: Move patch_exists() to nova.test.TestCase for reuse https://review.opendev.org/660500 | |
| 11:39:15 | lyarwood | has anyone ever seen LM try to claim resources on the source at the start of a migration? | |
| 11:39:36 | lyarwood | IHAC suggesting that's happening but that seems totally wrong | |
| 12:03:59 | openstackgerrit | Adam Spiers proposed openstack/nova master: Move patch_exists() to nova.test.TestCase for reuse https://review.opendev.org/660500 | |
| 12:09:02 | efried | stephenfin: in case you didn't see http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2019-05-24.log.html#t2019-05-24T06:31:39 | |
| 12:09:41 | openstackgerrit | Adam Spiers proposed openstack/nova master: Move patch_exists() to nova.test.TestCase for reuse https://review.opendev.org/660500 | |
| 12:15:18 | aspiers | god sometimes Python really pisses me off | |
| 12:16:45 | aspiers | Python 2/3 unicode stuff is a never-ending nightmare | |
| 12:17:05 | aspiers | I'm literally seeing heisenbugs with it | |
| 12:17:18 | aspiers | one minute everything's fine, the next it magically breaks | |
| 12:17:23 | cdent | i've never met a language that does it in a way that makes me happy. python is one of many that make me want to flip tables | |
| 12:17:59 | cdent | aspiers: bugs on your own machine or remote nodes? if the latter there are so many variables involved (system encoding, python build, blah blah) | |
| 12:18:08 | aspiers | local | |
| 12:18:32 | aspiers | I'm on the verge | |
| 12:18:54 | aspiers | I just want to slurp in the contents of __file__ (sounds weird I know, but there is a good reason) and then do a substring match on it | |
| 12:19:28 | aspiers | if I try to decode('utf-8') on each line I get UnicodeDecodeError: 'utf8' codec can't decode byte 0xf3 in position 1: unexpected end of data | |
| 12:19:54 | aspiers | despite it being an ASCII file (nova/nova/tests/unit/test_test.py) | |
| 12:20:34 | aspiers | ascii is no better: UnicodeDecodeError: 'ascii' codec can't decode byte 0xf3 in position 1: ordinal not in range(128) | |
| 12:24:34 | aspiers | WTF!! I literally ran the test twice in a row and it only failed the second time!! | |
| 12:27:35 | aspiers | OK, I've spotted a pattern | |
| 12:27:41 | aspiers | It always passes on py36 | |
| 12:28:02 | aspiers | It *only* passes on py27 if I run it after editing and saving the file | |
| 12:28:11 | aspiers | If I re-run it, subsequent runs fail | |
| 12:28:39 | aspiers | WT actual F | |
| 12:31:31 | cdent | if __file__ always the .py file and never the .pyc file? | |
| 12:31:38 | aspiers | ohhhhhhh | |
| 12:31:39 | cdent | s/if/is/ | |
| 12:31:42 | aspiers | you genius | |
| 12:31:44 | aspiers | that will be it | |
| 12:31:50 | cdent | it was a guess | |
| 12:32:01 | aspiers | that would perfectly explain it though | |
| 12:32:06 | cdent | my genius is never expecting things to be sensible | |
| 12:32:10 | aspiers | haha | |
| 12:32:19 | cdent | "what if it's this totally stupid thing that shouldn't happen" | |
| 12:32:20 | aspiers | my folly is always expecting things to be sensible | |
| 12:32:36 | cdent | how long have you been in this business ? :) | |
| 12:32:38 | aspiers | (except in politics. I'm not *that* stupid) | |
| 12:35:21 | aspiers | cdent: works! __file__.rstrip('c') | |
| 12:35:32 | cdent | wacky | |
| 12:35:44 | aspiers | you just saved me bashing my head over this for hours - THANK YOU | |
| 12:35:53 | cdent | you're very welcome | |
| 12:36:12 | aspiers | no amount of rubber-ducking would have helped on that one, I fear | |
| 12:36:49 | aspiers | :) | |
| 12:38:30 | openstackgerrit | Adam Spiers proposed openstack/nova master: Move selective patching of open() to nova.test for reuse https://review.opendev.org/661266 | |
| 12:38:47 | aspiers | cdent: ^^^ | |
| 12:39:02 | aspiers | BTW I think I fixed the contextlib issue in the parent of that one | |
| 12:41:17 | cdent | \o/ | |
| 12:51:53 | efried | tonyb: done | |
| 12:54:59 | aspiers | cdent: just noticed some functional tests are calling os.path.exists('/etc/policy.d') | |
| 12:55:13 | aspiers | presumably that's not good in a functional test environment | |
| 12:55:42 | cdent | There's a few bleeds like that in the nova functional tests | |
| 12:55:59 | aspiers | also .netrc, yikes | |
| 12:56:14 | cdent | I remember there were some (since fixed) I think, which would be change behavior by using the /etc/nova/nova.conf | |
| 12:58:38 | aspiers | my new helpers might help clean some of those cases up | |
| 13:06:45 | openstackgerrit | Silvan Kaiser proposed openstack/nova stable/stein: Fixes multi-registry config in Quobyte driver https://review.opendev.org/660706 | |
| 13:24:10 | openstackgerrit | Eric Fried proposed openstack/nova master: Clarify --before help text in nova manage https://review.opendev.org/661289 | |
| 13:29:52 | stephenfin | efried: I did not. Thanks for the heads up :) | |
| 13:35:45 | stephenfin | efried: So TripleO is being handled (mschuppert is on the case) and I'm working through the Kolla change. That's pretty much all we need, right? | |
| 13:37:50 | efried | stephenfin: IIUC from yesterday's meeting, yes. | |
| 13:37:55 | lyarwood | mriedem: prior to https://review.opendev.org/#/q/topic:bug/1469179+(status:open+OR+status:merged) the only real way to avoid over allocation issues when booting from volumes was to use flavors with a disk size of 0 right? | |
| 13:38:06 | efried | stephenfin: Assume you caught up on those logs (which prompted your ML thread)? | |
| 13:49:24 | mriedem | lyarwood: sounds right | |
| 13:49:46 | openstackgerrit | Dan Smith proposed openstack/nova master: Make nova-next archive using --before https://review.opendev.org/661002 | |
| 13:50:24 | mriedem | can i get a novaclient core to review https://review.opendev.org/#/c/659886/ since it's holding up a release? both tssurya and i have things in the upcoming novaclient release that are needed for enabling changes in other projects (osc and watcher) | |
| 13:55:31 | efried | dansmith: | |
| 13:55:31 | efried | efried@efried-ThinkPad-W520:~/openstack/nova$ f() { | |
| 13:55:31 | efried | > echo $1 | |
| 13:55:31 | efried | > } | |
| 13:55:31 | efried | efried@efried-ThinkPad-W520:~/openstack/nova$ f $(date) | |
| 13:55:31 | efried | Fri | |
| 13:55:31 | efried | efried@efried-ThinkPad-W520:~/openstack/nova$ f "$(date)" | |
| 13:55:32 | efried | Fri May 24 08:53:45 CDT 2019 | |
| 13:55:57 | efried | unless argparse is doing something magical... | |
| 13:56:03 | dansmith | efried: what? | |
| 13:56:16 | efried | see comment on https://review.opendev.org/661002 | |
| 13:56:52 | aspiers | Yeah, that comment looks right | |
| 13:56:57 | dansmith | oh yeah | |
| 13:57:22 | openstackgerrit | Dan Smith proposed openstack/nova master: Make nova-next archive using --before https://review.opendev.org/661002 | |
| 13:57:26 | aspiers | efried: BTW I can't believe you are still using a W520 | |
| 13:57:41 | aspiers | I mean, I am too, but only as a VPN server hidden under my desk | |
| 13:58:10 | efried | aspiers: Heh, it was my IBM-provided workstation, did a nice SSD and memory upgrade, it was really stable and solid, so I bought it back when I left. | |
| 13:58:15 | aspiers | Mine weighed a ton and the battery used to last 2 hours on a good day | |
| 13:58:39 | efried | My Intel-provided HP thingy has roughly equivalent specs, weighs about a quarter as much... and performs like a dog. | |
| 13:58:57 | efried | ...and won't run not-Windows. | |
| 13:59:06 | efried | (because of Intel gorp) | |
| 13:59:19 | efried | (without major hoops of fire) | |
| 14:05:33 | openstackgerrit | Silvan Kaiser proposed openstack/nova stable/stein: Fixes multi-registry config in Quobyte driver https://review.opendev.org/660706 | |
| 14:10:14 | mriedem | efried: we might want to see https://review.opendev.org/#/c/661002/ working in action before +2ing :) | |
| 14:11:19 | efried | mriedem: It worked at a previous PS | |
| 14:11:22 | efried | as in, didn't blow up | |