| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-05-26 | |||
| 13:03:03 | sdague | because of unicode mismatch | |
| 13:03:17 | cdent | sfinucan: leakypipes put his +2 on https://review.openstack.org/#/c/461494/ so it is safe for you to raise your +1 to +2 if you like. or anyone else if they are feeling like a quick win | |
| 13:03:43 | sfinucan | cdent: Yup, done | |
| 13:03:44 | leakypipes | sdague: ya. I've been meaning to ask alex_xu to submit fixups for those... | |
| 13:03:50 | cdent | sfinucan: yay! | |
| 13:04:26 | sdague | leakypipes: ok, cool | |
| 13:04:31 | leakypipes | sdague: but since it's now the weekend for alex_xu I'll put the word out to get that cleaned up. | |
| 13:04:43 | leakypipes | efried: feel like an easy one? :) | |
| 13:04:53 | sdague | yeh, that's fine, I just wanted to make sure it was known | |
| 13:05:15 | leakypipes | sdague: yup, thank you sir :) | |
| 13:06:21 | sdague | cross service request id tracking to cinder is Jenkins +1 - https://review.openstack.org/#/c/468355/ if anyone wants to push that in | |
| 13:11:18 | openstackgerrit | Sean Dague proposed openstack/nova master: Migrate to oslo request_id middleware - mv 2.46 https://review.openstack.org/467998 | |
| 13:22:39 | mriedem | sdague: i don't think a bug was reported for the dogpile.cache test failure | |
| 13:27:17 | mriedem | sdague: tsk tsk https://review.openstack.org/#/c/468355/1 | |
| 13:28:23 | sdague | mriedem: yeh, I was going to swing around with minimums at the end | |
| 13:28:44 | sdague | because otherwise it's another couple of days per patch | |
| 13:28:55 | mriedem | we're using a new thing on a minimum version, | |
| 13:28:59 | mriedem | we should require the minimum version | |
| 13:29:01 | sdague | and if you are CDing and not using upper constraints | |
| 13:29:03 | mriedem | regardless of u-c | |
| 13:29:04 | sdague | you are crazy | |
| 13:29:16 | mriedem | if i'm packaging and picking up when minimums bump... | |
| 13:29:20 | mriedem | at the least | |
| 13:29:26 | mriedem | anyway, can we just do that? it's trivial | |
| 13:29:31 | mriedem | and it's the right thing | |
| 13:29:53 | sdague | mriedem: we have to do g-r change to and Depends-On, that's fine, but I need to swing around on it | |
| 13:30:07 | mriedem | sure | |
| 13:30:13 | mriedem | or i can quick | |
| 13:30:46 | sdague | mriedem: I'll at least state for my defense, even trivial individual patches aren't so much when you have 12 of them outstanding in 7 repos, with depends-on :) | |
| 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? | |