| *** sarob has quit IRC | 00:03 | |
| *** sdake_ has joined #openstack-infra | 00:03 | |
| *** Guest98422 has quit IRC | 00:06 | |
| *** sdake has quit IRC | 00:07 | |
| *** yamamoto has joined #openstack-infra | 00:08 | |
| *** samueldmq has quit IRC | 00:10 | |
| *** bhunter71 has joined #openstack-infra | 00:11 | |
| *** sarob has joined #openstack-infra | 00:12 | |
| *** ddieterly has joined #openstack-infra | 00:13 | |
| *** achanda has joined #openstack-infra | 00:15 | |
| *** emagana has quit IRC | 00:16 | |
| *** julim has joined #openstack-infra | 00:17 | |
| *** hodos|3 has quit IRC | 00:17 | |
| *** achanda has quit IRC | 00:20 | |
| *** zul has joined #openstack-infra | 00:24 | |
| *** ociuhandu has quit IRC | 00:28 | |
| *** dboik_ has joined #openstack-infra | 00:32 | |
| *** signed8bit_ZZZzz is now known as signed8bit | 00:35 | |
| *** VijayTripathi has quit IRC | 00:36 | |
| *** koolhead17 has joined #openstack-infra | 00:37 | |
| *** yamamoto has quit IRC | 00:42 | |
| *** yamamoto has joined #openstack-infra | 00:43 | |
| *** yamamoto has quit IRC | 00:43 | |
| *** jogo has quit IRC | 00:47 | |
| *** markvoelker has joined #openstack-infra | 00:50 | |
| *** spzala has joined #openstack-infra | 00:51 | |
| *** stevemar has quit IRC | 00:53 | |
| *** markvoelker has quit IRC | 00:55 | |
| dougwig | clarkb: thank you. | 00:55 |
|---|---|---|
| dougwig | commented back. if it's a poor fit, no worries on abandoning it. but there is a maintenance win that's missing today. | 00:56 |
| *** sarob has quit IRC | 00:57 | |
| *** tiswanso has joined #openstack-infra | 00:57 | |
| *** SumitNaiksatam has joined #openstack-infra | 00:58 | |
| clarkb | dougwig: https://git.openstack.org/cgit/openstack-infra/devstack-gate/tree/devstack-vm-gate.sh#n89 you can update the defaults... | 00:58 |
| *** VijayTripathi has joined #openstack-infra | 00:59 | |
| clarkb | dougwig: you are basically reinventing ^ and some devstack magic | 00:59 |
| clarkb | I think you can just use ENABLED_SERVICES as is and it should work | 00:59 |
| dougwig | clarkb: two use cases: 1) remove swift, as it's a ram hog, or 2) swap q-lbaas for q-lbaasv2. the current mechanism is only adds, right? and i can live with that, but it makes some things awkward. | 00:59 |
| dougwig | clarkb: can i put subtractions in there? | 01:00 |
| clarkb | I think so | 01:00 |
| clarkb | devstack should process the subtractions | 01:00 |
| clarkb | I may be wrong though, but I thought that devstack understoof -service and last service entry wins | 01:01 |
| dougwig | clarkb: ooh, sweet. let me check on that. i definitely don't want that in there twice. | 01:01 |
| *** hdd has joined #openstack-infra | 01:02 | |
| fungi | i thought i remembered that too, so went digging in project-config | 01:03 |
| fungi | but didn't turn up any good examples | 01:03 |
| *** annegentle has joined #openstack-infra | 01:06 | |
| *** sputnik13 has quit IRC | 01:07 | |
| *** hichihara has quit IRC | 01:08 | |
| dougwig | clarkb: you're right, devstack has a '-', so your suggestion will work. thanks. | 01:09 |
| *** annegentle_ has joined #openstack-infra | 01:09 | |
| *** krtaylor has quit IRC | 01:10 | |
| dougwig | let me test it end-to-end, then i'll abandon. | 01:10 |
| *** yamamoto has joined #openstack-infra | 01:15 | |
| openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/shade: Fix caching bug in create_volume/list_volume https://review.openstack.org/171015 | 01:18 |
| SpamapS | Shrews: ^ | 01:18 |
| SpamapS | Shrews: I think there may be more places we're ignoring the cache flag. | 01:18 |
| *** tiswanso has quit IRC | 01:19 | |
| *** annegentle_ has quit IRC | 01:20 | |
| *** tiswanso has joined #openstack-infra | 01:20 | |
| *** koolhead_ has joined #openstack-infra | 01:21 | |
| *** annegentle has quit IRC | 01:22 | |
| *** koolhead17 has quit IRC | 01:23 | |
| *** annegentle has joined #openstack-infra | 01:24 | |
| *** asettle is now known as asettle-afk | 01:25 | |
| *** annegentle_ has joined #openstack-infra | 01:25 | |
| fungi | bandersnatch is still syncing. at least it finally got past django.* | 01:25 |
| openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/shade: Fix caching bug in create_volume/list_volume https://review.openstack.org/171015 | 01:25 |
| *** mtanino has quit IRC | 01:26 | |
| dstufft | fungi: did you get bandersnatch fixed? what was the problem? | 01:26 |
| Shrews | SpamapS: i'm in basketball-mode now, but wondering why if the backend is the null cache driver, why is *any* caching happening? | 01:28 |
| *** annegentle has quit IRC | 01:29 | |
| *** annegentle_ has quit IRC | 01:29 | |
| Shrews | maybe i misunderstand what the null backend does | 01:29 |
| *** annegentle has joined #openstack-infra | 01:29 | |
| *** annegent_ has joined #openstack-infra | 01:30 | |
| fungi | dstufft: we have bandersnatch running from cron under a timeout so it won't lock indefinitely in case it hangs | 01:30 |
| *** annegentle is now known as Guest56002 | 01:30 | |
| *** Longgeek has joined #openstack-infra | 01:30 | |
| fungi | dstufft: we merged a change that upgraded us from bandersnatch 1.7 to 1.8 but it bumps the generation in 1.8 (because some people might have deleted setuptools packages a while back) | 01:30 |
| fungi | dstufft: so the next bandersnatch run after the upgrade wanted to do a full refresh, but the timeout was killing it partway in. lather, rinse, repeat | 01:31 |
| dstufft | fungi: ahhh | 01:32 |
| *** jogo has joined #openstack-infra | 01:34 | |
| *** Guest56002 has quit IRC | 01:38 | |
| *** annegent_ has quit IRC | 01:39 | |
| *** hichihara has joined #openstack-infra | 01:39 | |
| *** sdake_ has quit IRC | 01:40 | |
| *** VijayTripathi has quit IRC | 01:41 | |
| *** sarob has joined #openstack-infra | 01:42 | |
| fungi | anyway, looks like it's got a couple more hours to go, so i'll double-check all our mirrors again when i get up in the morning | 01:42 |
| *** otter768 has joined #openstack-infra | 01:43 | |
| *** xyang1 has quit IRC | 01:45 | |
| *** jogo has quit IRC | 01:45 | |
| *** krtaylor has joined #openstack-infra | 01:46 | |
| *** sarob has quit IRC | 01:46 | |
| *** jogo has joined #openstack-infra | 01:48 | |
| *** otter768 has quit IRC | 01:48 | |
| *** baoli has joined #openstack-infra | 01:48 | |
| *** markvoelker has joined #openstack-infra | 01:51 | |
| *** carl_baldwin has joined #openstack-infra | 01:52 | |
| *** jogo has quit IRC | 01:53 | |
| *** baoli has quit IRC | 01:54 | |
| *** jogo has joined #openstack-infra | 01:55 | |
| *** markvoelker has quit IRC | 01:56 | |
| *** jogo has quit IRC | 01:57 | |
| *** adalbas has quit IRC | 01:57 | |
| *** melwitt has quit IRC | 01:58 | |
| *** jogo has joined #openstack-infra | 02:00 | |
| *** taron has quit IRC | 02:04 | |
| *** taron has joined #openstack-infra | 02:05 | |
| *** VijayTripathi has joined #openstack-infra | 02:07 | |
| *** taron has quit IRC | 02:07 | |
| *** taron has joined #openstack-infra | 02:07 | |
| *** signed8bit has quit IRC | 02:07 | |
| *** pal has joined #openstack-infra | 02:09 | |
| *** yamahata has quit IRC | 02:13 | |
| *** dhritishikhar_ has joined #openstack-infra | 02:14 | |
| *** Longgeek has quit IRC | 02:14 | |
| *** jogo has quit IRC | 02:17 | |
| SpamapS | Shrews: I am also confused by that, but thats phase 2 ;) | 02:18 |
| *** baoli has joined #openstack-infra | 02:19 | |
| *** taron has quit IRC | 02:19 | |
| *** taron has joined #openstack-infra | 02:20 | |
| *** jogo has joined #openstack-infra | 02:20 | |
| *** baoli has quit IRC | 02:25 | |
| *** carl_baldwin has quit IRC | 02:26 | |
| *** hdd has quit IRC | 02:26 | |
| *** dhritishikhar_ has quit IRC | 02:27 | |
| *** stevemar has joined #openstack-infra | 02:29 | |
| *** dhritishikhar_ has joined #openstack-infra | 02:30 | |
| *** achanda has joined #openstack-infra | 02:37 | |
| *** chlong has quit IRC | 02:42 | |
| *** baoli has joined #openstack-infra | 02:43 | |
| *** chlong has joined #openstack-infra | 02:44 | |
| *** marun has quit IRC | 02:46 | |
| *** achanda has quit IRC | 02:47 | |
| *** pal has quit IRC | 02:48 | |
| *** achanda has joined #openstack-infra | 02:48 | |
| *** dboik_ has quit IRC | 02:49 | |
| *** Longgeek has joined #openstack-infra | 02:51 | |
| *** markvoelker has joined #openstack-infra | 02:52 | |
| *** spzala has quit IRC | 02:54 | |
| *** wenlock has quit IRC | 02:54 | |
| *** dannywilson has joined #openstack-infra | 02:56 | |
| *** markvoelker has quit IRC | 02:56 | |
| *** baoli has quit IRC | 02:58 | |
| *** dims has quit IRC | 02:58 | |
| *** baoli has joined #openstack-infra | 02:58 | |
| *** dannywilson has quit IRC | 03:00 | |
| *** sdake has joined #openstack-infra | 03:02 | |
| *** patrickeast has quit IRC | 03:05 | |
| *** sdake_ has joined #openstack-infra | 03:06 | |
| *** dims has joined #openstack-infra | 03:07 | |
| *** dims_ has joined #openstack-infra | 03:08 | |
| *** ajmiller_ has joined #openstack-infra | 03:08 | |
| *** sdake has quit IRC | 03:09 | |
| *** Longgeek_ has joined #openstack-infra | 03:10 | |
| *** Longgeek has quit IRC | 03:12 | |
| *** asettle-afk is now known as asettle | 03:12 | |
| *** dims has quit IRC | 03:12 | |
| *** ajmiller__ has quit IRC | 03:12 | |
| openstackgerrit | Huang Rui proposed openstack-infra/project-config: Create neutron-zvm-plugin project on StackForge https://review.openstack.org/171030 | 03:17 |
| *** dims_ has quit IRC | 03:20 | |
| *** sdake has joined #openstack-infra | 03:21 | |
| *** baoli has quit IRC | 03:22 | |
| *** sdake_ has quit IRC | 03:24 | |
| *** ddieterly has quit IRC | 03:29 | |
| *** zigzg_ has joined #openstack-infra | 03:30 | |
| *** Longgeek_ has quit IRC | 03:32 | |
| *** tiswanso has quit IRC | 03:35 | |
| *** dkliban is now known as dkliban_afk | 03:41 | |
| *** otter768 has joined #openstack-infra | 03:44 | |
| *** coolsvap|afk is now known as coolsvap | 03:45 | |
| *** jamielennox is now known as jamielennox|away | 03:47 | |
| *** otter768 has quit IRC | 03:49 | |
| *** baoli has joined #openstack-infra | 03:49 | |
| *** gyee has quit IRC | 03:50 | |
| *** dhritishikhar_ has quit IRC | 03:50 | |
| *** Alexandra_ has joined #openstack-infra | 03:50 | |
| *** asettle has quit IRC | 03:50 | |
| *** Alexandra_ is now known as asettle | 03:51 | |
| *** pal has joined #openstack-infra | 03:51 | |
| *** markvoelker has joined #openstack-infra | 03:53 | |
| *** harlowja is now known as harlowja_away | 03:55 | |
| *** baoli has quit IRC | 03:55 | |
| *** markvoelker has quit IRC | 03:57 | |
| *** emagana has joined #openstack-infra | 03:57 | |
| *** koolhead_ has quit IRC | 03:59 | |
| *** koolhead17 has joined #openstack-infra | 04:00 | |
| *** Longgeek has joined #openstack-infra | 04:00 | |
| *** sputnik13 has joined #openstack-infra | 04:00 | |
| *** sdake_ has joined #openstack-infra | 04:01 | |
| *** tqtran has quit IRC | 04:03 | |
| *** koolhead17 has quit IRC | 04:04 | |
| *** sdake has quit IRC | 04:05 | |
| *** VijayTripathi1 has joined #openstack-infra | 04:07 | |
| *** VijayTripathi has quit IRC | 04:09 | |
| *** dhritishikhar_ has joined #openstack-infra | 04:11 | |
| *** achanda has quit IRC | 04:13 | |
| *** btully has joined #openstack-infra | 04:15 | |
| *** dhritishikhar_ has quit IRC | 04:15 | |
| *** achanda has joined #openstack-infra | 04:20 | |
| *** amotoki has joined #openstack-infra | 04:20 | |
| *** bhunter71 has quit IRC | 04:21 | |
| *** ayoung has quit IRC | 04:25 | |
| *** btully has quit IRC | 04:29 | |
| *** ddieterly has joined #openstack-infra | 04:30 | |
| *** prmtl has joined #openstack-infra | 04:34 | |
| *** ddieterly has quit IRC | 04:34 | |
| *** ajo has quit IRC | 04:38 | |
| *** coolsvap is now known as coolsvap|afk | 04:41 | |
| mordred | clarkb: btw ... | 04:43 |
| mordred | clarkb: GO DUKE | 04:43 |
| mordred | that's all | 04:43 |
| *** persia_ has quit IRC | 04:44 | |
| *** dhritishikhar_ has joined #openstack-infra | 04:45 | |
| *** persia_ has joined #openstack-infra | 04:45 | |
| *** unicell has joined #openstack-infra | 04:47 | |
| *** ajo has joined #openstack-infra | 04:47 | |
| *** coolsvap|afk is now known as coolsvap | 04:48 | |
| *** markvoelker has joined #openstack-infra | 04:53 | |
| *** markvoelker has quit IRC | 04:58 | |
| *** Krinkle is now known as Krinkle|detached | 04:58 | |
| clarkb | I figured you would be happy | 04:58 |
| *** chandankumar has joined #openstack-infra | 05:00 | |
| *** dhritishikhar_ has quit IRC | 05:00 | |
| mordred | clarkb: there may have been an amount of screaming, and also an amount of writhing on the floor. it was a really good game | 05:03 |
| *** prmtl has quit IRC | 05:06 | |
| clarkb | unfortunately I had to miss it | 05:06 |
| clarkb | but saw duke was down quite a but in the second half | 05:06 |
| *** jcoufal has joined #openstack-infra | 05:07 | |
| openstackgerrit | Deanna Hambleton proposed openstack-infra/jenkins-job-builder: Add support for Scoverage plugin https://review.openstack.org/171048 | 05:12 |
| *** jcoufal has quit IRC | 05:13 | |
| *** jcoufal has joined #openstack-infra | 05:16 | |
| *** mrmartin has joined #openstack-infra | 05:20 | |
| *** dhritishikhar_ has joined #openstack-infra | 05:28 | |
| *** sarob has joined #openstack-infra | 05:32 | |
| *** ddieterly has joined #openstack-infra | 05:36 | |
| *** ddieterly has quit IRC | 05:41 | |
| *** VijayTripathi1 has quit IRC | 05:41 | |
| *** sdake has joined #openstack-infra | 05:43 | |
| *** deepakcs has joined #openstack-infra | 05:45 | |
| *** otter768 has joined #openstack-infra | 05:45 | |
| *** Hal has joined #openstack-infra | 05:45 | |
| *** Hal is now known as Guest73676 | 05:46 | |
| *** sdake_ has quit IRC | 05:46 | |
| *** sarob has quit IRC | 05:47 | |
| *** prmtl has joined #openstack-infra | 05:50 | |
| *** dhritishikhar_ has quit IRC | 05:50 | |
| *** otter768 has quit IRC | 05:50 | |
| *** prmtl has quit IRC | 05:51 | |
| *** prmtl has joined #openstack-infra | 05:51 | |
| *** markvoelker has joined #openstack-infra | 05:54 | |
| *** markvoelker has quit IRC | 05:58 | |
| openstackgerrit | Joshua Hesketh proposed openstack-infra/zuul: Add in extra connections tests https://review.openstack.org/169300 | 06:01 |
| openstackgerrit | Joshua Hesketh proposed openstack-infra/zuul: Refactor sources out of triggers https://review.openstack.org/118993 | 06:01 |
| openstackgerrit | Joshua Hesketh proposed openstack-infra/zuul: Document the new connections in zuul https://review.openstack.org/169270 | 06:01 |
| openstackgerrit | Joshua Hesketh proposed openstack-infra/zuul: Add gerrit reviews into patchset approvals https://review.openstack.org/97390 | 06:01 |
| openstackgerrit | Joshua Hesketh proposed openstack-infra/zuul: Add support for negative requirements https://review.openstack.org/102726 | 06:01 |
| openstackgerrit | Joshua Hesketh proposed openstack-infra/zuul: Configure triggers dynamically https://review.openstack.org/119534 | 06:01 |
| openstackgerrit | Joshua Hesketh proposed openstack-infra/zuul: Add support for 'connection' concept https://review.openstack.org/121528 | 06:01 |
| openstackgerrit | Joshua Hesketh proposed openstack-infra/zuul: Add base class for triggers https://review.openstack.org/119532 | 06:01 |
| openstackgerrit | Joshua Hesketh proposed openstack-infra/zuul: Add base class for sources https://review.openstack.org/119531 | 06:01 |
| openstackgerrit | Joshua Hesketh proposed openstack-infra/zuul: Add base class for reporters https://review.openstack.org/119530 | 06:01 |
| *** dhritishikhar_ has joined #openstack-infra | 06:01 | |
| stevemar | everything seems to be broken :) | 06:03 |
| stevemar | looks like all projects are affected... | 06:04 |
| stevemar | AttributeError: 'InstallRequirement' object has no attribute 'url' | 06:04 |
| stevemar | when performing: python update.py /opt/stack/new/keystone | 06:04 |
| *** armax has quit IRC | 06:05 | |
| *** ZZelle has quit IRC | 06:07 | |
| stevemar | submitted this bug: https://bugs.launchpad.net/tempest/+bug/1440984 | 06:09 |
| openstack | Launchpad bug 1440984 in tempest "AttributeError: 'InstallRequirement' when running update.py" [Undecided,New] | 06:09 |
| *** Bsony has joined #openstack-infra | 06:14 | |
| openstackgerrit | Steve Martinelli proposed openstack-infra/elastic-recheck: Add elastic recheck for bug 1440984 https://review.openstack.org/171056 | 06:15 |
| openstack | bug 1440984 in tempest "AttributeError: 'InstallRequirement' when running update.py" [Undecided,New] https://launchpad.net/bugs/1440984 | 06:16 |
| rm_work | anything to do with pip 6.1.0 releasing? | 06:19 |
| *** ibiris_away is now known as ibiris | 06:19 | |
| rm_work | InstallRequirement comes from pip.req ? | 06:19 |
| stevemar | rm_work, i'm not sure, just reporting the problem | 06:20 |
| stevemar | the build stuff goes over my head | 06:20 |
| rm_work | we're running into this locally even | 06:20 |
| rm_work | on local devstacks | 06:20 |
| stevemar | but yeah, it looks like something is murky with requirements | 06:21 |
| rm_work | I have to assume it's related to the 6.1.0 pip update | 06:21 |
| stevemar | rm_work, considering it was released a few hours ago, i'd say you are bang on :) | 06:21 |
| *** Bsony has quit IRC | 06:22 | |
| juggler_ | hi all...is this a good place to inquire about devstack questions? | 06:23 |
| *** sputnik13 has quit IRC | 06:24 | |
| stevemar | juggler_, #openstack-dev would be better, but most channels are pretty quiet at this time | 06:24 |
| *** juggler_ is now known as juggler | 06:25 | |
| *** yamahata has joined #openstack-infra | 06:25 | |
| *** mrunge has joined #openstack-infra | 06:25 | |
| juggler | ah, thank you for that heads up stevemar | 06:26 |
| rm_work | yep | 06:26 |
| rm_work | https://github.com/pypa/pip/commit/e8e2566279879b7df04394edfcaa9c63c0ce9e67 | 06:27 |
| rm_work | ^^ that | 06:27 |
| rm_work | fff | 06:27 |
| stevemar | rm_work, that would do it | 06:28 |
| stevemar | rm_work, https://github.com/openstack/requirements/blob/643af1a899502a5eccc19b44b3f1f1815192fa97/update.py#L101 | 06:29 |
| rm_work | heh | 06:29 |
| rm_work | yeah | 06:29 |
| rm_work | that needs to be link | 06:30 |
| rm_work | or, some sort of hasattr check, or a try | 06:30 |
| rm_work | or SOMETHING | 06:30 |
| openstackgerrit | Steve Martinelli proposed openstack/requirements: Change install_requires to use link instead of url https://review.openstack.org/171060 | 06:31 |
| stevemar | rm_work, ^ | 06:31 |
| stevemar | yeah, hasattr check would be better | 06:32 |
| stevemar | well, lets see if this even works | 06:32 |
| stevemar | if it lasts more than 10-15 minutes in zuul we know it's working :) | 06:33 |
| stevemar | good detective work btw! on finding out what changed | 06:33 |
| *** andrey-mp has joined #openstack-infra | 06:34 | |
| *** Longgeek has quit IRC | 06:34 | |
| *** Longgeek has joined #openstack-infra | 06:34 | |
| rm_work | :) | 06:36 |
| *** ddieterly has joined #openstack-infra | 06:37 | |
| stevemar | rm_work, gah, doesn't seem to be working completely | 06:39 |
| rm_work | 6.1.0 is only used on 2.7.8 locally | 06:40 |
| rm_work | other python versions use different pip in my local testing, not sure if thats a pyenv thing or somehow global | 06:40 |
| rm_work | might need to allow for both | 06:40 |
| *** dhritishikhar__ has joined #openstack-infra | 06:41 | |
| *** asettle has quit IRC | 06:41 | |
| *** ddieterly has quit IRC | 06:42 | |
| *** dhritishikhar_ has quit IRC | 06:43 | |
| *** HeOS has quit IRC | 06:43 | |
| rm_work | seriously have no idea why some of my environments want to use the 1.x branch of pip and some the 6.x | 06:44 |
| stevemar | its a weird one. dstufft will figure this all out in 5 minutes and make us look like noobs :) | 06:46 |
| dstufft | whats up | 06:47 |
| *** jistr has joined #openstack-infra | 06:47 | |
| stevemar | oh hai | 06:47 |
| openstackgerrit | Deanna Hambleton proposed openstack-infra/jenkins-job-builder: Add support for Scoverage plugin https://review.openstack.org/171061 | 06:47 |
| stevemar | dstufft, the latest pip release seems to have broken everything | 06:47 |
| stevemar | :) | 06:47 |
| dstufft | oh goody | 06:47 |
| rm_work | dstufft: https://github.com/pypa/pip/commit/e8e2566279879b7df04394edfcaa9c63c0ce9e67 <-- we rely on .url | 06:47 |
| dstufft | breaking everything is my favorite pastetime | 06:47 |
| stevemar | code: https://review.openstack.org/171060 bug: https://bugs.launchpad.net/tempest/+bug/1440984 | 06:47 |
| openstack | Launchpad bug 1440984 in tempest "AttributeError: 'InstallRequirement' when running update.py" [Undecided,New] | 06:47 |
| stevemar | dstufft, having a hobby is always good | 06:48 |
| *** sarob has joined #openstack-infra | 06:49 | |
| dstufft | so pip doesn't support being used an API, so techincally folks get what they deserve for using it that way. It seems you have a fix already, but if that can't be merged quickly I can probably add a shim to InstallRequirement to unbreak things | 06:50 |
| *** woodster_ has quit IRC | 06:50 | |
| *** ildikov has joined #openstack-infra | 06:51 | |
| *** ajo has quit IRC | 06:51 | |
| rm_work | heh | 06:52 |
| stevemar | dstufft, i'm not sure what the core requirements folks will want going forward (using whatever is upstream makes sense to me). | 06:53 |
| stevemar | i guess priority one is to unbreak things | 06:53 |
| stevemar | but my patch doesn't seem to fix things :( | 06:53 |
| dstufft | you probably need link.url | 06:53 |
| dstufft | not just link | 06:53 |
| *** sarob has quit IRC | 06:54 | |
| *** afazekas has joined #openstack-infra | 06:54 | |
| *** markvoelker has joined #openstack-infra | 06:55 | |
| *** liusheng has joined #openstack-infra | 06:56 | |
| dstufft | stevemar: does that make sense? | 06:58 |
| *** aswadr has joined #openstack-infra | 06:58 | |
| lifeless | dstufft: perhaps we should make pip's guts raise if the process is neither py.test nor pip. | 06:59 |
| stevemar | dstufft, ohh didn't see that from the change in https://github.com/pypa/pip/commit/e8e2566279879b7df04394edfcaa9c63c0ce9e67 | 06:59 |
| *** markvoelker has quit IRC | 06:59 | |
| dstufft | stevemar: I think url was a string, and link is an object isntance of the Link class | 07:00 |
| *** mpavone has joined #openstack-infra | 07:01 | |
| openstackgerrit | Steve Martinelli proposed openstack/requirements: Change install_requires to use link instead of url https://review.openstack.org/171060 | 07:01 |
| dstufft | lifeless: meh, people reach into pip, it doesn't bother me a whole lot, as long they realize it's all internal so can break at any time. We have a ticket about making an actual API but step one is someone who wants to use pip as an API to come forward and iterate what they actually want from an API so we can expose something for them | 07:01 |
| stevemar | dstufft, yep, i'll admit i didn't take a thorough look, i saw the first few bits that changed url to link... and wanted to submit something to get us unbroken | 07:01 |
| stevemar | didn't realize that there was a Link class | 07:02 |
| *** ibiris is now known as ibiris_away | 07:02 | |
| *** achanda has quit IRC | 07:04 | |
| *** dtantsur|afk is now known as dtantsur | 07:04 | |
| *** spredzy|afk is now known as spredzy | 07:05 | |
| *** ajo has joined #openstack-infra | 07:07 | |
| *** Bsony has joined #openstack-infra | 07:07 | |
| *** Guest73676 has quit IRC | 07:08 | |
| *** achanda has joined #openstack-infra | 07:08 | |
| rm_work | yeah, though all the code is doing is seeing if it exists, so it shouldn't matter | 07:09 |
| rm_work | stevemar / dstufft ^^ | 07:09 |
| rm_work | a Link object will evaluate as True just as well as a string | 07:09 |
| *** sergsh has joined #openstack-infra | 07:10 | |
| rm_work | I don't think your patch2 is the solution either | 07:10 |
| rm_work | you probably want to do: | 07:10 |
| stevemar | rm_work, i suppose i should actually run it | 07:11 |
| stevemar | rather than wait for jenkins to crap out | 07:11 |
| rm_work | elif hasattr(install_require.link) and install_require.link: return pip | 07:11 |
| rm_work | elif hasattr(install_require.url) and install_require.url: return pip | 07:11 |
| rm_work | IE, have both | 07:11 |
| rm_work | and check they exist first before you check them for truth | 07:11 |
| stevemar | should the first be: | 07:11 |
| stevemar | elif hasattr(install_require.link.url) and install_require.link.url: return pip | 07:12 |
| rm_work | err | 07:12 |
| rm_work | yeah sorry | 07:12 |
| rm_work | err | 07:12 |
| rm_work | no | 07:12 |
| rm_work | but | 07:12 |
| rm_work | i did typo it | 07:12 |
| rm_work | elif hasattr(install_require, "url") and install_require.url: return pip | 07:12 |
| rm_work | elif hasattr(install_require, "link") and install_require.link: return pip | 07:12 |
| rm_work | we don't care about the ACTUAL url here, just that it is not None | 07:12 |
| rm_work | so if a Link object exists, I think it's fine | 07:12 |
| stevemar | ahhh | 07:12 |
| stevemar | good point | 07:12 |
| *** Ala has joined #openstack-infra | 07:14 | |
| openstackgerrit | Steve Martinelli proposed openstack/requirements: Change install_requires to use link instead of url https://review.openstack.org/171060 | 07:15 |
| openstackgerrit | Marton Kiss proposed openstack-infra/system-config: Pin askbot theme to stable version https://review.openstack.org/171066 | 07:15 |
| stevemar | rm_work, alright, one more try! | 07:15 |
| * stevemar was ready to sleep 2 hours ago | 07:15 | |
| rm_work | heh | 07:15 |
| rm_work | I have given up hope of ever going to sleep before about 3am where I am | 07:16 |
| stevemar | rm_work, google says you're a racker so you still have another 40 minutes | 07:17 |
| rm_work | yep | 07:17 |
| stevemar | i'm on east coast time :) | 07:17 |
| rm_work | heh | 07:17 |
| *** Bsony has quit IRC | 07:18 | |
| openstackgerrit | Jan Provaznik proposed openstack-infra/project-config: Create tripleo-common project under TripleO https://review.openstack.org/165433 | 07:18 |
| *** yamada-h has joined #openstack-infra | 07:18 | |
| rm_work | ah, IBM has offices everywhere I guess | 07:19 |
| stevemar | rm_work, that's an understatement :P | 07:19 |
| *** wicope has joined #openstack-infra | 07:20 | |
| rm_work | IBM offices, quantum-entangled with every particle in the universe -- or if you're a Star Trek fan, IBM offices have reached Warp 10 | 07:21 |
| *** teran has quit IRC | 07:22 | |
| stevemar | rm_work, that sounds about right. quantum-entangled with every particle on earth at least. | 07:22 |
| *** achanda has quit IRC | 07:23 | |
| stevemar | that bug i filed has been gaining lots of comments | 07:23 |
| stevemar | fun times | 07:23 |
| rm_work | heh | 07:23 |
| rm_work | wat | 07:24 |
| rm_work | "Anyone know how to use @Adam Harwell (adam-harwell) that github ?" | 07:24 |
| *** jogo has quit IRC | 07:25 | |
| stevemar | rm_work, yeah i was thoroughly confused by that one | 07:26 |
| *** achanda has joined #openstack-infra | 07:26 | |
| stevemar | gorram reavers! grenade failed | 07:26 |
| rm_work | i think maybe he thinks the github link i provided is somehow a fix | 07:26 |
| stevemar | probably | 07:26 |
| *** mmedvede has quit IRC | 07:27 | |
| *** ajo has quit IRC | 07:27 | |
| *** markus_z has joined #openstack-infra | 07:27 | |
| *** tkelsey has joined #openstack-infra | 07:28 | |
| *** soren_ has joined #openstack-infra | 07:29 | |
| stevemar | rm_work, gah! it has to be backported too, i think | 07:30 |
| *** tkelsey has quit IRC | 07:30 | |
| *** wicope has joined #openstack-infra | 07:30 | |
| *** soren_ has quit IRC | 07:30 | |
| rm_work | err | 07:30 |
| *** soren has joined #openstack-infra | 07:30 | |
| rm_work | backported? what does that mean in this context | 07:30 |
| stevemar | http://logs.openstack.org/60/171060/3/check/check-grenade-dsvm/71ad0ca/logs/grenade.sh.txt.gz | 07:30 |
| rm_work | are the dsvm tests not running on master? | 07:30 |
| rm_work | or, on this change? | 07:31 |
| rm_work | oh god | 07:31 |
| stevemar | rm_work, the requirements project will also run a grenade test, so it'll attempt to upgrade juno to master | 07:31 |
| *** BharatK has joined #openstack-infra | 07:31 | |
| rm_work | yeah i see it | 07:31 |
| rm_work | T_T | 07:31 |
| stevemar | it'll use the requirements from juno, which will have it's own stable brnach | 07:32 |
| stevemar | sigh | 07:32 |
| stevemar | though zuul seems to be happy with the rest | 07:32 |
| rm_work | so far | 07:32 |
| rm_work | they aren't done yet | 07:32 |
| *** yamada-h has quit IRC | 07:32 | |
| rm_work | so what is the procedure for this | 07:32 |
| rm_work | is there a procedure for this? :P | 07:32 |
| stevemar | hehe | 07:32 |
| stevemar | just cherry pick the change i guess | 07:33 |
| stevemar | https://review.openstack.org/#/c/171070/ | 07:33 |
| stevemar | love that button | 07:33 |
| rm_work | i have never figured out how to use it properly | 07:33 |
| rm_work | ah, i see why now, it's meant for REAL branches, not cherry-picking on top of other CRs like I assumed | 07:34 |
| *** andreykurilin_ has joined #openstack-infra | 07:34 | |
| stevemar | yep | 07:34 |
| *** chlong has quit IRC | 07:34 | |
| stevemar | for doing the latter, i just do `git review -x <ps#>` | 07:34 |
| rm_work | lol so the juno one needs to merge first, THEN the one for master can merge | 07:35 |
| *** shardy has joined #openstack-infra | 07:35 | |
| stevemar | yep | 07:35 |
| stevemar | the juno one has a grenade job too | 07:35 |
| stevemar | this could be murky | 07:35 |
| rm_work | T_________T | 07:36 |
| *** ibiris_away is now known as ibiris | 07:36 | |
| *** jlibosva has joined #openstack-infra | 07:36 | |
| stevemar | i feel bad for the non PST-EST devs, we left them with a mess today | 07:36 |
| *** achanda has quit IRC | 07:37 | |
| *** ddieterly has joined #openstack-infra | 07:38 | |
| stevemar | rm_work, it might need to be fixed in icehouse -> then juno -> then master | 07:38 |
| stevemar | fun times | 07:38 |
| rm_work | whelp | 07:38 |
| stevemar | the icehouse branch doesn't have a grenade job | 07:38 |
| stevemar | whelp indeed | 07:39 |
| rm_work | at least that's as deep as the rabbit-hole goes | 07:39 |
| *** e0ne has joined #openstack-infra | 07:40 | |
| stevemar | rm_work, well i'm happy, the code works for all but the grenade job | 07:41 |
| rm_work | yeah, that is good | 07:41 |
| stevemar | i'm taking that as a win, i'll let the early risers deal with the fallout in the morning | 07:41 |
| rm_work | did you propose for icehouse? | 07:41 |
| stevemar | yep | 07:41 |
| stevemar | click on the change id of the patch, | 07:42 |
| stevemar | the other patches will show up | 07:42 |
| *** ddieterly has quit IRC | 07:42 | |
| rm_work | it's always fun when everything hits the fan | 07:43 |
| rm_work | thanks for the co-authored, now i can have code retroactively in icehouse :P | 07:44 |
| stevemar | np at all, that'll be good retroactively good for a free ticket to the vancouver summit :P | 07:45 |
| rm_work | heh | 07:45 |
| rm_work | i already had two | 07:45 |
| rm_work | ATC and Speaker, soon I'll even know if I am going :) | 07:45 |
| *** otter768 has joined #openstack-infra | 07:46 | |
| *** YorikSar has quit IRC | 07:46 | |
| *** dguitarbite has joined #openstack-infra | 07:47 | |
| stevemar | ohh barbican talk, nice | 07:47 |
| *** devvesa has joined #openstack-infra | 07:49 | |
| stevemar | zzzz time for me, thanks for all the help rm_work ! | 07:50 |
| *** jistr has quit IRC | 07:50 | |
| rm_work | yeah, later :) | 07:50 |
| *** e0ne has quit IRC | 07:51 | |
| *** otter768 has quit IRC | 07:51 | |
| *** ibiris is now known as ibiris_away | 07:52 | |
| *** yamada-h has joined #openstack-infra | 07:53 | |
| *** stevemar has quit IRC | 07:54 | |
| *** markvoelker has joined #openstack-infra | 07:56 | |
| *** wicope has joined #openstack-infra | 07:57 | |
| *** yamada-h has quit IRC | 07:57 | |
| *** dhritishikhar__ has quit IRC | 08:00 | |
| *** zz_johnthetubagu is now known as johnthetubaguy | 08:00 | |
| *** markvoelker has quit IRC | 08:00 | |
| *** ajo has joined #openstack-infra | 08:00 | |
| openstackgerrit | Max Rydahl Andersen proposed openstack-infra/jenkins-job-builder: Add support for 'raw' to inject direct xml to job configs. https://review.openstack.org/160158 | 08:01 |
| *** dhritishikhar_ has joined #openstack-infra | 08:01 | |
| *** andreykurilin_ has quit IRC | 08:03 | |
| *** mmedvede has joined #openstack-infra | 08:03 | |
| *** luqas__ has joined #openstack-infra | 08:03 | |
| *** luqas__ has left #openstack-infra | 08:04 | |
| *** e0ne has joined #openstack-infra | 08:06 | |
| *** andrey-mp has quit IRC | 08:07 | |
| *** hashar has joined #openstack-infra | 08:07 | |
| *** ibiris_away is now known as ibiris | 08:08 | |
| *** arxcruz has joined #openstack-infra | 08:09 | |
| *** jistr has joined #openstack-infra | 08:09 | |
| rm_work | Alright, we'll need to start with merging this one ASAP: https://review.openstack.org/#/c/171071/ | 08:10 |
| rm_work | Then wait for it to pass through gate, and then merge https://review.openstack.org/#/c/171070/ | 08:11 |
| *** wicope has quit IRC | 08:11 | |
| *** dhritishikhar_ has quit IRC | 08:11 | |
| *** dhritishikhar_ has joined #openstack-infra | 08:14 | |
| *** e0ne has quit IRC | 08:15 | |
| *** MaxV has joined #openstack-infra | 08:16 | |
| *** e0ne has joined #openstack-infra | 08:17 | |
| *** dhritishikhar_ has quit IRC | 08:18 | |
| *** dhritishikhar_ has joined #openstack-infra | 08:18 | |
| *** dhritishikhar_ has quit IRC | 08:18 | |
| *** dhritishikhar_ has joined #openstack-infra | 08:18 | |
| *** dhritishikhar_ has joined #openstack-infra | 08:19 | |
| *** dhritishikhar_ has quit IRC | 08:19 | |
| *** dhritishikhar_ has joined #openstack-infra | 08:20 | |
| *** zigzg_ has quit IRC | 08:20 | |
| *** e0ne has quit IRC | 08:21 | |
| *** ociuhandu has joined #openstack-infra | 08:22 | |
| anteaya | ttx: have you stable requirements +2? | 08:22 |
| anteaya | ttx the tl;dr is that rm_work's two patches are needed to fix the world | 08:22 |
| *** wicope has joined #openstack-infra | 08:27 | |
| *** wicope has quit IRC | 08:27 | |
| *** eantyshev has joined #openstack-infra | 08:27 | |
| *** mrda is now known as mrda-away | 08:28 | |
| *** wicope has joined #openstack-infra | 08:28 | |
| *** HeOS has joined #openstack-infra | 08:29 | |
| eantyshev | Hi! Many thanks to you, anteaya, akerr and asselin for yesterday attention to my SCP plugin problem! | 08:30 |
| anteaya | eantyshev: wonderful | 08:30 |
| anteaya | join me in #openstack-meeting | 08:30 |
| anteaya | there is another third party meeting going on | 08:30 |
| anteaya | we can log our conversation | 08:31 |
| anteaya | others will wnt to know your progress | 08:31 |
| *** ihrachyshka has joined #openstack-infra | 08:31 | |
| *** yamahata has quit IRC | 08:32 | |
| openstackgerrit | Alexander Evseev proposed openstack-infra/system-config: New user — aevseev https://review.openstack.org/171085 | 08:37 |
| *** doude has joined #openstack-infra | 08:38 | |
| *** ddieterly has joined #openstack-infra | 08:38 | |
| *** YorikSar has joined #openstack-infra | 08:39 | |
| openstackgerrit | yolanda.robla proposed openstack-infra/puppet-pip: Add the ability to set pip.conf and configure it https://review.openstack.org/168309 | 08:41 |
| anteaya | eantyshev: that patch won't merge, the changes you made are for adding root access to all our servers for a infra core team member | 08:41 |
| anteaya | eantyshev: mind if I ask what motivated you to offer that as a patch? | 08:42 |
| anteaya | have we erred in our documentation somehow? | 08:42 |
| *** _skolekonov is now known as skolekonov | 08:43 | |
| *** dizquierdo has joined #openstack-infra | 08:43 | |
| *** ddieterly has quit IRC | 08:43 | |
| eantyshev | anteaya: you're addressing the wrong person | 08:45 |
| anteaya | eantyshev: sorry about that | 08:46 |
| *** yamamoto has quit IRC | 08:46 | |
| anteaya | eantyshev: sorry the timing and the names were close, so I guessed | 08:47 |
| *** ibiris is now known as ibiris_away | 08:48 | |
| ttx | anteaya: maybe, let me check | 08:48 |
| ttx | anteaya: where are those two patches ? | 08:48 |
| anteaya | https://review.openstack.org/#/c/171071/ | 08:49 |
| anteaya | https://review.openstack.org/#/c/171070/ | 08:49 |
| *** ssam2 has joined #openstack-infra | 08:49 | |
| anteaya | ttx: if not you, who? | 08:49 |
| ttx | booo to pip | 08:50 |
| ttx | anteaya: I haz rightz | 08:50 |
| ttx | approved | 08:50 |
| *** tnovacik has joined #openstack-infra | 08:51 | |
| anteaya | yay | 08:51 |
| ttx | If the second one can "depend" on the first one I can approve it now too | 08:51 |
| anteaya | the pipelines they must flow | 08:51 |
| *** Hal has joined #openstack-infra | 08:51 | |
| *** Hal is now known as Guest47799 | 08:51 | |
| anteaya | I don't see a dependency there | 08:52 |
| ttx | If I approve the juno one now without a depend it will fail tests | 08:52 |
| *** wicope has joined #openstack-infra | 08:53 | |
| *** wicope has quit IRC | 08:53 | |
| *** asrangne has joined #openstack-infra | 08:54 | |
| anteaya | yeah, that is a waste of time | 08:55 |
| *** aswadr has quit IRC | 08:56 | |
| *** asrangne__ has joined #openstack-infra | 08:56 | |
| *** markvoelker has joined #openstack-infra | 08:56 | |
| anteaya | almost time I can go back to bed again | 08:58 |
| *** dhritishikhar_ has left #openstack-infra | 08:59 | |
| *** asrangne has quit IRC | 09:00 | |
| anteaya | and I'm back to bed, then traveling to pycon, though I expect to have lovely train wifi for meeting season | 09:00 |
| anteaya | ttx: may many rc1 cuts be in your future | 09:00 |
| *** markvoelker has quit IRC | 09:01 | |
| *** e0ne has joined #openstack-infra | 09:01 | |
| *** e0ne is now known as e0ne_ | 09:01 | |
| ttx | anteaya: ack, gdnite | 09:02 |
| anteaya | :) | 09:02 |
| *** dtantsur is now known as dtantsur|brb | 09:03 | |
| *** claudiub has joined #openstack-infra | 09:03 | |
| *** yamada-h has joined #openstack-infra | 09:04 | |
| *** plol has joined #openstack-infra | 09:04 | |
| plol | In zuul, can I gate changes depending on external conditions? I e not mergingnew changes to master if a nightly build fails | 09:04 |
| openstackgerrit | yolanda.robla proposed openstack-infra/storyboard-webclient: Initial proposal for integration tests https://review.openstack.org/156217 | 09:05 |
| *** yamada-h has quit IRC | 09:08 | |
| *** e0ne_ is now known as e0ne | 09:10 | |
| *** yamada-h has joined #openstack-infra | 09:11 | |
| *** persia_ is now known as persia | 09:14 | |
| *** ociuhandu has quit IRC | 09:17 | |
| *** eantyshev has quit IRC | 09:18 | |
| *** wenlock_ has quit IRC | 09:18 | |
| *** wznoinsk has joined #openstack-infra | 09:20 | |
| *** wenlock has joined #openstack-infra | 09:23 | |
| *** Longgeek has quit IRC | 09:23 | |
| *** dizquierdo is now known as dizquierdo_afk | 09:24 | |
| *** pelix has joined #openstack-infra | 09:24 | |
| *** dizquierdo_afk has quit IRC | 09:29 | |
| *** dims has joined #openstack-infra | 09:29 | |
| *** koolhead17 has joined #openstack-infra | 09:30 | |
| *** teran has joined #openstack-infra | 09:32 | |
| *** teran has quit IRC | 09:33 | |
| *** teran has joined #openstack-infra | 09:33 | |
| *** dims has quit IRC | 09:35 | |
| *** ddieterly has joined #openstack-infra | 09:39 | |
| *** ddieterly has quit IRC | 09:44 | |
| *** wenlock has quit IRC | 09:44 | |
| *** wicope has joined #openstack-infra | 09:46 | |
| *** wicope has joined #openstack-infra | 09:47 | |
| *** otter768 has joined #openstack-infra | 09:47 | |
| *** wenlock has joined #openstack-infra | 09:48 | |
| *** wicope has joined #openstack-infra | 09:48 | |
| openstackgerrit | Andrew McDermott proposed openstack/requirements: pip 6.1.0 changed 'url' to 'link', amend update.py https://review.openstack.org/171106 | 09:49 |
| *** otter768 has quit IRC | 09:52 | |
| *** markvoelker has joined #openstack-infra | 09:57 | |
| *** markvoelker has quit IRC | 10:02 | |
| *** ociuhandu has joined #openstack-infra | 10:03 | |
| therve | Is anyone looking at https://review.openstack.org/#/c/171060 ? It's blocking the gate everywhere AFAICT | 10:04 |
| *** wicope has joined #openstack-infra | 10:04 | |
| therve | Ah there is https://review.openstack.org/171106 too... | 10:04 |
| *** chlong has joined #openstack-infra | 10:07 | |
| *** dtantsur|brb is now known as dtantsur | 10:07 | |
| therve | frobware, Is your patch backward compatible? | 10:08 |
| frobware | therve, good question. For this patch I went through the motions of actually capturing what was working for me, rather than stashing a patch locally and hoping somebody else would fix it. | 10:09 |
| therve | OK, I believe the other patch is on a better track | 10:09 |
| therve | We could also close the bug you linked as duplicate | 10:10 |
| frobware | therve, ok fine. Could you point me at the other patch/url, thx. | 10:10 |
| therve | frobware, https://bugs.launchpad.net/tempest/+bug/1440984 and https://review.openstack.org/#/c/171060 | 10:10 |
| openstack | Launchpad bug 1440984 in tempest "AttributeError: 'InstallRequirement' when running update.py" [Undecided,New] | 10:10 |
| openstackgerrit | yolanda.robla proposed openstack-infra/system-config: Start using puppet-puppet module to configure puppet https://review.openstack.org/171113 | 10:11 |
| ttx | therve: we need to get https://review.openstack.org/#/c/171071/ in first. just failed gate | 10:13 |
| *** wenlock has quit IRC | 10:13 | |
| *** wenlock has joined #openstack-infra | 10:14 | |
| ttx | Both patches have same ChangeID, but one depends on the other. Does anyone know if adding a "depends" on the same change ID would create a loop there ? | 10:14 |
| therve | ttx, Ah we need the icehouse version changed first? | 10:14 |
| therve | Fucking fantastic | 10:14 |
| *** doude has quit IRC | 10:14 | |
| ttx | yep, it's like a chain. The juno change can't merge until the icehouse one merged | 10:15 |
| openstackgerrit | yolanda.robla proposed openstack-infra/system-config: Start using puppet-puppet module to configure puppet https://review.openstack.org/171113 | 10:15 |
| *** koolhead17 has quit IRC | 10:15 | |
| ttx | (or at least, that's my understanding) | 10:15 |
| *** doude has joined #openstack-infra | 10:16 | |
| *** e0ne is now known as e0ne_ | 10:18 | |
| *** MaxV has quit IRC | 10:18 | |
| *** wenlock has quit IRC | 10:22 | |
| *** e0ne_ is now known as e0ne | 10:22 | |
| *** Longgeek has joined #openstack-infra | 10:24 | |
| *** wenlock has joined #openstack-infra | 10:25 | |
| *** cdent has joined #openstack-infra | 10:26 | |
| openstackgerrit | Mateusz Matuszkowiak proposed openstack-infra/project-config: Added MidoNet SDN Plugin for Fuel https://review.openstack.org/171116 | 10:26 |
| dstufft | ttx: if you wait a few I'll have a 6.1.1 out that will (temporarily) fix update.py | 10:28 |
| *** pal has quit IRC | 10:28 | |
| dstufft | you should still update though, because it's only a temp fix | 10:28 |
| *** mrunge has quit IRC | 10:29 | |
| dstufft | https://github.com/pypa/pip/pull/2650 | 10:29 |
| *** jistr has quit IRC | 10:30 | |
| deepakcs | anteaya, i sent a update mail to third-party announce w/o subscribing first, let me know if u can approve or i shud cancel and resubmit | 10:30 |
| *** Longgeek has quit IRC | 10:30 | |
| openstackgerrit | Paul Karikh proposed openstack-infra/project-config: Update jobs for Merlin https://review.openstack.org/171117 | 10:31 |
| *** andrey-mp has joined #openstack-infra | 10:34 | |
| *** dizquierdo_afk has joined #openstack-infra | 10:34 | |
| *** dizquierdo_afk is now known as dizquierdo | 10:36 | |
| *** wenlock has quit IRC | 10:37 | |
| *** dhritishikhar_ has joined #openstack-infra | 10:38 | |
| *** samueldmq has joined #openstack-infra | 10:38 | |
| *** dhritishikhar_ has quit IRC | 10:39 | |
| *** andrey-mp has quit IRC | 10:40 | |
| *** _nadya_ has joined #openstack-infra | 10:41 | |
| openstackgerrit | Paul Karikh proposed openstack-infra/project-config: Update jobs for Merlin https://review.openstack.org/171117 | 10:42 |
| *** Mmike has quit IRC | 10:47 | |
| *** pc_m has joined #openstack-infra | 10:49 | |
| *** Mmike has joined #openstack-infra | 10:49 | |
| *** Mmike has joined #openstack-infra | 10:49 | |
| *** jistr has joined #openstack-infra | 10:51 | |
| *** jistr is now known as jistr|biab | 10:52 | |
| *** pal has joined #openstack-infra | 10:53 | |
| mescanef | hi, any issues with gate-project-config-irc-access today? | 10:57 |
| *** markvoelker has joined #openstack-infra | 10:58 | |
| *** dhritishikhar_ has joined #openstack-infra | 10:58 | |
| *** marcusvrn has joined #openstack-infra | 11:02 | |
| *** markvoelker has quit IRC | 11:03 | |
| *** sdake_ has joined #openstack-infra | 11:03 | |
| *** tlbr has quit IRC | 11:04 | |
| *** MaxV has joined #openstack-infra | 11:05 | |
| *** dguitarbite has quit IRC | 11:06 | |
| *** dguitarbite has joined #openstack-infra | 11:06 | |
| *** sdake has quit IRC | 11:07 | |
| *** tlbr has joined #openstack-infra | 11:08 | |
| fungi | ttx: i believe having a change depends-on its own change-id is going to cause zuul to give up and ignore events for that change | 11:11 |
| fungi | it considers dependency loops an undefined behavior and errs on the side of caution by doing nothing | 11:11 |
| sdague | ttx: yeh, the duplicate ChangeId for backport policy basically breaks you here | 11:12 |
| dstufft | ttx: 6.1.1 should be released now | 11:13 |
| *** spredzy is now known as spredzy|afk | 11:16 | |
| *** cdent_ has joined #openstack-infra | 11:16 | |
| fungi | dstufft: thanks for the concession--i suspect we were close to having the fix in place but that helps | 11:18 |
| *** viktors|lunch has joined #openstack-infra | 11:18 | |
| dstufft | fungi: I was releasing 6.1.1 anyways for another issue that had unintended side effects | 11:19 |
| fungi | mescanef: you'd have to point me to a failure log--i have no idea what's broken without a description of the problem | 11:19 |
| dstufft | so I tossed in a quick shim for it | 11:19 |
| sdague | dstufft: thanks | 11:19 |
| *** cdent_ has quit IRC | 11:19 | |
| fungi | dstufft: awesome, and apologies for the hassle. i still think a lot of people don't realize that while you _can_ import pip, if it breaks you get to keep both pieces | 11:20 |
| dstufft | fungi: didn't bother me none :) | 11:20 |
| *** cdent has quit IRC | 11:20 | |
| *** pal has quit IRC | 11:21 | |
| sdague | fungi: I've got test patch in flight that might stop that, I actually have no idea about the origin of that pip parsing bit in that script | 11:21 |
| *** cdent has joined #openstack-infra | 11:22 | |
| sdague | ah, 6.1.1 doesn't actually fix it for us, it fails in a new way | 11:22 |
| *** dhritishikhar_ has quit IRC | 11:22 | |
| fungi | new and exciting | 11:22 |
| dstufft | sdague: oh :/ I didn't test it, I just assumed it would fix it | 11:22 |
| sdague | 2015-04-07 11:21:48.410 | File "/usr/local/lib/python2.7/dist-packages/pip/req/req_install.py", line 128, in url | 11:22 |
| sdague | 2015-04-07 11:21:48.410 | return self.link.url | 11:22 |
| sdague | 2015-04-07 11:21:48.410 | AttributeError: 'NoneType' object has no attribute 'url' | 11:22 |
| dstufft | oh | 11:22 |
| dstufft | durr | 11:22 |
| dstufft | forgot to guard it | 11:22 |
| dstufft | oh well | 11:22 |
| sdague | are you going to spin a 6.1.2 with that fixed? | 11:23 |
| sdague | we can probably disconnect this logic by the end of the day, but unless we have a version that works with the old code, all the openstack code is stopped until then | 11:25 |
| *** ibiris_away is now known as ibiris | 11:25 | |
| dstufft | sdague: probably not unless you really need it. it should only happen if someone is touching InstallRequirement.url so it's limited to openstack probably. I wasn't planning on releasing the shim at all except I was already doing a release so I tossed it in | 11:26 |
| sdague | well, this is supposed to be RC week | 11:26 |
| sdague | and this is going to mean a lost day more or less | 11:27 |
| *** dhritishikhar_ has joined #openstack-infra | 11:27 | |
| fungi | i'm on hand to get whatever fixes we need rammed through on our side | 11:27 |
| *** arxcruz has quit IRC | 11:28 | |
| *** devvesa has quit IRC | 11:28 | |
| *** arxcruz has joined #openstack-infra | 11:28 | |
| openstackgerrit | Sean M. Collins proposed openstack/requirements: Check for the url attribute before accessing https://review.openstack.org/171139 | 11:28 |
| *** mrunge has joined #openstack-infra | 11:28 | |
| marcusvrn | dstufft: sdague: hi! our CI started failing, returning the same error that sdague reported, while devstack is setup | 11:31 |
| *** dizquierdo has quit IRC | 11:32 | |
| fungi | marcusvrn: yes, it's almost certainly running the same script | 11:32 |
| ttx | sdague, fungi: considering pushing the change with a separate changeID just so that we can merge both at the same time | 11:32 |
| dstufft | sdague: sorry, got a bunch of people talking to me at once. fungi said he thought y'all were close to having a fix in place anyways. will it really burn a day? | 11:32 |
| *** yamamoto has joined #openstack-infra | 11:32 | |
| fungi | marcusvrn: we're relying on internal implementation details of a tool which has no defined stable api | 11:32 |
| ttx | s/at the same time/in a depends chain/ | 11:33 |
| dstufft | cutting a release will notify anyone who already has 6.1.1 that there's a 6.1.2 | 11:33 |
| *** dguitarbite has quit IRC | 11:33 | |
| ttx | though I guess we only win 28 min at this point | 11:33 |
| marcusvrn | fungi: hm... I see | 11:33 |
| *** tnovacik has quit IRC | 11:33 | |
| *** viktors|lunch is now known as viktors | 11:33 | |
| fungi | ttx: well, with nothing able to enter the integrated gate pipeline right now, there shouldn't be a lot of competition for these patches anyway | 11:34 |
| ttx | fungi: there is competition on the check queue | 11:34 |
| ttx | and that one is growing | 11:34 |
| ttx | Took my retry about 30min to get a node | 11:34 |
| fungi | i can take care of that. what's the change id? | 11:34 |
| ttx | fungi: the icehouse stuff is on gate, 1st slot | 11:35 |
| sdague | fungi: honestly, just ninja merge the existing fix on all branches | 11:35 |
| ttx | and the juno stuff can't get tested until the icehouse one merges | 11:35 |
| dstufft | sdague: fungi ttx I guess what I'm saying, if it's gonna burn a day I can release a 6.1.2, if it's only going to be an hour or so i'd prefer not | 11:35 |
| *** bhunter71 has joined #openstack-infra | 11:35 | |
| sdague | I'm working on a better fix, which will disconnect us from pip | 11:35 |
| ttx | sdague: +1 -- all for gaining a couple hours here | 11:35 |
| *** dguitarbite has joined #openstack-infra | 11:36 | |
| ttx | I have several RC1s blocked right now | 11:36 |
| sdague | ttx: is the fix up for master as well? | 11:36 |
| *** doug-fish has joined #openstack-infra | 11:36 | |
| fungi | this patch is trivial and fairly heavily tested now. i'm fine overriding gating to get it solved and move on | 11:36 |
| ttx | yes | 11:36 |
| ttx | https://review.openstack.org/#/q/Icf60f64f4e84f0c506263202821c8374c914e121,n,z | 11:36 |
| sdague | fungi: yeh, gate override ftw here | 11:36 |
| ttx | though Isaku has a -1 that Julien -1ed too there | 11:37 |
| *** pblaho has joined #openstack-infra | 11:37 | |
| ttx | fungi: you've hot our collective +2 here | 11:38 |
| ttx | goit* | 11:38 |
| *** amotoki has quit IRC | 11:38 | |
| *** yamamoto has quit IRC | 11:38 | |
| *** yamada-h has quit IRC | 11:38 | |
| fungi | yep, those -1's look like trivial nit-picking on a working implementation | 11:38 |
| ttx | you'll have to boot https://review.openstack.org/#/c/171071/ off the queue | 11:39 |
| fungi | it'll be fine. can you at least go ahead and approve the other two patches, for formality's sake? | 11:39 |
| fungi | then i'll merge them | 11:39 |
| ttx | on it, not W+1 it | 11:39 |
| openstackgerrit | Serg Melikyan proposed openstack-infra/project-config: Rename stackforge/murano to openstack/murano https://review.openstack.org/167934 | 11:40 |
| ttx | fungi: +2ed the other two | 11:40 |
| *** ddieterly has joined #openstack-infra | 11:40 | |
| fungi | ahh, good enough. i'll approve them then | 11:40 |
| *** ibiris is now known as ibiris_away | 11:41 | |
| *** enikanorov has quit IRC | 11:42 | |
| *** ldnunes has joined #openstack-infra | 11:42 | |
| *** yamada-h has joined #openstack-infra | 11:42 | |
| *** yamada-h has quit IRC | 11:42 | |
| openstackgerrit | Merged openstack/requirements: Change install_requires to use link instead of url https://review.openstack.org/171060 | 11:42 |
| ekarlso | hey guys | 11:42 |
| Kiall | Hey guys - something up with the DSVM gates at the moment? | 11:42 |
| Kiall | lol ekarlso | 11:42 |
| ekarlso | ... | 11:42 |
| Kiall | Was just about to ask ;) | 11:42 |
| fungi | okay, all three are merged to their respective branches now | 11:42 |
| ekarlso | ^ Kiall | 11:42 |
| Kiall | + python update.py /opt/stack/new/keystone is failing with " AttributeError: 'NoneType' object has no attribute 'url'" | 11:43 |
| Kiall | e.g. http://logs.openstack.org/97/171097/2/check/gate-designate-dsvm-powerdns/7dfe388/logs/devstacklog.txt.gz | 11:43 |
| *** enikanorov has joined #openstack-infra | 11:43 | |
| fungi | Kiall: ekarlso: yes, our integration tests were using pip internals in ways which changed with today's pip 6.1 release. the patches which adapt to it have just merged and things should start working now | 11:43 |
| Kiall | fungi: ah, perfect.. thanks :) | 11:44 |
| marcusvrn | fungi: thanks! | 11:44 |
| *** dizquierdo has joined #openstack-infra | 11:45 | |
| *** deepakcs has quit IRC | 11:45 | |
| *** ddieterly has quit IRC | 11:45 | |
| *** begal-sop__ has joined #openstack-infra | 11:46 | |
| *** begal-sop_ has quit IRC | 11:46 | |
| fungi | mescanef: http://logs.openstack.org/16/171116/1/check/gate-project-config-irc-access/c16153a/console.html is not a failure i recall seeing before. apparently the irc client received data it thought was utf8 but which it discovered was not decodable | 11:47 |
| *** andrey-mp has joined #openstack-infra | 11:47 | |
| *** sergmelikyan has joined #openstack-infra | 11:47 | |
| sergmelikyan | fungi, jeblair please review updated commit to rename Murano https://review.openstack.org/167934 | 11:48 |
| sergmelikyan | I've replied to all questions but I may missed something, please let me know if we need to update this commit with some details | 11:48 |
| *** otter768 has joined #openstack-infra | 11:48 | |
| andrey-mp | fungi, jeblair please review ec2-api gating fix :) https://review.openstack.org/#/c/170668/ | 11:49 |
| *** chlong has quit IRC | 11:49 | |
| mescanef | fungi: so what actions should I take here to get +1 from jenkins? :-) | 11:49 |
| *** jistr|biab is now known as jistr | 11:49 | |
| fungi | mescanef: i've rechecked that change to see if the error is consistent or if it was a transient behavior from the irc network | 11:50 |
| dstufft | fungi: things are fixed now? | 11:50 |
| fungi | dstufft: presumably, or we're at least freed up to discover what other new improvements may be breaking some of our assumptions | 11:50 |
| *** chlong has joined #openstack-infra | 11:51 | |
| dstufft | fungi: horray for postel's tarpit | 11:51 |
| fungi | yup | 11:51 |
| ttx | fungi: you might want to kill https://review.openstack.org/#/c/171071/ from queue now ? | 11:52 |
| ttx | it still appears in gate queue | 11:52 |
| *** otter768 has quit IRC | 11:52 | |
| *** Longgeek has joined #openstack-infra | 11:53 | |
| fungi | yeah, i can abort the one job still running for it | 11:56 |
| *** wenlock has joined #openstack-infra | 11:56 | |
| *** ibiris_away is now known as ibiris | 11:56 | |
| fungi | though zuul will simply refuse to merge it once that job finishes | 11:57 |
| fungi | since it's already merged | 11:57 |
| fungi | or rather gerrit will refuse to merge it when zuul makes the api call requesting it do so | 11:57 |
| fungi | anyway, job aborted | 11:58 |
| sdague | fungi: so ... https://review.openstack.org/#/c/171125/ look sane to you? | 11:58 |
| *** markvoelker has joined #openstack-infra | 11:59 | |
| *** coolsvap is now known as coolsvap|afk | 11:59 | |
| *** koolhead17 has joined #openstack-infra | 11:59 | |
| openstackgerrit | Mateusz Matuszkowiak proposed openstack-infra/project-config: Added MidoNet SDN Plugin for Fuel https://review.openstack.org/171116 | 12:00 |
| *** weshay has joined #openstack-infra | 12:00 | |
| fungi | sdague: yeah, i vaguely recall how the pkg_resources.Requirement class works and it looks like it should be a suitable replacement, though i haven't tested it personally | 12:01 |
| sdague | it seems to be passing so far | 12:01 |
| fungi | checking the logs now | 12:01 |
| *** mugsie has joined #openstack-infra | 12:02 | |
| sdague | I honestly think all the pip internals stuff was when we did things like let git urls into global requirements because of the way oslo used to do releases | 12:02 |
| fungi | that sounds right | 12:02 |
| *** spredzy|afk is now known as spredzy | 12:03 | |
| *** markvoelker has quit IRC | 12:03 | |
| fungi | yeah, http://logs.openstack.org/25/171125/4/check/check-tempest-dsvm-full/55c0f02//logs/devstacklog.txt.gz#_2015-04-07_11_40_15_473 seems to be doing the right thing | 12:04 |
| *** Guest47799 has quit IRC | 12:05 | |
| andrey-mp | ups, i pasted incorrect link for ec2 gating fix. right link is here - https://review.openstack.org/#/c/170646/ | 12:05 |
| ildikov | ttx: thanks for the 'recheck's | 12:06 |
| *** dkranz has joined #openstack-infra | 12:06 | |
| sdague | fungi: ok, I've got one last debug line in there, I'll nuke that, then propose it on all branches | 12:07 |
| *** wenlock has quit IRC | 12:07 | |
| fungi | sdague: i'll gladly +2 the master version | 12:08 |
| *** hogepodge has quit IRC | 12:08 | |
| fungi | i don't think i normally have +2 on its stable branches | 12:08 |
| *** prmtl has quit IRC | 12:10 | |
| *** woodster_ has joined #openstack-infra | 12:10 | |
| *** pal has joined #openstack-infra | 12:10 | |
| *** yamada-h has joined #openstack-infra | 12:13 | |
| *** BharatK has quit IRC | 12:13 | |
| *** _nadya_ has quit IRC | 12:15 | |
| *** rfolco has joined #openstack-infra | 12:15 | |
| fungi | also, bandersnatch refreshes completed after i went to bed and the mirrors are all updating normally again | 12:15 |
| *** baoli has joined #openstack-infra | 12:16 | |
| andrey-mp | thanks fungi :) | 12:16 |
| openstackgerrit | Sean Dague proposed openstack/requirements: attempt to stop using pip internals https://review.openstack.org/171157 | 12:16 |
| fungi | sdague: is that cherry-pick header in the commit message cruft? seems to reference a commit gerrit doesn't have | 12:17 |
| sdague | https://review.openstack.org/#/q/I10ada6860768017d4eb3c729f2870f2c7f4a46d7,n,z | 12:17 |
| *** yamada-h has quit IRC | 12:17 | |
| *** kerpan has joined #openstack-infra | 12:18 | |
| fungi | yeah, none of the patchsets of any of those seem to have a commit sha matching what's in your commit message though | 12:18 |
| *** fesp has joined #openstack-infra | 12:18 | |
| sdague | fungi: gah, apparently local issues? | 12:19 |
| *** fesp has quit IRC | 12:19 | |
| sdague | fungi: oh, I just forgot to push the icehouse one | 12:20 |
| *** hogepodge has joined #openstack-infra | 12:20 | |
| sdague | fixed now | 12:20 |
| *** dprince has joined #openstack-infra | 12:20 | |
| sdague | ttx: review of https://review.openstack.org/#/q/I10ada6860768017d4eb3c729f2870f2c7f4a46d7,n,z would be appreciated, I think this can get us away from pip issues in future | 12:21 |
| *** dtantsur_ has joined #openstack-infra | 12:21 | |
| *** dtantsur has quit IRC | 12:21 | |
| *** dtantsur_ is now known as dtantsur | 12:21 | |
| *** alexpilotti has joined #openstack-infra | 12:22 | |
| *** gulic has joined #openstack-infra | 12:22 | |
| *** devvesa has joined #openstack-infra | 12:23 | |
| ttx | sdague: on it | 12:23 |
| *** pcrews has quit IRC | 12:24 | |
| fungi | yep, the cherry-pick sha link resolves in gerrit now. also i apparently _do_ have +2 on reqs stable branches, i think we added that semi-recently | 12:24 |
| *** mwagner_lap has quit IRC | 12:25 | |
| sdague | fungi: I think infra inherits via release or something | 12:28 |
| *** baoli_ has joined #openstack-infra | 12:28 | |
| *** gordc has joined #openstack-infra | 12:28 | |
| *** mrunge is now known as mrunge_bbl | 12:28 | |
| *** smccully has joined #openstack-infra | 12:28 | |
| fungi | nah, it's added explicitly to the acl http://git.openstack.org/cgit/openstack-infra/project-config/tree/gerrit/acls/openstack/requirements.config | 12:29 |
| fungi | in fact, it was _your_ change that added us ;) https://review.openstack.org/150626 | 12:30 |
| *** baoli has quit IRC | 12:30 | |
| sdague | ah, yes :) | 12:30 |
| andrey-mp | sdague, can I ask you about review https://review.openstack.org/#/c/170646/ please? | 12:33 |
| *** pal_ has joined #openstack-infra | 12:33 | |
| sdague | andrey-mp: ok, I see you responded, +A | 12:34 |
| *** dims has joined #openstack-infra | 12:34 | |
| andrey-mp | thanks! | 12:35 |
| *** pal has quit IRC | 12:36 | |
| *** BharatK has joined #openstack-infra | 12:36 | |
| *** HeOS has quit IRC | 12:37 | |
| *** [HeOS] has joined #openstack-infra | 12:37 | |
| *** vlaza has joined #openstack-infra | 12:38 | |
| *** aysyd has joined #openstack-infra | 12:38 | |
| *** dims has quit IRC | 12:39 | |
| *** stevemar has joined #openstack-infra | 12:39 | |
| *** dkranz has quit IRC | 12:40 | |
| openstackgerrit | Sean Dague proposed openstack/requirements: add what-broke.py analayzer https://review.openstack.org/166997 | 12:41 |
| sdague | fungi / ttx while things are being reviewed, want to take a look at that ^^^ | 12:41 |
| sdague | I think it would be handy to have in tree | 12:41 |
| *** ddieterly has joined #openstack-infra | 12:41 | |
| openstackgerrit | Merged openstack-infra/project-config: Fix ec2-api gating job https://review.openstack.org/170646 | 12:42 |
| *** btully has joined #openstack-infra | 12:42 | |
| openstackgerrit | Julia Kreger proposed openstack-infra/shade: Change Ironic node lookups to support names https://review.openstack.org/171164 | 12:42 |
| openstackgerrit | Julia Kreger proposed openstack-infra/shade: WIP: Add Ironic update_machine method https://review.openstack.org/171165 | 12:42 |
| *** ddieterly has quit IRC | 12:46 | |
| *** rlandy has joined #openstack-infra | 12:48 | |
| *** vlaza is now known as Vic | 12:49 | |
| *** doug-fish has quit IRC | 12:51 | |
| *** macjack has joined #openstack-infra | 12:52 | |
| openstackgerrit | Joshua Hesketh proposed openstack-infra/os-loganalyze: Add support for non-text files https://review.openstack.org/107267 | 12:52 |
| *** macjack has quit IRC | 12:53 | |
| *** btully has quit IRC | 12:53 | |
| *** Vic is now known as vlaza | 12:56 | |
| *** andrey-mp has quit IRC | 12:57 | |
| *** Krinkle|detached is now known as Krinkle | 12:57 | |
| openstackgerrit | Merged openstack-infra/project-config: Added MidoNet SDN Plugin for Fuel https://review.openstack.org/171116 | 12:57 |
| *** annegentle has joined #openstack-infra | 12:58 | |
| *** macjack has joined #openstack-infra | 12:59 | |
| *** markvoelker has joined #openstack-infra | 12:59 | |
| *** amotoki has joined #openstack-infra | 13:00 | |
| *** macjack has quit IRC | 13:00 | |
| *** fifieldt has joined #openstack-infra | 13:00 | |
| *** sarob has joined #openstack-infra | 13:01 | |
| openstackgerrit | Merged openstack-infra/gerrit: Add username to stream-events queue entries https://review.openstack.org/170948 | 13:01 |
| *** ildikov has quit IRC | 13:02 | |
| *** prmtl has joined #openstack-infra | 13:02 | |
| *** ildikov has joined #openstack-infra | 13:03 | |
| stevemar | yay things are working! | 13:04 |
| *** markvoelker has quit IRC | 13:04 | |
| *** kgiusti has joined #openstack-infra | 13:06 | |
| *** annegentle has quit IRC | 13:06 | |
| *** bknudson has joined #openstack-infra | 13:06 | |
| *** annegentle has joined #openstack-infra | 13:06 | |
| *** openstackgerrit has quit IRC | 13:07 | |
| fungi | yep, gating all the things and thinging all the gates | 13:07 |
| *** openstackgerrit has joined #openstack-infra | 13:07 | |
| fungi | less hate, more gate | 13:07 |
| *** zigzg_ has joined #openstack-infra | 13:07 | |
| * fungi has a new slogan to propose for tee shirts | 13:08 | |
| *** e0ne is now known as e0ne_ | 13:08 | |
| *** prmtl has quit IRC | 13:08 | |
| *** e0ne_ is now known as e0ne | 13:08 | |
| stevemar | fungi, that is actually a good slogan... | 13:09 |
| fungi | would have worked a little better if we'd settled on "love" instead of "liberty" for the next release | 13:10 |
| *** dims has joined #openstack-infra | 13:11 | |
| annegentle | gaters gonna gate | 13:12 |
| *** mwagner_lap has joined #openstack-infra | 13:12 | |
| *** pal_ has quit IRC | 13:12 | |
| *** wicope has joined #openstack-infra | 13:13 | |
| *** tiswanso has joined #openstack-infra | 13:13 | |
| *** wicope has joined #openstack-infra | 13:13 | |
| *** wicope has joined #openstack-infra | 13:14 | |
| fungi | ooh! good one | 13:14 |
| * annegentle prices t-shirts | 13:14 | |
| *** wenlock has joined #openstack-infra | 13:15 | |
| *** doug-fish has joined #openstack-infra | 13:15 | |
| stevemar | love was my top choice for the name | 13:16 |
| mordred | lemming for life | 13:19 |
| bookwar | i am still feel sorry for Lizard which didn't make it :( | 13:19 |
| *** ZZelle has joined #openstack-infra | 13:21 | |
| *** trey has quit IRC | 13:23 | |
| *** wenlock has quit IRC | 13:24 | |
| *** trey has joined #openstack-infra | 13:25 | |
| *** Hal has joined #openstack-infra | 13:26 | |
| *** zigzg_ has quit IRC | 13:26 | |
| *** Hal is now known as Guest54708 | 13:26 | |
| *** gibi has quit IRC | 13:27 | |
| *** MaxV has quit IRC | 13:27 | |
| *** gibi has joined #openstack-infra | 13:27 | |
| *** MaxV has joined #openstack-infra | 13:27 | |
| openstackgerrit | Jeremy Stanley proposed openstack-infra/system-config: Revert "Add a helpful friend to gerrit reviews" https://review.openstack.org/169508 | 13:27 |
| *** ayoung has joined #openstack-infra | 13:28 | |
| sdague | fungi: https://review.openstack.org/#/q/I10ada6860768017d4eb3c729f2870f2c7f4a46d7,n,z - are all passing | 13:30 |
| sdague | so it might be nice to get those into the gate soon, and finally isolate ourselves from pip there | 13:30 |
| *** dustins has joined #openstack-infra | 13:31 | |
| sdague | mordred: you might be a good final set of eyes on those as well, removing pip internals usage from requirements update | 13:31 |
| mordred | sdague: looking | 13:32 |
| fungi | oh, i already went ahead and approved the stable branch ones since we had 2x+2 and it was proposed by a third core reviewer | 13:32 |
| sdague | fungi: yeh, that's fine as well | 13:32 |
| fungi | but i agree we can block gating before they merge if mordred sees something objectionable | 13:32 |
| fungi | it will take a while | 13:33 |
| sdague | yep, or we can figure out what to fix afterwards | 13:33 |
| mordred | nope. looks great | 13:33 |
| sdague | that import of pip internals comes from the original commit of the repo | 13:33 |
| sdague | git show b788389 | 13:33 |
| *** chandankumar has quit IRC | 13:34 | |
| *** ildikov has quit IRC | 13:35 | |
| *** dims has quit IRC | 13:36 | |
| *** devvesa has quit IRC | 13:36 | |
| *** devvesa has joined #openstack-infra | 13:36 | |
| *** markvoelker has joined #openstack-infra | 13:37 | |
| *** dims has joined #openstack-infra | 13:37 | |
| annegentle | maybe my understanding of pip is wrong... It seems like I should be able to install a local module using pip install -e /path/to/module (https://pip.pypa.io/en/latest/reference/pip_install.html#editable-installs) I did so, but I'm not picking up my changes to my sphinx theme when I build in the openstack-manuals directory... using pip freeze I see the git sha of the change I want to test locally... what am I missing | 13:37 |
| *** baoli_ has quit IRC | 13:37 | |
| annegentle | (sorry for the wall o' text) | 13:37 |
| *** baoli has joined #openstack-infra | 13:37 | |
| *** jgrimm is now known as zz_jgrimm | 13:37 | |
| *** amitgandhinz has joined #openstack-infra | 13:38 | |
| *** ianychoi has quit IRC | 13:38 | |
| fungi | not sure if there's something special about editable installs of sphinx themes | 13:38 |
| fungi | dhellmann: ^ do you happen to know? | 13:38 |
| sdague | annegentle: could it be a virtual env issue? | 13:38 |
| sdague | like the manual build is in a venv, so not picking that up? | 13:39 |
| *** eharney has joined #openstack-infra | 13:39 | |
| dhellmann | annegentle: did you rebuild the whole project, or just update? sphinx might not be rebuilding the output if it's not regenerating all of the pages | 13:39 |
| annegentle | sdague: oh could be, the manual build _is_ in a venv. Okay. | 13:39 |
| dhellmann | annegentle: remove the doc/build directory and build again and see if that does it | 13:39 |
| annegentle | dhellmann: this was with an sdist, trying to make sure the distribution has the change I want to test | 13:40 |
| *** tnovacik has joined #openstack-infra | 13:40 | |
| annegentle | I think it's the venv thing... for sure I'm switching venvs | 13:40 |
| dhellmann | ah, yeah, if you pip installed to somewhere other than where you're building that would do it | 13:40 |
| annegentle | but is pip freeze "lying" to me? :) | 13:41 |
| dhellmann | which pip are you running? | 13:41 |
| *** jckasper has joined #openstack-infra | 13:41 | |
| annegentle | pip 6.0.8 | 13:41 |
| annegentle | I see -e git+http://github.com/openstack/openstackdocstheme.git@606ec02d4a1b22cd5e4f05669eaa3756d5e4f8ed#egg=openstackdocstheme-gerrit_master in my 2nd venv | 13:41 |
| annegentle | for pip freeze results | 13:41 |
| *** yamamoto has joined #openstack-infra | 13:41 | |
| annegentle | and that's the git sha I want to test | 13:42 |
| *** mpavone has quit IRC | 13:42 | |
| *** ihrachyshka has quit IRC | 13:44 | |
| *** ihrachyshka has joined #openstack-infra | 13:44 | |
| *** tlbr has quit IRC | 13:45 | |
| annegentle | ok maybe it's my venv understanding that's limited... if I run tox -e venv python setup.py sdist how do I activate that venv? | 13:46 |
| dhellmann | annegentle: no, I mean are you running pip in the venv where you're also trying to run sphinx? | 13:46 |
| dhellmann | annegentle: if you want to install something into that venv, you can just run .tox/venv/bin/pip directly | 13:46 |
| annegentle | dhellmann: no I was running pip in a separate venv from what I created with tox -e venv python setup.py sdist | 13:46 |
| annegentle | dhellmann: ok, that's the part I was missing! | 13:46 |
| annegentle | doot doot doot | 13:47 |
| dhellmann | ah, ok, so pip is "smart" and looks at the packages where it is installed | 13:47 |
| *** adalbas has joined #openstack-infra | 13:47 | |
| annegentle | dhellmann: so do I "source .tox/venv/bin/pip " or is that a script itself? | 13:47 |
| mriedem_away | clarkb: i'll run a nova patch that depends-on https://review.openstack.org/#/c/170900/ to test it | 13:48 |
| *** tlbr has joined #openstack-infra | 13:48 | |
| dhellmann | annegentle: you don't have to "activate" a virtualenv to use it. The pip installed in .tox/venv/bin/pip is a copy of pip set up to use the python in that virtualenv automatically, so you can just run it as you would run pip normally | 13:48 |
| dhellmann | the activate script in a virtualenv is just for convenience, it puts the bin dir of the env first in your path so you don't have to type things out long-hand | 13:49 |
| *** mriedem_away is now known as mriedem | 13:49 | |
| *** otter768 has joined #openstack-infra | 13:49 | |
| *** dboik has joined #openstack-infra | 13:50 | |
| annegentle | dhellmann: ok... what does "run it as you would run pip normally" mean -- do I want that pip since it has my theme somehow? | 13:50 |
| dhellmann | annegentle: let's start at the beginning :-) what are you trying to do? | 13:50 |
| *** sarob has quit IRC | 13:50 | |
| *** chlong has quit IRC | 13:51 | |
| annegentle | dhellmann: heh. :) I have a new version of the openstackdocstheme | 13:51 |
| annegentle | dhellmann: I'm trying to test locally with content in openstack-manuals before releasing it | 13:51 |
| *** ildikov has joined #openstack-infra | 13:51 | |
| dhellmann | ok | 13:51 |
| dhellmann | and how do you build the manuals? | 13:51 |
| *** emagana has quit IRC | 13:51 | |
| annegentle | dhellmann: so my first thought was, make a distribution, make openstack-manuals use it. We have a tox -edocs build for the manuals. | 13:52 |
| dhellmann | yep, that'll work | 13:52 |
| dhellmann | so run: | 13:52 |
| dhellmann | to remove the existing build files: rm -rf doc/build | 13:52 |
| dhellmann | to build a new venv: tox -e docs -r | 13:53 |
| dhellmann | to install the theme: .tox/docs/bin/pip install path-to-the-theme-sdist | 13:53 |
| dhellmann | you might have to make that: .tox/docs/bin/pip install -U path-to-the-theme-sdist | 13:53 |
| annegentle | dhellmann: what's the -r in tox -e docs -r? Requirements.txt? | 13:53 |
| dhellmann | and then rebuild the docs with: tox -e docs | 13:53 |
| *** otter768 has quit IRC | 13:54 | |
| dhellmann | the -r arg to tox means "rebuild the virtualenv" | 13:54 |
| dhellmann | it removes it and recreates it | 13:54 |
| dhellmann | I included that to ensure that anything "bad" that had happened to your existing venv was cleaned up | 13:54 |
| annegentle | dhellmann: is the sdist the egg file? | 13:55 |
| dhellmann | annegentle: you know, I think I have a script that will make this much easier for you | 13:55 |
| dhellmann | stand by | 13:55 |
| dhellmann | sdist is "source distribution" and should be a .tar.gz file | 13:55 |
| annegentle | dhellmann: replace everything with a small shell script! LOL I need to understand the concepts too | 13:55 |
| annegentle | dhellmann: ok! that's helpful | 13:55 |
| dhellmann | annegentle: this is the script we use for running tests like this for oslo: http://git.openstack.org/cgit/openstack/oslotest/tree/tools/oslo_run_cross_tests | 13:56 |
| dhellmann | it has some comments to explain what is going on! :-) | 13:56 |
| annegentle | cool! thanks dhellmann | 13:56 |
| dhellmann | annegentle: run that from the directory where you have the *theme* and give it the path to the manuals repo and "docs" as args | 13:57 |
| *** prmtl has joined #openstack-infra | 13:57 | |
| annegentle | dhellmann: super efficient | 13:57 |
| *** prmtl has quit IRC | 13:57 | |
| *** e0ne is now known as e0ne_ | 13:58 | |
| *** e0ne_ is now known as e0ne | 13:58 | |
| *** wenlock has joined #openstack-infra | 13:58 | |
| *** jaypipes has joined #openstack-infra | 13:59 | |
| annegentle | dhellmann: success! | 13:59 |
| fungi | mrmartin: looks like a new commit on mockery's dev-master branch increased its minimum supported php such that the version on ubuntu precise no longer satisfies http://logs.openstack.org/99/165199/6/check/gate-openstackid-unittests/2dbde55/console.html | 13:59 |
| dhellmann | annegentle: \0/ | 13:59 |
| *** xyang0 has joined #openstack-infra | 13:59 | |
| *** bcourt has joined #openstack-infra | 13:59 | |
| fungi | mrmartin: how should openstackid go about pinning that? | 13:59 |
| *** marun has joined #openstack-infra | 14:00 | |
| *** mtanino has joined #openstack-infra | 14:00 | |
| dhellmann | annegentle: I'm going to be getting on a plane soon, but dims and bnemec in #openstack-oslo should be able to help with the script, or you can email me if you run into trouble and I'll reply when I land this evening | 14:00 |
| mrmartin | fungi: we should consider to upgrade the whole openstackid to trusty / php5.4 | 14:00 |
| *** bswartz has quit IRC | 14:01 | |
| *** ddieterly has joined #openstack-infra | 14:01 | |
| mrmartin | initially it was supported, we backported to precise, because didn't want to use external repos for php5.4 packages, and trusty was not released | 14:01 |
| fungi | mrmartin: well, pending an actual schedule to rebuild the production server, how should we go about getting its tests passing again? | 14:01 |
| *** bswartz has joined #openstack-infra | 14:02 | |
| *** yamahata has joined #openstack-infra | 14:02 | |
| *** carl_baldwin has joined #openstack-infra | 14:02 | |
| mrmartin | pin mockery to a previously working version | 14:02 |
| fungi | that's what i figured. i'll see if i can figure out how that works | 14:02 |
| mrmartin | https://github.com/openstack-infra/openstackid/blob/master/composer.json | 14:02 |
| mrmartin | don't use dev-master here | 14:02 |
| *** zz_dimtruck is now known as dimtruck | 14:03 | |
| mrmartin | need to change this single line, with a proper reference to a working version without this php5.4 limitation | 14:03 |
| fungi | yep, just looking for where these releases are listed so i can pick one | 14:04 |
| fungi | looks like they tagged a 0.9.4 version 5 days ago assuming it's https://github.com/padraic/mockery/tags | 14:05 |
| openstackgerrit | Darragh Bailey proposed stackforge/python-jenkins: Provider helper classes for plugin data management https://review.openstack.org/154316 | 14:06 |
| mrmartin | pin it to 0.9.3 | 14:06 |
| fungi | https://github.com/padraic/mockery/commit/56785d0 dropped php 5.3 support two days ago, so it's not part of the 0.9.4 tag | 14:06 |
| fungi | er, 5 days ago, but still it's after the tag | 14:06 |
| openstackgerrit | Paul Karikh proposed openstack-infra/project-config: Update jobs for Merlin https://review.openstack.org/171117 | 14:07 |
| mrmartin | anyway with php5.4/trusty have a packaging bug for php5-mcrypt which is required for openstackid: https://bugs.launchpad.net/ubuntu/+source/php-mcrypt/+bug/1318021 | 14:07 |
| openstack | Launchpad bug 1318021 in php-mcrypt (Ubuntu) "Not Find mcrypt.ini" [Undecided,Confirmed] | 14:07 |
| *** dkranz has joined #openstack-infra | 14:07 | |
| *** wenlock has quit IRC | 14:07 | |
| mrmartin | and it requires a lot of wizardry to workaround the bug which not looks nice in puppet | 14:07 |
| *** boris-42 has joined #openstack-infra | 14:08 | |
| *** btully has joined #openstack-infra | 14:08 | |
| fungi | anyway, i've proposed https://review.openstack.org/171212 to see if that works | 14:08 |
| fungi | i used 0.9.* on the assumption that the next tag will be a 0.10.x or 1.0.x or something | 14:09 |
| openstackgerrit | Merged openstack-infra/system-config: Revert "Add a helpful friend to gerrit reviews" https://review.openstack.org/169508 | 14:10 |
| fungi | since the author was wise enough to tag immediately prior to changing what php is supported | 14:10 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/shade: Add API method update_subnet() https://review.openstack.org/170516 | 14:11 |
| mrmartin | fungi: needs to pin to 0.9.4, 0.9.5 will fail, so 0.9.* won't work | 14:12 |
| fungi | mrmartin: i'm saying it looks like the author is sane enough not to tag a 0.9.5 from master | 14:12 |
| mrmartin | fungi: trust no one | 14:12 |
| mrmartin | it is safer to pin to 0.9.4 | 14:13 |
| fungi | <0.9.4 or are we asserting that this only works with 0.9.4 (not 0.9.3 or older and will probably not work with a 0.9.5 because the author is unlikely to branch that if he decides to tag one)? | 14:13 |
| *** vlaza has quit IRC | 14:14 | |
| fungi | er, <=0.9.4 | 14:14 |
| mrmartin | so, 0.9.4 | 14:14 |
| *** Guest54708 has quit IRC | 14:14 | |
| fungi | just noticing that all of the other requirements in that list which are setting versions assume semantic versioning | 14:16 |
| *** annegentle has quit IRC | 14:16 | |
| *** sigmavirus24_awa is now known as sigmavirus24 | 14:16 | |
| fungi | so not sure why we should assume the mockery authors won't do semantic versioning but the laravel/phpunit/zend/recaptcha/doctrine authors will | 14:17 |
| *** nelsnelson has joined #openstack-infra | 14:17 | |
| mrmartin | that's a nice dependency mess | 14:17 |
| *** mattfarina has joined #openstack-infra | 14:17 | |
| openstackgerrit | Timur Sufiev proposed openstack-infra/project-config: Update jobs for Merlin https://review.openstack.org/171117 | 14:17 |
| *** signed8bit has joined #openstack-infra | 14:19 | |
| *** signed8bit is now known as signed8bit_ZZZzz | 14:19 | |
| *** signed8bit_ZZZzz is now known as signed8bit | 14:19 | |
| *** wenlock has joined #openstack-infra | 14:20 | |
| *** unicell1 has joined #openstack-infra | 14:21 | |
| *** unicell has quit IRC | 14:21 | |
| *** Hal has joined #openstack-infra | 14:23 | |
| *** Hal is now known as Guest77014 | 14:23 | |
| *** zz_jgrimm is now known as jgrimm | 14:23 | |
| *** Sukhdev has joined #openstack-infra | 14:25 | |
| *** tonytan4ever has joined #openstack-infra | 14:26 | |
| *** dboik_ has joined #openstack-infra | 14:27 | |
| * mkj_wk returns from an out-of-the-country trip, pings for more reviews on https://review.openstack.org/#/c/158877/ to see if it can be merged... :) | 14:29 | |
| *** wenlock has quit IRC | 14:30 | |
| *** claudiub has quit IRC | 14:31 | |
| *** ddieterly has quit IRC | 14:31 | |
| *** ddieterly has joined #openstack-infra | 14:31 | |
| *** dboik has quit IRC | 14:32 | |
| *** nfedotov has joined #openstack-infra | 14:32 | |
| *** tsg_ has joined #openstack-infra | 14:33 | |
| *** erlon has joined #openstack-infra | 14:34 | |
| fungi | mrmartin: anyway, pinning that does seem to get the job passing again | 14:34 |
| mrmartin | great | 14:35 |
| *** _nadya_ has joined #openstack-infra | 14:35 | |
| *** vladiskuz has joined #openstack-infra | 14:35 | |
| openstackgerrit | Merged openstack-dev/cookiecutter: More portable to use setup.py test not testr https://review.openstack.org/170596 | 14:36 |
| *** carl_baldwin has quit IRC | 14:36 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/system-config: Add jenkins.openstack.org to the jenkins group https://review.openstack.org/171228 | 14:38 |
| *** devlaps has quit IRC | 14:38 | |
| *** wenlock has joined #openstack-infra | 14:39 | |
| vladiskuz | Hi everybody! I played with openstackid.org, and it does not respond to me now from my IP. Is it autoban ? | 14:40 |
| mrmartin | vladiskuz: :D | 14:41 |
| mrmartin | if you like to integrate openstackid with something I suggest to deploy a vagrant version | 14:41 |
| mordred | clarkb: ping when you're awake - I'd like to tie off on your comment on 164845 | 14:41 |
| mrmartin | localy, and play with that. | 14:41 |
| *** ddieterly has quit IRC | 14:41 | |
| *** ddieterl_ has joined #openstack-infra | 14:42 | |
| mordred | dstufft: btw - not for nothing - but if you think people shoudn't use your API, you should maybe name it with a leading _ ... | 14:43 |
| mordred | dstufft: cause while I hear what you're saying about it being private and non-stable - there is currently no way to know that without talking to you | 14:43 |
| *** zigzg has joined #openstack-infra | 14:43 | |
| vladiskuz | mrmartin: do you have vagrant script for deploy local server> | 14:43 |
| vladiskuz | mrmartin: ? | 14:44 |
| mrmartin | vladiskuz: sure, https://github.com/fremontlabs/vagrant-openstackid | 14:44 |
| *** hdd has joined #openstack-infra | 14:45 | |
| *** ddieterl_ has quit IRC | 14:45 | |
| *** ddieterly has joined #openstack-infra | 14:46 | |
| *** Guest77014 has quit IRC | 14:46 | |
| vladiskuz | mrmartin: Thank you very much! Do you know how I can unban on openstackid.org in future? | 14:46 |
| mrmartin | you can't, need to ask an admin. What you did anyway? | 14:46 |
| *** annegentle has joined #openstack-infra | 14:47 | |
| clarkb | mordred morning | 14:47 |
| *** ddieterly has quit IRC | 14:47 | |
| mrmartin | vladiskuz: I've removed your ip ban. | 14:48 |
| *** Hal has joined #openstack-infra | 14:49 | |
| *** Hal is now known as Guest81797 | 14:49 | |
| *** radez_g0n3 is now known as radez | 14:49 | |
| *** sarob has joined #openstack-infra | 14:50 | |
| vladiskuz | mrmartin: Thx!!! | 14:50 |
| openstackgerrit | Merged openstack-infra/system-config: Update ttx public SSH key https://review.openstack.org/170136 | 14:51 |
| vladiskuz | mrmartin: I want to use openid in Refstack https://github.com/stackforge/refstack. But documentation for openstackid is very дшееду | 14:51 |
| *** adalbas has quit IRC | 14:51 | |
| vladiskuz | mrmartin: * is very little | 14:52 |
| fungi | vladiskuz: have you seen https://review.openstack.org/165199 ? | 14:52 |
| mrmartin | vladiskuz: check this patch too: https://review.openstack.org/#/c/165199/ | 14:52 |
| *** adalbas has joined #openstack-infra | 14:52 | |
| fungi | vladiskuz: i'm currently trying to finish getting testing working there so it can merge and get published officially | 14:52 |
| vladiskuz | fungi: it will be cool! | 14:53 |
| fungi | vladiskuz: well, in the meantime, does that documentation help answer your questions? | 14:53 |
| openstackgerrit | Merged openstack-infra/system-config: Set $group at node scope for hiera lookups https://review.openstack.org/164842 | 14:54 |
| *** sarob has quit IRC | 14:54 | |
| vladiskuz | fungi: code will be more helpful than documentation I think | 14:54 |
| *** dboik_ has quit IRC | 14:54 | |
| *** wenlock_ has joined #openstack-infra | 14:54 | |
| *** dboik has joined #openstack-infra | 14:55 | |
| vladiskuz | fungi: Refstack will be the first project that use openstackid. Am I right? | 14:56 |
| fungi | vladiskuz: no, groups.openstack.org and www.openstack.org both use it already | 14:56 |
| fungi | vladiskuz: and ask.openstack.org will have it as an option soon, it sounds like | 14:56 |
| *** e0ne is now known as e0ne_ | 14:57 | |
| vladiskuz | fungi: groups.openstack.org used oauth I assume. Or openid? | 14:58 |
| *** annegentle has quit IRC | 14:58 | |
| fungi | vladiskuz: initially used openid but is using oauth to be able to make use of avatars/headshots | 14:59 |
| mordred | clarkb: your comment "Is there a more specific scope we can use here rather than a comment? Might be useful to explicitly use nodescope." doesn't make sense to me - can you say more words? | 14:59 |
| mrmartin | vladiskuz: if you don't like to use profile pictures, openid works perfectly | 14:59 |
| *** ddieterly has joined #openstack-infra | 15:00 | |
| vladiskuz | mrmartin: I thinking that OpenID for authentication and OAuth for authorization. Isn't it? | 15:01 |
| *** thingee has quit IRC | 15:01 | |
| clarkb | mordred: there is a comment that that says "no :: because group is set at nodescope" my suggestion was that instead of using a comment to imply what we want that we be explicit about it | 15:01 |
| *** sarob has joined #openstack-infra | 15:01 | |
| clarkb | mordred: but I think nibalizer said we cannot be explicit about this because puppet scopes are funny | 15:01 |
| mrmartin | vladiskuz: the Oauth part is not so flexible yet. | 15:01 |
| *** annegentle has joined #openstack-infra | 15:01 | |
| *** e0ne_ is now known as e0ne | 15:02 | |
| *** emagana has joined #openstack-infra | 15:02 | |
| mordred | clarkb: gotcha. I understand what your comment means now - and yes, I believe that nibalizer has indicated that as well | 15:03 |
| *** sarob has quit IRC | 15:03 | |
| *** Sukhdev has quit IRC | 15:04 | |
| *** dangers_away is now known as dangers | 15:06 | |
| *** annegent_ has joined #openstack-infra | 15:06 | |
| *** annegentle has quit IRC | 15:07 | |
| *** hodos|2 has joined #openstack-infra | 15:08 | |
| *** erikmwil_ has joined #openstack-infra | 15:08 | |
| *** erikmwilson is now known as Guest190 | 15:08 | |
| *** erikmwil_ is now known as erikmwilson | 15:08 | |
| *** erikmwilson_ has joined #openstack-infra | 15:08 | |
| *** sarob has joined #openstack-infra | 15:09 | |
| hashar | mordred: hello! I have just seen your Phabricator email. Teaser: Wikimedia will eventually migrate CodeReview to Phabricator as well and we believe we can hook it up with Zuul :) | 15:10 |
| hashar | mordred: I will reply to your email eventually :D | 15:10 |
| *** hichihara has quit IRC | 15:10 | |
| *** dhritishikhar_ has quit IRC | 15:11 | |
| mordred | hashar: yah - so, I'm explicitly kept that off the table for 2 reasons a) too complicated to think about all at one time b) gerrit is working fine for us right now | 15:11 |
| yolanda | mordred, jeblair, i did some work on nodepool benchmarks https://etherpad.openstack.org/p/Nodepool_benchmarks | 15:11 |
| mordred | hashar: however, I will be interested in watching how your zuul/phabricator work goes | 15:11 |
| hashar | mordred: yeah we are the same point right now. The migration from Gerrit to Phabricator review system is on hold pending engineering bandwith | 15:11 |
| mordred | hashar: turns out gerrit, java beast that it is, does its job pretty well | 15:12 |
| *** sarob has quit IRC | 15:12 | |
| hashar | mordred: there are some patches to Zuul by Joshua which would help adding more review system. There is even a patch to add support for GitHub pull requests :) | 15:12 |
| *** asrangne__ has quit IRC | 15:12 | |
| mordred | hashar: yes - I want the GH pull request patch to land very much | 15:12 |
| mordred | hashar: because I'd like to start running test in infra on PRs to upstream ansible | 15:12 |
| hashar | mordred: and yeah Gerrit backend is awesome. But it is end user experience sucks and we would like to tightly integrated review / patches and bugs reports | 15:12 |
| hashar | supporting GitHub pull requests should not be that much different from supporting Phabricator patches proposal. I guess it is "just" about sending some http notification with appropriate fields and teach zuul about them | 15:13 |
| hashar | not that trivial. But definitely doable | 15:13 |
| mordred | yup. :) | 15:13 |
| hashar | I just need to get Wikimedia to hiera a couple of james clones | 15:14 |
| hashar | s/hiera/hire/ | 15:14 |
| *** pcrews has joined #openstack-infra | 15:14 | |
| mordred | hehe. please tell me when you find such humans | 15:14 |
| mordred | I could do with a few more myself | 15:14 |
| hashar | yeah hiring is hard | 15:15 |
| hashar | though I am sure there are well hidden talents | 15:15 |
| *** sarob has joined #openstack-infra | 15:15 | |
| *** signed8bit is now known as signed8bit_ZZZzz | 15:15 | |
| *** dustins_ has joined #openstack-infra | 15:16 | |
| *** annegentle has joined #openstack-infra | 15:17 | |
| *** dannywilson has joined #openstack-infra | 15:17 | |
| *** annegent_ has quit IRC | 15:17 | |
| *** dhritishikhar_ has joined #openstack-infra | 15:18 | |
| *** armax has joined #openstack-infra | 15:18 | |
| *** signed8bit_ZZZzz is now known as signed8bit | 15:18 | |
| *** dustins has quit IRC | 15:18 | |
| *** hashar is now known as hasharCooking | 15:19 | |
| *** dannywilson has quit IRC | 15:19 | |
| *** duran has joined #openstack-infra | 15:19 | |
| *** dannywilson has joined #openstack-infra | 15:19 | |
| fungi | i actually have a genius developer (and circuit designer, firmware author, et cetera) whose language of choice is python and he's looking for a job now that the startup he was at has folded. no idea if anyone is interested in contact info | 15:20 |
| *** tsg_ has quit IRC | 15:20 | |
| jroll | fungi: would love to talk with him about working on ironic/onmetal | 15:21 |
| jroll | python things, open source things, hardware/firmware things | 15:22 |
| mordred | fungi: I would also like to talk to him about working on ironic, but not about onmetal | 15:22 |
| mordred | since I don't work at rackspace, so me hiring for onmetal would be weird | 15:22 |
| jroll | mordred: it's basically the same thing dev-wise :P | 15:22 |
| mordred | jroll: yup. although I'd imagine that the early-access to HP servers before they ship would be different in one place, and the access to a large deployment would be different in the other place | 15:23 |
| *** amitgandhinz has quit IRC | 15:23 | |
| jroll | mordred: early access to OCP gear could be fun too :) | 15:23 |
| * jroll digresses | 15:23 | |
| *** claudiub has joined #openstack-infra | 15:24 | |
| *** dhritishikhar_ has quit IRC | 15:24 | |
| *** dizquierdo has quit IRC | 15:24 | |
| *** ddieterly has quit IRC | 15:24 | |
| *** ddieterly has joined #openstack-infra | 15:24 | |
| fungi | jroll: mordred: i'll gauge his interest. he has no real experience with openstack other than hearing me prattle on, but does at least use free software pretty much exclusively and has contributed to a few random projects over the years | 15:24 |
| *** jlibosva has quit IRC | 15:25 | |
| *** ddieterly has quit IRC | 15:25 | |
| mordred | fungi: awesome | 15:25 |
| fungi | oh, and he's also only looking for remote/telework with maybe ~quarterly travel obligations | 15:25 |
| jroll | fungi: cool, please do let him know this wouldn't be only openstack :) thanks so much | 15:25 |
| jroll | yeah, remote is np | 15:26 |
| mordred | this is fun | 15:26 |
| jroll | heh | 15:26 |
| *** arxcruz has quit IRC | 15:27 | |
| mordred | fungi: let him know remote is no problem for me either - and that I have no interest in him working on things that are not openstack or free software | 15:27 |
| *** ajmiller_ has quit IRC | 15:27 | |
| jroll | mordred: wtb open firmware | 15:27 |
| *** dangers is now known as dangers_away | 15:27 | |
| mordred | nod. "open/free things" | 15:28 |
| *** krtaylor has quit IRC | 15:28 | |
| *** dangers_away has quit IRC | 15:28 | |
| *** tnovacik has quit IRC | 15:28 | |
| openstackgerrit | Deanna Hambleton proposed openstack-infra/jenkins-job-builder: Add support for Scoverage plugin https://review.openstack.org/171048 | 15:29 |
| *** mtanino has quit IRC | 15:29 | |
| *** dangers_away has joined #openstack-infra | 15:30 | |
| fungi | yeah, he was designing smart networked climate control systems | 15:31 |
| *** dhritishikhar_ has joined #openstack-infra | 15:31 | |
| fungi | saw some of the prototypes. awesome stuff | 15:32 |
| mordred | fungi: that would be superior to my dumb wall-mount heater | 15:32 |
| clarkb | I have a small networked heating device that I just installed as my home router :) | 15:33 |
| clarkb | turns out that fanless x86 machines get really hot | 15:33 |
| *** dangers_away is now known as dangers | 15:33 | |
| *** ajmiller has joined #openstack-infra | 15:33 | |
| *** amotoki has quit IRC | 15:34 | |
| clarkb | any chance I can get reviews for https://review.openstack.org/#/c/168408/6 to unstick multinode testing? | 15:34 |
| fungi | these things were arrays of sensors including gridded flow meters which tracked air movement inline inside of ducts and communicated over an rs422 chain | 15:35 |
| *** BharatK has quit IRC | 15:35 | |
| *** amitgandhinz has joined #openstack-infra | 15:36 | |
| fungi | i'm kinda sad that start-up went nowhere, because the tech they had was really amazing | 15:36 |
| *** Ala has quit IRC | 15:36 | |
| fungi | mrmartin: vladiskuz: here's the rendered draft of the openstackid docs now http://docs-draft.openstack.org/99/165199/7/check/gate-openstackid-docs/8797c5d//doc/build/html/ | 15:38 |
| mrmartin | awesome | 15:38 |
| *** sergsh has quit IRC | 15:39 | |
| vladiskuz | fungi: great! | 15:39 |
| *** devlaps has joined #openstack-infra | 15:40 | |
| pleia2 | I have returned \o/ | 15:40 |
| openstackgerrit | Darragh Bailey proposed openstack-infra/jenkins-job-builder: Use threads to update jobs in jenkins https://review.openstack.org/45371 | 15:41 |
| fungi | pleia2: so we can finally get this party started? | 15:41 |
| clarkb | mordred: I am going to approve 164845 now unless there is something that needs to be done with hiera files before hand | 15:41 |
| pleia2 | fungi: let's do it! | 15:41 |
| clarkb | mordred: I do not think we need to do anything because common is all we ever used | 15:41 |
| * pleia2 decides to catch up with email first | 15:41 | |
| ttx | pleia2: good choice. | 15:42 |
| fungi | pleia2: cinerama was shooting for being ready for you to launch the new zanata server as soon as you're caught up. let me know if you need any help and i'll be glad to assist | 15:42 |
| pleia2 | fungi: great news, thanks | 15:42 |
| clarkb | I think a couple puppet changes are ready for a second +2 and approval (assuming that second reviewer doesn't catch any issues I missed) | 15:43 |
| *** Longgeek has quit IRC | 15:45 | |
| *** funzo has joined #openstack-infra | 15:45 | |
| mordred | clarkb: I agree - I was going to land it earlier | 15:46 |
| clarkb | mordred: feel free to beat me to it | 15:47 |
| clarkb | I should make some breakfast | 15:47 |
| *** dtantsur is now known as dtantsur|afk | 15:48 | |
| *** ihrachyshka has quit IRC | 15:49 | |
| *** otter768 has joined #openstack-infra | 15:50 | |
| *** ddieterly has joined #openstack-infra | 15:52 | |
| *** gyee has joined #openstack-infra | 15:54 | |
| *** yamamoto has quit IRC | 15:54 | |
| *** otter768 has quit IRC | 15:55 | |
| clarkb | yolanda: around? curious if you considered using a single dictionary in 160885, and if not why? (maybe the stdlib merge function doesn't work recursively?) | 15:55 |
| yolanda | clarkb, following change done by notnownikki, but mostly for simplicity and isolation of content, i guess | 15:56 |
| *** hasharCooking is now known as hashar | 15:56 | |
| *** devvesa has quit IRC | 15:58 | |
| *** tonytan4ever has quit IRC | 15:58 | |
| clarkb | yolanda: ok, its fine as is, was mostly curious to know if a limitation of the tooling led to that | 15:58 |
| clarkb | I will try to apply that in a test run to the git backends today and approve it if it comes out clean | 15:59 |
| *** alexpilotti has quit IRC | 15:59 | |
| *** tiswanso has quit IRC | 15:59 | |
| *** emagana has quit IRC | 15:59 | |
| *** jogo has joined #openstack-infra | 16:00 | |
| yolanda | clarkb, actually looking at that stdlib merge, i found that https://projects.puppetlabs.com/issues/20200 | 16:00 |
| yolanda | but i didn't isolate due to that nesting really | 16:00 |
| *** ashleighfarnham has joined #openstack-infra | 16:00 | |
| *** yamamoto has joined #openstack-infra | 16:00 | |
| yolanda | also i see stdlib has a deep_merge function that does the merge recursively | 16:01 |
| *** dboik_ has joined #openstack-infra | 16:01 | |
| *** tiswanso has joined #openstack-infra | 16:01 | |
| clarkb | yolanda: deep merge shouldn't be required here due to the lack of nesting, but good to know we can do that if it does become necessary | 16:01 |
| *** jogo has quit IRC | 16:01 | |
| yolanda | yes, just i was curious as well | 16:02 |
| *** YorikSar has quit IRC | 16:02 | |
| yolanda | merging default and optional parameters should be a common pattern | 16:02 |
| *** ivar-lazzaro has joined #openstack-infra | 16:02 | |
| *** yamamoto has quit IRC | 16:02 | |
| *** emagana has joined #openstack-infra | 16:03 | |
| *** jogo has joined #openstack-infra | 16:04 | |
| *** dboik has quit IRC | 16:04 | |
| zaro | morning | 16:05 |
| openstackgerrit | Darragh Bailey proposed openstack-infra/jenkins-job-builder: Only query jenkins plugins if config provided https://review.openstack.org/158826 | 16:05 |
| *** bswartz has quit IRC | 16:06 | |
| *** ildikov has quit IRC | 16:06 | |
| *** SumitNaiksatam has quit IRC | 16:06 | |
| *** tsg_ has joined #openstack-infra | 16:07 | |
| *** jcoufal has quit IRC | 16:07 | |
| *** claudiub has quit IRC | 16:08 | |
| *** dhritishikhar_ has quit IRC | 16:09 | |
| *** alexpilotti has joined #openstack-infra | 16:10 | |
| *** dustins_ has quit IRC | 16:11 | |
| *** afazekas has quit IRC | 16:16 | |
| *** dhritishikhar_ has joined #openstack-infra | 16:17 | |
| *** signed8bit has quit IRC | 16:17 | |
| *** unicell1 has quit IRC | 16:18 | |
| *** tnovacik has joined #openstack-infra | 16:19 | |
| anteaya | pleia2: welomce back | 16:20 |
| * anteaya is on the train withe laggy wifi | 16:20 | |
| *** viktors has left #openstack-infra | 16:20 | |
| *** e0ne has quit IRC | 16:21 | |
| pleia2 | anteaya: thanks, enjoy the train ride (I always do!) | 16:21 |
| anteaya | I love it | 16:21 |
| anteaya | best way to travel | 16:21 |
| *** dhritishikhar_ has quit IRC | 16:22 | |
| *** alexpilotti_ has joined #openstack-infra | 16:24 | |
| *** soren has quit IRC | 16:24 | |
| *** alexpilotti has quit IRC | 16:24 | |
| *** alexpilotti_ is now known as alexpilotti | 16:24 | |
| *** flaviosr has joined #openstack-infra | 16:25 | |
| *** sputnik13 has joined #openstack-infra | 16:25 | |
| *** hashar is now known as hasharMeeting | 16:26 | |
| *** jistr has quit IRC | 16:27 | |
| *** signed8bit has joined #openstack-infra | 16:28 | |
| *** signed8bit is now known as signed8bit_ZZZzz | 16:28 | |
| openstackgerrit | Merged openstack-infra/system-config: Remove non-used ruby1_9_1 packages https://review.openstack.org/162124 | 16:30 |
| *** signed8bit_ZZZzz is now known as signed8bit | 16:30 | |
| *** koolhead17 has quit IRC | 16:31 | |
| cinerama | pleia2: hi there | 16:32 |
| *** dhritishikhar_ has joined #openstack-infra | 16:33 | |
| anteaya | she has no fodd in the house and went to get some breakfast | 16:36 |
| * SpamapS lurves fodd | 16:36 | |
| anteaya | a week away and all that | 16:36 |
| *** MaxV has quit IRC | 16:36 | |
| anteaya | fodd is godd | 16:36 |
| cinerama | anteaya, SpamapS :) | 16:36 |
| * SpamapS noms | 16:37 | |
| anteaya | laggy wifi, at a certain point you just keep typing and hit enter | 16:37 |
| SpamapS | anteaya: inorite? ;) | 16:37 |
| * SpamapS has abandoned all english | 16:37 | |
| anteaya | :) | 16:37 |
| anteaya | SpamapS: just keep hope | 16:37 |
| *** devlaps has quit IRC | 16:39 | |
| *** soren has joined #openstack-infra | 16:41 | |
| * SpamapS hums Billy Joel - Keepin The Faith | 16:43 | |
| openstackgerrit | Jeremy Stanley proposed openstack-infra/system-config: Add a centos-6 node label https://review.openstack.org/171286 | 16:44 |
| *** koolhead17 has joined #openstack-infra | 16:44 | |
| *** ddieterly has quit IRC | 16:45 | |
| *** ddieterly has joined #openstack-infra | 16:45 | |
| zaro | pelix: i don't understand your comment about tox here: https://review.openstack.org/#/c/75514/ | 16:45 |
| *** unicell has joined #openstack-infra | 16:46 | |
| openstackgerrit | Timur Sufiev proposed openstack-infra/project-config: Update jobs for Merlin https://review.openstack.org/171117 | 16:47 |
| *** markus_z has quit IRC | 16:47 | |
| *** dustins has joined #openstack-infra | 16:48 | |
| *** tkelsey has joined #openstack-infra | 16:49 | |
| *** tonytan4ever has joined #openstack-infra | 16:50 | |
| openstackgerrit | Jeremy Stanley proposed openstack-infra/bindep: Add codename and release platform profiles https://review.openstack.org/157979 | 16:51 |
| openstackgerrit | Jeremy Stanley proposed openstack-infra/bindep: Add platform support for Fedora https://review.openstack.org/171293 | 16:54 |
| *** krtaylor has joined #openstack-infra | 16:54 | |
| *** soren has quit IRC | 16:57 | |
| *** YorikSar has joined #openstack-infra | 16:58 | |
| openstackgerrit | Merged openstack-infra/bindep: Stop using intersphinx https://review.openstack.org/126158 | 16:59 |
| openstackgerrit | Herman Narkaytis proposed openstack-infra/project-config: Add Catalog project into Stackforge https://review.openstack.org/169671 | 16:59 |
| *** nfedotov has quit IRC | 17:01 | |
| openstackgerrit | Merged openstack/requirements: add what-broke.py analayzer https://review.openstack.org/166997 | 17:03 |
| *** patrickeast has joined #openstack-infra | 17:03 | |
| pelix | zaro: which one, first or second? | 17:03 |
| fungi | greghaynes: if you're around, did you see my question on https://review.openstack.org/165682 last week? | 17:04 |
| *** tsg_ has quit IRC | 17:04 | |
| * greghaynes is always around | 17:04 | |
| greghaynes | I did not | 17:04 |
| greghaynes | hahaa | 17:04 |
| fungi | funny, i pinged you in irc when i left that ;) | 17:04 |
| greghaynes | That is a really good comment | 17:04 |
| greghaynes | let me dig and see why | 17:05 |
| *** e0ne has joined #openstack-infra | 17:05 | |
| *** e0ne is now known as e0ne_ | 17:05 | |
| *** esker has joined #openstack-infra | 17:05 | |
| marcusvrn | fungi: I just cloned a fresh devstack and tried to setup it, but it stops on the same error (this morning): "AttributeError: 'NoneType' object has no attribute 'url'" | 17:06 |
| *** dboik_ has quit IRC | 17:06 | |
| *** dboik has joined #openstack-infra | 17:06 | |
| *** e0ne_ is now known as e0ne | 17:06 | |
| fungi | marcusvrn: the fix is in openstack/requirements not openstack-dev/devstack | 17:07 |
| *** dboik has quit IRC | 17:07 | |
| fungi | marcusvrn: make sure you're using the latest branch tips of the requirements repo | 17:07 |
| *** dboik has joined #openstack-infra | 17:07 | |
| *** sdake_ has quit IRC | 17:08 | |
| *** tiswanso has quit IRC | 17:08 | |
| marcusvrn | fungi: yep, but looking at the requirements repo (https://github.com/openstack/requirements/commits/master), the fix was not merged yet, was it? | 17:09 |
| marcusvrn | fungi: I think this one is the fix: "Change install_requires to use link instead of url" | 17:09 |
| fungi | marcusvrn: "Change install_requires to use link instead of url" is there | 17:09 |
| *** tiswanso_ has joined #openstack-infra | 17:09 | |
| *** yamamoto has joined #openstack-infra | 17:09 | |
| *** yamamoto has quit IRC | 17:10 | |
| marcusvrn | hm...it doesn't need to be merged? | 17:10 |
| fungi | what do you mean? | 17:10 |
| pleia2 | cinerama: hey, it's going to take some time for me to get caught up, want to give me the cliff notes version of zanata progress this past week? :) | 17:10 |
| * pleia2 noms on fruit | 17:10 | |
| marcusvrn | fungi: something like 'Merge "Change install_requires to use link instead of url"' commit | 17:11 |
| fungi | marcusvrn: if you're asking why there's no corresponding merge commit for that commit, it's because its parent was the previous commit in that branch, and gerrit's using its "merge when necessary" strategy | 17:11 |
| openstackgerrit | Darragh Bailey proposed openstack-infra/jenkins-job-builder: Initial work for supporting downstream-ext https://review.openstack.org/158117 | 17:11 |
| cinerama | pleia2: sure. if you have a look at the etherpad i've added my list of potential outstanding issues there. we still need to land a bunch of things, and there is at least one thing that we need to ask the zanata devs about | 17:12 |
| marcusvrn | fungi: hm... now it's clear! :) thanks | 17:12 |
| fungi | marcusvrn: so that commit was able to apply directly without needing an accompanying merge commit | 17:12 |
| greghaynes | fungi: I wonder if thats a base off master issue? compared to the parent patch I actually see that https://review.openstack.org/#/c/165682/ is strictly more coverage | 17:12 |
| cinerama | pleia2: i feel like what we've got under the topic is basically ready to be merged as is | 17:12 |
| *** tonytan4ever has quit IRC | 17:13 | |
| fungi | greghaynes: it tests against the branch tip, but the branch has moved since march 25 when that change was last tested, so i'll recheck and see if it improves at all | 17:13 |
| greghaynes | ok. I would be surprised if it is actually lest coverage - the only places where that should be true are the if "fake-" ... branches which I tried to remove | 17:14 |
| pleia2 | cinerama: starting at line 125? | 17:14 |
| greghaynes | but if it is true then that would be good to know :) | 17:14 |
| *** hdd has quit IRC | 17:14 | |
| *** vladiskuz|2 has joined #openstack-infra | 17:15 | |
| *** MaxV has joined #openstack-infra | 17:15 | |
| cinerama | pleia2: yup, that big block-o-stuff there | 17:15 |
| pleia2 | cinerama: great, I'll have a browse and note inline if I have any questions | 17:15 |
| *** hdd has joined #openstack-infra | 17:15 | |
| *** tosky has joined #openstack-infra | 17:16 | |
| *** yamamoto has joined #openstack-infra | 17:16 | |
| *** sergsh has joined #openstack-infra | 17:16 | |
| *** mrmartin has quit IRC | 17:16 | |
| cinerama | pleia2: so in the meeting yesterday we were going to get a server and you & i were going to learn how server is formed | 17:16 |
| krotscheck | Are colons allowed in job names? Example: https://review.openstack.org/#/c/170995/ | 17:17 |
| *** tqtran has joined #openstack-infra | 17:17 | |
| cinerama | pleia2: s/yesterday/last week/ | 17:17 |
| *** shardy has quit IRC | 17:17 | |
| vladiskuz|2 | mrmartin: I've been trying to install https://github.com/fremontlabs/vagrant-openstackid, but I have error https://etherpad.openstack.org/p/fail | 17:18 |
| pleia2 | cinerama: thanks, reviewing meeting logs is on my list too, I set up the backups server a few weeks back so I should be familiar enough with the basic process to get it done efficiently | 17:18 |
| *** dhritishikhar_ has quit IRC | 17:19 | |
| cinerama | pleia2: it might be okay if we do it inefficiently if that will help me learn the process :) | 17:19 |
| *** claudiub has joined #openstack-infra | 17:19 | |
| *** ddieterly has quit IRC | 17:20 | |
| *** ildikov has joined #openstack-infra | 17:22 | |
| *** dboik has quit IRC | 17:23 | |
| *** flaviosr has quit IRC | 17:24 | |
| *** spzala has joined #openstack-infra | 17:26 | |
| *** Sukhdev has joined #openstack-infra | 17:26 | |
| *** annegentle has quit IRC | 17:27 | |
| *** annegentle has joined #openstack-infra | 17:27 | |
| *** adalbas has quit IRC | 17:27 | |
| *** yamahata has quit IRC | 17:28 | |
| *** [HeOS] has quit IRC | 17:28 | |
| *** sigmavirus24 is now known as sigmavirus24_awa | 17:29 | |
| *** e0ne is now known as e0ne_ | 17:29 | |
| *** koolhead_ has joined #openstack-infra | 17:30 | |
| *** ddieterly has joined #openstack-infra | 17:30 | |
| *** harlowja_away is now known as harlowja | 17:31 | |
| *** annegent_ has joined #openstack-infra | 17:31 | |
| *** annegentle has quit IRC | 17:32 | |
| *** koolhead17 has quit IRC | 17:32 | |
| openstackgerrit | Herman Narkaytis proposed openstack-infra/project-config: Add Application Catalog project into Stackforge https://review.openstack.org/169671 | 17:34 |
| *** e0ne_ has quit IRC | 17:34 | |
| openstackgerrit | Timothy R. Chavez proposed openstack-dev/pbr: Sphinx >=1.3.1 init_values() requires an argument https://review.openstack.org/171309 | 17:36 |
| zaro | pelix: take a look at my comments https://review.openstack.org/#/c/75514/ | 17:36 |
| *** achanda has joined #openstack-infra | 17:36 | |
| zaro | pelix: ohh and the comment i don't understand "the tox change won't work by itself unless we change.." | 17:38 |
| *** dboik has joined #openstack-infra | 17:38 | |
| *** e0ne has joined #openstack-infra | 17:38 | |
| pelix | zaro: ah. yeah, the current behaviour of the code is to avoid the multi-threaded code when writing out the resulting XML files in test mode, so specifying the number or workers would have no effect | 17:39 |
| *** adalbas has joined #openstack-infra | 17:39 | |
| pelix | note, that the code generates and writes out the XML files if 'output' is not None, else it calls update_job(..) with the parallelize wrapper. | 17:40 |
| greghaynes | fungi: coverage report is in http://logs.openstack.org/82/165682/8/check/nodepool-coverage/6ecce38/cover/ | 17:41 |
| zaro | pelix: i don't see how that is related to tox? | 17:41 |
| openstackgerrit | Merged openstack-infra/bindep: Add CentOS RPM support https://review.openstack.org/157978 | 17:42 |
| zaro | pelix: and yes, i think i see that workers only applies to update that's why i asked to only put worker in update submenu. | 17:42 |
| fungi | greghaynes: looks like we're not doing post cover jobs on nodepool | 17:43 |
| *** koolhead17 has joined #openstack-infra | 17:43 | |
| *** ajmiller_ has joined #openstack-infra | 17:43 | |
| fungi | greghaynes: but i agree now with the recheck it's a 1% improvement over the one i linked in my earlier comment | 17:43 |
| greghaynes | yea | 17:43 |
| greghaynes | \O/ 1% | 17:43 |
| fungi | well, specifically it's taking nodepool/provider_manager from 67% to 71% and not touching the coverage for anything else | 17:44 |
| fungi | so for that one file it's a 4% improvement | 17:44 |
| greghaynes | yea, it also hit some bugs in there so I think its hitting some good parts :) | 17:45 |
| *** koolhead_ has quit IRC | 17:45 | |
| openstackgerrit | Merged openstack-infra/storyboard: Project Loader creates default branches. https://review.openstack.org/170942 | 17:46 |
| *** btully has quit IRC | 17:46 | |
| pelix | zaro: means that just running the test subcommand under tox with different worker argument values doesn't test anything. i.e. regarding you're comment " We could just execute that test twice, once with a single worker and again with multiple workers" | 17:46 |
| clarkb | fungi: 157979 reviewed | 17:47 |
| mescanef | hello, please add me (mmatuszkowiak@mirantis.com) to thee gerrits group members of stackforge/fuel-plugin-midonet (reference: https://review.openstack.org/#/c/171116/) | 17:47 |
| *** ajmiller has quit IRC | 17:47 | |
| zaro | pelix: it does seem wrong to apply workers on when no specifying '-o' flag. | 17:47 |
| *** teran has quit IRC | 17:48 | |
| zaro | pelix: ahh, your right. :( | 17:49 |
| pelix | would need to alter the code to use the threaded code when writing out separate XML files | 17:49 |
| pelix | but to use the single thread when writing to a pipe | 17:49 |
| *** _nadya_ has quit IRC | 17:50 | |
| zaro | that makes sense. | 17:50 |
| fungi | thanks clarkb | 17:50 |
| *** dprince has quit IRC | 17:51 | |
| *** otter768 has joined #openstack-infra | 17:51 | |
| zaro | pelix: anyways i believe i have found a regresssion there. | 17:51 |
| *** claudiub has quit IRC | 17:51 | |
| *** btully has joined #openstack-infra | 17:51 | |
| clarkb | fungi: oh derp I should open my eyes a bit more | 17:51 |
| zaro | pelix: do you use this parallel feature in your ci setup? | 17:51 |
| clarkb | fungi: I now see that the split is earlier than I expected. Sorry for the noise, will update my review | 17:51 |
| *** tonytan4ever has joined #openstack-infra | 17:52 | |
| fungi | clarkb: well, the lsb_release change is still worth doing | 17:52 |
| clarkb | fungi: and I think we want tests :) | 17:52 |
| openstackgerrit | Merged openstack-infra/nodepool: Monkeypatch Fake Clients for tests https://review.openstack.org/165682 | 17:53 |
| fungi | clarkb: that brings the fork down from 3 to 1 | 17:53 |
| clarkb | fungi: but I can at least note that the bit about needing to split on "." is my derp | 17:53 |
| clarkb | fungi: also note that the order of r,i,c flags to lsb_release does not change output order of those fields from what I can tell | 17:53 |
| *** dboik has quit IRC | 17:53 | |
| pelix | zaro: use the other patch instead at the moment, https://review.openstack.org/#/c/45371/ | 17:54 |
| pelix | be happy with either landing, but ya, need to find the source of that odd behaviour | 17:54 |
| clarkb | fungi: updated to note the split is already correct ut that we should have tests | 17:54 |
| tosky | sdague: can I ask for another reverify of https://review.openstack.org/#/c/169426/ now that the dependency is merged? - it seems that as a normal user I can't simply use reverify | 17:55 |
| *** bcourt has quit IRC | 17:55 | |
| clarkb | tosky: it needs to be reapproved, I can look at it for you | 17:55 |
| tosky | clarkb: thanks! | 17:55 |
| clarkb | tosky: this is necessary due to how gerrit doesn't write event values if the value has not changed | 17:55 |
| clarkb | which confuses zuul | 17:55 |
| tosky | clarkb: yeah, I thought that the Depends-On: is a bit of hack | 17:56 |
| *** otter768 has quit IRC | 17:56 | |
| clarkb | tosky: ok approved, it should make its way through the gate now | 17:56 |
| tosky | clarkb: thanks a lot | 17:56 |
| openstackgerrit | Timothy R. Chavez proposed openstack-dev/pbr: Sphinx >=1.3 init_values() requires an argument https://review.openstack.org/171309 | 17:56 |
| *** ivar-lazzaro has quit IRC | 17:58 | |
| clarkb | cinerama: question on 171000 but +2 regardless | 17:58 |
| *** dustins has quit IRC | 17:58 | |
| *** thingee has joined #openstack-infra | 17:58 | |
| *** cdent has quit IRC | 17:59 | |
| cinerama | clarkb: i thought it would be more user-friendly to let people specify the hostname rather than manipulating it themselves | 18:00 |
| *** dboik has joined #openstack-infra | 18:00 | |
| clarkb | cinerama: gotcha | 18:00 |
| *** tkelsey has quit IRC | 18:00 | |
| *** afazekas has joined #openstack-infra | 18:00 | |
| cinerama | clarkb: ideally we'd process the url | 18:01 |
| clarkb | that sounds unfun in puppet :) | 18:01 |
| *** jamielennox|away is now known as jamielennox | 18:01 | |
| *** yamamoto has quit IRC | 18:02 | |
| clarkb | woot the nodepool test monkey patch thing merged, I will need to rebase my changes after lunch | 18:03 |
| *** hdd has quit IRC | 18:03 | |
| clarkb | er lunch/meeting | 18:03 |
| fungi | who has a grasp of the nuances of the python coverage tool? openstack/heat-translator has metadata.name = heat translator in its setup.cfg but its source is all in a translator (not heat-translator) subdirectory. this seems to confuse coverage into thinking it has nothing to report | 18:03 |
| fungi | er, has metadata.name = heat-translator in its setup.cfg | 18:03 |
| *** dustins has joined #openstack-infra | 18:03 | |
| *** amitgandhinz has quit IRC | 18:03 | |
| *** mrmartin has joined #openstack-infra | 18:04 | |
| clarkb | I think you change that to translator | 18:04 |
| fungi | anyway, if i change setup.cfg to metadata.name = translator then coverage is able to find stuff | 18:04 |
| clarkb | yup | 18:04 |
| fungi | but that also changes the sdist name to translator | 18:04 |
| clarkb | oh I see, there should be a parameter to coverage you can set in setup.cfg that overrides just for coverage | 18:04 |
| clarkb | let me look that up | 18:04 |
| fungi | i was plowing through the coverage config docs but didn't see anything that looked like what i needed | 18:04 |
| fungi | still looking there | 18:05 |
| *** amitgandhinz has joined #openstack-infra | 18:05 | |
| clarkb | coverage_package_name | 18:05 |
| fungi | which is completely not documented at http://nedbatchelder.com/code/coverage/config.html#config | 18:06 |
| fungi | thanks! | 18:06 |
| clarkb | fungi: its not a coverage option | 18:06 |
| clarkb | fungi: its a pbr option | 18:06 |
| fungi | ahh... | 18:06 |
| fungi | so coverage_package_name goes in setup.cfg? in the metadata section? | 18:06 |
| clarkb | ya or onthe command line to setup.py test | 18:07 |
| *** SotK has quit IRC | 18:07 | |
| clarkb | fungi: also source = translation is what you want in coveragerc | 18:07 |
| zaro | pelix: haven't looked at your implmentation of parallel, what's the difference? | 18:07 |
| fungi | yeah, i found and fixed some other issues in .coveragerc where that was assumed from cookiecutter | 18:08 |
| *** johnthetubaguy is now known as zz_johnthetubagu | 18:08 | |
| *** yamahata has joined #openstack-infra | 18:08 | |
| *** bcourt has joined #openstack-infra | 18:08 | |
| pelix | zaro: just uses the multiprocessing threading library instead | 18:09 |
| pelix | of using threads and queues directly | 18:09 |
| *** dprince has joined #openstack-infra | 18:09 | |
| *** sdake has joined #openstack-infra | 18:10 | |
| clarkb | whats wrong with threads? | 18:10 |
| zaro | pelix: would you like me to try the same testing with your patch? | 18:10 |
| zaro | pelix: seems like using the multiprocessing lib is the better way to go | 18:11 |
| *** sdake_ has joined #openstack-infra | 18:11 | |
| asselin_ | clarkb, about the log server. what copyright headers should I add to the files? | 18:11 |
| asselin_ | clarkb, e.g. https://review.openstack.org/#/c/167425/7/manifests/logserver.pp | 18:12 |
| *** xyang0 has quit IRC | 18:12 | |
| *** MarkAtwood has joined #openstack-infra | 18:12 | |
| zaro | pelix: does it work for test command as well or just update? | 18:12 |
| yolanda | clarkb, about https://review.openstack.org/#/c/161695/ | 18:12 |
| yolanda | what parameter names are you talking about? i don't understand | 18:13 |
| pelix | zaro: only if the behaviour is more consistent, Ctrl+C is slower to react with the multiprocessing library since it has a separate management thread, so you end up having to catch and handle KeyboardInterrupt and then explicitly terminate all the threads otherwise it'll run to completion in the background | 18:13 |
| clarkb | asselin_: good question. Its probably sufficient to just use the apache header without copyright line? Otherwise you can git log -p openstack-infra/system-config/modules/static.pp filter out the changes just to the logserver stuff and add the appropriate copyright lines from there | 18:13 |
| clarkb | yolanda: the ones right above the comments. In some places its jenkins_gitemail and others its gitemail. They should be consistent | 18:13 |
| *** mattfarina has quit IRC | 18:13 | |
| openstackgerrit | Merged openstack-infra/project-config: Make saharaclient-dsvm-functional voting https://review.openstack.org/169426 | 18:14 |
| yolanda | clarkb, so i'm passing gitfullanem and gitemail to the jenkins::slave class https://git.openstack.org/cgit/openstack-infra/puppet-jenkins/tree/manifests/slave.pp | 18:14 |
| yolanda | but i thought it should be better if i just add it with jenkins_prefix to the openstack_project ones, so you can discriminate by other git parameters needed for other classes, if needed | 18:14 |
| clarkb | yolanda: ya so tahts the parameter to jenkins::slave so it doesn't need to change | 18:15 |
| *** sdake has quit IRC | 18:15 | |
| yolanda | do you prefer that i get rid of the jenkins_ prefix everywhere? | 18:15 |
| clarkb | yolanda: but the parameters to openstack-infra/system-config/modules/*.pp should be consistent | 18:15 |
| clarkb | yolanda: no I prefer to have the jenkins_ prefix everywhere in openstack-infra/system-config/modules/ | 18:15 |
| *** subscope_ has joined #openstack-infra | 18:16 | |
| yolanda | ah ok, i follow you , sorry | 18:16 |
| *** mattfarina has joined #openstack-infra | 18:16 | |
| *** sigmavirus24_awa is now known as sigmavirus24 | 18:17 | |
| *** sabeen has joined #openstack-infra | 18:18 | |
| pelix | zaro: the other implementation used to write out the XML files using threads, it got changed subsequently due to the changes in behaviour around supporting pipes for output. I think in either case we would have to alter the code to write out in test mode in parallel | 18:18 |
| fungi | clarkb: hrm... digging in the pbr source it looks like i need to do something along the lines of this, but it's not working: | 18:19 |
| fungi | commands = python setup.py testr --coverage --coverage-package-name=translator --testr-args='{posargs}' | 18:19 |
| fungi | error: option --coverage-package-name not recognized | 18:19 |
| fungi | hah, it's python setup.py test (not testr) if we want that to work | 18:20 |
| clarkb | fungi: that testr entrypoint may be served by testr | 18:20 |
| fungi | indeed it is! | 18:20 |
| clarkb | yes we should really be using test everywhere :) | 18:20 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/shade: Fix volume operations https://review.openstack.org/171319 | 18:21 |
| clarkb | fungi: as an alternative I think you can just set it in .coveragerc | 18:21 |
| zaro | pelix: yeah, i agree. | 18:21 |
| clarkb | fungi: whcih may be better so that any coverage run using testr or not works fine | 18:21 |
| openstackgerrit | yolanda.robla proposed openstack-infra/system-config: Expose jenkins_gitfullname and jenkins_gitemail https://review.openstack.org/161695 | 18:21 |
| fungi | i should propose switching testr to test in the cookiecutter tox.ini | 18:22 |
| fungi | but yeah | 18:22 |
| clarkb | yes we should fix cookiecutter. It is a bit annoying that I found a half dozen oddities with it and its our "standard" | 18:22 |
| clarkb | the multiprocessing import for example... | 18:22 |
| clarkb | zaro: pelix ^ one of the reasons why I ask about multiprocessing and why it may be necessary | 18:23 |
| clarkb | zaro: pelix but I don't have enough context about what you are talking about | 18:23 |
| *** afazekas has quit IRC | 18:24 | |
| fungi | clarkb: i think the problem with fixing it in .coveragerc is that's what the run.source option is for, but pbr is overriding what's in there unless you override its override via the command line | 18:24 |
| clarkb | fungi: really? | 18:24 |
| clarkb | oh I see ya | 18:24 |
| clarkb | boo :( | 18:24 |
| fungi | which is sort of annoying, but fixing would result in asking pbr to parse .coveragerc files | 18:24 |
| *** mtanino has joined #openstack-infra | 18:25 | |
| *** doude has quit IRC | 18:25 | |
| mordred | aroo? | 18:26 |
| fungi | oh, maybe we could just tweak pbr to not pass --source if coverage_package_name is None | 18:26 |
| fungi | i'll whip up a pbr patch for that too | 18:27 |
| fungi | no, wait, that won't work because then if you're relying on the package name in setup.cfg to determine the directory for coverage rather than setting source in .coveragerc, you're similarly screwed | 18:28 |
| fungi | mordred: http://git.openstack.org/cgit/openstack-dev/pbr/tree/pbr/testr_command.py#n125 | 18:28 |
| clarkb | fungi: mordred I think we should just force everyone to use .coveragerc | 18:29 |
| clarkb | its not a quick fix but it is most correct | 18:29 |
| fungi | mordred: in the case where metadata.name in your setup.cfg is not the same as run.source in your .coveragerc, pbr overrides coverage to what it finds in setup.cfg | 18:29 |
| *** hdd has joined #openstack-infra | 18:29 | |
| mordred | why not just teach pbr to read .coveragerc | 18:29 |
| mordred | and not override if it finds a value in .coveragerc ? | 18:29 |
| clarkb | mordred: because it doesn't need to | 18:29 |
| *** tosky has quit IRC | 18:29 | |
| *** _nadya_ has joined #openstack-infra | 18:29 | |
| clarkb | pbr shouldn't need to modify anything here, it just needs to run coverage instead of python | 18:30 |
| mordred | clarkb: I think you and I have different opinions on what pbr should do | 18:30 |
| clarkb | then .coveragerc should do the rest | 18:30 |
| mordred | but that assumes people have properly set up a .coveragerc in the first | 18:30 |
| mordred | place | 18:30 |
| mordred | which is one more place for people to make a mistake | 18:30 |
| clarkb | mordred: cookiecutter has basically shown me that we can't avoid that | 18:30 |
| asselin_ | clarkb, the log server was originally added by you back in 2012 94a7768dc3e58c833f087832e85c2c93e6241ae4 | 18:31 |
| clarkb | mordred: its full of wrong configs | 18:31 |
| clarkb | asselin_: that would be an HP copyright | 18:31 |
| mordred | so - if someone has a .coveragerc, we should assume that they know what they want | 18:31 |
| asselin_ | ok thanks | 18:31 |
| mordred | but if they _don't_ have a .coveragerc, then why not pass in a run.source on the command line? | 18:31 |
| *** _nadya_ has quit IRC | 18:31 | |
| mordred | we don't even have to teach it how to read it - we could just look for existence | 18:32 |
| clarkb | mordred: that I could live with | 18:32 |
| clarkb | its the teaching pbr how to read coverages config that bugs me | 18:32 |
| clarkb | but if its just a quick "does this exist" check thats fine | 18:32 |
| mordred | yah. I think that's all we need | 18:32 |
| fungi | so basically read it in as a safeconfigparser() file and look for run.source, then not pass a --source option to cover if it's set at all? | 18:34 |
| fungi | i can do that real fast | 18:34 |
| openstackgerrit | Ramy Asselin proposed openstack-infra/puppet-openstackci: Create a log server like http://logs.openstack.org https://review.openstack.org/167425 | 18:35 |
| dstufft | mordred: so I agree wrt leading underscore - main reason it's not is basically historical. Even though we don't support it we also try not to break things just because we can which I think has been the primary blocker to adding a leading underscope to everything | 18:36 |
| *** begal-sop__ has quit IRC | 18:36 | |
| *** mattfarina has quit IRC | 18:36 | |
| *** gulic has quit IRC | 18:36 | |
| pelix | clarkb: sorry, which part of the multiprocessing import are you looking for more context? | 18:37 |
| *** NobodyCam has quit IRC | 18:37 | |
| openstackgerrit | Ramy Asselin proposed openstack-infra/puppet-openstackci: Simple script to setup and test log server https://review.openstack.org/169117 | 18:38 |
| *** yamahata has quit IRC | 18:38 | |
| *** begal-sop__ has joined #openstack-infra | 18:38 | |
| clarkb | pelix: multiprocessing imports have to be worked around for some reason in setup.py (according to cookiecutter at least) so was curious to know what you plan to do with it | 18:39 |
| openstackgerrit | Merged openstack-infra/puppet-zanata: Add a zanata::client manifest https://review.openstack.org/167081 | 18:39 |
| *** NobodyCam has joined #openstack-infra | 18:39 | |
| pelix | odd, used for two things: 1) provides the cpu_count when you want to determine how many threads to put in a threadpool automatically based on cores | 18:40 |
| pelix | 2) it provides both multi-processing and multi-threaded libraries, with the same/similar API which makes switching between the two relatively trivial and it's multi-threading api has an inbuilt threadpool and manager thread. | 18:41 |
| clarkb | I think its the threadpooling that cookiecutter was working around by importing multiprocessing in setup.py. Its probably worth looking into | 18:42 |
| clarkb | https://git.openstack.org/cgit/openstack-dev/cookiecutter/tree/%7b%7bcookiecutter.repo_name%7d%7d/setup.py#n20 | 18:42 |
| clarkb | pelix: ^ | 18:42 |
| clarkb | since that is what I think pulls in the at_exit? | 18:42 |
| *** ajmiller has joined #openstack-infra | 18:44 | |
| *** ajmiller_ has quit IRC | 18:44 | |
| *** signed8bit is now known as signed8bit_ZZZzz | 18:44 | |
| *** tiswanso_ has quit IRC | 18:44 | |
| *** yamahata has joined #openstack-infra | 18:44 | |
| mordred | dstufft: nod | 18:45 |
| openstackgerrit | Julia Kreger proposed openstack-infra/shade: WIP: Update register_machine to use tasks https://review.openstack.org/171324 | 18:46 |
| openstackgerrit | Julia Kreger proposed openstack-infra/shade: WIP: Update unregister_machine to use tasks https://review.openstack.org/171325 | 18:46 |
| *** ibiris is now known as ibiris_away | 18:52 | |
| pelix | clarkb: that's annoying, difficult to determine if it's a problem with the multi threading or multi process API from the multiprocessing library | 18:53 |
| pelix | zaro: certainly seems like there is another reason to favour the direct use of Queue's in David Caro's patch if we can solve the regression. | 18:54 |
| *** [HeOS] has joined #openstack-infra | 18:54 | |
| *** signed8bit_ZZZzz is now known as signed8bit | 18:55 | |
| openstackgerrit | Julia Kreger proposed openstack-infra/shade: Change Ironic node lookups to support names https://review.openstack.org/171164 | 18:55 |
| openstackgerrit | Jeremy Stanley proposed openstack-dev/pbr: Don't override package when set in .coveragerc https://review.openstack.org/171331 | 18:55 |
| fungi | clarkb: mordred: something like ^ that? | 18:56 |
| *** tiswanso has joined #openstack-infra | 18:56 | |
| *** annegent_ has quit IRC | 18:56 | |
| *** annegentle has joined #openstack-infra | 18:57 | |
| *** gokrokve has joined #openstack-infra | 18:58 | |
| pleia2 | meeting time | 19:00 |
| fungi | thanks! | 19:00 |
| fungi | or it will be when keystone finishes anyway ;) | 19:00 |
| pleia2 | indeed :) | 19:00 |
| *** enikanorov has quit IRC | 19:01 | |
| stevemar | we're all done now ;) | 19:01 |
| *** hasharMeeting is now known as hashar | 19:01 | |
| *** _nadya_ has joined #openstack-infra | 19:02 | |
| openstackgerrit | Merged openstack-infra/puppet-zanata: Add OpenID login provider support to Zanata config https://review.openstack.org/166073 | 19:02 |
| *** doude has joined #openstack-infra | 19:03 | |
| openstackgerrit | Merged openstack-infra/project-config: Add puppet-diskimage_builder module https://review.openstack.org/167607 | 19:03 |
| *** bcourt has quit IRC | 19:04 | |
| *** xyang1 has joined #openstack-infra | 19:06 | |
| *** jistr has joined #openstack-infra | 19:07 | |
| openstackgerrit | Merged openstack-infra/puppet-zanata: Add zanata.ini file https://review.openstack.org/171000 | 19:08 |
| *** eharney has quit IRC | 19:08 | |
| *** _nadya_ has quit IRC | 19:10 | |
| *** annegent_ has joined #openstack-infra | 19:12 | |
| *** annegentle has quit IRC | 19:13 | |
| *** dboik has quit IRC | 19:14 | |
| openstackgerrit | David Shrewsbury proposed openstack-infra/shade: Update os-client-config min version https://review.openstack.org/171337 | 19:16 |
| *** annegent_ has quit IRC | 19:17 | |
| *** doude has quit IRC | 19:17 | |
| *** annegentle has joined #openstack-infra | 19:18 | |
| *** bcourt has joined #openstack-infra | 19:20 | |
| *** soren has joined #openstack-infra | 19:22 | |
| *** _nadya_ has joined #openstack-infra | 19:22 | |
| *** subscope_ has quit IRC | 19:22 | |
| *** openstackgerrit has quit IRC | 19:22 | |
| *** openstackgerrit has joined #openstack-infra | 19:23 | |
| *** achanda has quit IRC | 19:25 | |
| *** Sukhdev has quit IRC | 19:26 | |
| *** luqas has joined #openstack-infra | 19:26 | |
| *** sergsh has quit IRC | 19:27 | |
| *** bswartz has joined #openstack-infra | 19:31 | |
| *** luqas has quit IRC | 19:31 | |
| *** harlowja is now known as harlowja_away | 19:31 | |
| *** _nadya_ has quit IRC | 19:32 | |
| *** _nadya_ has joined #openstack-infra | 19:33 | |
| *** mrunge_bbl has quit IRC | 19:35 | |
| *** marcusvrn has quit IRC | 19:36 | |
| *** subscope_ has joined #openstack-infra | 19:37 | |
| *** achanda has joined #openstack-infra | 19:37 | |
| *** harlowja_away is now known as harlowja | 19:37 | |
| openstackgerrit | Timur Sufiev proposed openstack-infra/project-config: Update jobs for Merlin https://review.openstack.org/171117 | 19:38 |
| *** chandankumar has joined #openstack-infra | 19:41 | |
| *** tsg has joined #openstack-infra | 19:42 | |
| *** hodos|2 has quit IRC | 19:43 | |
| *** sarob has quit IRC | 19:45 | |
| *** tkelsey has joined #openstack-infra | 19:46 | |
| *** e0ne has quit IRC | 19:46 | |
| *** achanda has quit IRC | 19:47 | |
| *** akrivoka has joined #openstack-infra | 19:48 | |
| *** ddieterly has quit IRC | 19:48 | |
| akrivoka | any chance we could get a final +2 on this: https://review.openstack.org/#/c/169715/ | 19:49 |
| clarkb | akrivoka: that requires a gerrit downtime, so no not yet | 19:50 |
| *** tkelsey has quit IRC | 19:50 | |
| akrivoka | clarkb: oh really? I wasn't aware of that | 19:50 |
| *** otter768 has joined #openstack-infra | 19:52 | |
| *** dangers is now known as dangers_away | 19:53 | |
| pleia2 | akrivoka: yeah, that's what mordred meant by "rename window" in his comment (we do a batch of renames all at once, since gerrit downtime) | 19:54 |
| openstackgerrit | Spencer Krum proposed openstack-infra/puppet-puppet: Add .gitreview https://review.openstack.org/171344 | 19:54 |
| *** ddieterly has joined #openstack-infra | 19:54 | |
| *** _nadya_ has quit IRC | 19:54 | |
| openstackgerrit | Spencer Krum proposed openstack-infra/puppet-puppet: Module is puppet, not simpleproxy https://review.openstack.org/171345 | 19:54 |
| *** jistr has quit IRC | 19:55 | |
| openstackgerrit | Merged openstack-infra/nodepool: Add support for requsting that config drive be used. https://review.openstack.org/155770 | 19:55 |
| akrivoka | pleia2: I see, thanks for the clarification | 19:56 |
| *** otter768 has quit IRC | 19:56 | |
| mordred | clarkb: your dib-nodepool changes have test fail unhappy (I tried to land them) | 19:56 |
| akrivoka | pleia2: and how often are these renames happening? | 19:56 |
| *** melwitt has joined #openstack-infra | 19:57 | |
| clarkb | mordred: yes, greghaynes' fake change in nodepool made them fail | 19:57 |
| clarkb | mordred: I need to rebase to make it all happy | 19:57 |
| clarkb | mordred: will work on that after the meeting | 19:57 |
| pleia2 | akrivoka: it depends, "every few weeks" is about as specific as we can get | 19:57 |
| akrivoka | alright, thanks for the info pleia2 and clarkb | 19:57 |
| greghaynes | clarkb: actually, I think it legit is not jsut a rebase | 19:58 |
| greghaynes | clarkb: I can fix though if you want | 19:58 |
| clarkb | greghaynes: well its legit in that rebase needs edits | 19:58 |
| clarkb | greghaynes: I can get them | 19:58 |
| greghaynes | ok | 19:58 |
| openstackgerrit | Merged openstack-infra/system-config: Document openstackid deployment https://review.openstack.org/170974 | 19:58 |
| *** eharney has joined #openstack-infra | 19:58 | |
| *** dprince has quit IRC | 19:59 | |
| *** annegent_ has joined #openstack-infra | 19:59 | |
| yolanda | here, so no time, but wanted to expose that https://etherpad.openstack.org/p/Nodepool_benchmarks | 20:00 |
| yolanda | jeblair not here, but wanted to share with others as well | 20:00 |
| *** gmurphy has joined #openstack-infra | 20:01 | |
| *** doug-fish has quit IRC | 20:01 | |
| *** tsg has quit IRC | 20:01 | |
| fungi | pleia2: okay, so that was translate-dev you wanted deleted from dns, yeah? | 20:02 |
| sdague | fungi: can we ninja merge 171157, it bounced off the gate, and is a bunch of hours behind now, and locally the old fix is still breaking somewhere for me | 20:02 |
| *** Shrews has quit IRC | 20:02 | |
| yolanda | it's mostly to improve nodepool performance, by having independent task managers, mordred, clarkb, i'd like if you can take a look | 20:02 |
| *** tiswanso has quit IRC | 20:02 | |
| *** annegentle has quit IRC | 20:02 | |
| *** tiswanso has joined #openstack-infra | 20:03 | |
| *** Shrews has joined #openstack-infra | 20:03 | |
| *** doug-fish has joined #openstack-infra | 20:03 | |
| fungi | sdague: it's not urgent at this point other than the minimal risk that there will be a new pip release in the next few hours which changes behavior we're using in that script again, right? | 20:04 |
| sdague | fungi: so, honestly, maybe it's a mirroring problem? | 20:05 |
| openstackgerrit | Stephanie Miller proposed openstack-infra/puppet-zanata: Add apache configuration for zanata https://review.openstack.org/164011 | 20:05 |
| *** tsg has joined #openstack-infra | 20:05 | |
| sdague | I can see if this is 100% reproducable, but I was kind of blown away that it was nuking again like this | 20:05 |
| fungi | sdague: what's a mirroring problem? | 20:05 |
| sdague | getting an older pip | 20:05 |
| fungi | sdague: maybe. are we still getting failures on the url parameter thing? | 20:06 |
| sdague | I saw it locally | 20:06 |
| sdague | I don't know about in the system | 20:06 |
| *** chandankumar has quit IRC | 20:06 | |
| *** Rockyg has joined #openstack-infra | 20:06 | |
| *** samueldmq has quit IRC | 20:07 | |
| fungi | oh, locally. were you running with a stale requirements commit? | 20:07 |
| clarkb | greghaynes: I get to freshen up on getopt :) | 20:07 |
| greghaynes | clarkb: the joys of dib :) | 20:08 |
| *** ayoung has quit IRC | 20:08 | |
| sdague | no, it looks right | 20:09 |
| sdague | it's grenade testing, I'm deleting /opt/stack every go | 20:09 |
| *** ssam2 has quit IRC | 20:09 | |
| openstackgerrit | Matthew Treinish proposed openstack-infra/project-config: Ensure we run the full tempest job serially without admin https://review.openstack.org/171351 | 20:10 |
| fungi | that's... strange. but do you have a devstack-gate git cache | 20:10 |
| openstackgerrit | Stephanie Miller proposed openstack-infra/puppet-zanata: Add SMTP configuration options for Zanata https://review.openstack.org/168167 | 20:10 |
| fungi | devstack-gate will repopulate your devstack git-cache from /opt/git if it finds repos there | 20:11 |
| cody-somerville | jeblair: mordred: Do you guys use just trove or do you manually have dbs and if the latter how did you do backups? | 20:12 |
| sdague | fungi: I don't, I was about to set it up though | 20:12 |
| fungi | got it. was just wondering if you had an outdated /opt/git/openstack/requirements clone or something | 20:13 |
| sdague | yeh, I checked, didn't | 20:13 |
| fungi | i mean, you shouldn't be seeing that bug with latest requirements from any branch. the workaround merged including all of its cherry-picks | 20:13 |
| fungi | and the 171157 fix is to the same repo, so if you're getting an old enough requirements to not have the workaround from earlier then you won't get getting that fix when it merges either | 20:14 |
| fungi | er, won't be getting | 20:14 |
| nibalizer | yolanda: 171113 makes me so happy :) | 20:15 |
| *** tqtran_ has joined #openstack-infra | 20:15 | |
| sdague | yeh, I'm confused, I'll try to hack around it | 20:15 |
| *** signed8bit is now known as signed8bit_ZZZzz | 20:15 | |
| *** teran has joined #openstack-infra | 20:16 | |
| openstackgerrit | Clark Boylan proposed openstack-infra/nodepool: Use correct diskimage when building diskimages https://review.openstack.org/169474 | 20:16 |
| openstackgerrit | Clark Boylan proposed openstack-infra/nodepool: Support per provider image build settings https://review.openstack.org/126747 | 20:16 |
| openstackgerrit | Clark Boylan proposed openstack-infra/nodepool: Just set qcow2 version https://review.openstack.org/169383 | 20:16 |
| clarkb | greghaynes: ^ I think I got that right | 20:16 |
| *** signed8bit_ZZZzz is now known as signed8bit | 20:16 | |
| *** tqtran has quit IRC | 20:17 | |
| clarkb | cody-somerville: we use trove and puppet created DBs both | 20:17 |
| mordred | cody-somerville: both - depending on the db | 20:17 |
| mordred | cody-somerville: for backups for both of them, use mysqldump | 20:17 |
| mordred | cody-somerville: and dump to a filesystem that will get backed up by bup | 20:17 |
| clarkb | cody-somerville: in both cases we have a daily cron that mysqldumps the database onto the server using the DB. These dumps are logrotated and bup copies them offsite | 20:17 |
| mordred | :) | 20:18 |
| mordred | yay! clarkb and I agree | 20:18 |
| *** dustins has quit IRC | 20:18 | |
| fungi | pleia2: translate-dev was 50.56.185.32 and 2001:4800:780e:510:3bc3:d7f6:ff00:c2ca for the record, now deleted | 20:19 |
| pleia2 | woo, we know how our infra works \o/ | 20:19 |
| pleia2 | fungi: thank you | 20:19 |
| fungi | mrmartin: ask.openstack.org ttls have been raised back to an hour now | 20:19 |
| mrmartin | fungi: ack. | 20:20 |
| *** vladiskuz|2 has quit IRC | 20:20 | |
| cody-somerville | mordred: clarkb: What about cinder snapshots? | 20:20 |
| *** tnovacik has quit IRC | 20:20 | |
| pleia2 | clarkb: "Life of an Event in Logstash" video & slides: https://www.elastic.co/elasticon/2015/sf/life-of-an-event-in-logstash | 20:20 |
| mordred | cody-somerville: we back up cross-cloud | 20:20 |
| clarkb | we do not do cinder snapshots | 20:20 |
| *** subscop__ has joined #openstack-infra | 20:20 | |
| pleia2 | ^^ and anyone else who is interested, it's good | 20:20 |
| clarkb | mordred: well cross region but yes | 20:20 |
| clarkb | pleia2: thanks | 20:21 |
| mordred | cody-somerville: cinder snapshots have had the result of pausing a server for a long time, btw | 20:21 |
| * pleia2 seeks lunch | 20:21 | |
| *** AJaeger has joined #openstack-infra | 20:21 | |
| *** AJaeger has quit IRC | 20:22 | |
| greghaynes | clarkb: replied on https://review.openstack.org/#/c/126747/ - sorry for the funky dib behavior there | 20:22 |
| clarkb | greghaynes: we do handle the lack of -t with the outtypes initialized value | 20:23 |
| *** subscope_ has quit IRC | 20:23 | |
| clarkb | greghaynes: and nodepool will only ever write -o without a suffix (and this should test that if we don't) | 20:23 |
| greghaynes | ah, right, we need to handle the no extension specified though | 20:23 |
| *** ayoung has joined #openstack-infra | 20:24 | |
| greghaynes | oh, that changed too? | 20:24 |
| greghaynes | ok, nvm then | 20:24 |
| clarkb | greghaynes: ya in that change | 20:24 |
| clarkb | greghaynes: prior to that change it was always adding the suffix iirc | 20:24 |
| greghaynes | I think prior it was always not adding | 20:24 |
| clarkb | oh maybe not, in any case now it never adds it | 20:24 |
| clarkb | and relies on -t to set it | 20:24 |
| greghaynes | ok, SGTM | 20:24 |
| *** subscop__ has quit IRC | 20:25 | |
| *** adalbas has quit IRC | 20:25 | |
| fungi | clarkb: eventually cross-cloud, once we get back to finishing the hpcloud backup server | 20:28 |
| *** achanda has joined #openstack-infra | 20:29 | |
| *** hodos has joined #openstack-infra | 20:29 | |
| *** tsg_ has joined #openstack-infra | 20:29 | |
| clarkb | mordred: if 126747 and its children come back as jenkins +1 then they should be ready for merging now | 20:30 |
| mordred | clarkb: woot | 20:30 |
| mordred | clarkb: I look forward to merging our new multi-format overlords | 20:30 |
| clarkb | I also squashed the change that added a test for 126747 into 126747 since I was editing it anyways | 20:30 |
| mordred | clarkb: once they merge, I'll make sure my patches still work on top of them | 20:30 |
| *** hodos has left #openstack-infra | 20:30 | |
| *** tsg has quit IRC | 20:30 | |
| *** hodos has joined #openstack-infra | 20:30 | |
| *** tgohad has joined #openstack-infra | 20:31 | |
| *** melwitt has quit IRC | 20:31 | |
| *** dboik has joined #openstack-infra | 20:32 | |
| *** melwitt has joined #openstack-infra | 20:32 | |
| *** tnovacik has joined #openstack-infra | 20:33 | |
| *** tsg_ has quit IRC | 20:33 | |
| *** alex_xu has quit IRC | 20:34 | |
| *** heyongli has quit IRC | 20:36 | |
| *** tonytan4ever has quit IRC | 20:37 | |
| clarkb | hrm my nodepool change doesn't seem to be returning from testing | 20:37 |
| *** bcourt has quit IRC | 20:37 | |
| clarkb | no errors yet in the console log but it should've completed already. I will have to look into that after lunch | 20:38 |
| openstackgerrit | Merged openstack-infra/elastic-recheck: Add elastic recheck for bug 1440984 https://review.openstack.org/171056 | 20:38 |
| openstack | bug 1440984 in OpenStack-Gate "AttributeError: 'InstallRequirement' when running update.py" [Undecided,New] https://launchpad.net/bugs/1440984 | 20:38 |
| *** koolhead17 has quit IRC | 20:40 | |
| openstackgerrit | Spencer Krum proposed openstack-infra/system-config: Refactor o_p::template into o_p::server https://review.openstack.org/171359 | 20:40 |
| *** claudiub has joined #openstack-infra | 20:41 | |
| zigzg | Hi everyone! I'm Deanna Hambleton, and I'm interested in participating in OpenDaylight's Jenkins Job Builder project for a remote internship sponsored through the Outreachy program. This program requires me to submit an initial contribution to the project in order for me to apply - I managed to submit a small patch last night, and it'd mean a lot to me if any committers could review it and give me feedback. | 20:41 |
| zigzg | Since the written application deadline was today (I already submitted it), it'd be great if my patch could be reviewed as soon as possible (hopefully this week), so that my full application can be completed. I'm so sorry to be asking for this on such short notice (I know the review process normally takes a few weeks on average), but I'm very interested in working on this project for Outreachy, so I'd love to get my contribution | 20:41 |
| *** akrivoka has quit IRC | 20:41 | |
| zigzg | Here's the link for my patch: https://review.openstack.org/#/c/171048/ Thank you all so much! :) | 20:41 |
| *** dustins has joined #openstack-infra | 20:42 | |
| openstackgerrit | Joe Gordon proposed openstack-infra/devstack-gate: Refactor cleanup code to improve aiopcpu logs https://review.openstack.org/169919 | 20:42 |
| *** kgiusti has left #openstack-infra | 20:44 | |
| *** tonytan4ever has joined #openstack-infra | 20:45 | |
| clarkb | zigzg: neat, one question for you inline. But otherwise it looks good to me | 20:49 |
| jogo | greghaynes: how is the pypi mirror coming along? | 20:50 |
| fungi | ianw: i assume you meant to ping me in here, and not in the tc meeting? ;) | 20:50 |
| greghaynes | jogo: got 1 +2, need one more | 20:50 |
| jogo | greghaynes: woot | 20:50 |
| *** melwitt has quit IRC | 20:50 | |
| *** melwitt has joined #openstack-infra | 20:51 | |
| ianw | fungi: doh | 20:51 |
| greghaynes | https://review.openstack.org/#/q/status:open+branch:master+topic:feature/wheel-mirror,n,z | 20:51 |
| zigzg | clarkb: Thank you! I'll go look at it right now | 20:51 |
| ianw | fungi: yeah, anyway, i can not get the glusterfs job to run out of memory on a host i'm setting up. the only extra thing i can think is running on nodepool hosts is the jenkins stuff | 20:52 |
| fungi | ianw: sure, i'll hold you one momentarily | 20:53 |
| *** ivar-lazzaro has joined #openstack-infra | 20:54 | |
| *** pc_m has quit IRC | 20:55 | |
| openstackgerrit | Joe Gordon proposed openstack-infra/devstack-gate: Reflect the fact that devstack takes 10-15 minutes now https://review.openstack.org/171367 | 20:56 |
| *** esker has quit IRC | 20:56 | |
| fungi | ianw: ssh jenkins@104.239.166.75 | 20:57 |
| *** sdake_ has quit IRC | 20:57 | |
| *** hashar has quit IRC | 20:57 | |
| jogo | sdague: ^ | 20:58 |
| jogo | sdague: not sure if 20 is the right number, but don't think 15 is either | 20:58 |
| sdague | jogo: it should be except for terrible hpcloud nodes | 20:59 |
| sdague | at least once the wheel cache is in nodepool | 20:59 |
| openstackgerrit | Spencer Krum proposed openstack-infra/system-config: Pull o_p::server invocation out of afs roles https://review.openstack.org/171369 | 20:59 |
| jogo | sdague: the 15 limit you mean? | 21:00 |
| jogo | sdague: yeah this changes once the wheel stuff is in place | 21:00 |
| jogo | sdague: https://review.openstack.org/#/c/170608/2 should help too | 21:02 |
| openstackgerrit | Spencer Krum proposed openstack-infra/system-config: Refactor o_p::template into o_p::server https://review.openstack.org/171359 | 21:02 |
| jogo | haven't heard back from SergeyLukjanov about doing the same for sahara yet | 21:02 |
| *** pelix has quit IRC | 21:03 | |
| openstackgerrit | Spencer Krum proposed openstack-infra/system-config: Capitalize references to resoruces https://review.openstack.org/171370 | 21:04 |
| *** Krinkle is now known as Krinkle|detached | 21:04 | |
| jogo | sdague: want to review the 2 dependencies https://review.openstack.org/#/c/170605/1 ? | 21:04 |
| openstackgerrit | Stephanie Miller proposed openstack-infra/system-config: Add infra setup for zanata https://review.openstack.org/169074 | 21:04 |
| *** dustins has quit IRC | 21:04 | |
| openstackgerrit | Merged openstack-infra/project-config: Add puppet-pgsql_backup as a separate puppet module https://review.openstack.org/167631 | 21:06 |
| *** ociuhandu has quit IRC | 21:06 | |
| *** gokrokve has quit IRC | 21:07 | |
| *** mattfarina has joined #openstack-infra | 21:08 | |
| *** fifieldt has quit IRC | 21:08 | |
| *** radez is now known as radez_g0n3 | 21:08 | |
| openstackgerrit | Merged openstack-infra/system-config: Add a clouds.yaml file in anticipation of shade https://review.openstack.org/168601 | 21:09 |
| *** aysyd has quit IRC | 21:09 | |
| openstackgerrit | Merged openstack-infra/system-config: Document the vhd-util ppa https://review.openstack.org/169075 | 21:09 |
| openstackgerrit | Clark Boylan proposed openstack-infra/nodepool: Use correct diskimage when building diskimages https://review.openstack.org/169474 | 21:11 |
| openstackgerrit | Clark Boylan proposed openstack-infra/nodepool: Support per provider image build settings https://review.openstack.org/126747 | 21:11 |
| openstackgerrit | Clark Boylan proposed openstack-infra/nodepool: Just set qcow2 version https://review.openstack.org/169383 | 21:11 |
| clarkb | mordred: greghaynes ^ that missing : was important to getopt but should work now | 21:11 |
| openstackgerrit | Stephanie Miller proposed openstack-infra/system-config: Add infra setup for zanata https://review.openstack.org/169074 | 21:11 |
| greghaynes | clarkb: oh, derp, right | 21:11 |
| jeblair | fungi: i caught up on the meeting log, thanks | 21:12 |
| *** annegent_ has quit IRC | 21:13 | |
| fungi | jeblair: no worries. have a good conferencing | 21:13 |
| *** pblaho has quit IRC | 21:13 | |
| krotscheck | Is it permitted to have a ':' in a job name? (https://review.openstack.org/#/c/170995/1/zuul/layout.yaml) | 21:14 |
| *** ldnunes has quit IRC | 21:14 | |
| jeblair | krotscheck: no | 21:14 |
| krotscheck | jeblair: Thanks. Will rename relevant commands. | 21:15 |
| jeblair | krotscheck: zuul uses ':' as part of the zuul-gearman protocol when talking to jenkins | 21:15 |
| *** tiswanso has quit IRC | 21:15 | |
| *** ayoung has quit IRC | 21:16 | |
| *** fifieldt has joined #openstack-infra | 21:21 | |
| *** mrda-away is now known as mrda | 21:21 | |
| greghaynes | clarkb: oh, fail again - are we now passing in -x? | 21:22 |
| greghaynes | clarkb: I think if we are we should at least document that in the commit msg | 21:22 |
| *** rfolco has quit IRC | 21:22 | |
| *** signed8bit has quit IRC | 21:23 | |
| clarkb | greghaynes: I thought we always did | 21:23 |
| greghaynes | oh, huh, thats not changed behavior | 21:23 |
| greghaynes | yea | 21:23 |
| clarkb | greghaynes: but something may have pulled it out | 21:23 |
| clarkb | anyways I will fix | 21:23 |
| openstackgerrit | Merged openstack-infra/project-config: Make tempest-dsvm-heat job voting https://review.openstack.org/170604 | 21:24 |
| clarkb | greghaynes: I think getopt fails on that but doesn't matter typically | 21:24 |
| clarkb | greghaynes: however the -t arg to fake-image-create isn't quite right os it breaks | 21:24 |
| *** mrmartin has quit IRC | 21:24 | |
| greghaynes | clarkb: thats wierd | 21:25 |
| openstackgerrit | Merged openstack-infra/project-config: Remove integrated-gate template from heat https://review.openstack.org/170605 | 21:25 |
| clarkb | greghaynes: nah, its tests doing their job. Thank you for updating this :) | 21:25 |
| *** chlong has joined #openstack-infra | 21:25 | |
| clarkb | the old fakes didn't catch it | 21:26 |
| greghaynes | yea, I just dont get how its not currently failing | 21:26 |
| openstackgerrit | James E. Blair proposed stackforge/gertty: Make project name a search link in change screen https://review.openstack.org/171376 | 21:26 |
| openstackgerrit | James E. Blair proposed stackforge/gertty: Make topic a search link in change screen https://review.openstack.org/171377 | 21:26 |
| openstackgerrit | James E. Blair proposed stackforge/gertty: Make change-id a search link in change screen https://review.openstack.org/171378 | 21:26 |
| openstackgerrit | James E. Blair proposed stackforge/gertty: Do not clear history when opening a dashboard https://review.openstack.org/171379 | 21:26 |
| openstackgerrit | James E. Blair proposed stackforge/gertty: Add a key to return to the project list https://review.openstack.org/171380 | 21:26 |
| openstackgerrit | James E. Blair proposed stackforge/gertty: Highlight own name on change screen https://review.openstack.org/171381 | 21:26 |
| openstackgerrit | James E. Blair proposed stackforge/gertty: Open internal URLs in commentlinks https://review.openstack.org/171382 | 21:26 |
| openstackgerrit | James E. Blair proposed stackforge/gertty: Display file header in top line of diff https://review.openstack.org/171383 | 21:26 |
| openstackgerrit | James E. Blair proposed stackforge/gertty: Fix crash on opening a change with missing commits https://review.openstack.org/171384 | 21:26 |
| openstackgerrit | James E. Blair proposed stackforge/gertty: Fix error in double upload https://review.openstack.org/171385 | 21:26 |
| openstackgerrit | James E. Blair proposed openstack-infra/infra-specs: WIP: Add Zuul v3 spec. https://review.openstack.org/164371 | 21:27 |
| greghaynes | hah, did someone just get off a plane? | 21:27 |
| jeblair | indeed! :) | 21:27 |
| jeblair | but i did as much reviewing as hacking | 21:27 |
| clarkb | greghaynes: ya, the getopt errors don't cause fake-image-create to error | 21:27 |
| clarkb | greghaynes: so I am just going to leave that alone in this chagne | 21:27 |
| greghaynes | clarkb: ah! | 21:27 |
| greghaynes | clarkb: sounds good | 21:27 |
| *** pelix has joined #openstack-infra | 21:28 | |
| *** annegentle has joined #openstack-infra | 21:28 | |
| sdague | jogo: it would be nice to go through an analysis of what's making our setup time so much slower now | 21:28 |
| openstackgerrit | Clark Boylan proposed openstack-infra/nodepool: Use correct diskimage when building diskimages https://review.openstack.org/169474 | 21:28 |
| openstackgerrit | Clark Boylan proposed openstack-infra/nodepool: Support per provider image build settings https://review.openstack.org/126747 | 21:28 |
| openstackgerrit | Clark Boylan proposed openstack-infra/nodepool: Just set qcow2 version https://review.openstack.org/169383 | 21:28 |
| *** luqas has joined #openstack-infra | 21:29 | |
| clarkb | greghaynes: and ^ that should actually really fix things. yay for tests | 21:29 |
| openstackgerrit | Spencer Krum proposed openstack-infra/system-config: Refactor o_p::server out of role for elasticsearch https://review.openstack.org/171387 | 21:29 |
| greghaynes | clarkb: Yep, it is kind of nice to see things actually fail | 21:29 |
| *** ayoung has joined #openstack-infra | 21:29 | |
| jeblair | clarkb, greghaynes: i might have approved some changes while on the plane too.... :) | 21:30 |
| jeblair | mordred: my flight had wifi! it did not work. | 21:30 |
| mordred | jeblair: yay! and boo! | 21:31 |
| clarkb | sdague: I thought we did and found that we compile lxml a gazillion times | 21:31 |
| sdague | clarkb: nope | 21:31 |
| sdague | at least not what I can see | 21:31 |
| mescanef | hello, please add me (mmatuszkowiak@mirantis.com) to thee gerrits group members of stackforge/fuel-plugin-midonet (reference: https://review.openstack.org/#/c/171116/) | 21:31 |
| mescanef | fungi: ^^ | 21:31 |
| clarkb | oh well then maybe wheels won't be super helpful | 21:31 |
| mescanef | :-) | 21:31 |
| sdague | http://logs.openstack.org/57/171157/1/check/check-tempest-dsvm-full/6416003/logs/devstacklog.txt.gz - it's a 13 minute log | 21:32 |
| sdague | lxml is installed from packages | 21:32 |
| clarkb | zigzg: were you able to find my comment? (just making sure it ddn't get lost) | 21:32 |
| openstackgerrit | Joshua Harlow proposed openstack-infra/gerritlib: Add ability for this thread to die https://review.openstack.org/171393 | 21:32 |
| ttx | zuul in event processing overdrive | 21:32 |
| ttx | Queue lengths: 505 events, 187 results. | 21:32 |
| *** dkranz has quit IRC | 21:33 | |
| *** luqas has quit IRC | 21:33 | |
| openstackgerrit | Joshua Harlow proposed openstack-infra/gerritlib: Add ability for this thread to die https://review.openstack.org/171393 | 21:34 |
| mtreinish | jeblair: I just updated the gertty packages on arch | 21:35 |
| mtreinish | I actually had someone complain that they weren't working | 21:35 |
| mtreinish | so you have at least one archlinux user besides me | 21:35 |
| *** marun has quit IRC | 21:35 | |
| jogo | sdague: I skimmed through things a few times and didn't see anything that really stood out as a single issue | 21:35 |
| jogo | sdague: we do spend 5+ minutes installing packages though | 21:36 |
| *** YorikSar has quit IRC | 21:36 | |
| jogo | and every OSC command takes 1-3 seconds | 21:36 |
| *** Rockyg has quit IRC | 21:36 | |
| jeblair | ttx, sdague, fungi, jhesketh: it appears zuul is still being slammed by status.json requests from review.o.o | 21:36 |
| *** marun has joined #openstack-infra | 21:36 | |
| jeblair | (refrerer from review.o.o) | 21:36 |
| openstackgerrit | Joshua Harlow proposed openstack-infra/gerritlib: Add ability for this thread to die https://review.openstack.org/171393 | 21:36 |
| *** smccully has quit IRC | 21:37 | |
| *** dboik_ has joined #openstack-infra | 21:37 | |
| openstackgerrit | Merged openstack/requirements: attempt to stop using pip internals https://review.openstack.org/171157 | 21:37 |
| *** openstackgerrit has quit IRC | 21:37 | |
| *** openstackgerrit has joined #openstack-infra | 21:37 | |
| jeblair | since we reverted that change yesterday, this would be from people who simply still have review windows up | 21:38 |
| jeblair | think the only way to reduce the load would be to do some mod_rewrite stuff to block status.json with that referrer | 21:38 |
| jeblair | i'm about to board a plane so won't be able to do that myself | 21:38 |
| *** mriedem is now known as mriedem_away | 21:39 | |
| openstackgerrit | Joshua Harlow proposed openstack-infra/gerritlib: Add ability for this thread to die https://review.openstack.org/171393 | 21:39 |
| jeblair | sdague, clarkb, fungi, mordred, jhesketh: ^ can one of you do that? | 21:39 |
| jogo | sdague: and we have about 140 or so OSC calls | 21:40 |
| jogo | so that is a few minutes right there | 21:40 |
| clarkb | sdague: a couple things stand out after a quick skim. We spend a lot of time doing db migrations, can't we just deploy at latest version and avoid that? also we build tox venvs twice. One venv for the test env and the other to check a config file. That kills about a minute on its own. Maybe we can use a single venv for both | 21:40 |
| *** Krinkle|detached is now known as Krinkle | 21:40 | |
| *** dboik has quit IRC | 21:41 | |
| jogo | clarkb: we build tox venvs 2x? | 21:41 |
| clarkb | sdague: also the openstackclient calls are serialized and relatively slow. Maybe we can run some of those in parallel once keystone is configured | 21:41 |
| clarkb | jogo: yes | 21:41 |
| clarkb | jeblair: ya we can just remove the CORS stuff that allows it I guess | 21:41 |
| *** dboik_ has quit IRC | 21:41 | |
| jogo | clarkb: can you show me where it happens in the logs ttp://logs.openstack.org/08/170608/2/check/check-tempest-dsvm-full/daac19d/logs/devstacklog.txt.gz | 21:41 |
| clarkb | fungi: sdague ^ removing the CORS allowal for review.o.o against status.json seem sane? | 21:41 |
| clarkb | jogo: http://logs.openstack.org/57/171157/1/check/check-tempest-dsvm-full/6416003/logs/devstacklog.txt.gz#_2015-04-07_16_47_26_458 and http://logs.openstack.org/57/171157/1/check/check-tempest-dsvm-full/6416003/logs/devstacklog.txt.gz#_2015-04-07_16_55_01_256 | 21:42 |
| clarkb | jogo: just grep for 'tox -' | 21:42 |
| jogo | clarkb: clarkb ahh | 21:43 |
| dtroyer | clarkb: there is some batching that can be done with OSC to do multiple commands on a single auth…I don't hink it merged but I did a poc a few weeks ago | 21:44 |
| *** tonytan4ever has quit IRC | 21:44 | |
| jogo | clarkb: why use -r the second time? | 21:45 |
| *** melwitt has quit IRC | 21:45 | |
| jogo | dtroyer: you said that only saved 30 seconds though right? | 21:45 |
| jogo | if so that isn't really a lot of saved time | 21:45 |
| mtreinish | clarkb: oh yeah we probably can fix the tempest venv thing | 21:45 |
| mtreinish | one sec I'll push a patch for that | 21:45 |
| *** melwitt has joined #openstack-infra | 21:45 | |
| clarkb | jogo: I think because the config depends on versions of stuff, so you need latest of everything | 21:45 |
| clarkb | jogo: but that doesn't make a difference in the gate because its all single use nodes | 21:45 |
| jogo | clarkb: right | 21:46 |
| clarkb | fungi: sdague I am not seeing where we allowt he status.json GET from review.o.o on zuul's vhost | 21:46 |
| jogo | clarkb mtreinish: so that change should shave 30 seconds off itself | 21:46 |
| clarkb | fungi: sdague is that because we weren't using ajax before? | 21:46 |
| dtroyer | jogo: yeah, it isn't huge, and it doesn't work when you need to use a return value, like looking up an ID as there is no DSL, it's just piping commands to stdin | 21:46 |
| mtreinish | clarkb: but that's only on most gating jobs, if it runs a different tox definition (like for the heat, or trove jobs or on grenade) it'll still be making 2 | 21:46 |
| clarkb | mtreinish: why? | 21:46 |
| dougwig | clarkb: i think that devstack thing will work, though it uncovered a bug in the regex. ENABLED_SERVICES="-q-lbaas,q-lbaasv2" results in the following ending up getting passed down "..,v2". the leading q-lbaas gets stripped. | 21:47 |
| clarkb | mtreinish: all of those envs should be able to check their own config I hope | 21:47 |
| *** pelix has quit IRC | 21:47 | |
| mtreinish | clarkb: it's a matter of having devstack create it ahead of time | 21:47 |
| *** tkelsey has joined #openstack-infra | 21:47 | |
| mtreinish | clarkb: devstack doesn't know which job d-g will use | 21:47 |
| clarkb | mtreinish: oh devstack will only do the -efull | 21:47 |
| clarkb | mtreinish: I see | 21:47 |
| clarkb | dougwig: which regex? | 21:47 |
| jogo | mtreinish: why have the first run anyway? | 21:47 |
| clarkb | you could have something other than devstack build the env | 21:48 |
| mtreinish | the config verify script? | 21:48 |
| mtreinish | jogo: ^^^ | 21:48 |
| clarkb | since running tox to run the tests will build the env too | 21:48 |
| dougwig | clarkb: not sure yet, i'm guessing it's an issue with disable_negated_services | 21:48 |
| jogo | http://logs.openstack.org/57/171157/1/check/check-tempest-dsvm-full/6416003/logs/devstacklog.txt.gz#_2015-04-07_16_47_26_458 | 21:48 |
| mtreinish | jogo: so that's done to ensure we install tempest-lib from git in the tox venv we use to run | 21:49 |
| clarkb | I need to step out for a few, if others have ideas on making zuul happier let me know | 21:49 |
| mtreinish | jogo: clarkb and I were talking about: http://logs.openstack.org/57/171157/1/check/check-tempest-dsvm-full/6416003/logs/devstacklog.txt.gz#_2015-04-07_16_55_01_256 | 21:49 |
| mtreinish | jogo: right now in the worse case 3 venvs are being created for a tempest run (only on a job which runs a job other than -efull) | 21:49 |
| jogo | can we re-use the -efull from the first tox run and in the verify section? | 21:51 |
| mtreinish | jogo: yes that's the change I'm working on | 21:51 |
| *** tkelsey has quit IRC | 21:51 | |
| fungi | mescanef: done | 21:52 |
| jogo | mtreinish: excellent | 21:52 |
| jogo | mtreinish: and that is the one that will save about 30 seconds | 21:52 |
| jogo | clarkb: but greghaynes patch will save way more time I bet | 21:52 |
| clarkb | jogo I dont expect much | 21:52 |
| *** otter768 has joined #openstack-infra | 21:52 | |
| clarkb | sdague is right we dont spend a ton of time installing from python source | 21:52 |
| clarkb | its like 3 minutes | 21:53 |
| clarkb | the db migrations and osc setup really hurt | 21:53 |
| *** ddieterly has quit IRC | 21:53 | |
| * ttx can't go to sleep until zuul gives a node to his rc1 tag tarball job | 21:53 | |
| *** MaxV has quit IRC | 21:53 | |
| dougwig | clarkb: got it, sec. | 21:54 |
| ttx | ah. apparently I only had to ask politely. | 21:54 |
| fungi | clarkb: yeah, we weren't doing ajax, just direct access from javascript. i think what jeblair outlined is probably about the only solution (other than letting the traffic die off from attrition) | 21:55 |
| jogo | clarkb: installing takes a lot of time in unit tests at least | 21:55 |
| jogo | 10 minutes or so for nova | 21:55 |
| fungi | clarkb: i can look at it when i get back from dinner, but it's national beer day, so i might be a few hours | 21:55 |
| *** ddieterly has joined #openstack-infra | 21:55 | |
| fungi | the burrito bar up the road has a local agave ipa called "el guapo" on draft at the moment | 21:56 |
| fungi | also, while load on zuul is high, it's keeping up and utilizing available quota when needed | 21:57 |
| *** otter768 has quit IRC | 21:57 | |
| fungi | we're up at 700+ jobs per hour, which is way better than the past few weeks | 21:57 |
| mordred | \o/ | 21:58 |
| *** stevemar has quit IRC | 21:58 | |
| fungi | if mrmartin pops back in, https://git.openstack.org/cgit/openstack-infra/puppet-pgsql_backup/ officially exists now | 21:58 |
| *** asettle has joined #openstack-infra | 21:58 | |
| fungi | anyway, back later | 21:59 |
| clarkb | jogo I think this is an important distinction though | 21:59 |
| clarkb | openstack is slow | 21:59 |
| clarkb | this makes devstack slow | 21:59 |
| clarkb | you can speed up installs but that doesnt address openstack performance | 21:59 |
| jogo | clarkb: agreed | 22:00 |
| openstackgerrit | Stephanie Miller proposed openstack-infra/system-config: Add infra setup for zanata https://review.openstack.org/169074 | 22:00 |
| *** jckasper has quit IRC | 22:03 | |
| *** sabeen has quit IRC | 22:03 | |
| *** erikmwilson has quit IRC | 22:04 | |
| *** mwagner_lap has quit IRC | 22:04 | |
| jogo | clarkb: I don't think db syncs are that slow | 22:05 |
| jogo | >10 seconds each | 22:05 |
| clarkb | 10*20 = lots of seconds | 22:05 |
| jogo | cinder is 3 seconds or so | 22:05 |
| jogo | *5 not 20 | 22:05 |
| jogo | http://logs.openstack.org/57/171157/1/check/check-tempest-dsvm-full/6416003/logs/devstacklog.txt.gz#_2015-04-07_16_53_36_336 | 22:06 |
| jogo | 3 seconds for heat-manage db sync | 22:06 |
| mtreinish | clarkb: 200? | 22:06 |
| mtreinish | or am I missing something :) | 22:06 |
| openstackgerrit | K Jonathan Harker proposed openstack-infra/zuul: Handle NullChange "merges" gracefully. https://review.openstack.org/85217 | 22:06 |
| openstackgerrit | K Jonathan Harker proposed openstack-infra/zuul: Add a remote url override location https://review.openstack.org/76057 | 22:06 |
| clarkb | nova cinder glance neutron heat keystone ironic and thats just off the top of my head | 22:07 |
| *** doug-fish has left #openstack-infra | 22:07 | |
| jogo | we don't run ironic every time | 22:07 |
| jogo | and moving heat out as well | 22:07 |
| morganfainberg | hopefully keystone's migrations are getting better since we're doing collapses. | 22:08 |
| *** erikmwilson has joined #openstack-infra | 22:08 | |
| *** Krinkle is now known as Krinkle|detached | 22:08 | |
| clarkb | also horizon has a non trivial setup step (but not db migrations) | 22:08 |
| *** duran has quit IRC | 22:10 | |
| clarkb | is there a reason we don't just deploy the db as it should be out of the box without needing to do a bunch of migrations? | 22:11 |
| *** mestery has quit IRC | 22:11 | |
| clarkb | surely you can tell alembic to just apply the schema if db did not exist before it ran | 22:11 |
| *** s0nea has quit IRC | 22:11 | |
| clarkb | then set the migration level to latest | 22:11 |
| *** s0nea has joined #openstack-infra | 22:11 | |
| *** gyee has quit IRC | 22:11 | |
| *** Mmike has quit IRC | 22:11 | |
| *** duran has joined #openstack-infra | 22:12 | |
| *** mestery has joined #openstack-infra | 22:12 | |
| mtreinish | clarkb: well that assumes everything uses alembic, also what if the patch is changing the schema | 22:12 |
| *** praveenkumar has quit IRC | 22:12 | |
| clarkb | mtreinish: then you would apply the new schema | 22:12 |
| *** alexpilotti has quit IRC | 22:12 | |
| clarkb | it doesn't have to assume everything uses alembic, I just think that db syncing in general could be smarter | 22:12 |
| openstackgerrit | Michael Krotscheck proposed openstack-infra/storyboard-webclient: Renamed NPM commands for testing. https://review.openstack.org/171409 | 22:13 |
| *** alexpilotti has joined #openstack-infra | 22:13 | |
| clarkb | but maybe the cost is lower than I think it is and this is simplest | 22:13 |
| *** gyee has joined #openstack-infra | 22:13 | |
| *** alexpilotti has quit IRC | 22:14 | |
| mtreinish | clarkb: it might be, tbh I haven't thought too much about it | 22:14 |
| *** praveenkumar has joined #openstack-infra | 22:14 | |
| *** Mmike has joined #openstack-infra | 22:14 | |
| jogo | clarkb: https://review.openstack.org/171410 | 22:16 |
| jogo | that should help give us a few numbers to work off of | 22:16 |
| clarkb | jogo: we already have that data ... | 22:16 |
| jogo | clarkb: but its hard to parse right now | 22:16 |
| clarkb | jogo: fwiw nova takes ~1.5 minutes to install and only 5 seconds of that is pip install | 22:20 |
| *** gordc has quit IRC | 22:20 | |
| jogo | db sync is 16 seconds http://paste.openstack.org/show/199885/ | 22:21 |
| jogo | for tempst-dsvm-full | 22:21 |
| *** Guest81797 has quit IRC | 22:23 | |
| *** zigzg has quit IRC | 22:23 | |
| openstackgerrit | melanie witt proposed openstack-infra/project-config: Fix regex for excluding slow tests in Nova Cells V1 job https://review.openstack.org/171414 | 22:24 |
| openstackgerrit | K Jonathan Harker proposed openstack-infra/nodepool: Reset the client object after proxy timeouts https://review.openstack.org/74943 | 22:24 |
| openstackgerrit | Michael Krotscheck proposed openstack-infra/project-config: Added NPM executions for StoryBoard Tests. https://review.openstack.org/170995 | 22:24 |
| *** dkehn has quit IRC | 22:25 | |
| openstackgerrit | Joshua Harlow proposed openstack-infra/gerritlib: Add ability for this thread to die/cleanup https://review.openstack.org/171393 | 22:26 |
| *** marun has quit IRC | 22:27 | |
| *** esker has joined #openstack-infra | 22:28 | |
| *** annegentle has quit IRC | 22:29 | |
| *** erikmwilson has quit IRC | 22:29 | |
| *** amitgandhinz has quit IRC | 22:30 | |
| *** annegentle has joined #openstack-infra | 22:30 | |
| *** dims_ has joined #openstack-infra | 22:30 | |
| *** dims_ has quit IRC | 22:30 | |
| *** dims_ has joined #openstack-infra | 22:31 | |
| *** sdake has joined #openstack-infra | 22:31 | |
| *** bknudson has quit IRC | 22:33 | |
| *** dims has quit IRC | 22:34 | |
| *** duran has quit IRC | 22:34 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/system-config: Run yum update on Centos7 hosts https://review.openstack.org/171419 | 22:35 |
| ianw | fungi: ^ so that's one rather large difference between my images and what is being tested in the gate | 22:36 |
| *** sigmavirus24 is now known as sigmavirus24_awa | 22:36 | |
| *** sdake_ has joined #openstack-infra | 22:37 | |
| *** soren has quit IRC | 22:37 | |
| openstackgerrit | Clark Boylan proposed openstack-infra/puppet-zuul: Allow for ability to block status.json referers https://review.openstack.org/171420 | 22:37 |
| openstackgerrit | Clark Boylan proposed openstack-infra/system-config: Block review.o.o as a referer to zuul status.json https://review.openstack.org/171421 | 22:38 |
| *** sputnik13 has quit IRC | 22:38 | |
| clarkb | fungi: jeblair sdague ^ I think I got that right | 22:38 |
| *** nelsnels_ has joined #openstack-infra | 22:38 | |
| *** boris-42 has quit IRC | 22:38 | |
| greghaynes | clarkb: jogo the reason were not spending a lot of time installing is because were using system numpy and pylxml, the wheels thing might let us use non-system of those though without the big perf hit which I think is a win | 22:38 |
| *** dkehn has joined #openstack-infra | 22:38 | |
| greghaynes | agreed that 3mins which is the max gain isnt a huge gain though | 22:38 |
| clarkb | ianw: +2 | 22:39 |
| *** baoli has quit IRC | 22:39 | |
| jhesketh | Morning | 22:40 |
| dougwig | clarkb: that negation thing, https://review.openstack.org/#/c/171402/ | 22:40 |
| *** sdake has quit IRC | 22:40 | |
| NobodyCam | morning infra folk | 22:41 |
| NobodyCam | :) | 22:41 |
| clarkb | jhesketh: good morning. Please see comment on https://review.openstack.org/#/c/107267/ | 22:41 |
| clarkb | jhesketh: I +2'd anyways but want to make sure we agree on a path forward to fix the thing the comment points out there | 22:41 |
| NobodyCam | is there a place I can check failed logins. I'm helping someone test there ssh connection | 22:42 |
| clarkb | jhesketh: (I don't mind picking that up in a subsequent change either if it helps) | 22:42 |
| *** nelsnelson has quit IRC | 22:42 | |
| clarkb | NobodyCam: no we don't publish our auth.log | 22:42 |
| clarkb | NobodyCam: can you be a bit more specific and possibly paste commands with output so we can see what is going on? | 22:42 |
| *** nelsnels_ has quit IRC | 22:43 | |
| NobodyCam | ssh -p 29418 john-stafford@review.openstack.org | 22:43 |
| NobodyCam | permssion denied (public key) | 22:43 |
| *** alexpilotti has joined #openstack-infra | 22:43 | |
| *** annegentle has quit IRC | 22:43 | |
| NobodyCam | I've had him add a couple -v's and it appears to be attempting to send the key | 22:44 |
| greghaynes | clarkb: I wonder if for the zuul thing we should just make zuul write out the status.json to disk and serve that with $webserver | 22:44 |
| clarkb | NobodyCam: go to https://review.openstack.org/#/settings/ and check that the username matches john-stafford then check https://review.openstack.org/#/settings/ssh-keys for the public key that you have locall | 22:44 |
| jroll | NobodyCam: how sure are you that the public key is correct in gerrit (i.e. no extra line breaks or whatever) | 22:44 |
| *** ashleighfarnham has quit IRC | 22:44 | |
| clarkb | greghaynes: or be more agressive about caching it in apache (we already do cache it) | 22:45 |
| NobodyCam | jroll: we just uploaded a couple of minutes ago | 22:45 |
| jroll | NobodyCam: some terminals don't handle line breaks correctly, might be worth checking | 22:45 |
| greghaynes | clarkb: yea, if you can be forceful there | 22:45 |
| jhesketh | clarkb: thanks for the review. Yeah not sure if it's better here or in another change. Following the Apache rules closer means probably just matching off the filename and the work to get the headers isn't useful. Maybe we want to use a swift cdn for this and keep os-loganalyze to it's name (ie just for formatting logs) | 22:48 |
| openstackgerrit | Adam Gandelman proposed openstack-infra/project-config: python-ironicclient: Make src job voting, drop pxe_ssh job https://review.openstack.org/171426 | 22:49 |
| clarkb | jhesketh: the idea to use the cdn came up but thats a huge pita with rax | 22:49 |
| clarkb | NobodyCam: I see no user with a username of john-stafford in the database fwiw | 22:49 |
| clarkb | jhesketh: its doable though, we would just have to have someone set it up by hand then update the apache vhost to 302 to it | 22:50 |
| *** dboik has joined #openstack-infra | 22:50 | |
| clarkb | jhesketh: My preference is to have an htmlify/ and passthrough/ path that apache or other wsgi server can call into as appropriate | 22:51 |
| *** jgrimm has quit IRC | 22:51 | |
| *** maurosr has quit IRC | 22:51 | |
| *** jgrimm has joined #openstack-infra | 22:52 | |
| jhesketh | clarkb: okay. I wonder if sdague has an opinion ^ | 22:52 |
| *** maurosr has joined #openstack-infra | 22:52 | |
| *** dboik_ has joined #openstack-infra | 22:53 | |
| clarkb | my concern with using the cdn is both the manual setup required since this is a raxism iirc and its possible that swift may not be publicly accessible in all clouds so anyone else wanting to do similar would have to write the above anyways | 22:54 |
| *** mattfarina has quit IRC | 22:54 | |
| *** _nadya_ has joined #openstack-infra | 22:55 | |
| jhesketh | Yeah I think it's fair enough, I'm just thinking out loud ;-) | 22:55 |
| *** mwagner_lap has joined #openstack-infra | 22:55 | |
| *** dboik has quit IRC | 22:55 | |
| *** bknudson has joined #openstack-infra | 22:56 | |
| *** dimtruck is now known as zz_dimtruck | 22:56 | |
| jhesketh | clarkb: reading the backlog, has somebody had a chance to revert the zuul status checker? | 22:56 |
| clarkb | jhesketh: yes, its reverted but any changes that are still loaded will continue to hit the zuul status.json page | 22:56 |
| openstackgerrit | Thanh Ha proposed openstack-infra/nodepool: Clarify meaning of Jenkins master configuration settings in target https://review.openstack.org/167878 | 22:58 |
| *** bknudson1 has joined #openstack-infra | 22:58 | |
| jhesketh | Okay, good to know | 22:59 |
| *** jgrimm is now known as zz_jgrimm | 22:59 | |
| *** _nadya_ has quit IRC | 22:59 | |
| *** bknudson has quit IRC | 23:00 | |
| *** rlandy has quit IRC | 23:03 | |
| *** achanda has quit IRC | 23:05 | |
| openstackgerrit | Joshua Harlow proposed openstack-infra/gerritlib: Add ability for this thread to die/cleanup https://review.openstack.org/171393 | 23:08 |
| *** Krinkle|detached is now known as Krinkle | 23:08 | |
| *** Sukhdev has joined #openstack-infra | 23:08 | |
| *** btully has quit IRC | 23:10 | |
| *** Krinkle is now known as Krinkle|detached | 23:18 | |
| *** achanda has joined #openstack-infra | 23:21 | |
| *** tnovacik has quit IRC | 23:22 | |
| *** ayoung has quit IRC | 23:22 | |
| openstackgerrit | Stephanie Miller proposed openstack-infra/puppet-zanata: Update default zanata client version to latest available https://review.openstack.org/171430 | 23:24 |
| *** Krinkle|detached is now known as Krinkle | 23:24 | |
| *** yamamoto has joined #openstack-infra | 23:25 | |
| *** dboik_ has quit IRC | 23:25 | |
| *** ZZelle has quit IRC | 23:27 | |
| *** ddieterly has quit IRC | 23:28 | |
| *** luqas has joined #openstack-infra | 23:30 | |
| *** hichihara has joined #openstack-infra | 23:31 | |
| clarkb | asselin_: by the way I think https://pypi.python.org/pypi/envassert is what nibalizer wants to do to test puppet modules | 23:32 |
| *** ociuhandu has joined #openstack-infra | 23:33 | |
| *** Krinkle is now known as Krinkle|detached | 23:34 | |
| asselin_ | clarkb, ok will take a look at that | 23:34 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Rename auth_plugin to auth_type https://review.openstack.org/171432 | 23:35 |
| *** luqas has quit IRC | 23:35 | |
| *** davideagnello has quit IRC | 23:35 | |
| clarkb | asselin_: with that we might be able to do something as simple as `tox` to test the module then tox would run puppet and then run envassert tests | 23:35 |
| clarkb | asselin_: the nice thing about that is we already know how to run tox pretty well | 23:35 |
| openstackgerrit | Mathieu Gagné proposed openstack-infra/jenkins-job-builder: Add ability to configure releaseEnvVar for maven-deploy publisher https://review.openstack.org/163626 | 23:36 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Rename auth_plugin to auth_type https://review.openstack.org/171432 | 23:37 |
| *** bhunter71 has quit IRC | 23:37 | |
| mordred | clarkb: ^^ just a rebase to remove a dep ... also, 170220 gets us docs in the docs fwiw | 23:37 |
| asselin_ | clarkb, +1 | 23:38 |
| clarkb | mordred: +2 on both | 23:38 |
| mordred | clarkb: thanks! | 23:39 |
| *** amitgandhinz has joined #openstack-infra | 23:41 | |
| nibalizer | asselin_: clarkb yip! im a bit underwater with things right now, haven't had time to get it working yet | 23:42 |
| openstackgerrit | Stephanie Miller proposed openstack-infra/system-config: Add infra setup for zanata https://review.openstack.org/169074 | 23:43 |
| *** annegentle has joined #openstack-infra | 23:44 | |
| asselin_ | clarkb, nibalizer yeah...that was the next thing I was going to look at: how to get it to run on each patch set. tox is the way to go | 23:45 |
| *** amitgandhinz has quit IRC | 23:45 | |
| *** teran_ has joined #openstack-infra | 23:48 | |
| clarkb | mtreinish: any idea why https://jenkins04.openstack.org/job/gate-tempest-dsvm-neutron-pg/2430/consoleFull is taking so long? | 23:48 |
| clarkb | mtreinish: the vpnaas tests don't seem to be consuming much time | 23:49 |
| *** yamamoto has quit IRC | 23:49 | |
| clarkb | mtreinish: also that is just smoke tests not full | 23:49 |
| clarkb | so I am extra confused | 23:50 |
| *** annegentle has quit IRC | 23:50 | |
| *** teran has quit IRC | 23:50 | |
| *** hdd has quit IRC | 23:53 | |
| *** otter768 has joined #openstack-infra | 23:53 | |
| *** boris-42 has joined #openstack-infra | 23:54 | |
| *** hodos has quit IRC | 23:55 | |
| *** otter768 has quit IRC | 23:58 | |
| *** MarkAtwood has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!