*** salv-orlando has quit IRC | 00:04 | |
*** yamamoto has quit IRC | 00:05 | |
*** yamahata has quit IRC | 00:06 | |
*** amotoki has joined #openstack-infra | 00:14 | |
*** yamahata has joined #openstack-infra | 00:18 | |
*** dboik has quit IRC | 00:22 | |
*** mahito has joined #openstack-infra | 00:23 | |
*** dhritishikhar has quit IRC | 00:25 | |
openstackgerrit | Thanh Ha proposed openstack-infra/jenkins-job-builder: Add support for Sonatype CLM plugin https://review.openstack.org/164446 | 00:25 |
---|---|---|
*** sdake has quit IRC | 00:29 | |
*** sputnik13 has quit IRC | 00:31 | |
mordred | fungi: oh - maybe we just trigger a fake republication then | 00:33 |
mordred | fungi: it's only a few repos worth of stuff anyway | 00:33 |
*** sputnik13 has joined #openstack-infra | 00:36 | |
*** signed8bit has joined #openstack-infra | 00:58 | |
*** woodster_ has quit IRC | 01:00 | |
*** dboik has joined #openstack-infra | 01:04 | |
*** salv-orlando has joined #openstack-infra | 01:05 | |
*** a13x2121 has quit IRC | 01:06 | |
Kennan | hi . could we update requests as following? | 01:07 |
Kennan | https://review.openstack.org/#/c/173290/ | 01:07 |
*** signed8bit has quit IRC | 01:09 | |
*** salv-orlando has quit IRC | 01:11 | |
*** sputnik13 has quit IRC | 01:12 | |
*** luqas has joined #openstack-infra | 01:17 | |
*** alex_xu has quit IRC | 01:18 | |
*** luqas has quit IRC | 01:21 | |
*** alex_xu has joined #openstack-infra | 01:23 | |
*** mriedem has quit IRC | 01:26 | |
*** yamamoto has joined #openstack-infra | 01:29 | |
*** salv-orlando has joined #openstack-infra | 01:32 | |
*** salv-orlando has quit IRC | 01:35 | |
*** mahito_ has joined #openstack-infra | 01:36 | |
*** mahito has quit IRC | 01:40 | |
*** dboik has quit IRC | 01:40 | |
*** ivar-lazzaro has joined #openstack-infra | 01:40 | |
*** ddieterl_ has joined #openstack-infra | 01:41 | |
*** ddieterly has quit IRC | 01:41 | |
*** salv-orlando has joined #openstack-infra | 01:41 | |
anteaya | heads up I have had to reject 3 emails from huawei to -announce requesting a gerrit account for themselves, I continue to tell them they create the account themselves and provide a link to http://ci.openstack.org/third_party.html#creating-a-service-account but I'm not sure how many more attempts it will take before they read my reply | 01:46 |
anteaya | just in case they wander by and ask in here | 01:46 |
*** julim has quit IRC | 01:47 | |
*** julim has joined #openstack-infra | 01:48 | |
*** julim has quit IRC | 01:48 | |
*** ivar-lazzaro has quit IRC | 01:50 | |
*** salv-orlando has quit IRC | 01:51 | |
*** achanda has joined #openstack-infra | 01:53 | |
*** baoli has quit IRC | 01:54 | |
*** vilobhmm1 has joined #openstack-infra | 01:56 | |
*** mahito_ has quit IRC | 01:56 | |
*** mahito has joined #openstack-infra | 01:58 | |
*** zz_dimtruck is now known as dimtruck | 02:00 | |
tchaypo | Kennan: REading the description on https://review.openstack.org/#/c/173290/3 again | 02:03 |
tchaypo | if docker-py requires >2.2.1, I’m curious about why that isn’t installed when docker-py gets insatlled | 02:04 |
tchaypo | (not doubting your explanation, I’m just wondering how that dependency gets missed | 02:04 |
Kennan | :tchaypo which part do you refer > 2.2.1 ? reuqests ? | 02:07 |
tchaypo | yeah - the bit where docker-py won’t run because requests is too old | 02:07 |
tchaypo | I’m sure I worked out the answer to this last week but now I don’t remember it | 02:07 |
Kennan | the new version od docker-py require new versions of requests | 02:08 |
Kennan | https://github.com/docker/docker-py/blob/master/setup.py | 02:08 |
Kennan | but in devstack gate jobs | 02:08 |
*** cody-somerville has joined #openstack-infra | 02:08 | |
Kennan | it default use | 02:08 |
Kennan | But in devstack stack.sh install in ubuntu, it uses requests 2.2.1 | 02:08 |
Kennan | It seems use system package 2.2.1-1ubuntu0.2 | 02:08 |
tchaypo | http://logs.openstack.org/54/173154/1/check/check-functional-dsvm-magnum/7fa79f3/logs/devstacklog.txt.gz#_2015-04-14_05_37_28_996 | 02:08 |
tchaypo | it looks like it’s pip-installed docker-py-1.1.0 at that point | 02:09 |
tchaypo | so presumably at that point there was a version of requests installed that met the requirements | 02:09 |
Kennan | :tchaypo yes, not but seems it not get new requests install finally | 02:12 |
Kennan | as I did not find requests installed is >2.5.* | 02:12 |
tchaypo | 2015-04-14 05:31:34.265 | /opt/stack/new/devstack/tmp-venv-Lh8J/local/lib/python2.7/site-packages/pip/_vendor/requests/packages/urllib3/util/ssl_.py:79: InsecurePlatformWarning: A true SSLContext object is not available. This prevents urllib3 from configuring SSL appropriately and may cause certain SSL connections to fail. For more information, see | 02:13 |
tchaypo | https://urllib3.readthedocs.org/en/latest/security.html#insecureplatformwarning. | 02:13 |
tchaypo | 2015-04-14 05:33:11.477 | Requirement already satisfied (use --upgrade to upgrade): requests!=2.4.0,>=2.2.0 in /usr/lib/python2.7/dist-packages (from keystonemiddleware>=1.5.0) | 02:13 |
tchaypo | in the first line it looks like it’s using requests in a virtualenv; in the second line it seems to be pointing at the system version | 02:13 |
*** vilobhmm1 has quit IRC | 02:13 | |
*** vilobhmm1 has joined #openstack-infra | 02:14 | |
*** vilobhmm1 has quit IRC | 02:14 | |
*** stevemar has joined #openstack-infra | 02:14 | |
*** vilobhmm1 has joined #openstack-infra | 02:14 | |
tchaypo | http://logs.openstack.org/54/173154/1/check/check-functional-dsvm-magnum/7fa79f3/logs/pip-freeze.txt.gz - definitely 2.2.1 installed by that stage | 02:16 |
Kennan | yes | 02:16 |
Kennan | so not know why use that version | 02:16 |
tchaypo | wait | 02:20 |
*** ayoung has quit IRC | 02:20 | |
tchaypo | I may have just understood something lifeless said a few days ago | 02:20 |
* tchaypo will go test | 02:20 | |
*** sdake has joined #openstack-infra | 02:21 | |
lifeless | tchaypo: ? | 02:22 |
*** woodster_ has joined #openstack-infra | 02:28 | |
*** sputnik13 has joined #openstack-infra | 02:28 | |
*** vilobhmm1 has quit IRC | 02:39 | |
*** otter768 has quit IRC | 02:50 | |
*** zhiwei has joined #openstack-infra | 02:54 | |
*** baoli has joined #openstack-infra | 02:55 | |
*** david-lyle has quit IRC | 02:55 | |
*** dhritishikhar has joined #openstack-infra | 02:57 | |
*** jamielennox is now known as jamielennox|away | 02:58 | |
*** baoli has quit IRC | 02:59 | |
*** achanda has quit IRC | 03:00 | |
*** jamielennox|away is now known as jamielennox | 03:03 | |
openstackgerrit | Brant Knudson proposed openstack/requirements: Recap keystoneclient for stable/icehouse https://review.openstack.org/175234 | 03:08 |
*** dhritishikhar has quit IRC | 03:08 | |
*** cody-somerville has quit IRC | 03:09 | |
*** dhritishikhar has joined #openstack-infra | 03:10 | |
*** dhritishikhar_ has joined #openstack-infra | 03:11 | |
*** dhritishikhar has quit IRC | 03:11 | |
*** armax has joined #openstack-infra | 03:14 | |
*** sdake has quit IRC | 03:23 | |
*** dboik has joined #openstack-infra | 03:30 | |
*** dhritishikhar_ has quit IRC | 03:33 | |
*** dhritishikhar_ has joined #openstack-infra | 03:35 | |
*** armax has quit IRC | 03:37 | |
*** coolsvap|afk is now known as coolsvap | 03:39 | |
*** dhritishikhar_ has quit IRC | 03:39 | |
*** dhritishikhar_ has joined #openstack-infra | 03:39 | |
*** salv-orlando has joined #openstack-infra | 03:45 | |
*** salv-orlando has quit IRC | 03:51 | |
*** ddieterl_ has quit IRC | 03:51 | |
*** sarob has joined #openstack-infra | 03:52 | |
*** sarob has quit IRC | 03:52 | |
*** sarob has joined #openstack-infra | 03:52 | |
*** vilobhmm1 has joined #openstack-infra | 03:54 | |
*** dhritishikhar_ has quit IRC | 03:57 | |
*** dhritishikhar_ has joined #openstack-infra | 03:57 | |
*** dboik has quit IRC | 04:03 | |
*** otter768 has joined #openstack-infra | 04:03 | |
*** otter768 has quit IRC | 04:13 | |
*** yfried|afk is now known as yfried__ | 04:18 | |
*** subscope has joined #openstack-infra | 04:25 | |
*** Kennan2 has joined #openstack-infra | 04:26 | |
*** Kennan has quit IRC | 04:26 | |
*** sabeen has joined #openstack-infra | 04:28 | |
*** aswadr has joined #openstack-infra | 04:32 | |
*** ajmiller has quit IRC | 04:34 | |
*** Longgeek has joined #openstack-infra | 04:35 | |
*** Longgeek has quit IRC | 04:35 | |
*** Longgeek has joined #openstack-infra | 04:36 | |
*** BharatK has joined #openstack-infra | 04:43 | |
*** subscope has quit IRC | 04:47 | |
*** sks has joined #openstack-infra | 04:48 | |
*** ibiris_away is now known as ibiris | 04:48 | |
*** yfried__ is now known as yfried|afk | 04:49 | |
*** juggler has joined #openstack-infra | 04:50 | |
*** btran has joined #openstack-infra | 04:50 | |
*** yfried|afk is now known as yfried__ | 04:51 | |
*** ddieterly has joined #openstack-infra | 04:51 | |
*** sputnik13 has quit IRC | 04:52 | |
*** HeOS has quit IRC | 04:53 | |
*** luqas has joined #openstack-infra | 04:55 | |
*** sputnik13 has joined #openstack-infra | 04:56 | |
*** ddieterly has quit IRC | 04:56 | |
openstackgerrit | David Pursehouse proposed openstack-infra/lodgeit: Use SQLAlchemy version 0.9.7 https://review.openstack.org/175259 | 04:58 |
*** boris-42 has quit IRC | 04:58 | |
*** ibiris is now known as ibiris_away | 04:58 | |
*** luqas has quit IRC | 04:59 | |
*** dimtruck is now known as zz_dimtruck | 05:00 | |
openstackgerrit | David Pursehouse proposed openstack-infra/lodgeit: Update README https://review.openstack.org/172293 | 05:01 |
openstackgerrit | David Pursehouse proposed openstack-infra/lodgeit: Replace .hgignore with .gitignore https://review.openstack.org/175260 | 05:01 |
openstackgerrit | David Pursehouse proposed openstack-infra/lodgeit: Replace .hgignore with .gitignore https://review.openstack.org/175260 | 05:02 |
*** yfried__ has quit IRC | 05:02 | |
*** ibiris_away is now known as ibiris | 05:03 | |
*** sputnik13 has quit IRC | 05:07 | |
*** deepakcs has joined #openstack-infra | 05:10 | |
*** pal has joined #openstack-infra | 05:13 | |
*** sabeen has quit IRC | 05:16 | |
openstackgerrit | David Pursehouse proposed openstack-infra/lodgeit: Add a requirements.txt file https://review.openstack.org/175262 | 05:16 |
*** pal has quit IRC | 05:19 | |
*** tnovacik has joined #openstack-infra | 05:24 | |
*** sputnik13 has joined #openstack-infra | 05:31 | |
*** sputnik13 has quit IRC | 05:35 | |
*** achanda has joined #openstack-infra | 05:37 | |
*** sputnik13 has joined #openstack-infra | 05:38 | |
*** ildikov has quit IRC | 05:43 | |
*** hdd has joined #openstack-infra | 05:50 | |
tchaypo | lifeless: you’re installing via “pip -r requirements.txt”, and requirements.txt has a constraint (such as ‘requests>=2.2.1”) which is narrowed in a dependency (such as docker-py requiring requests>=2.5.2), the tighter requirement is ignored | 05:52 |
tchaypo | there was meant to be an “if” there | 05:52 |
*** ddieterly has joined #openstack-infra | 05:53 | |
tchaypo | whereas if you “pip install docker-py”, docker-py’s install_requires will mean that the tighter constraint is applied | 05:55 |
*** ddieterly has quit IRC | 05:57 | |
*** salv-orlando has joined #openstack-infra | 05:57 | |
openstackgerrit | Merged openstack-infra/project-config: Adds gate-group-based-policy-specs-python27 check https://review.openstack.org/173133 | 05:58 |
*** yamahata has quit IRC | 05:58 | |
*** mrunge has joined #openstack-infra | 05:58 | |
*** yamahata has joined #openstack-infra | 05:59 | |
*** salv-orlando has quit IRC | 06:06 | |
openstackgerrit | Merged openstack-infra/pypi-mirror: Workflow documentation is now in infra-manual https://review.openstack.org/139429 | 06:09 |
*** afazekas_ has joined #openstack-infra | 06:10 | |
*** garyk has joined #openstack-infra | 06:12 | |
*** takehiro-kaneko has joined #openstack-infra | 06:13 | |
*** otter768 has joined #openstack-infra | 06:14 | |
*** yamahata has quit IRC | 06:15 | |
takehiro-kaneko | hi | 06:18 |
*** otter768 has quit IRC | 06:18 | |
*** zz_jgrimm is now known as jgrimm | 06:19 | |
*** yfried__ has joined #openstack-infra | 06:19 | |
takehiro-kaneko | I'm managing a stackforge project and want to integrate the project with launchpad bugs/blueprint. | 06:20 |
takehiro-kaneko | How can I setup the integration? | 06:21 |
*** scheuran has joined #openstack-infra | 06:21 | |
*** dhritishikhar_ has quit IRC | 06:22 | |
*** dhritishikhar_ has joined #openstack-infra | 06:26 | |
lifeless | tchaypo: yes | 06:28 |
*** yfried__ is now known as yfried|afk | 06:30 | |
*** woodster_ has quit IRC | 06:30 | |
*** vlaza has joined #openstack-infra | 06:32 | |
*** yfried|afk is now known as yfried__ | 06:34 | |
*** sarob has quit IRC | 06:34 | |
openstackgerrit | Huang Rui proposed openstack-infra/project-config: Create neutron-zvm-plugin project on StackForge https://review.openstack.org/171030 | 06:36 |
*** sergsh has joined #openstack-infra | 06:37 | |
*** stevemar has quit IRC | 06:40 | |
*** mmedvede has quit IRC | 06:44 | |
*** flaper87 has quit IRC | 06:45 | |
*** flaper87 has joined #openstack-infra | 06:45 | |
openstackgerrit | Merged openstack-infra/project-config: Add pep8 check for haos project https://review.openstack.org/172948 | 06:45 |
openstackgerrit | Merged openstack-infra/project-config: Move GBP experimental job to check queue https://review.openstack.org/172726 | 06:47 |
*** gillllllliard is now known as gilliard | 06:48 | |
*** jamielennox is now known as jamielennox|away | 06:52 | |
*** ddieterly has joined #openstack-infra | 06:53 | |
*** stevemar has joined #openstack-infra | 06:54 | |
*** jyuso1 has joined #openstack-infra | 06:54 | |
*** ddieterly has quit IRC | 06:58 | |
*** sushilkm has joined #openstack-infra | 06:58 | |
*** sushilkm has left #openstack-infra | 06:58 | |
*** mmedvede has joined #openstack-infra | 07:00 | |
*** rakhmerov has joined #openstack-infra | 07:00 | |
*** katyafervent_awa has joined #openstack-infra | 07:02 | |
*** dteselkin has joined #openstack-infra | 07:03 | |
*** tsufiev has joined #openstack-infra | 07:04 | |
*** zhiwei has quit IRC | 07:05 | |
*** asilenkov has joined #openstack-infra | 07:05 | |
*** sergsh has quit IRC | 07:06 | |
*** teran has quit IRC | 07:07 | |
*** katyafervent_awa is now known as katyafervent | 07:07 | |
*** tkelsey has joined #openstack-infra | 07:09 | |
*** HeOS_ has joined #openstack-infra | 07:10 | |
*** ildikov has joined #openstack-infra | 07:12 | |
*** mpaolino has joined #openstack-infra | 07:15 | |
*** teran has joined #openstack-infra | 07:18 | |
*** claudiub has joined #openstack-infra | 07:21 | |
*** tkelsey has quit IRC | 07:23 | |
*** devvesa has joined #openstack-infra | 07:24 | |
*** ociuhandu has joined #openstack-infra | 07:27 | |
*** tkelsey has joined #openstack-infra | 07:27 | |
*** chlong has quit IRC | 07:29 | |
*** takehiro-kaneko has quit IRC | 07:30 | |
*** ihrachyshka has joined #openstack-infra | 07:31 | |
*** e0ne has joined #openstack-infra | 07:31 | |
*** e0ne has quit IRC | 07:34 | |
*** ildikov has quit IRC | 07:34 | |
*** jlanoux has joined #openstack-infra | 07:37 | |
*** luqas has joined #openstack-infra | 07:38 | |
*** MaxV has joined #openstack-infra | 07:38 | |
*** e0ne has joined #openstack-infra | 07:38 | |
openstackgerrit | Sergey Kraynev proposed openstack-infra/project-config: Enable ceilometer service on Heat gates https://review.openstack.org/174394 | 07:39 |
*** tkelsey has quit IRC | 07:39 | |
*** vilobhmm1 has quit IRC | 07:39 | |
*** e0ne has quit IRC | 07:40 | |
*** kaisers has joined #openstack-infra | 07:42 | |
*** kaisers has left #openstack-infra | 07:42 | |
*** ociuhandu has quit IRC | 07:45 | |
*** arxcruz has joined #openstack-infra | 07:45 | |
*** e0ne has joined #openstack-infra | 07:45 | |
*** jistr has joined #openstack-infra | 07:47 | |
*** achanda has quit IRC | 07:47 | |
*** ildikov has joined #openstack-infra | 07:49 | |
*** weshay has joined #openstack-infra | 07:52 | |
*** sputnik13 has quit IRC | 07:52 | |
*** claudiub has quit IRC | 07:53 | |
*** sputnik13 has joined #openstack-infra | 07:54 | |
*** ddieterly has joined #openstack-infra | 07:54 | |
*** Hal has joined #openstack-infra | 07:54 | |
*** Hal is now known as Guest89583 | 07:55 | |
*** claudiub has joined #openstack-infra | 07:56 | |
*** ddieterly has quit IRC | 07:59 | |
*** tkelsey has joined #openstack-infra | 08:00 | |
*** AJaeger has joined #openstack-infra | 08:00 | |
*** AJaeger has joined #openstack-infra | 08:00 | |
*** HeOS has joined #openstack-infra | 08:00 | |
*** dguitarbite has joined #openstack-infra | 08:02 | |
*** e0ne has quit IRC | 08:04 | |
*** hashar has joined #openstack-infra | 08:05 | |
*** Ala has joined #openstack-infra | 08:05 | |
*** sergsh has joined #openstack-infra | 08:05 | |
*** fhubik has joined #openstack-infra | 08:06 | |
*** Ala has joined #openstack-infra | 08:06 | |
*** stevemar has quit IRC | 08:06 | |
*** claudiub has quit IRC | 08:06 | |
*** salv-orlando has joined #openstack-infra | 08:07 | |
*** mmedvede has quit IRC | 08:07 | |
*** e0ne has joined #openstack-infra | 08:08 | |
*** e0ne is now known as e0ne_ | 08:09 | |
openstackgerrit | Igor Belikov proposed openstack-infra/jenkins-job-builder: Add slack notification support, largely inspired by hipchat module https://review.openstack.org/136627 | 08:09 |
*** dhritishikhar_ has quit IRC | 08:10 | |
*** dhritishikhar_ has joined #openstack-infra | 08:10 | |
*** salv-orlando has quit IRC | 08:11 | |
openstackgerrit | Fabien Boucher proposed openstack-infra/system-config: Move server class call outside of jenkins*.pp class https://review.openstack.org/170487 | 08:11 |
*** e0ne_ is now known as e0ne | 08:11 | |
*** claudiub has joined #openstack-infra | 08:11 | |
openstackgerrit | Fabien Boucher proposed openstack-infra/system-config: Move some hardcoded values as parameters for zuul manifests https://review.openstack.org/168330 | 08:12 |
*** sergsh has quit IRC | 08:12 | |
*** jlibosva has joined #openstack-infra | 08:15 | |
openstackgerrit | takehirokaneko proposed openstack-infra/project-config: Add groups list to stackforge/rack project config https://review.openstack.org/175294 | 08:15 |
*** jlibosva has quit IRC | 08:15 | |
*** salv-orlando has joined #openstack-infra | 08:17 | |
*** mmedvede has joined #openstack-infra | 08:17 | |
*** e0ne has quit IRC | 08:17 | |
*** derekh has joined #openstack-infra | 08:19 | |
*** vponomaryov has joined #openstack-infra | 08:19 | |
*** shardy has joined #openstack-infra | 08:20 | |
*** tkelsey has quit IRC | 08:20 | |
*** salv-orlando has quit IRC | 08:21 | |
*** salv-orlando has joined #openstack-infra | 08:23 | |
*** fhubik is now known as fhubik_afk | 08:25 | |
*** sergsh has joined #openstack-infra | 08:25 | |
BharatK | AJaeger: ping | 08:27 |
BharatK | AJaeger: Regarding the bug https://review.openstack.org/#/c/173408/ | 08:27 |
*** salv-orlando has quit IRC | 08:28 | |
*** Guest89583 has quit IRC | 08:28 | |
*** zz_johnthetubagu is now known as johnthetubaguy | 08:29 | |
AJaeger | BharatK, please talk with sdague. He might be up in three hours... | 08:30 |
*** dmellado has joined #openstack-infra | 08:30 | |
BharatK | AJaeger: Okay sure Thank you. | 08:30 |
*** teran has quit IRC | 08:30 | |
*** doude has joined #openstack-infra | 08:34 | |
*** mahito has quit IRC | 08:34 | |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config: Mark python-saharaclient python34 jobs master only https://review.openstack.org/175303 | 08:35 |
*** andreykurilin__ has joined #openstack-infra | 08:39 | |
AJaeger | ttx, SergeyLukjanov , please review the patch above which is needed by stable/kilo python-saharaclient. SergeyLukjanov, see https://review.openstack.org/#/c/173856/ also | 08:39 |
*** garyk1 has joined #openstack-infra | 08:39 | |
openstackgerrit | Merged openstack-dev/pbr: Reinstate pure-git-hashes-are-illegal test. https://review.openstack.org/174625 | 08:40 |
ttx | AJaeger: I think we need to disable the functional test job as well | 08:40 |
AJaeger | ttx, that's what I'm wondering as well... | 08:40 |
ttx | Sergey told me in an email " The DSVM job implementation on client side is done only in master as well." | 08:40 |
AJaeger | Ah, ok will add it. | 08:40 |
ttx | (pasted his comment on the failing job) | 08:40 |
AJaeger | Wasn't sure about this | 08:40 |
AJaeger | Will amend my patch | 08:41 |
ttx | so both should be skipped on stable since they don't have the required code in | 08:41 |
*** fhubik_afk is now known as fhubik | 08:41 | |
ttx | AJaeger: ok, ping me when done and I'll +1 it | 08:41 |
*** garyk has quit IRC | 08:42 | |
AJaeger | ttx, ^ | 08:43 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config: Mark some python-saharaclient jobs master only https://review.openstack.org/175303 | 08:43 |
AJaeger | ttx, now ^ | 08:43 |
ttx | +1, thx! | 08:44 |
*** yamamoto has quit IRC | 08:45 | |
AJaeger | ttx, anything else I can help today quickly? | 08:46 |
*** ssam2 has joined #openstack-infra | 08:46 | |
ttx | AJaeger: I'll doublecheck with dhellmann but likely we'll open RC2 windows for some projects today/tomorrow | 08:48 |
ttx | since stable/kilo is mostly in good shape at this point | 08:48 |
AJaeger | great! | 08:48 |
ttx | which means we can push the translations patches | 08:48 |
ttx | AJaeger: hmm, actually we could already push those patches now and -2 them temporarily | 08:48 |
openstackgerrit | Derek Higgins proposed openstack-infra/tripleo-ci: Switch Fedora job to F21 https://review.openstack.org/172369 | 08:49 |
openstackgerrit | Derek Higgins proposed openstack-infra/tripleo-ci: Remove use of the qemu-pinned element https://review.openstack.org/175308 | 08:49 |
AJaeger | ok, let me do the patches now and I'll let you -2 them... | 08:49 |
ttx | AJaeger: your call depending on when you have the most time | 08:50 |
*** garyk1 has quit IRC | 08:50 | |
AJaeger | ttx, let me start now ;) | 08:50 |
*** garyk has joined #openstack-infra | 08:50 | |
ttx | AJaeger: although I guess we should idellay do them once the last patches are pushed | 08:50 |
*** devvesa has quit IRC | 08:50 | |
ttx | ideally | 08:50 |
*** chlong has joined #openstack-infra | 08:50 | |
ttx | so that line numbers match | 08:50 |
*** salv-orlando has joined #openstack-infra | 08:51 | |
ttx | (is that even important ?) | 08:51 |
AJaeger | ttx, I'll start with them now and can redo if there's time | 08:51 |
ttx | right | 08:51 |
AJaeger | Not really important | 08:51 |
ttx | At least I'll have something to merge if I need to | 08:51 |
AJaeger | yep | 08:51 |
ttx | ++ | 08:51 |
openstackgerrit | Jerry Zhao proposed openstack-infra/nodepool: add option to use ipv6 for image update and node launching https://review.openstack.org/156178 | 08:53 |
*** ddieterly has joined #openstack-infra | 08:55 | |
*** devvesa has joined #openstack-infra | 08:56 | |
*** BobBall_AWOL is now known as BobBall | 08:57 | |
*** e0ne has joined #openstack-infra | 08:59 | |
*** ddieterly has quit IRC | 09:00 | |
*** ihrachyshka has quit IRC | 09:00 | |
*** Hal has joined #openstack-infra | 09:02 | |
*** Hal is now known as Guest29527 | 09:03 | |
AJaeger | ttx, https://review.openstack.org/#/q/branch:stable/kilo+topic:translations,n,z - I'll ping you once I'm finished... | 09:03 |
ttx | ack | 09:03 |
*** hdd has quit IRC | 09:04 | |
openstackgerrit | Bharat Kumar Kobagana proposed openstack-infra/project-config: Skip test_volume_boot_pattern for GlusterFS until bug 1444806 is fixed https://review.openstack.org/173408 | 09:04 |
openstack | bug 1444806 in Cinder "test_volume_boot_pattern tempest test failure for glusterfs backend - Part 2" [Undecided,New] https://launchpad.net/bugs/1444806 | 09:04 |
*** tkelsey has joined #openstack-infra | 09:05 | |
*** sambetts has joined #openstack-infra | 09:06 | |
*** pblaho has joined #openstack-infra | 09:11 | |
*** ihrachyshka has joined #openstack-infra | 09:11 | |
*** e0ne is now known as e0ne_ | 09:13 | |
*** e0ne_ is now known as e0ne | 09:13 | |
*** jlanoux_ has joined #openstack-infra | 09:17 | |
*** alexpilotti has joined #openstack-infra | 09:17 | |
*** chlong has quit IRC | 09:20 | |
*** jlanoux has quit IRC | 09:20 | |
*** teran has joined #openstack-infra | 09:23 | |
*** pelix has joined #openstack-infra | 09:23 | |
*** hichihara has quit IRC | 09:23 | |
*** pcaruana has quit IRC | 09:27 | |
*** teran has quit IRC | 09:27 | |
*** teran has joined #openstack-infra | 09:27 | |
*** sushilkm has joined #openstack-infra | 09:29 | |
*** sushilkm has left #openstack-infra | 09:29 | |
*** pcaruana has joined #openstack-infra | 09:30 | |
openstackgerrit | Merged openstack-infra/project-config: Mark some python-saharaclient jobs master only https://review.openstack.org/175303 | 09:32 |
*** sergsh has quit IRC | 09:32 | |
*** chlong has joined #openstack-infra | 09:33 | |
*** fhubik is now known as fhubik_afk | 09:33 | |
*** amotoki has quit IRC | 09:35 | |
*** fhubik_afk is now known as fhubik | 09:37 | |
*** vdo_ has joined #openstack-infra | 09:44 | |
*** sergsh has joined #openstack-infra | 09:44 | |
*** tim__ has joined #openstack-infra | 09:47 | |
*** tim__ is now known as tirmc- | 09:47 | |
tirmc- | erm | 09:47 |
*** tirmc- is now known as timrc- | 09:47 | |
*** yfried__ is now known as yfried|afk | 09:48 | |
*** garyk has quit IRC | 09:48 | |
*** garyk has joined #openstack-infra | 09:49 | |
*** coolsvap is now known as coolsvap|afk | 09:51 | |
*** fhubik is now known as fhubik_afk | 09:51 | |
*** boris-42 has joined #openstack-infra | 09:52 | |
*** yfried|afk is now known as yfried__ | 09:53 | |
openstackgerrit | Renat Akhmerov proposed openstack-infra/project-config: Rename stackforge/mistral to openstack/mistral https://review.openstack.org/175328 | 09:53 |
*** hdd has joined #openstack-infra | 09:53 | |
*** fhubik_afk is now known as fhubik | 09:54 | |
*** Guest29527 has quit IRC | 09:55 | |
*** ddieterly has joined #openstack-infra | 09:56 | |
openstackgerrit | Renat Akhmerov proposed openstack-infra/project-config: Rename stackforge/mistral to openstack/mistral https://review.openstack.org/175328 | 09:59 |
AJaeger | ttx, all done besides horizon - akamito does those. Please check that I have all repos, I skipped magnum, rally, murano, congress since those have no translations setup | 10:00 |
*** andreykurilin__ has quit IRC | 10:00 | |
*** ddieterly has quit IRC | 10:00 | |
*** claudiub has quit IRC | 10:00 | |
ttx | AJaeger: do we need to cover the neutron-*aas ? | 10:02 |
AJaeger | ttx, they are not setup for translations | 10:02 |
ttx | sahara ? | 10:03 |
ttx | trove? | 10:03 |
AJaeger | Will do sahara and trove now - thakns for double checking | 10:03 |
ttx | barbican, manila ? | 10:03 |
ttx | NB: barbican doesn't have stable/kilo yet, just asking for completeness | 10:03 |
AJaeger | will do manila- barbican is setup | 10:04 |
*** kaisers has joined #openstack-infra | 10:05 | |
*** jcoufal has joined #openstack-infra | 10:07 | |
*** doude has quit IRC | 10:07 | |
*** wznoinsk_ has joined #openstack-infra | 10:07 | |
openstackgerrit | Alexey Deryugin proposed openstack-infra/project-config: [gate-murano-devstack-dsvm] Fix bug with neutron misconfiguration https://review.openstack.org/172447 | 10:08 |
*** wznoinsk has quit IRC | 10:09 | |
*** pc_m has joined #openstack-infra | 10:09 | |
AJaeger | ttx, manila, sahara, trove done | 10:09 |
ttx | thx | 10:11 |
*** dhritishikhar_ has quit IRC | 10:12 | |
*** kaisers has quit IRC | 10:14 | |
openstackgerrit | Merged openstack-infra/storyboard: One-To-One relationship for Access and Refresh Tokens https://review.openstack.org/173140 | 10:15 |
*** otter768 has joined #openstack-infra | 10:15 | |
openstackgerrit | Merged openstack-infra/storyboard: Fixed scheduler management conflict. https://review.openstack.org/173143 | 10:16 |
openstackgerrit | Merged openstack-infra/storyboard: Token Cleaner plugin now uses proper session. https://review.openstack.org/173144 | 10:16 |
*** adiantum has joined #openstack-infra | 10:17 | |
AJaeger | ttx, it's sad to see failures for such a change ;( | 10:17 |
*** kaisers has joined #openstack-infra | 10:18 | |
ttx | Heh, I got failures on a .gitreview update. Those aren't even part of the code tarball | 10:18 |
*** otter768 has quit IRC | 10:20 | |
AJaeger | ;( | 10:20 |
*** cdent has joined #openstack-infra | 10:22 | |
*** yfried__ is now known as yfried|afk | 10:25 | |
*** yfried|afk is now known as yfried__ | 10:31 | |
*** kaisers has quit IRC | 10:34 | |
boris-42 | AJaeger: hi there | 10:34 |
*** sushilkm has joined #openstack-infra | 10:35 | |
*** sushilkm has left #openstack-infra | 10:35 | |
boris-42 | AJaeger: could you help me | 10:36 |
*** dhritishikhar_ has joined #openstack-infra | 10:36 | |
boris-42 | AJaeger: I would like to add meeting bot to #openstack-rally room | 10:37 |
boris-42 | AJaeger: but I don't how it is called / where to find it.. | 10:37 |
*** funzo has joined #openstack-infra | 10:42 | |
*** mrunge has quit IRC | 10:45 | |
openstackgerrit | Bharat Kumar Kobagana proposed openstack-infra/project-config: Skip test_volume_boot_pattern for GlusterFS until bug 1444806 is fixed https://review.openstack.org/173408 | 10:46 |
openstack | bug 1444806 in Cinder "test_volume_boot_pattern tempest test failure for glusterfs backend - Part 2" [Undecided,In progress] https://launchpad.net/bugs/1444806 - Assigned to Bharat Kumar Kobagana (bharat-kobagana) | 10:46 |
*** vdo_ has quit IRC | 10:46 | |
*** fhubik is now known as fhubik_afk | 10:47 | |
*** mrunge has joined #openstack-infra | 10:48 | |
AJaeger | boris-42, http://ci.openstack.org/irc.html explains most of it | 10:48 |
AJaeger | And then there's some doku in http://docs.openstack.org/infra/manual/creators.html | 10:48 |
*** dhritishikhar_ has quit IRC | 10:49 | |
*** jcoufal_ has joined #openstack-infra | 10:50 | |
*** jcoufal has quit IRC | 10:53 | |
*** e0ne is now known as e0ne_ | 10:56 | |
*** ddieterly has joined #openstack-infra | 10:56 | |
*** e0ne_ is now known as e0ne | 10:58 | |
*** yolanda has joined #openstack-infra | 10:59 | |
*** ddieterly has quit IRC | 11:01 | |
*** fhubik_afk is now known as fhubik | 11:02 | |
*** fhubik is now known as fhubik_afk | 11:04 | |
*** e0ne is now known as e0ne_ | 11:16 | |
*** Krinkle|detached is now known as Krinkle | 11:17 | |
boris-42 | AJaeger: so thank you for links | 11:20 |
boris-42 | AJaeger: but I didn't get how to setup for my chat=) | 11:20 |
boris-42 | AJaeger: should I install meetbot on my hardware? | 11:20 |
boris-42 | AJaeger: or is there some std way for openstack projects? | 11:21 |
AJaeger | boris-42, exactly, those documents describe the standard way ;) | 11:21 |
AJaeger | for meetings you should use one of #openstack-meeting rooms | 11:21 |
boris-42 | AJaeger: and we can't use our room? like #openstack-rally? | 11:22 |
AJaeger | but if you add accessbot and gerritbot to your channel and enable logging, you might be able to use meetbot as well - but better ask somebody else from infra team for advise | 11:22 |
AJaeger | boris-42, project-config is the repo for accessbot/gerritbot, system-config the one for logging | 11:23 |
*** afazekas_ has quit IRC | 11:24 | |
*** yfried__ is now known as yfried|afk | 11:25 | |
*** ildikov has quit IRC | 11:25 | |
*** vlaza has quit IRC | 11:25 | |
*** e0ne_ has quit IRC | 11:26 | |
*** yfried|afk is now known as yfried__ | 11:26 | |
boris-42 | AJaeger: so let me just try to do something=) | 11:26 |
*** tnovacik has quit IRC | 11:26 | |
AJaeger | go ahead | 11:27 |
*** _nadya_ has joined #openstack-infra | 11:28 | |
*** afazekas_ has joined #openstack-infra | 11:28 | |
*** jcoufal_ has quit IRC | 11:28 | |
*** kashyap has joined #openstack-infra | 11:29 | |
*** BobBall has quit IRC | 11:29 | |
*** jcoufal has joined #openstack-infra | 11:29 | |
boris-42 | AJaeger: so this seems valid https://github.com/openstack-infra/project-config/blob/master/gerritbot/channels.yaml#L983-L992 | 11:30 |
boris-42 | AJaeger: gerrit bot is added properly | 11:30 |
kashyap | Is there a timeline as to when would existing projects (e.g Nova, etc) will be moved to Storyboard? | 11:31 |
openstackgerrit | Dmitry Teselkin proposed openstack-infra/zuul: Typo in processQueue https://review.openstack.org/175359 | 11:32 |
*** fhubik_afk is now known as fhubik | 11:33 | |
*** _nadya_ has quit IRC | 11:33 | |
*** adiantum has quit IRC | 11:35 | |
fbo | asselin, Hi I want to help on the openstackci effort but I need some guidances from you to start working on some tasks described in storyboard. So if you have time to discuss about that I'll be happy | 11:35 |
*** doude has joined #openstack-infra | 11:36 | |
*** doude has quit IRC | 11:37 | |
*** doude has joined #openstack-infra | 11:37 | |
mordred | kashyap: no, there is not. there will be a discussion at the summit about next steps | 11:40 |
*** BobBall has joined #openstack-infra | 11:40 | |
*** baoli has joined #openstack-infra | 11:41 | |
kashyap | mordred: Okay, I won't be there this time around, will follow the updates via interwebs. | 11:41 |
kashyap | Thanks. | 11:41 |
mordred | kashyap: sure thing! | 11:41 |
mordred | tchaypo, lifeless: actually, I'm pretty sure that the tighter version dep will _never_ be applied, regardless of mechanism of install | 11:42 |
mordred | tchaypo, lifeless: unless you give pip a -U | 11:42 |
mordred | tchaypo, lifeless: if there is already a version of the library installed | 11:42 |
mordred | (it's possible this is what you're describing, just in slightly different words) | 11:42 |
tchaypo | In my testing it was | 11:42 |
mordred | even if there was already a requests installed? | 11:43 |
mordred | oh - yes - we're describing the same thing | 11:43 |
mordred | if requirements.txt itself has requests >= 2.2.0 and also docker-py and docker-py has requests dep that is tighter | 11:44 |
tchaypo | If 2.0 is installed, and I pip install foo, and foo install_requires>=2.1, then 2.1 got installed | 11:44 |
mordred | _really_ | 11:44 |
mordred | that is not what I would expect from my experience | 11:44 |
mordred | so it seems it's even more crazy than I thought | 11:44 |
*** fhubik_afk has joined #openstack-infra | 11:45 | |
tchaypo | Now you've got me out of bed so I can verify this again | 11:45 |
mordred | tchaypo: dangit | 11:45 |
mordred | tchaypo: you go back to bed | 11:45 |
*** kashyap has left #openstack-infra | 11:45 | |
mordred | tchaypo: I just know we've gotten screwed before by doing a pip install . of a new commit of zuul | 11:45 |
mordred | where zuul had a requirements version bump | 11:45 |
mordred | on a library | 11:45 |
mordred | and that did not get upgraded | 11:46 |
*** fhubik_afk has quit IRC | 11:46 | |
*** baoli has quit IRC | 11:46 | |
*** woodster_ has joined #openstack-infra | 11:46 | |
*** fhubik_afk has joined #openstack-infra | 11:46 | |
*** fhubik_afk is now known as fhubik_meeting | 11:46 | |
*** Hal has joined #openstack-infra | 11:46 | |
tchaypo | https://www.irccloud.com/pastebin/70kutYTy | 11:46 |
*** fhubik has quit IRC | 11:46 | |
*** Hal is now known as Guest72462 | 11:47 | |
mordred | tchaypo: well then. I stand corrected | 11:47 |
tchaypo | vs | 11:47 |
tchaypo | https://www.irccloud.com/pastebin/FnLqXH7G | 11:47 |
mordred | tchaypo: WOW | 11:48 |
mordred | *brainsplodes* | 11:48 |
tchaypo | so if zuul’s install_requires was bumped but we installed by pointing it at a requirements.txt, that would lead to screwage | 11:48 |
mordred | wow | 11:48 |
tchaypo | it’s also *possible* that the test packages I’m using here are screwy | 11:48 |
mordred | seriously | 11:48 |
mordred | no | 11:48 |
mordred | I mean | 11:48 |
mordred | wow | 11:48 |
mordred | ok - I need to process that - it would explain _SO_MANY_ISSUES_ | 11:49 |
tchaypo | you can see the sources at https://github.com/jamezpolley/161047-foo and https://github.com/jamezpolley/161047-bar and https://github.com/jamezpolley/161047-requirements | 11:49 |
tchaypo | while you’re here | 11:49 |
tchaypo | i was scrolling back through the weekend and i saw you talking a bit more about this and referring to a conversation on distutils-sig (which I haven’t seen) | 11:49 |
mordred | tchaypo, lifeless: this means I believe the way we're handling this in pbr is flat wrong | 11:49 |
tchaypo | and if I understood correctly, your position on tight pinning seems to be driven by wanting to provide one obvious way for end users to install the packages | 11:50 |
*** _nadya_ has joined #openstack-infra | 11:50 | |
mordred | yes | 11:50 |
tchaypo | I think that’s something we do need to cater for as a primary use case | 11:50 |
*** fhubik_meeting has quit IRC | 11:50 | |
mordred | as in - I think making "pip install foo" needs to work and if that means removing some flexibility from folks who want to inject other deps, I think that's the tradeoff | 11:50 |
tchaypo | but it seems like it would be good if we can also avoid making life more difficult than it has to be for distro maintainers | 11:50 |
mordred | if we can get both - awesome | 11:51 |
mordred | sure | 11:51 |
mordred | I agree | 11:51 |
tchaypo | good | 11:51 |
tchaypo | :) | 11:51 |
tchaypo | i like it when we agree | 11:51 |
mordred | I also think the distro maintainers question is a red herring a bit though | 11:51 |
mordred | because there is nothing in our pinning that affects them | 11:51 |
mordred | since they don't consume it | 11:51 |
*** deepakcs has quit IRC | 11:51 | |
mordred | and while they _Could_ look at our requirements, see it says requests==2.2.0 and think they can't make a package that depends on 2.3.0 | 11:52 |
tchaypo | When you say “they don’t consume it”, I think you mean “we suggest that people preparing packages disable pbr, so the things pbr does to read requirements.txt into install_requires shouldn’t affect them" | 11:52 |
mordred | what we're saying is "we assert that 2.2.0 works and we don't know that anything else works" | 11:53 |
mordred | tchaypo: yes | 11:53 |
mordred | tchaypo: and by | 11:53 |
mordred | tchaypo: and by suggest | 11:53 |
mordred | I mean that distro packaging cannot work with pbr enabled | 11:53 |
mordred | they all disable it | 11:53 |
mordred | or, with pbr pip installation enabled | 11:53 |
tchaypo | except for the person i was talking to last week who said they maintain gentoo packages and do use pbr to prepare them | 11:53 |
mordred | they disable it by either setting the env var to false, or to just patching pbr out | 11:54 |
mordred | so, they let pbr download pip requirements? | 11:54 |
tchaypo | I didn’t have time to go into that | 11:54 |
tchaypo | but I’m going to assume that you know more about this than I do, since my exposure to people who maintain packages of openstack stuff consists of one 10 minute conversation with one person... | 11:55 |
mordred | they should be setting http://docs.openstack.org/developer/pbr/packagers.html#dependencies SKIP_PIP_INSTALL=1 in their build scripts | 11:55 |
mordred | tchaypo: :) | 11:55 |
tchaypo | getting back to an earlier point | 11:55 |
mordred | tchaypo: your original point is totally valid | 11:55 |
mordred | we should not make distro maintainers lives hard | 11:55 |
tchaypo | if the distro maintainers issue is less important because we expect them to disable pip | 11:55 |
mordred | however, I mainly want to contend that it's not our job to do their job | 11:55 |
tchaypo | and our primary desire is to make “pip install nova” come up with a known-sane set of packages | 11:55 |
tchaypo | then I feel like our proposed solution of putting pip freeze output $SOMEWHERE is wrong | 11:56 |
*** mmedvede has quit IRC | 11:56 | |
openstackgerrit | Boris Pavlovic proposed openstack-infra/system-config: Add #openstack-rally to eavesdrop https://review.openstack.org/175365 | 11:56 |
mordred | yah | 11:56 |
boris-42 | AJaeger: ^ so one line was missing=) | 11:56 |
mordred | tchaypo: although - I think it is still a helpful thing to do | 11:56 |
tchaypo | btw - I should mention that in the output I pasted you earlier, those packages *aren’t* using pbr at all, they have dependencies hard-coded in install_requires in setup.py | 11:57 |
AJaeger | boris-42, ah... | 11:57 |
mordred | tchaypo: because having the ability to go back and reconstruct what a commit was tested against is useful | 11:57 |
*** hdd has quit IRC | 11:57 | |
mordred | tchaypo: yah - we do the depend injection a little differently - which I think means we have a bug | 11:57 |
*** jistr has quit IRC | 11:57 | |
mordred | tchaypo: what pbr should be doing is making it seem as if the things in requirements.txt were in install_requires | 11:58 |
tchaypo | but since my understanding is that pbr just stuffs the contents of requirements.txt into install_requires I’m (perhaps naively) not expecting any differences between what I did, and what pbr would do if those requirements were in requirements.txt | 11:58 |
mordred | but we're bypassing that at the moment possibly at the wrong point in the stack | 11:58 |
*** chlong has quit IRC | 11:58 | |
mordred | yes. that is what I also thing | 11:58 |
mordred | think | 11:58 |
*** afazekas_ has quit IRC | 11:58 | |
tchaypo | oh wow. | 11:58 |
mordred | the big problem is that we ignore install_requires itself to avoid easy_install invocation | 11:58 |
mordred | which is _Bad_ | 11:58 |
*** vlaza has joined #openstack-infra | 11:59 | |
tchaypo | so if I’d put those into requirements.txt and used pbr, even the “pip -r requirements.txt” case would result in bar being upgraded? | 11:59 |
mordred | but I think instead of what we're doing, which is re-invoking pip ourselves | 11:59 |
*** kaisers has joined #openstack-infra | 11:59 | |
mordred | tchaypo: no, I think the opposite | 11:59 |
mordred | tchaypo: I think we're increasing the number of cases where bar does not get upgraded | 11:59 |
*** fhubik has joined #openstack-infra | 11:59 | |
tchaypo | oh. | 11:59 |
mordred | I'm thinking what we may need to do instead of re-invoking pip ourselves | 12:00 |
mordred | is to hack into the egg_info command | 12:00 |
*** fhubik is now known as fhubik_meeting | 12:00 | |
mordred | so that we produce a foo.egg-info/requires.txt that has the things we want | 12:00 |
mordred | which is what you'd get if you put the thing into install_requires in setup.py and then ran egg_info | 12:00 |
tchaypo | I am sad to say that I’ve never before looked into the egg format enough to know what foo.egg-info/requires.txt is and how it’s different from install_requires | 12:01 |
tchaypo | and sadder to say that I expect to be learning this tomorrow | 12:01 |
mordred | tchaypo: http://git.openstack.org/cgit/openstack-dev/pbr/tree/pbr/packaging.py#n90 | 12:01 |
mordred | tchaypo: this is the function where we're overriding some behavior that I think we actually should not be | 12:01 |
tchaypo | I feel like it would also be *strongly* in our interest to investigate changing pip’s behaviour | 12:02 |
mordred | tchaypo: I'll try to write up a few notes later in my day to help point you in a few directions that I think would be worthy of investigation for this | 12:02 |
mordred | tchaypo: well, I think that's the path lifeless has been embarking on | 12:02 |
tchaypo | at least in so far as making sure that if requirements.txt says >=2.0 and foo says >=2.1 pip goes with the stronger requirement | 12:02 |
*** SergK has quit IRC | 12:02 | |
*** jistr has joined #openstack-infra | 12:03 | |
mordred | I think the thing is | 12:03 |
mordred | that pip doesn't see those files the way we use them | 12:03 |
mordred | which is why dstufft has been telling us we're using them wrong | 12:03 |
*** ihrachyshka has quit IRC | 12:03 | |
tchaypo | yeah, I agree with that | 12:03 |
mordred | I think they are not intended from pip's pov to be dependencies of a thing | 12:03 |
mordred | as much as a list of top-level things you want to install yourself | 12:04 |
mordred | and so the behavior for each of them is as if you've run pip install on them on the command line | 12:04 |
*** kgiusti has joined #openstack-infra | 12:04 | |
mordred | whereas if we put them into install_requires properly, pip should see them as _dependencies_ | 12:04 |
*** mmedvede has joined #openstack-infra | 12:04 | |
mordred | so - this is a disturbing morning | 12:04 |
*** SergK has joined #openstack-infra | 12:04 | |
tchaypo | that’s my understanding, in theory, and I’d like to go towards that | 12:05 |
mordred | but thank you - because I think I finally grok something I did not before | 12:05 |
mordred | tchaypo: ++ | 12:05 |
tchaypo | I think the stuff lifeless suggested about being able to seperate install_requires from requirements.txt is part of it, and will bring pbr closer to pythonic expectations | 12:05 |
mordred | yah | 12:05 |
*** ildikov has joined #openstack-infra | 12:05 | |
tchaypo | oooh, let me try something | 12:05 |
mordred | tchaypo: fwiw, I tink we should remove this: http://git.openstack.org/cgit/openstack-dev/pbr/tree/pbr/packaging.py#n195 | 12:06 |
tchaypo | okay, that was unexpected | 12:06 |
tchaypo | pip install bar>=2.0 foo | 12:06 |
tchaypo | results in nothing. | 12:07 |
tchaypo | i thought i could do multiple packages | 12:07 |
*** rfolco has joined #openstack-infra | 12:07 | |
*** vlaza has quit IRC | 12:08 | |
*** vlaza1 has joined #openstack-infra | 12:08 | |
openstackgerrit | Boris Pavlovic proposed openstack-infra/project-config: Add new rally-ci project https://review.openstack.org/166962 | 12:08 |
*** bswartz has quit IRC | 12:08 | |
tchaypo | oh. silly me. | 12:08 |
tchaypo | Bash helpfully sent output to a file named '=2.0' | 12:09 |
mordred | hahahaha | 12:09 |
mordred | :) | 12:09 |
mordred | tchaypo: yeah - that line I just pasted - I actually think it might be as simple as removing that LocalInstall class altogether | 12:09 |
tchaypo | fwiw | 12:10 |
mordred | tchaypo: we actually already inject the depends into install_requires | 12:10 |
tchaypo | pip install 'bar>=2.0' foo | 12:10 |
tchaypo | does the same as the requirements.txt example - foo gets installed, bar is left at 2.0 | 12:10 |
* mgedmin wonders how often people use pip install foo[extra] without realizing that [ and ] define a glob pattern that might match one day | 12:10 | |
mordred | yah | 12:10 |
mordred | mgedmin: :) | 12:10 |
*** kaisers has quit IRC | 12:10 | |
tchaypo | When I see docs that use < > to surround a sample value that needs to be replaced, I cry | 12:11 |
tchaypo | I’ve learnt to check for them before I paste, but even so I miss them occasionally | 12:12 |
* tchaypo stared at http://git.openstack.org/cgit/openstack-dev/pbr/tree/pbr/packaging.py#n195 for a bit, but 10pm + leaving the glasses beside my bed == no comprende. I’ve made a note to take a look in the morning. | 12:13 | |
mordred | ++ | 12:13 |
tchaypo | It does look like, if I wanted a simple way to test what happens if that code is removed, I can just change the condition on line 215 and make it always skip the _pip_install | 12:13 |
*** tiswanso has joined #openstack-infra | 12:14 | |
tchaypo | … or just change 217 to pass | 12:14 |
*** _nadya_ has quit IRC | 12:14 | |
openstackgerrit | Tristan Cacqueray proposed openstack-infra/puppet-jenkins: Use jenkins group instead of nogroup https://review.openstack.org/175368 | 12:14 |
tchaypo | anyway, bedtime. I’m looking forward to this discussion happening in a room in the place we’re going to | 12:14 |
openstackgerrit | Monty Taylor proposed openstack-dev/pbr: Stop re-invoking pip https://review.openstack.org/175369 | 12:15 |
mordred | tchaypo: ^^ | 12:15 |
tchaypo | I’d forgotten it was vancouver for a moment | 12:15 |
mordred | tchaypo: try testing with that when you wake up | 12:15 |
* mordred must also run now | 12:15 | |
*** tiswanso has quit IRC | 12:16 | |
*** tiswanso has joined #openstack-infra | 12:16 | |
*** otter768 has joined #openstack-infra | 12:16 | |
*** ihrachyshka has joined #openstack-infra | 12:19 | |
*** ihrachyshka has quit IRC | 12:19 | |
*** ihrachyshka has joined #openstack-infra | 12:19 | |
*** otter768 has quit IRC | 12:21 | |
*** adiantum has joined #openstack-infra | 12:22 | |
*** hdd has joined #openstack-infra | 12:24 | |
*** bknudson has quit IRC | 12:26 | |
*** e0ne has joined #openstack-infra | 12:26 | |
*** salv-orlando has quit IRC | 12:28 | |
*** nfedotov has joined #openstack-infra | 12:28 | |
*** sarob has joined #openstack-infra | 12:30 | |
*** arxcruz has quit IRC | 12:30 | |
*** jamielennox|away is now known as jamielennox | 12:31 | |
*** dprince has joined #openstack-infra | 12:31 | |
*** jamielennox is now known as jamielennox|away | 12:32 | |
*** hrubi has quit IRC | 12:32 | |
*** yolanda has quit IRC | 12:32 | |
*** hrubi has joined #openstack-infra | 12:33 | |
*** jaypipes has joined #openstack-infra | 12:33 | |
*** _nadya_ has joined #openstack-infra | 12:33 | |
*** sarob has quit IRC | 12:34 | |
*** erlon has joined #openstack-infra | 12:34 | |
*** BharatK has quit IRC | 12:36 | |
*** gordc has joined #openstack-infra | 12:37 | |
*** baoli has joined #openstack-infra | 12:38 | |
*** dhritishikhar has joined #openstack-infra | 12:38 | |
*** rlandy has joined #openstack-infra | 12:39 | |
*** prad has quit IRC | 12:40 | |
*** prad has joined #openstack-infra | 12:40 | |
*** yolanda has joined #openstack-infra | 12:42 | |
*** arxcruz has joined #openstack-infra | 12:42 | |
*** tiswanso has quit IRC | 12:45 | |
*** ihrachyshka has quit IRC | 12:46 | |
*** zz_dimtruck is now known as dimtruck | 12:46 | |
*** timrc- has quit IRC | 12:49 | |
*** esker has quit IRC | 12:52 | |
*** baoli has quit IRC | 12:54 | |
*** cdent has quit IRC | 12:54 | |
*** dhritishikhar has quit IRC | 12:54 | |
*** AJaeger has quit IRC | 12:54 | |
*** baoli has joined #openstack-infra | 12:54 | |
*** dhritishikhar has joined #openstack-infra | 12:54 | |
*** bknudson has joined #openstack-infra | 12:55 | |
*** bswartz has joined #openstack-infra | 12:55 | |
*** hdd_ has joined #openstack-infra | 12:55 | |
*** sdake has joined #openstack-infra | 12:56 | |
*** salv-orlando has joined #openstack-infra | 12:56 | |
*** mrunge has quit IRC | 12:56 | |
*** dimtruck is now known as zz_dimtruck | 12:56 | |
*** doug-fish has joined #openstack-infra | 12:57 | |
*** ddieterly has joined #openstack-infra | 12:58 | |
*** hdd has quit IRC | 12:58 | |
*** ihrachyshka has joined #openstack-infra | 12:58 | |
openstackgerrit | Merged openstack-infra/project-config: [gate-murano-devstack-dsvm] Fix bug with neutron misconfiguration https://review.openstack.org/172447 | 12:59 |
*** baoli has quit IRC | 12:59 | |
*** baoli has joined #openstack-infra | 13:00 | |
*** dizquierdo has joined #openstack-infra | 13:01 | |
*** sks has quit IRC | 13:02 | |
*** ddieterly has quit IRC | 13:03 | |
*** Krinkle is now known as Krinkle|detached | 13:03 | |
*** bcourt has joined #openstack-infra | 13:04 | |
*** lifeless has quit IRC | 13:05 | |
*** zhiwei has joined #openstack-infra | 13:05 | |
teran | Hello infra folks, I have the user Denis Meltsaykin <dmeltsaykin@mirantis.com> in review.openstack.org with two accounts(with the same name and email), could you help him please? | 13:05 |
*** dhritishikhar has quit IRC | 13:07 | |
*** fhubik_ has joined #openstack-infra | 13:08 | |
*** wznoinsk_ has quit IRC | 13:08 | |
*** vlaza1 has quit IRC | 13:09 | |
*** vlaza has joined #openstack-infra | 13:09 | |
fungi | teran: i'll take a look and see what can be done | 13:09 |
teran | fungi: thanks! | 13:09 |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: redo devstack-gate grenade selection as a case statement https://review.openstack.org/175390 | 13:10 |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: add support for stable/kilo https://review.openstack.org/175391 | 13:10 |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: add support for stable/kilo https://review.openstack.org/175391 | 13:11 |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: redo devstack-gate grenade selection as a case statement https://review.openstack.org/175390 | 13:12 |
*** timrc- has joined #openstack-infra | 13:12 | |
sdague | fungi: opinions on - https://review.openstack.org/#/c/175390/ would be appreciated, it seemed a bit cleaner to me | 13:12 |
*** fhubik_meeting has quit IRC | 13:12 | |
*** sks has joined #openstack-infra | 13:16 | |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: add support for stable/kilo https://review.openstack.org/175391 | 13:18 |
*** fhubik_ is now known as fhubik | 13:18 | |
openstackgerrit | Sergey Lukjanov proposed openstack-infra/project-config: Add all Sahara images build to check pipeline as non-voting https://review.openstack.org/173939 | 13:20 |
openstackgerrit | Sergey Lukjanov proposed openstack-infra/project-config: Setup non-voting job for sahara-extra to build artifacts https://review.openstack.org/175395 | 13:20 |
*** isviridov_away is now known as isviridov | 13:20 | |
*** dhritishikhar has joined #openstack-infra | 13:22 | |
anteaya | sdague: I could follow both patches | 13:24 |
anteaya | sdague: how long do we keep icehouse support | 13:24 |
sdague | anteaya: mid summer some time IIRC | 13:26 |
*** miqui has joined #openstack-infra | 13:26 | |
*** peristeri has joined #openstack-infra | 13:27 | |
anteaya | k, thanks | 13:27 |
tristanC | iirc, Icehouse support is 15 months | 13:27 |
*** zz_dimtruck is now known as dimtruck | 13:28 | |
anteaya | tristanC: great, thank you | 13:29 |
*** sslypushenko has quit IRC | 13:30 | |
*** superdan is now known as dansmith | 13:31 | |
*** radez_g0n3 is now known as radez | 13:34 | |
*** annegentle has joined #openstack-infra | 13:35 | |
*** jistr_ has joined #openstack-infra | 13:35 | |
*** claudiub has joined #openstack-infra | 13:36 | |
*** tnovacik has joined #openstack-infra | 13:38 | |
*** jistr has quit IRC | 13:38 | |
*** esker has joined #openstack-infra | 13:40 | |
*** russellb has quit IRC | 13:40 | |
*** doude_ has joined #openstack-infra | 13:41 | |
*** julim has joined #openstack-infra | 13:41 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 13:42 | |
*** adrian_otto has joined #openstack-infra | 13:43 | |
*** dboik has joined #openstack-infra | 13:44 | |
*** jistr_ has quit IRC | 13:44 | |
*** jistr has joined #openstack-infra | 13:44 | |
*** tnovacik has quit IRC | 13:44 | |
*** dboik has quit IRC | 13:45 | |
*** mattfarina has joined #openstack-infra | 13:45 | |
*** dboik has joined #openstack-infra | 13:45 | |
*** russellb has joined #openstack-infra | 13:45 | |
*** sarob has joined #openstack-infra | 13:45 | |
*** dhritishikhar has quit IRC | 13:46 | |
*** dhritishikhar has joined #openstack-infra | 13:46 | |
*** mriedem has joined #openstack-infra | 13:46 | |
*** ddieterly has joined #openstack-infra | 13:47 | |
*** samuelBartel has joined #openstack-infra | 13:47 | |
samuelBartel | Hi all | 13:47 |
samuelBartel | I have a question about the creation of a new project on stackforge | 13:48 |
*** doude has quit IRC | 13:48 | |
*** claudiub has quit IRC | 13:48 | |
anteaya | samuelBartel: hello | 13:48 |
*** doude_ has quit IRC | 13:48 | |
anteaya | samuelBartel: this is a good guide for creating a project: http://docs.openstack.org/infra/manual/creators.html | 13:49 |
samuelBartel | anteaya, hello | 13:49 |
anteaya | its aim is for a project into the openstack space, but many of the steps are the same | 13:49 |
anteaya | you don't have to submit a patch to the governance repo for a stackforge project | 13:49 |
anteaya | and most of the things are documented what you need and what are options | 13:50 |
*** eharney has joined #openstack-infra | 13:50 | |
anteaya | I suggest you begin with this guide and work your way through asking questions as they come up | 13:50 |
anteaya | and having a patch to reference when you ask is always helpful for us | 13:50 |
*** sarob has quit IRC | 13:50 | |
anteaya | and welcome | 13:50 |
*** ddieterly has quit IRC | 13:52 | |
samuelBartel | anteaya, thanks a lot, I should be able to manage with it | 13:52 |
anteaya | samuelBartel: it is a well written guide | 13:52 |
anteaya | samuelBartel: dhellmann did the majority of the work on it | 13:52 |
*** amitgandhinz has joined #openstack-infra | 13:53 | |
*** e0ne is now known as e0ne_ | 13:53 | |
*** amitgandhinz has quit IRC | 13:54 | |
*** amitgandhinz has joined #openstack-infra | 13:54 | |
*** kaisers has joined #openstack-infra | 13:58 | |
*** wznoinsk has joined #openstack-infra | 13:58 | |
*** fhubik has quit IRC | 13:59 | |
*** sdake has quit IRC | 13:59 | |
*** garyk has quit IRC | 14:00 | |
*** garyk has joined #openstack-infra | 14:00 | |
*** e0ne_ is now known as e0ne | 14:02 | |
*** dhritishikhar has quit IRC | 14:03 | |
*** jgrimm is now known as zz_jgrimm | 14:04 | |
*** timrc- has quit IRC | 14:04 | |
*** smccully has joined #openstack-infra | 14:04 | |
EmilienM | maybe I've miss the topic a long time ago, but is there any objection to install vagrant on nodepool? | 14:05 |
*** sdake has joined #openstack-infra | 14:05 | |
*** dboik_ has joined #openstack-infra | 14:06 | |
*** tonytan4ever has joined #openstack-infra | 14:06 | |
*** doude has joined #openstack-infra | 14:06 | |
nibalizer | EmilienM: what do you want to do with it? virt on virt? | 14:08 |
*** ajmiller has joined #openstack-infra | 14:08 | |
*** dboik has quit IRC | 14:08 | |
*** mtanino has joined #openstack-infra | 14:08 | |
EmilienM | nibalizer: it's about our Puppet beaker jobs. I would like to run vagrant so I can test Ubuntu *AND* CentOS | 14:08 |
*** sks has quit IRC | 14:08 | |
*** mwagner_lap has joined #openstack-infra | 14:09 | |
*** doude has quit IRC | 14:09 | |
*** samuelBartel has quit IRC | 14:10 | |
*** doude has joined #openstack-infra | 14:10 | |
*** doude_ has joined #openstack-infra | 14:10 | |
nibalizer | i think you can just get a centos node | 14:10 |
nibalizer | they are available | 14:11 |
nibalizer | the feedback sdauge gave me once was to get the testing where I wanted it for ubuntu, then spread it to centos | 14:11 |
openstackgerrit | Louis Taylor proposed openstack-infra/project-config: Add gerrit notifications to WSME https://review.openstack.org/175416 | 14:12 |
*** doude_ has quit IRC | 14:12 | |
greghaynes | EmilienM: Yes, make jobs with the node types you want. The alternative is youre going to be doing nested virt which is unbearably slow | 14:15 |
annegentle | morning, jeblair are you around? Or anyone with docs site ftp creds they can call me with? | 14:15 |
annegentle | Want to hand them over to Lana and I also need to do some deletions and don't have the password stored | 14:16 |
jeblair | annegentle: lemme see if i have them | 14:16 |
*** tnovacik has joined #openstack-infra | 14:17 | |
*** sabeen1 has joined #openstack-infra | 14:17 | |
EmilienM | greghaynes, nibalizer: ack | 14:17 |
*** otter768 has joined #openstack-infra | 14:17 | |
annegentle | jeblair: thanks | 14:17 |
*** vlaza1 has joined #openstack-infra | 14:17 | |
*** vlaza has quit IRC | 14:18 | |
*** afazekas_ has joined #openstack-infra | 14:18 | |
jeblair | annegentle: yep i can call you with them. i seem to have misplaced your number though, so could you please pm it to me? | 14:19 |
EmilienM | nibalizer, greghaynes: there is no bare-centos7, only centos6. For centos7, I only find devstack-centos7. Can I do something? | 14:19 |
*** dizquierdo has quit IRC | 14:20 | |
annegentle | jeblair: yep | 14:20 |
*** dizquierdo has joined #openstack-infra | 14:20 | |
greghaynes | EmilienM: use the devstack-centos7? | 14:21 |
EmilienM | greghaynes: well, I'm not sure this is a good idea. Is the image clean? | 14:21 |
EmilienM | why not having bare-centos7 for consistency? | 14:21 |
*** sdake has quit IRC | 14:21 | |
fungi | EmilienM: i'm working on that, but recall that the devstack-.* images are actually a lot more "clean" than the poorly-named bare-.* ones | 14:22 |
*** otter768 has quit IRC | 14:22 | |
fungi | we preinstall lots of stuff on our bare-.* workers and almost nothing on our devstack-.* workers (because it's assumed that a job is going to have devstack install things instead) | 14:22 |
*** sdake has joined #openstack-infra | 14:23 | |
EmilienM | fungi: ack. | 14:23 |
greghaynes | Yes, I think the right thing is to use all devstack- if you can | 14:23 |
fungi | getting rid of the bare-.* workers and renaming the devstack-.* workers to something less confusing is in the works | 14:23 |
greghaynes | or at least thats what I was told last time I did this ;) | 14:23 |
nibalizer | and what EmilienM is doing is much closer to running devstack anyway | 14:23 |
fungi | i've already got an experimental job which successfully runs nova's unit tests on a devstack-trusty worker, for example | 14:23 |
EmilienM | fungi: good to know | 14:24 |
*** vlaza has joined #openstack-infra | 14:24 | |
*** vlaza1 has quit IRC | 14:24 | |
*** sdake_ has joined #openstack-infra | 14:24 | |
*** mdenny has joined #openstack-infra | 14:25 | |
*** garyk has quit IRC | 14:26 | |
*** sputnik13 has quit IRC | 14:26 | |
*** garyk has joined #openstack-infra | 14:26 | |
*** peristeri has quit IRC | 14:26 | |
*** afazekas_ has quit IRC | 14:26 | |
*** peristeri has joined #openstack-infra | 14:26 | |
*** dboik_ has quit IRC | 14:27 | |
*** dboik has joined #openstack-infra | 14:27 | |
*** ayoung has joined #openstack-infra | 14:28 | |
*** emagana has joined #openstack-infra | 14:28 | |
*** sdake has quit IRC | 14:28 | |
*** dhritishikhar has joined #openstack-infra | 14:29 | |
*** afazekas_ has joined #openstack-infra | 14:30 | |
*** vlaza has quit IRC | 14:31 | |
*** vlaza has joined #openstack-infra | 14:31 | |
*** andrey-mp has joined #openstack-infra | 14:31 | |
*** lifeless has joined #openstack-infra | 14:32 | |
*** alevine has joined #openstack-infra | 14:32 | |
*** achanda has joined #openstack-infra | 14:32 | |
*** samuelBartel has joined #openstack-infra | 14:33 | |
*** andrey-mp has quit IRC | 14:34 | |
*** thedodd has joined #openstack-infra | 14:34 | |
*** alevine has quit IRC | 14:35 | |
*** afazekas_ has quit IRC | 14:36 | |
openstackgerrit | Louis Taylor proposed openstack-infra/project-config: Add gerrit notifications to WSME https://review.openstack.org/175416 | 14:37 |
openstackgerrit | Louis Taylor proposed openstack-infra/project-config: Add #wsme to the list of irc channels https://review.openstack.org/175424 | 14:37 |
*** vlaza1 has joined #openstack-infra | 14:38 | |
*** vlaza has quit IRC | 14:38 | |
*** stevemar has joined #openstack-infra | 14:39 | |
*** achanda has quit IRC | 14:39 | |
*** adrian_otto has quit IRC | 14:41 | |
*** sushilkm has joined #openstack-infra | 14:41 | |
*** sushilkm has left #openstack-infra | 14:41 | |
asselin | fbo, glad to hear it! I will be happy to help! | 14:43 |
*** sdake has joined #openstack-infra | 14:44 | |
*** dguitarbite has quit IRC | 14:44 | |
*** gordc has quit IRC | 14:44 | |
*** emagana has quit IRC | 14:44 | |
*** emagana has joined #openstack-infra | 14:45 | |
*** gordc has joined #openstack-infra | 14:46 | |
openstackgerrit | Matt Riedemann proposed openstack-infra/elastic-recheck: Add query for neutron functional test bug 1446261 https://review.openstack.org/175432 | 14:47 |
openstack | bug 1446261 in neutron "gate-neutron-dsvm-functional race fails HA/DVR tests with network namespace not found" [Undecided,New] https://launchpad.net/bugs/1446261 | 14:47 |
*** afazekas_ has joined #openstack-infra | 14:47 | |
*** ddieterly has joined #openstack-infra | 14:47 | |
*** sdake_ has quit IRC | 14:47 | |
*** jlanoux_ has quit IRC | 14:47 | |
*** dangers_away is now known as dangers | 14:48 | |
openstackgerrit | Ihar Hrachyshka proposed openstack-infra/project-config: Disable all python34 jobs for stable/icehouse https://review.openstack.org/175433 | 14:48 |
*** russellb has quit IRC | 14:48 | |
*** asselin_ has joined #openstack-infra | 14:50 | |
*** hdd_ has quit IRC | 14:50 | |
*** vlaza1 is now known as vlaza | 14:53 | |
*** cdent has joined #openstack-infra | 14:53 | |
*** russellb has joined #openstack-infra | 14:53 | |
samuelBartel | anteaya, a question about the documentation, "In your commit message, include a reference to the governance change to make your project-config change depend on it:" do you know where I can find such an ID? | 14:53 |
*** sslypushenko has joined #openstack-infra | 14:54 | |
openstackgerrit | Ryan Carey proposed openstack-infra/jenkins-job-builder: Add support for Artifactory plugin https://review.openstack.org/173932 | 14:55 |
openstackgerrit | Emilien Macchi proposed openstack-infra/project-config: puppet: run beaker on both trusty & centos7 https://review.openstack.org/175434 | 14:55 |
*** nelsnelson has joined #openstack-infra | 14:55 | |
EmilienM | fungi, nibalizer: really WIP but any feedback is welcome | 14:56 |
*** annegentle has quit IRC | 14:57 | |
anteaya | samuelBartel: you are creating a stackforge project, you will not be submitted a patch to the governance repo | 14:58 |
anteaya | samuelBartel: you may safely skip that step | 14:58 |
samuelBartel | anteaya, ok thank you | 14:59 |
*** annegentle has joined #openstack-infra | 14:59 | |
anteaya | samuelBartel: you're welcome | 15:00 |
*** MarkAtwood has joined #openstack-infra | 15:00 | |
openstackgerrit | Merged openstack-infra/project-config: Add all Sahara images build to check pipeline as non-voting https://review.openstack.org/173939 | 15:01 |
openstackgerrit | Merged openstack-infra/project-config: Setup non-voting job for sahara-extra to build artifacts https://review.openstack.org/175395 | 15:01 |
*** tonytan4ever has quit IRC | 15:01 | |
*** BharatK has joined #openstack-infra | 15:01 | |
*** rwsu has joined #openstack-infra | 15:02 | |
openstackgerrit | Merged openstack-infra/project-config: upgrade ec2api jobs https://review.openstack.org/173945 | 15:03 |
*** marun has joined #openstack-infra | 15:03 | |
*** annegentle has quit IRC | 15:04 | |
*** zz_jgrimm is now known as jgrimm | 15:04 | |
*** annegentle has joined #openstack-infra | 15:05 | |
*** achanda has joined #openstack-infra | 15:05 | |
*** annegentle has quit IRC | 15:06 | |
BharatK | Hi sdague, Please review the patch https://review.openstack.org/#/c/173408/ | 15:06 |
*** rainya has left #openstack-infra | 15:06 | |
BharatK | sdague: Please let me know if any modifications required. | 15:06 |
*** vlaza is now known as vlaza_offline | 15:07 | |
*** achanda has quit IRC | 15:07 | |
*** afazekas_ has quit IRC | 15:08 | |
*** mrmartin has joined #openstack-infra | 15:08 | |
openstackgerrit | Victor Stinner proposed openstack-infra/project-config: oslo.messaging: enable Python 3.4 check https://review.openstack.org/172221 | 15:08 |
*** hdd_ has joined #openstack-infra | 15:09 | |
*** annegentle has joined #openstack-infra | 15:10 | |
*** beekneemech is now known as bnemec | 15:10 | |
*** e0ne is now known as e0ne_ | 15:10 | |
*** tnovacik has quit IRC | 15:11 | |
*** zhiwei has left #openstack-infra | 15:13 | |
*** jistr has quit IRC | 15:14 | |
openstackgerrit | Jan Hruban proposed openstack-infra/jenkins-job-builder: Allow inclusion of multiple files https://review.openstack.org/171591 | 15:15 |
*** arxcruz has quit IRC | 15:16 | |
openstackgerrit | Ihar Hrachyshka proposed openstack-infra/project-config: Disable all python34 jobs for stable/icehouse https://review.openstack.org/175433 | 15:16 |
*** tonytan4ever has joined #openstack-infra | 15:16 | |
*** e0ne_ is now known as e0ne | 15:17 | |
*** annegentle has quit IRC | 15:18 | |
*** Guest72462 has quit IRC | 15:20 | |
*** afazekas_ has joined #openstack-infra | 15:21 | |
*** armax_ has joined #openstack-infra | 15:22 | |
*** weshay has quit IRC | 15:23 | |
*** hdd_ has quit IRC | 15:23 | |
*** flip214 has quit IRC | 15:24 | |
*** weshay has joined #openstack-infra | 15:24 | |
*** rfolco has quit IRC | 15:24 | |
*** harlowja_at_home has joined #openstack-infra | 15:24 | |
openstackgerrit | Victor Stinner proposed openstack-infra/project-config: oslo.messaging: enable Python 3.4 check https://review.openstack.org/172221 | 15:24 |
*** dizquierdo has quit IRC | 15:24 | |
*** rfolco has joined #openstack-infra | 15:25 | |
*** ZZelle has quit IRC | 15:26 | |
*** rfolco has quit IRC | 15:26 | |
*** ZZelle has joined #openstack-infra | 15:26 | |
*** afazekas_ has quit IRC | 15:26 | |
*** ihrachyshka has quit IRC | 15:27 | |
*** rfolco has joined #openstack-infra | 15:27 | |
*** MaxV has quit IRC | 15:28 | |
*** jistr has joined #openstack-infra | 15:28 | |
openstackgerrit | Wayne Warren proposed openstack-infra/jenkins-job-builder: Update 'timeout' wrapper module https://review.openstack.org/129467 | 15:29 |
*** dkranz has quit IRC | 15:30 | |
openstackgerrit | Matt Riedemann proposed openstack-infra/elastic-recheck: Add query for neutron functional test bug 1446261 https://review.openstack.org/175432 | 15:30 |
openstack | bug 1446261 in neutron "gate-neutron-dsvm-functional race fails HA/DVR tests with network namespace not found" [High,Confirmed] https://launchpad.net/bugs/1446261 - Assigned to Maru Newby (maru) | 15:30 |
openstackgerrit | Samuel BARTEL proposed openstack-infra/project-config: add fuel-plugin-nova-nfs-project Change-Id: Ibc53fdf69287059cba12423333ca21d8f523e626 https://review.openstack.org/175452 | 15:33 |
waynr | pelix: ping | 15:34 |
*** nfedotov has quit IRC | 15:34 | |
waynr | actually nevermind, i was going to ask you a question about one of your comments but i just grokked it | 15:35 |
*** dkranz has joined #openstack-infra | 15:37 | |
openstackgerrit | Wayne Warren proposed openstack-infra/jenkins-job-builder: Update 'timeout' wrapper module https://review.openstack.org/129467 | 15:37 |
*** afazekas_ has joined #openstack-infra | 15:37 | |
*** dhritishikhar_ has joined #openstack-infra | 15:38 | |
*** dhritishikhar_ has quit IRC | 15:39 | |
*** dhritishikhar has quit IRC | 15:39 | |
*** dannywilson has joined #openstack-infra | 15:39 | |
*** dhritishikhar_ has joined #openstack-infra | 15:40 | |
*** MaxV has joined #openstack-infra | 15:41 | |
*** dhritishikhar_ has quit IRC | 15:41 | |
*** dhritishikhar_ has joined #openstack-infra | 15:41 | |
openstackgerrit | Emilien Macchi proposed openstack-infra/project-config: puppet: run beaker on both trusty & centos7 https://review.openstack.org/175434 | 15:42 |
*** Swami has joined #openstack-infra | 15:42 | |
*** dannywilson has quit IRC | 15:42 | |
*** dannywilson has joined #openstack-infra | 15:43 | |
*** dhritishikhar_ has quit IRC | 15:43 | |
*** dhritishikhar_ has joined #openstack-infra | 15:44 | |
*** vkmc has joined #openstack-infra | 15:44 | |
*** gyee has joined #openstack-infra | 15:45 | |
*** yfried__ has quit IRC | 15:45 | |
openstackgerrit | Wayne Warren proposed openstack-infra/jenkins-job-builder: (hipchat) Accept list of hipchat rooms. https://review.openstack.org/153831 | 15:45 |
*** Ala has quit IRC | 15:46 | |
*** dboik_ has joined #openstack-infra | 15:47 | |
*** afazekas_ has quit IRC | 15:47 | |
*** dboik has quit IRC | 15:50 | |
*** dannywilson has quit IRC | 15:51 | |
*** jlvillal has quit IRC | 15:52 | |
*** jlvillal has joined #openstack-infra | 15:52 | |
*** rm_work is now known as rm_work|away | 15:52 | |
*** tnovacik has joined #openstack-infra | 15:52 | |
*** jistr_ has joined #openstack-infra | 15:53 | |
*** russellb has quit IRC | 15:53 | |
*** tqtran has joined #openstack-infra | 15:55 | |
*** adrian_otto has joined #openstack-infra | 15:55 | |
*** mrmartin has quit IRC | 15:55 | |
*** garyk has quit IRC | 15:56 | |
*** jistr has quit IRC | 15:56 | |
*** sdake_ has joined #openstack-infra | 15:57 | |
*** russellb has joined #openstack-infra | 15:58 | |
*** ihrachyshka has joined #openstack-infra | 15:58 | |
*** nelsnels_ has joined #openstack-infra | 15:59 | |
*** sergsh has quit IRC | 15:59 | |
*** BharatK has quit IRC | 16:00 | |
*** nelsnels_ has quit IRC | 16:00 | |
*** Longgeek has quit IRC | 16:00 | |
*** sdake has quit IRC | 16:00 | |
*** nelsnelson has quit IRC | 16:01 | |
*** nelsnels_ has joined #openstack-infra | 16:01 | |
*** sdake has joined #openstack-infra | 16:02 | |
*** sabeen1 has quit IRC | 16:02 | |
*** funzo has quit IRC | 16:02 | |
*** adiantum has quit IRC | 16:02 | |
openstackgerrit | Ryan Carey proposed openstack-infra/jenkins-job-builder: Add support for Artifactory plugin https://review.openstack.org/173932 | 16:02 |
zaro | morning | 16:02 |
waynr | howdy | 16:02 |
waynr | i am finding some zuul code i am looking at a little confusing | 16:03 |
*** sputnik13 has joined #openstack-infra | 16:04 | |
fungi | zaro: so, when i tried to restart gerrit on review.o.o with the latest autobuilt 2.8.x tarball which includes jeblair's stream-events task info patch, the service wouldn't start. i suspect something is broken with our builds. the error is mentioned in my commit message at https://review.openstack.org/175098 | 16:04 |
pleia2 | good morning | 16:04 |
*** sdake_ has quit IRC | 16:05 | |
*** weshay has quit IRC | 16:05 | |
*** jistr_ is now known as jistr | 16:05 | |
*** scheuran has quit IRC | 16:05 | |
*** adreznec has quit IRC | 16:06 | |
*** mfink has quit IRC | 16:06 | |
openstackgerrit | Darren Birkett proposed openstack-infra/jeepyb: Handle projects not using stable/ branch names https://review.openstack.org/153057 | 16:07 |
*** spredzy is now known as spredzy|afk | 16:08 | |
*** xyang1 has joined #openstack-infra | 16:09 | |
clarkb | good morning | 16:09 |
fbo | asselin, maybe I will need your feedback/support for this review https://review.openstack.org/#/c/168330/4 regarding the zuul refactor for puppet-openstackci, so if you have time to check it I'll be happy | 16:09 |
greghaynes | waynr: that never happens ;) | 16:10 |
openstackgerrit | Andreas Jaeger proposed openstack/requirements: Remove doc repositories from projects.txt https://review.openstack.org/173201 | 16:10 |
*** arielb has joined #openstack-infra | 16:10 | |
*** arielb has quit IRC | 16:10 | |
*** arielb has joined #openstack-infra | 16:10 | |
*** jcoufal has quit IRC | 16:11 | |
*** Krinkle|detached is now known as Krinkle | 16:11 | |
*** unicell has quit IRC | 16:12 | |
zaro | fungi: i think there might be duplicate bouncycastle libs in the review_site/lib folder. can check there? | 16:13 |
*** hashar has quit IRC | 16:13 | |
zaro | fungi: that error happens when java loads the same package from different jar files. | 16:14 |
*** mfink has joined #openstack-infra | 16:15 | |
*** adreznec has joined #openstack-infra | 16:15 | |
*** mrmartin has joined #openstack-infra | 16:15 | |
*** ashleighfarnham has joined #openstack-infra | 16:17 | |
*** otter768 has joined #openstack-infra | 16:18 | |
fungi | zaro: thanks! checking now... | 16:19 |
fungi | zaro: depends i guess on what you mean by duplicate. there aren't more than one of each linked there but perhaps we've suddenly started also bundling a copy in the gerrit.war itself? http://paste.openstack.org/show/204889/ | 16:20 |
fungi | zaro: because rolling back to the previous gerrit.war was sufficient to be able to start gerrit successfully again | 16:22 |
*** Krinkle is now known as Krinkle|detached | 16:22 | |
openstackgerrit | Merged openstack-infra/elastic-recheck: Add query for neutron functional test bug 1446261 https://review.openstack.org/175432 | 16:22 |
openstack | bug 1446261 in neutron "gate-neutron-dsvm-functional race fails HA/DVR tests with network namespace not found" [High,Confirmed] https://launchpad.net/bugs/1446261 - Assigned to Maru Newby (maru) | 16:22 |
openstackgerrit | Merged openstack-infra/jeepyb: Log project names as the projects are processed https://review.openstack.org/169435 | 16:22 |
*** otter768 has quit IRC | 16:23 | |
*** sdake has quit IRC | 16:23 | |
zaro | fungi: unlikely, there was no change to build scripts. let me see if i can deploy on review-dev and get back to you. | 16:24 |
fungi | zaro: thanks | 16:24 |
openstackgerrit | David Shrewsbury proposed openstack-infra/shade: Search methods for networks, subnets and routers https://review.openstack.org/175027 | 16:24 |
*** thedodd has quit IRC | 16:24 | |
*** jistr has quit IRC | 16:24 | |
*** sdake has joined #openstack-infra | 16:25 | |
openstackgerrit | Merged openstack-infra/project-config: Puppet OpenStack: enable Puppet 4.0 as non voting https://review.openstack.org/175145 | 16:25 |
zaro | fungi: ohh darn, review-dev db is on gerrit 2.10. i'll test on my VM instead. | 16:26 |
*** Kennan2 has quit IRC | 16:29 | |
openstackgerrit | Sergey Lukjanov proposed openstack-infra/project-config: The sahara-tox-py27-scenario-unit job should be voting since kilo https://review.openstack.org/175474 | 16:29 |
*** baoli has quit IRC | 16:29 | |
*** Kennan has joined #openstack-infra | 16:29 | |
*** dboik_ has quit IRC | 16:30 | |
*** dboik has joined #openstack-infra | 16:30 | |
*** rm_work|away is now known as rm_work | 16:31 | |
*** dboik has quit IRC | 16:33 | |
*** baoli_ has joined #openstack-infra | 16:33 | |
*** dboik has joined #openstack-infra | 16:33 | |
openstackgerrit | Merged openstack-infra/project-config: Allow gnocchi-release to create branch https://review.openstack.org/174387 | 16:35 |
*** yolanda has quit IRC | 16:35 | |
*** unicell has joined #openstack-infra | 16:41 | |
*** claudiub has joined #openstack-infra | 16:41 | |
*** dhritishikhar_ has quit IRC | 16:41 | |
*** Swami has quit IRC | 16:41 | |
*** harlowja_at_home has quit IRC | 16:41 | |
*** Krinkle|detached is now known as Krinkle | 16:43 | |
*** ashleighfarnham has quit IRC | 16:43 | |
*** luqas has quit IRC | 16:43 | |
clarkb | armax_: I have just held the nodes running https://jenkins03.openstack.org/job/check-tempest-dsvm-neutron-multinode-full/5/ so we can take a look at things once tempest is done there | 16:43 |
armax_ | great | 16:44 |
*** dhritishikhar_ has joined #openstack-infra | 16:44 | |
asselin_ | fbo, done | 16:46 |
*** frobware_ has joined #openstack-infra | 16:46 | |
*** armax_ has left #openstack-infra | 16:46 | |
*** armax_ has joined #openstack-infra | 16:47 | |
*** dizquierdo has joined #openstack-infra | 16:47 | |
*** Swami has joined #openstack-infra | 16:48 | |
fbo | asselin_, thanks ! | 16:49 |
*** devvesa has quit IRC | 16:51 | |
*** derekh has quit IRC | 16:52 | |
*** ashleighfarnham has joined #openstack-infra | 16:52 | |
openstackgerrit | Ryan Carey proposed openstack-infra/jenkins-job-builder: Add support for Artifactory plugin https://review.openstack.org/173932 | 16:52 |
asselin_ | fbo, btw, I'm ok with some basic refactoring like that to be done as part of migrating to puppet-openstackci. | 16:53 |
*** russellb has quit IRC | 16:55 | |
*** vilobhmm1 has joined #openstack-infra | 16:55 | |
*** vilobhmm11 has joined #openstack-infra | 16:55 | |
*** Krinkle is now known as Krinkle|detached | 16:56 | |
*** sdake_ has joined #openstack-infra | 16:57 | |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: add support for stable/kilo https://review.openstack.org/175391 | 16:57 |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: redo devstack-gate grenade selection as a case statement https://review.openstack.org/175390 | 16:57 |
*** russellb has joined #openstack-infra | 16:58 | |
*** asselin__ has joined #openstack-infra | 16:59 | |
*** bswartz has quit IRC | 16:59 | |
*** dguitarbite has joined #openstack-infra | 16:59 | |
*** vilobhmm1 has quit IRC | 16:59 | |
*** sdake has quit IRC | 17:00 | |
*** baoli_ has quit IRC | 17:01 | |
*** dboik has quit IRC | 17:02 | |
*** dboik has joined #openstack-infra | 17:02 | |
*** asselin_ has quit IRC | 17:02 | |
*** dboik has quit IRC | 17:03 | |
*** dboik has joined #openstack-infra | 17:03 | |
*** armax_ has quit IRC | 17:03 | |
*** frobware_ has quit IRC | 17:04 | |
*** vilobhmm11 has left #openstack-infra | 17:04 | |
*** armax_ has joined #openstack-infra | 17:04 | |
openstackgerrit | Samuel BARTEL proposed openstack-infra/project-config: add plugin-fuel-nova-nfs project alphabetical order https://review.openstack.org/175487 | 17:04 |
*** sdake has joined #openstack-infra | 17:06 | |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: add liberty to feature grid https://review.openstack.org/175493 | 17:07 |
*** cody-somerville has joined #openstack-infra | 17:07 | |
*** dboik has quit IRC | 17:07 | |
*** dboik has joined #openstack-infra | 17:07 | |
*** mpaolino has quit IRC | 17:08 | |
*** ivar-lazzaro has joined #openstack-infra | 17:08 | |
openstackgerrit | Ryan Carey proposed openstack-infra/jenkins-job-builder: Add support for Artifactory plugin https://review.openstack.org/173932 | 17:08 |
*** ivar-lazzaro has quit IRC | 17:09 | |
*** ivar-lazzaro has joined #openstack-infra | 17:09 | |
*** sdake_ has quit IRC | 17:09 | |
fungi | heads up, i have an errand to head out to in about half an hour, and then i'll be grabbing a late lunch, but will be back after | 17:09 |
* clarkb prepares for all the clouds to go sidways again >_> | 17:10 | |
fungi | oh, they haven't already? i didn't even check... ;) | 17:10 |
*** patrickeast has joined #openstack-infra | 17:10 | |
fungi | zuul is extremely busy, but our overall resource pool seems remarkably healthy even if running on a reduced quota | 17:11 |
clarkb | ya, itseems to mostly be keeping up | 17:12 |
clarkb | the check lag time is ~20 minutes | 17:12 |
jeblair | clarkb, fungi: anything urgent need my attention while a have a few hours between transits? :) | 17:13 |
openstackgerrit | Matthew Treinish proposed openstack-infra/elastic-recheck: Add query for bug 1446309 https://review.openstack.org/175496 | 17:13 |
openstack | bug 1446309 in tempest "tempest.tests.common.test_accounts.TestAccount.test__get_creds_by_roles_no_admin race fails" [High,In progress] https://launchpad.net/bugs/1446309 - Assigned to Matthew Treinish (treinish) | 17:13 |
*** ssam2 has quit IRC | 17:13 | |
clarkb | jeblair: not that I have seen. Assisting sdague and dtroyer cutting kilo for grenade and devstack. https://review.openstack.org/#/c/175390/3 can use a second review to help that along | 17:14 |
openstackgerrit | Merged openstack/requirements: Bump PyECLib version from 1.0.3 to 1.0.7 https://review.openstack.org/174171 | 17:14 |
clarkb | if it stays relatively quiet like this I am hopefully going to be able to do reviews of our priority changes today | 17:14 |
openstackgerrit | Matt Riedemann proposed openstack-infra/elastic-recheck: Add query for tempest unit test bug 1446309 https://review.openstack.org/175497 | 17:14 |
jeblair | clarkb: ack | 17:14 |
fungi | jeblair: yeah, i think we're set. enjoy your transit! | 17:14 |
openstackgerrit | Merged openstack/requirements: Bump PyECLib version from 1.0.3 to 1.0.7 https://review.openstack.org/174167 | 17:15 |
openstackgerrit | Eric Larson proposed openstack-infra/project-config: Create oslo.cache project https://review.openstack.org/165224 | 17:15 |
*** Guest36304 is now known as mgagne | 17:16 | |
*** mgagne has joined #openstack-infra | 17:16 | |
pleia2 | elarson: thanks, I'll review | 17:16 |
clarkb | jeblair: https://review.openstack.org/#/c/107267/ is my selfish stack needing reviews. jhesketh wrote it to make os-loganalyze configurable for passthrough of files from swift | 17:16 |
jeblair | clarkb, sdague: it looks like we currently regex match sideways, presumably to support sidewase-foo or something. but i take it we only currently have literal 'sideways' in use, since the case-based code full-matches on that. is that right? | 17:16 |
clarkb | looks like SergeyLukjanov actually +2'd those so I can merge them and babysit later | 17:17 |
*** deepakcs has joined #openstack-infra | 17:17 | |
openstackgerrit | Matthew Treinish proposed openstack-infra/elastic-recheck: Add query for bug 1446309 https://review.openstack.org/175496 | 17:17 |
openstack | bug 1446309 in tempest "tempest.tests.common.test_accounts.TestAccount.test__get_creds_by_roles_no_admin race fails" [High,In progress] https://launchpad.net/bugs/1446309 - Assigned to Matthew Treinish (treinish) | 17:17 |
fungi | after lunch i'm hoping to do some final manual re-testing of the state of bindep since all open changes have now merged, and then tag a 1.0.0 and make sure it ends up on pypi | 17:17 |
pleia2 | clarkb, fungi - btw, I'm done conferencing and things aside from summit pretty much through august (shocking, right?), so please keep me in mind if there are pleia2-should-learn things | 17:18 |
clarkb | jeblair: actually I think you are right, we do sideways-ironic and sideways-neutron | 17:18 |
clarkb | jeblair: sdague so that should be case sideways-* | 17:18 |
clarkb | pleia2: will do! | 17:18 |
clarkb | pleia2: maybe I should drag you into the swift log hosting <_< | 17:18 |
pleia2 | clarkb: maybe! | 17:18 |
fungi | pleia2: you got it. also if you see me mention anything in here which you also want to learn, just say so. cross-training other core reviewers and root sysadmins is top priority as far as i'm concerned | 17:18 |
pleia2 | fungi: great, will do | 17:19 |
clarkb | pleia2: https://review.openstack.org/#/c/107267/ is the start of changes to os-loganalyze for that. tl;dr is some code reorg to support configurable modifications to files or passthrough when hosted via that wsgi app | 17:19 |
pleia2 | clarkb: thanks, I'll have a look | 17:19 |
clarkb | once we get that in place we will need a system-config change to tell os-loganalyze to pass through things that aren't log files | 17:19 |
clarkb | fungi: +1 | 17:19 |
*** davideagnello has joined #openstack-infra | 17:20 | |
jeblair | clarkb, sdague: okay, left -1 with inline comment. otherwise, +2 in case i'm not around to re-review. | 17:22 |
clarkb | jeblair: ty | 17:22 |
jeblair | pleia2: awesome, good to know! :) | 17:22 |
sdague | oh, so I guess I missed the sideways regex, will fix | 17:22 |
*** achanda has joined #openstack-infra | 17:24 | |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: add support for stable/kilo https://review.openstack.org/175391 | 17:25 |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: redo devstack-gate grenade selection as a case statement https://review.openstack.org/175390 | 17:25 |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: add liberty to feature grid https://review.openstack.org/175493 | 17:25 |
sdague | jeblair / clarkb thanks for the catch | 17:25 |
*** sushilkm has joined #openstack-infra | 17:26 | |
*** sushilkm has left #openstack-infra | 17:26 | |
zaro | clarkb: where is the documentation for the version of puppet we are using? I want to verify that tidy is available | 17:26 |
clarkb | zaro: https://docs.puppetlabs.com/references/3.stable/type.html should be latest stable 3.x release | 17:27 |
zaro | thanks | 17:27 |
*** yolanda has joined #openstack-infra | 17:28 | |
*** MaxV has quit IRC | 17:29 | |
*** ildikov has quit IRC | 17:29 | |
fungi | sdague: possible concern marked inline on https://review.openstack.org/175390 (not sure if we want to always assume subtypes of those?) | 17:30 |
openstackgerrit | Wayne Warren proposed openstack-infra/zuul: WIP: Support GitHub PR webhooks https://review.openstack.org/163117 | 17:31 |
openstackgerrit | Emilien Macchi proposed openstack-infra/project-config: puppet: run beaker on both trusty & centos7 https://review.openstack.org/175434 | 17:31 |
*** nelsnelson has joined #openstack-infra | 17:31 | |
*** HeOS has quit IRC | 17:32 | |
*** aswadr has quit IRC | 17:32 | |
clarkb | fungi: yes, the comment just above the case statement should explain the cases | 17:32 |
jeblair | fungi, sdague: the only extant uses of both have hyphens, so i think it's okay | 17:32 |
fungi | cool. +2 from me then | 17:33 |
*** cody-somerville has quit IRC | 17:34 | |
*** nelsnels_ has quit IRC | 17:35 | |
clarkb | uh python3.4 bug for trusty just went fix released | 17:35 |
* clarkb does some apt-get update | 17:35 | |
fungi | oh? | 17:35 |
greghaynes | clarkb: wat | 17:35 |
zaro | fungi: i am able to run 2.8.4.19 no problem. i'm wondering about this puppet http://git.openstack.org/cgit/openstack-infra/puppet-gerrit/tree/manifests/init.pp#n720 | 17:36 |
* greghaynes drops the baloons | 17:36 | |
clarkb | my mirror doesn't seem to have it yet | 17:36 |
*** russellb has quit IRC | 17:36 | |
fungi | clarkb: see the comments on bug 1348954 | 17:36 |
openstack | bug 1348954 in python3.4 (Ubuntu) "update Python3 for trusty" [Undecided,Confirmed] https://launchpad.net/bugs/1348954 | 17:36 |
zaro | fungi: that tidy might may not remove the those duplicate files before gerrit starts | 17:36 |
fungi | clarkb: apparently a changelog in another package mentioned the wrong bug number | 17:36 |
clarkb | arg | 17:37 |
zaro | fungi: i think we need to make sure that tidy is done before file { '/etc/init.d/gerrit | 17:37 |
*** dboik has quit IRC | 17:37 | |
fungi | zaro: oh, perhaps... though once that puppet was applied i tried multiple times to start the gerrit service, always with that same error | 17:37 |
*** dboik has joined #openstack-infra | 17:38 | |
zaro | fungi: but i'm not sure how to change the require on gerrit init to a tidy | 17:38 |
*** melwitt has joined #openstack-infra | 17:38 | |
*** harlowja_away is now known as harlowja | 17:38 | |
zaro | fungi: well then that probably debunks my theory | 17:38 |
*** kmartin_ has quit IRC | 17:39 | |
fungi | zaro: does your test environment have contactstore enabled in the config like we do on review(-dev)? | 17:39 |
fungi | it's possible gerrit only notices the problem when it tries to use features from that library | 17:40 |
*** russellb has joined #openstack-infra | 17:40 | |
*** dboik has quit IRC | 17:40 | |
*** _nadya_ has quit IRC | 17:40 | |
*** isviridov is now known as isviridov_away | 17:41 | |
*** dboik has joined #openstack-infra | 17:41 | |
*** AJaeger_ has joined #openstack-infra | 17:41 | |
*** dustins has joined #openstack-infra | 17:41 | |
AJaeger_ | dolphm: around? | 17:41 |
zaro | fungi: ohh yeah, i had that commented out. let me try again | 17:42 |
*** luqas has joined #openstack-infra | 17:44 | |
fungi | okay, heading out to errands/late lunch. back in a couple hours | 17:44 |
*** ZZelle has quit IRC | 17:45 | |
*** baoli has joined #openstack-infra | 17:46 | |
*** nelsnelson has quit IRC | 17:47 | |
*** russellb has quit IRC | 17:47 | |
*** yfried__ has joined #openstack-infra | 17:48 | |
*** luqas has quit IRC | 17:48 | |
*** bswartz has joined #openstack-infra | 17:50 | |
*** david-lyle has joined #openstack-infra | 17:50 | |
*** e0ne has quit IRC | 17:51 | |
*** isviridov_away is now known as isviridov | 17:51 | |
*** russellb has joined #openstack-infra | 17:52 | |
*** crc32 has joined #openstack-infra | 17:54 | |
*** nelsnelson has joined #openstack-infra | 17:54 | |
openstackgerrit | Merged openstack-infra/elastic-recheck: Add query for bug 1446309 https://review.openstack.org/175496 | 17:55 |
openstack | bug 1446309 in tempest "tempest.tests.common.test_accounts.TestAccount.test__get_creds_by_roles_no_admin race fails" [High,In progress] https://launchpad.net/bugs/1446309 - Assigned to Matthew Treinish (treinish) | 17:55 |
clarkb | armax_: I think we are getting close to the end of that job and a few tests have failed. Want to point me at an ssh public key so yo ucan get into the env and poke around? | 17:57 |
armax_ | Swami: ^^ | 17:58 |
*** pelix has quit IRC | 17:58 | |
openstackgerrit | Gary W. Smith proposed openstack-infra/project-config: Add manila-ui to OpenStack https://review.openstack.org/175063 | 17:59 |
Swami | armax_: hi | 17:59 |
*** teran has quit IRC | 17:59 | |
*** unicell has quit IRC | 17:59 | |
armax_ | clarkb: hang on | 17:59 |
*** unicell has joined #openstack-infra | 17:59 | |
armax_ | clarkb: https://launchpad.net/~armando-migliaccio/+sshkeys | 17:59 |
mriedem | this doesn't appear to be an issue anymore, can someone from infra confirm? https://bugs.launchpad.net/openstack-gate/+bug/1421452 | 18:00 |
openstack | Launchpad bug 1421452 in OpenStack-Gate "We reverted back to https://pypi.python.org" [Undecided,New] | 18:00 |
*** dguitarbite has quit IRC | 18:00 | |
*** david-lyle_ has joined #openstack-infra | 18:01 | |
clarkb | armax_: jenkins@162.242.212.53 is the primary node running the controller and jenkins@162.242.212.93 is the subnode compute host | 18:01 |
clarkb | armax_: we should let tempest finish before making changes though | 18:01 |
*** dboik_ has joined #openstack-infra | 18:01 | |
clarkb | armax_: that way you don't fight tempest | 18:01 |
*** annegentle has joined #openstack-infra | 18:01 | |
*** tkelsey has quit IRC | 18:01 | |
*** david-lyle has quit IRC | 18:01 | |
armax_ | clarkb: makes sense | 18:02 |
*** shashankhegde has joined #openstack-infra | 18:03 | |
armax_ | clarkb: I am in, I’ll wait for the build to complete | 18:04 |
*** mattfarina has quit IRC | 18:04 | |
*** dboik has quit IRC | 18:05 | |
Shrews | fungi: clarkb: If either of you get some time today, care to put some eyes on https://review.openstack.org/173994 and https://review.openstack.org/173998? Both very similar and one +2 already. | 18:06 |
*** david-lyle_ is now known as david-lyle | 18:06 | |
openstackgerrit | Kurt Taylor proposed openstack-infra/project-config: Add third party CI working group tools repository https://review.openstack.org/175520 | 18:08 |
*** miqui has quit IRC | 18:13 | |
*** tonytan4ever has quit IRC | 18:13 | |
*** AJaeger_ has quit IRC | 18:14 | |
*** sdake_ has joined #openstack-infra | 18:14 | |
*** spredzy|afk is now known as spredzy | 18:15 | |
*** mattfarina has joined #openstack-infra | 18:16 | |
*** tsg has joined #openstack-infra | 18:17 | |
*** sdake has quit IRC | 18:18 | |
*** dboik_ has quit IRC | 18:19 | |
*** otter768 has joined #openstack-infra | 18:19 | |
*** david-lyle has quit IRC | 18:20 | |
*** david-lyle has joined #openstack-infra | 18:20 | |
*** bhuvan has joined #openstack-infra | 18:20 | |
*** ihrachyshka has quit IRC | 18:20 | |
*** vlaza_offline has quit IRC | 18:21 | |
clarkb | armax_: its done | 18:21 |
*** ayoung has quit IRC | 18:22 | |
*** otter768 has quit IRC | 18:24 | |
*** dhritishikhar_ has quit IRC | 18:25 | |
*** arosen has left #openstack-infra | 18:25 | |
openstackgerrit | Samuel BARTEL proposed openstack-infra/project-config: add plugin-fuel-nova-nfs project alphabetical order https://review.openstack.org/175487 | 18:29 |
*** e0ne has joined #openstack-infra | 18:29 | |
*** tkelsey has joined #openstack-infra | 18:30 | |
*** sdake has joined #openstack-infra | 18:30 | |
*** dizquierdo has quit IRC | 18:31 | |
*** dboik has joined #openstack-infra | 18:33 | |
*** ZZelle has joined #openstack-infra | 18:33 | |
*** sdake_ has quit IRC | 18:34 | |
*** tkelsey has quit IRC | 18:34 | |
openstackgerrit | Eric Larson proposed openstack-infra/project-config: Create oslo.cache project https://review.openstack.org/165224 | 18:35 |
openstackgerrit | Matt Riedemann proposed openstack-infra/elastic-recheck: Remove old queries for "fixed" bugs https://review.openstack.org/175529 | 18:36 |
*** sushilkm has joined #openstack-infra | 18:36 | |
*** sushilkm has left #openstack-infra | 18:36 | |
openstackgerrit | Samuel BARTEL proposed openstack-infra/project-config: add fuel-plugin-nova-nfs-project Change-Id: Ibc53fdf69287059cba12423333ca21d8f523e626 https://review.openstack.org/175452 | 18:38 |
samuelBartel | Hi | 18:40 |
samuelBartel | i am a little been lost with gerrit | 18:40 |
*** tonytan4ever has joined #openstack-infra | 18:40 | |
samuelBartel | i tried to push a past to a review and it seems i overwrite the previous patch in the same time | 18:40 |
samuelBartel | is it a way to crrect it? | 18:41 |
*** david-lyle has quit IRC | 18:41 | |
*** baoli has quit IRC | 18:41 | |
clarkb | samuelBartel: yes, gerrit changes keep a history of patches with the most recent being up for review to be merged | 18:41 |
*** doude has quit IRC | 18:42 | |
clarkb | samuelBartel: this way when code merges you merge the code everyone is happy with and not a half dozen "whoops fixed X" commits too | 18:42 |
*** ZZelle has quit IRC | 18:42 | |
samuelBartel | clarkb, ok thank, it make sense | 18:43 |
openstackgerrit | Sean Dague proposed openstack-infra/devstack-gate: only force off KEYSTONE_USE_WSGI up until kilo https://review.openstack.org/175531 | 18:43 |
*** arielb has quit IRC | 18:44 | |
*** e0ne is now known as e0ne_ | 18:46 | |
*** deepakcs has quit IRC | 18:47 | |
*** arielb has joined #openstack-infra | 18:47 | |
*** dboik has quit IRC | 18:48 | |
mordred | clarkb: did you see/read the scrollback between tchaypo and I from this morning? I believe we've uncovered a bug in how pbr and pip interact that may have caused install vs. upgrade to be more confusing | 18:48 |
*** amitgandhinz has quit IRC | 18:48 | |
*** ZZelle has joined #openstack-infra | 18:48 | |
clarkb | I saw but didnt have enough context to understand the problem | 18:48 |
mordred | clarkb: cool. it's ok - no need to understand yet | 18:49 |
mordred | clarkb: I've got a patch I'm going to test out | 18:49 |
mordred | clarkb: if it does what I think it will, then I'll braindump to you | 18:49 |
clarkb | ok | 18:49 |
mordred | clarkb: tl;dr is pip install -r requirements.txt does a different thing than pip install foo where foo has requirements in install_requires | 18:50 |
mordred | clarkb: so pbr executing pip install -r requirements.txt in install phase produces wrong behavior, that I think can be fixed by just deleting code | 18:50 |
*** isviridov is now known as isviridov_away | 18:51 | |
*** MaxV has joined #openstack-infra | 18:51 | |
*** dannywilson has joined #openstack-infra | 18:52 | |
*** e0ne_ has quit IRC | 18:54 | |
*** imcsk8 has quit IRC | 18:54 | |
*** imcsk8 has joined #openstack-infra | 18:54 | |
openstackgerrit | Kurt Taylor proposed openstack-infra/project-config: Add third party CI working group tools repository https://review.openstack.org/175520 | 18:55 |
*** andreykurilin__ has joined #openstack-infra | 18:55 | |
*** amitgandhinz has joined #openstack-infra | 18:56 | |
openstackgerrit | Samuel BARTEL proposed openstack-infra/project-config: add project fuel-plugin-nova-nfs Change-Id: Ibc53fdf69287059cba12423333ca21d8f523e626 https://review.openstack.org/175452 | 18:56 |
*** dannywilson has quit IRC | 18:56 | |
*** hdd has joined #openstack-infra | 18:57 | |
*** dboik has joined #openstack-infra | 18:57 | |
*** e0ne has joined #openstack-infra | 18:58 | |
openstackgerrit | Merged openstack/requirements: Recap keystoneclient for stable/icehouse https://review.openstack.org/175234 | 18:58 |
krtaylor | asselin, thanks for the review, re: jobs, I used specs because it seemed the least strict, I just removed it for now | 18:58 |
anteaya | mriedem: I have not seen a lot of chat about timeouts with pypi.python.org: https://bugs.launchpad.net/openstack-gate/+bug/1421452 | 18:58 |
openstack | Launchpad bug 1421452 in OpenStack-Gate "We reverted back to https://pypi.python.org" [Undecided,Incomplete] | 18:58 |
anteaya | mriedem: I can't speak for the others though | 18:59 |
clarkb | mordred we did that because easy install is used otherwise iirc | 19:00 |
asselin__ | krtaylor, yeah...we'll need to figure out how to setup the jobs | 19:00 |
clarkb | mordred so its two bad thing pick one | 19:00 |
asselin__ | krtaylor, err...which jobs to run | 19:00 |
mordred | clarkb: right - except easy_install doesn't get invoked the otherway | 19:01 |
mordred | clarkb: we did that because we _thought_ that's what would happen | 19:01 |
lifeless | mordred: no | 19:01 |
mordred | lifeless: no? | 19:01 |
lifeless | mordred: pip will upgrade things if thats needed to meet the dep on a new install | 19:01 |
mordred | lifeless: yes - but not if you pip install -r requirements.txt | 19:01 |
mordred | lifeless: (if you haven't read the whole tchaypo scrollback yet - we figured that out) | 19:02 |
lifeless | mordred: no, I did a lastlog lifeless | 19:02 |
lifeless | and saw a wall of mentions | 19:02 |
lifeless | mordred: I'm glad you've reached consensus ;) | 19:02 |
mordred | clarkb: what pip does is run python setup.py egg_info - and then looks at the requires.txt in the egg-info dir - and then pip installs those knowing that they are dependencies | 19:02 |
mordred | clarkb: what it does whe you do -r requirements.txt is that it treats each requirement as an individual request for install - not as a dependency | 19:03 |
mordred | clarkb: so our workaround to avoid using easy_install not only is not needed, it's causing a wrong behavior | 19:03 |
lifeless | pip doesn't consider dep vs request to install differently | 19:03 |
*** ildikov has joined #openstack-infra | 19:04 | |
lifeless | I'd like to tease this apart, I'm worried I'm jumping into a many hours long conversation without enough context | 19:04 |
lifeless | is there a bug or some worked summary of the thing? | 19:04 |
mordred | lifeless: nope | 19:05 |
mordred | lifeless: tchaypo and I spoke for about 45 minutes this morning before I had a meeting | 19:05 |
mordred | lifeless: I recommend just reading his and my scrollback starting at 11:42:20 UTC | 19:05 |
mordred | lifeless: I'm using the word dependency to describe an observed behavior, not a declared action | 19:06 |
mordred | lifeless: if bar is installed at ==2.0 | 19:06 |
mordred | lifeless: if foo has an install_requires=['bar>=2.2'] and you do "pip install foo" - pip will install foo and upgrade bar to 2.2 to meet the depend | 19:07 |
mordred | lifeless: if you have a requirements.txt file listing 'bar>=2.2' and you do "pip install -r requirements.txt" - pip will not upgrade bar, because you did not ask it to upgrade | 19:07 |
mordred | those are actually reasonable actions I believe | 19:07 |
clarkb | >= says please upgrade to me | 19:08 |
clarkb | or at least error early | 19:08 |
mordred | clarkb: right - except that requirements.txt is not intended to be used the way we are using it | 19:08 |
mordred | clarkb: it's just a list of things that pip should pretend you said "pip install $thing" on the command line | 19:09 |
lifeless | mordred: a few thoughts as I go through: I still fundamentally disagree about known-good vs known-bad, the known-good desire is causing us huge grief - thats what we were etherpad speccing last week, how to get that out of the install_requires! | 19:09 |
*** aduarte has joined #openstack-infra | 19:09 | |
mordred | lifeless: oh - well, then we should swing back to that - but let's clear up this one first | 19:09 |
mordred | lifeless: also - good morning! | 19:10 |
clarkb | mordred I think it shouldstill at least error in that case | 19:11 |
clarkb | mordred I asked it to install something and it failed | 19:11 |
mordred | clarkb: I agree | 19:11 |
mordred | clarkb: but I can at least understand why it doesn't upgrade in that case | 19:11 |
mordred | clarkb: but does in the other | 19:11 |
lifeless | mordred: oh, I hadn't realised we wrapped pip rather than exporting to pip, thats clearly a bug | 19:11 |
mordred | from its point of view | 19:11 |
mordred | lifeless: yah. it's a holdover from a long time ago when we did not realize exactly what the paths for triggering easy_install all were | 19:12 |
* anteaya thought one of the problems was that with bar==2.0 and requirements specifying bar<=2.0 with pip installing foo with bar 2.1 available foo was downloaded and bar became 2.1: https://www.irccloud.com/pastebin/70kutYTy https://github.com/jamezpolley/161047-requirements/blob/master/tmp/161047/requirements.txt | 19:12 | |
mordred | lifeless: (mainly was a "please god never let easy install run" thing) | 19:12 |
clarkb | anteaya ya == is differebt behavior in pip | 19:12 |
anteaya | clarkb: okay thank you | 19:12 |
clarkb | if you use == it eill install that version | 19:12 |
clarkb | unlike >= | 19:12 |
anteaya | <= was in the requirements file | 19:13 |
anteaya | bar==2.0 was freeze output: https://www.irccloud.com/pastebin/70kutYTy | 19:13 |
*** dboik_ has joined #openstack-infra | 19:13 | |
*** dboik has quit IRC | 19:13 | |
dstufft | so two things | 19:13 |
lifeless | ok, so three discussions at once :) | 19:13 |
*** yfried__ is now known as yfried|afk | 19:14 | |
dstufft | pip doesn't consider the state of what's already installed into the environment _at all_ | 19:14 |
lifeless | mordred: so the invoking pip thing has to stop. I'd like to do that *after* we release master. | 19:14 |
lifeless | dstufft: I've opened a bug for that | 19:14 |
*** deepakcs has joined #openstack-infra | 19:14 | |
*** tsg has quit IRC | 19:14 | |
lifeless | dstufft: also, it does to a degree already, just not pervasively | 19:14 |
lifeless | dstufft: if you have a missing dep of a dep thats already installed, it will get filled out automatically. | 19:14 |
lifeless | four discussions at once | 19:15 |
dstufft | lol | 19:15 |
mordred | lifeless: yes - I'm pretty sure I agree that it's a but in pbr and has to stop | 19:15 |
* anteaya wonders if someone feels like populating an etherpad | 19:15 | |
lifeless | mordred: ok, great. Willing to wait for 0.12 ? | 19:15 |
mordred | lifeless: there is one thing I think we need to sort out that I could use mild help on - you may know off the top of your head | 19:15 |
*** teran has joined #openstack-infra | 19:15 | |
mordred | lifeless: yes | 19:15 |
lifeless | mordred: ok, I'm going to mark to mark it WIP | 19:15 |
mordred | lifeless: which is - one of the side-effects of our LocalInstall class is that we NEVER install an egg, we always install expanded files (a la single-version-externally-managed) | 19:16 |
openstackgerrit | Kyle Rockman proposed openstack-infra/jenkins-job-builder: Adding hipchat publisher https://review.openstack.org/174036 | 19:16 |
mordred | lifeless: I'd like to keep that behavior, but I don't just want to force-set single-version-externally-managed in the class, because that _Also_ causes it to not emit requires.txt into the egg-info file IIRC | 19:16 |
lifeless | mordred: right, pip doesn't install eggs either | 19:17 |
dstufft | I don't think that install_requires and -r requirements.txt acts differently as to how pip resolves them, other than pip only respects 1 version specifier for any particular name and a top level name in requirements.txt will be rpeferred over one inside of a install_requires | 19:17 |
mordred | lifeless: oh - good | 19:17 |
lifeless | mordred: easy_install will only trigger for setup_requires | 19:17 |
*** hashar has joined #openstack-infra | 19:17 | |
lifeless | dstufft: right, I've already said that above ;) | 19:17 |
mordred | dstufft: nope - we empirically tested that it's different earlier today | 19:17 |
lifeless | mordred: thats not due to what you think it is | 19:17 |
mordred | lifeless: I don't care - the behavior is demonstrably different | 19:17 |
lifeless | mordred: sure, but it can be produced with two packages with deps | 19:18 |
mordred | lifeless: (although I do care to learn) | 19:18 |
mordred | yes | 19:18 |
mordred | the key is that a dep is different than a reqlist | 19:18 |
lifeless | mordred: e.g. you don't need either -r requirements.txt *or* explicit-on-command-line use to trigger the pathology | 19:18 |
lifeless | mordred: no, thats the thing, its not. | 19:18 |
*** baoli has joined #openstack-infra | 19:18 | |
lifeless | mordred: tell you what, lets not spiral into insanity on this minor angle. | 19:19 |
*** johnthetubaguy is now known as zz_johnthetubagu | 19:19 | |
mordred | lifeless: right. I'm sure we could sort it - but I'm not sure it's worth sorting :) | 19:19 |
lifeless | mordred: if you or tchaypo file a bug on pbr with what you learnt summarised | 19:19 |
*** arielb has quit IRC | 19:19 | |
lifeless | mordred: I'll take it from there | 19:19 |
mordred | lifeless: woot | 19:19 |
*** ddieterly has quit IRC | 19:20 | |
*** baoli has quit IRC | 19:21 | |
*** baoli has joined #openstack-infra | 19:21 | |
*** ZZelle_ has joined #openstack-infra | 19:22 | |
*** yfried|afk is now known as yfried__ | 19:22 | |
*** shashankhegde has quit IRC | 19:24 | |
openstackgerrit | Merged openstack-infra/infra-manual: Creates a sandbox repo tutorial https://review.openstack.org/138195 | 19:25 |
lifeless | mordred: https://review.openstack.org/#/c/174646/ needs reviews | 19:25 |
lifeless | jd__: dhellmann: ^ | 19:26 |
*** tkelsey has joined #openstack-infra | 19:28 | |
*** Darkwan has joined #openstack-infra | 19:29 | |
*** ajmiller_ has joined #openstack-infra | 19:29 | |
clarkb | armax_: how is it going with direct env interaction? | 19:29 |
mordred | lifeless: https://bugs.launchpad.net/pbr/+bug/1446358 | 19:30 |
openstack | Launchpad bug 1446358 in PBR "pbr subs out to pip unnecessarily" [Undecided,New] | 19:30 |
armax_ | clarkb: I have logged in, but got distracted, poking now | 19:30 |
lifeless | mordred: thanks; thats the pip one. The bug I was asking for though was the one about differing behaviour with things in install_requires vs requirements.txt that you've seen | 19:32 |
lifeless | mordred: perhaps you're saying you think that is explained by invoking pip | 19:32 |
*** tkelsey has quit IRC | 19:32 | |
lifeless | anteaya: ok, now your pip issue. | 19:32 |
*** HeOS has joined #openstack-infra | 19:33 | |
*** ajmiller has quit IRC | 19:33 | |
mordred | lifeless: it's the same thign in my mind - and I think we decided that digging in to why it's different was not a good use of our brainholes? | 19:33 |
*** ddieterly has joined #openstack-infra | 19:33 | |
* anteaya listens | 19:33 | |
lifeless | mordred: then lets leave it there for now and you/tchaypo can try to reproduce once pip installation is ripped out | 19:34 |
*** ajmiller_ is now known as ajmiller | 19:34 | |
pleia2 | running out to dr appt+lunch, bbiab | 19:34 |
mordred | lifeless: yup | 19:34 |
* anteaya moves back to other things | 19:34 | |
openstackgerrit | Merged openstack-infra/project-config: oslo.messaging: enable Python 3.4 check https://review.openstack.org/172221 | 19:34 |
lifeless | anteaya: what you described is not a problem | 19:34 |
anteaya | lifeless: oh good | 19:34 |
lifeless | anteaya: we want all the requirements to be satisfied, assuming they can be. | 19:34 |
mordred | lifeless: tchaypo's original reproduction had nothing to do with pbr | 19:34 |
anteaya | except the way I interpreted that was that requirements specifyed bar<=2.0 and bar 2.1 was installed | 19:35 |
lifeless | anteaya: the behaviour described was the use of a dependency list to constrain an initial install, followed by installing something which conflicted with an installed version - and the installed version getting automtically upgraded | 19:35 |
lifeless | mordred: which is why I thought it was different to the invokes-pip thing | 19:35 |
openstackgerrit | Wayne Warren proposed openstack-infra/infra-specs: Develop gitub-integration spec. https://review.openstack.org/163637 | 19:36 |
anteaya | lifeless: ah, then I mis-interpreted the files | 19:36 |
anteaya | lifeless: thank you | 19:36 |
*** annegentle has quit IRC | 19:36 | |
mordred | lifeless: so - in my brain hole - tchaypo found a seeming difference in behavior in the two approaches, and the behavior he saw was similar to issues we've faced doing CD deploys of pbr things from git - and I can see how the invocation of pip in pbr would trigger the behavior | 19:38 |
mordred | lifeless: it's entirely likely that the situation is trickier | 19:38 |
mordred | lifeless: but the invocation is certainly bong | 19:38 |
mordred | lifeless: so we might as well sort that in parallel | 19:39 |
*** isviridov_away is now known as isviridov | 19:39 | |
lifeless | mordred: righto. So for clarity: I don't want to investigate the bong; lets sort that once we've flushed our tech debt into a release. | 19:39 |
*** gary-smith_ has joined #openstack-infra | 19:39 | |
anteaya | pleia2: thanks for letting us know | 19:39 |
lifeless | But | 19:39 |
lifeless | the chance that tehre is a nasty subtle thing biting us is worth investigating | 19:39 |
lifeless | so I'd like to have tchaypo's thing that looks like CD thing issues captured somewhere where I can poke closely at it with a stick | 19:40 |
mordred | lifeless: ++ | 19:40 |
lifeless | I'd like that to be a separate bug to the 'runs pip' bug | 19:40 |
mordred | lifeless: I'm not sure I can do that succinctly | 19:40 |
lifeless | tchaypo: ^ task for thou | 19:40 |
lifeless | mordred: now, there's a raft of pbr patches I WIP'd last week | 19:41 |
lifeless | mordred: from you | 19:41 |
mordred | lifeless: neat | 19:41 |
lifeless | mordred: I think they are dups from issues you merged other ways, but not sure. Could you please clean that up? | 19:41 |
clarkb | mordred: 167608 needs a rebase | 19:42 |
openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/shade: Add test for caching in list_images https://review.openstack.org/174717 | 19:42 |
openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/shade: Test flavor cache and add invalidation https://review.openstack.org/174707 | 19:42 |
openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/shade: Fix major update_user issues https://review.openstack.org/174704 | 19:42 |
openstackgerrit | Clint 'SpamapS' Byrum proposed openstack-infra/shade: Add design for an object interface https://review.openstack.org/164506 | 19:42 |
mordred | lifeless: ValueError: Unknown remainder ['post22'] in '1.1.0.post22' | 19:43 |
SpamapS | dohhh | 19:43 |
SpamapS | errant cherry-pick fail | 19:43 |
mordred | lifeless: I just got that trying to start gertty | 19:43 |
SpamapS | mordred: you should probably submit a new 164506 that isn't tied to my patch-series-of-doom | 19:43 |
mordred | lifeless: it's possible I'm not rebased on master of pbr -one sec | 19:43 |
mordred | lifeless: yah - nevermind- false alarm | 19:44 |
SpamapS | I really need to add an alternative keymap to gertty that doesn't have x == cherry pick (because x = "go back" in my mail client) | 19:44 |
mordred | SpamapS: for some reason I want to type "u" to go back | 19:45 |
* SpamapS rewrites gertty interface to work like zork | 19:46 | |
*** luqas has joined #openstack-infra | 19:46 | |
SpamapS | "review patch" | 19:46 |
SpamapS | "You see a patch on the wall. It is dripping with bugs." | 19:46 |
lifeless | mordred: postN is 0.10 behaviour | 19:47 |
asselin__ | clarkb, fungi so just to confirm, we're still going to change puppet-modules metadata.json name prefix from openstackci to openstackinfra? | 19:47 |
clarkb | asselin__: correct | 19:48 |
lifeless | mordred: oh yeah, and trunk can parse it now, so wtf :) | 19:48 |
openstackgerrit | Monty Taylor proposed openstack-infra/system-config: Add puppet-diskimage_builder to our module list https://review.openstack.org/167608 | 19:48 |
clarkb | asselin__: fbo re 168330, I have zero interest in making site.pp reconsumable | 19:48 |
mordred | clarkb: done | 19:48 |
openstackgerrit | Khai Do proposed openstack-infra/puppet-gerrit: Update bouncy castle libs for Gerrit 2.10 https://review.openstack.org/172534 | 19:48 |
Swami | clarkb: hi | 19:48 |
clarkb | asselin__: fbo site.pp is well site specific. I can be convinced that some things in openstack_project should be better but to suggest that our site.pp should also be reconsumable is :( | 19:48 |
clarkb | Swami: hello | 19:48 |
clarkb | asselin__: fbo but maybe I am misreading the comments there | 19:49 |
Swami | clarkb: can you add my ssh keys to the jenkins node server so that I can poke in. | 19:49 |
*** jose-idar has joined #openstack-infra | 19:49 | |
clarkb | Swami: can you point me at a key? | 19:49 |
Swami | sure: | 19:49 |
Swami | clarkb: #link https://launchpad.net/~swaminathan-vasudevan/+sshkeys | 19:49 |
*** dprince has quit IRC | 19:50 | |
asselin__ | clarkb, taking another look | 19:50 |
clarkb | Swami: done | 19:50 |
*** ayoung_ has joined #openstack-infra | 19:50 | |
Swami | clarkb: thanks | 19:50 |
jose-idar | Can someone take a look at this pull request, just need to know what the next step to getting it merged is: https://review.openstack.org/#/c/159642/ | 19:51 |
*** luqas has quit IRC | 19:51 | |
jose-idar | Please let me know if there's anything wrong with it too! | 19:51 |
asselin__ | clarkb, I think the approach fbo is taking is different than what I did, but the result is the same....(I think) | 19:51 |
clarkb | jose-idar: I think ajaeger commented on an issue | 19:52 |
clarkb | jose-idar: you want it to be non voting but it is voting | 19:52 |
asselin__ | clarkb, fbo is doing the refactor first. Then the puppet-zuul portions can be moved to openstackci. | 19:52 |
jose-idar | clarkb: indeed! i talked to the rest of the team and we all agreed that a voting job would be better, and so left it as-is | 19:52 |
asselin__ | clarkb, alternatively, fbo could refactor those parts, and then when getting infra to consume the new puppet scripts, it will look more like what his patch set shows currently. | 19:53 |
openstackgerrit | Merged openstack-infra/puppet-pip: Add the ability to set pip.conf and configure it https://review.openstack.org/168309 | 19:53 |
clarkb | asselin__: sure, I am not necessarily against the change itself but the comment response to nibalizer implies that site.pp is meant to be reconsumable | 19:53 |
asselin__ | clarkb, fbo but maybe it's more clear / less confusing to first refactor, then consume? | 19:53 |
clarkb | asselin__: that is what I am taking issue with | 19:53 |
asselin__ | clarkb, will look some more.... | 19:54 |
*** miqui has joined #openstack-infra | 19:54 | |
*** shashankhegde has joined #openstack-infra | 19:55 | |
*** kaisers has quit IRC | 19:56 | |
*** tsg has joined #openstack-infra | 19:56 | |
*** AJaeger has joined #openstack-infra | 19:57 | |
AJaeger | dolphm, ping | 19:57 |
*** julim has quit IRC | 19:58 | |
*** liusheng has quit IRC | 19:58 | |
*** hdd has quit IRC | 19:58 | |
*** liusheng has joined #openstack-infra | 19:59 | |
*** andreykurilin__ has quit IRC | 19:59 | |
*** Rockyg has joined #openstack-infra | 19:59 | |
anteaya | asselin__ fbo clarkb I had thought that I was clear when I said that the intent of the work was to leave system-config alone and make the split out puppet modules consumable | 20:00 |
*** andreykurilin__ has joined #openstack-infra | 20:00 | |
anteaya | now if I am wrong in saying that please correct me | 20:00 |
clarkb | anteaya: no you are not wrong | 20:00 |
anteaya | but I had thought that was the focus of the spec | 20:00 |
asselin__ | clarkb, fbo anteaya I guess that may be an issue with the "running your own" documentation / approach. | 20:01 |
anteaya | clarkb: thank you | 20:01 |
anteaya | asselin__: lets see if we can clear that up | 20:01 |
clarkb | anteaya: problem is people want to start with system-config because its already there. | 20:01 |
asselin__ | http://ci.openstack.org/running-your-own.html#initial-setup | 20:01 |
anteaya | since having new folks spinning their wheels is counterproductive and I want to support the new folks contributing | 20:01 |
anteaya | clarkb: ah yes, that makes sense, let's see if we can figure out a better place to begin? | 20:02 |
anteaya | ah ha | 20:02 |
anteaya | Clone the CI config repository and adjust it as necessary. | 20:02 |
anteaya | that may be the line we are stumbling on? | 20:02 |
clarkb | anteaya: I think so | 20:02 |
*** mrmartin has quit IRC | 20:03 | |
clarkb | http://ci.openstack.org/running-your-own.html#modules-openstack-project but if you read a little further | 20:03 |
asselin__ | clarkb, anteaya that's what fbo references to in his comment as welll | 20:03 |
clarkb | its not a great answer but it specifically says don't do what people are doing | 20:03 |
openstackgerrit | Merged openstack-infra/project-config: Adding pep8 gate check to Opencafe https://review.openstack.org/159642 | 20:04 |
*** tkelsey has joined #openstack-infra | 20:04 | |
anteaya | okay so do we have any better thoughts on how to get them redirected to a better behaviour | 20:04 |
dolphm | AJaeger: o/ | 20:04 |
lifeless | mordred: are there any other pbr things in the air atm ? | 20:04 |
AJaeger | dolphm: I'm just researching history for you for the translation setup... | 20:05 |
anteaya | I'll throw out an idea, asselin__ what does your config version look like and can we get it somwhere by itself that people can start with that? | 20:05 |
*** kgiusti has left #openstack-infra | 20:06 | |
dolphm | AJaeger: if you find justification, it should really go into comments in that script. otherwise, there's no reason not to change it to any other number, including 0 or 100 | 20:06 |
asselin__ | anteaya, I use the plain old puppet apply version: https://github.com/rasselin/os-ext-testing/blob/master/puppet/install_master.sh#L196 | 20:06 |
AJaeger | dolphm: the 75 % was the outcome of a discussion by the translation team. | 20:07 |
anteaya | asselin__: do you have any file or dir in your setup that is based off of system-config in any way? | 20:07 |
*** isviridov is now known as isviridov_away | 20:07 | |
AJaeger | See this bug: https://bugs.launchpad.net/horizon/+bug/1317794 and for example this message: http://lists.openstack.org/pipermail/openstack-i18n/2014-May/000622.html | 20:07 |
openstack | Launchpad bug 1317794 in openstack i18n "Complete translations are too big" [Undecided,In progress] - Assigned to Andreas Jaeger (jaegerandi) | 20:07 |
asselin__ | anteaya, no | 20:07 |
AJaeger | I can add the bug number to the script, would that help? | 20:07 |
dolphm | AJaeger: i mean that actual reasoning used to produce both 75% and 66% - not just "people talked about this once" | 20:08 |
anteaya | okay great, perhaps them maybe we can eliminate the line in the spec that directs people to do that? | 20:08 |
clarkb | asselin__: right thats the direction I think we want to push people, some of that will come through openstackinfra-openstackci but in general make your own composition layer | 20:08 |
anteaya | asselin__: and instead guides them to taking your approach? | 20:08 |
asselin__ | anteaya, I clone & depend on it: https://github.com/rasselin/os-ext-testing/blob/master/puppet/install_master.sh#L40 | 20:08 |
clarkb | asselin__: because all in one zuul + jenkins + nodepool isn't something system-config will do but others seem to want it | 20:08 |
anteaya | asselin__: oh | 20:08 |
asselin__ | anteaya, clarkb (this is from jaypipes) | 20:08 |
anteaya | clarkb: do any of my suggestions support your goals here? | 20:09 |
jose-idar | clarkb: thanks! | 20:09 |
AJaeger | dolphm: Those are not scientific numbers, these are the numbers the translators agreed on. | 20:09 |
anteaya | except perhaps depending on system-config may not be what we want | 20:09 |
armax_ | clarkb: I have mixed news | 20:09 |
asselin__ | anteaya, clarkb yes, it's what you need for a "small" 3rd party ci setup. No need to setup gerrit, etc. | 20:09 |
dolphm | AJaeger: that's not justification of any sort | 20:09 |
dolphm | AJaeger: i literally cannot fathom why you want to discard translation work - much less using some arbitrary threshhold. i'm looking for a WHY | 20:10 |
anteaya | how do we feel about creating a config file or repo which isn't system-config but which does what a third party ci setup needs | 20:10 |
dolphm | AJaeger: if a module in nova is only partially commented, should we discard half the comments there? | 20:10 |
clarkb | anteaya: that is what openstackinfra-openstackci is supposed to be | 20:10 |
openstackgerrit | Khai Do proposed openstack-infra/gerritlib: Update project docs https://review.openstack.org/165979 | 20:10 |
AJaeger | dolphm: we're not discarting translation work at all! | 20:10 |
anteaya | clarkb: ah ha | 20:10 |
AJaeger | dolphm: All the work is done in transifex! We just copy it over! | 20:11 |
asselin__ | :) | 20:11 |
anteaya | I was chasing my own tail | 20:11 |
dolphm | AJaeger: but you're removing some translation support from the deliverable, correct? | 20:11 |
AJaeger | dolphm: Yes, we do - temporarily. Once the translation team has translated more, we add it back in. | 20:11 |
AJaeger | dolphm: No single translation is lost since everything is in transifex | 20:12 |
asselin__ | anteaya, clarkb my goal is to migrate away from my github fork of jaypipes. Have an upstream solution and meets the needs of 3rd party ci. | 20:12 |
*** tsg has quit IRC | 20:12 | |
anteaya | asselin__: so what step adds the parts of system-config to openstackinfra-openstackci so that it can take the place of system-config for the work? | 20:13 |
dolphm | AJaeger: so why should translations be excluded from the deliverable? | 20:13 |
AJaeger | dolphm: The translators represent our international users and they dediced not to ship files that are not sufficient enough translated. | 20:13 |
anteaya | asselin__: so that fbo can make changes to openstackinfra-openstackci rather than system-config for his zuul patch? | 20:13 |
*** tkelsey has quit IRC | 20:13 | |
anteaya | because I believe that is what would work better for clarkb, am I correct, clarkb? | 20:14 |
dolphm | AJaeger: again, i'm looking for a WHY - the actual reasoning that brought them to that decision. the WHY should live in your script which is implementing the resulting action | 20:14 |
dolphm | AJaeger: the only assertion made in the script / commit message is that they would "not be useful" and i directly disagree with that assertion, as vague as it is. | 20:14 |
clarkb | anteaya: Ideally we would coordinate on the common goal yes. I was mostly just ocncerned that it seemd that site.pp was maybe something people thought should be reconsumable? | 20:14 |
*** tkelsey has joined #openstack-infra | 20:15 | |
asselin__ | anteaya, fbo can refactor the zuul portions, remove hard-coded parameters and submit that patch to puppet-openstackci. | 20:15 |
*** jogo has joined #openstack-infra | 20:15 | |
anteaya | ah yes, I made the point it won't be reconsumable in the meeting when it came up, but it bears repeating | 20:15 |
clarkb | armax_: what did you find? | 20:15 |
anteaya | asselin__: I personally would be in support of such a direction | 20:15 |
*** andreykurilin__ has quit IRC | 20:16 | |
armax_ | clarkb: so…wanna hear the bad or the good news first? | 20:16 |
asselin__ | anteaya, fbo can then propose a new patch to system-config with cross-project depends-on to the other patch, removing zuul from system-config, and using the common solution. | 20:16 |
clarkb | armax_: bad news | 20:16 |
AJaeger | dolphm: Just a sidenote, please check https://review.openstack.org/#/c/94541/ - I wonder why this comes up today | 20:16 |
anteaya | asselin__: so puppet-openstackci is the puppet module for openstackinfra-openstackci? | 20:16 |
anteaya | so many repos | 20:16 |
Swami | armax_:clarkb:hi | 20:16 |
asselin__ | anteaya, yes | 20:16 |
anteaya | asselin__: great, thank you for your patience with me | 20:17 |
*** andreykurilin__ has joined #openstack-infra | 20:17 | |
dolphm | AJaeger: i recall the ones being removed as being *completely* untranslated, not *mostly*. skimming the deleted files again, i actually don't see any files with translations? | 20:17 |
armax_ | clarkb: well the somewhat bad news is that the multi-node setup is helping manifest internal races to the DVR code | 20:17 |
clarkb | armax_: that sounds like good news :) | 20:18 |
armax_ | clarkb: that’s why I am saying somewhat bad news :) | 20:18 |
Swami | armax_: what was the race that you found. | 20:18 |
AJaeger | dolphm: we deleted them across projects, might be that keystone was different there | 20:18 |
armax_ | clarkb: in the sense that there’s work to do, but the tests seem to work as they should | 20:18 |
armax_ | clarkb: and the setup is rock solid | 20:18 |
armax_ | clarkb: if I execute the tests steps manually | 20:18 |
Swami | armax_: clarkb: that is good news | 20:19 |
armax_ | clarkb: or even execute the failed tempest tests manually one by one, they pass | 20:19 |
anteaya | armax_ clarkb congratulations | 20:19 |
clarkb | armax_: cool so the environment is fine, its just a matter of working throuhg the bugs it exposes | 20:19 |
armax_ | clarkb: indeed | 20:19 |
dolphm | AJaeger: is it not possible to have po files contain *either* a translated message, or exclude the message altogether in favor of the original in code? | 20:20 |
armax_ | clarkb: as bad as it sounds, my current working theory is that dvr has latent races that are exposed via the multi-node level of concurrency and the fact that now we have two l3 agents pounding the server | 20:20 |
*** otter768 has joined #openstack-infra | 20:20 | |
clarkb | fungi: for 171286, we are likely going to need more disk space on nodepool.o.o before adding that diskimage | 20:20 |
*** yfried__ is now known as yfried|afk | 20:20 | |
armax_ | clarkb: not that great if you ask me, but the fact that tests pass when called in isolation or even manual steps seem to imply | 20:21 |
AJaeger | dolphm: you might have two strings displayed one ofter the other, the first one translated, the second one not. | 20:21 |
armax_ | clarkb: that besides the odd condition, there’s nothing persistently broken | 20:21 |
fungi | wow. >300 lines of scrollback while i was outside. if any of it was important for me to look at, let me know ;) | 20:21 |
AJaeger | dolphm: Our i18n tools always try to translate. And if you have let's say 10 % translated, then 1 out of 10 strings is for example in German, the rest in English. | 20:21 |
dolphm | AJaeger: why/where would you have that? | 20:21 |
*** achanda has quit IRC | 20:22 | |
clarkb | fungi: just the ping a couple lines up | 20:22 |
fungi | yeah, saw the pings | 20:22 |
dolphm | AJaeger: right, okay, so can you exclude all the original english from the po files? | 20:22 |
armax_ | clarkb: so the bad/good news is that we’d need to painstakingly go through the logs to try and identify why the l3 agents fail to complete the setup at the pace that tests require | 20:22 |
*** fawadkhaliq has joined #openstack-infra | 20:22 | |
fungi | clarkb: and thanks, looking into it now | 20:22 |
anteaya | asselin__: also I'm wondering if we should re-evaluate the wording of this first paragraph: http://specs.openstack.org/openstack-infra/infra-specs/specs/openstackci.html#proposed-change | 20:22 |
dolphm | AJaeger: so what the po files contain is 100% translated strings, even if that only covers 10% of the strings in the project? | 20:22 |
clarkb | armax_: ok, that sounds useful since we don't hit this cloud with that much load | 20:22 |
armax_ | Swami: I haven’t identified a precise bug per se | 20:22 |
AJaeger | dolphm: yes, we could filter our po files that way. | 20:22 |
*** hdd has joined #openstack-infra | 20:23 | |
Swami | armax_: got it. | 20:23 |
zaro | fungi: I was hoping that Gerrit far enough for me to get that error on my VM when I uncommented the contact store configs but instead it hangs pretty early without giving me that error. | 20:23 |
AJaeger | dolphm: but that's still 10 % coverage of the totally number of strings | 20:23 |
dolphm | AJaeger: understood - is there a reason why you would not do it that way? | 20:23 |
anteaya | asselin__: perhaps something like take the necessary information from system-config and create generic puppet scripts in puppet-openstackci | 20:23 |
AJaeger | translators can edit directly the po files and upload/download them. | 20:23 |
armax_ | clarkb: we went through the same exercise of understanding what bottlenecks were causing spurious failures for dvr and the single node deployment | 20:23 |
zaro | fungi: i would hate to have to go back to 2.8 on review-dev to verify that error. can you think of anything else to try? | 20:24 |
AJaeger | dolphm: The standard tools also do not support it directly. | 20:24 |
armax_ | clarkb: I guess now it’s the time to identify those for multi-node | 20:24 |
asselin__ | anteaya, yes, possibly, the spec when through a lot of changes. We can certainly increase clarity now | 20:24 |
dolphm | AJaeger: openstack tooling? | 20:24 |
zaro | fungi: if not i can revert review-dev back to 2.8 and try to repro that problem. | 20:24 |
clarkb | fungi: greghaynes also I am annoyed that centos6 has a centos element and centos7 has a centos7 element | 20:24 |
AJaeger | dolphm: we could enhance our toolchain to download files, do some preprocessing on them as you suggest and go from there. | 20:24 |
AJaeger | dolphm: The usual gettext and sphinx tools do not handle it. | 20:24 |
fungi | zaro: nope, though i can try again when we have a quiet period some evening, if you can think of other things i should check on review.o.o while it's failing to start | 20:24 |
*** otter768 has quit IRC | 20:24 | |
*** andreykurilin__ has quit IRC | 20:24 | |
anteaya | asselin__: we can offer the patch and it can be reviewed for it, if others disagree they can say so, you can mention the initial understanding new contributors to the spec have | 20:25 |
dolphm | AJaeger: wouldn't gettext just fallback to the original string it's being asked to translate? | 20:25 |
AJaeger | dolphm: we could experiment with your suggestion - but not for kilo, please ;) | 20:25 |
fungi | clarkb: agreed. that's marvellously inconsistent | 20:25 |
armax_ | clarkb: perhaps mine is not a great finding, just yet, but only thing I can say is that I ‘can’t reproduce it' | 20:25 |
armax_ | clarkb: even on the ‘faulty’ box | 20:25 |
clarkb | armax_: probably going to need to make logging better and trace through logs to find the races | 20:25 |
armax_ | clarkb: yes | 20:25 |
AJaeger | dolphm: gettext should fallback, yes | 20:26 |
anteaya | asselin__: but in the meantime I think we might have found the more productive direction | 20:26 |
asselin__ | anteaya, ok...I think the issue people haven't read the spec yet / aren't aware of its existence | 20:26 |
*** shardy is now known as shardy_z | 20:26 | |
dolphm | AJaeger: what would sphinx do? (does sphinx not use gettext?) | 20:27 |
AJaeger | dolphm: sphinx uses gettext libraries | 20:27 |
anteaya | asselin__: oh yes, well that certainly would add to the difficulty | 20:27 |
AJaeger | dolphm: ignore me, for keystone sphinx is not part of the toolchain. | 20:28 |
armax_ | clarkb: even if I ran just the scenario tests at once, they are passing | 20:28 |
* AJaeger has been hacking on RST translation using Sphinx too much the last weeks | 20:28 | |
clarkb | armax_: but serially right? | 20:28 |
armax_ | clarkb: no | 20:28 |
*** tsg has joined #openstack-infra | 20:28 | |
armax_ | clarkb: I am just running ./run-tempest.sh tempest.scenario, from what I can tell they run concurrently | 20:29 |
openstackgerrit | Kyle Rockman proposed openstack-infra/jenkins-job-builder: Adding hipchat publisher https://review.openstack.org/174036 | 20:29 |
armax_ | clarkb: that said, I don’t think that having us get hold of the nodes is gonna give us much more information than the one that we already have through the logs, especially when all the neutron data related to the tests is wiped | 20:29 |
clarkb | armax_: ok | 20:30 |
openstackgerrit | Andreas Jaeger proposed openstack-infra/release-tools: Add script to cleanup of translations https://review.openstack.org/172827 | 20:30 |
mordred | lifeless: I don't think so | 20:31 |
*** _nadya_ has joined #openstack-infra | 20:31 | |
mordred | lifeless: well, Daviey has a patch up that is important to him | 20:31 |
armax_ | clarkb: a somewhat interesting finding is the fact that if I disable the compute/l3-agent on the primary node, I never even get the occasional failure | 20:32 |
armax_ | clarkb: this leads me to believe that there’s a load induced problem | 20:32 |
clarkb | armax_: its not doing dvr at that point though right? | 20:32 |
clarkb | that is essentially equivalent to the non dvr job since only q-agt is running on the controller | 20:33 |
armax_ | clarkb: well, I only disabled n-cpu from the nova-manage console | 20:33 |
armax_ | clarkb: so the VM’s are forced to boot on the subnode | 20:33 |
clarkb | sorry s/controller/compute node/ | 20:33 |
armax_ | clarkb: no, it’s still dvr, but it’s like it ran single node | 20:34 |
Swami | armax_: yesterday I saw in my patch, that there were two tests that failed, those were the "selve-instance test" and the "volume" test. | 20:34 |
clarkb | armax_: whats distributed about it? | 20:34 |
armax_ | clarkb: it’s basically the same test that Swami did with https://review.openstack.org/#/c/175056/ | 20:34 |
Swami | armax_: when i forced everything to a single compute node. | 20:34 |
AJaeger | dolphm: msgattrib --translated should allow such filtering. | 20:35 |
*** achanda has joined #openstack-infra | 20:35 | |
*** tsg has quit IRC | 20:35 | |
armax_ | clarkb: well, the FIP is ‘distributed’ because it’s local to the compute host | 20:35 |
armax_ | the control plane logic is still DVR | 20:35 |
AJaeger | dolphm: I'll add this to my todo list for the openstack translation toolchain and will see whether this works fine. | 20:35 |
greghaynes | clarkb: yes, I think it is universally agreed that the centos vs centos7 is a bad thing | 20:35 |
lifeless | mordred: that landed | 20:35 |
greghaynes | clarkb: If I had a reason to id be up for fixing... | 20:36 |
lifeless | mordred: I fixed it up | 20:36 |
armax_ | clarkb: and there’s still code paths that recognize the distributed nature of the deployment | 20:36 |
mordred | greghaynes, clarkb: centos7 / centos is fixed in the minimal elements :) | 20:36 |
armax_ | clarkb: however the data plane nature of DVR is only triggered when you have multiple VM’s hitting each other | 20:36 |
mordred | greghaynes: o hai! | 20:37 |
clarkb | armax_: ok so ^ could explain why it fails when in the other configuration? | 20:37 |
greghaynes | yep, I should see if I can dig up the reason for that, I remember it came up in review | 20:37 |
*** deepakcs has quit IRC | 20:37 | |
greghaynes | mordred: hello there | 20:37 |
mordred | greghaynes: so, my brain cannot wrap its head aroudn what needs to happen in those tests in nodepool | 20:37 |
greghaynes | mordred: awesome. that is next on my list of things to hack on, one I get these slides fixed up... | 20:38 |
greghaynes | I dont entirely remember either | 20:38 |
fawadkhaliq | folks quick question: running this command "ssh -i KEY 29418 plumgrid-ci@review.openstack.org gerrit review -m '"msg"' --verified 1 CHANGETSET,PATCH" does not show +1 verified on gerrit. only the message shows up. Any help would be appreciated! | 20:38 |
armax_ | clarkb: you mean when both primary and subnode are active | 20:38 |
armax_ | ? | 20:38 |
clarkb | armax_: yes | 20:38 |
fawadkhaliq | seems to me..the permissions for this user could not have verified option allowed/enabled, possible? | 20:39 |
clarkb | fawadkhaliq: you rpobably do not have permissions to leave a verified +1 on that project | 20:39 |
anteaya | fawadkhaliq: I hope you are testing on http://git.openstack.org/cgit/openstack-dev/ci-sandbox/ | 20:39 |
fawadkhaliq | clarkb: perfect. just what I thought. | 20:40 |
clarkb | zaro: for 173052 should we be upgrading that plugin prior to upgrading gerrit to 2.10? | 20:40 |
*** sdake_ has joined #openstack-infra | 20:40 | |
clarkb | zaro: if not maybe it should be WIP? | 20:40 |
fawadkhaliq | clarkb: and thanks! | 20:40 |
dolphm | AJaeger: cool! | 20:41 |
armax_ | clarkb: I don’t have definite proof yet, but my working theory right now is that the two l3 agents stomp on each other….not a very analytical and accurate theory I must admit, but I’d need to dive into the logs a lot deeper to see what’s going on | 20:41 |
clarkb | nibalizer: still awake? for the openstack_project::server move into site.pp node defs should that happen every your merge for template and server happens? | 20:42 |
armax_ | clarkb: I don’t have a better answer for you right now | 20:42 |
clarkb | armax_: gotcha, just making sure we have a next step | 20:42 |
clarkb | sounds like we do | 20:42 |
*** amitgandhinz has quit IRC | 20:42 | |
armax_ | clarkb: well, yes, we need to cross reference logs and figure out what’s the exact sequence of actions and where the glitches happen | 20:42 |
mordred | greghaynes: yay! | 20:43 |
*** kevinbenton has joined #openstack-infra | 20:43 | |
armax_ | clarkb: in the process we may need to beef up logs to gather more intelligence | 20:43 |
armax_ | clarkb: past experience showed us that we don’t necessary need to dive into the namespaces | 20:43 |
*** sdake has quit IRC | 20:44 | |
armax_ | clarkb: these failure modes, the inactive fip and the ssh failures are dear to Swami | 20:44 |
*** kevinbenton has quit IRC | 20:44 | |
armax_ | :) | 20:44 |
Swami | armax_: sure, they are well known behavior of DVR | 20:44 |
armax_ | clarkb: and me | 20:44 |
*** kevinbenton has joined #openstack-infra | 20:44 | |
armax_ | clarkb: we’d need to figure out why/when they happen more recurrently during multinode | 20:44 |
clarkb | armax_: I'm curious why namespaces are used at all considering that ovs is already controlling the data flows | 20:45 |
*** hashar has quit IRC | 20:45 | |
clarkb | eg there shouldn't be any unwanted cross talking right? | 20:45 |
clarkb | fungi: pleia2 https://review.openstack.org/#/c/141182/ would be a good one to get in soon as there are at least a couple changes that want to change our puppet module deps | 20:45 |
openstackgerrit | yolanda.robla proposed openstack-infra/puppet-jeepyb: Add helpers to execute manage_projects https://review.openstack.org/171538 | 20:46 |
clarkb | fungi: pleia2 it has the necessary +2s so I will likely approve unless you all wanted to take a look first | 20:46 |
*** _nadya_ has quit IRC | 20:46 | |
*** samuelBartel has quit IRC | 20:46 | |
armax_ | clarkb: to this aim, I wonder what you think if we switched the job to the check queue at least just for the neutron project; that would help us gather vital intelligence to identify the reliability gap between multi vs single node and would give us an awful lot of data points to go through | 20:47 |
*** amitgandhinz has joined #openstack-infra | 20:47 | |
armax_ | clarkb: to some degree we’ve established that besides, these two failure modes, we’re free from other nastier races | 20:47 |
openstackgerrit | Merged openstack-infra/project-config: Correct platform codenames in bindep fallback https://review.openstack.org/172674 | 20:47 |
clarkb | armax_: it would be good to have a better understanding of why it fails before we run a test that fails 95% of the time a few thousands times per day | 20:47 |
clarkb | armax_: IME it has been rare that we actually need to run it at that scale to make a job reliable | 20:47 |
armax_ | clarkb: true, but can we tell we’re at 95? | 20:47 |
clarkb | armax_: I have not seen it pass on any of my patches | 20:48 |
clarkb | armax_: so I think its actually closer to 100% | 20:48 |
clarkb | armax_: but, we can check logstash and graphite | 20:48 |
*** dboik has joined #openstack-infra | 20:48 | |
openstackgerrit | Merged openstack-infra/system-config: update Gerrit javamelody plugin on review-dev.o.o https://review.openstack.org/173050 | 20:48 |
armax_ | clarkb: we were still troubleshooting setup issues at that point though | 20:48 |
armax_ | clarkb: that said we can wait a little longer I suppose | 20:49 |
clarkb | armax_: not for the last week | 20:49 |
armax_ | clarkb: oh ok, then I think it’s wiser to hold on for a tad longer | 20:49 |
*** _nadya_ has joined #openstack-infra | 20:50 | |
armax_ | clarkb: I’ll circle back with Swami to work out the next steps | 20:50 |
openstackgerrit | yolanda.robla proposed openstack-infra/puppet-jeepyb: Add helpers to execute manage_projects https://review.openstack.org/171538 | 20:50 |
Swami | armax_: clarkb: yes we can work on a plan | 20:51 |
armax_ | if Swami doesn’t need the node any longer, I guess you get free them up | 20:51 |
*** dboik_ has quit IRC | 20:51 | |
*** sdake has joined #openstack-infra | 20:52 | |
clarkb | no rush | 20:52 |
armax_ | clarkb: ok | 20:52 |
armax_ | that’s been great help though, thank you! | 20:53 |
*** MaxV has quit IRC | 20:53 | |
*** MaxV has joined #openstack-infra | 20:53 | |
nibalizer | clarkb: those changes are orthoginal | 20:53 |
Swami | clarkb: thanks clarkb | 20:53 |
*** eharney has quit IRC | 20:54 | |
dougwig | mordred: ping, did you want to chat about how to setup your cloud with non-private ip's? | 20:54 |
clarkb | nibalizer: ok, Ithought they were related | 20:54 |
openstackgerrit | Tristan Cacqueray proposed openstack-infra/puppet-openstackci: Create a jenkins server like http://jenkins.openstack.org https://review.openstack.org/175571 | 20:54 |
fungi | clarkb: i'm skimming back through 141182 quickly... any easy explanation as to why there are several additional modules brought in by the patch? dependencies? | 20:54 |
openstackgerrit | Merged openstack-infra/system-config: Remove duplicate puppet-simpleproxy https://review.openstack.org/173281 | 20:54 |
mordred | dougwig: I don't have a cloud right now - but I understand that just using "provider network" is the thing I probably want, yeah? | 20:55 |
*** e0ne has quit IRC | 20:55 | |
openstackgerrit | yolanda.robla proposed openstack-infra/system-config: Move manage_projects logic to puppet-jeepyb https://review.openstack.org/171565 | 20:55 |
*** annegentle has joined #openstack-infra | 20:55 | |
*** annegent_ has joined #openstack-infra | 20:55 | |
*** annegentle is now known as Guest30256 | 20:56 | |
*** sdake_ has quit IRC | 20:56 | |
fungi | concat, staging and wget. seem likely second-order transitive deps | 20:56 |
nibalizer | clarkb: same spec, but both are forward compatible refactors | 20:56 |
nibalizer | fungi: if concat is < 2.0 run the other direction | 20:57 |
fungi | aha, commit message does mention transitive deps. approved | 20:57 |
*** Guest30256 has quit IRC | 20:57 | |
nibalizer | concat 2.0 is a srsbsns break and should be viewed with caution | 20:57 |
clarkb | fungi: yup, since we aren't using puppet module to install we need to handle transitive deps ourselves | 20:57 |
nibalizer | idunno what wget is but staging and archive are ++good | 20:57 |
*** MaxV has quit IRC | 20:57 | |
clarkb | fungi: but this gives us much greater control and the total number of transitive deps is tiny so worth the human cost imo | 20:57 |
clarkb | fungi: alternative is we end up having no idea what we are running. Also ty | 20:57 |
fungi | clarkb: you mean the current situation ;) | 20:58 |
*** stevemar has quit IRC | 20:58 | |
clarkb | fungi: exactly | 20:58 |
fungi | it's not like we need any additional excuses to drink | 20:58 |
*** AJaeger has quit IRC | 20:58 | |
clarkb | fungi: I did a new thing yesterday. I sat on the patio in the sun reading a book with a glass of bruichladdich. | 20:59 |
fungi | from its name, i'm going to guess that's a scotch i haven't tried yet | 20:59 |
mordred | clarkb: that sounds like a wonderful use of a sunday | 20:59 |
clarkb | was an excellent non computered afternoon | 20:59 |
mordred | clarkb: computers suck | 20:59 |
fungi | aye, scotch it be | 21:00 |
mordred | clarkb: I did fish butchering yesterday | 21:00 |
clarkb | fungi: its a non peated islay | 21:00 |
clarkb | mordred: nice did you catch the fish? | 21:00 |
*** jose-idar has left #openstack-infra | 21:00 | |
mordred | clarkb: I did not. but I cut it all up anyway | 21:00 |
nibalizer | clarkb: for 141182 one thing im wondering | 21:00 |
zaro | fungi: so one thing i did notice is that 'TeeOutputStream.class' is in both 'WEB-INF/lib/commons-io-1.4.jar' and 'WEB-INF/plugins/replication.jar' | 21:00 |
nibalizer | is the script smart enough to move a module from 'installed by module-tool' to 'git' ? | 21:01 |
nibalizer | basiclly it would have to remove the dir and clone in that case right? | 21:01 |
clarkb | nibalizer: ya it deletes the old one then clones in the new one | 21:01 |
nibalizer | sweeet | 21:01 |
*** _nadya_ has quit IRC | 21:01 | |
clarkb | nibalizer: and it doesn't race with puppet agents because the run script does that first then runs agents on all the nodes | 21:01 |
nibalizer | oh awesome | 21:01 |
nibalizer | we're all kinds of figure-8 races downstream with puppet modules | 21:01 |
nibalizer | its annoying | 21:01 |
nibalizer | i should fix it | 21:02 |
zaro | fungi: i'm wondering if the core plugins got installed? http://git.openstack.org/cgit/openstack-infra/puppet-gerrit/tree/manifests/init.pp#n573 | 21:02 |
openstackgerrit | Stefano Maffulli 'reed' proposed openstack-infra/infra-manual: Adds a link for new developers to use sandbox https://review.openstack.org/104951 | 21:03 |
*** tkelsey has quit IRC | 21:04 | |
mordred | clarkb: I was thinking over the weekend that we should add a lockfile to our run_all.sh script - so that when we're doing a maint task, we can "turn off puppet" just by touching the lockfile instead of having to edit the crontab | 21:04 |
*** BobBall is now known as BobBall_AWOL | 21:04 | |
mordred | clarkb, fungi: ^^ (this came into my head while hacking on the rename-projects playbook) | 21:05 |
clarkb | mordred: I thought we do have one | 21:05 |
mordred | we do? | 21:05 |
clarkb | ya in the cron job so that two runs don't overlap | 21:05 |
mordred | ah - in the cronjob | 21:05 |
*** bswartz has quit IRC | 21:09 | |
fungi | zaro: it's possible something about that patch was causing it not to update those, though all the non-disabled plugins have a last-updated time of when i pushed the revert, so it worked when going back to the war we had been running | 21:09 |
*** tiswanso has joined #openstack-infra | 21:09 | |
openstackgerrit | Merged openstack-infra/system-config: Install all puppet modules from source https://review.openstack.org/141182 | 21:09 |
dougwig | mordred: yes, though it's one of our less well documented operating modes, so feel free to ping when you get there. i added a few gray hairs getting it setup, but now it works dandy, including single NIC. | 21:11 |
mordred | dougwig: woot! because I think we'll actually be doing single NIC | 21:12 |
*** amitgandhinz has quit IRC | 21:12 | |
mordred | dougwig: I'll be sure to poke you when we get there | 21:12 |
clarkb | fungi: 141182 should apply in just a couple minutes, looks like the *:15 run is the next puppet run all | 21:13 |
fungi | excellent | 21:13 |
zaro | fungi: yeah, i have no other idea why you would get that error. it's most likely caused by 2 different bouncy castle jars getting loaded. | 21:14 |
fungi | as for the additional nodepool image, we seem to have 18gb free in /opt right now, 36gb in use by /opt/nodepool_dib | 21:14 |
mordred | clarkb: so - I can't think of a CLEAN way to do the thing I want to do WRT to that lock file - hopefully you can tell me a smarter thing | 21:15 |
zaro | fungi: any hidden files in the review_site/lib or review_site/plugins folder? | 21:15 |
mordred | clarkb: what I really want to do is put a task at the top of the rename-project playbook which grabs the lockfile - and one at the end that releases it | 21:15 |
mordred | clarkb: because we _don't_ want it to timeout or release after runnign ansible - in case there was an error | 21:15 |
clarkb | mordred: I think you just flock -en, do your stuff, then flock -u | 21:15 |
fungi | zaro: nope, no dotfiles in those at all | 21:15 |
mordred | clarkb: but doesn't it wrap a command? | 21:16 |
*** tiswanso has quit IRC | 21:16 | |
clarkb | mordred: doesn't have to, thats what flock -n says don't block just get lock or fail | 21:16 |
mordred | clarkb: on my box, you have to give flock either a command or a file descriptor | 21:16 |
*** tiswanso has joined #openstack-infra | 21:17 | |
greghaynes | make a subshell reading from an fd, then flock it | 21:17 |
clarkb | oh hrm | 21:17 |
mordred | greghaynes: but I have to keep that subshell around | 21:17 |
mordred | no? | 21:17 |
zaro | fungi: well. looks like that TeeOutputStream.class is in bcprov lib | 21:18 |
greghaynes | yes, I just popped in at the end... must be some context im missing | 21:18 |
greghaynes | that is just the bash lock a command recipe | 21:18 |
mordred | greghaynes: I want a command that grabs the flock and returns and holds the lock until I flock -u | 21:18 |
*** dangers is now known as dangers_away | 21:18 | |
mordred | because I want to prevent an automtaed tasks from running until I'm done | 21:18 |
*** radez is now known as radez_g0n3 | 21:19 | |
greghaynes | oh, I see, you have an existing lock | 21:20 |
openstackgerrit | Dave McCowan proposed openstack-infra/project-config: Promote Bandit Gate from Experimental to Check Non-voting for Barbican https://review.openstack.org/175577 | 21:20 |
fungi | we could tweak the cronjob to pass flock a specific filename. you could grab the lock by wrapping a background sleep | 21:21 |
fungi | </random_idea> | 21:21 |
mordred | fungi: yeah - I think I'll have to wrap a background something | 21:21 |
*** cdent has quit IRC | 21:21 | |
* mordred frowns | 21:21 | |
openstackgerrit | yolanda.robla proposed openstack-infra/system-config: Bump stankevich-python version https://review.openstack.org/174820 | 21:21 |
clarkb | puppet modules appear to have installed via git | 21:21 |
mordred | \o/ | 21:22 |
clarkb | (I checked the wget module and it has a .git in it) | 21:22 |
clarkb | so keep an eye out for oddities on puppetbaord (I will be watching there and have the run all log tailed too) | 21:22 |
*** hdd has quit IRC | 21:22 | |
openstackgerrit | yolanda.robla proposed openstack-infra/system-config: Bump stankevich-python version https://review.openstack.org/174820 | 21:23 |
clarkb | now, I think that if we propose the module bump for puppetdb/postgres it should fail | 21:23 |
clarkb | I think | 21:24 |
fungi | clarkb: what's our ~%/diskimage in /opt? we're at 75% of 72gb used. wondering how big a cinder volume we want to attach | 21:24 |
fungi | or if we need to do some cleanup in there for deleted images | 21:25 |
*** jgrimm is now known as zz_jgrimm | 21:25 | |
clarkb | fungi: each image is 3-5GB depending on file format and what gets cached. And we need 3*image size for each image build iirc | 21:25 |
fungi | looks like we're at 4 diskimages in nodepool.yaml at the moment | 21:25 |
clarkb | fungi: that free space is necessary to build the image | 21:25 |
*** dboik has quit IRC | 21:26 | |
mordred | we'll need more once we start doing vhd's too | 21:26 |
mordred | and if we add dreamhost as a provider, we'll also need raw's | 21:26 |
*** jaypipes has quit IRC | 21:26 | |
fungi | did we get a bump to our cinder quota a while back? | 21:26 |
clarkb | yes so we should probably figure 15GB per image with ~50GB of free space used to build + cache | 21:26 |
greghaynes | we cant consolodate at the same time we do vhds? | 21:26 |
clarkb | greghaynes: you need one copy of each format | 21:27 |
greghaynes | oh right, clouds not being same format :( | 21:27 |
*** dboik has joined #openstack-infra | 21:27 | |
mordred | greghaynes: what's the deal with profiledir.0TIpkZ ? | 21:27 |
clarkb | sorry the 50GB of free space is me thinking about size enough to make one image and any file caching we need to do (and its not a conservative estimate) | 21:27 |
greghaynes | mordred: wah? in the cachedir? | 21:28 |
mordred | yah | 21:28 |
clarkb | mordred: I think that happens when you have a build fail | 21:28 |
clarkb | dib isn't good about cleaning up in all fail cases | 21:28 |
mordred | we have 29 of those | 21:28 |
mordred | awesome | 21:28 |
greghaynes | hrm, I dont have that... unsure | 21:29 |
openstackgerrit | Merged openstack-infra/elastic-recheck: Remove old queries for "fixed" bugs https://review.openstack.org/175529 | 21:29 |
mordred | we also have 97 .d files in the nodepool_dib dir | 21:29 |
mordred | we also have 97 .d directories in the nodepool_dib dir | 21:29 |
greghaynes | those are probably manifests | 21:29 |
*** e0ne has joined #openstack-infra | 21:30 | |
greghaynes | 1 per image name * number of times your changed image names? | 21:30 |
greghaynes | I am curious about this profiledir business though | 21:31 |
mordred | so since we have 4 image types * 2 for each (for fallback) - that's 8 images at the moment | 21:31 |
clarkb | oh and we keep two copies of each image file so double the 15GB | 21:31 |
mordred | clarkb: well, but you don't need the 3x for both | 21:31 |
clarkb | mordred: you need 3x for one build iirc | 21:31 |
mordred | yah - but then it goes away | 21:31 |
clarkb | mordred: ya so you count it once | 21:31 |
clarkb | not for each image, its just build overhead | 21:31 |
mordred | clarkb: so really it's in the 50G | 21:32 |
mordred | of build overhead | 21:32 |
clarkb | mordred: yes | 21:32 |
greghaynes | then youall have some job to purge old images? | 21:32 |
clarkb | mordred: then 30GB is 2 days of images times each fomrat | 21:32 |
mordred | well then - if we count 15G per image | 21:32 |
mordred | yah | 21:32 |
mordred | so 120G of images AT LEAST | 21:32 |
clarkb | raw, qcow2, vhd | 21:32 |
*** amitgandhinz has joined #openstack-infra | 21:32 | |
greghaynes | any reason you need more than just the raw for old images? | 21:32 |
fungi | i'm going to eyeball our cacti graphs too | 21:32 |
greghaynes | its easy to remake the other formats if you need it | 21:32 |
mordred | greghaynes: because we don't have a way to ask for dib to do the conversion for us | 21:33 |
clarkb | greghaynes: simplicity, would rather throw disk at a problem then figure out conversions later | 21:33 |
mordred | ++ | 21:33 |
greghaynes | yea, thats fair | 21:33 |
clarkb | its also faster | 21:33 |
mordred | right now, dib contains the logic for conversinos | 21:33 |
fungi | hrm, we don't have /opt tracked in cacti? | 21:33 |
clarkb | fungi: thats odd | 21:33 |
clarkb | since boot script sets up /opt for us and then reboots which should kick snmpd | 21:34 |
mordred | clarkb: btw - raw and vhd are 50% bigger than qcow2 locally for me | 21:34 |
clarkb | mordred: I see almost twice as big. qcow2 is roghly 3GB for our images and 5GB for vhd/raw | 21:34 |
clarkb | mordred: this is also why all clouds should accept qcow2 | 21:34 |
mordred | clarkb: so we might want to consider it 20G / image - so 40G * 4 formats for 160G of image | 21:34 |
fungi | though the memory graph is pretty amusing. you can distinctly see each of the four diskimage builds happening in series | 21:34 |
greghaynes | mordred: any idea how much smaller the -mininal is versus not? | 21:34 |
clarkb | greghaynes: its not that much smaller | 21:34 |
greghaynes | :( | 21:34 |
*** hichihara has joined #openstack-infra | 21:34 | |
clarkb | greghaynes: because all of our disk is in git and image caching | 21:34 |
mordred | yah | 21:34 |
greghaynes | womp womp | 21:34 |
mordred | minimal-trusty without git caching is 845M | 21:34 |
*** vkmc has left #openstack-infra | 21:35 | |
*** tonytan4ever has quit IRC | 21:35 | |
*** e0ne has quit IRC | 21:35 | |
mordred | I haven't built one WITH repo caching in a while | 21:35 |
mordred | because size | 21:35 |
mordred | clarkb: did I tell you flaper87 added cloud-side image conversions in kilo glance? | 21:36 |
mordred | clarkb: currently it only does qcow2 <-> raw ... but I'd imagine that making a patch in rackspace's case to go from either to vhd should be pretty easy | 21:37 |
clarkb | mordred: http://puppetboard.openstack.org/report/nodepool.openstack.org/f5b5049537265a5e8e7a85439f6b082c0bc3beb7 is an issue | 21:37 |
*** mattfarina has quit IRC | 21:37 | |
mordred | now - if only we could convince our clouds to deploy such a thing | 21:37 |
mordred | looking | 21:37 |
fungi | okay, so 20gb per diskimage means we should be able to support 25 diskimages in a half-terabyte volume. too big? not quite too big enough? | 21:37 |
* clarkb is also looking at http://puppetboard.openstack.org/report/proposal.slave.openstack.org/e28ef19e51bbf81c07ebb9a1eff255a6008e5e4b | 21:37 | |
mordred | clarkb: I uploaded a new package version to fix ... oh, piddle | 21:38 |
clarkb | fungi: its ~40 because we keep two days | 21:38 |
*** bswartz has joined #openstack-infra | 21:38 | |
fungi | clarkb: okay, so 1tb for 25 diskimages | 21:38 |
mordred | clarkb: I will fix | 21:38 |
mordred | why do we think 25 diskimages? | 21:38 |
fungi | mordred: i'm picking a number out of the air | 21:38 |
mordred | what past the 4 we have now do we expect to grow? | 21:38 |
mordred | k | 21:38 |
clarkb | ideally we would have centos6, centos7, f21, precise, trusty | 21:38 |
fungi | so we can have a high-end to compare thoughts against | 21:38 |
mordred | I mean, we should ultimately only have one per distro release | 21:38 |
clarkb | but ya thinking high end is good | 21:39 |
mordred | fungi: I think a half terra is plenty | 21:39 |
fungi | that's a dozen diskimage types | 21:39 |
fungi | sounds like plenty to me | 21:39 |
mordred | yah | 21:39 |
clarkb | should give us plenty of room to add arch/gentoo/suse/ and a handful more if we need to | 21:39 |
mordred | debian stable, debian unstable, centos 6, centos 7, fedora 21, fedora 22, ubuntu precise, ubuntu trusty, gentoo | 21:40 |
fungi | opensystem | 21:40 |
pleia2 | well, once we have the ability to do more, I could see requests coming in | 21:40 |
anteaya | pleia2: isn't that the way of it | 21:40 |
pleia2 | anteaya: seems so! | 21:40 |
greghaynes | mordred: any idea what the contents of those profiledir's you were seeing are? | 21:41 |
fungi | i can see review requests coming in for new image types people have tested out already. i can also see ignoring random "please gimme platform x" requests | 21:41 |
*** erlon has quit IRC | 21:41 | |
mordred | greghaynes: start_01-ccache start_10-fedora-cloud-image stop_01-ccache | 21:41 |
clarkb | I just ran puppet agent --test on proposal.slave.openstack.org without error so unsure of the error in the last run | 21:43 |
clarkb | will have to keep an eye on it | 21:43 |
clarkb | mordred: what is problem with nodepool? | 21:44 |
mordred | clarkb: overlapping files that we don't need - packaging issue - fixing | 21:44 |
clarkb | Swami: armax_ were you done with those nodes? it wasn't clear if you were both done ( I think armax_ is ) | 21:45 |
*** tiswanso has quit IRC | 21:45 | |
openstackgerrit | Jeremy Stanley proposed openstack-infra/system-config: Correct internal doc anchor for cinder vol mgmt https://review.openstack.org/175584 | 21:45 |
fungi | that ^ was an amusing error | 21:45 |
*** tiswanso has joined #openstack-infra | 21:45 | |
Swami | clarkb: You can take back the nodes. If I need more info to debug I will request you for access later. | 21:45 |
clarkb | Swami: thanks | 21:46 |
*** mwagner_lap has quit IRC | 21:46 | |
fungi | the current text at the end of http://ci.openstack.org/static.html ended up pretty silly as a result | 21:46 |
*** dustins has quit IRC | 21:47 | |
armax_ | clarkb: go ahead | 21:47 |
pleia2 | fungi: hah, indeed | 21:47 |
clarkb | Swami: armax_ all done | 21:47 |
Swami | Swami: I am doing anything right now with the node. | 21:48 |
*** shashankhegde has quit IRC | 21:48 | |
anteaya | Swami: that is a confusing statement | 21:48 |
*** tsg has joined #openstack-infra | 21:49 | |
Swami | anteaya: armax_ already mentioned that he had done with his testing in the previous message. | 21:49 |
anteaya | right | 21:49 |
Swami | anteaya: I am not currently using it, so I mentioned that those nodes can be taken away. | 21:49 |
anteaya | Swami: right saw that too | 21:49 |
Swami | If required I will grant access again. | 21:49 |
anteaya | yup, great | 21:49 |
Swami | Am I confusing. | 21:50 |
anteaya | no worries, you clarified your position, thank you | 21:50 |
yolanda | mordred, i saw your change to remove ansible playbooks from puppetmaster. It makes sense, but i was puppetizing that downstream to use templates, and substitute some hardcoded vars such as puppetmaster, list of hosts, etc... what's the plan for that? are you in upstream thinking in having something to make this more generic? | 21:50 |
*** tiswanso has quit IRC | 21:50 | |
mordred | yolanda: yup - I have another change for you | 21:51 |
mordred | yolanda: https://review.openstack.org/#/c/163538/ | 21:51 |
mordred | yolanda: I would like to reduce the amount that we use puppet to construct ansible that we run to run puppet - because at some point we're going to lose track of our sanity :) | 21:52 |
mordred | yolanda: I'm not sure that's the _exact_ write form of that - but somethign along those lines | 21:53 |
*** mriedem is now known as mriedem_away | 21:53 | |
yolanda | mm , ok, this updates the puppetmaster. But what about hosts like review.openstack.org, or even the git repo? | 21:53 |
yolanda | if you are on a testing environment your host is not really review.openstack.org | 21:54 |
*** yamahata has joined #openstack-infra | 21:54 | |
mordred | yah - things like that should become params taht are overridable | 21:54 |
clarkb | yolanda: re 171526 I don't understand how that helps testing | 21:54 |
clarkb | yolanda: we should test puppet-lodgeit not openstack_project::paste | 21:54 |
clarkb | yolanda: sorry we should test ^ if we want to test those parameters | 21:54 |
clarkb | we only need to test openstack_project::paste in the context of running a paste with thoseparameters | 21:54 |
mordred | yolanda: I should sketch out more of what my brain was thinking in a spec perhaps | 21:55 |
mordred | clarkb: ok - I just pushed a new change to the ppa that should fix that | 21:55 |
*** yamamoto has joined #openstack-infra | 21:56 | |
yolanda | clarkb, from my opinion, everything that encapsulates logic , even as a hardcoded parameter inside the module, is bad. If you are on the case where you want to setup a whole system-config, for integration for example, that hidden logic makes thing more complicated | 21:56 |
yolanda | mordred, are you here tomorrow? | 21:56 |
yolanda | i'd like to chat with you guys a bit about it | 21:56 |
mordred | yolanda: unfortunately no - I got pulled into a last-minute customer meeting - but jeblair will be there | 21:56 |
reed | is this change now ready to merge now or is there something else needed https://review.openstack.org/#/c/104951/ ? | 21:56 |
yolanda | oh, bad thing | 21:56 |
clarkb | yolanda: its not hidden, its there and understood. Its just not configurable because we have decided that the best way to run lodgeit is with a local db using innodb for example | 21:56 |
yolanda | but i'm happy to have jeblair | 21:57 |
*** nelsnelson has quit IRC | 21:57 | |
clarkb | yolanda: there is no reason to ever make innodb configurable | 21:57 |
yolanda | clarkb, but the logic is there in the form of a default parameter | 21:57 |
clarkb | yolanda: but why have it at all? | 21:57 |
clarkb | do we want to suggest that myisam is ok? | 21:57 |
clarkb | it is ok to have constants where it makes sense | 21:58 |
yolanda | clarkb, i can agree that this innodb is a superflous one . But some others like host, vhost, user, pass | 21:58 |
clarkb | and innodb seems like something that should be constant | 21:58 |
yolanda | i would be ok with removing engine, port | 21:59 |
mordred | clarkb: https://github.com/emonty/vhd-util/commit/3b80a60c0c61cfc64310997bf7b2c31e3b7cc7d5 in case you're interested - basically, I took the simplest way making the package the first time and it turns out it includes some files that are in xe-guest-utilities which rax puts on the machiens because xen | 21:59 |
clarkb | yolanda: ok let me leave a proper review | 21:59 |
mordred | clarkb: but ultimately we don't need either those files in the vhd-util package _or_ xe-guest-utilities on the box | 21:59 |
yolanda | ok | 21:59 |
mordred | but I think we should wait before removing xen things from rax images until we're doing that as part of a larger effort | 21:59 |
*** apmelton is now known as apmelton_away | 22:00 | |
*** shashankhegde has joined #openstack-infra | 22:00 | |
yolanda | clarkb, also i saw some inconsistency there. Why do you allow to setup a password, but hardcode the user, and the host? | 22:00 |
greghaynes | mordred: the vhd-util on https://launchpad.net/~openstack-ci-core/+archive/ubuntu/vhd-util has a missing reference to libvhd.so.1.0 | 22:00 |
mordred | greghaynes: yup. just fixed that in -5 too | 22:00 |
greghaynes | ok | 22:00 |
openstackgerrit | Merged openstack-infra/infra-manual: Adds a link for new developers to use sandbox https://review.openstack.org/104951 | 22:01 |
clarkb | yolanda: because we are saying don't run this on trove it is broken | 22:01 |
fungi | reed: lgtm. we should probably reorganize the top-level index a bit since one of these things is not like the others (ideally that tutorial should be a sub-page/chapter of the developers guide rather than a separate top-level guide) | 22:01 |
fungi | reed: but that can happen later when someone has time to do it | 22:01 |
reed | fungi, i think that some documentation is better than no documentation at all | 22:01 |
clarkb | yolanda: also the password is secret info so can't be hardcoded | 22:01 |
fungi | reed: couldn't agree more! | 22:01 |
mordred | fungi: should we land the infra docs changes and retrigger docs builds? | 22:02 |
fungi | mordred: probably | 22:02 |
mordred | fungi: since we're both here :) | 22:02 |
reed | that patch was initially proposed in July last year, if it was a wiki page it would have existed and been adjusted with subsequent edits | 22:02 |
* fungi looks back over the list of changes for that | 22:02 | |
mordred | fungi: https://review.openstack.org/#/q/status:open+topic:move-infra-docs,n,z | 22:03 |
fungi | mordred: oh, right--i updated all their topics for consistency! ;) | 22:04 |
*** david-lyle has joined #openstack-infra | 22:04 | |
mordred | fungi: maybe we should land the two project-config changes - then retrigger docs builds - then when we're happy, land the redirects? | 22:04 |
*** melwitt has quit IRC | 22:04 | |
fungi | mordred: the redirects can land far ahead. the won't take effect until someone updates dns | 22:05 |
fungi | er, they won't take effect | 22:05 |
fungi | the current ci.o.o is on old-wiki.o.o. the redirects will be hosted on static.o.o | 22:05 |
yolanda | clarkb, in terms of automation and integration testing, the only thing that needs to change from my point of view is vhost_name, so you can refer to your real one. But puppet-lodgeit module is offering the possibility, exposing the parameters, why should we be hiding them on system-config? i'll wait for your review anyway, i'm flexible on that | 22:06 |
mordred | fungi: ah - neat | 22:07 |
anteaya | fungi reed the original link was a sub-page | 22:07 |
clarkb | yolanda: we should hide them because system-config specifically says this is how we run the paste server | 22:07 |
clarkb | yolanda: we make puppet-lodgeit generic and reusable then we layer over the top to do it our way | 22:07 |
*** Swami has quit IRC | 22:07 | |
clarkb | yolanda: some things should be configurable in that case but we aren't going to use a remote and local db at the same time. We aren't going to use myisam. | 22:07 |
fungi | mordred: so the redirect and index job changes can land early and in any order. once the publisher change lands, our docs will start going stale so that's when we need to retrigger docs jobs for branch tip of each of those repos | 22:07 |
yolanda | clarkb, are we ok then to parameterize some thing like vhost? that is for sure needed | 22:08 |
fungi | and once they check out, then we switch dns to send people to the new ci.o.o redirect vhost | 22:08 |
clarkb | yolanda: yes I think thats fine. should have comment posted shortly | 22:08 |
yolanda | cool, i'm good with that | 22:08 |
asselin__ | clarkb, fungi I can propose the openstackci->openstackinfra change now...or wait until later if it will be too much noise for now.... | 22:09 |
mordred | fungi: I agree. | 22:09 |
yolanda | so my intention when i refactor something is to expose all parameters as possible, with default constants to don't alter system-config. But i can understand that some of the things are not really needed | 22:09 |
clarkb | asselin__: whenever is fine with me, but I don't think its a massive priority until we start pushing stuff to the forge | 22:09 |
fungi | mordred: clarkb: i have an rsync of all the cache content going on nodepool.o.o right now. will swap the new volume in as soon as it completes | 22:09 |
*** dboik has quit IRC | 22:09 | |
clarkb | yolanda: my problem with it is in many of these cases only the default config will work | 22:09 |
clarkb | yolanda: and thats what we had before so the change isn't useful | 22:09 |
clarkb | yolanda: for example this change will not work without a local db | 22:09 |
asselin__ | clarkb, it's done...just need to git review it | 22:09 |
*** dboik has joined #openstack-infra | 22:11 | |
clarkb | asselin__: oh don't let me stop you from pushing it | 22:11 |
yolanda | clarkb, you are right with that, sure. I was thinking in the case of having some remote mysql server, and just add databases there instead of having local ones for each service. But that's not the case of system-config | 22:11 |
clarkb | should be quick reviews for that name update too | 22:11 |
clarkb | yolanda: yes except remote db won't work in that change either | 22:11 |
clarkb | yolanda: because puppet-mysql doesn't know how to manage remote dbs | 22:12 |
yolanda | no? ah, didn't know about it | 22:12 |
yolanda | sounded like some pretty basic stuff to me | 22:12 |
clarkb | yolanda: nope, if the local root user doesn't have access to the db via default user conf then puppet breaks | 22:12 |
clarkb | yolanda: its the same reason you can't sudo to use puppet in this case | 22:12 |
Rockyg | Hi, folks. If you've got the time, I could use a little brainstorming/direction finding for Refstack. | 22:13 |
yolanda | clarkb, thx, good info to me | 22:13 |
mordred | Rockyg: sup? | 22:13 |
Rockyg | Refstack would like to demo the server and client sw for Vancouver and needs a server. Funny that, you need a server for server software | 22:13 |
Rockyg | Refstack.org is now owned by the foundation. | 22:14 |
Rockyg | Yay! | 22:14 |
*** amitgandhinz has quit IRC | 22:14 | |
Rockyg | and, when it all comes down to it, the foundation will own the site. | 22:14 |
*** amitgandhinz has joined #openstack-infra | 22:14 | |
Rockyg | But, how do we get a demo server up and how do we get the right process in place for a smooth transition sometime in the future? | 22:15 |
fungi | Rockyg: i don't suppose anyone's written a puppet module to automate deployment details for that software? | 22:15 |
Rockyg | Ha ha ha ha ha. I think the only one who understands the complexities of a real, public facing server is hogepodge | 22:15 |
fungi | Rockyg: if they have, then standing up a server managed by the project infrastructure team is quick and straightforward | 22:15 |
Rockyg | Ah. Cool. Is there a list of requirements? | 22:16 |
mordred | Rockyg: dare I ask what the server is written in? | 22:16 |
clarkb | Rockyg: I think you are providing the requirements :) | 22:16 |
Rockyg | right now, I really think it's simply the python api, a db, and a swing frontent. And someone is working on authentication | 22:16 |
fungi | Rockyg: also be aware that it needs to be something well-documented enough that our root sysadmins can see how to easily help the refstack developers troubleshoot deployment and operation problems, since the developers won't have shells on that server | 22:16 |
openstackgerrit | Merged openstack-infra/system-config: Add redirects from ci.o.o to docs.o.o/infra https://review.openstack.org/174437 | 22:17 |
Rockyg | Ah, great! point one. No shell access, so smart hands (sorry for the insult, guys) management | 22:18 |
* fungi is insulted that you called him "smart" | 22:18 | |
reed | no insult taken :) | 22:18 |
reed | no hands at all | 22:18 |
fungi | look ma, no hands! | 22:18 |
mordred | it seems to be a pecan-based thing - so setting it up should be very similar to the server portion of storyboard | 22:18 |
Rockyg | ah. News to me. | 22:19 |
*** amitgandhinz has quit IRC | 22:19 | |
*** unicell has quit IRC | 22:19 | |
*** unicell has joined #openstack-infra | 22:20 | |
mordred | I'm assuming the stackforge/refstack repo is the right repo? | 22:20 |
openstackgerrit | yolanda.robla proposed openstack-infra/system-config: Allow configuring all lodgeit parameters on paste https://review.openstack.org/171526 | 22:20 |
Rockyg | So, I haven't checked recentlyu (missed today's meeting), but I *think* they are trying to get an authenticated login set up? | 22:20 |
Rockyg | and yse, stackforge/refstack | 22:20 |
*** Darkwan has quit IRC | 22:21 | |
mordred | k. so - it's not going to be a tiny amount of work - but it's not going to be a lot of learning either | 22:21 |
fungi | Rockyg: yeah, there's a thread going on the openstack-infra ml where they're discussing authentication/authorization design for leveraging openstackid.org | 22:21 |
*** otter768 has joined #openstack-infra | 22:21 | |
mordred | the storyboard modules can almost certainly be lifted wholesale for the most part | 22:21 |
mordred | oh. OH. my eye | 22:21 |
fungi | sounds like we could probably nudge hogepodge to work with us on adding a puppet-refstack module in that case | 22:21 |
Rockyg | You could likely get his help. Predicated on the knowledge that it *is* a foundation site. | 22:22 |
*** Krinkle|detached is now known as Krinkle | 22:22 | |
fungi | also clarkb can be sent 'round to rough him up if we need | 22:23 |
mordred | we might want to get the repo split into refstack and refstack-webclient | 22:23 |
mordred | because we've got teh "angularjs web frontend to pecan/sqlalchemy backend" pattern set up | 22:23 |
mordred | and I'd rather not solve it slightly diffrently a second time | 22:24 |
fungi | that could also make test-driving auto-generated drafts of changes to the frontend easier | 22:24 |
*** fawadkhaliq has quit IRC | 22:24 | |
Rockyg | Angular right, not swint. Toop freaking many meaningless names! | 22:24 |
mordred | Rockyg: if you ask real nice, you might be able to convince krotscheck to help with re-orging the repo into two so that it fits lessons learned | 22:24 |
krotscheck | Eh? Wha? | 22:24 |
* krotscheck reads backscroll | 22:24 | |
mordred | krotscheck: ^^ the refstack folks have a repo that's pecan + angular | 22:25 |
mordred | krotscheck: but it's not _quite_ the way you've been dong things, do re-using your existing infra work isn't quite there | 22:25 |
krotscheck | OH HAI | 22:25 |
*** tsg has quit IRC | 22:25 | |
greghaynes | mordred: you saw your vhd-util build failed? | 22:25 |
mordred | greghaynes: bah. I'm going to have to actually build locally aren't I? | 22:25 |
greghaynes | seems so | 22:25 |
*** otter768 has quit IRC | 22:25 | |
greghaynes | isnt packaging fun! | 22:25 |
krotscheck | Rockyg: Gimme 10 minutes, lemme go dig into code. I'm assuming stackforge/refstack? | 22:26 |
mordred | krotscheck: yah | 22:26 |
* clarkb should remember to point at this next time people want to "just set up a package mirror for packages we build" | 22:26 | |
*** bknudson has quit IRC | 22:26 | |
*** gordc has quit IRC | 22:26 | |
greghaynes | just say "you first"? | 22:26 |
mordred | clarkb: to be fair - if we had a package mirror for the packages we built, the check job on my patch would have failed | 22:27 |
clarkb | mordred: sure but that means ou would have to write those jobs and make the packages and all that | 22:27 |
mordred | clarkb: which si one of the reasons I want to make sure that if/when we make one - it's incorporated into our workflow | 22:27 |
Rockyg | Thanks krotscheck! Typing that name has lots of backspaces in it;-) | 22:27 |
clarkb | thing which everyone implies are ridiculously easy but don't step up to do | 22:27 |
krotscheck | Rockyg: Tab completion is your friend :D | 22:27 |
mordred | clarkb: yup | 22:27 |
mordred | clarkb: "just make a package" - easy "make a packaging system that allows people to land patches and get packages out the other end" ... not so much | 22:28 |
*** ddieterly has quit IRC | 22:28 | |
*** peristeri has quit IRC | 22:28 | |
krotscheck | Rockyg: Looks like you're committing all your dependencies to source? | 22:29 |
krotscheck | Rockyg: Does refstack have packaging requirements? i.e. if we run npm and bower during the build, will the package maintainers yell at us? | 22:29 |
Rockyg | krotscheck: no clue. I'm not doing anything with the code. If it's bad, we can change it. | 22:29 |
Rockyg | The team is like five people, none of which are packagers. anyone who want to package will be looked upon as a godsend. | 22:30 |
mordred | krotscheck: let's say "they should be happy about doing the right things" | 22:30 |
krotscheck | Rockyg: Not saying it's bad. It's a design decision that's usually made specificially to address the requirements imposed by package maintainers. I'm simply clarifying :) | 22:31 |
*** achanda has quit IRC | 22:31 | |
fungi | also might be worth seeing about making refstack an official development effort. if memory serves, the only reason it was originally proposed to stackforge instead is that mckenty said he'd pull all his resources off it if "openstack" took over the refstack project (he was fairly opposed to the code development even being hosted on our infra in stackforge, but eventually got talked into it anyway) | 22:31 |
Rockyg | Big problem on the project is that we have performance engineers, developers and no ops folks except hogpadge. and ops is the ugly stepchild (sorry hogepodge) | 22:31 |
fungi | at least i got called smart | 22:32 |
mordred | fungi: ++ | 22:32 |
Rockyg | The guys doing the angular are just learning, so following the "book" that they google | 22:33 |
fungi | well, okay, my _hands_ got called smart, but it's true they are my smartest feature | 22:33 |
Rockyg | Problem with "official" is that refstack currently doesn't have critical mass. | 22:33 |
*** yolanda has quit IRC | 22:33 | |
krotscheck | Rockyg: Right. Well, infra's support for javascript applications are all based around executing NPM. | 22:33 |
krotscheck | Rockyg: So it's library-agnostic. You could do reactjs if you wanted to, or ember. | 22:34 |
fungi | yeah, we can (and do) host services openstack finds useful even if they're not officially developed as a part of the openstack big tent | 22:34 |
Rockyg | It's got breadth, but not clear who would stick around for the long haul. | 22:34 |
mordred | yah - what krotscheck says | 22:34 |
Rockyg | krotscheck: kewl! | 22:34 |
clarkb | greghaynes: commented on 173585 | 22:34 |
Rockyg | krotscheck: you know, Refstack would *love* a web guy on the team ;-) | 22:34 |
mordred | I think we're well set up to run a bower/npm + pecan/wsme app with not a ton of effort - so best effort is likely spent making refstack look like storyboard | 22:34 |
krotscheck | mordred: Gating one on the other is still tricky | 22:35 |
greghaynes | clarkb: ah, nice | 22:35 |
mordred | rather than making refstack people learn enough infra to do infra things for a differently shaped model | 22:35 |
krotscheck | yolanda was working on that. | 22:35 |
mordred | nod | 22:35 |
krotscheck | Rockyg: Ok, I think I have the gist of what's going on here. | 22:36 |
krotscheck | Rockyg: And while it'd be fun to have me on the team, I'm also trying to push 12516236425 other javascript things inside the openstack ecosystem :) | 22:36 |
Rockyg | Excellent! Would it help if we came up with either a demo script or a couple of demo scenarios to help direct work effort? | 22:36 |
krotscheck | Rockyg: But I would LOVE to help you guys get ot a happy place. | 22:36 |
openstackgerrit | Morgan Fainberg proposed openstack-infra/project-config: Add keystoneauth library and testing infrastructure https://review.openstack.org/175596 | 22:36 |
Rockyg | krotscheck: Fantastic! | 22:37 |
krotscheck | Rockyg: Is there a #refstack channel that I can talk to the whole team with? | 22:37 |
*** jaypipes has joined #openstack-infra | 22:37 | |
Rockyg | Yes. #refstack It can often be crickets, though. I can get you some email addresses if you don't get a response | 22:37 |
*** baoli has quit IRC | 22:40 | |
clarkb | btw I am going to single approve things in project-config that look safe. We are falling behind there and there are simple things like stop running py3k tests on this project or add service to this functional test suite | 22:41 |
*** Swami has joined #openstack-infra | 22:41 | |
fungi | mordred: you have a shell running on nodepool.o.o with its cwd in /opt. could you cd out of there so i can remount the new volume on it? | 22:41 |
*** sabeen1 has joined #openstack-infra | 22:41 | |
mordred | fungi: yes | 22:42 |
mordred | fungi: done | 22:42 |
fungi | mordred: thanks! looks good now | 22:42 |
pleia2 | clarkb: lmk if you want other eyes on any of those changes | 22:43 |
clarkb | pleia2: will do, they are all pretty straightforward otherwise I am just +2ing | 22:43 |
* pleia2 nods | 22:43 | |
fungi | clarkb: mordred: okay, i've got the new 0.5tb /opt in place now, and the old opt temporarily mounted at /oldopt in case there's something wrong with the new one | 22:43 |
clarkb | pleia2: eg 174997 | 22:43 |
clarkb | fungi: if you rsync the old one in that will speed up image builds tomorrow | 22:44 |
fungi | also restarted snmpd in hopes of picking up that new device | 22:44 |
fungi | clarkb: i already did before i swapped them | 22:44 |
clarkb | fungi: and you can double check it works by `sudo -H -u nodepool nodepool image-build devstack-trusry` | 22:44 |
clarkb | fungi: awesome | 22:44 |
fungi | yep, going now | 22:44 |
clarkb | woot! | 22:44 |
fungi | except spelled correctly | 22:44 |
clarkb | even better | 22:45 |
fungi | though devstack-treasury sounds like somewhere i'd like to find | 22:45 |
mordred | fungi: ++ | 22:46 |
Krinkle | Hm.. someone got ideas for https://review.openstack.org/#/c/147101/ ? I keep running into git locks in Jenkins. Zuul cancels jobs when it makes up its mind about how to manage the pipeline, and leaves behind corrupted workspaces that it itself is unable to restore. | 22:46 |
anteaya | can we get one of those? | 22:46 |
mordred | krotscheck: do you know the magical git foo to split a repo into two repos and keep the history correct in both? | 22:46 |
*** wuhg has joined #openstack-infra | 22:47 | |
krotscheck | mordred: Depends on the approach. Can't we clone the original repo, and explicitly commit a delete of all things not-that-language? | 22:47 |
fungi | Krinkle: i think in the bad old days where we still had a lot of long-lived jenkins slaves, we had our git checkout script set up to clear the workspace and reclone the repo if updating it failed | 22:48 |
*** luqas has joined #openstack-infra | 22:48 | |
fungi | though it's been a while, so i might be misremembering | 22:48 |
krotscheck | mordred: Alternatively, there are rewrite-history things. | 22:48 |
mordred | krotscheck: you can. there is also a trickier one using git filter-branch | 22:49 |
krotscheck | mordred: Yes, that one. | 22:49 |
openstackgerrit | Merged openstack-infra/project-config: networking-ovn: enable mysql and rabbitmq https://review.openstack.org/173607 | 22:49 |
mordred | krotscheck: your first way is probably easier | 22:49 |
krotscheck | mordred: git filter-branch --tree-filter 'rm -f passwords.txt' HEAD | 22:49 |
Krinkle | fungi: Yeah, we don't have local git caches. it's not so much persistent workspaces as persistent slaves that's the issue. They go hand-in-hand. | 22:49 |
fungi | krotscheck: the benefit of filter-branch is that it omits any irrelevant history, keeping the resulting split-out repo history cleaner | 22:49 |
Krinkle | When creating fresh slaves from nodepool setting up a local git cache is relatively easy, but I'm not sure how to pull it off on a live slave. | 22:50 |
Krinkle | without race conditions. | 22:50 |
anteaya | krotscheck: this was the series of commands I ran when I split the project-config repo from what was then config: http://paste.openstack.org/show/204928/ | 22:50 |
fungi | Krinkle: how about a cache update job? | 22:50 |
clarkb | should oslo projects have stable branches now? | 22:51 |
fungi | Krinkle: as long as your executors is set to 1 per slave, you should be able to have them run a periodic cache update job which will prevent any other jobs from running on them while the git cache is being updated? | 22:51 |
clarkb | I seem to recall that libs were doing that | 22:51 |
clarkb | looking at 165224 | 22:51 |
Krinkle | fungi: Hm.. interesting. We'll need a way to inverse the concurrency limit to make the slave run only that one job. | 22:51 |
Krinkle | We have 4 executors per slave (quarcore openstack vms) | 22:51 |
fungi | Krinkle: ahh, so you currently run with concurrency > 1 | 22:51 |
fungi | Krinkle: perhaps each executor can have its own separate cache? | 22:52 |
fungi | i've not delved deeply into how jenkins handles that since we never ran concurrent jobs | 22:52 |
Krinkle | Interesting | 22:52 |
openstackgerrit | Merged openstack-infra/system-config: mailing list for Analyst Relations Working Group https://review.openstack.org/173991 | 22:52 |
openstackgerrit | Merged openstack-infra/project-config: Cerberus and Sticks: remove python3 tests https://review.openstack.org/174358 | 22:53 |
*** luqas has quit IRC | 22:53 | |
* clarkb leaves a note that we may want to update that later and approves 165224 as is | 22:53 | |
Krinkle | I guess we could consider raising our nova project quotas and using more basic nodes. | 22:53 |
openstackgerrit | Merged openstack-infra/project-config: Enable ceilometer service on Heat gates https://review.openstack.org/174394 | 22:53 |
Krinkle | fungi: Thanks :) | 22:53 |
Krinkle | and decreasing executors to 1 | 22:53 |
Krinkle | would make it more like how it is with nodepool anyway | 22:53 |
Rockyg | Hey people! Thanks for getting this kicked off for us! | 22:53 |
fungi | Krinkle: instead we run non-concurrent on each slave and make sure our jobs are n-way multi-threaded, so they run faster by taking advantage of the available processors | 22:53 |
openstackgerrit | Merged openstack-infra/project-config: Experimental bandit job for anchor https://review.openstack.org/174761 | 22:53 |
Krinkle | fungi: yeah | 22:53 |
fungi | though coding concurrency-safe tests takes some attention to races and cross-test interactions | 22:54 |
fungi | so you need properly discrete tests or barriers when that's not possible. fixtures are a great way to wrangle a lot of that | 22:55 |
lifeless | fungi: I want us to distribute over multiple machines | 22:55 |
lifeless | fungi: just never had the time to push on it | 22:55 |
fungi | lifeless: we are! | 22:56 |
fungi | lifeless: though not in the way you're wanting, yet anyway | 22:56 |
lifeless | fungi: since when? | 22:56 |
lifeless | hah | 22:56 |
lifeless | yes | 22:56 |
*** annegent_ has quit IRC | 22:57 | |
clarkb | ya I'm not sure it makes a big difference | 22:57 |
*** sabeen1 has quit IRC | 22:57 | |
clarkb | and the current setup better accomodates projects that aren't n way distributable | 22:57 |
fungi | lifeless: though the multi-node nodepool feature is working out well for devstack-based testing, so probably someone could start poking at seeing what sort of speedup we get from trying to distribute testr | 22:57 |
jhesketh | Morning | 22:57 |
clarkb | also keeps the load against git repos lower | 22:57 |
clarkb | jhesketh: good morning | 22:57 |
anteaya | morning jhesketh | 22:58 |
jhesketh | o/ | 22:58 |
lifeless | clarkb: load against git would be constant | 22:58 |
lifeless | clarkb: since its count(nodes) | 22:58 |
*** otter768 has joined #openstack-infra | 22:58 | |
lifeless | clarkb: I'm not aware of any of our projects being incompatible with testr distribution, but you never know | 22:58 |
mordred | morning jhesketh | 22:58 |
mtreinish | lifeless: I would expect it to work fine | 22:58 |
mtreinish | although it feels slightly weird to have both your controller and compute talking to the api server at the same time to run tests | 22:59 |
clarkb | lifeless: horizon and swift | 22:59 |
mtreinish | not that it's wrong | 23:00 |
clarkb | lifeless: the problem is zuul mergers host the specific refs for testing, and only one merger hosts the ref for that job | 23:00 |
lifeless | mtreinish: well,unless tempest is the bottleneck, I wouldn't be worried about distributing that part of the load yet; I'd increase the cluster size for the cloud under test to get more actors, then up tempest concurrency | 23:00 |
clarkb | lifeless: so today that means you get ~15 nodes that all grab the ref at about the same time. As soon as you ditribute across nodes like you intendits 600 | 23:00 |
lifeless | clarkb: well, *if* you use all 600 nodes to run one job | 23:00 |
lifeless | clarkb: I suspect the sweet spot is somewhere south of that :) | 23:00 |
mordred | NO. MUST USE ALL NODES FOR ALL CHANGES!!!! | 23:01 |
mordred | MOAR CARTESIAN PRODUCTS!!! | 23:01 |
lifeless | clarkb: how are horizon and swift incompatible? Other than horizon not being testr enabled yet | 23:01 |
mtreinish | lifeless: oh, I assumed you were talking about using testr distrubuted to run tempst on the existing multinode jobs, nm | 23:01 |
mordred | lifeless: neither are testr enabled | 23:01 |
clarkb | fungi: I just approved 167443 which may confict with your centos-6 change :/ sorry | 23:01 |
notmyname | what does "testr enabled" mean? I thought that was something done long ago | 23:02 |
clarkb | lifeless: you don't use all 600 for one job, you use 100 for python26, 100 for python3.4 100 for python2.7 and so on | 23:02 |
mordred | notmyname: did we? oh - neat! | 23:02 |
notmyname | mordred: I don't know :-) | 23:02 |
mordred | notmyname: nope. you use nose in swift | 23:02 |
notmyname | oh, ok | 23:02 |
clarkb | notmyname: https://git.openstack.org/cgit/openstack/swift/tree/tox.ini#n21 that uses nosetests | 23:03 |
fungi | clarkb: no worries, that's an easy rebase since it's a script-generated file | 23:03 |
notmyname | ah, it's python-swiftclient that uses testr | 23:03 |
mordred | yah. | 23:03 |
krotscheck | crinkle, nibalizer: I hear there's a puppet team now? | 23:04 |
*** aduarte has quit IRC | 23:04 | |
mordred | notmyname: you know - I think nose2 does the test discover protocol properly now - I could imagine a world where we made tox.ini in swift use testr but still had .unittets call nose or nose2 | 23:04 |
mordred | notmyname: it's low on my personal list | 23:05 |
notmyname | mordred: what's the difference, from the code-being-tested perspective? ie would it require changes to anything in swift/test/*? | 23:05 |
mordred | notmyname: but if we do reach somethign liek what lifeless and clarkb are talking about above where there was a more explicitly pressing need to get something going | 23:05 |
Rockyg | krotscheck: Yeah. I think yolanda is the ptl? | 23:05 |
mordred | notmyname: very minimal - the only nose code you use in your unittests are nose.SkipTest it seems | 23:06 |
mordred | notmyname: and those are easily replaceable by other thigns called skiptest | 23:06 |
*** MarkAtwood has quit IRC | 23:06 | |
mordred | notmyname: on first glance, it seems like way less work than last time I looked at it | 23:06 |
mordred | krotscheck, Rockyg: EmilienM is the ptl | 23:07 |
mordred | krotscheck, Rockyg: but that's puppet-openstack (except not named puppet-openstack) | 23:07 |
*** ddieterly has joined #openstack-infra | 23:07 | |
notmyname | mordred: I get that a lot when people look at swift. "once I looked at it, it's way less <whatever> than I thought" ;-) | 23:07 |
mordred | notmyname: :) | 23:07 |
Rockyg | thanks mordred. Wasn't sure. But, with all the modules, if Refstack is gonna make it to official status, there will be a module for that, too;) | 23:08 |
clarkb | jogo: why are 167779 and 167780 separate changes? is there some ordering that we need to do there? | 23:08 |
mordred | Rockyg: well.... | 23:08 |
pleia2 | clarkb: it's the right time of day (no meetings) if you want to approve 175365 | 23:09 |
Rockyg | mordred: yeah..... | 23:09 |
mordred | Rockyg: it kinda depends ... do you expect people operating openstack clouds to deploy a refstack? | 23:09 |
krotscheck | mordred: Wait, so.... is that "puppet things to install openstack", or "puppet things in openstack" | 23:09 |
mordred | Rockyg: or do you expect there to be one refstack instance that people report data to | 23:09 |
Rockyg | mordred: Darn! no :( | 23:09 |
mordred | krotscheck: that's "puppet things to install/operate openstack" - doesn't relate to any infra puppet activities | 23:10 |
*** achanda has joined #openstack-infra | 23:10 | |
krotscheck | mordred: Gotcha. | 23:10 |
Rockyg | One Refstack to collect them; one refstack to grade them; one refstack to display them | 23:10 |
mordred | :) | 23:10 |
krotscheck | mordred: So this'd be more of an openstack-infra/puppet-refstack thing. | 23:10 |
mordred | yup | 23:10 |
mordred | one more for the menagerie | 23:10 |
krotscheck | wheeeee | 23:10 |
fungi | we could get zoo keeper costumes and start charging admission | 23:11 |
mordred | fungi: I dont' want to run zookeeper | 23:11 |
fungi | oh, great point | 23:11 |
mordred | http://mcfunley.com/choose-boring-technology | 23:11 |
krotscheck | Wat | 23:11 |
Rockyg | ++ with signs: feed the infra-critters beer | 23:11 |
clarkb | zaro: see comment on 167816 | 23:11 |
clarkb | pleia2: done | 23:12 |
clarkb | HenryG: have a moment to talk about 167847? | 23:13 |
lifeless | mtreinish: you could totally do that, but you fix bottlenecks first | 23:13 |
clarkb | HenryG: have we tried just running devstack with that turned on? | 23:13 |
openstackgerrit | Boris Pavlovic proposed openstack-infra/project-config: Add new rally-ci project https://review.openstack.org/166962 | 23:14 |
*** dimtruck is now known as zz_dimtruck | 23:14 | |
*** amitgandhinz has joined #openstack-infra | 23:15 | |
openstackgerrit | Clark Boylan proposed openstack-infra/devstack-gate: DO NOT MERGE Try turning on ipv6 https://review.openstack.org/175607 | 23:15 |
clarkb | HenryG: ^ /me finds out | 23:16 |
*** mdenny has quit IRC | 23:16 | |
*** sabeen1 has joined #openstack-infra | 23:16 | |
*** pc_m has quit IRC | 23:17 | |
*** yamamoto has quit IRC | 23:18 | |
*** thinrichs has joined #openstack-infra | 23:18 | |
*** achanda_ has joined #openstack-infra | 23:18 | |
*** achanda_ has quit IRC | 23:18 | |
*** achanda_ has joined #openstack-infra | 23:19 | |
*** achanda has quit IRC | 23:19 | |
*** amitgandhinz has quit IRC | 23:19 | |
clarkb | fungi: 168036 is one that you have previously reviewed that should be ready | 23:19 |
clarkb | pleia2: ^ | 23:19 |
*** amitgandhinz has joined #openstack-infra | 23:20 | |
*** adrian_otto has quit IRC | 23:21 | |
pleia2 | yay simplified | 23:22 |
*** rlandy has quit IRC | 23:25 | |
jogo | clarkb: no ordering, they can be merged into one patch if you like | 23:25 |
jogo | clarkb: was basing it roughly on similar work that jeblair did | 23:26 |
clarkb | jogo: two is fine, just wanted ot make sure I wasn't missing something | 23:26 |
clarkb | jogo: did you see they need a rebase? | 23:26 |
jogo | yeah, rebasing now | 23:27 |
* tchaypo reads backscroll | 23:27 | |
*** esker has quit IRC | 23:27 | |
openstackgerrit | Merged openstack-infra/project-config: Blacklist last failing Tempest tests for cells job https://review.openstack.org/175015 | 23:28 |
clarkb | mtreinish: 171351 disbales tenant isolation but the commit says it makes the job run serially. Is that a side effect that makes tempest run serially? | 23:28 |
openstackgerrit | Joe Gordon proposed openstack-infra/project-config: Add tempest-dsvm-sahara https://review.openstack.org/167779 | 23:29 |
openstackgerrit | Joe Gordon proposed openstack-infra/project-config: Add sahara tempest job to sahara and tempest https://review.openstack.org/167780 | 23:29 |
*** tnovacik has quit IRC | 23:30 | |
openstackgerrit | Merged openstack-infra/project-config: Change kolla notifications to #kolla https://review.openstack.org/175173 | 23:30 |
*** thedodd has joined #openstack-infra | 23:30 | |
mtreinish | clarkb: yeah, it is. If the tests weren't tenant isolated we would potentially pollute others running at the same time | 23:30 |
openstackgerrit | Merged openstack-infra/project-config: Change python-magnumclient's bandit gate https://review.openstack.org/175175 | 23:30 |
mtreinish | like trying to do a list on something if 3 other tests are messing with that resource | 23:31 |
*** rm_work is now known as rm_work|away | 23:31 | |
mtreinish | clarkb: in that particular case that d-g flag does 2 things passes the flag to turn of tenant isolation in the tempest.conf into the localrc and tells tempest to run serially | 23:32 |
clarkb | gotcha | 23:32 |
fungi | clarkb: DIB image 118861 with file devstack-trusty is built | 23:33 |
clarkb | fungi: woot thanks for checking | 23:33 |
*** dboik has quit IRC | 23:33 | |
*** sabeen3 has joined #openstack-infra | 23:33 | |
clarkb | mestery: in 172982 isn't that typically the openstack release manager's job? | 23:34 |
*** chlong has joined #openstack-infra | 23:34 | |
fungi | yeah, i think we discussed that change in here when he was drafting it, and i suggested he connect with ttx on the correct/intended workflow | 23:36 |
clarkb | pleia2: 175433 is a lsightly more interesting change that updates jobs for projects | 23:36 |
*** baoli has joined #openstack-infra | 23:37 | |
*** sabeen1 has quit IRC | 23:37 | |
*** shashankhegde has quit IRC | 23:37 | |
tchaypo | okay, scrollback reading done | 23:40 |
tchaypo | 05:17:07 <dstufft> I don't think that install_requires and -r requirements.txt acts differently as to how pip resolves them, other than pip only respects 1 version specifier for any particular name and a top level name in requirements.txt will be rpeferred over one inside of a install_requires | 23:40 |
*** otter768 has quit IRC | 23:40 | |
*** Rockyg has quit IRC | 23:41 | |
pleia2 | clarkb: ah yeah, I looked at that one quickly earlier and wasn't sure, I'll have a closer look | 23:41 |
tchaypo | I think that “other than pip only respects 1 version specifier for any particular name” sounds like a bug, or at least an incomplete feature | 23:41 |
*** marun has quit IRC | 23:42 | |
*** ZZelle_ has quit IRC | 23:42 | |
*** crc32 has quit IRC | 23:43 | |
*** david-ly_ has joined #openstack-infra | 23:44 | |
*** jamielennox|away is now known as jamielennox | 23:45 | |
*** tsg has joined #openstack-infra | 23:45 | |
*** david-lyle has quit IRC | 23:45 | |
*** sigmavirus24 is now known as sigmavirus24_awa | 23:46 | |
*** david-ly_ is now known as david-lyle | 23:46 | |
*** mtanino has quit IRC | 23:46 | |
*** amitgandhinz has quit IRC | 23:47 | |
*** amitgandhinz has joined #openstack-infra | 23:48 | |
openstackgerrit | greghaynes proposed openstack-infra/project-config: Add diskimage-builder functests https://review.openstack.org/173585 | 23:48 |
*** ashleighfarnham has quit IRC | 23:49 | |
openstackgerrit | Thanh Ha proposed openstack-infra/jenkins-job-builder: Add support to CFP in builders sections https://review.openstack.org/164504 | 23:49 |
*** ajmiller has quit IRC | 23:51 | |
greghaynes | clarkb: ^ I think addressed your comment | 23:52 |
*** amitgandhinz has quit IRC | 23:53 | |
mordred | tchaypo: so - are you set to write a bug for lifeless? | 23:54 |
tchaypo | First I’ll write reproduction instructions | 23:55 |
lifeless | tchaypo: please do. In a bug. | 23:55 |
*** yamamoto has joined #openstack-infra | 23:55 | |
mordred | lifeless: oh - btw - you asked about other patches | 23:55 |
mordred | lifeless: oh - nevermind | 23:55 |
mordred | lifeless: ignore me | 23:55 |
lifeless | mordred: I asked you to prune your cruft and fixup any that weren't :) | 23:56 |
lifeless | mordred: separately I asked about any other discussions we need to have | 23:56 |
*** thinrichs has quit IRC | 23:56 | |
lifeless | mordred: and you said none came to mind | 23:56 |
lifeless | mordred: lastly we need to drill into the known-good vs known-bad stuff, but thats contingent on a bunch of other stuff, so doing it now is premature IMO, we've sketched enough already | 23:57 |
*** annegentle has joined #openstack-infra | 23:57 | |
tchaypo | lifeless: how about I write it up first, and we dissect what I’ve done wrong, and then figure out which bugs need to be filed? | 23:58 |
*** openstackgerrit has quit IRC | 23:58 | |
*** openstackgerrit has joined #openstack-infra | 23:58 | |
openstackgerrit | Ian Wienand proposed openstack/requirements: Keep pyOpenSSL < 0.14 https://review.openstack.org/169596 | 23:58 |
tchaypo | I feel like I want to file one for pip (or maybe add to the existing bug about needing a dependency resolver or something), and another for pbr (if it turns out it does do something different from pip) | 23:58 |
lifeless | tchaypo: I don't have the brainspace right now. | 23:58 |
lifeless | tchaypo: I want a bug open on pbr with reproduction instructions, expected behaviour. | 23:59 |
lifeless | tchaypo: so that we don't lose the info thats been dug up | 23:59 |
lifeless | tchaypo: then I want to focus on pip + pbr 0.11 release, not to mention TC etc etc. | 23:59 |
lifeless | tchaypo: then setup-requires | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!