| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-05-26 | |||
| 13:31:12 | sdague | I'll do it | |
| 13:31:39 | mriedem | just pushed the reqs change | |
| 13:31:42 | sdague | ok | |
| 13:31:51 | mriedem | https://review.openstack.org/468417 | |
| 13:32:10 | sdague | oh, you want to bump cinderclient as well | |
| 13:32:14 | mriedem | sure | |
| 13:32:14 | sdague | if you want to be strict about it | |
| 13:32:17 | mriedem | i do | |
| 13:33:41 | mriedem | done | |
| 13:33:50 | mriedem | https://review.openstack.org/468419 | |
| 13:36:06 | stvnoyes | mriedem: johnthetubaguy: when you get a minute, please take a look at https://review.openstack.org/#/c/463987/ - cinder v3 migrate update. thanks. | |
| 13:36:09 | dansmith | who is responsible for the bindep stuff? is that fungi? | |
| 13:37:30 | leakypipes | figleaf: you pushing up a new patch for the send instance UUIDs in param changeset? | |
| 13:38:03 | sdague | dansmith: mostly, what's up though? | |
| 13:38:28 | dansmith | sdague: was just trying it on a fresh centos7 and it just does nothing, "file not found" or something.. I can repro here in a sec | |
| 13:38:34 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Send request_id on neutron calls https://review.openstack.org/468356 | |
| 13:38:34 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Send request_id on cinder calls https://review.openstack.org/468355 | |
| 13:38:35 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Send request_id on glance calls https://review.openstack.org/467242 | |
| 13:38:41 | mriedem | sdague: ^ updated with depends-on | |
| 13:38:50 | leakypipes | dansmith: don't you need to find a phone booth to change your outfit in? | |
| 13:38:51 | mriedem | marked neutron and glance ones as wips until the related clients are in g-r | |
| 13:39:07 | dansmith | leakypipes: no, I'm more of a public exposure kinda guy | |
| 13:39:13 | sdague | mriedem: ok, thanks | |
| 13:39:13 | leakypipes | lmao | |
| 13:39:41 | sdague | mriedem: can you squash those 2 g-r changes? | |
| 13:40:04 | sdague | because otherwise we're running about 20 jobs for each and they don't actually even test anything | |
| 13:40:17 | sdague | or, I'll do it | |
| 13:40:19 | mriedem | the cinderclient one could cause someting to fail | |
| 13:40:27 | mriedem | that's why i separated them | |
| 13:42:02 | superdan | sdague: https://pastebin.com/pUjH5vf4 | |
| 13:42:30 | sdague | mriedem: it actually can't | |
| 13:42:45 | sdague | mriedem: it's a test hole, we're always testing with u-c | |
| 13:43:00 | sdague | the g-r non uc changes are just burning nodes | |
| 13:43:30 | mriedem | done | |
| 13:44:06 | fungi | superdan: i'm not really around most of today, but odds are the error you're seeing is because bindep wants the lsb_release utility so it can identify your operating system | |
| 13:44:32 | fungi | it could probably benefit from a patch to produce a more useful error message under that scenario | |
| 13:44:39 | superdan | fungi: okay, lemme see if that's it | |
| 13:45:28 | fungi | it used to be that pretty much all distros preinstalled that by default, but now that the lsb itself has fallen by the wayside distros are starting to omit the tool | |
| 13:45:44 | sdague | mriedem: thanks | |
| 13:46:12 | fungi | well, s/omit/made optional/ | |
| 13:46:51 | superdan | fungi: and I'm running on a super stripped "cloud-ready" install of centos | |
| 13:46:57 | fungi | aha | |
| 13:46:59 | superdan | missing all kinds of stuff I would normally expect | |
| 13:47:11 | superdan | the desire to omit wget these days is super annoying | |
| 13:47:19 | fungi | more than anything i hate when they leave out manpages | |
| 13:47:19 | superdan | fungi: yeah that was it | |
| 13:47:31 | superdan | fungi: is bindep in each project or a library? | |
| 13:47:45 | mriedem | each project | |
| 13:47:54 | superdan | so it really needs fixing everywhere? | |
| 13:48:00 | fungi | superdan: it's a python utility on pypi | |
| 13:48:01 | mriedem | i think so | |
| 13:48:09 | mriedem | the txt file is in each project | |
| 13:48:21 | superdan | mriedem: this sounds like it's a core problem though | |
| 13:49:27 | fungi | if you look at nova's tox.ini, the bindep target just installs bindep from pypi into a tox venv and then runs it (and it checks the packages installed on your system against the list of package names in the bindep.txt file in the nova repo) | |
| 13:49:57 | superdan | yep, i see | |
| 13:50:13 | superdan | fungi: recede behind the curtain, I'm unblocked and we can discuss more later if needed :) | |
| 13:50:27 | fungi | glad i could help! | |
| 13:51:09 | superdan | yep thanks :) | |
| 13:52:02 | mriedem | if there is one thing i hate, it's needing to mock a context manager | |
| 13:52:28 | leakypipes | mriedem: just one thing, eh? | |
| 13:52:35 | mriedem | that's the only thing i hate | |
| 13:52:40 | mriedem | right this minute | |
| 14:00:22 | alex_xu | sdague: leakypipes yea, I can take a look at the trait log tomorrow, do you have any example? | |
| 14:00:39 | sdague | alex_xu: I was just running tox -e functional locally | |
| 14:00:52 | alex_xu | sdague: ok, got it, i will try | |
| 14:01:02 | sdague | it's warnings in the stream | |
| 14:01:11 | leakypipes | alex_xu: oh, it's all the same warning... just need to use six.text_type() for a few things. | |
| 14:01:41 | alex_xu | leakypipes: ah, got it, I will check that tomorrow | |
| 14:01:55 | leakypipes | coolio. | |
| 14:10:21 | danpawlik | mriedem: Hi. If you have some time can you reply me in my comment: https://review.openstack.org/#/c/459753/ ? Please | |
| 14:16:43 | mriedem | danpawlik: reply how? | |
| 14:19:08 | danpawlik | mriedem: maybe question is not directed to you, but it will be nice to know if we still wait for reply from Claudio or we can focus on merging it ;) | |
| 14:19:23 | mriedem | danpawlik: i understand that you're eager to get this change merged, | |
| 14:19:32 | mriedem | but we have ~72 proposed blueprints for this release, | |
| 14:19:37 | mriedem | and other things going on at the same time, | |
| 14:19:39 | mriedem | so it's on my list | |
| 14:19:43 | mriedem | but not top priority atm | |
| 14:20:04 | melwitt | mriedem: fyi it looks like this patch is working on the same thing you are about cell0 context https://review.openstack.org/#/c/468322 | |
| 14:20:45 | mriedem | melwitt: ah, it's also incomplete | |
| 14:21:00 | melwitt | here's the bug he opened too https://bugs.launchpad.net/nova/+bug/1693721 | |
| 14:21:00 | openstack | Launchpad bug 1693721 in OpenStack Compute (nova) "Error handling for HostMappingNotFound in conductor manager does not work properly" [Undecided,In progress] - Assigned to Takashi NATSUME (natsume-takashi) | |
| 14:21:21 | melwitt | if you want to pick which one to dupe | |
| 14:21:34 | mriedem | i'll dupe his | |
| 14:21:44 | melwitt | k | |
| 14:22:32 | danpawlik | mriedem: Ok, sorry for pushing you. I just want to move forward.I'll be patient | |
| 14:22:32 | mriedem | so given takashin is probably done for the weekend | |
| 14:22:35 | mriedem | and his patch is incomplete | |
| 14:22:41 | mriedem | i'm not sure if i'll just continue working on mine or clean his up | |
| 14:22:45 | mriedem | plus the trove patches are dependent on mine | |
| 14:29:08 | mriedem | sdague: melwitt: superdan: looks like the trove job failed but it's not failing on the bury in cell0 InstanceNotFound thing with my patch http://logs.openstack.org/47/467447/15/check/gate-trove-scenario-dsvm-mariadb-multi-ubuntu-xenial-nv/30d878e/logs/screen-n-cond.txt.gz?level=TRACE | |
| 14:29:27 | mriedem | now i just need to get unit tests fixed up | |
| 14:30:49 | openstackgerrit | Jay Pipes proposed openstack/nova master: integrate trait cache with TraitList and Trait https://review.openstack.org/462770 | |
| 14:31:14 | sdague | mriedem: nice | |
| 14:32:08 | efried | leakypipes Just saw a pop from an hour and a half ago - something about unicode mismatches? | |
| 14:33:07 | leakypipes | efried: ya, it's cool though. alex_xu is gonna do it tomorrow morning. | |
| 14:33:17 | efried | ight | |
| 14:48:08 | efried | sdague Not sure if you saw, but glanceclient plans to cut a release next Wed. (This is regarding global req id work items) | |
| 14:48:22 | sdague | efried: I did not see that | |
| 14:48:41 | sdague | efried: do glance unit tests run for you? | |
| 14:48:46 | efried | You're wanting https://review.openstack.org/#/c/467592/ to get in there? | |