Katherine | how will the two be viewed as dependent? | 00:00 |
---|---|---|
jeblair | Katherine: you have described a case of conflicting reviews, that's not related to dependencies | 00:00 |
dstufft | oh right | 00:00 |
Katherine | what we've seen in our env is two changes like this getting merged in quick succession | 00:00 |
*** beekneemech has joined #openstack-infra | 00:00 | |
dstufft | salt only runs so often | 00:00 |
*** bnemec has quit IRC | 00:00 | |
jeblair | Katherine: based a git commit's parent | 00:00 |
dstufft | asselin: try now, salt just ran at 00 | 00:00 |
Katherine | ok. | 00:00 |
asselin | ok | 00:00 |
Katherine | so the problem i described above would not be caught by zuul | 00:01 |
Katherine | this is the problem we're trying to solve in our env | 00:01 |
asselin | dstufft, jeblair fungi ok works for me | 00:02 |
*** thedodd has quit IRC | 00:02 | |
asselin | dstufft, thanks! | 00:02 |
dstufft | asselin: sorry about that, that's what I get for copy/pasting without testing | 00:02 |
fungi | mtreinish: any guesses what's going on here? http://logs.openstack.org/04/143504/1/check/check-grenade-dsvm-ironic-sideways/c673096/logs/grenade.sh.txt.gz#_2014-12-22_23_50_35_099 | 00:02 |
jeblair | Katherine: zuul should not allow any change that fails tests to land; so if you have testing in place, then a patch which removes something needed by tests should not land | 00:02 |
jeblair | Katherine: however, if you have developers removing tests, then that's not something zuul can help with :) | 00:03 |
Katherine | got the git commit parent dependencies. thanks for that explanation | 00:03 |
fungi | mtreinish: that's the "new" failure i'm seeing on the ironic sideways job now that we're past the other pip 6.0 errors | 00:03 |
Katherine | let me see if i can be more clear. these changes were approved in quick succession of one another. each passed the gate individually | 00:04 |
jeblair | Katherine: we also have a new feature in zuul that we're using that might be relevant | 00:04 |
Katherine | does o~s run into that scenario? | 00:04 |
Katherine | what's that? | 00:04 |
dstufft | fungi: could be related to the fact that pip caches by default to ~/.cache/pip on Linux | 00:04 |
dstufft | though that error doens't look like pip | 00:04 |
jeblair | Katherine: the 'merge-check' pipeline that we defined will check all outstanding changes for merge conflict each time a change merges | 00:04 |
dstufft | pip doens't use mkdir | 00:04 |
Katherine | jeblair: got it | 00:05 |
fungi | dstufft: good eye! same error at http://logs.openstack.org/04/143504/1/check/check-tempest-dsvm-ironic-pxe_ssh/beacd18/logs/devstacklog.txt.gz#_2014-12-22_23_49_11_665 too | 00:05 |
Katherine | i'll look into that | 00:05 |
mtreinish | fungi: hmm, dunno. I'm not too familiar with dib or the ironic devstack paths | 00:05 |
Katherine | jeblair: what's the new feature? i'm curious | 00:05 |
*** achanda has joined #openstack-infra | 00:05 | |
mtreinish | dstufft: oh, yeah that's a good call | 00:06 |
fungi | mtreinish: dstufft: looks like maybe on the ironic jobs we're creating /opt/stack/new/.cache and then pip is barfing all over because it can't write there | 00:06 |
jeblair | Katherine: if i'm following you, i think it will be difficult to solve that problem with technology -- it sounds like there are developers that don't agree on the direction | 00:06 |
mtreinish | fungi: does dib use pip internally? | 00:06 |
jeblair | Katherine: but there is one more bit of tech that might help... | 00:06 |
Katherine | hmm... good point | 00:06 |
Katherine | great. tell me | 00:06 |
dstufft | fungi: you can turn off pip's caching | 00:06 |
mtreinish | fungi: or I guess just creating .cache would do it :) | 00:06 |
dstufft | you might want to for devstack I guess? | 00:07 |
jeblair | Katherine: the new change screen in gerrit will tell you if a patch conflicts with any other patches in gerrit, so you might find cases where developers are attempting to change the same code sooner | 00:07 |
fungi | mtreinish: dstufft: might be the other way around. pip already created .cache and there's an mkdir in the deploy script which is missing -p | 00:07 |
dstufft | yea I think the order fungi said is correct | 00:07 |
Katherine | jeblair: i love the sounds of that! that sounds like something we could really use | 00:07 |
*** baoli has joined #openstack-infra | 00:07 | |
*** baoli has quit IRC | 00:08 | |
dstufft | pip should just silently not cache if it can't write/read to the cache dir | 00:08 |
*** baoli has joined #openstack-infra | 00:08 | |
fungi | so perhaps a bug in ramdisk-image-create | 00:08 |
fungi | looks like we can possibly work around this in lib/baremetal and lib/ironic bu clearing that path first | 00:09 |
jeblair | Katherine: https://gerrit-review.googlesource.com/Documentation/user-review-ui.html#conflicts-with | 00:09 |
*** beekneemech is now known as bnemec | 00:10 | |
*** reazem has quit IRC | 00:10 | |
*** achanda has quit IRC | 00:10 | |
adam_g | mkdir: cannot create directory '/opt/stack/new/.cache': Permission denied | 00:10 |
dstufft | fungi: for complete-ness sake, you can both disable pip's caching and you can point it to a diff dir | 00:11 |
adam_g | $HOME/.cachce is where DIB caches build things like base cloud image, etc | 00:11 |
dstufft | $HOME/.cache is a standard cache dir on Linux yea, pip uses it now too by default | 00:12 |
Katherine | jeblair: thanks for the pointer | 00:12 |
jeblair | Katherine: so i'd say the ideal workflow for that kind of situation would be: developers use that feature (or otherwise discover) they are working on the same code; they agree on a way forward; they rebase their changes on each other as appropriate so they merge in the correct series. | 00:12 |
*** signed8bit is now known as signed8bit_ZZZzz | 00:12 | |
fungi | adam_g: based on the error messages there, it looks less like it's failing to create /opt/stack/new/.cache because it exists, but rather because it's owned by some other account (possibly root from pip being run under sudo) | 00:13 |
adam_g | fungi, yeah | 00:14 |
adam_g | we can just point DIB to $DATA_DIR/dib-cache or something? | 00:14 |
jogo | are things working again? | 00:14 |
fungi | adam_g: oh, maybe | 00:14 |
jogo | things == pip 6.0 issues fixed | 00:14 |
dstufft | if you're running pip with sudo, you might want sudo -H pip | 00:14 |
fungi | jogo: no, blocked on ironic and pip fighting over the same cachedir now | 00:14 |
dstufft | so that pip uses /root/.cache | 00:14 |
fungi | dstufft: that's a stellar suggestion. seeing if i can find all the possible places that's happening | 00:15 |
*** zz_zz_zz_zz_zz_s is now known as zz_zz_zz_zz_zz_z | 00:15 | |
jogo | fungi: thanks | 00:15 |
*** Masahiro has joined #openstack-infra | 00:20 | |
adam_g | fungi, https://review.openstack.org/143573 | 00:20 |
*** dims has joined #openstack-infra | 00:22 | |
*** dims has quit IRC | 00:23 | |
fungi | adam_g: dstufft: updated https://review.openstack.org/143504 to see if that helps. otherwise we can try squashing 143573 into the initial patchset there | 00:23 |
adam_g | fungi, nice | 00:24 |
*** Masahiro has quit IRC | 00:24 | |
*** SumitNaiksatam has quit IRC | 00:27 | |
dstufft | fungi: using sudo -H with pip is a good idea anyways because we're going to increasingly be using ~/<stuff> automatically | 00:28 |
*** dims has joined #openstack-infra | 00:28 | |
*** salv-orlando has quit IRC | 00:32 | |
*** ryanpetrello has joined #openstack-infra | 00:35 | |
*** dims has quit IRC | 00:36 | |
*** ryanpetrello has quit IRC | 00:42 | |
*** hemna is now known as hemnafk | 00:46 | |
*** hichtakk has quit IRC | 00:47 | |
*** hichtakk has joined #openstack-infra | 00:48 | |
*** grue_pm has quit IRC | 00:51 | |
*** grue_pm has joined #openstack-infra | 00:52 | |
*** unicell has joined #openstack-infra | 00:57 | |
*** hichtakk has quit IRC | 00:58 | |
*** hichtakk has joined #openstack-infra | 00:58 | |
*** dmsimard is now known as dmsimard_away | 00:59 | |
fungi | mtreinish: dstufft: adam_g: looks like the new https://review.openstack.org/143504 is working for juno, so i'll make similar edits to fix it on master as well | 01:01 |
*** harlowja has joined #openstack-infra | 01:01 | |
*** liusheng has quit IRC | 01:01 | |
fungi | dtroyer: ^ | 01:02 |
*** dannywilson has quit IRC | 01:03 | |
adam_g | w00t | 01:03 |
*** sigmavirus24_awa is now known as sigmavirus24 | 01:05 | |
fungi | looks like stack.sh didn't need editing on master, just functions-common | 01:06 |
fungi | dtroyer: https://review.openstack.org/143501 is updated now | 01:07 |
fungi | and the juno change looks like it's about to pop into the gate pipeline if we're really lucky | 01:08 |
*** sigmavirus24 is now known as sigmavirus24_awa | 01:08 | |
anteaya | zaro: nice christmas present | 01:08 |
anteaya | :D | 01:09 |
dtroyer | fungi: I just +2 it for now…am going to dinner soon so if it looks good, go ahead and approve | 01:09 |
fungi | dtroyer: okay | 01:09 |
fungi | thanks! | 01:09 |
dstufft | fungi: awesome, does that mean that as of right now the pip release wasn't very bad? :D | 01:10 |
fungi | dstufft: yep! | 01:11 |
*** fandi_ has quit IRC | 01:11 | |
dstufft | awesome | 01:12 |
dstufft | it wouldn't be a pip release without a little breakage | 01:12 |
*** gyee has quit IRC | 01:12 | |
fungi | dstufft: i was worried it would tail into a multi-day scramble | 01:12 |
fungi | kudos! | 01:12 |
fungi | and thanks for all the help | 01:12 |
dstufft | no problem :) glad it went relatively smoothly | 01:13 |
zaro | anteaya: thanks! hoping to get a similar response from my wife :) | 01:13 |
zaro | although i feel odds are not as good. | 01:13 |
fungi | zaro: it's awesome that your wife is such a huge fan of storyboard | 01:14 |
fungi | i can't convince my wife to care about any of this stuff | 01:14 |
*** camunoz has quit IRC | 01:15 | |
fungi | okay, the juno fix has now made it into the gate. i'll recheck the master fix once that merges | 01:16 |
dstufft | my wife and daughter are putting up the christmas tree, it has disco balls and a strobe light | 01:16 |
dstufft | it's like christmas at a rave | 01:16 |
fungi | dstufft: i'm envious | 01:16 |
fungi | backpacks and pacifiers all 'round | 01:16 |
*** ddieterly has quit IRC | 01:16 | |
dstufft | I should get some of them to complete the look! | 01:16 |
fungi | dstufft: i hope you have enough blacklights | 01:17 |
*** grue_pm has quit IRC | 01:17 | |
*** ddieterly has joined #openstack-infra | 01:17 | |
dstufft | it's a pink tree with a zebra skirt, basically our tree looks like snooki puked on it | 01:17 |
*** reed has quit IRC | 01:17 | |
*** grue_pm has joined #openstack-infra | 01:17 | |
*** hichtakk has quit IRC | 01:18 | |
*** hichtakk has joined #openstack-infra | 01:18 | |
*** adalbas has quit IRC | 01:19 | |
*** andreykurilin_ has joined #openstack-infra | 01:19 | |
fungi | we have the biggest black prelit tree i could find, painstakingly re-bulbed with all purple lights | 01:19 |
dstufft | now they are asking me if I know where the smoke machine is | 01:20 |
dstufft | fungi: a black tree would be great | 01:20 |
*** ddieterly has quit IRC | 01:21 | |
jogo | sigmavirus24_awa: just tried your patch, very neat | 01:23 |
*** sdake has quit IRC | 01:24 | |
*** dims has joined #openstack-infra | 01:28 | |
*** cdent has quit IRC | 01:30 | |
*** andreykurilin_ has quit IRC | 01:32 | |
*** salv-orlando has joined #openstack-infra | 01:32 | |
*** dims has quit IRC | 01:34 | |
anteaya | dstufft: where you you get a zebra skirt | 01:39 |
anteaya | I'm feeling like spruce is all out of place | 01:39 |
dstufft | anteaya: It's actually a blanket for a bed I think, but we use it as a skirt | 01:40 |
anteaya | ah okay | 01:40 |
*** grue_pm has quit IRC | 01:40 | |
anteaya | when I am inclined to tree I just go spruce or balsam | 01:40 |
*** grue_pm has joined #openstack-infra | 01:40 | |
anteaya | head outside, knock something down, done | 01:41 |
dstufft | I haven't had a real tree in like 10 years | 01:41 |
anteaya | wow | 01:41 |
anteaya | the smell is what I like | 01:41 |
anteaya | cleaning up the needles is tons of fun too | 01:41 |
anteaya | not sure how I would deal with a pink tree and a smoke machine | 01:41 |
dstufft | this house isn't big enough for a big tree | 01:42 |
*** marcusvrn has quit IRC | 01:42 | |
*** grue_pm has left #openstack-infra | 01:43 | |
dstufft | my child hood house was though | 01:43 |
dstufft | we cut it down and the whole 9 yards | 01:43 |
*** grue_pm has joined #openstack-infra | 01:43 | |
anteaya | the tree I hope | 01:44 |
*** otter768 has quit IRC | 01:44 | |
anteaya | yeah we would make a day of it | 01:44 |
anteaya | in high school my brother and I would spend the day getting the old snowmachines working | 01:45 |
anteaya | drive around in circles in the fields for a few hours | 01:45 |
anteaya | then cut a tree, drag it behind the snowmachine round about dark | 01:45 |
*** melwitt has quit IRC | 01:45 | |
dstufft | I used to have a big sheet of plywood that I had a model railroad tracks nailed down on, and it was sprayed with fake snow and such for the base | 01:45 |
dstufft | dunno if my parents still have it | 01:46 |
anteaya | put the machines away, eat dinner with our grandparents and head home | 01:46 |
anteaya | dstufft: sounds awesome | 01:46 |
*** marcusvrn has joined #openstack-infra | 01:47 | |
dstufft | I remember being blown out of my mind when we went to a tree farm that has the machine that drills a hole in the base of the tree | 01:47 |
dstufft | those hole in the tree things are so much better than the screws for standing up a tree | 01:47 |
anteaya | I don't know what you are talking about | 01:47 |
anteaya | we just nailed a circle of plywood to the base and stood the whole business up in a plastic 5 gallon pail and added rocks | 01:48 |
anteaya | I have never seen a machine that drills a hole in the base of the tree | 01:48 |
dstufft | anteaya: it's a hole, drilled in the bottom of the tree, the tree farm levels the tree out and drills a hole in it, then you get a stand like this -> https://cdn.shopify.com/s/files/1/0194/1253/products/standstrait_1024x1024.jpg?v=1353464744 which has a big spike in it | 01:49 |
dstufft | you just stick the tree on the spike and it's leveled and all that for you | 01:49 |
anteaya | cool | 01:50 |
*** grue_pm has left #openstack-infra | 01:50 | |
anteaya | and makes sense | 01:50 |
*** grue_pm has joined #openstack-infra | 01:50 | |
anteaya | so did you pick your own tree at the tree farm? | 01:50 |
dstufft | ya | 01:50 |
dstufft | went and cut them | 01:50 |
anteaya | nice | 01:51 |
anteaya | spruce? | 01:51 |
dstufft | then just took em to the people there and they put the hole in it and sent it through the baler to wrap string around it to bind it tight so that it was easier to move | 01:51 |
anteaya | nice | 01:51 |
dstufft | yea I believe they were spruce | 01:52 |
*** bdpayne has quit IRC | 01:52 | |
anteaya | spruce has pointy needles and when you pick one you can roll it around between your fingers | 01:52 |
anteaya | balsam has flat needles, you can't roll them, and the are not pointy | 01:52 |
anteaya | wrapping your hand around a balsam branch doesn't hurt | 01:52 |
dstufft | yea it was spruce then | 01:53 |
anteaya | cool | 01:53 |
anteaya | I like them both | 01:53 |
anteaya | but I slightly favour spruce | 01:53 |
jogo | after https://review.openstack.org/#/c/143504/ land are things working again? | 01:54 |
dstufft | jogo: I think there's a similar patch for master | 01:54 |
dstufft | that can't land until that lands | 01:54 |
*** harlowja has quit IRC | 01:54 | |
dstufft | https://review.openstack.org/#/c/143501/ | 01:54 |
jogo | ahh thanks | 01:55 |
anteaya | "that can't land until that lands" I want that on a shirt | 01:55 |
jogo | anteaya: hehe | 01:56 |
*** grue_pm has quit IRC | 01:57 | |
*** grue_pm has joined #openstack-infra | 01:58 | |
*** markmcclain has quit IRC | 01:59 | |
anteaya | yeah, I too want to hear how excited zaro's wife is about the storyboard plugin | 02:01 |
krotscheck | Urm what? | 02:01 |
* krotscheck is missing context. | 02:01 | |
anteaya | ha ha ha | 02:02 |
anteaya | backscroll | 02:02 |
anteaya | it is better than if i try to explain | 02:02 |
anteaya | have the -dev ml handy while you read | 02:02 |
krotscheck | anteaya: I… uh… know what? That didn’t help much | 02:02 |
*** markmcclain has joined #openstack-infra | 02:03 | |
*** koolhead17 has joined #openstack-infra | 02:04 | |
*** stevemar has joined #openstack-infra | 02:04 | |
anteaya | zaro posted to the ml about the gerrit plugin for storyboard | 02:07 |
anteaya | I said nice christmas present | 02:07 |
anteaya | he said he was hoping for the same response from his wife | 02:07 |
anteaya | and fungi was surprised zaro's wife is such a big fan of storyboard | 02:07 |
*** Masahiro has joined #openstack-infra | 02:09 | |
*** ZZelle has quit IRC | 02:11 | |
anteaya | it loses something in the telling | 02:11 |
fungi | jogo: it's rechecked now and should hopefully pop into the gate as soon as the grenade jobs for it pass | 02:11 |
fungi | jogo: https://review.openstack.org/143501 | 02:11 |
* fungi disappears again | 02:11 | |
*** Masahiro has quit IRC | 02:13 | |
*** ddieterly has joined #openstack-infra | 02:14 | |
*** camunoz has joined #openstack-infra | 02:17 | |
*** ddieterly has quit IRC | 02:19 | |
*** signed8bit_ZZZzz is now known as signed8bit | 02:20 | |
*** ddieterly has joined #openstack-infra | 02:26 | |
*** baoli has quit IRC | 02:27 | |
*** baoli has joined #openstack-infra | 02:29 | |
*** baoli has quit IRC | 02:30 | |
*** baoli has joined #openstack-infra | 02:30 | |
*** ddieterly has quit IRC | 02:30 | |
*** jerryz has joined #openstack-infra | 02:33 | |
*** yamahata has joined #openstack-infra | 02:34 | |
*** yamahata has quit IRC | 02:34 | |
*** dims has joined #openstack-infra | 02:34 | |
*** yamahata has joined #openstack-infra | 02:34 | |
*** yaguang has joined #openstack-infra | 02:35 | |
*** dims has quit IRC | 02:39 | |
*** zz_dimtruck is now known as dimtruck | 02:42 | |
*** Ryan_Lane has quit IRC | 02:47 | |
*** hdd has quit IRC | 02:48 | |
*** SumitNaiksatam has joined #openstack-infra | 02:51 | |
*** jyuso has quit IRC | 02:54 | |
openstackgerrit | Wayne Warren proposed openstack-infra/jenkins-job-builder: Add plugins_info to module registry object. https://review.openstack.org/132927 | 02:55 |
openstackgerrit | Wayne Warren proposed openstack-infra/jenkins-job-builder: Update 'timeout' wrapper module https://review.openstack.org/129467 | 02:55 |
*** teran has quit IRC | 02:55 | |
*** salv-orlando has quit IRC | 02:58 | |
*** marcusvrn has quit IRC | 03:00 | |
*** marcusvrn has joined #openstack-infra | 03:03 | |
openstackgerrit | Wayne Warren proposed openstack-infra/jenkins-job-builder: Add plugins_info to module registry object. https://review.openstack.org/132927 | 03:04 |
openstackgerrit | Wayne Warren proposed openstack-infra/jenkins-job-builder: Update 'timeout' wrapper module https://review.openstack.org/129467 | 03:04 |
*** wuhg has joined #openstack-infra | 03:11 | |
*** boris-42 has quit IRC | 03:13 | |
*** mfer has joined #openstack-infra | 03:15 | |
*** mfer has quit IRC | 03:20 | |
*** markmcclain has quit IRC | 03:20 | |
*** Ryan_Lane has joined #openstack-infra | 03:22 | |
*** ayoung has quit IRC | 03:23 | |
*** jyuso has joined #openstack-infra | 03:23 | |
*** ddieterly has joined #openstack-infra | 03:26 | |
*** spzala has quit IRC | 03:30 | |
*** ddieterly has quit IRC | 03:31 | |
*** hichtakk has quit IRC | 03:33 | |
*** hichtakk has joined #openstack-infra | 03:33 | |
*** otter768 has joined #openstack-infra | 03:45 | |
*** Ryan_Lane has quit IRC | 03:48 | |
*** asettle has quit IRC | 03:50 | |
*** otter768 has quit IRC | 03:50 | |
*** marcusvrn has quit IRC | 03:51 | |
*** marcusvrn has joined #openstack-infra | 03:55 | |
*** teran has joined #openstack-infra | 03:56 | |
fungi | devstack should be unbroken again. i've rechecked the setuptools 8 stack for pbr now to see if they'll finally merge | 03:57 |
*** Masahiro has joined #openstack-infra | 03:57 | |
*** dannywilson has joined #openstack-infra | 03:59 | |
*** signed8bit has quit IRC | 04:00 | |
*** teran has quit IRC | 04:01 | |
*** Masahiro has quit IRC | 04:01 | |
*** salv-orlando has joined #openstack-infra | 04:02 | |
*** asettle has joined #openstack-infra | 04:03 | |
*** dannywilson has quit IRC | 04:04 | |
*** dimtruck is now known as zz_dimtruck | 04:10 | |
*** baoli has quit IRC | 04:12 | |
jogo | fungi: nice, time to do some rechecks then | 04:20 |
*** otter768 has joined #openstack-infra | 04:24 | |
*** patrickeast has quit IRC | 04:25 | |
*** ddieterly has joined #openstack-infra | 04:26 | |
*** hdd has joined #openstack-infra | 04:27 | |
*** armax has quit IRC | 04:27 | |
*** ddieterly has quit IRC | 04:30 | |
*** mbacchi has quit IRC | 04:35 | |
*** stevemar has quit IRC | 04:40 | |
*** stevemar has joined #openstack-infra | 04:41 | |
*** mtanino has quit IRC | 04:41 | |
dougwig | for job in ALL_JOBS_IN_GERRIT(): "recheck" | 04:46 |
dougwig | :) | 04:46 |
*** hichtakk has quit IRC | 04:51 | |
jogo | dougwig: I think its for job in https://review.openstack.org/#/q/is:open+label:Workflow%253E%253D1+label:verified%253C%253D0%252Cjenkins+age:2hour,n,z | 04:51 |
*** teran has joined #openstack-infra | 04:57 | |
*** yamahata has quit IRC | 04:58 | |
*** hichtakk has joined #openstack-infra | 05:00 | |
*** devanand1 is now known as devananda | 05:00 | |
*** teran has quit IRC | 05:02 | |
*** hichtakk has quit IRC | 05:03 | |
*** garyh has quit IRC | 05:09 | |
*** gpocentek has quit IRC | 05:10 | |
*** mfink- has quit IRC | 05:15 | |
*** mfink_ has joined #openstack-infra | 05:16 | |
*** ddieterly has joined #openstack-infra | 05:17 | |
*** erlon_ has quit IRC | 05:19 | |
*** ddieterly has quit IRC | 05:22 | |
*** salv-orlando has quit IRC | 05:32 | |
*** pcrews has quit IRC | 05:42 | |
*** adriant has quit IRC | 05:44 | |
*** yfried has quit IRC | 05:44 | |
*** Masahiro has joined #openstack-infra | 05:46 | |
*** achanda has joined #openstack-infra | 05:50 | |
*** Masahiro has quit IRC | 05:51 | |
*** wuhg has quit IRC | 05:53 | |
*** sputnik13 has joined #openstack-infra | 05:59 | |
*** achanda has quit IRC | 06:02 | |
*** marcusvrn has quit IRC | 06:03 | |
*** achanda has joined #openstack-infra | 06:07 | |
*** marcusvrn has joined #openstack-infra | 06:08 | |
*** hichtakk has joined #openstack-infra | 06:15 | |
*** garyk has joined #openstack-infra | 06:16 | |
*** stevemar has quit IRC | 06:20 | |
*** achanda has quit IRC | 06:26 | |
*** hdd has quit IRC | 06:36 | |
*** hdd has joined #openstack-infra | 06:36 | |
*** yfried has joined #openstack-infra | 06:39 | |
*** koolhead17 has quit IRC | 06:43 | |
*** denis_makogon has quit IRC | 06:50 | |
*** denis_makogon has joined #openstack-infra | 06:50 | |
*** vponomaryov has quit IRC | 06:51 | |
*** vponomaryov has joined #openstack-infra | 06:51 | |
*** camunoz has quit IRC | 06:56 | |
openstackgerrit | Alice Ferrazzi proposed openstack-infra/lodgeit: Install pillow instead of PIL https://review.openstack.org/128895 | 06:57 |
*** hdd has quit IRC | 06:58 | |
*** otter768 has quit IRC | 06:58 | |
*** Masahiro has joined #openstack-infra | 07:01 | |
*** HeOS has quit IRC | 07:04 | |
*** k4n0 has joined #openstack-infra | 07:07 | |
*** salv-orlando has joined #openstack-infra | 07:17 | |
*** jamielennox is now known as jamielennox|away | 07:24 | |
*** mfink_ has quit IRC | 07:25 | |
*** teran has joined #openstack-infra | 07:25 | |
*** mfink_ has joined #openstack-infra | 07:30 | |
*** yamahata has joined #openstack-infra | 07:37 | |
*** pblaho_ has joined #openstack-infra | 07:39 | |
*** hichtakk has quit IRC | 07:41 | |
*** teran_ has joined #openstack-infra | 07:42 | |
*** teran has quit IRC | 07:43 | |
*** subscope has joined #openstack-infra | 07:48 | |
*** salv-orlando has quit IRC | 07:50 | |
openstackgerrit | Merged openstack-dev/pbr: Port in git sha changes from 0.10 line https://review.openstack.org/142931 | 07:50 |
*** salv-orlando has joined #openstack-infra | 07:50 | |
*** yamahata has quit IRC | 07:55 | |
*** teran has joined #openstack-infra | 07:57 | |
*** pblaho_ has quit IRC | 07:57 | |
*** yfried is now known as yfried|afk | 08:00 | |
*** teran_ has quit IRC | 08:00 | |
*** isaacb has joined #openstack-infra | 08:01 | |
*** EmilienM|afk is now known as EmilienM | 08:02 | |
*** hichtakk has joined #openstack-infra | 08:02 | |
*** garyk has quit IRC | 08:03 | |
*** jgallard_ has joined #openstack-infra | 08:05 | |
openstackgerrit | Merged openstack-dev/pbr: Properly check for git before getting git dir https://review.openstack.org/142841 | 08:07 |
openstackgerrit | Merged openstack-dev/pbr: Move write_pbr_json to avoid issues with nose https://review.openstack.org/143146 | 08:07 |
*** yfried|afk is now known as yfried | 08:08 | |
*** e0ne has joined #openstack-infra | 08:10 | |
*** Masahiro has quit IRC | 08:12 | |
*** ttx has quit IRC | 08:13 | |
*** ttx has joined #openstack-infra | 08:14 | |
*** ttx has quit IRC | 08:14 | |
*** ttx has joined #openstack-infra | 08:14 | |
*** e0ne has quit IRC | 08:14 | |
*** Longgeek has joined #openstack-infra | 08:22 | |
*** hichtakk has quit IRC | 08:23 | |
*** amuller has joined #openstack-infra | 08:29 | |
*** teran has quit IRC | 08:30 | |
*** HeOS has joined #openstack-infra | 08:33 | |
openstackgerrit | Sumit Naiksatam proposed openstack-infra/project-config: Update GBP projects for tarballs, pypi & gerritbot https://review.openstack.org/143526 | 08:34 |
*** denis_makogon has quit IRC | 08:34 | |
*** denis_makogon has joined #openstack-infra | 08:35 | |
*** Hal_ has joined #openstack-infra | 08:40 | |
*** yfried is now known as yfried|afk | 08:40 | |
*** yfried|afk is now known as yfried | 08:42 | |
*** hichtakk has joined #openstack-infra | 08:46 | |
*** Ala has joined #openstack-infra | 08:50 | |
*** jpich has joined #openstack-infra | 08:51 | |
*** skolekonov has joined #openstack-infra | 08:53 | |
*** e0ne has joined #openstack-infra | 08:54 | |
*** e0ne has quit IRC | 08:55 | |
*** otter768 has joined #openstack-infra | 08:59 | |
*** yfried is now known as yfried|afk | 08:59 | |
*** yfried|afk is now known as yfried | 08:59 | |
anteaya | omrim: I am still here for a few minutes if you want to keep working on your tox problem | 09:01 |
anteaya | if I don't hear a response in 2 minutes I am off to bed | 09:02 |
omrim | anteaya: I jeust clone a devstack | 09:02 |
omrim | anteaya: And then I run the tox | 09:03 |
anteaya | okay great | 09:03 |
omrim | anteaya: How can I see the tox.ini of the gerrit? | 09:03 |
*** otter768 has quit IRC | 09:03 | |
anteaya | well it is the tox.ini that that job uses | 09:03 |
anteaya | which is the tox.ini file of the system-config repo | 09:04 |
anteaya | which should be the same tox file you are invoking when you run tox within the system-config repo after you have made changes to the third_party.rst file | 09:04 |
anteaya | omrim: so when you say you clone the devstack and run tox | 09:05 |
omrim | Now..during the meeting | 09:05 |
anteaya | you are running tox within the system-config root are you not? | 09:06 |
anteaya | this is the tox file for system-config | 09:06 |
anteaya | http://git.openstack.org/cgit/openstack-infra/system-config/tree/tox.ini | 09:06 |
anteaya | omrim: sorry are you saying you have another meeting? | 09:06 |
omrim | anteaya: No I meant during our meeting..:) | 09:07 |
anteaya | ah | 09:07 |
anteaya | what directory are you in when you run tox locally? | 09:08 |
omrim | anteaya: Oh...anteaya I think you figure it out | 09:09 |
anteaya | I did | 09:09 |
anteaya | so can you get the expected results from you local tox now? | 09:09 |
omrim | anteaya: Now I run it with the tox.ini that you sent me ant it failed | 09:09 |
anteaya | well you should be in the system-config repo and run tox | 09:10 |
omrim | anteaya: Why there is a different between tox.ini of the devstack to the infra? | 09:10 |
anteaya | and when you are in the correct repo, tox will find the tox.ini file in that repo | 09:10 |
anteaya | tox will find the tox.ini file of whereever it is and run that | 09:10 |
anteaya | and system-config is not part of devstack | 09:11 |
anteaya | so when you tell me you run tox in devstack I am very confused | 09:11 |
omrim | anteaya: Thanks a lot! have a good night :) | 09:11 |
anteaya | since to get your tox results for the third_party.rst file you have to be in the system-config repo when you run tox | 09:11 |
anteaya | omrim: I won't turn down the opportunity to go to bed, I hope you have solved your problem | 09:12 |
anteaya | good night | 09:12 |
anteaya | omrim: you are doing good work on your patch, keep it up | 09:12 |
* anteaya goes to bed | 09:12 | |
omrim | anteaya: Yes it's make sense my mistake | 09:12 |
omrim | anteaya: Thanks a lot you are doing a great job too.. | 09:13 |
*** mfink_ has quit IRC | 09:13 | |
*** Masahiro has joined #openstack-infra | 09:13 | |
*** hichtakk has quit IRC | 09:17 | |
*** garyk has joined #openstack-infra | 09:17 | |
*** Masahiro has quit IRC | 09:17 | |
heyongli | anteaya, are you still there? | 09:19 |
*** yfried is now known as yfried|afk | 09:20 | |
*** yfried|afk is now known as yfried | 09:23 | |
*** salv-orlando has quit IRC | 09:24 | |
heyongli | anteaya, don't worry, see you next year. | 09:24 |
*** ihrachyshka has joined #openstack-infra | 09:31 | |
openstackgerrit | Merged openstack-infra/storyboard: Plugins may now register cron workers. https://review.openstack.org/129609 | 09:35 |
*** ildikov has joined #openstack-infra | 09:36 | |
*** ociuhandu has joined #openstack-infra | 09:43 | |
openstackgerrit | omri marcovitch proposed openstack-infra/system-config: Add Third Party FAQ paragraph https://review.openstack.org/141817 | 09:45 |
*** dizquierdo has joined #openstack-infra | 09:54 | |
*** ZZelle has joined #openstack-infra | 09:56 | |
*** praveenkumar has quit IRC | 09:56 | |
*** andreykurilin_ has joined #openstack-infra | 09:57 | |
*** teran has joined #openstack-infra | 09:58 | |
*** nuritv has joined #openstack-infra | 10:01 | |
*** salv-orlando has joined #openstack-infra | 10:01 | |
*** praveenkumar has joined #openstack-infra | 10:06 | |
*** e0ne has joined #openstack-infra | 10:08 | |
*** vigneshvar has joined #openstack-infra | 10:10 | |
*** _nadya_ has joined #openstack-infra | 10:11 | |
*** abhijeetm has joined #openstack-infra | 10:12 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/infra-manual: Explain changes and patch sets https://review.openstack.org/143644 | 10:13 |
*** yfried is now known as yfried|afk | 10:14 | |
*** Masahiro has joined #openstack-infra | 10:14 | |
*** Masahiro has quit IRC | 10:18 | |
*** EmilienM is now known as EmilienM|afk | 10:22 | |
*** yfried|afk is now known as yfried | 10:22 | |
*** sputnik13 has quit IRC | 10:24 | |
*** yolanda has quit IRC | 10:29 | |
*** garyk has quit IRC | 10:33 | |
*** andreykurilin_ has quit IRC | 10:35 | |
*** yolanda has joined #openstack-infra | 10:41 | |
*** garyk has joined #openstack-infra | 10:41 | |
*** yaguang has quit IRC | 10:41 | |
openstackgerrit | Yair Fried proposed openstack-infra/project-config: Remove large-ops-{stable_branch} gates from tempest https://review.openstack.org/143648 | 10:44 |
*** garyk has quit IRC | 10:52 | |
*** unicell has quit IRC | 10:56 | |
*** e0ne is now known as e0ne_ | 10:59 | |
*** otter768 has joined #openstack-infra | 11:00 | |
*** habib has joined #openstack-infra | 11:04 | |
*** otter768 has quit IRC | 11:04 | |
*** EmilienM|afk is now known as EmilienM | 11:07 | |
*** salv-orlando has quit IRC | 11:08 | |
*** e0ne_ has quit IRC | 11:09 | |
*** e0ne has joined #openstack-infra | 11:21 | |
*** _nadya_ has quit IRC | 11:21 | |
*** jgallard_ has quit IRC | 11:22 | |
*** habib has quit IRC | 11:27 | |
*** enikanorov_ has quit IRC | 11:30 | |
*** rfolco has joined #openstack-infra | 11:33 | |
*** rfolco has quit IRC | 11:38 | |
*** e0ne is now known as e0ne_ | 11:45 | |
*** e0ne_ is now known as e0ne | 11:46 | |
*** dims has joined #openstack-infra | 11:56 | |
*** andreaf has quit IRC | 12:00 | |
*** andreaf has joined #openstack-infra | 12:00 | |
openstackgerrit | Emilien Macchi proposed openstack-infra/project-config: Bring stackforge/puppet modules to StoryBoard https://review.openstack.org/143655 | 12:02 |
*** Masahiro has joined #openstack-infra | 12:02 | |
*** teran has quit IRC | 12:05 | |
*** teran has joined #openstack-infra | 12:07 | |
*** Masahiro has quit IRC | 12:07 | |
*** garyk has joined #openstack-infra | 12:07 | |
*** ff has joined #openstack-infra | 12:07 | |
*** salv-orlando has joined #openstack-infra | 12:08 | |
*** salv-orlando has quit IRC | 12:11 | |
*** salv-orlando has joined #openstack-infra | 12:12 | |
*** e0ne_ has joined #openstack-infra | 12:15 | |
*** subscope has quit IRC | 12:20 | |
*** vigneshvar has quit IRC | 12:24 | |
*** jyuso has quit IRC | 12:24 | |
*** mwagner_lap has quit IRC | 12:24 | |
*** annegentle has quit IRC | 12:24 | |
*** shayneburgess has quit IRC | 12:24 | |
*** rkukura has quit IRC | 12:24 | |
*** dims has quit IRC | 12:24 | |
*** jpich has quit IRC | 12:24 | |
*** yfried has quit IRC | 12:24 | |
*** asettle has quit IRC | 12:24 | |
*** pabelanger has quit IRC | 12:24 | |
*** krtaylor has quit IRC | 12:24 | |
*** vhoward has quit IRC | 12:24 | |
*** sc68cal has quit IRC | 12:24 | |
*** tchaypo has quit IRC | 12:24 | |
*** zhiyan has quit IRC | 12:24 | |
*** gus has quit IRC | 12:24 | |
*** cloudnull has quit IRC | 12:24 | |
*** StevenK has quit IRC | 12:24 | |
*** sweston has quit IRC | 12:24 | |
*** vishy has quit IRC | 12:24 | |
*** dougwig has quit IRC | 12:24 | |
*** zz_zz_zz_zz_zz_z has quit IRC | 12:24 | |
*** masayukig has quit IRC | 12:24 | |
*** zigo has quit IRC | 12:24 | |
*** jamielennox|away has quit IRC | 12:24 | |
*** erw_ has quit IRC | 12:24 | |
*** briancline has quit IRC | 12:24 | |
*** hogepodge has quit IRC | 12:24 | |
*** freyes has quit IRC | 12:24 | |
*** mattoliverau has quit IRC | 12:24 | |
*** Guest7756 has quit IRC | 12:24 | |
*** erikmwilson has quit IRC | 12:24 | |
*** Adri2000 has quit IRC | 12:24 | |
*** Hunner has quit IRC | 12:24 | |
*** yolanda has quit IRC | 12:24 | |
*** notmyname has quit IRC | 12:24 | |
*** xianghui has quit IRC | 12:24 | |
*** dansmith has quit IRC | 12:24 | |
*** tteggel_ has quit IRC | 12:24 | |
*** alazarev has quit IRC | 12:24 | |
*** hyakuhei_ has quit IRC | 12:24 | |
*** gothicmindfood has quit IRC | 12:24 | |
*** dkehn has quit IRC | 12:24 | |
*** juice has quit IRC | 12:24 | |
*** LinuxJedi has quit IRC | 12:24 | |
*** ianw has quit IRC | 12:24 | |
*** jesusaurus has quit IRC | 12:24 | |
*** dkliban_afk has quit IRC | 12:24 | |
*** e0ne has quit IRC | 12:24 | |
*** denis_makogon has quit IRC | 12:24 | |
*** Longgeek has quit IRC | 12:24 | |
*** Daviey has quit IRC | 12:24 | |
*** mfink has quit IRC | 12:24 | |
*** dtantsur has quit IRC | 12:24 | |
*** maurosr has quit IRC | 12:24 | |
*** jraim has quit IRC | 12:24 | |
*** kevinbenton_ has quit IRC | 12:24 | |
*** BobBall_Xmas has quit IRC | 12:24 | |
*** rcarrillocruz has quit IRC | 12:24 | |
*** funzo_ has quit IRC | 12:24 | |
*** hamzy_vacation has quit IRC | 12:24 | |
*** primeministerp has quit IRC | 12:24 | |
*** doude_ has quit IRC | 12:24 | |
*** dhp has quit IRC | 12:24 | |
*** dstufft has quit IRC | 12:24 | |
*** gingerjiang has quit IRC | 12:24 | |
*** dteselkin has quit IRC | 12:24 | |
*** NikitaKonovalov has quit IRC | 12:24 | |
*** ruhe has quit IRC | 12:24 | |
*** zul has quit IRC | 12:24 | |
*** andreaf_ has quit IRC | 12:24 | |
*** HeOS_ has quit IRC | 12:24 | |
*** davidlenwell has quit IRC | 12:24 | |
*** Apsu has quit IRC | 12:24 | |
*** antonym has quit IRC | 12:24 | |
*** eikke has quit IRC | 12:24 | |
*** SpamapS has quit IRC | 12:24 | |
*** EntropyWorks has quit IRC | 12:24 | |
*** amuller has quit IRC | 12:24 | |
*** jerryz has quit IRC | 12:24 | |
*** Mmike has quit IRC | 12:24 | |
*** redrobot_away has quit IRC | 12:24 | |
*** sressot has quit IRC | 12:24 | |
*** mugsie has quit IRC | 12:24 | |
*** gnuoy has quit IRC | 12:24 | |
*** JoshNang has quit IRC | 12:24 | |
*** vladan has quit IRC | 12:24 | |
*** jokke_ has quit IRC | 12:24 | |
*** alaski has quit IRC | 12:24 | |
*** jhesketh has quit IRC | 12:24 | |
*** paul-- has quit IRC | 12:24 | |
*** vigneshvar has joined #openstack-infra | 12:25 | |
*** jyuso has joined #openstack-infra | 12:25 | |
*** annegentle has joined #openstack-infra | 12:25 | |
*** mwagner_lap has joined #openstack-infra | 12:25 | |
*** shayneburgess has joined #openstack-infra | 12:25 | |
*** rkukura has joined #openstack-infra | 12:25 | |
*** dims has joined #openstack-infra | 12:26 | |
*** jpich has joined #openstack-infra | 12:26 | |
*** yfried has joined #openstack-infra | 12:26 | |
*** asettle has joined #openstack-infra | 12:26 | |
*** pabelanger has joined #openstack-infra | 12:26 | |
*** krtaylor has joined #openstack-infra | 12:26 | |
*** vhoward has joined #openstack-infra | 12:26 | |
*** sc68cal has joined #openstack-infra | 12:26 | |
*** tchaypo has joined #openstack-infra | 12:26 | |
*** zhiyan has joined #openstack-infra | 12:26 | |
*** gus has joined #openstack-infra | 12:26 | |
*** cloudnull has joined #openstack-infra | 12:26 | |
*** StevenK has joined #openstack-infra | 12:26 | |
*** vishy has joined #openstack-infra | 12:26 | |
*** sweston has joined #openstack-infra | 12:26 | |
*** dougwig has joined #openstack-infra | 12:26 | |
*** zz_zz_zz_zz_zz_z has joined #openstack-infra | 12:26 | |
*** masayukig has joined #openstack-infra | 12:26 | |
*** zigo has joined #openstack-infra | 12:26 | |
*** jamielennox|away has joined #openstack-infra | 12:26 | |
*** erw_ has joined #openstack-infra | 12:26 | |
*** freyes has joined #openstack-infra | 12:26 | |
*** hogepodge has joined #openstack-infra | 12:26 | |
*** briancline has joined #openstack-infra | 12:26 | |
*** mattoliverau has joined #openstack-infra | 12:26 | |
*** Guest7756 has joined #openstack-infra | 12:26 | |
*** erikmwilson has joined #openstack-infra | 12:26 | |
*** Adri2000 has joined #openstack-infra | 12:26 | |
*** Hunner has joined #openstack-infra | 12:26 | |
*** yolanda has joined #openstack-infra | 12:26 | |
*** notmyname has joined #openstack-infra | 12:26 | |
*** xianghui has joined #openstack-infra | 12:26 | |
*** dansmith has joined #openstack-infra | 12:26 | |
*** tteggel_ has joined #openstack-infra | 12:26 | |
*** alazarev has joined #openstack-infra | 12:26 | |
*** hyakuhei_ has joined #openstack-infra | 12:26 | |
*** gothicmindfood has joined #openstack-infra | 12:26 | |
*** dkehn has joined #openstack-infra | 12:26 | |
*** juice has joined #openstack-infra | 12:26 | |
*** LinuxJedi has joined #openstack-infra | 12:26 | |
*** ianw has joined #openstack-infra | 12:26 | |
*** jesusaurus has joined #openstack-infra | 12:26 | |
*** dkliban_afk has joined #openstack-infra | 12:26 | |
*** denis_makogon has joined #openstack-infra | 12:27 | |
*** Longgeek has joined #openstack-infra | 12:27 | |
*** eikke has joined #openstack-infra | 12:27 | |
*** Daviey has joined #openstack-infra | 12:27 | |
*** mfink has joined #openstack-infra | 12:27 | |
*** dtantsur has joined #openstack-infra | 12:27 | |
*** maurosr has joined #openstack-infra | 12:27 | |
*** jraim has joined #openstack-infra | 12:27 | |
*** kevinbenton_ has joined #openstack-infra | 12:27 | |
*** BobBall_Xmas has joined #openstack-infra | 12:27 | |
*** rcarrillocruz has joined #openstack-infra | 12:27 | |
*** funzo_ has joined #openstack-infra | 12:27 | |
*** hamzy_vacation has joined #openstack-infra | 12:27 | |
*** primeministerp has joined #openstack-infra | 12:27 | |
*** doude_ has joined #openstack-infra | 12:27 | |
*** dhp has joined #openstack-infra | 12:27 | |
*** dstufft has joined #openstack-infra | 12:27 | |
*** gingerjiang has joined #openstack-infra | 12:27 | |
*** dteselkin has joined #openstack-infra | 12:27 | |
*** NikitaKonovalov has joined #openstack-infra | 12:27 | |
*** ruhe has joined #openstack-infra | 12:27 | |
*** zul has joined #openstack-infra | 12:27 | |
*** Apsu has joined #openstack-infra | 12:27 | |
*** andreaf_ has joined #openstack-infra | 12:27 | |
*** HeOS_ has joined #openstack-infra | 12:27 | |
*** EntropyWorks has joined #openstack-infra | 12:27 | |
*** davidlenwell has joined #openstack-infra | 12:27 | |
*** antonym has joined #openstack-infra | 12:27 | |
*** SpamapS has joined #openstack-infra | 12:27 | |
*** jhesketh has joined #openstack-infra | 12:27 | |
*** paul-- has joined #openstack-infra | 12:27 | |
*** mwagner_lap has quit IRC | 12:28 | |
*** dims has quit IRC | 12:28 | |
*** ddieterly has joined #openstack-infra | 12:28 | |
*** amuller has joined #openstack-infra | 12:28 | |
*** jerryz has joined #openstack-infra | 12:28 | |
*** Mmike has joined #openstack-infra | 12:28 | |
*** redrobot_away has joined #openstack-infra | 12:28 | |
*** sressot has joined #openstack-infra | 12:28 | |
*** mugsie has joined #openstack-infra | 12:28 | |
*** gnuoy has joined #openstack-infra | 12:28 | |
*** JoshNang has joined #openstack-infra | 12:28 | |
*** vladan has joined #openstack-infra | 12:28 | |
*** jokke_ has joined #openstack-infra | 12:28 | |
*** alaski has joined #openstack-infra | 12:28 | |
*** alexpilotti has joined #openstack-infra | 12:32 | |
*** andreaf has quit IRC | 12:33 | |
*** koolhead17 has joined #openstack-infra | 12:34 | |
*** andreaf has joined #openstack-infra | 12:34 | |
openstackgerrit | Emilien Macchi proposed openstack-infra/project-config: Bring stackforge/puppet modules to StoryBoard https://review.openstack.org/143655 | 12:39 |
*** yfried is now known as yfried|afk | 12:39 | |
*** _nadya_ has joined #openstack-infra | 12:40 | |
*** baoli has joined #openstack-infra | 12:41 | |
*** baoli has quit IRC | 12:42 | |
*** baoli has joined #openstack-infra | 12:43 | |
*** koolhead17 has quit IRC | 12:45 | |
*** dims has joined #openstack-infra | 12:51 | |
*** jerryz is now known as jerryz_ | 12:53 | |
*** jerryz_ is now known as jerryz | 12:53 | |
*** yfried|afk is now known as yfried | 12:54 | |
*** amuller has quit IRC | 13:00 | |
*** otter768 has joined #openstack-infra | 13:00 | |
*** Masahiro has joined #openstack-infra | 13:03 | |
*** unicell has joined #openstack-infra | 13:04 | |
*** dkliban_afk is now known as dkliban | 13:05 | |
*** yfried is now known as yfried|afk | 13:05 | |
*** otter768 has quit IRC | 13:05 | |
*** fandi has joined #openstack-infra | 13:06 | |
*** isaacb has quit IRC | 13:06 | |
*** Masahiro has quit IRC | 13:07 | |
*** ff has quit IRC | 13:09 | |
*** mbacchi has joined #openstack-infra | 13:14 | |
openstackgerrit | Merged openstack-infra/subunit2sql: Add a db api method to extract run_times for a single test https://review.openstack.org/142309 | 13:14 |
*** derekh has joined #openstack-infra | 13:15 | |
*** mwagner_lap has joined #openstack-infra | 13:16 | |
*** bswartz has quit IRC | 13:20 | |
*** jgallard_ has joined #openstack-infra | 13:21 | |
*** mika has quit IRC | 13:24 | |
*** yfried|afk is now known as yfried | 13:24 | |
*** vigneshvar has quit IRC | 13:24 | |
*** alexpilotti has quit IRC | 13:25 | |
*** spzala has joined #openstack-infra | 13:26 | |
*** isaacb has joined #openstack-infra | 13:26 | |
*** e0ne has joined #openstack-infra | 13:28 | |
*** e0ne_ has quit IRC | 13:30 | |
*** bhunter71 has quit IRC | 13:32 | |
*** mase_x200 has joined #openstack-infra | 13:33 | |
*** kgiusti has joined #openstack-infra | 13:33 | |
*** pblaho_ has joined #openstack-infra | 13:35 | |
*** mfink has quit IRC | 13:35 | |
*** dprince has joined #openstack-infra | 13:36 | |
*** garyk has quit IRC | 13:36 | |
*** ryanpetrello has joined #openstack-infra | 13:37 | |
*** garyk has joined #openstack-infra | 13:37 | |
*** andreykurilin_ has joined #openstack-infra | 13:38 | |
openstackgerrit | Denis M. proposed openstack/requirements: Add aioeventlet dependency https://review.openstack.org/138750 | 13:38 |
*** andreykurilin_ has quit IRC | 13:39 | |
*** ayoung has joined #openstack-infra | 13:41 | |
*** wuhg has joined #openstack-infra | 13:50 | |
*** mase_x200 has quit IRC | 13:55 | |
openstackgerrit | Thierry Carrez proposed openstack-infra/release-tools: Use PEP440 normalized versions https://review.openstack.org/143039 | 13:57 |
derekh | Are CI jobs having trouble with pip 6.0.X ? Having trouble in tripleo jobs and wondering if its happening in other places http://logs.openstack.org/17/98817/11/check-tripleo/check-tripleo-ironic-overcloud-f20-nonha/ae39aa3/console.html#_2014-12-23_08_13_33_100 | 13:57 |
*** bswartz has joined #openstack-infra | 13:59 | |
*** mfink has joined #openstack-infra | 13:59 | |
ashp | derekh: we had similar troubles at work yesterday with pbr | 14:00 |
ashp | I wasn't involved in it but I saw lots of pip 6.x talk | 14:00 |
derekh | ashp: ok, thanks | 14:01 |
ashp | i think the solution was to wait for upstream, aka you, to fix it :D | 14:01 |
ashp | derekh: https://www.youtube.com/watch?v=JYc05gZFly0 | 14:02 |
derekh | ashp: lol, will see if I can figure out whats going on | 14:02 |
*** Ala has quit IRC | 14:02 | |
*** mfer has joined #openstack-infra | 14:04 | |
ashp | good luck! I think we ended up pinning on 5.x for now as it was a pain | 14:04 |
*** Ala has joined #openstack-infra | 14:04 | |
*** mfer has quit IRC | 14:05 | |
*** yfried is now known as yfried|afk | 14:05 | |
*** abhijeetm has left #openstack-infra | 14:06 | |
*** mfer has joined #openstack-infra | 14:06 | |
*** mfer has quit IRC | 14:08 | |
dstufft | derekh: in pip 6 / setuptools 8 the meaning of specifiers changed in some ways | 14:10 |
*** mfer has joined #openstack-infra | 14:10 | |
fungi | derekh: ashp: that error indicates that ceilometer needs to approve their requirements sync change | 14:10 |
dstufft | one of that ways is instead of a confusing and mostly incomprehensible meaning of what , means, it changes it so that , means AND | 14:10 |
fungi | derekh: ashp: that sqlalchemy version specifier is not pep 440 compliant, and all the equivalents in the openstack/requirements repo have been fixed for weeks | 14:11 |
derekh | dstufft: fungi: ok, I'll got poke ceilometer | 14:11 |
*** redixin has joined #openstack-infra | 14:13 | |
dstufft | fungi: techincally the specifier is compliant, it's just nonsense because it's impossible to have a version which satisfies all the constraints </pedant> | 14:14 |
ashp | oh our issue might have been different then fungi, i just heard pip 6 and issues and assumed it was the same | 14:14 |
fungi | ashp: yeah, at this point i think we have all the setuptools 8/pip 6/virtualenv 12 issues we were seeing solved | 14:16 |
fungi | after a slew of changes to devstack and pbr in particular | 14:17 |
*** cdent has joined #openstack-infra | 14:17 | |
*** cnesa has joined #openstack-infra | 14:25 | |
*** yfried|afk is now known as yfried | 14:26 | |
*** amuller has joined #openstack-infra | 14:28 | |
*** ddieterly has quit IRC | 14:30 | |
*** ddieterly has joined #openstack-infra | 14:31 | |
*** baoli has quit IRC | 14:33 | |
*** imcsk8 has quit IRC | 14:34 | |
*** imcsk8 has joined #openstack-infra | 14:34 | |
*** ddieterly has quit IRC | 14:35 | |
*** yfried is now known as yfried|afk | 14:36 | |
*** darvon has quit IRC | 14:37 | |
*** ryanpetrello has quit IRC | 14:37 | |
*** darvon has joined #openstack-infra | 14:37 | |
*** baoli has joined #openstack-infra | 14:38 | |
*** ryanpetrello has joined #openstack-infra | 14:40 | |
*** dkranz has joined #openstack-infra | 14:40 | |
*** yfried|afk is now known as yfried | 14:40 | |
*** _nadya_ has quit IRC | 14:42 | |
openstackgerrit | Aleksey proposed openstack-infra/storyboard: Team and project groups delete methods https://review.openstack.org/143675 | 14:42 |
*** mfink_ has joined #openstack-infra | 14:44 | |
openstackgerrit | Aleksey proposed openstack-infra/storyboard: Team and project groups delete methods https://review.openstack.org/143675 | 14:44 |
*** radez_g0n3 is now known as radez | 14:44 | |
EmilienM | fungi: are you familiar with storyboard? | 14:46 |
EmilienM | fungi: If I want to add stackforge/puppet projects in StoryBoard, I just have to update project-config - gerrit/projects.yml with use-storyboard: true for the projects? | 14:46 |
EmilienM | I tried something: https://review.openstack.org/143655 | 14:47 |
EmilienM | I know you guys are using it, but not much doc on it yet | 14:47 |
*** enikanorov has joined #openstack-infra | 14:50 | |
*** yfried is now known as yfried|afk | 14:51 | |
*** Masahiro has joined #openstack-infra | 14:52 | |
*** mfer has quit IRC | 14:56 | |
*** Masahiro has quit IRC | 14:56 | |
*** yfried|afk is now known as yfried | 14:58 | |
dteselkin | Hi! Have you faced an issue when console logs from Jenkins jobs partially copied to node with logs? | 14:59 |
dteselkin | That occurs not for every job, though | 14:59 |
*** otter768 has joined #openstack-infra | 15:01 | |
fungi | EmilienM: right now we've only put infra projects on storyboard, mostly as an incentive to our developers to report on storyboard's shortcomings/needed features and as an incentive to work on fixing them. it's still missing features you probably expect in a general issue tracker | 15:03 |
*** dmsimard_away is now known as dmsimard | 15:04 | |
EmilienM | fungi: I was about setting up a Trello board for our project | 15:04 |
EmilienM | and sbadia suggested storyboard | 15:04 |
*** ryanpetrello has quit IRC | 15:05 | |
fungi | EmilienM: searching is limited, there's no e-mail functionality, no integration with our code review system yet, et cetera | 15:05 |
EmilienM | oh | 15:05 |
*** amitgandhinz has joined #openstack-infra | 15:06 | |
*** teran has quit IRC | 15:06 | |
*** otter768 has quit IRC | 15:06 | |
EmilienM | fungi: so you mean it's too early stage for our needs, right now? | 15:06 |
fungi | EmilienM: also no client library for the api yet | 15:07 |
EmilienM | fungi: fair enough | 15:07 |
*** teran has joined #openstack-infra | 15:07 | |
fungi | EmilienM: yeah, it's still in the early stages. we're actively discouraging random projects from using it yet unless their going to be working on tackling the work needed to implement the missing parts | 15:08 |
fungi | s/their/they're/ | 15:08 |
EmilienM | fungi: I would appreciate if you could put your words in the review | 15:08 |
EmilienM | because people from Puppet community is not reading this channel, but they will see the review | 15:08 |
EmilienM | fungi: thanks a lot for your inputs, I was doing wrong here | 15:09 |
fungi | EmilienM: sure. luckily gertty works from the car. gerrit's web interface is a bit laggy over wireless modem | 15:09 |
fungi | i'll see what i can do to summarize in a review comment | 15:09 |
EmilienM | fungi: I use gerrty every day | 15:09 |
EmilienM | gertty* | 15:09 |
fungi | best...thing...evar | 15:09 |
EmilienM | ahah | 15:09 |
EmilienM | very useful when I do Montreal <-> Paris | 15:10 |
*** ddieterly has joined #openstack-infra | 15:10 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 15:16 | |
sigmavirus24 | jogo glad you like it | 15:19 |
*** _nadya_ has joined #openstack-infra | 15:21 | |
*** jerryz has quit IRC | 15:24 | |
openstackgerrit | Sylvain Bauza proposed openstack-infra/elastic-recheck: Add query for Tempest bug 1405204 https://review.openstack.org/143681 | 15:25 |
uvirtbot | Launchpad bug 1405204 in tempest "SSH times out when trying to validate Tempest authentication" [Undecided,New] https://launchpad.net/bugs/1405204 | 15:25 |
*** k4n0 has quit IRC | 15:25 | |
openstackgerrit | Yair Fried proposed openstack-infra/project-config: Remove large-ops-{stable_branch} from tempest gate https://review.openstack.org/143648 | 15:25 |
*** hdd has joined #openstack-infra | 15:26 | |
*** _nadya_ has quit IRC | 15:26 | |
*** tonytan4ever has joined #openstack-infra | 15:26 | |
*** mjturek has joined #openstack-infra | 15:27 | |
*** lttrl has quit IRC | 15:28 | |
jaypipes | jeblair: so... I was able to figure out a solution for the 401 Unauthorized problem in gertty. :) | 15:29 |
*** dims has quit IRC | 15:30 | |
*** ihrachyshka has quit IRC | 15:31 | |
openstackgerrit | Merged openstack-dev/pbr: Merge tag '0.10.3' into master https://review.openstack.org/143325 | 15:31 |
jaypipes | jeblair: instead of re-using a requests.Session object in between calls to sync.get|post|put, I modified the code to create a new session for each call. Seems much more stable now (been running uninterrupted for >2 hours now) but a little slower of course. Would you be opposed to me adding a conf option to toggle the "new session for each HTTP call" behaviour? | 15:31 |
jaypipes | jeblair: oh, and in case you're wondering, I did try setting session.keep_alive = False, but it had no effect. | 15:31 |
*** isaacb has quit IRC | 15:32 | |
*** ryanpetrello has joined #openstack-infra | 15:33 | |
*** baoli has quit IRC | 15:33 | |
*** FlorianSW has joined #openstack-infra | 15:35 | |
*** achanda has joined #openstack-infra | 15:36 | |
jeblair | jaypipes: hrm; i still haven't had time to get fully up to speed on this, but istr i wrote it like that originally and then someone had a good reason for making the change to reuse it | 15:39 |
*** jerryz has joined #openstack-infra | 15:39 | |
anteaya | fungi: this might be the same chap who had difficulty with the multiple gerrit accounts, in any case he would like to run glusterfs on our nodes: https://review.openstack.org/#/c/143309/ | 15:40 |
anteaya | fungi: do we know if it is possible to do so? | 15:40 |
*** ddieterl_ has joined #openstack-infra | 15:41 | |
anteaya | I'll also ask in cinder to find out if they are aware of this proposed addition | 15:41 |
*** ddieterly has quit IRC | 15:41 | |
jeblair | jaypipes: i don't see "No JSON object could be decoded" in my .gertty.log (running for 11 days) | 15:44 |
jaypipes | jeblair: yeah, I don't know if it's something to do with the particular projects I'm subscribed to, or something particular to my setup :( | 15:45 |
mtreinish | anteaya: none of the support for doing that has landed anywhere yet: https://review.openstack.org/133102 and https://review.openstack.org/#/c/143308/1 | 15:45 |
mtreinish | anteaya: so it's probably a bit premature to discuss adding a job for it. | 15:46 |
jaypipes | jeblair: but I assure you, it happens on the hour when using the stock gertty :) | 15:46 |
anteaya | mtreinish: do we have any expectation the support will land? | 15:46 |
*** pblaho_ has quit IRC | 15:46 | |
*** ryanpetrello has quit IRC | 15:47 | |
dstufft | fungi: is the pip 6 + setuptools 8 stack holding up now? Nothing new that needs addressed? | 15:47 |
mtreinish | anteaya: no I'm not sure, sdague had strongish objections to the devstack patch before. Definitely not until after the holidays at least | 15:47 |
anteaya | and dtroyer is also not in favour | 15:47 |
anteaya | as there is a qa spec up outlining a plug in approach | 15:48 |
fungi | dstufft: i think just someone needs to review/approve https://review.openstack.org/142320 | 15:48 |
clarkb | fungi I will do that shortly | 15:49 |
fungi | clarkb: thanks | 15:49 |
dstufft | ah cool | 15:50 |
dstufft | I don't have +2 power on that so ;D | 15:50 |
jeblair | anteaya, mtreinish: however, configuring the job to run experimentally on the devstack and devstack-gate repos may be worth doing so that the changes to devstack and devstack-gate can be tested. if you want to do that i would suggest... | 15:51 |
jeblair | anteaya, mtreinish: not adding it to the integrated-gate template, but rather just to those specific projects; and additing it with a note of who is working on it, and a deadline for when it should either be finished or removed. | 15:52 |
jeblair | (i will put that in the project-config review) | 15:52 |
openstackgerrit | Merged openstack-dev/pbr: Merge tag '0.10.4' into master https://review.openstack.org/143326 | 15:53 |
openstackgerrit | Merged openstack-dev/pbr: Merge tag '0.10.5' into master https://review.openstack.org/143327 | 15:53 |
openstackgerrit | Merged openstack-dev/pbr: Merge tag '0.10.6' into master https://review.openstack.org/143328 | 15:53 |
dstufft | dem merges | 15:53 |
anteaya | jeblair: ah wonderful thank you, I had just finished commenting | 15:53 |
*** reed has joined #openstack-infra | 15:54 | |
anteaya | jeblair: so you are advising I lift my -2 | 15:54 |
jeblair | anteaya: yes, and i am surprised you would -2 that in the first place. that's rather strong. | 15:54 |
anteaya | okay | 15:55 |
anteaya | that was the sense I got from mtreinish comments and the comments on the other patches | 15:55 |
jeblair | anteaya: i did not get the sense that "people are opposed to this and it must never merge" | 15:56 |
anteaya | oh, that was what I read from dtroyers comments | 15:56 |
anteaya | perhaps I read his comments incorrectly | 15:56 |
jeblair | anteaya: dtroyer said he wanted it to use plugins | 15:57 |
*** zz_dimtruck is now known as dimtruck | 15:57 | |
* dtroyer wakes up | 15:57 | |
anteaya | yes | 15:57 |
anteaya | dtroyer: I'm using your name in vain do stop me | 15:57 |
jeblair | anteaya: that is not the same as dtroyer does not think glusterfs should ever be tested | 15:57 |
anteaya | true | 15:58 |
dtroyer | yes, we want the gluster review to be the first to use the extended plugin where the devstack code is not in the devstack repo…sdague is still working on that... | 15:58 |
dtroyer | I'm neutral on testing it, I don't want it in the devstack repo | 15:58 |
dtroyer | ceph either fwiw, I'll propose we change it to the same thing when this is working | 15:58 |
*** pcrews has joined #openstack-infra | 15:59 | |
anteaya | so does the approach for whether it is in devstack or accessed via a plugin change this patch? https://review.openstack.org/#/c/143309/ | 15:59 |
jeblair | anteaya: at any rate, there's a chicken and egg situation with some of these jobs, where it can be quite difficult to ensure that the associated changes are working correctly. in these cases it's often helpful to configure the jobs to run before those changes land so that the changes are self-testing | 15:59 |
anteaya | oh I didn't know that | 15:59 |
anteaya | as I have been operating on the belief that the changes should be expected to work by the time the job lands | 15:59 |
anteaya | so that is a new piece of information for me | 16:00 |
anteaya | thank you | 16:00 |
clarkb | fungi: did we revert the devstack setuptools pin on icehouse and juno too? Probably less urgent do to those | 16:00 |
jeblair | anteaya: the experimental pipeline is specifically to help get new jobs into shape | 16:00 |
anteaya | jeblair: so for an experimental job it is fine if the first piece is the job in the experimental pipeline and nothing else exists? | 16:01 |
jeblair | anteaya: regarding 143309, my proposal is that if the change were going into devstack, 309 should only enable it for the devstack (and perhaps devstack-gate repo), so it would at least need to change for that. if it's not going into devstack, then presumably it would need to be enabled for the cinder repo instead. so yes, it may still be too early to merge 309 in any form | 16:02 |
anteaya | okay thanks for clearing that up, and I appreciate you taking the time to teach me | 16:02 |
anteaya | saves me continuing to make mistakes | 16:02 |
jeblair | anteaya: and yes, it is okay for it to be the first piece. however, it should be balanced with whether it's desirable to run that job (if people -2 devstack patches with "we will never test this!" then probably not worth it), and also making sure that we don't end up with a bunch of orphaned jobs no one cares about (thus my suggestion about adding comments in the files with names/deadlines) | 16:03 |
*** markmcclain has joined #openstack-infra | 16:04 | |
openstackgerrit | Merged openstack-infra/system-config: Revert "Pin version of setuptools" https://review.openstack.org/142320 | 16:05 |
anteaya | right the piece about orphaned jobs is important (andreas the house cleaner would be unhappy) also I noticed you had an item up about the procedural -2 on the meeting last week | 16:05 |
anteaya | I am hoping we can get to it this week | 16:05 |
anteaya | I have been using -2 as a process for ensuring things don't get merged before other pieces are in place, either code or agreements | 16:06 |
anteaya | and I look forward to learning how to use that better and more in alignment with expectations | 16:06 |
anteaya | as it appears I am not there yet | 16:06 |
jeblair | jaypipes: i believe you :) i'm still kind of stumped | 16:06 |
jeblair | anteaya: you have to be careful with that; if you are on vacation, other people can't remove your -2 | 16:07 |
jaypipes | jeblair: :) hehe, yeah, I am too. I've been looking into oepnstackclient's use of requests, along with keystoneclient.Session usage of requests to try and see if there's something I'm missing... no luck yet :) | 16:07 |
jeblair | jaypipes: i see a handful of these: ProtocolError: ('Connection aborted.', BadStatusLine("''",)) | 16:08 |
jeblair | jaypipes: in my log, and some dns errors from yesterday during the rax ddos, but nothing in the form you quoted | 16:08 |
jaypipes | jeblair: those are from requests logging, IIRC. I saw those too, but didn't think much of them. | 16:08 |
jaypipes | jeblair: and they didn't happen on the hour mark, like this Unauthorized thing does. | 16:09 |
jeblair | jaypipes: so you figure, based on your additional local code/testing, that when you get the 401, your session object is hosed and must be replaced? or does it ever recover on its own? | 16:09 |
anteaya | jeblair: right | 16:09 |
jeblair | jaypipes: when you say 'the hour mark', you mean elapsed time, right? not wall-clock time? | 16:09 |
jaypipes | jeblair: yes, the session object is hosed. I tried to reset the session object by doing self.session = requests.Session() but that didn't work. and yes, elapsed time from starting gertty. | 16:10 |
jeblair | jaypipes: didn't you just propose resetting the session for each call as a solution? are you saying that that works as long as you do it _before_ the 401, but after the 401, even that is not helpful? | 16:11 |
*** e0ne is now known as e0ne_ | 16:11 | |
*** e0ne_ is now known as e0ne | 16:12 | |
*** Ala has quit IRC | 16:12 | |
jaypipes | jeblair: sorry, I was unclear... I was referring to a previous attempt at a solution that, upon getting a 401 Unauthorized, I would "reset" the session object and retry the same HTTP call. But that did not work. However, a solution that simply creates a brand new session object for each HTTP call does work... very odd. | 16:13 |
jeblair | jaypipes: yeah, it's hard for me to see how those would produce different behavior | 16:14 |
jaypipes | jeblair: I know, right? :) | 16:14 |
* dtroyer speculates wildly | 16:15 | |
dtroyer | is that auth cookie based? | 16:15 |
dtroyer | is the session reset not clearing cookies maybe? | 16:15 |
jaypipes | dtroyer: https://github.com/stackforge/gertty/blob/master/gertty/sync.py#L909-L914 <-- I put a block around that code there and if I got r.status_code == 401, I re-created the self.session object and retried the session.request() call. | 16:17 |
jeblair | jaypipes: what version of requests do you have? | 16:18 |
sigmavirus24 | dtroyer: define session reset? | 16:18 |
jaypipes | sigmavirus24: self.session = requests.Session() <-- session reset :) | 16:19 |
dtroyer | sigmavirus24: I was reusing jaypipes word…I think the link he posted is what he meant… | 16:19 |
sigmavirus24 | ah | 16:19 |
openstackgerrit | Ramy Asselin proposed openstack-infra/infra-specs: Add optimization opportunities https://review.openstack.org/143689 | 16:19 |
sigmavirus24 | I was going to say, we don't have a Session reset method =P | 16:19 |
jaypipes | sigmavirus24: do you have any idea why after an hour a requests.Session() would return a 401 Unauthorized from Gerrit? If I create a new requests.Session for each HTTP call to gerrit (in gertty), I don't get the 401 Unauthorized. But if I re-used the same session, exactly one hour after makign the first HTTP call with the requests.Session, I will get a 401 from Gerrit. really werid. | 16:21 |
jeblair | this should be http digest auth | 16:22 |
sigmavirus24 | jaypipes: if it is digest auth, I suspect you need to reinitialize the auth handler | 16:23 |
sigmavirus24 | The current HTTPDigestAuth handler does not check time and if it's already been used to authenticate continues to use what it has | 16:23 |
jeblair | so maybe the server has expired the nonce, or otherwise that got out of sync? | 16:23 |
sigmavirus24 | yeah | 16:23 |
sigmavirus24 | that's my best guess | 16:24 |
jeblair | jaypipes: can you check the requests version you are using? | 16:24 |
jeblair | (i'm also trying to figure out why i am not seeing this behavior but jaypipes is) | 16:24 |
jaypipes | jeblair: ah, sure, one sec, sorry missed your query above. | 16:24 |
sigmavirus24 | oh so, we recently improved some behaviour in the Digest Auth handler (in 2.5.0 maybe?) | 16:24 |
jeblair | sigmavirus24: i'm on 2.4.0 | 16:24 |
sigmavirus24 | One of 2.4.3 or 2.5.0 | 16:24 |
anteaya | pleia2: what repo do we have with a 'feature/gearman' branch? I've tried gear and gearman-plugin repos and so far no luck | 16:25 |
jaypipes | jeblair: 2.5.0 | 16:26 |
sigmavirus24 | So this is the specific improvement I'm thinking of: https://github.com/kennethreitz/requests/pull/2253 | 16:26 |
sigmavirus24 | But it only matters when the flow is 401 -> 302 -> 401 | 16:26 |
anteaya | clarkb: what repo has a 'logstash' branch? | 16:27 |
sigmavirus24 | Oh I bet I know what's going on jeblair/jaypipes | 16:27 |
openstackgerrit | Jeremy Stanley proposed openstack-dev/pbr: Use unsafe OpenPGP keys for testing https://review.openstack.org/142884 | 16:28 |
sigmavirus24 | jaypipes: can you patch L185 in requests/auth.py and remove the addition taking place on the call to setattr? | 16:28 |
jaypipes | sigmavirus24: I can try that, sure... | 16:28 |
sigmavirus24 | I bet it's not even trying to re-auth and it's skipping the 401 at that point | 16:28 |
jaypipes | sigmavirus24: gimme about one hour and 2 minutes to test :) | 16:28 |
jeblair | sigmavirus24: do you mean remove the whole line? | 16:28 |
sigmavirus24 | jeblair: no just switch it to setattr(self, 'num_401_calls', 1) | 16:29 |
* sigmavirus24 is going to through toegether a PR because I'm pretty sure this is blatantly wrong and my fault. If it works for jaypipes I'll push out 2.5.1 this afternoon | 16:29 | |
sigmavirus24 | s/ugh/w/ | 16:29 |
clarkb | anteaya: is there context for that? | 16:31 |
jaypipes | sigmavirus24: so, that line is line 157 in my /usr/local/lib/python2.7/dist-packages/requests/auth.py ... | 16:31 |
jaypipes | sigmavirus24: should I change that one? | 16:31 |
*** dims has joined #openstack-infra | 16:31 | |
jaypipes | sigmavirus24: sorry, no, line 171 looks like this: | 16:31 |
sigmavirus24 | jaypipes: sorry, you're changing setattr(self, 'num_401_calls', num_401_calls + 1) to setattr(self, 'num_401_calls', 1) | 16:31 |
jaypipes | sigmavirus24: setattr(self, 'num_401_calls', num_401_calls + 1) | 16:31 |
sigmavirus24 | L191 | 16:31 |
anteaya | clarkb: trying to verify https://review.openstack.org/#/c/143197/ | 16:31 |
sigmavirus24 | sorry I read the wrong line number jaypipes | 16:32 |
sigmavirus24 | jaypipes: https://github.com/kennethreitz/requests/pull/2253/files#diff-9f3a95293a5d26032b1c588167760362R191 | 16:32 |
anteaya | clarkb: I'm sure they exist, I would just like to learn where they are coming from | 16:32 |
sigmavirus24 | revert the change on that line | 16:32 |
jaypipes | sigmavirus24: got it. thx. | 16:32 |
sigmavirus24 | jaypipes: you're welcome | 16:32 |
clarkb | anteaya: I think feature/gearman is zuul. from when zuul grew gearman support. not sure of logstash branch | 16:32 |
*** jgallard_ has quit IRC | 16:33 | |
*** Hal_ has quit IRC | 16:33 | |
fungi | infra publications repo branches? | 16:33 |
anteaya | clarkb: I only see the master branch in zuul: http://git.openstack.org/cgit/openstack-infra/zuul/ | 16:33 |
jaypipes | sigmavirus24: ok, gertty started. will let you know if all is right in the world after an hour :) | 16:33 |
anteaya | fungi: well there are several publication branches I have verified | 16:34 |
anteaya | and a 'logstash' branch and a 'feature/gearman' branch I am working on verifying | 16:34 |
jeblair | sigmavirus24, jaypipes: i know we're past this, but i did confirm that gerrit expires digest tokens after 1 hour. it's also hardcoded, so i can't easily make it shorter for testing purposes :(. | 16:34 |
jaypipes | jeblair: oh, that's excellent snooping. good to know! :) | 16:35 |
sigmavirus24 | jeblair: no worries | 16:35 |
anteaya | I just learned 'debian/sid' branch comes from zuul/packaging | 16:35 |
jaypipes | jeblair: glad to know I'm not going totally crazy :) | 16:35 |
sigmavirus24 | jeblair: also, I think the HTTPDigestAuth class would otherwise renegotiate a nonce if it weren't buggy | 16:35 |
*** dims has quit IRC | 16:35 | |
sigmavirus24 | Of course when we tested that behaviour all seemed right with the world but we didn't know about this use case | 16:36 |
*** yfried has quit IRC | 16:37 | |
*** sputnik13 has joined #openstack-infra | 16:40 | |
jeblair | krotscheck: filed https://storyboard.openstack.org/#!/story/2000092 (posted a comment and had to reload to see it) | 16:40 |
*** Masahiro has joined #openstack-infra | 16:41 | |
*** carl_baldwin has joined #openstack-infra | 16:41 | |
*** dims has joined #openstack-infra | 16:41 | |
*** rkukura_ has joined #openstack-infra | 16:42 | |
sigmavirus24 | jeblair: jaypipes https://github.com/kennethreitz/requests/pull/2389 | 16:43 |
sigmavirus24 | Ping me there if this solves your problem | 16:43 |
jaypipes | k, will do. | 16:44 |
*** habib has joined #openstack-infra | 16:44 | |
*** rkukura has quit IRC | 16:44 | |
*** rkukura_ is now known as rkukura | 16:44 | |
*** sabeen has joined #openstack-infra | 16:45 | |
*** Masahiro has quit IRC | 16:45 | |
*** _nadya_ has joined #openstack-infra | 16:46 | |
*** hdd has quit IRC | 16:46 | |
*** markmcclain has quit IRC | 16:48 | |
sigmavirus24 | Also feel free to ping me here or elsewhere about requests related stuff as you need. Just don't bring up the v word because Kenneth won't ever let us change that | 16:49 |
*** boris-42 has joined #openstack-infra | 16:50 | |
*** skolekonov has quit IRC | 16:50 | |
*** garyk has quit IRC | 16:51 | |
*** cnesa has quit IRC | 16:52 | |
*** ChuckC has joined #openstack-infra | 16:54 | |
*** mriedem has joined #openstack-infra | 16:57 | |
krotscheck | jeblair: Ack | 16:57 |
* krotscheck wants to be done with the stupid email thing so he can get back to writing javascript :/ | 16:57 | |
*** sarob has joined #openstack-infra | 16:58 | |
*** andreaf has quit IRC | 17:00 | |
*** andreaf has joined #openstack-infra | 17:00 | |
zaro | morning | 17:00 |
openstackgerrit | Merged openstack-infra/project-config: ec2-driver: Added gate checks https://review.openstack.org/142908 | 17:01 |
anteaya | morning zaro | 17:02 |
*** HeOS has quit IRC | 17:02 | |
anteaya | asselin: are you about? | 17:02 |
*** otter768 has joined #openstack-infra | 17:02 | |
anteaya | asselin: just wanted to express my concerns about a workflow that hinges on a script which others may or may not be inclined to run | 17:03 |
*** cnesa has joined #openstack-infra | 17:03 | |
anteaya | asselin: I just want to ensure we are clear that the spliting out of modules offers the script running on cron as an option not a requirement | 17:03 |
anteaya | context: https://review.openstack.org/#/c/143689/1/specs/puppet-modules.rst | 17:04 |
*** sarob has quit IRC | 17:07 | |
*** otter768 has quit IRC | 17:07 | |
*** sabeen has quit IRC | 17:07 | |
pleia2 | anteaya: replied to your comments re: branches | 17:08 |
*** sarob has joined #openstack-infra | 17:08 | |
anteaya | jeblair clarkb or fungi I'm waiting to hear from a nodepool admin on this patch which enables injection of urls in the xenserver nodepool scripts: https://review.openstack.org/#/c/136700/ | 17:08 |
anteaya | pleia2: thanks | 17:08 |
*** sarob_ has joined #openstack-infra | 17:09 | |
clarkb | anteaya: we aren't consuming those scripts directly (at least not yet) so should be fine to update that if it comes from the xenserver folks or has been reviewed by them | 17:09 |
anteaya | pleia2: ah sorry, so you didn't add those branches | 17:09 |
*** Katherine has quit IRC | 17:09 | |
anteaya | clarkb: great, thanks and jeblair has +2'd | 17:10 |
pleia2 | anteaya: no, they're just reorganized, that's why they show up in the diff | 17:10 |
*** e0ne is now known as e0ne_ | 17:10 | |
anteaya | pleia2: your goal was just to add the publication branches? | 17:10 |
pleia2 | anteaya: only 4 branches were added (they're in the publications list) | 17:10 |
anteaya | ah okay my mistake thanks | 17:11 |
anteaya | I'll follow in with a patch offering some cleanup | 17:11 |
pleia2 | anteaya: yes, if you look at the first patchset that's all I did, then ajaeger asked me to organize them | 17:11 |
anteaya | sorry I didnt' look | 17:11 |
pleia2 | no worries, thanks for the review :) | 17:11 |
*** sarob has quit IRC | 17:12 | |
*** yfried has joined #openstack-infra | 17:13 | |
*** dimtruck is now known as zz_dimtruck | 17:15 | |
*** sarob_ has quit IRC | 17:16 | |
anteaya | asselin: also are you and jesusaurus in co-ordination with his lodgeit module split out and your large patch? https://review.openstack.org/#/c/142917/ | 17:16 |
*** sweston_ has joined #openstack-infra | 17:16 | |
*** baoli has joined #openstack-infra | 17:17 | |
openstackgerrit | Merged openstack-infra/project-config: XenServer: Enable injection of XVA and ISO urls https://review.openstack.org/136700 | 17:17 |
*** sweston is now known as spw | 17:17 | |
*** sweston_ has quit IRC | 17:17 | |
*** sweston has joined #openstack-infra | 17:18 | |
anteaya | this patch adds a nodepool script for mtreinish's subunit2sql work, would appreciate some nodepool admin eyes: https://review.openstack.org/#/c/136234/9 | 17:19 |
*** e0ne_ has quit IRC | 17:20 | |
*** esker has joined #openstack-infra | 17:20 | |
clarkb | mtreinish: anteaya: do we want to call that script from the devstack preparation scripst so that the testreposiroty DBs end up on our devstack test nodes? | 17:21 |
openstackgerrit | Merged openstack-infra/project-config: Add more publications branches to gerritbot https://review.openstack.org/143197 | 17:22 |
anteaya | clarkb: a good question | 17:22 |
openstackgerrit | Thierry Carrez proposed openstack-infra/infra-specs: StoryBoard: story types specification https://review.openstack.org/129267 | 17:22 |
*** yfried has quit IRC | 17:23 | |
*** jpich has quit IRC | 17:23 | |
mtreinish | clarkb: https://review.openstack.org/#/c/142940/ | 17:23 |
mtreinish | I can squash them together I guess | 17:24 |
clarkb | mtreinish: maybe? I do have a commetn for the second ne though | 17:24 |
anteaya | oh sorry I had thought you had seen that clarkb | 17:24 |
clarkb | mtreinish: so that will need to be addressed. | 17:25 |
clarkb | actually we may accomodate that so nevermind | 17:25 |
*** dannywilson has joined #openstack-infra | 17:25 | |
mtreinish | clarkb: ? | 17:26 |
clarkb | mtreinish: trying to figure out if that should be added to the dib stuff too | 17:26 |
mtreinish | ah, ok | 17:27 |
clarkb | ya I think it should be | 17:27 |
*** habib has quit IRC | 17:27 | |
clarkb | I thought maybe that we were running the cache script out of scripts/ for that but we do use dib elements for all of it | 17:27 |
mtreinish | the other thing I've thought of is I'll have to update d-g to not use .testrepository/0 anymore as part of the artifact collection | 17:27 |
openstackgerrit | Khai Do proposed stackforge/python-jenkins: Fix cancel_queue() method https://review.openstack.org/138964 | 17:28 |
clarkb | mtreinish: oh ya, we can just change that to do a sort and grab highest | 17:28 |
clarkb | or similar | 17:28 |
*** sressot has quit IRC | 17:29 | |
*** thedodd has joined #openstack-infra | 17:29 | |
mtreinish | clarkb: yeah I was probably just gonna switch it to do testr last --subunit > testrepository.subunit I think that should work fine | 17:29 |
*** teran has quit IRC | 17:30 | |
clarkb | mtreinish: I commneted on the second change. I think we can leave it unsquahsed and I will review the script itself now | 17:30 |
*** teran has joined #openstack-infra | 17:30 | |
clarkb | mtreinish: how is subunit2sql installed on the image build nodes? it is imported by your script but I don't expect it to be available? | 17:30 |
mtreinish | ok, thanks. I'll work on respinning the 2nd one now | 17:30 |
*** sressot has joined #openstack-infra | 17:31 | |
mtreinish | oh, that's a good point, I guess i'ts not being installed anywhere | 17:31 |
*** e0ne has joined #openstack-infra | 17:31 | |
openstackgerrit | Anita Kuno proposed openstack-infra/project-config: Cleanup gerritbot branches reporting to -infra https://review.openstack.org/143699 | 17:31 |
anteaya | pleia2: ^ | 17:32 |
clarkb | mtreinish: commented | 17:33 |
*** mwagner_lap has quit IRC | 17:33 | |
clarkb | jeblair: did you happen to restart geard on zuul to deal with the extra logging? | 17:33 |
clarkb | jeblair: if not I can likely do that today | 17:33 |
*** sdake has joined #openstack-infra | 17:34 | |
*** sdake has quit IRC | 17:34 | |
*** sdake has joined #openstack-infra | 17:34 | |
*** sdake_ has joined #openstack-infra | 17:34 | |
*** sdake_ has quit IRC | 17:34 | |
*** sdake_ has joined #openstack-infra | 17:34 | |
*** openstackgerrit has quit IRC | 17:34 | |
*** openstackgerrit has joined #openstack-infra | 17:35 | |
jeblair | clarkb: i have not. i'm digging into geard ssl. it looks like SSL_WANT_READ is actually just masking EAGAIN | 17:35 |
pleia2 | anteaya: thanks, looks like feature/gearman used to be a zuul branch | 17:35 |
jeblair | clarkb: it's pretty easy for me to restart it, why don't i? | 17:35 |
jaypipes | jeblair, sigmavirus24: \o/ it's been >1 hour and no errors in my log. looks like that one liner change worked. | 17:36 |
*** sdake is now known as steak | 17:36 | |
jeblair | jaypipes: woo! | 17:36 |
jaypipes | jeblair: I'll submit a patch that bumps requests requirement to 2.5.1. | 17:36 |
jaypipes | jeblair: once sigmavirus24 releases it :) | 17:36 |
jeblair | jaypipes: hopefully that patch is self-testing ;) | 17:36 |
sigmavirus24 | heh | 17:37 |
jaypipes | jeblair: indeed. | 17:37 |
*** koolhead17 has joined #openstack-infra | 17:37 | |
jeblair | actually, heh, it has absolutely no tests, so even that isn't self-testing. :) | 17:38 |
*** adalbas has joined #openstack-infra | 17:38 | |
*** jerryz has quit IRC | 17:38 | |
*** fandi has quit IRC | 17:39 | |
*** cnesa has quit IRC | 17:39 | |
*** cnesa has joined #openstack-infra | 17:40 | |
*** hichtakk has joined #openstack-infra | 17:41 | |
clarkb | jeblair go for it | 17:41 |
*** amuller has quit IRC | 17:41 | |
anteaya | pleia2: ah | 17:42 |
openstackgerrit | Matthew Treinish proposed openstack-infra/project-config: Use prepare_tempest_testrepository as part of devstack node script https://review.openstack.org/142940 | 17:43 |
mtreinish | clarkb: ^^^ I'm not sure that's right | 17:43 |
anteaya | pleia2: how do you see a branch that used to exist? | 17:43 |
*** derekh has quit IRC | 17:43 | |
pleia2 | anteaya: some google searching and some assumptions | 17:44 |
pleia2 | led me to find some patches in zuul against that branch in gerrit | 17:44 |
*** SumitNaiksatam has quit IRC | 17:45 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/infra-manual: Explain changes and patch sets https://review.openstack.org/143644 | 17:47 |
anteaya | pleia2: not that important, just a passing curiosity | 17:47 |
*** stevemar has joined #openstack-infra | 17:47 | |
jeblair | restarting geard | 17:49 |
*** fandi has joined #openstack-infra | 17:49 | |
*** ildikov_ has joined #openstack-infra | 17:51 | |
*** s0nea has quit IRC | 17:52 | |
*** wuhg has quit IRC | 17:53 | |
*** s0nea has joined #openstack-infra | 17:54 | |
*** ildikov has quit IRC | 17:54 | |
openstackgerrit | Matthew Treinish proposed openstack-infra/system-config: Add support to the subunit workers to reuse zuul uuids https://review.openstack.org/139750 | 17:55 |
*** e0ne has quit IRC | 17:55 | |
*** Ryan_Lane has joined #openstack-infra | 17:55 | |
openstackgerrit | Ian Cordasco proposed stackforge/gertty: Do not use requests 2.5.0 https://review.openstack.org/143703 | 17:56 |
sigmavirus24 | jeblair: jaypipes ^ 2.5.1 is out | 17:56 |
jaypipes | sigmavirus24: rock on. ty sir! | 17:57 |
*** zz_dimtruck is now known as dimtruck | 17:57 | |
sigmavirus24 | yw | 17:57 |
anteaya | zaro: can you look at https://review.openstack.org/#/c/143699/1 please? our current gerrit branch is openstack/2.8.4 is it not? | 17:58 |
anteaya | zaro: I assume we are working on the latest of our branches, yes? | 17:58 |
openstackgerrit | Merged stackforge/gertty: Do not use requests 2.5.0 https://review.openstack.org/143703 | 17:59 |
*** armax has joined #openstack-infra | 18:02 | |
*** lttrl has joined #openstack-infra | 18:05 | |
anteaya | looks like there might be a meeting topic issue in -meeting, will know for sure after keystone finishes | 18:05 |
*** bdpayne has joined #openstack-infra | 18:07 | |
openstackgerrit | Merged openstack-infra/project-config: Remove icehouse compat checks from glance/keystoneclient https://review.openstack.org/143566 | 18:08 |
*** SumitNaiksatam has joined #openstack-infra | 18:09 | |
*** teran has quit IRC | 18:09 | |
*** _nadya_ has quit IRC | 18:11 | |
vponomaryov | Have anyone faced same sudden error using py26 job? - https://jenkins06.openstack.org/job/gate-manila-python26/444/console | 18:13 |
*** vigneshvar has joined #openstack-infra | 18:14 | |
openstackgerrit | Khai Do proposed stackforge/python-jenkins: Fix cancel_queue() method https://review.openstack.org/138964 | 18:14 |
*** garyk has joined #openstack-infra | 18:15 | |
clarkb | vponomaryov: that failed bceause zero tests were run. The test listing discovered zero tests so ran no tests | 18:15 |
vponomaryov | clarkb: test run was before and it was the only one. Now there are two of them and second fails | 18:15 |
*** patrickeast has joined #openstack-infra | 18:17 | |
asselin | anteaya, I'm around to coordinate with jesusaurus | 18:18 |
asselin | anteaya, jesusaurus if ledgeit module is ready to go, then push it through. I'll rebase from there. | 18:19 |
zaro | anteaya: taking a look | 18:20 |
clarkb | jeblair: I think gertty may have trouble with those tag merge commits. Haven't been able to debug beyond that though | 18:21 |
anteaya | asselin: great thanks, just want to ensure there is a clear way forward | 18:23 |
anteaya | zaro: thank you | 18:23 |
*** achanda has quit IRC | 18:25 | |
*** achanda has joined #openstack-infra | 18:25 | |
jeblair | clarkb: yeah, gertty does not handle multiple parents at all yet | 18:26 |
jeblair | clarkb: implementation status is basically "TODO: handle multiple parents" :) | 18:26 |
jeblair | (it's not even in the db schema) | 18:26 |
*** achanda has quit IRC | 18:26 | |
*** achanda has joined #openstack-infra | 18:26 | |
*** Masahiro has joined #openstack-infra | 18:29 | |
*** carwatt has joined #openstack-infra | 18:32 | |
clarkb | fungi: are you still around today? 141666 and 142323 should be abandoned? and pbr master is basically happy now? | 18:32 |
jesusaurus | asselin: yes, i think the lodgeit module is ready | 18:32 |
jesusaurus | clarkb: jeblair: either of you feel like reviewing https://review.openstack.org/#/c/142917/? | 18:33 |
bnemec | vponomaryov: I saw that on an oslo.messaging patch too. It looks like something is getting confused by the tox -epy26 -- pbr freeze call. | 18:34 |
jesusaurus | asselin: why do you think the current split-out process is too involved? | 18:34 |
*** Masahiro has quit IRC | 18:34 | |
asselin | jesusaurus, because there are a few changes spread out in multiple files | 18:34 |
*** amuller has joined #openstack-infra | 18:34 | |
*** harlowja has joined #openstack-infra | 18:35 | |
bnemec | As in, it runs all the tests, then it runs the pbr freeze, which doesn't run any tests, and then it checks whether tests were run, and correctly finds that the last tox run had 0 tests. | 18:35 |
clarkb | bnemec: oh you may be right I think that is a bug in the script `tox -epy26 -- pbr freeze` says run the pbr freeze tests | 18:35 |
clarkb | which doesn't make sense | 18:35 |
*** e0ne has joined #openstack-infra | 18:35 | |
bnemec | clarkb: Yeah, something seems wrong there. | 18:36 |
*** andreaf has quit IRC | 18:36 | |
clarkb | bnemec: ya it should be doing `.tox/py26/bin/pbr freeze` not `tox -epy26 -- pbr freeze` | 18:37 |
*** andreaf has joined #openstack-infra | 18:37 | |
clarkb | I am not finding freezecmd in our scripts though | 18:38 |
openstackgerrit | Khai Do proposed stackforge/python-jenkins: Fix cancel_queue() method https://review.openstack.org/138964 | 18:38 |
clarkb | oh my git was out of date | 18:38 |
jesusaurus | asselin: i like keeping the changes together to make it easier to follow the logical story backwards through the commit history, but will defer to the cores' opinions | 18:39 |
fungi | sorry we stopped and got out for lunch, back now | 18:39 |
fungi | clarkb: yeah, i'll abandon them now that the other stuff merged | 18:40 |
*** ihrachyshka has joined #openstack-infra | 18:40 | |
clarkb | bnemec: I am whipping up the fix | 18:40 |
clarkb | bnemec: thank you for pointing that out | 18:40 |
openstackgerrit | James E. Blair proposed openstack-infra/gear: Add SSL functional test https://review.openstack.org/143235 | 18:40 |
openstackgerrit | James E. Blair proposed openstack-infra/gear: Add simple functional test https://review.openstack.org/143217 | 18:40 |
openstackgerrit | James E. Blair proposed openstack-infra/gear: Fix SSL non-blocking IO support https://review.openstack.org/143716 | 18:40 |
carl_baldwin | Greetings, I have a quick question. Is there something going on with Jenkins creating false merge conflicts? | 18:41 |
anteaya | carl_baldwin: what prompts you to ask? | 18:41 |
carl_baldwin | I saw some strange merge conflicts yesterday and I wondered. Today I saw this one: https://review.openstack.org/#/c/134339/ | 18:41 |
bnemec | clarkb: Thanks. vponomaryov gets credit though - I was still waiting to see if it would happen again on the recheck. :-) | 18:41 |
carl_baldwin | anteaya: ^ | 18:41 |
anteaya | carl_baldwin: looking | 18:42 |
carl_baldwin | Here is one that failed after the gate and I don’t see how the conflict could have happened: https://review.openstack.org/#/c/143179/ | 18:42 |
*** spzala has quit IRC | 18:42 | |
jeblair | clarkb: i think i have addressed the ssl issues in https://review.openstack.org/143716 | 18:42 |
jeblair | clarkb: going to run some longer tests now | 18:43 |
asselin | jesusaurus, updating the spec. you can post your comments there. | 18:43 |
carl_baldwin | anteaya: Here is a third one that didn’t seem like it should have conflicted: https://review.openstack.org/#/c/118491/ | 18:43 |
asselin | jesusaurus, https://review.openstack.org/#/c/143689/ | 18:43 |
openstackgerrit | Clark Boylan proposed openstack-infra/project-config: We want to run freese in venv not with tox target https://review.openstack.org/143717 | 18:44 |
clarkb | bnemec: vponomaryov ^ that is the fix once that is in I will rebuild images for all the things | 18:44 |
anteaya | carl_baldwin: you ask a good question, I am uncertain how there can be a merge conflict with a patch that introduces a new directory and file | 18:44 |
clarkb | jeblair: fungi anteaya ^ | 18:44 |
clarkb | mordred: ^ | 18:44 |
openstackgerrit | Ramy Asselin proposed openstack-infra/infra-specs: Add module-split workflow optimizations https://review.openstack.org/143689 | 18:45 |
carl_baldwin | anteaya: So, with these three patches that don’t appear to conflict with anything in master, I’m now wondering if something is broken. | 18:45 |
*** dizquierdo has quit IRC | 18:45 | |
bnemec | clarkb: Nice, thanks | 18:45 |
clarkb | carl_baldwin: gerrit specifically calls out the conflict in a comment on 143179 | 18:46 |
clarkb | carl_baldwin: same thing with 118491 | 18:46 |
carl_baldwin | clarkb: Nothing has changed around that comment in over a month. | 18:47 |
carl_baldwin | clarkb: b8930e8f (Gary Kotton 2014-11-10 08:19:06 -0800 | 18:47 |
jeblair | carl_baldwin: 134338 could be fallout from some geard issues we were having last week, you can 'recheck' to be sure. | 18:47 |
carl_baldwin | I’m not a git newby. This shouldn’t conflict. | 18:47 |
*** FallenPegasus has joined #openstack-infra | 18:48 | |
carl_baldwin | jeblair: I will recheck them. | 18:48 |
clarkb | jeblair: no rechecking won't help | 18:48 |
clarkb | carl_baldwin: ^ | 18:48 |
clarkb | gerrit is saying there is a path conflict | 18:48 |
jeblair | carl_baldwin: jgit is not as good as cgit in resolving merges, in those cases they just need to be rebased | 18:48 |
clarkb | it is not zuul/jenkins reporting the merge conflict. and since it is gerrit you get a specific message about it | 18:48 |
fungi | was it part of a long patch series? is this the jgit octomerge bug? | 18:48 |
jeblair | clarkb: i was specifically referring to a change that has not merged | 18:48 |
clarkb | fungi: I think its actually neutron's symlink mess | 18:48 |
fungi | oh | 18:49 |
jeblair | clarkb: s/merged/attempted to merge | 18:49 |
clarkb | jeblair: oh missed the number | 18:49 |
jeblair | clarkb: and the change i was referring to has no comments from gerrit | 18:49 |
*** steak has quit IRC | 18:49 | |
clarkb | fungi: since apparently the "fix" for that was to complain about pbr then copy pasta code | 18:50 |
clarkb | fungi: the copy pasta movement likely explains the path conflicts | 18:50 |
carl_baldwin | jeblair: I understand the differences between jgit and cgit. But, these examples show a conflict where there is no code changing anywhere close. | 18:51 |
fungi | carl_baldwin: for the one where gerrit complained about the merge, if it was part of a patch series of more than a few patches, gerrit attempts to merge them serially but has to do so with merge commits since their original parents aren't the branch tip, and after a few of those in a row it gets crabby and decides it can't calculate the merge and complains that there's a conflict even though there | 18:52 |
fungi | isn't | 18:52 |
*** markmcclain has joined #openstack-infra | 18:52 | |
carl_baldwin | fungi: Which one are you talking about exactly? | 18:53 |
clarkb | fungi: though looking at the file lists tests/contrib and tests/functional/contrib are not in them so maybe not the file movement | 18:53 |
*** HeOS has joined #openstack-infra | 18:53 | |
fungi | carl_baldwin: whichever one was gerrit refusing to merge the approved change after it passed tests | 18:54 |
fungi | i'm pulling up your examples now to get specifics | 18:54 |
clarkb | https://review.openstack.org/#/c/143179/ for example | 18:54 |
clarkb | and https://review.openstack.org/#/c/118491/ | 18:54 |
clarkb | both have the message about path conflicts from gerrits diffy cuckoo bird | 18:55 |
fungi | 143179 doesn't look like the patch series jgit bug at least | 18:55 |
jhesketh | Morning | 18:56 |
anteaya | morning jhesketh | 18:56 |
anteaya | jhesketh: I see a meeting in your future :D | 18:56 |
jhesketh | Heh :-) | 18:56 |
*** mjturek has quit IRC | 18:56 | |
fungi | manually rebasing 143179 seems to work currently | 18:57 |
fungi | though i wonder if the changing state of master means that there were conflicts since reverted | 18:58 |
carl_baldwin | https://review.openstack.org/#/c/143179/ is such a simple change. There is no patch series and there isn’t any patch in the current master that touches any line close to the change and isn’t already reachable from the commit. | 18:59 |
carl_baldwin | fungi: I’ve manually rebased them all successfully. However, I used cgit. Are you manually rebasing with cgit? | 18:59 |
*** ddieterl_ has quit IRC | 19:00 | |
fungi | carl_baldwin: yeah, i tested with cgit and that definitely worked | 19:00 |
fungi | also there have been no merges to neutron master which would explain that conflict from an hour ago being invalidated | 19:00 |
pleia2 | meeting time :) | 19:00 |
carl_baldwin | fungi: A revert of a conflict would show up in git annotate. I don’t see any such revert. | 19:00 |
*** koolhead17 has quit IRC | 19:01 | |
fungi | most recent merges to master were 75 minutes ago and 8 hours ago | 19:01 |
jeblair | fungi, carl_baldwin: the conflict from an hour ago is still probably the geard thing. | 19:01 |
*** markmcclain has quit IRC | 19:01 | |
jeblair | the conflict from gerrit was probably a jgit-cgit incompatibility | 19:01 |
carl_baldwin | jeblair: geard is where my skill set stops. :) | 19:01 |
vponomaryov | clarkb: thanks! | 19:02 |
*** otter768 has joined #openstack-infra | 19:03 | |
carl_baldwin | jeblair: I don’t see how jgit could have messed up https://review.openstack.org/#/c/143179. I’m trying to show above how this patch has no merge complexity that possibly could have tripped up jgit. | 19:03 |
*** hdd has joined #openstack-infra | 19:04 | |
*** ryanpetrello has joined #openstack-infra | 19:06 | |
carl_baldwin | jeblair: But, jgit does seem to be unable to handle it. I cannot see why. I created https://review.openstack.org/#/c/143721/ so that I could hit the rebase button in the UI. It failed. | 19:07 |
*** otter768 has quit IRC | 19:08 | |
*** vhoward has left #openstack-infra | 19:08 | |
jeblair | carl_baldwin: aiui, the octomerge jgit issue that fungi was referring to does not involve complexity as much as git features it just doesn't implement | 19:08 |
carl_baldwin | jeblair: This shouldn’t be an octomerge. | 19:09 |
openstackgerrit | Merged openstack-infra/project-config: We want to run freese in venv not with tox target https://review.openstack.org/143717 | 19:09 |
carl_baldwin | There is no patch series. | 19:09 |
clarkb | carl_baldwin: 881344280ce47a1af41d8b74d8fd779959510068 maybe the conflict | 19:10 |
carl_baldwin | clarkb: I checked that one. It is already a decendent of this patch. | 19:11 |
*** subscope has joined #openstack-infra | 19:11 | |
carl_baldwin | clarkb: Shown by this command: “git merge-base 4e79a4d3cda7905d77a56d6bc97f59790d113d1f 881344280ce47a1af41d8b74d8fd779959510068” | 19:12 |
*** carwatt has quit IRC | 19:12 | |
clarkb | ya its in the log of the change | 19:13 |
carl_baldwin | clarkb: That too. | 19:13 |
*** garyk has quit IRC | 19:14 | |
clarkb | carl_baldwin: you know I wonder if we are focusing on the diff too much. I wonder if it is due to the function contrib test move stuff | 19:14 |
clarkb | is that change on one side and master on the other | 19:15 |
*** prad has joined #openstack-infra | 19:15 | |
clarkb | gerrit shows you diff against HEAD^ not diff against target branch | 19:15 |
carl_baldwin | clarkb: ^ I understand what you’re trying to tell me about git diff. | 19:17 |
*** ryanpetrello has quit IRC | 19:17 | |
*** dimtruck is now known as zz_dimtruck | 19:17 | |
carl_baldwin | The command I ran was merge-base and that was a simple way to show that the patch is reachable in the commit graph. | 19:18 |
clarkb | carl_baldwin: not that patch | 19:20 |
clarkb | carl_baldwin: I think you are right that its not at fault | 19:20 |
*** FallenPegasus has quit IRC | 19:20 | |
clarkb | carl_baldwin: instead I think that the movement of neutron/tests/**/contrib is likely at fault | 19:20 |
carl_baldwin | clarkb: This must be some drawback with jgit that I don’t understand and haven’t seen before. None of these patches touch files in that area. Even a very naive path merge should not have any problem ignoring it. Do we know any jgit maintainers who might want to take a look at this case? | 19:24 |
*** weshay has joined #openstack-infra | 19:24 | |
carl_baldwin | clarkb: Meanwhile, I’ll rebase using cgit and resubmit the patches to the gate. | 19:25 |
clarkb | carl_baldwin: I think jgit is intentionally more conservative than cgit? iirc one of the major ocmplaints with git is it is often thought to be too helpful with conflict resolution | 19:25 |
clarkb | carl_baldwin: but I don't know any jgit maintainers | 19:26 |
clarkb | so can't have them comment on that | 19:26 |
carl_baldwin | clarkb: fair enough | 19:26 |
carl_baldwin | clarkb: Could you guys provide me the version of jgit in use on our gerrit? | 19:29 |
*** ddieterly has joined #openstack-infra | 19:30 | |
*** ddieterly has quit IRC | 19:32 | |
*** kgiusti has left #openstack-infra | 19:32 | |
*** ddieterly has joined #openstack-infra | 19:32 | |
carl_baldwin | clarkb: I see our gerrit version is 2.8.4-15-g6dc8444 but I’m not sure if that is sufficient to nail down the version of jgit it uses. | 19:33 |
clarkb | carl_baldwin: in theory it is http://git.openstack.org/cgit/openstack-infra/gerrit/tree/?h=openstack/2.8.4 but I have no idea how to figure out from buck what the version is | 19:34 |
clarkb | zaro: ^ | 19:34 |
*** mfer has joined #openstack-infra | 19:34 | |
carl_baldwin | clarkb: Thanks. | 19:35 |
*** ddieterly has quit IRC | 19:37 | |
openstackgerrit | Matthew Treinish proposed openstack-infra/devstack-gate: Handle more than one stream stored testrepository https://review.openstack.org/143727 | 19:38 |
*** spzala has joined #openstack-infra | 19:42 | |
*** kgiusti has joined #openstack-infra | 19:45 | |
*** teran has joined #openstack-infra | 19:45 | |
*** zz_zz_zz_zz_zz_z is now known as sabari | 19:49 | |
*** otter768 has joined #openstack-infra | 19:54 | |
*** rlandy has joined #openstack-infra | 19:55 | |
*** ildikov_ has quit IRC | 19:55 | |
*** KurtMartin is now known as kmartin | 19:55 | |
zaro | carl_baldwin, clarkb : git grep jgit in 'BUCK' files show ver is 3.5.1.201410131835-r | 19:55 |
carl_baldwin | zaro: thanks. | 19:56 |
zaro | carl_baldwin: ohh sorry, that was on master. please try to git grep on the 2.8.4 branch to get the version. | 19:57 |
carl_baldwin | zaro: I will. | 19:57 |
*** baoli has quit IRC | 19:58 | |
fungi | since there's no tc meeting to gawk at, i'm going to afk and take over driving for the afternoon. i'll check back in at some point this evening | 20:02 |
clarkb | ok I am going to start spinning up new nodepool nodes now | 20:02 |
clarkb | fungi: have fun | 20:02 |
anteaya | fungi: happy trails | 20:02 |
fungi | thanks. pavement as far as the eye can see | 20:02 |
clarkb | fungi: you don't have windy mountain roads? | 20:02 |
clarkb | those are fun | 20:02 |
fungi | those come in a couple hours | 20:02 |
*** mika has joined #openstack-infra | 20:03 | |
*** fifieldt has quit IRC | 20:03 | |
pleia2 | need to scoot off for a bit to handle some family things, bbiab | 20:04 |
*** bswartz has quit IRC | 20:05 | |
dteselkin | Hi, could anyone help me a bit with scp plugin for Jenkins? | 20:06 |
*** yfried has joined #openstack-infra | 20:06 | |
clarkb | dteselkin: sure what is the question? | 20:07 |
dteselkin | clarkb, I'm trying to figure out how it copies console.log to the static node | 20:08 |
dteselkin | I have a problem with the logs on that node | 20:08 |
dteselkin | Some of them are truncated | 20:08 |
dteselkin | I know that you use a custom version of plugin | 20:08 |
dteselkin | So, first of all, where can I get it to try? | 20:09 |
clarkb | dteselkin: its not custom as much as it is unreleased. | 20:09 |
dteselkin | Well, but it works :) | 20:09 |
clarkb | dteselkin: https://github.com/jenkinsci/scp-plugin is the code | 20:09 |
dteselkin | Thanks, I'll try | 20:09 |
clarkb | dteselkin: and http://tarballs.openstack.org/ci/scp.jpi should be our most recent build of it | 20:10 |
dteselkin | Cool, thanks! | 20:10 |
dteselkin | Am I right that this plugin takes console log using some kind of API ? | 20:10 |
clarkb | dteselkin: it runs within jenkins so it just gets it internally | 20:11 |
dteselkin | Hmm, that's strange | 20:11 |
*** teran_ has joined #openstack-infra | 20:11 | |
clarkb | there is a file like java object that the plugin reads from and writes out via scp | 20:11 |
dteselkin | I mean, I looked at the console log file on jenkins noe | 20:11 |
dteselkin | *node | 20:11 |
dteselkin | It have a bit different format | 20:12 |
clarkb | the format should be the same. it reads the same underlying html file | 20:12 |
clarkb | so it gets properly escaped and all that | 20:12 |
dteselkin | There is some kind of prefix before each line, and that prefix looks like base64 string | 20:13 |
dteselkin | So jenkins should store console log as html file? | 20:13 |
clarkb | no jenkins actually stores the log in base64 iirc | 20:13 |
clarkb | but the data served via http to your web browser is the same data read by the plugin internally | 20:14 |
*** teran has quit IRC | 20:14 | |
dteselkin | os_loganalyzer does that? | 20:14 |
clarkb | the plugin doesn't read it from disk. It reads it from the specail java file object thing that applies all of the appopriate filters to the stream | 20:14 |
clarkb | dteselkin: no | 20:14 |
dteselkin | Ah, ok. So scp plugin uses some Jenkins internals and read it as html file, then copies to the destination server? | 20:15 |
clarkb | yes | 20:15 |
dteselkin | Ok, so probably the problem is in the plugin in my case | 20:16 |
dteselkin | One more question | 20:16 |
dteselkin | How can I enable debug logging for the plugin? | 20:16 |
*** fifieldt has joined #openstack-infra | 20:16 | |
clarkb | dteselkin: I think you can do that via jenkins log config management in the gui | 20:17 |
dteselkin | And is it possible to add more debugging outputs manually to the plugin? | 20:17 |
clarkb | dteselkin: there is also a way to set it up in a config file on disk but I forget how that works | 20:17 |
clarkb | dteselkin: you can edit the plugin and rebuild it with more logging info | 20:17 |
dteselkin | Ok, I'll dig into this | 20:17 |
dteselkin | One stupid question - how to rebuild it? | 20:18 |
dteselkin | Sorry, I'm not familiar with Java :( | 20:18 |
*** amuller has quit IRC | 20:18 | |
*** Masahiro has joined #openstack-infra | 20:18 | |
*** ociuhandu has quit IRC | 20:19 | |
clarkb | dteselkin: `mvn package` will run maven to build it | 20:19 |
dteselkin | clarkb, ok, I'll try. Thank you! | 20:22 |
*** Masahiro has quit IRC | 20:22 | |
*** teran_ has quit IRC | 20:24 | |
*** bswartz has joined #openstack-infra | 20:26 | |
*** dizquierdo has joined #openstack-infra | 20:26 | |
*** dprince has quit IRC | 20:29 | |
*** teran has joined #openstack-infra | 20:30 | |
*** sabari is now known as zz_sabari | 20:31 | |
*** ddieterly has joined #openstack-infra | 20:31 | |
* anteaya goes for a walk | 20:34 | |
*** teran has quit IRC | 20:35 | |
*** ddieterly has quit IRC | 20:36 | |
*** otter768 has quit IRC | 20:37 | |
*** yamahata has joined #openstack-infra | 20:38 | |
*** sdake_ has quit IRC | 20:38 | |
*** kgiusti has left #openstack-infra | 20:42 | |
*** esker has quit IRC | 20:42 | |
*** dizquierdo has quit IRC | 20:43 | |
*** harlowja has quit IRC | 20:44 | |
*** yamahata has quit IRC | 20:45 | |
*** teran has joined #openstack-infra | 20:47 | |
*** markmcclain has joined #openstack-infra | 20:47 | |
*** harlowja has joined #openstack-infra | 20:48 | |
*** ihrachyshka has quit IRC | 20:49 | |
*** yamahata has joined #openstack-infra | 20:52 | |
*** mfer has quit IRC | 20:52 | |
*** achanda has quit IRC | 20:54 | |
*** achanda has joined #openstack-infra | 20:54 | |
*** carl_baldwin has quit IRC | 20:55 | |
*** steak has joined #openstack-infra | 20:57 | |
*** ihrachyshka has joined #openstack-infra | 20:58 | |
*** achanda has quit IRC | 20:59 | |
*** markmcclain1 has joined #openstack-infra | 21:00 | |
*** teran has quit IRC | 21:00 | |
*** ryanpetrello has joined #openstack-infra | 21:00 | |
*** markmcclain has quit IRC | 21:00 | |
*** markmcclain1 has quit IRC | 21:02 | |
*** markmcclain has joined #openstack-infra | 21:02 | |
*** markmcclain has quit IRC | 21:03 | |
*** EmilienM is now known as EmilienM|afk | 21:04 | |
clarkb | image builds are still going. centos6 should be all done and trusty is one its way | 21:04 |
clarkb | precise and variants will be next. Iam avoiding devstack-* since they don't use those scripts so don't need the updates | 21:05 |
*** mfer has joined #openstack-infra | 21:05 | |
*** rlandy has quit IRC | 21:06 | |
*** ddieterly has joined #openstack-infra | 21:07 | |
*** teran_ has joined #openstack-infra | 21:07 | |
*** steak has quit IRC | 21:08 | |
*** zz_dimtruck is now known as dimtruck | 21:09 | |
*** e0ne has quit IRC | 21:10 | |
mtreinish | clarkb: on 136234 to install the deps do I just need to do something like: https://review.openstack.org/#/c/142940/2/nodepool/elements/cache-devstack/extra-data.d/51-cache-testrepository-db | 21:10 |
mtreinish | or is there something else I need to do? | 21:10 |
*** subscope has quit IRC | 21:12 | |
clarkb | mtreinish: I think thats basically it. We install zuul onto the nodes in a venv though. Maybe we should do what zuul does? | 21:12 |
clarkb | mtreinish: except that appears broken :( | 21:13 |
clarkb | jeblair: http://paste.openstack.org/show/154229/ | 21:13 |
clarkb | jeblair: is that similar to what you ran into in the past? | 21:13 |
mtreinish | ok sure, that should be simple enough | 21:14 |
mtreinish | oh, heh | 21:14 |
clarkb | jeblair: note that doesn't appear to happen on hpcloud image builds | 21:14 |
clarkb | just rax so far | 21:14 |
mtreinish | wasn't there a new virtualenv release recently? | 21:14 |
clarkb | mtreinish: yup | 21:14 |
clarkb | I think we should use the same version on both though? | 21:14 |
clarkb | likely something to look into as the cause | 21:15 |
clarkb | I am also rebuilding just to see if it is transient | 21:15 |
*** ociuhandu has joined #openstack-infra | 21:15 | |
*** e0ne has joined #openstack-infra | 21:18 | |
*** dimtruck is now known as zz_dimtruck | 21:19 | |
openstackgerrit | Matthew Treinish proposed openstack-infra/project-config: Add nodepool script to preseed testrepository from subunit2sql https://review.openstack.org/136234 | 21:23 |
*** zz_sabari is now known as sabari | 21:23 | |
mtreinish | clarkb: ^^^ I think that should setup the venvs correctly | 21:23 |
*** thedodd has quit IRC | 21:23 | |
clarkb | I lied hpcloud is affected too | 21:23 |
clarkb | mtreinish: thanks | 21:23 |
clarkb | I need to go do a last round of grocery shopping before cooking stuff for thursday | 21:24 |
clarkb | will try to get back and debug the venv issues during image builds. In theory this should be easy to replicate | 21:24 |
clarkb | install latest venv, make venv, look for explosion | 21:24 |
*** mfer has quit IRC | 21:25 | |
mattoliverau | morning all | 21:26 |
*** baoli has joined #openstack-infra | 21:26 | |
*** zz_dimtruck is now known as dimtruck | 21:26 | |
*** eharney has quit IRC | 21:28 | |
*** ryanpetrello has quit IRC | 21:30 | |
*** ryanpetrello has joined #openstack-infra | 21:32 | |
*** eharney has joined #openstack-infra | 21:33 | |
jeblair | clarkb: i don't think i've seen this before | 21:36 |
jeblair | clarkb: virtualenv 1.11.6 (which i happened to have on a test vm) works, 12.0.2 does not | 21:37 |
jeblair | clarkb: 11.11.6 is from may 16, 12.0 is from yesterday. | 21:38 |
ekarlso- | hmmm, whne enable_service cue-api in devstack why does it install cinder instead ? :/ | 21:39 |
jeblair | dstufft: virtualenv 12.0.2 produces http://paste.openstack.org/show/154229/ on ubuntu 12.04 wheras virtualenv 1.11.6 does not | 21:40 |
dstufft | jeblair: hrm | 21:41 |
dstufft | that's strange | 21:41 |
dstufft | I think travis uses ubuntu 12.04 | 21:42 |
dstufft | though it doesn't uses ubuntu's python | 21:42 |
jeblair | oh, theres some missing output from that, let me get more verbose | 21:42 |
jeblair | dstufft, clarkb: http://paste.openstack.org/show/154240/ | 21:42 |
jeblair | IOError: invalid Python installation: unable to open /usr/zuul-env/local/include/python2.7/pyconfig.h (No such file or directory) | 21:43 |
dstufft | jeblair: I'm only half here (cooking atm), do you have python-dev installed? | 21:43 |
jeblair | yep | 21:44 |
jeblair | ii python-dev 2.7.3-0ubuntu2.2 header files and a static library for Python (default) | 21:44 |
jeblair | though python-minimal supplies pyconfig.h, but that's installed too | 21:44 |
jeblair | python2.7-minimal: /usr/include/python2.7/pyconfig.h | 21:44 |
openstackgerrit | Elizabeth K. Joseph proposed openstack-infra/infra-manual: Remove third party page from the manual https://review.openstack.org/143737 | 21:45 |
*** EmilienM|afk is now known as EmilienM | 21:48 | |
jeblair | clarkb, dstufft: it _looks_ like we have built rhat related images (centos6,centos7,f20) within the past few hours, so this _may_ be a debian-only problem. i'll try to confirm | 21:49 |
*** dims has quit IRC | 21:51 | |
anteaya | morning mattoliverau | 21:52 |
*** spzala has quit IRC | 21:53 | |
*** dimtruck is now known as zz_dimtruck | 21:55 | |
*** prad has quit IRC | 21:55 | |
*** ChanServ changes topic to "Discussion of OpenStack Developer Infrastructure | Current Python distutils issues: https://etherpad.openstack.org/p/pydistutils-issues" | 21:56 | |
*** dims has joined #openstack-infra | 21:56 | |
jeblair | clarkb, fungi, anteaya, dstufft: https://etherpad.openstack.org/p/pydistutils-issues | 21:56 |
jeblair | so we can keep up to date as we become highly async | 21:57 |
clarkb | ++ | 21:57 |
dstufft | local/include/whatever looks suspect | 21:58 |
*** e0ne has quit IRC | 21:58 | |
dstufft | Debian patches their python to move pip install stuff to /usr/local | 21:58 |
dstufft | I wonder if that is breaking things | 21:58 |
*** mbacchi has quit IRC | 21:59 | |
anteaya | jeblair: thanks | 22:00 |
dstufft | luckily | 22:01 |
*** dannywil_ has joined #openstack-infra | 22:01 | |
dstufft | virtualenv doesn't change much (mostly because I think we're all afraid to change it much) | 22:01 |
dstufft | https://github.com/pypa/virtualenv/compare/1.11.6...12.0.2 | 22:01 |
*** harlowja_ has joined #openstack-infra | 22:01 | |
dstufft | shouldn't be too hard to narrow it down | 22:01 |
dstufft | will do it once I eat | 22:01 |
*** harlowja has quit IRC | 22:02 | |
*** jamielennox|away is now known as jamielennox | 22:03 | |
*** ddieterl_ has joined #openstack-infra | 22:04 | |
*** dannywilson has quit IRC | 22:04 | |
dstufft | I wonder if I can just convince python-dev to backport venv to 2.7 too and then virtualenv can just become a shim :D | 22:04 |
*** mfink has quit IRC | 22:05 | |
*** dannywil_ has quit IRC | 22:05 | |
anteaya | here is a commit that deletes a sys.path: https://github.com/pypa/virtualenv/commit/dde220efe38b99020cd61e3c3a88c5d44922c544 | 22:07 |
*** Masahiro has joined #openstack-infra | 22:07 | |
*** ddieterly has quit IRC | 22:07 | |
anteaya | and another that addresses sys.path: https://github.com/pypa/virtualenv/commit/25cea9d5a416b17efa9098c0564a2077c99741c7 | 22:08 |
*** dannywilson has joined #openstack-infra | 22:08 | |
anteaya | and a third: https://github.com/pypa/virtualenv/commit/920d7c71d800ba4c0da8fcac605cdf0adfe46f3a | 22:09 |
*** dannywilson has quit IRC | 22:09 | |
* dstufft spins up a precise image | 22:09 | |
*** dannywilson has joined #openstack-infra | 22:09 | |
*** patrickeast_ has joined #openstack-infra | 22:10 | |
clarkb | trusty is where it failed on us. though in theory precise should fail similarly | 22:11 |
jeblair | anteaya, dstufft: reverting https://github.com/pypa/virtualenv/commit/dde220efe38b99020cd61e3c3a88c5d44922c544 appears to fix it for me | 22:11 |
jeblair | clarkb: yeah, i'm testing on precise | 22:11 |
jeblair | (it's what i had laying around) | 22:11 |
*** patrickeast has quit IRC | 22:11 | |
*** patrickeast_ is now known as patrickeast | 22:11 | |
dstufft | I hate python | 22:11 |
*** dims has quit IRC | 22:11 | |
dstufft | that commit was there to fix a bleed through of the stdlib | 22:11 |
*** Masahiro has quit IRC | 22:11 | |
dstufft | because virtualenv's -p thing does something reaaaaly hacky | 22:12 |
dstufft | https://github.com/pypa/virtualenv/issues/673 | 22:13 |
*** teran has joined #openstack-infra | 22:13 | |
anteaya | my shot in the dark was a lucky one | 22:13 |
dstufft | I bet it needs to be smarter about what it deletes | 22:14 |
jeblair | oh, so we should test any fixes on trusty and make sure it can still use an alternate interpreter | 22:15 |
* anteaya steps away to chop some vegetables | 22:15 | |
pleia2 | anteaya: happy chopping | 22:15 |
*** teran_ has quit IRC | 22:15 | |
dstufft | maybe a better answer is to be even more hacky and just copy virtualenv.py into a temporary directory before we execute it | 22:16 |
*** mfer has joined #openstack-infra | 22:16 | |
*** esmute has quit IRC | 22:17 | |
*** zaro has quit IRC | 22:18 | |
*** gothicmindfood has quit IRC | 22:18 | |
*** jesusaurus has quit IRC | 22:18 | |
*** ryanpetrello has quit IRC | 22:22 | |
*** signed8bit has joined #openstack-infra | 22:24 | |
dstufft | jeblair: https://bpaste.net/show/b2f9c2d8e1cd does that fix it? | 22:27 |
jeblair | dstufft: i'll start testing it | 22:28 |
dstufft | https://github.com/pypa/virtualenv/pull/687 pull request for completeness sake | 22:29 |
*** harlowja_ has quit IRC | 22:31 | |
*** harlowja has joined #openstack-infra | 22:31 | |
krotscheck | jeblair: I’m currently drafting the promotion email for yolanda - Given how many western countries are currently on vacation, I’d like to keep the review period open until the 9th rather than just a week. Thoughts? | 22:32 |
jeblair | krotscheck: seems reasonable | 22:33 |
*** mfer has quit IRC | 22:34 | |
jeblair | dstufft: btw i found that current HEAD (not including your change) works if you do "-p python3" and also if you omit "-p python2". it seems to only be the case where you are specifying the default interpreter that it fails | 22:36 |
jeblair | (on precise) | 22:36 |
*** ihrachyshka has quit IRC | 22:36 | |
*** otter768 has joined #openstack-infra | 22:38 | |
dstufft | jeblair: yea, it's a an error that comes from like, 12 different layers of hacks in virtualenv | 22:42 |
*** hdd has quit IRC | 22:43 | |
jeblair | dstufft: the 6 tests described at the bottom of https://etherpad.openstack.org/p/pydistutils-issues all work with your patch | 22:43 |
*** otter768 has quit IRC | 22:44 | |
jeblair | dstufft: i'm running those again, but with virtualenv running under python3 | 22:46 |
*** esmute has joined #openstack-infra | 22:46 | |
*** gothicmindfood has joined #openstack-infra | 22:47 | |
*** Longgeek has quit IRC | 22:49 | |
*** jesusaurus has joined #openstack-infra | 22:49 | |
*** zaro has joined #openstack-infra | 22:55 | |
jeblair | dstufft: okay, so all of the python2/3 cross testing i can think to do on precise and trusty works with your patch (see etherpad for full list) | 22:57 |
*** teran_ has joined #openstack-infra | 22:58 | |
*** FlorianSW has quit IRC | 22:58 | |
*** unicell has quit IRC | 22:59 | |
*** zz_dimtruck is now known as dimtruck | 23:00 | |
*** dannywilson has quit IRC | 23:01 | |
*** otter768 has joined #openstack-infra | 23:01 | |
*** teran has quit IRC | 23:01 | |
*** dannywilson has joined #openstack-infra | 23:04 | |
*** esker has joined #openstack-infra | 23:05 | |
*** esker has quit IRC | 23:05 | |
*** esker has joined #openstack-infra | 23:06 | |
*** marun has quit IRC | 23:08 | |
*** tonytan4ever has quit IRC | 23:08 | |
openstackgerrit | Michael Krotscheck proposed openstack-infra/storyboard: [WIP] Normalized event data https://review.openstack.org/143744 | 23:08 |
*** salv-orlando has quit IRC | 23:10 | |
*** vigneshvar has quit IRC | 23:10 | |
*** dims has joined #openstack-infra | 23:12 | |
*** hichtakk has quit IRC | 23:12 | |
*** harlowja has quit IRC | 23:12 | |
*** hichtakk has joined #openstack-infra | 23:12 | |
*** radez is now known as radez_g0n3 | 23:13 | |
*** dims has quit IRC | 23:16 | |
*** dmsimard is now known as dmsimard_away | 23:20 | |
*** harlowja has joined #openstack-infra | 23:20 | |
*** yamahata has quit IRC | 23:22 | |
*** jamielennox is now known as jamielennox|away | 23:24 | |
clarkb | jeblair so I guess we wait for new virtualenv before worrying about those images? | 23:28 |
dstufft | I'll be releasing a new virtualenv in a bit | 23:31 |
*** esker has quit IRC | 23:37 | |
*** amitgandhinz has quit IRC | 23:37 | |
*** esker has joined #openstack-infra | 23:38 | |
clarkb | dstufft so the original fix detailed in 673 was insufficient? | 23:38 |
*** dkliban is now known as dkliban_afk | 23:38 | |
*** naohirot has joined #openstack-infra | 23:38 | |
dstufft | clarkb: honestly I understand like 50% of what virtualenv is doing | 23:39 |
clarkb | ok I understand less so if its not worth grokking thats fine by me :) | 23:39 |
dstufft | I just noticed that it was failing trying to import ``site`` when we subshell out to python /path/to/site-packages/virtualenv.py | 23:39 |
dstufft | so I randomly tried -S and it fixed it | 23:39 |
dstufft | why importing site failing? I have no idea | 23:40 |
*** [HeOS] has joined #openstack-infra | 23:42 | |
*** HeOS has quit IRC | 23:43 | |
naohirot | good morning | 23:43 |
naohirot | this is my first time to join this channel | 23:45 |
*** ddieterly has joined #openstack-infra | 23:45 | |
naohirot | gate-tempest-dsvm-neutron-src-python-heatclient becomes error at the gate | 23:46 |
*** cnesa has quit IRC | 23:46 | |
*** miqui_ has quit IRC | 23:46 | |
*** dimtruck is now known as zz_dimtruck | 23:46 | |
clarkb | naohirot can you link to logs for failing runs? | 23:46 |
naohirot | Is this channel appropriate place to discuss such that gate problem? | 23:46 |
naohirot | clarkb: Hi | 23:46 |
*** ddieterl_ has quit IRC | 23:46 | |
clarkb | possibly. having logs would definitely help | 23:47 |
*** hichtakk has quit IRC | 23:47 | |
naohirot | clarkb: this is OK log http://logs.openstack.org/59/143059/1/check/gate-tempest-dsvm-neutron-src-python-heatclient/ff49780/ | 23:47 |
naohirot | clarkb: this is NG log http://logs.openstack.org/59/143059/4/check/gate-tempest-dsvm-neutron-src-python-heatclient/5817272/ | 23:47 |
naohirot | clarkb: the program is same | 23:48 |
naohirot | clarkb: the gerrit review is here https://review.openstack.org/#/c/143059/ | 23:50 |
clarkb | those look like timeouts when testing heat. is it possible that the change is at fault? | 23:50 |
clarkb | I think those may be valid failures | 23:50 |
naohirot | clarkb: sorry what do you mean by "at fault"? | 23:52 |
naohirot | clarkb: No, definitely No | 23:52 |
clarkb | naohirot: I mean that test fails in code related to heatclient | 23:53 |
clarkb | naohirot: so it seems likely that the change to heatclient is causing those tests to fail but let me see the change | 23:53 |
naohirot | clarkb: because the patch set 1 was OK, the patch set 4 was NG, and 1 and 4 are exactly the same code | 23:54 |
clarkb | http://logs.openstack.org/59/143059/4/check/gate-tempest-dsvm-neutron-src-python-heatclient/5817272/logs/screen-h-eng.txt.gz appears to be the error | 23:54 |
clarkb | so it is indirectly related to heatclient. Now why are we using a version of pbr that doesn't work fo rthis | 23:55 |
*** achanda has joined #openstack-infra | 23:55 | |
*** sabari is now known as zz_sabari | 23:56 | |
*** Masahiro has joined #openstack-infra | 23:56 | |
naohirot | clarkb: Yes the url you just put here is the log of the patch set 4. | 23:56 |
clarkb | looks like we install pbr from ubuntu | 23:56 |
clarkb | sdague: ^ if you are not on vacation yet can we please please please not install packages like that from the distros? | 23:57 |
naohirot | clarkb: can you compare the code between patch set 1 and 4 at https://review.openstack.org/#/c/143059/, you will set the same. | 23:57 |
*** dannywilson has quit IRC | 23:57 | |
*** ociuhandu has quit IRC | 23:58 | |
naohirot | clarkb: regarding "pbr", | 23:58 |
clarkb | naohirot: yes its not the codes fault it appears to be devstack doing something funny | 23:58 |
naohirot | clarkb: I didn't do anything. | 23:58 |
clarkb | naohirot: I know | 23:59 |
naohirot | clarkb: Okay, so should I submit a bug report ? | 23:59 |
*** achanda has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!