*** Hal1 has quit IRC | 00:04 | |
*** rlandy_ has joined #openstack-infra | 00:05 | |
*** rlandy_ has quit IRC | 00:05 | |
*** rlandy has quit IRC | 00:07 | |
*** baoli has joined #openstack-infra | 00:10 | |
anteaya | the moon venus conjunction is a sight to behold in the sky | 00:11 |
---|---|---|
*** ZZelle_ has quit IRC | 00:13 | |
*** ddieterl_ has joined #openstack-infra | 00:14 | |
*** ddieterly has quit IRC | 00:14 | |
*** baoli has quit IRC | 00:15 | |
*** markvoelker has joined #openstack-infra | 00:17 | |
*** julim has joined #openstack-infra | 00:17 | |
*** yamamoto has joined #openstack-infra | 00:19 | |
*** markvoelker has quit IRC | 00:22 | |
*** achanda has quit IRC | 00:22 | |
*** yamamoto has quit IRC | 00:24 | |
*** dkranz has quit IRC | 00:25 | |
*** liusheng has quit IRC | 00:26 | |
*** liusheng has joined #openstack-infra | 00:26 | |
*** yamamoto has joined #openstack-infra | 00:26 | |
*** baoli has joined #openstack-infra | 00:32 | |
openstackgerrit | Joshua Hesketh proposed openstack/requirements: Remove pip_missing_reqs https://review.openstack.org/166664 | 00:33 |
*** emagana has quit IRC | 00:34 | |
openstackgerrit | Lars Kellogg-Stedman proposed openstack-infra/project-config: Add "heat-coe-templates" project https://review.openstack.org/164806 | 00:37 |
*** ajmiller__ is now known as ajmiller | 00:40 | |
*** garyh has joined #openstack-infra | 00:45 | |
*** asettle has quit IRC | 00:51 | |
*** garyh has quit IRC | 00:51 | |
*** julim has quit IRC | 00:53 | |
*** achanda has joined #openstack-infra | 00:53 | |
ianw | mordred: 166640 ... one typo, then fails in cloud-init ... but i'm not sure what the intentiosn are there because cloud-init isn't installed | 00:54 |
*** amitgandhinz has joined #openstack-infra | 00:56 | |
*** sputnik13 has quit IRC | 01:00 | |
*** liusheng has quit IRC | 01:00 | |
*** sdake has joined #openstack-infra | 01:00 | |
*** dims has joined #openstack-infra | 01:01 | |
*** dims has quit IRC | 01:01 | |
*** sdake__ has quit IRC | 01:04 | |
*** yamamoto has quit IRC | 01:04 | |
*** ghostpl_ has joined #openstack-infra | 01:04 | |
*** Longgeek has joined #openstack-infra | 01:08 | |
*** ghostpl_ has quit IRC | 01:12 | |
*** dims__ has joined #openstack-infra | 01:15 | |
*** fbo has quit IRC | 01:16 | |
*** fbo has joined #openstack-infra | 01:17 | |
*** tiswanso has joined #openstack-infra | 01:18 | |
*** markvoelker has joined #openstack-infra | 01:18 | |
*** esker has joined #openstack-infra | 01:23 | |
*** markvoelker has quit IRC | 01:23 | |
*** ianw has quit IRC | 01:25 | |
*** ianw has joined #openstack-infra | 01:26 | |
*** yamamoto has joined #openstack-infra | 01:28 | |
*** stevemar has quit IRC | 01:34 | |
*** sputnik13 has joined #openstack-infra | 01:40 | |
*** oomichi_ has joined #openstack-infra | 01:44 | |
*** Slower has quit IRC | 01:50 | |
*** sputnik13 has quit IRC | 01:51 | |
*** garyh has joined #openstack-infra | 01:51 | |
*** Slower has joined #openstack-infra | 01:52 | |
*** mestery_ is now known as mestery | 01:53 | |
*** asettle has joined #openstack-infra | 01:53 | |
*** unicell has quit IRC | 01:53 | |
*** Alexandra_ has joined #openstack-infra | 01:56 | |
*** sputnik13 has joined #openstack-infra | 01:57 | |
*** achanda has quit IRC | 01:57 | |
*** asettle has quit IRC | 01:58 | |
*** baoli has quit IRC | 01:59 | |
*** hdd has joined #openstack-infra | 01:59 | |
*** armax has joined #openstack-infra | 02:00 | |
*** garyh has quit IRC | 02:02 | |
*** sabeen2 has joined #openstack-infra | 02:03 | |
*** sabeen has quit IRC | 02:03 | |
*** baoli has joined #openstack-infra | 02:04 | |
*** armax has quit IRC | 02:04 | |
*** armax has joined #openstack-infra | 02:06 | |
*** armax has quit IRC | 02:07 | |
*** jyuso1 has joined #openstack-infra | 02:08 | |
*** mtanino has joined #openstack-infra | 02:08 | |
*** Alexandra_ is now known as asettle | 02:13 | |
*** mfink_ has joined #openstack-infra | 02:17 | |
*** sdake__ has joined #openstack-infra | 02:18 | |
*** markvoelker has joined #openstack-infra | 02:19 | |
*** tkelsey has joined #openstack-infra | 02:19 | |
*** mfink_ has quit IRC | 02:22 | |
*** sdake has quit IRC | 02:22 | |
*** markvoelker has quit IRC | 02:23 | |
*** tkelsey has quit IRC | 02:24 | |
*** kaisers has joined #openstack-infra | 02:27 | |
*** kaisers1 has quit IRC | 02:28 | |
greghaynes | mordred: Do you have a link to your vhd-util ppa? | 02:42 |
*** ddieterl_ has quit IRC | 02:47 | |
*** dboik has joined #openstack-infra | 02:53 | |
*** ghostpl_ has joined #openstack-infra | 02:56 | |
*** achanda has joined #openstack-infra | 02:58 | |
*** bdpayne has quit IRC | 02:59 | |
*** dims__ has quit IRC | 03:01 | |
*** ghostpl_ has quit IRC | 03:01 | |
*** garyh has joined #openstack-infra | 03:03 | |
*** achanda has quit IRC | 03:03 | |
*** sdake has joined #openstack-infra | 03:09 | |
*** sdake__ has quit IRC | 03:12 | |
*** garyh has quit IRC | 03:12 | |
*** trey has quit IRC | 03:13 | |
*** woodster_ has joined #openstack-infra | 03:14 | |
*** trey has joined #openstack-infra | 03:14 | |
*** amitgandhinz has quit IRC | 03:18 | |
openstackgerrit | Merged openstack-infra/system-config: Allow HTTPS through iptables on zuul.openstack.org https://review.openstack.org/165584 | 03:20 |
*** coolsvap|afk is now known as coolsvap | 03:24 | |
*** hdd has quit IRC | 03:24 | |
*** Longgeek has quit IRC | 03:30 | |
openstackgerrit | Merged openstack-infra/project-config: Add pypi jobs for stackforge/networking-[odl,ovn] https://review.openstack.org/166522 | 03:31 |
*** ddieterly has joined #openstack-infra | 03:48 | |
openstackgerrit | Merged openstack-infra/project-config: Reduce timeout for heat functional job https://review.openstack.org/166320 | 03:50 |
*** ujuc has joined #openstack-infra | 03:52 | |
*** ddieterly has quit IRC | 03:52 | |
openstackgerrit | Merged openstack-infra/puppet-user: Ensure home and home/.ssh folders are created https://review.openstack.org/165029 | 03:53 |
*** dhp has joined #openstack-infra | 04:00 | |
*** dims__ has joined #openstack-infra | 04:06 | |
*** baoli has quit IRC | 04:07 | |
*** achanda has joined #openstack-infra | 04:08 | |
*** baoli has joined #openstack-infra | 04:10 | |
*** stevemar has joined #openstack-infra | 04:15 | |
*** baoli has quit IRC | 04:15 | |
*** Hal1 has joined #openstack-infra | 04:17 | |
*** Hal1 has quit IRC | 04:18 | |
*** dboik has quit IRC | 04:21 | |
*** dboik has joined #openstack-infra | 04:21 | |
*** sabeen2 has quit IRC | 04:22 | |
*** esker has quit IRC | 04:23 | |
*** Longgeek has joined #openstack-infra | 04:30 | |
openstackgerrit | Merged openstack-infra/nodepool: Don't die while doing alien list https://review.openstack.org/165792 | 04:30 |
openstackgerrit | Merged openstack-infra/nodepool: Don't die while doing alien list https://review.openstack.org/165792 | 04:30 |
openstackgerrit | Merged openstack-infra/nodepool: Dont die on alien-image-list failure https://review.openstack.org/166132 | 04:31 |
*** achanda has quit IRC | 04:34 | |
*** dims__ has quit IRC | 04:36 | |
*** sdake__ has joined #openstack-infra | 04:40 | |
*** sdake has quit IRC | 04:44 | |
*** ghostpl_ has joined #openstack-infra | 04:45 | |
*** ddieterly has joined #openstack-infra | 04:49 | |
*** sdake has joined #openstack-infra | 04:51 | |
*** ghostpl_ has quit IRC | 04:52 | |
*** ddieterly has quit IRC | 04:54 | |
*** yamahata has joined #openstack-infra | 04:54 | |
openstackgerrit | Merged openstack-infra/nodepool: Ignore stderr for documentation program output https://review.openstack.org/166057 | 04:55 |
*** sdake__ has quit IRC | 04:55 | |
openstackgerrit | Merged openstack-infra/os-loganalyze: unwind test class multiple inheritance https://review.openstack.org/165851 | 04:55 |
*** asettle has quit IRC | 04:57 | |
*** ajmiller has quit IRC | 05:03 | |
*** kiseok7_ has joined #openstack-infra | 05:06 | |
*** pcaruana has quit IRC | 05:07 | |
*** achanda has joined #openstack-infra | 05:08 | |
*** Hal1 has joined #openstack-infra | 05:17 | |
*** sdake__ has joined #openstack-infra | 05:18 | |
*** Hal1 has quit IRC | 05:21 | |
*** sdake has quit IRC | 05:22 | |
*** achanda has quit IRC | 05:24 | |
*** achanda has joined #openstack-infra | 05:26 | |
*** bdpayne has joined #openstack-infra | 05:26 | |
*** garyh has joined #openstack-infra | 05:28 | |
*** sdake__ has quit IRC | 05:33 | |
*** sdake has joined #openstack-infra | 05:34 | |
*** garyh has quit IRC | 05:34 | |
*** nilasae has joined #openstack-infra | 05:38 | |
*** yamahata has quit IRC | 05:41 | |
*** sushilkm has joined #openstack-infra | 05:47 | |
*** sushilkm has left #openstack-infra | 05:47 | |
*** trinaths has joined #openstack-infra | 05:48 | |
trinaths | Hi, For my CI, I get this error, http://paste.openstack.org/show/195182/ . With respect to the recent changes Is there any Upgrade I need to do. Kindly help me to resolve this issue | 05:49 |
*** BharatK has joined #openstack-infra | 05:49 | |
*** ddieterly has joined #openstack-infra | 05:50 | |
*** sdake has quit IRC | 05:51 | |
*** ddieterly has quit IRC | 05:55 | |
ianw | jhesketh: while i take your point on validating the .erb, is there room to convince you that maybe that problem could be solved if and when it becomes a problem? unless you have suggestions on how to get it done in a practical way quickly | 06:02 |
openstackgerrit | Merged openstack-infra/system-config: Add a cronjob frequency to pypi_mirror https://review.openstack.org/163014 | 06:03 |
*** tnovacik has joined #openstack-infra | 06:03 | |
*** nilasae has quit IRC | 06:06 | |
*** Hal1 has joined #openstack-infra | 06:09 | |
*** yfried|afk has quit IRC | 06:12 | |
*** yfried|afk has joined #openstack-infra | 06:12 | |
*** trinaths has quit IRC | 06:13 | |
jhesketh | ianw: so a quick google does not show an easy way to do it, so I think I could be convinced | 06:18 |
*** trinaths has joined #openstack-infra | 06:18 | |
jhesketh | a hack could be to render the template with the erb tooling but we'd have to pass in the expected values explicitly | 06:18 |
jhesketh | I think your suggestion is fine until we have a need for better testing | 06:18 |
ianw | jhesketh: yeah, that's what i mean, to deploy it fully, i'm not sure if puppet with noop would do it | 06:19 |
jhesketh | ianw: yeah and from what I can see there is no easy way to pull it out of a noop | 06:19 |
ianw | jhesketh: and i think the likelihood of templating out anything other than the paths is low | 06:20 |
jhesketh | yep | 06:21 |
*** yfried|afk has quit IRC | 06:21 | |
*** otter768 has joined #openstack-infra | 06:21 | |
*** mtanino has quit IRC | 06:22 | |
*** dims__ has joined #openstack-infra | 06:22 | |
trinaths | Hi, For my CI, I get this error, http://paste.openstack.org/show/195182/ . With respect to the recent changes Is there any Upgrade I need to do. Kindly help me to resolve this issue | 06:22 |
*** otter768 has quit IRC | 06:25 | |
jhesketh | trinaths: it could be because of the gerrit host move... try restarting zuul | 06:26 |
jhesketh | and checking the IP for gerrit is 104.130.159.134 | 06:27 |
*** yfried|afk has joined #openstack-infra | 06:27 | |
openstackgerrit | Merged openstack-infra/system-config: Add nodepool config-validation environment https://review.openstack.org/164901 | 06:28 |
*** jamielennox is now known as jamielennox|away | 06:28 | |
trinaths | jhesketh: okay. Let the check and test the same as suggested. Thanks sir | 06:29 |
trinaths | jhesketh: I restarted Zuul, It still gives the same error | 06:31 |
jhesketh | trinaths: check that you can ssh to gerrit as the same user you've configured zuul with | 06:31 |
*** dannywilson has joined #openstack-infra | 06:31 | |
*** bdpayne has quit IRC | 06:32 | |
*** yfried|afk has quit IRC | 06:32 | |
trinaths | jhesketh: sure sir | 06:33 |
trinaths | jhesketh: thanks sir, Its an IP address related Issue. Its resolved now. Thanks a lot. | 06:34 |
jhesketh | no trouble :-) | 06:35 |
*** garyh has joined #openstack-infra | 06:35 | |
*** ghostpl_ has joined #openstack-infra | 06:36 | |
openstackgerrit | Merged openstack-infra/devstack-gate: force_config_drive always=>True https://review.openstack.org/162339 | 06:39 |
*** yolanda has quit IRC | 06:40 | |
*** deepakcs has joined #openstack-infra | 06:40 | |
*** dannywilson has quit IRC | 06:40 | |
*** HeOS has quit IRC | 06:40 | |
*** ghostpl_ has quit IRC | 06:43 | |
*** ujuc has quit IRC | 06:43 | |
*** garyh has quit IRC | 06:44 | |
*** stevemar has quit IRC | 06:48 | |
*** ddieterly has joined #openstack-infra | 06:51 | |
*** yfried|afk has joined #openstack-infra | 06:53 | |
*** dims__ has quit IRC | 06:54 | |
*** ddieterly has quit IRC | 06:56 | |
*** mrunge has joined #openstack-infra | 06:58 | |
*** pcaruana has joined #openstack-infra | 06:58 | |
*** Bsony has joined #openstack-infra | 06:59 | |
*** garyk has joined #openstack-infra | 07:03 | |
*** yfried has joined #openstack-infra | 07:11 | |
*** yfried|afk has quit IRC | 07:12 | |
*** yfried has quit IRC | 07:15 | |
openstackgerrit | Merged openstack-infra/project-config: Add Python 2.6 jobs to Castellan https://review.openstack.org/165913 | 07:16 |
*** yfried has joined #openstack-infra | 07:18 | |
*** yfried is now known as yfried|afk | 07:18 | |
*** Somay has joined #openstack-infra | 07:22 | |
*** scheuran has joined #openstack-infra | 07:24 | |
*** trinaths has left #openstack-infra | 07:26 | |
*** achanda has quit IRC | 07:29 | |
*** gsagie has joined #openstack-infra | 07:30 | |
gsagie | Hello all, i commit to a stackforge project and i wanted to add images to the README file, (https://github.com/stackforge/dragonflow) how ever they are not displayed correctly as you can see, i tried changing the README.rst to README.md but that fails in Jenkins, any ideas what am i missing? | 07:32 |
*** yamahata has joined #openstack-infra | 07:36 | |
*** yfried has joined #openstack-infra | 07:39 | |
*** yfried|afk has quit IRC | 07:43 | |
openstackgerrit | Joshua Hesketh proposed openstack-infra/system-config: Add tempest help footers to indexes https://review.openstack.org/166719 | 07:44 |
*** saggi has joined #openstack-infra | 07:44 | |
*** nilasae has joined #openstack-infra | 07:46 | |
saggi | Hi, we have a project on stackforge ( https://launchpad.net/dragonflow ) and we want to start filling up wiki pages. Is there some kind of process to start doing that on openstack.org/wiki/ or do we just start creating pages? | 07:46 |
*** mrmartin has joined #openstack-infra | 07:49 | |
*** woodster_ has quit IRC | 07:50 | |
*** dtantsur|afk is now known as dtantsur | 07:50 | |
*** yamahata has quit IRC | 07:53 | |
ttx | saggi: no process, just create a dragonflow page there | 07:55 |
*** yfried is now known as yfried|afk | 07:57 | |
*** chlong has quit IRC | 07:58 | |
*** YorikSar has quit IRC | 07:58 | |
saggi | ttx: thanks | 07:59 |
*** ibiris_away is now known as ibiris | 08:00 | |
*** Somay has quit IRC | 08:02 | |
*** yfried|afk is now known as yfried | 08:03 | |
*** oomichi_ has quit IRC | 08:04 | |
*** Somay has joined #openstack-infra | 08:05 | |
*** mrmartin has quit IRC | 08:05 | |
*** ghostpl_ has joined #openstack-infra | 08:06 | |
*** ghostpl_ has quit IRC | 08:07 | |
*** ghostpl_ has joined #openstack-infra | 08:07 | |
*** mrda is now known as mrda-away | 08:07 | |
*** sergsh has joined #openstack-infra | 08:07 | |
*** ghostpl_ has quit IRC | 08:07 | |
*** gsagie has left #openstack-infra | 08:08 | |
openstackgerrit | Jan Provaznik proposed openstack-infra/project-config: Create os-cloud-management project under TripleO https://review.openstack.org/165433 | 08:12 |
*** sergsh has quit IRC | 08:13 | |
*** markus_z has joined #openstack-infra | 08:14 | |
*** sergsh has joined #openstack-infra | 08:16 | |
*** soren has joined #openstack-infra | 08:18 | |
*** soren has joined #openstack-infra | 08:18 | |
*** yfried is now known as yfried|afk | 08:21 | |
*** otter768 has joined #openstack-infra | 08:22 | |
*** mpavone has joined #openstack-infra | 08:22 | |
*** yfried|afk is now known as yfried | 08:22 | |
*** doude has joined #openstack-infra | 08:23 | |
*** markvoelker has joined #openstack-infra | 08:24 | |
*** ihrachyshka has joined #openstack-infra | 08:25 | |
*** otter768 has quit IRC | 08:26 | |
*** markvoelker has quit IRC | 08:28 | |
*** yfried_ has joined #openstack-infra | 08:28 | |
*** ildikov has joined #openstack-infra | 08:30 | |
*** sergsh has quit IRC | 08:30 | |
*** sergsh has joined #openstack-infra | 08:31 | |
*** tkelsey has joined #openstack-infra | 08:31 | |
*** yfried has quit IRC | 08:32 | |
*** soren has quit IRC | 08:32 | |
*** ominakov has joined #openstack-infra | 08:32 | |
*** YorikSar has joined #openstack-infra | 08:34 | |
*** yfried_ has quit IRC | 08:37 | |
*** yfried_ has joined #openstack-infra | 08:37 | |
*** dims__ has joined #openstack-infra | 08:39 | |
*** andreykurilin_ has joined #openstack-infra | 08:40 | |
*** Somay has quit IRC | 08:41 | |
*** ghostpl_ has joined #openstack-infra | 08:42 | |
*** yfried_ has quit IRC | 08:44 | |
*** yfried__ has joined #openstack-infra | 08:44 | |
*** dtantsur is now known as dtantsur|brb | 08:48 | |
*** soren has joined #openstack-infra | 08:48 | |
*** soren has joined #openstack-infra | 08:48 | |
*** ihrachyshka has quit IRC | 08:48 | |
*** ddieterly has joined #openstack-infra | 08:52 | |
*** hashar has joined #openstack-infra | 08:56 | |
*** ddieterly has quit IRC | 08:57 | |
*** garyh has joined #openstack-infra | 09:00 | |
*** e0ne has joined #openstack-infra | 09:02 | |
*** jistr has joined #openstack-infra | 09:02 | |
*** ildikov has quit IRC | 09:03 | |
*** ondergetekende_ has quit IRC | 09:03 | |
*** garyh has quit IRC | 09:06 | |
*** e0ne has quit IRC | 09:08 | |
*** ibiris is now known as ibiris_away | 09:09 | |
*** dims__ has quit IRC | 09:12 | |
*** katyafervent is now known as katyafervent_awa | 09:14 | |
*** katyafervent_awa is now known as katyafervent | 09:17 | |
*** yamamoto has quit IRC | 09:17 | |
*** ibiris_away is now known as ibiris | 09:18 | |
*** ujuc has joined #openstack-infra | 09:18 | |
*** zz_johnthetubagu is now known as johnthetubaguy | 09:18 | |
*** HeOS has joined #openstack-infra | 09:21 | |
*** ujuc has quit IRC | 09:22 | |
*** markvoelker has joined #openstack-infra | 09:24 | |
*** derekh has joined #openstack-infra | 09:25 | |
*** andreykurilin_ has quit IRC | 09:26 | |
*** jlanoux has joined #openstack-infra | 09:26 | |
*** markvoelker has quit IRC | 09:29 | |
*** afazekas_ has joined #openstack-infra | 09:29 | |
*** dalgaaf has joined #openstack-infra | 09:40 | |
*** dhritishikhar has joined #openstack-infra | 09:40 | |
*** liusheng has joined #openstack-infra | 09:41 | |
*** dannywilson has joined #openstack-infra | 09:41 | |
*** jedimike has joined #openstack-infra | 09:43 | |
*** jedimike has quit IRC | 09:43 | |
*** notnownikki has joined #openstack-infra | 09:44 | |
*** ZZelle has quit IRC | 09:44 | |
*** ZZelle has joined #openstack-infra | 09:44 | |
*** dannywilson has quit IRC | 09:46 | |
openstackgerrit | Merged openstack-infra/storyboard: Story events may now be filtered by event_type. https://review.openstack.org/160462 | 09:47 |
*** jyuso1 has quit IRC | 09:51 | |
*** jyuso1 has joined #openstack-infra | 09:51 | |
*** Ala has joined #openstack-infra | 09:51 | |
*** dims__ has joined #openstack-infra | 09:51 | |
*** yfried__ is now known as yfried|afk | 09:53 | |
*** ujuc has joined #openstack-infra | 09:53 | |
*** yamamoto has joined #openstack-infra | 09:53 | |
*** yfried|afk is now known as yfried__ | 09:54 | |
*** e0ne has joined #openstack-infra | 09:54 | |
*** tiswanso has quit IRC | 09:55 | |
*** ssam2 has joined #openstack-infra | 09:57 | |
*** pelix has joined #openstack-infra | 09:57 | |
*** e0ne is now known as e0ne_ | 10:00 | |
*** _nadya_ has joined #openstack-infra | 10:00 | |
openstackgerrit | Merged openstack-infra/storyboard-webclient: Removed username. https://review.openstack.org/165193 | 10:01 |
*** e0ne_ has quit IRC | 10:05 | |
*** pelix has quit IRC | 10:06 | |
*** pelix has joined #openstack-infra | 10:07 | |
*** rlandy has joined #openstack-infra | 10:07 | |
*** garyh has joined #openstack-infra | 10:07 | |
*** Hal1 has quit IRC | 10:08 | |
*** Hal1 has joined #openstack-infra | 10:10 | |
*** mpavone has quit IRC | 10:10 | |
*** Hal1 has quit IRC | 10:10 | |
*** MrAboii has joined #openstack-infra | 10:10 | |
*** enikanorov_ has quit IRC | 10:11 | |
*** enikanorov_ has joined #openstack-infra | 10:12 | |
*** ondergetekende has joined #openstack-infra | 10:13 | |
*** e0ne has joined #openstack-infra | 10:14 | |
*** freerunner has quit IRC | 10:14 | |
*** freerunner has joined #openstack-infra | 10:14 | |
*** _nadya_ has quit IRC | 10:15 | |
*** garyh has quit IRC | 10:17 | |
*** cdent has joined #openstack-infra | 10:18 | |
*** Hal1 has joined #openstack-infra | 10:20 | |
*** otter768 has joined #openstack-infra | 10:23 | |
*** ihrachyshka has joined #openstack-infra | 10:23 | |
*** markvoelker has joined #openstack-infra | 10:25 | |
*** mpavone has joined #openstack-infra | 10:25 | |
*** otter768 has quit IRC | 10:27 | |
*** yamamoto has quit IRC | 10:27 | |
openstackgerrit | Aleksey Ripinen proposed openstack-infra/storyboard: Added events for tags https://review.openstack.org/165074 | 10:28 |
*** yamamoto has joined #openstack-infra | 10:28 | |
*** asettle has joined #openstack-infra | 10:29 | |
*** markvoelker has quit IRC | 10:30 | |
*** dhritishikhar has quit IRC | 10:33 | |
*** dtantsur|brb is now known as dtantsur | 10:34 | |
*** soren has quit IRC | 10:37 | |
*** vigneshvar has joined #openstack-infra | 10:41 | |
*** Somay has joined #openstack-infra | 10:43 | |
*** yamamoto has quit IRC | 10:43 | |
*** ominakov has quit IRC | 10:44 | |
*** dhritishikhar has joined #openstack-infra | 10:45 | |
*** ihrachyshka has quit IRC | 10:47 | |
*** ghostpl_ has quit IRC | 10:49 | |
*** ghostpl_ has joined #openstack-infra | 10:51 | |
*** Hal1 has quit IRC | 10:51 | |
*** Hal1 has joined #openstack-infra | 10:52 | |
*** ddieterly has joined #openstack-infra | 10:54 | |
*** mpavone has quit IRC | 10:57 | |
*** ddieterly has quit IRC | 10:59 | |
*** dhritishikhar has quit IRC | 11:00 | |
*** Bsony_ has joined #openstack-infra | 11:00 | |
*** gmann_ has joined #openstack-infra | 11:03 | |
*** gmann_ has left #openstack-infra | 11:03 | |
*** Bsony has quit IRC | 11:04 | |
*** e0ne is now known as e0ne_ | 11:10 | |
*** coolsvap is now known as coolsvap|afk | 11:11 | |
openstackgerrit | Kota Tsuyuzaki proposed openstack-infra/project-config: Add new tox funxtional test and gate to swift3 https://review.openstack.org/166568 | 11:13 |
*** e0ne_ is now known as e0ne | 11:17 | |
*** garyh has joined #openstack-infra | 11:17 | |
*** yfried__ is now known as yfried|afk | 11:20 | |
openstackgerrit | Chris Dent proposed openstack-infra/project-config: Make ceilometerclient-dsvm-functional voting https://review.openstack.org/166784 | 11:23 |
*** mmedvede has joined #openstack-infra | 11:24 | |
*** markvoelker has joined #openstack-infra | 11:26 | |
*** jcoufal has joined #openstack-infra | 11:27 | |
*** garyh has quit IRC | 11:27 | |
*** marcusvrn has joined #openstack-infra | 11:28 | |
openstackgerrit | Aleksey Ripinen proposed openstack-infra/storyboard: Refactor Tasks to work as a Subcontroller https://review.openstack.org/150052 | 11:30 |
*** mwagner_lap has quit IRC | 11:30 | |
*** ajo has joined #openstack-infra | 11:30 | |
*** markvoelker has quit IRC | 11:30 | |
*** zul has joined #openstack-infra | 11:32 | |
*** dizquierdo has joined #openstack-infra | 11:37 | |
*** _nadya_ has joined #openstack-infra | 11:38 | |
*** _nadya_ has quit IRC | 11:38 | |
openstackgerrit | Merged openstack-infra/puppet-zuul: Modernize template syntax https://review.openstack.org/157136 | 11:39 |
*** yfried|afk is now known as yfried__ | 11:40 | |
openstackgerrit | Merged openstack-infra/system-config: Configure puppetmaster server from manifest https://review.openstack.org/164670 | 11:41 |
*** shardy has joined #openstack-infra | 11:42 | |
*** pc_m has joined #openstack-infra | 11:42 | |
*** mpavone has joined #openstack-infra | 11:44 | |
*** Longgeek has quit IRC | 11:46 | |
*** markvoelker has joined #openstack-infra | 11:46 | |
*** ibiris is now known as ibiris_away | 11:47 | |
*** _nadya_ has joined #openstack-infra | 11:47 | |
*** _nadya_ has quit IRC | 11:48 | |
*** yamamoto_ has joined #openstack-infra | 11:53 | |
*** marcusvrn has quit IRC | 11:53 | |
*** ddieterly has joined #openstack-infra | 11:55 | |
*** AJaeger has joined #openstack-infra | 11:55 | |
*** AJaeger has joined #openstack-infra | 11:55 | |
*** claudiub has joined #openstack-infra | 11:56 | |
*** soren has joined #openstack-infra | 11:56 | |
AJaeger | HI, anybody around that could hell with setuptools and pbr, please? | 11:56 |
*** weshay has joined #openstack-infra | 11:56 | |
*** pblaho has joined #openstack-infra | 11:57 | |
*** ibiris_away is now known as ibiris | 11:58 | |
*** asettle has quit IRC | 11:59 | |
*** ddieterly has quit IRC | 11:59 | |
*** ldnunes has joined #openstack-infra | 12:00 | |
*** kgiusti has joined #openstack-infra | 12:01 | |
sdague | mordred: if hpcloud is still having issues on the delete side, could you see if they can tune it with the following index change (it's in kilo) - https://review.openstack.org/#/c/160195/2/nova/db/sqlalchemy/migrate_repo/versions/255_update_instances_project_id_index.py,cm | 12:04 |
*** e0ne is now known as e0ne_ | 12:05 | |
*** bradjones has quit IRC | 12:05 | |
*** ominakov has joined #openstack-infra | 12:06 | |
*** doug-fish has joined #openstack-infra | 12:06 | |
*** bradjones has joined #openstack-infra | 12:06 | |
*** marcusvrn has joined #openstack-infra | 12:13 | |
*** _nadya_ has joined #openstack-infra | 12:13 | |
*** sreshetnyak has quit IRC | 12:18 | |
*** dkranz has joined #openstack-infra | 12:18 | |
*** ominakov has quit IRC | 12:19 | |
*** mjturek1 has joined #openstack-infra | 12:20 | |
*** jcoufal has quit IRC | 12:20 | |
*** mwagner_lap has joined #openstack-infra | 12:23 | |
*** otter768 has joined #openstack-infra | 12:24 | |
*** gmann_ has joined #openstack-infra | 12:25 | |
*** gordc has joined #openstack-infra | 12:27 | |
*** aysyd has joined #openstack-infra | 12:28 | |
*** otter768 has quit IRC | 12:28 | |
*** e0ne_ has quit IRC | 12:29 | |
*** Hal1 has quit IRC | 12:31 | |
*** ihrachyshka has joined #openstack-infra | 12:31 | |
*** dims__ has quit IRC | 12:34 | |
*** dims__ has joined #openstack-infra | 12:34 | |
*** jcoufal has joined #openstack-infra | 12:36 | |
*** notnownikki has quit IRC | 12:37 | |
AJaeger | sdague, as requirements expert, could you briefly have a look at https://review.openstack.org/#/c/166796/ - and my email? I wonder why this passed the gate at all... | 12:37 |
AJaeger | sdague, email link: http://lists.openstack.org/pipermail/openstack-dev/2015-March/059628.html | 12:38 |
*** ominakov has joined #openstack-infra | 12:38 | |
*** tnovacik has quit IRC | 12:40 | |
sdague | there was a whole discussion on friday | 12:41 |
sdague | ~= requires really new setuptools and pip, which are not available in all the infra | 12:41 |
*** ominakov has quit IRC | 12:41 | |
AJaeger | sdague, I missed that ;( | 12:42 |
sdague | however, we did a full fix | 12:42 |
AJaeger | sdague, not for the translation jobs | 12:42 |
AJaeger | https://jenkins.openstack.org/job/glance-propose-translation-update/341/console was run this morning and failed | 12:43 |
AJaeger | sdague, so, I should abandon my patch then? | 12:43 |
sdague | right, but there should be a proposal bot patch up | 12:43 |
sdague | did you look? | 12:43 |
*** weshay has quit IRC | 12:44 | |
*** ominakov has joined #openstack-infra | 12:44 | |
AJaeger | I don't see one - and didn't stumble upon one on project-config | 12:45 |
sdague | so it would be worth figuring out why openstack proposal bot didn't propose to glance | 12:48 |
openstackgerrit | Aleksey Ripinen proposed openstack-infra/storyboard: Fix docs https://review.openstack.org/166803 | 12:49 |
sdague | it did to everyone else once we landed it | 12:49 |
*** ayoung has joined #openstack-infra | 12:51 | |
*** praveenkumar has quit IRC | 12:52 | |
AJaeger | https://jenkins.openstack.org/job/propose-requirements-updates/489/consoleFull - because of the ~= | 12:52 |
*** dims__ is now known as dims | 12:52 | |
AJaeger | sdague, search for glance in the link - that's the latest proposal bot run | 12:53 |
AJaeger | errror is ValueError: ('Expected version spec in', 'oslo.policy~=0.3.1 # apache-2.0', 'at', '~=0.3.1 # apache-2.0') | 12:53 |
AJaeger | So, I guess I should restore my patch again? | 12:53 |
openstackgerrit | Valeriy Ponomaryov proposed openstack-infra/project-config: Add tempest + postgres job for manila https://review.openstack.org/165935 | 12:54 |
sdague | AJaeger: ok, so that's something else that's a bug right? | 12:55 |
*** tiswanso has joined #openstack-infra | 12:55 | |
openstackgerrit | Aleksey Ripinen proposed openstack-infra/storyboard: Fix docs https://review.openstack.org/166803 | 12:55 |
AJaeger | sdague, it'S the same underlying problem IMHO - our tools do not cope with ~= | 12:55 |
*** ddieterly has joined #openstack-infra | 12:55 | |
*** xyang1 has joined #openstack-infra | 12:55 | |
AJaeger | sdague, this was fixed in requirements with https://review.openstack.org/#/c/166377/ | 12:55 |
sdague | oh... so someone in glance bumped outside of proposal bot, and proposal bot can't revert | 12:55 |
*** weshay has joined #openstack-infra | 12:56 | |
AJaeger | sdague, exactly! | 12:56 |
AJaeger | sdague, thanks for helping me to understand this | 12:56 |
*** garyh has joined #openstack-infra | 12:56 | |
sdague | AJaeger: yeh, so you are correct, you need to restore yours | 12:57 |
sdague | also, I'd say you should do a full manual sync | 12:57 |
*** ominakov has quit IRC | 12:57 | |
AJaeger | let me ask on the glance list... | 12:57 |
sdague | because glance is unlikely to get a requirements bump | 12:57 |
sdague | as we've basically frozen now | 12:57 |
dims | until that line is fixed... | 12:57 |
sdague | dims: it's event based though | 12:57 |
AJaeger | ok, later - need to meet now... | 12:57 |
sdague | until we land new requirements it won't try to repropose | 12:58 |
*** dboik has quit IRC | 12:58 | |
dims | ah i see | 12:58 |
*** dprince has joined #openstack-infra | 12:59 | |
*** tiswanso has quit IRC | 12:59 | |
*** ddieterly has quit IRC | 13:00 | |
*** mpaolino has joined #openstack-infra | 13:00 | |
*** amotoki has quit IRC | 13:01 | |
*** Bsony has joined #openstack-infra | 13:01 | |
*** praveenkumar has joined #openstack-infra | 13:03 | |
*** Somay has quit IRC | 13:04 | |
*** garyh has quit IRC | 13:04 | |
*** Bsony_ has quit IRC | 13:04 | |
*** Hal1 has joined #openstack-infra | 13:04 | |
*** bhunter71 has joined #openstack-infra | 13:04 | |
mordred | sdague: sure- thanks! - btw - sake of argument, shouldn't that covering index also have instance_id in it? | 13:04 |
*** rfolco has joined #openstack-infra | 13:05 | |
sdague | mordred: IIRC we've got a ton of indexes there already | 13:05 |
mordred | sdague: I'm assuming that the query is something like "select instance_id from instance where project_id=X and deleted=Y" | 13:05 |
sdague | let me look | 13:05 |
*** soren has quit IRC | 13:05 | |
sdague | that's actually the backport prep patch, so the actually kilo change is already in there | 13:05 |
*** ominakov has joined #openstack-infra | 13:05 | |
mordred | cool | 13:05 |
*** amotoki has joined #openstack-infra | 13:06 | |
mordred | (this is just me putting on database consultant hat with only half of the data) | 13:06 |
sdague | yep, that's fair | 13:07 |
mordred | oh - so - wow | 13:07 |
mordred | that table does not have a primary key | 13:07 |
mordred | oh - there it is | 13:07 |
*** baoli has joined #openstack-infra | 13:07 | |
mordred | nevermind - was reading the python wrong | 13:08 |
*** dkranz has quit IRC | 13:08 | |
openstackgerrit | Lars Kellogg-Stedman proposed openstack-infra/project-config: Add "heat-coe-templates" project https://review.openstack.org/164806 | 13:08 |
sdague | fwiw, it came from a reported bug that showed a 2 order of magnitude time saving with some of these queries with this (from >20s to < 0.1s) | 13:09 |
*** ddieterly has joined #openstack-infra | 13:09 | |
mordred | awesome | 13:09 |
*** vigneshvar has quit IRC | 13:09 | |
sdague | so, it's in master, a backport is being preped for juno | 13:09 |
sdague | however, the index could just be added to hpcloud and see if it gets better | 13:09 |
*** mjturek1 has quit IRC | 13:11 | |
*** yamamoto_ has quit IRC | 13:12 | |
*** leakypipes has quit IRC | 13:13 | |
*** annegentle has joined #openstack-infra | 13:15 | |
*** Hal2 has joined #openstack-infra | 13:17 | |
*** Hal1 has quit IRC | 13:17 | |
*** ddieterly has quit IRC | 13:18 | |
*** gmann_ has quit IRC | 13:18 | |
mordred | sdague: I have sent an email to a set of people | 13:18 |
*** dboik has joined #openstack-infra | 13:18 | |
sdague | mordred: cool | 13:18 |
*** ildikov has joined #openstack-infra | 13:19 | |
mordred | sdague: thank you! I would really love for us to be able to delete instances | 13:19 |
*** yamamoto_ has joined #openstack-infra | 13:19 | |
sdague | yeh, well the fix was already out there, I just noticed it this morning while reviewing code | 13:21 |
*** dkranz has joined #openstack-infra | 13:21 | |
*** dustins has joined #openstack-infra | 13:23 | |
*** peristeri has joined #openstack-infra | 13:23 | |
*** yfried__ is now known as yfried|afk | 13:24 | |
*** coolsvap|afk is now known as coolsvap | 13:25 | |
*** yolanda has joined #openstack-infra | 13:26 | |
*** weshay has quit IRC | 13:26 | |
*** mriedem1 has joined #openstack-infra | 13:27 | |
*** mriedem has quit IRC | 13:27 | |
*** claudiub has quit IRC | 13:28 | |
*** mattfarina has joined #openstack-infra | 13:30 | |
*** alexpilotti has joined #openstack-infra | 13:31 | |
*** tiswanso has joined #openstack-infra | 13:31 | |
*** mjturek1 has joined #openstack-infra | 13:33 | |
fungi | aside from known issues with tripleo, things don't look horribly broken yet this morning | 13:34 |
fungi | strange for a monday | 13:34 |
openstackgerrit | Aleksey Ripinen proposed openstack-infra/storyboard: Session per request https://review.openstack.org/155751 | 13:35 |
*** yfried|afk is now known as yfried__ | 13:35 | |
openstackgerrit | yolanda.robla proposed openstack-infra/system-config: Don't hardcode pip.conf values https://review.openstack.org/166252 | 13:36 |
*** Somay has joined #openstack-infra | 13:37 | |
openstackgerrit | Aleksey Ripinen proposed openstack-infra/storyboard: Fix docs https://review.openstack.org/166803 | 13:38 |
mordred | fungi: wohoo! | 13:38 |
*** BharatK has quit IRC | 13:38 | |
fungi | maybe it means that i can get some more productive hacking done after my next meeting | 13:38 |
mordred | fungi: I did some work over the weekend on the debian base elements in dib | 13:38 |
fungi | i saw! | 13:38 |
mordred | fungi: if you're back around to hacking on that - they may be of interest | 13:39 |
mordred | :) | 13:39 |
*** superdan is now known as dansmith | 13:39 | |
mordred | although I imagine your current project does not care one way or the other about the changes I made, tbh | 13:39 |
fungi | indeed, i should give it a shot, though first i need to get the database setup builder macros added to jobs so that they don't rely on mysql and postgres to be set up by dib | 13:39 |
mordred | ++ | 13:40 |
fungi | well, they're sort of intertwined there | 13:40 |
fungi | stepping up the job runtime databse setup work was to allow us to dib bare-.* nodes so that i could dib a debian worker to run py3k jobs (because one of our providers lacked a jessie base image) | 13:41 |
mordred | yes. | 13:41 |
*** dkliban_afk is now known as dkliban | 13:41 | |
mordred | and then eventually so that there can be no difference between bare-* and devstack-* nodes | 13:42 |
*** woodster_ has joined #openstack-infra | 13:42 | |
fungi | right, that's the impetus for bindep since it's basically the only missing piece to that puzzle | 13:42 |
mordred | we're certainly getting some where | 13:42 |
mordred | I'm hoping that soon I'll have the ability to have debian|fedora|ubuntu|centos base images that can be built and work in both of our clouds | 13:42 |
mordred | which means we can use them as the base for dib nodepool | 13:43 |
mordred | but also so that we can control all of our own base images for long-lived things | 13:43 |
mordred | which would make me the hapies | 13:43 |
fungi | how close are we to viable dib for nodepool in rackspace now? | 13:43 |
mordred | fungi: VERY close | 13:43 |
mordred | fungi: it's really about init-script hacking | 13:43 |
* fungi rubs hands together | 13:43 | |
mordred | and by init-script, I mean "make sure this script runs correctly in upstart, sysvinit and systemd" | 13:43 |
fungi | just for our custom bits right? | 13:44 |
mordred | fungi: Ng, greghaynes, and SpamapS are all looking at it ... Ng has been running in to trouble with rackspace LON being a completely different cloud | 13:44 |
mordred | fungi: yes - just for the one script we need run at boot | 13:44 |
mordred | fungi: so, it's a totally tractable problem | 13:44 |
*** sabeen1 has joined #openstack-infra | 13:44 | |
mordred | just needs someone who isn't me to look at it for a bit | 13:44 |
*** mrmartin has joined #openstack-infra | 13:45 | |
openstackgerrit | yolanda.robla proposed openstack-infra/puppet-zuul: Add fetch_url and poll parameters to zuul.conf https://review.openstack.org/162591 | 13:45 |
mordred | fungi: once we have that - I also have an ansible playbook already written that does "build image once, upload to all the clouds" - so managing our own base-images in our long-lived providers should be completely driveable from commits in system-config | 13:47 |
fungi | what did the solution end up being to the different image format requirements between providers? | 13:48 |
mordred | fungi: image conversion | 13:48 |
mordred | fungi: I mean, we essentially build one raw image, and then convert it to both qcow2 and vhd | 13:48 |
fungi | outside of dib itself? oh, right, you did something docker-based for that yeah? | 13:48 |
mordred | fungi: it can also be debian packaged - it's just a utility | 13:49 |
mordred | I have debian packaging for it 80% done | 13:49 |
fungi | oh, so docker was just a workaround for lack of packaged dependencies? | 13:49 |
*** sabeen1 has quit IRC | 13:49 | |
mordred | fungi: docker was a workaround for making debian packages being a pita | 13:49 |
*** salv-orlando has joined #openstack-infra | 13:50 | |
fungi | aha, that. well yes, it can be | 13:50 |
mordred | fungi: https://github.com/emonty/vhd-util/blob/master/build.sh that's what you have to do to build the util | 13:50 |
*** mpaolino has quit IRC | 13:50 | |
mordred | fungi: I made this: https://github.com/emonty/deb-vhd-util to translate that into debian ... but something in the ppa upload broke and I haven't cared to fix it yet | 13:51 |
mordred | greghaynes: ^^ btw - I think you were asking about that over the weekend | 13:51 |
mordred | greghaynes: also https://launchpad.net/~mordred/+archive/ubuntu/vhd-util | 13:51 |
*** e0ne has joined #openstack-infra | 13:51 | |
fungi | given it's just the typical configure&&make&&make install dance, dh-make ought to deal with it pretty well | 13:52 |
fungi | i'm guessing there was some odd reason it didn't | 13:52 |
*** dprince has quit IRC | 13:52 | |
*** dprince has joined #openstack-infra | 13:52 | |
*** sabeen has joined #openstack-infra | 13:53 | |
mordred | nah - it's mostly fine | 13:53 |
mordred | right now I think we're breaking on missing a wget depend :) | 13:53 |
*** sdake has joined #openstack-infra | 13:57 | |
cdent | can I get a quick review on changing an experimental job to voting https://review.openstack.org/#/c/166784/ ? The removal of the related tests from tempest got approved before this change was in place, so the ceilometerclient is lacking a bit of coverage in the gate | 13:57 |
*** nilasae has quit IRC | 13:57 | |
cdent | please and thank you | 13:57 |
*** mrunge is now known as mrunge_bbl | 13:57 | |
*** armax has joined #openstack-infra | 13:58 | |
AJaeger | sdague, could you +1 https://review.openstack.org/#/c/166796/, please? I've updated as suggested... | 13:59 |
mtreinish | cdent: oh, I didn't realize that wasn't approved when I +2d the tempest removal. Maybe we should have -2d/WIPd it | 13:59 |
cdent | mtreinish: isn't a huge deal: there's nothing important pending on ceilometerclient at the moment | 13:59 |
*** sdake__ has joined #openstack-infra | 13:59 | |
openstackgerrit | yolanda.robla proposed openstack-infra/puppet-jenkins: Add extension settings in jjb https://review.openstack.org/162152 | 14:00 |
*** deepakcs has quit IRC | 14:00 | |
cdent | I just wanted to be sure it didn't get missed | 14:00 |
openstackgerrit | yolanda.robla proposed openstack-infra/puppet-jenkins: Add extension settings in jjb https://review.openstack.org/162152 | 14:00 |
*** ominakov has quit IRC | 14:01 | |
*** sdake has quit IRC | 14:01 | |
fungi | depends-on header could also have been used there | 14:01 |
fungi | to prevent the tempest change landing prior to the project-config change | 14:02 |
*** hogepodge has joined #openstack-infra | 14:02 | |
sdague | flaper87: can you land - https://review.openstack.org/#/c/166796/ | 14:02 |
*** ominakov has joined #openstack-infra | 14:02 | |
sdague | cdent: +2 from me | 14:02 |
cdent | fungi, yeah I realized that over the weekend, but by then it was too late. My original mistake was that I forgot to mark the tempest change as a wip. I got distracted by something in the midst and completely forgot. | 14:03 |
sdague | yeh, Depends-On is really good for blocking merges | 14:03 |
cdent | Will try to remain focused for the next time. | 14:03 |
cdent | Last week was _bad_ for focus. | 14:03 |
sdague | I think we probably need to expose some visualization around it | 14:03 |
sdague | because it's not always clear why things aren't landing | 14:03 |
openstackgerrit | Nikita Konovalov proposed openstack-infra/storyboard-webclient: Support for tags https://review.openstack.org/166847 | 14:03 |
*** prad has joined #openstack-infra | 14:04 | |
*** coolsvap is now known as coolsvap|afk | 14:05 | |
*** nelsnelson has joined #openstack-infra | 14:06 | |
openstackgerrit | Sean Dague proposed openstack/requirements: Bump cryptography version to latest https://review.openstack.org/164289 | 14:07 |
*** julim has joined #openstack-infra | 14:07 | |
*** jerryz has joined #openstack-infra | 14:08 | |
*** dhritishikhar has joined #openstack-infra | 14:08 | |
*** ctlaugh has quit IRC | 14:08 | |
*** mpaolino has joined #openstack-infra | 14:09 | |
*** stevemar has joined #openstack-infra | 14:10 | |
*** ddieterly has joined #openstack-infra | 14:10 | |
openstackgerrit | Merged openstack-infra/project-config: Make ceilometerclient-dsvm-functional voting https://review.openstack.org/166784 | 14:11 |
*** ctlaugh has joined #openstack-infra | 14:12 | |
*** mfink_ has joined #openstack-infra | 14:12 | |
*** radez_g0n3 is now known as radez | 14:12 | |
*** zz_jgrimm- is now known as jgrimm | 14:12 | |
*** yamamoto_ has quit IRC | 14:12 | |
*** eharney has joined #openstack-infra | 14:13 | |
*** dhritishikhar has quit IRC | 14:15 | |
*** tnovacik has joined #openstack-infra | 14:15 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 14:16 | |
*** sdake has joined #openstack-infra | 14:16 | |
*** yfried__ is now known as yfried|afk | 14:17 | |
*** yfried|afk is now known as yfried__ | 14:18 | |
*** ddieterly has quit IRC | 14:18 | |
*** coolsvap has joined #openstack-infra | 14:19 | |
*** sdake__ has quit IRC | 14:20 | |
*** Hal2 has quit IRC | 14:21 | |
*** mtanino has joined #openstack-infra | 14:21 | |
*** tiswanso has quit IRC | 14:21 | |
*** timcline has joined #openstack-infra | 14:23 | |
*** zz_dimtruck is now known as dimtruck | 14:23 | |
*** otter768 has joined #openstack-infra | 14:24 | |
*** asselin_ has joined #openstack-infra | 14:25 | |
vponomaryov | who is available from i18n/translation team? | 14:26 |
AJaeger | vponomaryov, I'm here. There's also a separate openstack-i18n channel | 14:26 |
AJaeger | vponomaryov, I might be able to help | 14:27 |
vponomaryov | AJaeger: thanks, i will write there | 14:27 |
vponomaryov | AJaeger: err, is "openstack-i18n" correct channel name? | 14:27 |
AJaeger | vponomaryov, sorry: openstack-translation | 14:28 |
*** otter768 has quit IRC | 14:29 | |
*** annegentle has quit IRC | 14:29 | |
*** dangers_away is now known as dangers | 14:29 | |
openstackgerrit | Monty Taylor proposed openstack-infra/project-config: Add config-drive element https://review.openstack.org/154132 | 14:30 |
openstackgerrit | Monty Taylor proposed openstack-infra/project-config: Add elements for Infra servers https://review.openstack.org/140840 | 14:30 |
*** ddieterly has joined #openstack-infra | 14:31 | |
mordred | fungi: updated https://review.openstack.org/166650 in dib to work | 14:31 |
*** amitgandhinz has joined #openstack-infra | 14:32 | |
*** wenlock has joined #openstack-infra | 14:32 | |
*** dannywilson has joined #openstack-infra | 14:33 | |
HeOS | Hi to all! I have following strange problem: I have "Administrator" permissions in gerrit - I can see project "foo-bar" in web-ui, but I can't see this project in output of "ls-projects" command. Do you know a reasons of this problem? | 14:33 |
*** sdake__ has joined #openstack-infra | 14:34 | |
*** sdake__ has quit IRC | 14:34 | |
*** achanda has joined #openstack-infra | 14:34 | |
*** annegentle has joined #openstack-infra | 14:34 | |
*** sdake__ has joined #openstack-infra | 14:34 | |
*** annegentle has quit IRC | 14:34 | |
fungi | HeOS: i'm guessing you don't mean in our (review.openstack.org) gerrit. have you asked in #gerrit? | 14:35 |
*** ddieterly has quit IRC | 14:35 | |
*** ddieterly has joined #openstack-infra | 14:35 | |
fungi | we don't actually write gerrit, we just run a copy of it for the openstack developer community | 14:36 |
*** ctlaugh has quit IRC | 14:36 | |
*** ctlaugh has joined #openstack-infra | 14:36 | |
*** asselin__ has joined #openstack-infra | 14:36 | |
*** sdake has quit IRC | 14:37 | |
HeOS | fungi, I asked them, but I know, that you have experience in using of gerrit. That is why I've wrote my question to this channel too. :) | 14:38 |
*** asselin_ has quit IRC | 14:40 | |
*** asselin_ has joined #openstack-infra | 14:40 | |
*** achanda has quit IRC | 14:41 | |
*** asselin__ has quit IRC | 14:41 | |
*** pelix has quit IRC | 14:41 | |
*** Hal1 has joined #openstack-infra | 14:42 | |
*** mpaolino has quit IRC | 14:42 | |
*** garyh has joined #openstack-infra | 14:44 | |
*** sarob has joined #openstack-infra | 14:44 | |
*** bnemec has quit IRC | 14:46 | |
*** MarkAtwood has joined #openstack-infra | 14:46 | |
fungi | HeOS: you see some projects listed in ls-projects output but not foo-bar? make sure the "state" on that project is one of active or read-only, not hidden | 14:47 |
*** pelix has joined #openstack-infra | 14:47 | |
*** jedimike has joined #openstack-infra | 14:48 | |
*** jedimike has quit IRC | 14:48 | |
fungi | i have no idea whether admins can see hidden projects in ls-projects, since we don't have any hidden projects | 14:48 |
*** sarob has quit IRC | 14:48 | |
*** bnemec has joined #openstack-infra | 14:49 | |
*** notnownikki has joined #openstack-infra | 14:49 | |
*** r-daneel has joined #openstack-infra | 14:49 | |
*** ddieterly has quit IRC | 14:50 | |
HeOS | fungi, yes, state of this project is "Active". | 14:50 |
openstackgerrit | Julien Danjou proposed openstack-dev/pbr: Handle PEP426 markers https://review.openstack.org/165015 | 14:51 |
*** annegentle has joined #openstack-infra | 14:51 | |
dhellmann | lifeless, mordred, dstufft : I would appreciate your input on this pbr change, because I'm confused: https://review.openstack.org/#/c/165015/1 | 14:51 |
fungi | no clue then. we configure our gerrit so that all users can see everything because we don't like to hide anything from our community. as such we don't run into a lot of these sorts of issues | 14:51 |
fungi | HeOS: if you want everything in your gerrit visible, i suggest checking your configuration against ours http://ci.openstack.org/gerrit.html#gerrit-configuration | 14:52 |
HeOS | fungi, thanks. :) | 14:54 |
fungi | dhellmann: i thought the outcome of the discussion favored providing a fallback mechanism to translate the current split files to markers too | 14:54 |
fungi | though i was only skimming | 14:54 |
mordred | dhellmann: looking | 14:55 |
fungi | though i agree that not doing that and being purely forward-looking simplifies the implementation | 14:55 |
mordred | I don't think that implementation removes the other thing | 14:55 |
mordred | and I think we need to keep the other thing for some time because of compatability with things we've released that use pbr and have those files | 14:56 |
mordred | however, I think we shoudl add the new capability and then strongly suggest to projects using the old thing to move to the new thing | 14:56 |
mordred | so that we could maybe one day remove the old code path | 14:57 |
*** thedodd has joined #openstack-infra | 14:57 | |
fungi | maybe add a deprecation warning when handling split requirements | 14:57 |
*** dannywilson has quit IRC | 14:57 | |
fungi | though again that's something that will annoy users and not necessarily come to the immediate attention of projects using pbr in that way | 14:57 |
*** dannywilson has joined #openstack-infra | 14:58 | |
fungi | so maybe not a great idea | 14:58 |
mordred | yes. I don't think it's super important to deprecate it | 14:58 |
mordred | honestly | 14:58 |
mordred | I think there are like 3 users of the feature | 14:58 |
fungi | well, as far as we know | 14:58 |
mordred | everyone who isn't openstack hates pbr :) | 14:58 |
fungi | but i agree it's probably reasonable in this case to assume it's openstack projects only | 14:59 |
mordred | even so though - because we cannot version pbr - I think it is completely unreasonable to remove support for the thing since we know we have stuff out in the wild, and it's 100% possible that us removing support will screw those people ina weird way | 14:59 |
anteaya | don't a few folks who identify as openstack look sideways at pbr as well? | 14:59 |
mordred | anteaya: yes | 14:59 |
anteaya | okay | 14:59 |
*** tiswanso has joined #openstack-infra | 15:01 | |
openstackgerrit | Mateusz Matuszkowiak proposed openstack-infra/project-config: Added fuel-plugin-cisco-aci fuel plugin to SF https://review.openstack.org/166872 | 15:01 |
clarkb | mordred shouldnt it understand what reqs to drop as well? | 15:02 |
NobodyCam | morning -Infra folk | 15:02 |
clarkb | I think that is dhellmanns confusion | 15:03 |
*** carl_baldwin has joined #openstack-infra | 15:03 | |
clarkb | either it needs to do it or something else | 15:03 |
NobodyCam | is there something up with etherpad.openstack.org? | 15:03 |
fungi | NobodyCam: the service is up. what else do you see up>? | 15:04 |
NobodyCam | fungi: this is not at all correct https://etherpad.openstack.org/p/IronicWhiteBoard | 15:04 |
*** pblaho has quit IRC | 15:05 | |
NobodyCam | all historical version also look the same? | 15:05 |
fungi | NobodyCam: what's not corect about it? i see a bunch of rst | 15:05 |
jroll | fungi: over a year of history including what was there when we last saw it is gone :| | 15:06 |
fungi | oh, i think we've seen the "etherpad forgets historical versions" issue with other pads before | 15:06 |
mordred | clarkb: I don't understand your question? | 15:06 |
NobodyCam | fungi: so everything is gone? | 15:07 |
fungi | jroll: NobodyCam: it rolled back to an earlier version on its own? | 15:07 |
NobodyCam | fungi: that is like the sample page | 15:07 |
*** dhritishikhar has joined #openstack-infra | 15:07 | |
fungi | NobodyCam: huh? i see a bunch of restructuredtext content | 15:07 |
fungi | NobodyCam: not the sample page | 15:07 |
clarkb | mordred its ignoring the data instead of using it | 15:07 |
jroll | fungi: it was lists of bugs/specs/ongoing work etc for ironic | 15:07 |
NobodyCam | fungi: is your first line: An h1 header | 15:08 |
jroll | fungi: I don't remember it ever being a RST formatting document, in the year I've been on the project | 15:08 |
fungi | jroll: looks like something someone has pasted in | 15:08 |
*** MrAboii has quit IRC | 15:09 | |
jroll | fungi: sure. maybe. no history to be found :| | 15:09 |
fungi | the template says something like "Welcome to Etherpad Lite! blah, blah..." | 15:09 |
jroll | fungi: if there's going to be no way to get it back, let us know now and we'll start from scratch :) | 15:09 |
*** bdpayne has joined #openstack-infra | 15:09 | |
fungi | jroll: i can probably look at it after my current meeting is over | 15:10 |
NobodyCam | fungi: Thank you | 15:10 |
fungi | will require some spelunking in the backend database | 15:10 |
NobodyCam | :( | 15:10 |
jroll | fungi: that would be awesome, thank you. looking for "gate status", "subteam status updates", etc | 15:10 |
NobodyCam | its like will over a years history for us | 15:10 |
mordred | clarkb: oh - hrm. ok - I understand the confusion more now - I agree, we don't want to lose that information if it's there | 15:10 |
fungi | NobodyCam: jroll: i would personally feel very uncomfortable trying to maintain revision control history of important data in an etherpad, but maybe that's just me | 15:12 |
openstackgerrit | Marton Kiss proposed openstack-infra/puppet-askbot: Added missing askbot cron jobs https://review.openstack.org/166882 | 15:12 |
jroll | fungi: it isn't a horrible loss IMO, we literally treat it like a long-running whiteboard | 15:12 |
NobodyCam | fungi: its more like a current status with other important bits of data attached | 15:13 |
*** yamamoto_ has joined #openstack-infra | 15:13 | |
HeOS | fungi, I found the reason of this problem. :) This project had no "master" branch. | 15:13 |
*** yfried has joined #openstack-infra | 15:13 | |
fungi | HeOS: very odd behavior. i guess we don't see it because we make sure all projects are created or imported with a master branch | 15:14 |
*** yfried has quit IRC | 15:14 | |
*** yfried has joined #openstack-infra | 15:14 | |
mordred | dhellmann, clarkb: I tink the change right now might be "stop erroring on environment markers if they exist" - with a follow up needed later of "support environment markers" perhaps? | 15:15 |
*** yfried__ has quit IRC | 15:16 | |
HeOS | fungi, this project was created by hands from web-ui and he is was marked as "parent" for another project. :) | 15:16 |
mordred | since there is two problems - if pip doesn't support them directly in the install_requires at the moment, then people putting them in ...oh - wait | 15:16 |
mordred | let me look at something | 15:16 |
dhellmann | mordred: why add them if nothing is going to use them? I thought this was a migration away from using separate files? jd__ ^^ | 15:16 |
mordred | yeah - I think I just realized a different problem that might make this whole thing go away - but let me look ... | 15:16 |
mordred | because i'm still coffeeing | 15:17 |
jd__ | dhellmann: pip isn't using them? | 15:17 |
*** hashar is now known as hasharAwa | 15:17 | |
*** hasharAwa is now known as hasharAway | 15:17 | |
dhellmann | jd__: I don't know, that's what I'm trying to find out. | 15:17 |
*** ociuhandu has joined #openstack-infra | 15:18 | |
jd__ | it is using them, I read the source - or at least it tries to :-) | 15:18 |
*** yamamoto_ has quit IRC | 15:19 | |
mordred | awesome. then we shouldn't need to do anything but pass them through like we do now - can you describe what breaks in the current code? | 15:19 |
mordred | dhellmann: I think we have a code path in pbr that is historical that we don't need anymore, btw | 15:19 |
dhellmann | mordred: I wouldn't be surprised to learn that :-)( | 15:20 |
mordred | dhellmann: back in the early days, we made sure to parse requirements.txt and inject it into install_requires so that things would have access to it | 15:20 |
mordred | BUT | 15:20 |
mordred | we override install and sub-contract that to pip now | 15:20 |
mordred | AND | 15:20 |
mordred | we override egg_info which is what pip calls to find out what to install for dependencies | 15:20 |
dhellmann | mordred: so pip is calling setuptools is calling pbr is calling pip? | 15:20 |
mordred | yes | 15:20 |
mordred | so really, there is zero benefit to actually parsing and putting thigns into install_requires, since we don't support easy_install at all | 15:21 |
mordred | (taht woudl be the only benefit of injecting the data into that variable) | 15:21 |
mordred | but we weren't as aggressive about that in the early days - so there is still a bunch of complexity in there | 15:21 |
dhellmann | excellent, maybe we can make pbr just throw an error "we don't do easy_install"? | 15:21 |
fungi | "pip is thatway ->" | 15:22 |
*** sputnik13 has quit IRC | 15:22 | |
dhellmann | fungi: ++ | 15:22 |
mordred | yup | 15:22 |
mordred | I'll work on that patch - I think it'll make what jd__ wants to accomplish much easier to reason about | 15:22 |
jd__ | if we drop install_requires it's clear everything's going to be simpler indeed :) | 15:23 |
mordred | that, combined with the patch lifeless is working on to upstream pip to handle setup_requires in setup.cfg - should get us to a land of ponies and rainbows | 15:23 |
mordred | jd__: ++ | 15:23 |
dhellmann | mordred: I feel like we're losing a lot of the why's behind these changes because we only ever talk about pbr on irc. I'm not sure what to do about that. | 15:23 |
*** emagana has joined #openstack-infra | 15:23 | |
mordred | dhellmann: I agree | 15:23 |
mordred | dhellmann: and I agree that I'm not sure what to do about it | 15:24 |
clarkb | commit messages? this is why they exist | 15:24 |
dhellmann | my first reaction is to ask for specs, but we had one for semver and it didn't really help there | 15:24 |
mordred | the problem is that it's a large, smushy and tricky subject area to page in to working RAM in your brain | 15:24 |
dhellmann | clarkb: yeah, they should help, but it can be hard to piece them together over a long series of changes | 15:25 |
mordred | so even if we wrote all the things down, the amount of reading you'd have to do to re-get the picture is probably greater than coming into channel and having a half-hour conversation with someone | 15:25 |
dhellmann | mordred: yeah, I feel like I have to relearn the whole thing every time I start looking at it, because that's only once a month or so | 15:25 |
mordred | yup | 15:25 |
mordred | I should write a stem-to-stern WHY document though | 15:25 |
*** sdague has quit IRC | 15:26 | |
mordred | that has a beginning, middle, and end | 15:26 |
*** yfried has quit IRC | 15:26 | |
dhellmann | ++ | 15:26 |
*** claudiub has joined #openstack-infra | 15:27 | |
mordred | dhellmann: YES - looking through the code I believe I can make a crap-ton of things simpler | 15:27 |
mordred | fungi: https://launchpad.net/~mordred/+archive/ubuntu/infra <-- vhd_util successfully built in debian package | 15:29 |
fungi | mordred: dhellmann: clarkb: i think that a big part of the issue is that understanding the why means first having a grasp of lots of rather obscure nuances of python packaging | 15:29 |
dhellmann | fungi: that, too | 15:29 |
*** mclaren has joined #openstack-infra | 15:29 | |
mordred | fungi: ++ | 15:29 |
notnownikki | ello :) who can help me with progress on https://review.openstack.org/#/c/163734/ and https://review.openstack.org/#/c/163402/ for the puppet-bandersnatch module project? | 15:30 |
*** harlowja_at_home has joined #openstack-infra | 15:30 | |
mclaren | anyone know if it's possible to change the author of a commit after it's merged? | 15:30 |
mordred | mclaren: it is not possible | 15:31 |
hogepodge | Can canyone help out with this question? http://lists.openstack.org/pipermail/openstack-infra/2015-March/002536.html | 15:31 |
fungi | mclaren: only possible to change anything about a commit before it's merged. after it's merged the train has sailed | 15:31 |
mclaren | mordred: (ok, thanks, case of a slightly stolen patch) | 15:31 |
hogepodge | The refstack team is looking at integrating with openstack id, but needs some help | 15:31 |
fungi | notnownikki: the governance change needs to be approved by the tc chair after more tc members vote on it | 15:31 |
jeblair | mclaren: you could revert it and reapply it with the correct information | 15:31 |
mordred | jeblair: good point | 15:32 |
fungi | doesn't really change the commit, but does introduce equivalent code | 15:32 |
jeblair | fungi: "git express 999" | 15:32 |
*** ajmiller has joined #openstack-infra | 15:33 | |
*** ujuc has quit IRC | 15:33 | |
*** sslypushenko has joined #openstack-infra | 15:33 | |
*** bdpayne has quit IRC | 15:33 | |
morganfainberg | hogepodge: I use a script that does lp integration, but it opens a web browser, openid is better than oidc for console. Unfortunately, these sso technologies tend to not play nice on the console. | 15:33 |
mclaren | jeblair: thanks (I'll just live with it) | 15:33 |
notnownikki | fungi, ok, thanks :) | 15:34 |
*** mclaren has quit IRC | 15:34 | |
*** vladiskuz_ has joined #openstack-infra | 15:34 | |
*** MrAboii has joined #openstack-infra | 15:35 | |
morganfainberg | hogepodge: I think we need to add the same endpoint info that lp has to the openstackid when you login. That said we probably shouldn't be carrying custom auth technology (custom oo-php) long term | 15:35 |
fungi | jeblair: sounds like a matsumoto film | 15:36 |
* fungi gets busy re-dubbing the original | 15:37 | |
fungi | morganfainberg: hogepodge: there is openstackid documentation proposed but not merged or published yet. smarcet and i have some patches up which might help the situation | 15:37 |
*** sushilkm has joined #openstack-infra | 15:38 | |
*** sushilkm has left #openstack-infra | 15:38 | |
*** yamahata has joined #openstack-infra | 15:39 | |
*** sdague has joined #openstack-infra | 15:40 | |
*** Liu has joined #openstack-infra | 15:41 | |
*** jaypipes has joined #openstack-infra | 15:42 | |
mordred | clarkb, jeblair, fungi: the neutron port leak issue: https://bugs.launchpad.net/nova/+bug/1324934 - in case we didn't have that in our notes before | 15:42 |
openstack | Launchpad bug 1324934 in OpenStack Compute (nova) icehouse "Neutron port leak when connection is dropped during port create in instance boot." [Undecided,Fix committed] | 15:42 |
*** afazekas_ has quit IRC | 15:42 | |
hogepodge | morganfainberg: fungi: thanks. They're going to reach out to tipit also. They're trying to set up public/private key auth similar to what gerrit does | 15:42 |
hogepodge | mediated through openstackid | 15:43 |
fungi | hogepodge: morganfainberg: smarcet's documentation is proposed in https://review.openstack.org/165199 if that helps | 15:44 |
*** jamesmcarthur has joined #openstack-infra | 15:44 | |
*** smarcet has joined #openstack-infra | 15:45 | |
fungi | hogepodge: also, totally unrelated topic, i was pondering the implications of the nova api extensions ml thread on defcore... i wonder how we might get future iterations of defcore to discourage nonstandard extensions to the apis it covers | 15:45 |
fungi | something which would need to be proposed to the committee i know, just pondering whether it's even something we'd be able to detect in the first place | 15:46 |
clarkb | mordred: the os-loganalyze install on static.o.o isn't quite right | 15:47 |
clarkb | mordred: there are a bunch of egg-info files in dist-packages including the correct one but pip freeze reports an older version | 15:47 |
*** dkranz has quit IRC | 15:47 | |
jamesmcarthur | @hogepodge: Hi. I understand there have been some questions around OpenStackID. We’ve just completed the documentation and I think it’s been +1’d. You can see that here: https://review.openstack.org/#/c/165199/3/doc/source/index.rst | 15:48 |
clarkb | mordred: seems like in the past the fix has been to wipe out an install and reinstall it completely? anything else you want ot look at for that? | 15:48 |
jamesmcarthur | hogepodge: is there something else specifically we can answer? | 15:48 |
clarkb | mordred: also thanks for the port leak bug | 15:48 |
hogepodge | jamesmcarthur: morganfainberg had some questions related to the system in general. | 15:49 |
*** dannywilson has quit IRC | 15:49 | |
hogepodge | sslypushenko: jamesmcarthur can help out with any specific questions you have with regards to refstack integration | 15:49 |
hogepodge | sslypushenko: has worked up a nice flow diagram of how he thinks console auth would work | 15:49 |
zaro | morning | 15:51 |
fungi | jamesmcarthur: i pointed him to that change just before you showed up. if we can get https://review.openstack.org/165660 approved, then drop the conf.py from https://review.openstack.org/165199 and rebase it, we should have an automated draft you can browse, and once it's approved it will get published to ci.openstack.org along with our other infrastructure project documentation | 15:51 |
jamesmcarthur | hogepodge: sslypushenko: smarcet: I saw the request for console auth and haven’t had a chance to respond yet. At the moment, we’re not prepared to do that. We need to add support for oAuth2 | 15:51 |
anteaya | morning zaro | 15:51 |
fungi | jamesmcarthur: it's just oauth (v1) and openid at the moment, right? | 15:52 |
smarcet | hogepodge: sslypushenko: oauth2 is not for authentication | 15:52 |
jamesmcarthur | fungi: that’s correct | 15:52 |
smarcet | hogepodge: sslypushenko: its just for authorization, the only authenication protocol that openstackid so far supports is openid 2.0 and its meant for web apps | 15:53 |
pabelanger | hasharAway, once you are back, lets talk packaging | 15:53 |
*** dhritishikhar has quit IRC | 15:53 | |
*** rkukura has joined #openstack-infra | 15:53 | |
*** dhritishikhar has joined #openstack-infra | 15:53 | |
jamesmcarthur | hogepodge: sslypushenko: In general, oAuth should not be used for authentication. It’s not good security practice. | 15:54 |
*** emagana has quit IRC | 15:54 | |
hogepodge | smarcet: sslypushenko: hmm. That's disappointing. I've been pushing pretty hard to use openstack id | 15:54 |
hogepodge | jamesmcarthur: what's the plan for projects like gerrit then? | 15:55 |
clarkb | mordred: interesting info on that, I ran pip install a few times and pip freeze reported a different version each time. Not necessarily in a semver version increasing manner either | 15:55 |
*** emagana has joined #openstack-infra | 15:55 | |
clarkb | mordred: however pip freeze does report the correct version right now | 15:55 |
*** reed has joined #openstack-infra | 15:55 | |
jamesmcarthur | hogepodge: the reason that we didn’t start with OpenID Connect is b/c Gerrit wouldn’t work with it. Gerrit could technically start using OpenStackID as soon as they want. | 15:55 |
hogepodge | jamesmcarthur: smarcet: the use case is having an account on refstack that's hooked into openstackid, then upload results to that account in secure way. | 15:56 |
*** YorikSar has quit IRC | 15:56 | |
fungi | hogepodge: we're planning to use openid for gerrit | 15:56 |
fungi | not oauth | 15:56 |
fungi | gerrit is a webapp | 15:56 |
fungi | so should be fine there | 15:56 |
hogepodge | fungi: openstack id supports openid, right? | 15:56 |
fungi | yep | 15:57 |
jamesmcarthur | hogepodge: fungi: yes sir. | 15:57 |
*** emagana has quit IRC | 15:57 | |
hogepodge | fungi: yes, refstack is too, but isn't there a public/private exchange to upload patch sets? | 15:57 |
sslypushenko | jamesmcarthur fungi Where I can get info/examples how to use openid from OpenstackID? | 15:57 |
*** emagana has joined #openstack-infra | 15:57 | |
fungi | hogepodge: gerrit handles the ssh key management and auth itself. it only uses openid for its web interface | 15:58 |
*** ddieterly has joined #openstack-infra | 15:58 | |
*** asselin_ has quit IRC | 15:58 | |
jamesmcarthur | sslypushenko: we’re still working on a couple of config issues, but as soon as we can get https://review.openstack.org/165199 approved, it will be available on ci.opesntack.org | 15:58 |
sslypushenko | jamesmcarthur Thx! | 15:58 |
*** emagana has quit IRC | 15:59 | |
*** garyk has quit IRC | 15:59 | |
smarcet | hogepodge: we support this workflow on oauth2 https://tools.ietf.org/html/rfc6749#section-4.4 | 15:59 |
hogepodge | sslypushenko: so I would assume web auth and account creation with openstackid, some implementation of ssl auth in refstack for uploading results to that account | 15:59 |
smarcet | hogepodge: that workflow doesnt requires authentication | 15:59 |
smarcet | hogepodge: its assuming that u are a service | 16:00 |
ctlaugh | amotoki: Hi, in the third-party meeting, you mentioned configuring gearman to offline nodes -- could you point me to where that configuration is? I am running 0.0.7 | 16:00 |
*** emagana has joined #openstack-infra | 16:00 | |
hogepodge | sslypushenko: can you share your diagram with smarcet? | 16:00 |
*** garyh has quit IRC | 16:00 | |
smarcet | hogepodge: perhaps its could be usefull for your intended purpose | 16:00 |
fungi | ctlaugh: you get zuul to pass an OFFLINE_NODE_WHEN_COMPLETE=1 parameter in your jobs | 16:01 |
fungi | ctlaugh: we do that by default for all our jobs using a zuul parameter script | 16:01 |
ctlaugh | fungi: ah, ok. I misunderstood - I thought it was a config option in Jenkins (and I couldn't see it) | 16:01 |
clarkb | ok various console logs are coming through with the log level header still, will probably need to properly clean up the os-loganalyze install | 16:01 |
ctlaugh | fungi: I've got that paramater in zuul, so I should be ok there | 16:01 |
ctlaugh | fungi: I don't see any indication that multiple jobs are trying to be sent to the same slave | 16:02 |
sslypushenko | hogepodge I need improve it a litte | 16:02 |
hogepodge | kk | 16:02 |
amotoki | clarkb: sorry I am in another meeting and discusing in a different channel. will be back later. | 16:02 |
*** garyh has joined #openstack-infra | 16:02 | |
*** dkranz has joined #openstack-infra | 16:03 | |
fungi | ctlaugh: right, as soon as a job completes, if you look at the status page for the slave that ran the job you should see it marked offline with a comment stating that it was due to a gearman request | 16:03 |
sslypushenko | hogepodge I think that Refstack should work like gerrit in terms on auth | 16:03 |
fungi | ctlaugh: and you should also be able to see that parameter in the list of parameters for a running/completed job | 16:03 |
sslypushenko | smarcet hogepodge Where I should put this workflow diagram? Link on google drive will be fine? | 16:04 |
ctlaugh | fungi: Yes, I've seen them go offline and that param is there. I need to keep looking... Thanks! | 16:04 |
*** zzz_pczesno has quit IRC | 16:05 | |
fungi | sslypushenko: eventually i hope you'll include any design diagrams in some public documentation for the service | 16:05 |
smarcet | sslypushenko: its ok with me | 16:05 |
hogepodge | sslypushenko: that should be fine, doc to refstack for design review is good too | 16:05 |
*** thedodd has quit IRC | 16:07 | |
*** Somay has quit IRC | 16:08 | |
*** Bsony has quit IRC | 16:08 | |
sslypushenko | fungi smarcet hogepodge kk | 16:08 |
clarkb | zaro: fwiw I have mostly narrowed the jenkins thread leak issue down. https://issues.jenkins-ci.org/browse/JENKINS-27514 has the latest. Any ideas on why that method is synchronized? | 16:08 |
openstackgerrit | Merged openstack/requirements: Bump cryptography version to latest https://review.openstack.org/164289 | 16:09 |
*** gyee has joined #openstack-infra | 16:09 | |
*** MarkAtwood has quit IRC | 16:10 | |
*** jcoufal has quit IRC | 16:11 | |
*** Liu has quit IRC | 16:13 | |
*** weshay has joined #openstack-infra | 16:14 | |
openstackgerrit | Merged openstack-infra/project-config: Disable -dibtest jobs https://review.openstack.org/166302 | 16:16 |
*** jcoufal has joined #openstack-infra | 16:17 | |
*** bdpayne has joined #openstack-infra | 16:18 | |
*** yamahata has quit IRC | 16:19 | |
*** Hal1 has quit IRC | 16:19 | |
*** coolsvap has quit IRC | 16:19 | |
*** yamahata has joined #openstack-infra | 16:19 | |
*** gyee has quit IRC | 16:21 | |
fungi | clarkb: i'm starting to look into NobodyCam and jroll's etherpad problem... in the past when we've seen a pad get stuck and start showing the same content for all historical revisions, we've been able to export an earlier version via the admin api right? | 16:22 |
fungi | in this case padID=IronicWhiteBoard is showing 29056 revisions | 16:22 |
clarkb | ya, if you find the version you want you can get all its content via the api then paste it back in | 16:23 |
*** jamesmcarthur has quit IRC | 16:23 | |
*** gyee has joined #openstack-infra | 16:23 | |
clarkb | fungi: the only time I did that I did a binary search using the api to find roughly where the content was good | 16:24 |
fungi | though i think they're hoping to restore all the history because they refer back to starred change points | 16:24 |
clarkb | I hae not do that | 16:24 |
clarkb | you could probably script it though by replaying old versions back again | 16:25 |
*** otter768 has joined #openstack-infra | 16:25 | |
fungi | did we upgrade our production etherpad version yet? wondering if this might be related (very old pad with data in a format unsupported by current release?) | 16:26 |
fungi | they've basically been updating this pad continuously since ironic came into being, sounds like | 16:26 |
clarkb | we have not | 16:26 |
clarkb | at least I haven't | 16:26 |
fungi | okay, so it spontaneously broke. that helps narrow this down at least | 16:26 |
clarkb | fungi: note that previous occurrences of this issue were basically a client defacing the pad | 16:26 |
clarkb | fungi: this is one thing about public rw etherpads, any client can explode them | 16:27 |
fungi | yeah. wondering if i might be able to manually roll back recent changes | 16:27 |
fungi | i may have to save this for after lunch, but that gives me a few things to try. thanks! | 16:27 |
asselin | anteaya, I did notice that the version of urllib3 got downgraded somehow: urllib3==1.7.1 vs urllib3==1.10.2 | 16:27 |
*** hodos has joined #openstack-infra | 16:28 | |
boris-42 | fungi: hi there | 16:28 |
boris-42 | fungi: is there any way to remove project from stackforge? | 16:28 |
clarkb | the regex at https://git.openstack.org/cgit/openstack-infra/os-loganalyze/tree/os_loganalyze/filter.py#n21 is matching -trove jobs because \b doesn't treat - as a word character | 16:28 |
anteaya | asselin: okay, that is interesting | 16:29 |
clarkb | my regex foo is failing at finding a good way to updating that regex to fix | 16:29 |
clarkb | boris-42: you can move it to the attic but we don't delete it completely | 16:29 |
*** armax has quit IRC | 16:29 | |
ctlaugh | one question I've been meaning to ask... when I see a Jenkins job that says "aborted", what does that typically mean? Is this something that Jenkins is doing itself, or is it something that zuul is telling it to do? | 16:29 |
asselin | anteaya, yeah I remember an issue with proxies using an older version of urllib3. have a meeting now, but will look into that later. | 16:29 |
boris-42 | clarkb: I mean I just would like to remove useless project (house keeping) | 16:29 |
clarkb | ctlaugh: usually zuul is telling it to stop running the job because the job is no longer valid or necessary | 16:30 |
boris-42 | clarkb: does it make sense? | 16:30 |
fungi | boris-42: basically we move it into a namespace which makes it obvious the project is retired and then set it read-only so that nobody can propose new changes to it in gerrit | 16:30 |
*** otter768 has quit IRC | 16:30 | |
fungi | boris-42: gerrit doesn't have a "delete project" feature | 16:30 |
anteaya | asselin: okay thanks, yes urllib3 is also giving insecureplatform warnings on gertty, so the fix was to disable | 16:30 |
fungi | boris-42: optionally, before we retire the project, it's good to commit a final change deleting all files and replacing them with a readme saying that the project is no longer maintained and why | 16:31 |
*** armax has joined #openstack-infra | 16:31 | |
*** dboik_ has joined #openstack-infra | 16:31 | |
*** baoli has quit IRC | 16:31 | |
*** Ala has quit IRC | 16:31 | |
*** baoli has joined #openstack-infra | 16:32 | |
clarkb | hpcloud seems to be in a happier place now. About 110 used nodes and twice that either building or deleting | 16:34 |
*** harlowja_at_home has quit IRC | 16:34 | |
*** YorikSar has joined #openstack-infra | 16:34 | |
*** dboik has quit IRC | 16:35 | |
mordred | clarkb: yay! | 16:35 |
*** tsg_ has joined #openstack-infra | 16:35 | |
*** peristeri has quit IRC | 16:35 | |
*** marun has joined #openstack-infra | 16:35 | |
mordred | clarkb: I have started poking people internally about getting some patches applied related to the delete performance | 16:35 |
boris-42 | fungi: ok | 16:35 |
nibalizer | mordred: yay | 16:36 |
nibalizer | anteaya: i just rol9led requests back to 2.5.0 | 16:36 |
anteaya | asselin: sorry not disable, downgrade | 16:36 |
nibalizer | er rolled, dunno where that 9 came from | 16:36 |
anteaya | 9's show up in the oddest places | 16:36 |
clarkb | mordred are these upstream patches we can help with at all? | 16:36 |
*** dkranz has quit IRC | 16:37 | |
*** sergsh has quit IRC | 16:37 | |
*** bdpayne has quit IRC | 16:37 | |
anteaya | nibalizer: rolled requests back to 2.5.0, are we discussing urllib3 or something else? | 16:37 |
mordred | clarkb: no - there is a db delete performance related patch upstream already - but it involves doing a schema migration and adding an index - which will take testing and planning | 16:37 |
fungi | boris-42: anyway, if you want that done, just propose it like any project rename and we batch it up with other pending renames during a maintenance window (since it requires gerrit downtime) | 16:37 |
*** hdd has joined #openstack-infra | 16:38 | |
mordred | clarkb: beause, let's face it - doing a schema transformation on a live large openstack isn't exaclty people's favorite thing to do | 16:38 |
clarkb | mordred we can create another outage window if it helps | 16:39 |
openstackgerrit | Mateusz Matuszkowiak proposed openstack-infra/project-config: Added fuel-plugin-cisco-aci fuel plugin to SF https://review.openstack.org/166872 | 16:39 |
*** Hal1 has joined #openstack-infra | 16:39 | |
nibalizer | anteaya: i think so, requests requires urllib which i think is the problem | 16:39 |
openstackgerrit | Darragh Bailey proposed openstack-infra/jenkins-job-builder: Treat non-existant output files as empty files https://review.openstack.org/166062 | 16:40 |
anteaya | nibalizer: okay my conversation with asselin is a continuation of a conversation that started during the third-party meeting | 16:40 |
*** unicell has joined #openstack-infra | 16:40 | |
anteaya | nibalizer: he is experiencing errors in his devstack set up starting from 3am this morning | 16:41 |
*** yamahata has quit IRC | 16:41 | |
boris-42 | fungi: ok thank you | 16:41 |
mordred | clarkb: hahaha | 16:41 |
*** yamahata has joined #openstack-infra | 16:41 | |
anteaya | nibalizer: http://15.126.198.151/97/164697/6/check/3par-fc-driver-master-client-pip-eos10-dsvm/5af4187/logs/devstacklog.txt.gz#_2015-03-23_10_43_00_435 | 16:41 |
fungi | hasharAway: pabelanger: just saw http://bugs.debian.org/781027 cross the devian-devel ml. awesome! | 16:41 |
openstack | Debian bug 781027 in wnpp "ITP: nodepool -- nodepool a system to launch disposable OpenStack VM for testing purposes" [Wishlist,Open] | 16:41 |
mordred | hasharAway, pabelanger: NEAT! | 16:42 |
anteaya | nibalizer: so far he is the only third party op seeing this issue, so we don't know if it is just his setup or not | 16:42 |
hasharAway | fungi: yeah I am giving it a try. Will probably reach out to zigo about :) | 16:42 |
*** ajo has quit IRC | 16:42 | |
nibalizer | oh i thought we were talking about gertty. my bad | 16:42 |
*** hasharAway is now known as hashar | 16:42 | |
anteaya | nibalizer: ah okay, I just mentioned gertty in the context of asselin's urllib3 comment | 16:44 |
anteaya | nibalizer: may or may not be related | 16:44 |
anteaya | nibalizer: but thanks :) | 16:45 |
*** reed has quit IRC | 16:45 | |
*** nelsnelson has quit IRC | 16:45 | |
*** reed has joined #openstack-infra | 16:46 | |
*** bhunter71 has quit IRC | 16:47 | |
* fungi disappears to find food... bbiab | 16:48 | |
zaro | clarkb: taking a look | 16:48 |
asselin | anteaya, ok i'm back. the requests packages is the same version b/w working and non-working. | 16:48 |
*** emagana has quit IRC | 16:49 | |
anteaya | asselin: hmmmm | 16:49 |
anteaya | so you have a working version still? | 16:49 |
asselin | anteaya, past log files such as yesterday's :) | 16:49 |
anteaya | ah | 16:50 |
*** dkranz has joined #openstack-infra | 16:50 | |
*** sushilkm has joined #openstack-infra | 16:50 | |
*** sushilkm has left #openstack-infra | 16:50 | |
*** ihrachyshka has quit IRC | 16:51 | |
*** vladiskuz_ has quit IRC | 16:53 | |
*** yamahata has quit IRC | 16:54 | |
*** yamahata has joined #openstack-infra | 16:55 | |
*** tqtran has joined #openstack-infra | 16:55 | |
pleia2 | so much email | 16:57 |
*** emagana has joined #openstack-infra | 16:57 | |
*** e0ne is now known as e0ne_ | 16:57 | |
*** e0ne_ is now known as e0ne | 16:58 | |
notmyname | jeblair: clarkb: last June, Swift merged a feature branch. we're about to do the same thing again, and I'd like to do it the same way as last time: land one merge commit for the patch chain rather than each individual patch in the chain as a ff merge | 16:59 |
*** annegentle has quit IRC | 16:59 | |
*** emagana has quit IRC | 16:59 | |
jeblair | notmyname: are all the commits on the feature branch merged? | 16:59 |
notmyname | jeblair: that's what we're finishing this week | 16:59 |
jeblair | notmyname: so when that's done, you should be able to create a merge commit that merges feature/foo into master and propose that | 17:00 |
*** bdpayne has joined #openstack-infra | 17:00 | |
jeblair | notmyname: that single merge commit is reviewable in gerrit | 17:00 |
notmyname | right, but IIRC there was an issue with doing it at a merge commit | 17:00 |
notmyname | if it's set up to do already, then that part is easy | 17:01 |
*** emagana has joined #openstack-infra | 17:01 | |
*** sarob has joined #openstack-infra | 17:01 | |
*** sarob has quit IRC | 17:01 | |
notmyname | I know it works as master->feature/foo | 17:01 |
*** sarob has joined #openstack-infra | 17:01 | |
jeblair | notmyname: my fuzzy memory of last time is that there was an issue with the contents of the feature branch not actually being what you wanted merged into master or something... | 17:01 |
*** soren has joined #openstack-infra | 17:01 | |
*** spredzy is now known as spredzy|afk | 17:01 | |
notmyname | jeblair: that's part 2 of what I wanted to say :-) | 17:01 |
*** scheuran has quit IRC | 17:02 | |
jeblair | notmyname: but regardless, if feature/foo is what you want, then feature/foo -> master should be straightforward | 17:02 |
jeblair | notmyname: http://docs.openstack.org/infra/manual/drivers.html#feature-branches | 17:02 |
jeblair | notmyname: now with docs! :) | 17:02 |
notmyname | ok, great | 17:02 |
notmyname | yay | 17:02 |
jeblair | http://docs.openstack.org/infra/manual/drivers.html#merge-feature-branch-into-master | 17:02 |
jeblair | notmyname: let me know if that's wrong ;) | 17:02 |
*** bhunter71 has joined #openstack-infra | 17:02 | |
notmyname | jeblair: clarkb: ok, the other part is that we want to refactor the current feature branch to clean it up for merge to master. so we can either create a new feature branch (eg feature/ec_merge) or we can do what we did last time | 17:02 |
notmyname | jeblair: last time we tagged the current feature/foo dev work and recreated the feature/foo branch with the cleaned up commits | 17:03 |
*** ddieterly has quit IRC | 17:03 | |
jeblair | notmyname: what is it that you want to 'clean up' ? | 17:03 |
notmyname | either way, there's a little feature branch wrangling to do | 17:03 |
pabelanger | fungi, hashar, mordred: Nice. I have some packaging some place, but cool to get it merged into debian. I'll likely start up on RHEL stuff shortly | 17:03 |
*** ddieterly has joined #openstack-infra | 17:04 | |
*** yamahata has quit IRC | 17:04 | |
notmyname | jeblair: the current feature branch is nearly a year's worth of ideas and rewrites and etc. it's all over the map. we want to clean it up into the logical pieces for 2 reasons. immediately, it helps with reviewing it for merge. later, it helps with understanding what comes in with the feature branch merge | 17:04 |
pabelanger | FWIW: I'd be interested in some sort of generic build environments hooked into -infra. I know talking to fungi before, snapshots could be leveraged | 17:04 |
*** davide has joined #openstack-infra | 17:04 | |
*** jlanoux has quit IRC | 17:04 | |
notmyname | jeblair: same as last time, actually (with storage policies) | 17:05 |
pabelanger | I tend to use pbuilder, but know others prefer cowbuilder environments | 17:05 |
hashar | pabelanger: fungi: mordred: I will poke the list eventually when I have some debian packaging progress to share :) | 17:06 |
notmyname | jeblair: mostly I wanted to let you know what's coming (that I'm going to be asking for this next week) instead of next week saying "I need this yesterday!!eleventy!!111!!" | 17:06 |
jeblair | notmyname: i feel like i'm missing something in this process -- i would expect you to get feature/foo into the shape you want it to be in, then merge it into master | 17:07 |
*** dan has quit IRC | 17:07 | |
pabelanger | hashar, I assume you are using git-buildpackage. What are you using for chroots? | 17:07 |
notmyname | jeblair: yes. at a high level, that's the plan | 17:07 |
hashar | pabelanger: cowbuilder :) | 17:07 |
jeblair | notmyname: i thought last time we talked about how it was not done correctly and we would avoid the same thing again | 17:07 |
pabelanger | hashar, any specific reason for cowbuilder? | 17:07 |
hashar | pabelanger: I have crafted a cow image for each of the distribution we are using (and sid of course). Make it easy to build. | 17:07 |
hashar | pabelanger: it is fast? :) | 17:07 |
pabelanger | hashar, unsure about speed, never compared. | 17:08 |
hashar | pabelanger: I have started a very basic doc at https://wikitech.wikimedia.org/wiki/Cowbuilder | 17:08 |
mordred | pabelanger, hashar: I actually had some thoughts about how to incorporate this into infra tooling/workflow earlier this morning | 17:08 |
mordred | my next step is to write those up for comment | 17:08 |
*** ghostpl_ has quit IRC | 17:08 | |
mordred | so I'll send the to you - and between us I betcha we can solve the world :) | 17:09 |
hashar | pabelanger: we even have some puppet wrapper to setup the initial images https://gerrit.wikimedia.org/r/#/c/194471/ | 17:09 |
pabelanger | hashar, When talking to fungi we envisioned bootstrapping a snapshot for nodepool building all the cowbuilder /pbuilder chroots. Then, when the new patchset was fired, git-buildpackage just did an update (vs create). To speed up the processes | 17:09 |
*** annegentle has joined #openstack-infra | 17:10 | |
hashar | mordred: you could surely have Jenkins jobs wrapping around http://jenkins-debian-glue.org/ . A set of shell script that makes it trivial to integration debian packaging in jenkins. | 17:10 |
greghaynes | mordred: so re vhd-util PPA, that link doesnt seem to work... | 17:10 |
*** sushilkm has joined #openstack-infra | 17:10 | |
notmyname | jeblair: the code as it stands now on the feature branch, isn't good for integration into master. there are false starts and undos, elements split across multiple commits, and no logical grouping of what's been going on. it's been a working area for a long time as we've been figuring out this complicated feature. in order to integrate it into master we want a succinct set of commits like "this one deals with refactoring X" and "this o | 17:10 |
notmyname | ne implements new daemon Y" | 17:10 |
pabelanger | hashar, Ya, a few years a go I did: https://github.com/kickstandproject/kickstandproject-pbuilder/ | 17:10 |
*** sushilkm has left #openstack-infra | 17:11 | |
*** vigneshvar has joined #openstack-infra | 17:11 | |
pabelanger | worked well at the time | 17:11 |
mordred | greghaynes: sorry - made a new one | 17:11 |
hashar | pabelanger: yeah that is exactly what http://jenkins-debian-glue.org/ does. It would init some cow images using cowbuilder, then you can trigger a new build that would link that image which is all available already | 17:11 |
mordred | greghaynes: replace vhd-util with infra | 17:11 |
pabelanger | but, been some time since I played with it | 17:11 |
mordred | hashar: I'm way less concerned about that part | 17:11 |
mordred | hashar: as I am in the human workflow around proposing patches to gerrit | 17:11 |
mordred | hashar: and the asumptions gbp makes about how your git environment works | 17:11 |
notmyname | jeblair: so refactoring what is currently on feature/ec into those commits is what we want to do. but to do that, we need a place to do it (either a new branch or recreating the existing one after tagging the current for posterity) | 17:11 |
hashar | mordred: well that is "just" yet another convention to stick to :) | 17:12 |
mordred | :) | 17:12 |
pabelanger | mordred, Ya, some spec or discussion would be great. I see a lot of potential for packagers using -infra. | 17:12 |
jeblair | notmyname: my opinion is that you should merge it as-is. re-writing the history will screw up authorship information and lose the context of why things were actually added. however.... | 17:12 |
*** dan has joined #openstack-infra | 17:12 | |
hashar | mordred: the gbp defaults are sane enough though and can always be overridden per repo as may fit. | 17:12 |
mordred | hashar: yup. I just think it needs to be walked through and written up | 17:13 |
hashar | mordred: yup. My aim is to get the packages to auto build from wikimedia Gerrit. I will surely share my result :) | 17:14 |
jeblair | notmyname: if you just want to make a series of commits out of it, you could create patch series against master and merge it normally. or i suppos if it's still a large effort with many commits, you could create a new feature branch. | 17:14 |
notmyname | jeblair: right. the new patch series is what I'm talking about, except with the property of being merged as a single merge commit (which looks simple enough to do with existing tooling) | 17:15 |
pabelanger | hashar, don't think the workflow for auto build will be too hard, obvious some design stuff to figure out. Am curious the process how to serve up said packages. EG: static repo list or per patchset build. That would be fun to manage | 17:15 |
pabelanger | I was using reprepro for my stuff, but fungi had some other thoughts | 17:15 |
lxsli | gate-nova-docs said "TypeError: init_values() takes exactly 2 arguments (1 given)" - is this a known issue please? | 17:15 |
jeblair | notmyname: yeah, i want to have a conversation with you at some point, not necessarily now, about why you think your existing commits are somehow unfit for being in master :) | 17:16 |
pabelanger | lxsli, review URL? | 17:16 |
lxsli | pabelanger: https://review.openstack.org/#/c/138686 | 17:16 |
notmyname | jeblair: sounds great (after I'm out of this EC craziness!) | 17:16 |
*** sarob_ has joined #openstack-infra | 17:18 | |
*** amotoki has quit IRC | 17:19 | |
*** jamesmcarthur has joined #openstack-infra | 17:19 | |
pabelanger | lxsli, have you checked pbr project for open issues? | 17:21 |
*** sputnik13 has joined #openstack-infra | 17:21 | |
*** melwitt has joined #openstack-infra | 17:22 | |
*** MrAboii has quit IRC | 17:22 | |
lxsli | Quick search shows: https://bugs.launchpad.net/oslotest/+bug/1379998 | 17:23 |
openstack | Launchpad bug 1379998 in PBR "pbr incompatible with sphinx 1.3b1" [Medium,Triaged] | 17:23 |
*** derekh has quit IRC | 17:23 | |
*** patrickeast has joined #openstack-infra | 17:24 | |
lxsli | It's from Oct 2014 though and I can't see what versions of pbr + sphinx the gate is using | 17:24 |
*** dannywilson has joined #openstack-infra | 17:24 | |
*** dannywilson has quit IRC | 17:24 | |
hashar | pabelanger: I am pretty sure jenkins-debian-glue comes with support to add packages in a reprepro repository. Haven't played much with it though. | 17:25 |
*** dannywilson has joined #openstack-infra | 17:25 | |
hashar | pabelanger: I am off for dinner etc. Will be back in roughly 3 hours | 17:25 |
anteaya | zaro: so when I look here: https://review.openstack.org/#/q/status:open+topic:gerrit-upgrade,n,z I don't see the close connections cherry pick patch | 17:27 |
*** sarob_ has quit IRC | 17:27 | |
anteaya | zaro: do you think it should have the gerrit-upgrade topic? I would like to see it included | 17:27 |
*** harlowja has joined #openstack-infra | 17:27 | |
*** rlandy is now known as rlandy|afk | 17:27 | |
*** bdpayne has quit IRC | 17:28 | |
*** asselin has quit IRC | 17:28 | |
*** achanda has joined #openstack-infra | 17:29 | |
*** asselin has joined #openstack-infra | 17:29 | |
*** hdd has quit IRC | 17:29 | |
*** carl_baldwin has quit IRC | 17:30 | |
*** spzala has joined #openstack-infra | 17:33 | |
*** hodos has quit IRC | 17:34 | |
*** hodos has joined #openstack-infra | 17:34 | |
*** e0ne has quit IRC | 17:36 | |
*** amotoki has joined #openstack-infra | 17:36 | |
*** amotoki has quit IRC | 17:36 | |
*** annegentle has quit IRC | 17:37 | |
*** carl_baldwin has joined #openstack-infra | 17:38 | |
*** HeOS has quit IRC | 17:38 | |
*** dkliban is now known as dkliban_brb | 17:39 | |
anteaya | nibalizer: I'm going through infra priority patches, do you want to see if you can make jenkins happy here? https://review.openstack.org/#/c/162830/ | 17:39 |
*** jamesmcarthur has quit IRC | 17:42 | |
nibalizer | anteaya: ya will do | 17:43 |
*** carl_baldwin has quit IRC | 17:44 | |
openstackgerrit | Boris Pavlovic proposed openstack-infra/project-config: Add new project nerd-reviewer https://review.openstack.org/166945 | 17:44 |
anteaya | nibalizer: thanks | 17:44 |
*** ivar-lazzaro has joined #openstack-infra | 17:45 | |
*** ivar-lazzaro has quit IRC | 17:45 | |
mtreinish | jeblair, clarkb, mordred: if you get a sec https://review.openstack.org/#/c/165183/ should be quick (it's a 1 liner) | 17:45 |
*** garyh has quit IRC | 17:46 | |
greghaynes | oh wow, is there not a way to tell puppet to use bash for execs | 17:48 |
*** ivar-lazzaro has joined #openstack-infra | 17:49 | |
*** yamahata has joined #openstack-infra | 17:49 | |
mtreinish | greghaynes: tbh, I didn't really look. But when that didn't work I just pushed the easy fix which didn't require me looking anything up | 17:50 |
pleia2 | greghaynes: I think what we've just done is 'bash foo.sh' as the command | 17:51 |
*** garyh has joined #openstack-infra | 17:51 | |
greghaynes | yea yea, I think its fine to fix that I just know we assume bash in all our DIB stuff and other places to that seems like a pattern thats likely to repeat | 17:51 |
zaro | anteaya: i guess the only reason why it doesn't have that topic is because it's not required when we upgrade. it can be done after. | 17:52 |
greghaynes | pleia2: ah | 17:52 |
mtreinish | pleia2: well it this case it would be something like bash -c 'subunit2sql-db-manage --config-file /etc/subunit2sql.conf upgrade head &> /var/log/subunit2sql_migration.log &' :) | 17:52 |
pleia2 | mtreinish: you have too much fun ;) | 17:52 |
anteaya | zaro: oh okay, thanks well as long as we can get the functionality in sometime, if we did it afterward the patch would merge and then what? just a gerrit restart to pick it up? | 17:53 |
mtreinish | all to use '&>' | 17:53 |
openstackgerrit | Spencer Krum proposed openstack-infra/system-config: Refactor o_p::base into o_p::template https://review.openstack.org/162830 | 17:53 |
openstackgerrit | yolanda.robla proposed openstack-infra/system-config: Don't hardcode pip.conf values https://review.openstack.org/166252 | 17:54 |
*** dannywilson has quit IRC | 17:54 | |
anteaya | mtreinish: ah so that was the motivation to work &> in there | 17:54 |
mtreinish | pleia2: oh I forgot to ping you above, I'll remember to bug you for reviews eventually :) | 17:54 |
*** armax has quit IRC | 17:54 | |
*** jistr has quit IRC | 17:54 | |
* anteaya hands pleia2 her camo coat | 17:54 | |
pleia2 | mtreinish: no rush! | 17:55 |
zaro | anteaya: the process would be, change merges, new gerrit build gets created, new change to update to the new gerrit build, change to update gerrit merges, then func is available. | 17:55 |
pleia2 | hehe | 17:55 |
anteaya | zaro: how long on average does that process take? | 17:55 |
zaro | anteaya: ohh, i think restart would also be required on gerrit updates | 17:55 |
anteaya | thought so | 17:56 |
mtreinish | anteaya: well https://review.openstack.org/#/c/165183/1 I did the above without the bash -c before and it didn't work (it redirected nothing, which makes debug fun :) ) | 17:56 |
zaro | anteaya: depends on review queue? | 17:56 |
*** jamesmcarthur has joined #openstack-infra | 17:56 | |
*** hashar is now known as hasharDinner | 17:56 | |
*** tonytan4ever has joined #openstack-infra | 17:56 | |
anteaya | zaro: fair enough, your call | 17:56 |
*** sushilkm has joined #openstack-infra | 17:57 | |
*** sushilkm has left #openstack-infra | 17:57 | |
*** achanda has quit IRC | 17:57 | |
*** _nadya_ has quit IRC | 17:57 | |
zaro | anteaya: if you feel strongly that it should be included with gerit 2.9 upgrade then you can probably add the topic and explain in comments? | 17:58 |
anteaya | mtreinish: okay thanks | 17:58 |
zaro | clarkb: take a look https://github.com/jenkinsci/jenkins/blob/master/core/src/main/java/hudson/slaves/ComputerLauncher.java#L107 | 17:58 |
anteaya | zaro: can I change the topic if I'm not a core reviewer or the patch owner? | 17:58 |
zaro | anteaya: yes | 17:59 |
anteaya | okay thanks I will try | 17:59 |
anteaya | zaro: next question, this patch merged: https://review.openstack.org/#/c/164815/ what will it take for a 2.9 tarball to appear on tarballs.o.o? | 18:00 |
anteaya | this patch needs it https://review.openstack.org/#/c/155463/ | 18:00 |
anteaya | zaro: I do not have the ability to change the topic in the gui on https://review.openstack.org/#/c/166025/ | 18:01 |
anteaya | I see no notepad icon | 18:01 |
*** vladiskuz_ has joined #openstack-infra | 18:02 | |
zaro | anteaya: ohh that's interesting. i think that might be a bug. | 18:02 |
anteaya | okay | 18:02 |
anteaya | zaro: let me know what you need from me next | 18:02 |
*** stevemar has quit IRC | 18:02 | |
zaro | anteaya: you can push the change with a new topic, just change the commit message slightly. | 18:02 |
zaro | *the/the same | 18:03 |
openstackgerrit | Elizabeth K. Joseph proposed openstack-infra/system-config: Update review.o.o IP addresses in Zuul ssh config https://review.openstack.org/166948 | 18:03 |
*** sandywalsh_ has joined #openstack-infra | 18:03 | |
pleia2 | amusingly I use zuul_prod.pp as an example file for a talk I was prepping for this morning, so look what I found ^^ | 18:03 |
*** AJaeger_ has joined #openstack-infra | 18:03 | |
*** stevemar has joined #openstack-infra | 18:03 | |
zaro | anteaya: so i'm not sure why there's a restriction on allowing any registered user to change the topic | 18:03 |
*** armax has joined #openstack-infra | 18:04 | |
jpeeler | how can a review fail pep8 checks on files that aren't part of the commit? https://review.openstack.org/#/c/165150/ | 18:04 |
*** Somay has joined #openstack-infra | 18:04 | |
anteaya | pleia2: well done | 18:04 |
anteaya | zaro: I'm pretty sure it was limited to patch owner and core reviewers on that repo | 18:04 |
zaro | anteaya: Gerrit tarball won't get built until there is a change to the openstack-infra/gerrit repo | 18:05 |
zaro | anteaya: neither of those change that repo therefore no gerrit tarball | 18:05 |
*** sandywalsh has quit IRC | 18:06 | |
*** marun has quit IRC | 18:06 | |
*** vladiskuz_ has quit IRC | 18:06 | |
zaro | anteaya: one of thesse needs to be approved for tarball to get created | 18:06 |
*** sarob has quit IRC | 18:06 | |
zaro | https://review.openstack.org/#/q/project:openstack-infra/gerrit+status:open,n,z | 18:06 |
hodos | hi did any1 have this problem: BadRequest: Invalid key_name provided. with the nodepool trying to update the image | 18:07 |
*** jamesmcarthur has left #openstack-infra | 18:08 | |
*** ChuckC has quit IRC | 18:08 | |
anteaya | zaro: okay thanks | 18:08 |
*** dan has quit IRC | 18:08 | |
*** garyh has quit IRC | 18:09 | |
anteaya | clarkb: when you are about, I have a question about https://review.openstack.org/#/c/166596/1 how can we test this? | 18:09 |
*** hdd has joined #openstack-infra | 18:09 | |
zaro | anteaya: no, it doesn't look like an ACL setting. it's hwo gerrit works by default. might be nice to lift that restriction. | 18:09 |
anteaya | clarkb: we have a fair few jobs using swift for logging now, when it was just infra I was fine testing it in production, now I'm hesitant | 18:09 |
*** ildikov has quit IRC | 18:10 | |
anteaya | clarkb: and I don't know how to test it in non-production | 18:10 |
anteaya | zaro: well perhaps I am mis-remembering then, perhaps there is no restriction, I don't know | 18:10 |
openstackgerrit | melanie witt proposed openstack-infra/project-config: Adjust regression exceptions for Nova Cells V1 job https://review.openstack.org/166396 | 18:11 |
anteaya | zaro: my expectation however is that as a registered user I can't go around and randomly change commit messages or topics, but perhaps I can | 18:11 |
*** emagana has quit IRC | 18:11 | |
*** dustins has quit IRC | 18:12 | |
zaro | clarkb: i don't know why that afterDisconnect method is syncroized. it seems like the only reason might be for use cases when the same slave is connected/disconnected | 18:12 |
zaro | clarkb: but that would not affect us because our slaves are always uniquely named correct? | 18:13 |
* anteaya is going to take 20 minutes to row | 18:13 | |
*** emagana has joined #openstack-infra | 18:13 | |
zaro | clarkb: also it seems like the syncronization should be on the slaveComputer and not the entire method? | 18:13 |
*** nelsnelson has joined #openstack-infra | 18:13 | |
*** nelsnelson has quit IRC | 18:13 | |
*** ghostpl_ has joined #openstack-infra | 18:14 | |
*** dank_ has joined #openstack-infra | 18:14 | |
clarkb | anteaya using infra was our test | 18:14 |
zaro | anteaya: hmm, how about the ability for any registered user to upload another version of the owner's patch? wouldn't that be worse than just chaning the topic? | 18:14 |
*** nelsnelson has joined #openstack-infra | 18:14 | |
*** ociuhandu has quit IRC | 18:15 | |
clarkb | zaro I think we can remove the synchronized on the method and add a lock around the session and connwction | 18:15 |
*** achanda has joined #openstack-infra | 18:15 | |
clarkb | so that its only locked per instance not every instance sharing a lock | 18:15 |
*** dannywilson has joined #openstack-infra | 18:15 | |
*** dtantsur is now known as dtantsur|afk | 18:15 | |
*** dizquierdo has quit IRC | 18:16 | |
zaro | clarkb: that makes sense and worth a try. | 18:16 |
*** flashgordon is now known as jogo | 18:16 | |
*** enikanorov__ has joined #openstack-infra | 18:16 | |
*** mpavone has quit IRC | 18:17 | |
*** ominakov has quit IRC | 18:17 | |
*** ociuhandu has joined #openstack-infra | 18:17 | |
*** marun has joined #openstack-infra | 18:17 | |
*** emagana has quit IRC | 18:18 | |
*** harlowja has quit IRC | 18:18 | |
fungi | hasharDinner: pabelanger: just got back, skimmed scrollback, i think we could update a pbuilder chroot when building bare-jessie, or just not bother with pbuilder since we'd be starting from very minimally-configured virtual machines anyway | 18:18 |
*** enikanorov has quit IRC | 18:19 | |
fungi | single-use workers make a lot of the argument for pbuilder go away | 18:19 |
*** achanda has quit IRC | 18:19 | |
*** achanda has joined #openstack-infra | 18:20 | |
pabelanger | fungi, Right, my only concern is people downstream trying to consume scripts, which don't have single-use workers. But, aside from that, I think that approach works | 18:20 |
fungi | pabelanger: agreed. pbuilder is mostly doing what nodepool already does for us anyway | 18:20 |
*** ssam2 has quit IRC | 18:20 | |
pabelanger | yar | 18:20 |
fungi | when we last discussed this (a year or more ago), nodepool wasn't nearly so ubiquitous in our environment | 18:21 |
*** harlowja has joined #openstack-infra | 18:21 | |
*** Somay has quit IRC | 18:22 | |
*** peristeri has joined #openstack-infra | 18:23 | |
pabelanger | Ya, really looking forward to what can be leveraged by nodepool. Lots of potential for packagers | 18:23 |
*** otter768 has joined #openstack-infra | 18:26 | |
*** samueldmq has joined #openstack-infra | 18:27 | |
*** achanda has quit IRC | 18:27 | |
*** Somay has joined #openstack-infra | 18:27 | |
*** achanda has joined #openstack-infra | 18:28 | |
*** emagana has joined #openstack-infra | 18:28 | |
*** thedodd has joined #openstack-infra | 18:29 | |
*** woodster_ has quit IRC | 18:30 | |
*** otter768 has quit IRC | 18:31 | |
greghaynes | mordred: do you have a link to the vhd-util patch you used? I want to add it to DIB docs | 18:31 |
openstackgerrit | Matthew Treinish proposed openstack-infra/system-config: Add docs for subunit2sql https://review.openstack.org/164397 | 18:32 |
mtreinish | anteaya: ^^^ if you get a sec, I'd appreciate your feedback on that | 18:33 |
openstackgerrit | Spencer Krum proposed openstack-infra/system-config: Refactor o_p::base into o_p::template https://review.openstack.org/162830 | 18:34 |
openstackgerrit | Sean Dague proposed openstack-infra/os-loganalyze: fix path matching redux https://review.openstack.org/166960 | 18:34 |
*** dannywilson has quit IRC | 18:35 | |
*** dannywilson has joined #openstack-infra | 18:35 | |
openstackgerrit | Boris Pavlovic proposed openstack-infra/project-config: Add rally-ci project https://review.openstack.org/166962 | 18:35 |
*** garyh has joined #openstack-infra | 18:36 | |
anteaya | clarkb: but python-jobs are now using swift for logs too, so that affects more than just infra | 18:36 |
anteaya | clarkb: were we going to implement the footer code on just infra swift logs jobs? | 18:36 |
samueldmq | mtreinish, there is a trailing whitespace on your change, please remove it :) | 18:37 |
anteaya | zaro: you have a point, it just felt weird to me | 18:37 |
*** ParsectiX has joined #openstack-infra | 18:37 | |
samueldmq | mtreinish, I wont -1/+1 since I am not familiar with that doc | 18:37 |
anteaya | mtreinish: we are acknowledging that I am not a core reviewer on system-config, yes? | 18:37 |
anteaya | mtreinish: I'll review but I don't want you to have expectations that I can't meet | 18:38 |
mtreinish | anteaya: yes, I know :) but, your feedback was helpful on my last infra doc | 18:38 |
openstackgerrit | Valeriy Ponomaryov proposed openstack/requirements: Bump ddt to min version 0.7.0 https://review.openstack.org/166162 | 18:39 |
anteaya | mtreinish: okay, I'll take my +1 and dive in | 18:39 |
reed | fungi, I received the list of ATCs, thanks, proceeding to send invites | 18:39 |
*** ParsectiX has quit IRC | 18:39 | |
mtreinish | samueldmq: yeah, I'll fix that on the inevitable next revision | 18:39 |
samueldmq | mtreinish, k :) | 18:39 |
openstackgerrit | Valeriy Ponomaryov proposed openstack-infra/project-config: Add tempest + postgres job for manila https://review.openstack.org/165935 | 18:39 |
*** soren has quit IRC | 18:39 | |
*** ParsectiX has joined #openstack-infra | 18:40 | |
*** ParsectiX has quit IRC | 18:40 | |
*** dboik_ has quit IRC | 18:40 | |
clarkb | anteaya right | 18:41 |
clarkb | the footer is only for dsvm jobs and we only swift logs on d-g jobs | 18:41 |
anteaya | clarkb: ah | 18:42 |
anteaya | I somehow missed that | 18:42 |
* anteaya checks the commit message again | 18:42 | |
*** peristeri has quit IRC | 18:42 | |
fungi | reed: awesome--let me know if you need anything else for that | 18:42 |
reed | if anyone asks, I just sent another batch of invites | 18:42 |
anteaya | reed: thank you | 18:43 |
reed | fungi, I think we're done for the season :) | 18:43 |
anteaya | reed: is that your last batch? | 18:43 |
anteaya | cool | 18:43 |
reed | from now on I'll send only to those who ask for one | 18:43 |
mordred | greghaynes: yes! I also have working PPA now | 18:44 |
fungi | reed: great! though i'm not done for the season. it's almost time for me to start lather-rinse-repeating for technical election rolls | 18:44 |
reed | anteaya, I'd say so since we're in feature freeze and there shouldn't be many new contributors at this point | 18:44 |
mordred | greghaynes: https://github.com/emonty/vhd-util/blob/master/debian/patches/citrix | 18:44 |
reed | fungi, yeah... we so need to have better tools for that | 18:44 |
mordred | greghaynes: https://launchpad.net/~mordred/+archive/ubuntu/infra | 18:44 |
anteaya | reed: okay | 18:44 |
greghaynes | mordred: Yes, my VHD-util patch seems to mostly work | 18:44 |
mordred | greghaynes: woot | 18:44 |
*** Swami has joined #openstack-infra | 18:44 | |
*** sushilkm has joined #openstack-infra | 18:45 | |
fungi | reed: agreed. i think this can be rewritten now to use the gerrit http api | 18:45 |
mordred | greghaynes: if you just point folks to ppa:mordred/infra - it _should_ work from apt-get install vhd-util | 18:45 |
*** sushilkm has left #openstack-infra | 18:45 | |
mordred | greghaynes: at least it did for me | 18:45 |
zaro | anteaya: ohh, i'm wrong about it not being an ACL, https://review.openstack.org/Documentation/access-control.html#category_edit_topic_name | 18:45 |
greghaynes | mordred: Yea, I doing that, just also wanted to mention why in docs | 18:45 |
openstackgerrit | Boris Pavlovic proposed openstack-infra/project-config: Add new project nerd-reviewer https://review.openstack.org/166945 | 18:45 |
mordred | greghaynes: ++ | 18:45 |
mordred | greghaynes: see, there you go with being sane | 18:45 |
reed | fungi, or openstack-web | 18:46 |
* greghaynes will try to prevent future incidents of this | 18:46 | |
mordred | greghaynes: I also update the ansible-build-image role to not use the docker thing, fwiw | 18:46 |
samueldmq | Hi folks ! I am from Keystone project and I am now willing to contribute to infra projects | 18:46 |
greghaynes | nice! | 18:46 |
fungi | reed: well, maybe, assuming it tracks info on what projects a user has contributed to (not just how recently) | 18:46 |
zaro | anteaya: i think it makes sense to allow all registered users to edit topics | 18:46 |
*** marcusvrn1 has joined #openstack-infra | 18:47 | |
samueldmq | I am still reading up the docs .. and got a question, do you maintain all those projects under github.com/openstack-infra ? | 18:47 |
reed | fungi, right ... I don't know | 18:47 |
*** ildikov has joined #openstack-infra | 18:47 | |
anteaya | zaro: okay however as we did establish I can't do that to your patch via the gui | 18:47 |
fungi | zaro: i think we in fact already do, just not with a convenient click in the webui, so by the same argument we used for the rebase and revert buttons, should be fine to turn on for everyone | 18:47 |
anteaya | samueldmq: hi | 18:49 |
samueldmq | anteaya, hi :) | 18:49 |
anteaya | git.o.o/openstack-infra yes we do | 18:49 |
*** marcusvrn has quit IRC | 18:49 | |
samueldmq | anteaya, hmm, looks a lot of project, just became more interesting :) | 18:49 |
anteaya | samueldmq: we have something for everyone | 18:50 |
samueldmq | anteaya, and you have core teams for the whole set of projects, or each project has its own one? | 18:50 |
fungi | samueldmq: thanks! i certainly think it's interesting anyway | 18:50 |
mordred | samueldmq: you'll never run out of fun around here | 18:50 |
anteaya | samueldmq: our meetings are on tuesdays at 1900: https://wiki.openstack.org/wiki/Meetings/InfraTeamMeeting | 18:50 |
anteaya | samueldmq: one core team for everything | 18:50 |
mordred | samueldmq: one core team for all - some projects have a sub-team which has core in addition to the main core team | 18:50 |
openstackgerrit | David Kranz proposed openstack-infra/project-config: Create new periodic job to run tempest without admin creds https://review.openstack.org/166964 | 18:50 |
anteaya | and some projects have additional members | 18:50 |
samueldmq | anteaya, just after keystone meetings I guess | 18:50 |
anteaya | samueldmq: right after | 18:51 |
samueldmq | anteaya, so I just need to stay for 1 more after, k | 18:51 |
anteaya | samueldmq: :) makes it easy | 18:51 |
samueldmq | fungi, mordred yes I hope, thanks :) | 18:51 |
fungi | yep, i often catch the end of the keystone meetings. and heckle people | 18:51 |
anteaya | samueldmq: you well feel overwhelmed for about 4 weeks, just expect that | 18:51 |
fungi | it gets them to wrap up more quickly ;) | 18:51 |
pabelanger | So I'm starting down the rabbit hole of dib with nodepool, but confused about some elements. If I understand correctly, nodepool will use the following elements from https://github.com/openstack-infra/project-config/tree/master/nodepool/elements but, I don't see where the fedora and vm elements are from? | 18:51 |
pabelanger | https://github.com/openstack-infra/system-config/blob/master/modules/openstack_project/templates/nodepool/nodepool.yaml.erb#L1029 | 18:51 |
anteaya | samueldmq: just keep showing up in channel and at the meetings | 18:51 |
pabelanger | do they come from diskimage-builder directly? | 18:51 |
anteaya | samueldmq: read things ask questions look at log files, review patches | 18:52 |
anteaya | samueldmq: just keep coming back everyday | 18:52 |
fungi | pabelanger: yes, dib has its own basic building block elements and we build on those | 18:52 |
openstackgerrit | David Kranz proposed openstack-infra/project-config: Create new periodic job to run tempest without admin creds https://review.openstack.org/166964 | 18:52 |
anteaya | samueldmq: and eventually something will make sense | 18:52 |
*** dboik has joined #openstack-infra | 18:52 | |
samueldmq | mordred, k got it .. so general core team + main core teams per project | 18:52 |
pabelanger | fungi, perfect. That's what I figured | 18:52 |
*** dkliban_brb is now known as dkliban | 18:52 | |
fungi | pabelanger: https://git.openstack.org/cgit/openstack/diskimage-builder/tree/elements for reference | 18:53 |
samueldmq | anteaya, k got it ... thanks for those advices :-) | 18:53 |
samueldmq | anteaya, I'll be coming here regularly (still have to keep working on keystone) | 18:53 |
*** vigneshvar has quit IRC | 18:54 | |
pabelanger | fungi, Danke | 18:54 |
anteaya | mtreinish: any inclination to flesh out http://git.openstack.org/cgit/openstack-infra/puppet-subunit2sql/tree/README.md a bit, maybe make it an .rst file? | 18:54 |
anteaya | samueldmq: sure | 18:55 |
anteaya | samueldmq: if you see something that interests you ask about it | 18:55 |
anteaya | samueldmq: don't try to understand everything, you won't be able to | 18:55 |
anteaya | samueldmq: just find something you feel you can get into and start with that | 18:55 |
mtreinish | anteaya: sure I can do that | 18:56 |
zxiiro | question, for nodepool do I need gearman to use it or is gearman optional? | 18:56 |
*** soren has joined #openstack-infra | 18:56 | |
*** peristeri has joined #openstack-infra | 18:57 | |
samueldmq | anteaya, great, makes sense | 18:57 |
sdague | fungi: seeing a bunch of docs failures in gate and check now | 18:57 |
samueldmq | anteaya, the test jobs run when we send new patch sets are managed by devstack-gate, right? | 18:58 |
lifeless | mordred: pong | 18:58 |
sdague | fungi: http://paste.openstack.org/show/195509/ | 18:58 |
mtreinish | anteaya: to be honest I forgot that it had separate docs because it was moved into a separate repo | 18:58 |
sdague | maybe that should be a dhellmann ask as well | 18:58 |
anteaya | mtreinish: where is the code that is on docs.o.o about subunit2SQL, maybe just link to that? | 18:58 |
fungi | sdague: multiple projects? | 18:58 |
*** Rockyg has joined #openstack-infra | 18:58 | |
anteaya | mtreinish: where are the docs.o.o docs coming from? | 18:58 |
SpamapS | greghaynes: did you have any luck on rax btw? | 18:58 |
sdague | fungi: yes | 18:58 |
mtreinish | anteaya: I thought I did link to the docs.o.o subunit2sql page | 18:59 |
sdague | nova & keystone in the gate right now | 18:59 |
mtreinish | oh you meant in the puppet repo | 18:59 |
SpamapS | greghaynes: I'm just circled back to where I should probably create an account on rax and see if I can get it to boot from my image. | 18:59 |
mtreinish | not the systemconfig patch | 18:59 |
sdague | that patch that I posted was an oslo.messaging one | 18:59 |
fungi | sdague: no new sphinx or pbr today fwiw | 18:59 |
sdague | fungi: new tox today though, right? | 19:00 |
anteaya | sdague: test jobs are defined in project-config/zuul/layout.yaml and project-config/jenkins/project.yaml | 19:00 |
anteaya | samueldmq: ^^ | 19:00 |
mordred | lifeless: did I ping you? | 19:00 |
anteaya | sdague: sorry tab complete fail | 19:00 |
sdague | anteaya: no worries | 19:00 |
fungi | i need to disable the ctrl-q hotkey in firefox. on my keyboard it's right next to ctrl-w which i use. maybe this is an argument in favor of dvorak | 19:00 |
mordred | lifeless: oh - i was mentioning the pip setup_requires work | 19:00 |
samueldmq | anteaya, great thanks, I think it's good place to start | 19:00 |
mtreinish | anteaya: the docs are from: https://git.openstack.org/cgit/openstack-infra/subunit2sql/tree/doc/source | 19:00 |
mordred | lifeless: no need for anything just now | 19:00 |
anteaya | samueldmq: devstack-gate has jobs that have a devstack-gate component, yes | 19:00 |
anteaya | samueldmq: yes it is | 19:00 |
*** garyh has quit IRC | 19:01 | |
fungi | sdague: indeed! new tox. must have missed discussion while lunching | 19:01 |
samueldmq | anteaya, nice thanks :) I will be back once I get into trouble | 19:01 |
mordred | SpamapS: I recommend getting one on general principle - having the ability to boot something in rackspace can be highly helpful | 19:01 |
openstackgerrit | Boris Pavlovic proposed openstack-infra/project-config: Add new project nerd-reviewer https://review.openstack.org/166945 | 19:01 |
anteaya | mtreinish: ah yeah, just pointing the puppet-module readme at the subunit2SQL readme makes sense to me | 19:01 |
*** johnthetubaguy is now known as zz_johnthetubagu | 19:02 | |
greghaynes | SpamapS: Getting there, I got all my acct stuff setup, trying to use the vhd-util to boot a rax node | 19:02 |
mtreinish | anteaya: ok sure, that should be simple enough. I'll push a patch up for that soonish | 19:02 |
anteaya | mtreinish: yes the empty puppet repo readme was what I was looking at | 19:02 |
anteaya | mtreinish: sure, two nits in your patch and then it lgtm | 19:02 |
mtreinish | anteaya: ok cool, thanks. I'll respin it now | 19:02 |
greghaynes | SpamapS: You just need to sign up, wait for the call, tell them to enable glance and youre good to go | 19:02 |
*** mfink_ has quit IRC | 19:02 | |
mordred | SpamapS, greghaynes: oh, I dno' tknow if you noticed, but I put ina "HOW-TO-TEST" doc in the project-config patch | 19:02 |
greghaynes | mordred: what!? docs!? | 19:03 |
*** fcorral_hp has joined #openstack-infra | 19:03 | |
anteaya | samueldmq: try to be back before then | 19:03 |
mordred | greghaynes: https://review.openstack.org/#/c/154132/17/tools/HOW-TO-TEST.rst | 19:03 |
anteaya | samueldmq: but everyone has their own workflow | 19:03 |
mordred | greghaynes: I don't have docs for manual testing on HP - but the ansible playbook will upload to hp | 19:03 |
*** otter768 has joined #openstack-infra | 19:03 | |
openstackgerrit | Matthew Treinish proposed openstack-infra/system-config: Add docs for subunit2sql https://review.openstack.org/164397 | 19:03 |
greghaynes | mordred: ok, that answers my questions about 'what elements are youall using' | 19:04 |
mtreinish | anteaya: ^^^ | 19:04 |
*** raginbajin has quit IRC | 19:04 | |
*** frickler has quit IRC | 19:04 | |
*** grantbow has quit IRC | 19:04 | |
*** SpamapS has quit IRC | 19:04 | |
*** SpamapS has joined #openstack-infra | 19:04 | |
*** SpamapS has quit IRC | 19:04 | |
*** SpamapS has joined #openstack-infra | 19:04 | |
*** raginbajin has joined #openstack-infra | 19:04 | |
*** jklare has quit IRC | 19:04 | |
*** otherwiseguy has quit IRC | 19:04 | |
*** mikal has quit IRC | 19:04 | |
*** nikhil_k has quit IRC | 19:04 | |
*** heyongli has quit IRC | 19:04 | |
*** frickler has joined #openstack-infra | 19:04 | |
*** nikhil_k has joined #openstack-infra | 19:04 | |
*** mikal has joined #openstack-infra | 19:04 | |
*** otherwiseguy has joined #openstack-infra | 19:04 | |
*** jklare has joined #openstack-infra | 19:04 | |
fungi | sdague: mordred: dhellmann: so tox is running distutils cmd_obj.run() which is passing fewer arguments than pbr's _sphinx_run() expects | 19:04 |
fungi | setuptools api change? | 19:05 |
*** baoli has quit IRC | 19:05 | |
*** mfink_ has joined #openstack-infra | 19:05 | |
mordred | greghaynes: those elements are good ones to start with - the full nodepool ones take FOREVER to build because they cache everything | 19:05 |
mordred | fungi: maybe? | 19:05 |
fungi | new setuptools was released on friday | 19:05 |
mordred | sounds likely | 19:05 |
mordred | greghaynes: if you can get those elements to boot in rackspace, then adding more content should be fine | 19:06 |
samueldmq | anteaya, I just didn't want to come here with a lot of newbie questions (I think I'll have a lot, since I don't have too much xp with infra) | 19:06 |
mordred | greghaynes: the process-config-drive element is the one where the init scripts / systemd / upstart may not be solid yet - so the script may not be running in the right sequence | 19:07 |
greghaynes | mordred: ok. For now I am just trying to get an ubuntu element image booting for the sake of landing the vhd-util patch, but next is trying to replicate what youall are doing | 19:07 |
*** pelix has quit IRC | 19:07 | |
samueldmq | anteaya, but ok then, I'll be here before getting into trouble :) | 19:07 |
sdague | fungi: I think we need to print T-shirts "new setuptools released on friday" | 19:07 |
*** MarkAtwood has joined #openstack-infra | 19:07 | |
*** jerryz has quit IRC | 19:07 | |
anteaya | mtreinish: +1 thanks | 19:07 |
samueldmq | anteaya, thanks | 19:07 |
clarkb | sdague +1 | 19:07 |
mordred | greghaynes: totally correct choice on your part | 19:07 |
anteaya | samueldmq: there we go | 19:07 |
fungi | sdague: or a captioned beer stein but yes | 19:07 |
samueldmq | anteaya, yes thx o/ | 19:08 |
anteaya | samueldmq: just ask questions and someone will do their best to read backscroll and reply | 19:08 |
mordred | greghaynes: well, as an FYI - you're goign to have networking problems on rax with that | 19:08 |
openstackgerrit | Boris Pavlovic proposed openstack-infra/project-config: Add rally-ci project https://review.openstack.org/166962 | 19:08 |
greghaynes | mordred: perfect :) | 19:08 |
mordred | greghaynes: because cloud-init does not understand their patches to config-drive | 19:08 |
greghaynes | yea | 19:08 |
anteaya | samueldmq: if we are in the middle of a fire, we might be a while before we can reply but we do try our best | 19:08 |
mordred | greghaynes: which is why we have the process-config-drive element | 19:08 |
mordred | greghaynes: BUT - you can test via web console just fine :) | 19:08 |
greghaynes | yep, as long as it boots though ill consider that a success for that patch | 19:08 |
* mordred shuts up - lets greghaynes go be smart and stuff | 19:09 | |
sdague | fungi: do you currently have a script to generate the last week of releases by global requirements? (or equiv) or are you manually poking at this like I often do | 19:09 |
sdague | because we seem to be problem determining this quite often of "oh crap, what released" | 19:09 |
fungi | sdague: after enough iterations of this, i just know which usual suspects to check first | 19:09 |
sdague | fungi: yeh, me to | 19:09 |
fungi | sdague: in this case distutils in the traceback makes me take a look at setuptools | 19:10 |
sdague | but it's still time consuming, and too much tribal knowlegde | 19:10 |
sdague | it's a fragmented day for me, maybe I'll work on banging something out for future | 19:10 |
*** erlon_away has joined #openstack-infra | 19:10 | |
fungi | wondering if today's tox changed its dependency on (vendoring of?) setuptools, or just changed the entrypoint it's using to run commands | 19:11 |
samueldmq | anteaya, k fair enough, you cannot train an intern when you need to extinguish the fire :) | 19:11 |
fungi | samueldmq: sometimes we use new recruits in the bucket line though | 19:11 |
samueldmq | fungi, sure, I'll be glad to help if I can | 19:12 |
anteaya | samueldmq: sure you can, you train them to step back and do crowd control | 19:12 |
*** ZZelle_ has joined #openstack-infra | 19:12 | |
*** HeOS has joined #openstack-infra | 19:12 | |
openstackgerrit | Spencer Krum proposed openstack-infra/project-config: No need to run the puppet-unit tests on setproxy https://review.openstack.org/166976 | 19:12 |
openstackgerrit | Khai Do proposed openstack-infra/jenkins-job-builder: Allow setting of only "single-use" option for jclouds plugin https://review.openstack.org/122333 | 19:12 |
samueldmq | anteaya, ++ | 19:12 |
fungi | sdague: this is the point at which i start plugging random bits of the traceback into a web search and seeing if it turns up interesting ideas, because stuffing debug/breakpoints into that call chain is a pita | 19:12 |
anteaya | samueldmq: I've been on crowd control for sometime | 19:12 |
anteaya | samueldmq: I'll teach you want I know | 19:13 |
mordred | jeblair: 19:12:39 openstackgerrit | Khai Do proposed openstack-infra/jenkins-job-builder: Allow setting of only "single-use" option for jclouds plugin | 19:13 |
mordred | jeblair: look ^^ jclouds plugin has single-use now | 19:13 |
fungi | neat that we wrote a jclouds replacement, and now it's copying features from us ;) | 19:13 |
samueldmq | anteaya, great, thanks a lot for being willing to teach me :) | 19:14 |
openstackgerrit | Khai Do proposed openstack-infra/jenkins-job-builder: Allow setting of only 'single-use' option for jclouds plugin https://review.openstack.org/122333 | 19:14 |
openstackgerrit | Boris Pavlovic proposed openstack-infra/project-config: Add new project nerd-reviewer https://review.openstack.org/166945 | 19:14 |
anteaya | samueldmq: sure, actually teaching someone else is my best method of learning what I don't know | 19:14 |
fungi | boris-42: i really hope that ^ will help me do a better job of reviewing nerds | 19:14 |
boris-42 | fungi: =) | 19:15 |
boris-42 | fungi: actually I would like to get automatically review that I am trying to do mad stuff with infra=) | 19:15 |
samueldmq | anteaya, yes I agree 100%, I was teaching assistant during my undergraduation, and I share this thought | 19:15 |
*** armax has quit IRC | 19:15 | |
boris-42 | fungi: and not spend time of reviewers for that | 19:16 |
boris-42 | =) | 19:16 |
*** markus_z has quit IRC | 19:16 | |
fungi | boris-42: a lot of us have gone mad already, so makes sense to me | 19:16 |
anteaya | samueldmq: cool | 19:16 |
anteaya | :) | 19:16 |
* mordred runs around waving arms in the air and screaming | 19:16 | |
*** yamamoto_ has joined #openstack-infra | 19:16 | |
fungi | mordred: i've decided i'm going mad. care to join me? | 19:17 |
boris-42 | =) | 19:17 |
mordred | fungi: I'm all in on that party | 19:17 |
*** dhritishikhar has quit IRC | 19:17 | |
fungi | sdague: it's funny when searching abroad brings me back home again... see bug 1379998 | 19:18 |
openstack | bug 1379998 in PBR "pbr incompatible with sphinx 1.3b1" [Medium,Triaged] https://launchpad.net/bugs/1379998 | 19:18 |
* boris-42 being mad is not so bad | 19:18 | |
fungi | sdague: _could_ be a coincidence, but the "new" traceback bears a striking resemblance to that | 19:19 |
*** marcusvrn has joined #openstack-infra | 19:19 | |
*** yamamoto_ has quit IRC | 19:21 | |
*** woodster_ has joined #openstack-infra | 19:22 | |
*** timcline has quit IRC | 19:22 | |
*** marcusvrn1 has quit IRC | 19:22 | |
*** Somay has quit IRC | 19:22 | |
*** erlon_away is now known as erlon | 19:23 | |
fungi | sdague: also unfortunate that the job in question only runs pip freeze (well, pbr freeze) when it succeeds. though that still may be enough for me to work out what changed | 19:23 |
openstackgerrit | Boris Pavlovic proposed openstack-infra/project-config: Add new rally-ci project https://review.openstack.org/166962 | 19:23 |
fungi | sdague: last successful run used Sphinx==1.2.3 | 19:24 |
fungi | sdague: i bet something's got us using 1.3.1 (latest release) in the past few hours | 19:24 |
*** garyh has joined #openstack-infra | 19:25 | |
openstackgerrit | Boris Pavlovic proposed openstack-infra/project-config: Add new project nerd-reviewer https://review.openstack.org/166945 | 19:26 |
fungi | doing some local testing to see if i can nail down what | 19:28 |
rlucio | hi guys, by chance is there a way (for 3rdparty ci) to tell zuul only to run jobs on a project master branch? | 19:29 |
openstackgerrit | Imran Hayder Malik proposed openstack-infra/project-config: Change exit code to 1 for when test fails https://review.openstack.org/166980 | 19:29 |
pabelanger | fungi, well, this is going to be fun. Looks like diskimage-builder doesn't like tar on ubuntu 12.04 | 19:30 |
pabelanger | sudo tar -C /tmp/image.uKWC0wip/mnt --numeric-owner --xattrs '--xattrs-include=*' --xattrs-exclude=security.selinux -xzf /home/nodepool/.cache/image-create/20-Cloud-x86_64-20.tgz | 19:30 |
pabelanger | tar: unrecognized option '--xattrs' | 19:30 |
fungi | pabelanger: yep, we upgraded our nodepool server to 14.04 to deal with that | 19:30 |
fungi | pabelanger: the alternative is not to dib centos7/rhel7/recent fedora | 19:31 |
pabelanger | fungi, yar, suspected that | 19:31 |
pabelanger | okay | 19:31 |
*** dboik has quit IRC | 19:31 | |
mordred | greghaynes, SpamapS: ^^ I don't suppose it would be possible to support older, would it? | 19:31 |
pabelanger | let me start with Debian dib | 19:31 |
zaro | anteaya, fungi : i think only admins have access to push changes on refs/meta/config for review correct? | 19:31 |
*** dboik has joined #openstack-infra | 19:32 | |
*** prad has quit IRC | 19:32 | |
zaro | therefore only admins can proprose that change to enable editing of topics for all projects? | 19:32 |
anteaya | zaro: i can't confirm what we do, but that would make sense to me that we do that | 19:32 |
fungi | zaro: we usually tackle that by using a change to the configuration instructions in docs/source/gerrit.rst in system-config as a proxy for having a gerrit admin add the same to the global configuration | 19:33 |
fungi | zaro: so if the change to the config documentation gets consensus, then whoever approves it should make sure it gets set | 19:34 |
zaro | anteaya: so looksl ike a doc change is what is required. | 19:34 |
fungi | we'd like to have that under configuration management too, but it changes infrequently enough that we've gotten by keeping it updated manually so far | 19:35 |
zaro | anteaya: the workaround is change the commit message then 'git review -t new_topic' | 19:35 |
*** sarob has joined #openstack-infra | 19:35 | |
*** otter768 has quit IRC | 19:36 | |
*** timcline has joined #openstack-infra | 19:36 | |
zaro | jhesketh: do my comments make sense? https://review.openstack.org/165228 | 19:37 |
openstackgerrit | Boris Pavlovic proposed openstack-infra/project-config: Add new project nerd-reviewer https://review.openstack.org/166945 | 19:37 |
anteaya | zaro: well I can submit a new patch no problem, I hadn't expect a snowball to have to change permissions | 19:37 |
*** armax has joined #openstack-infra | 19:38 | |
*** dustins has joined #openstack-infra | 19:39 | |
*** alexpilotti has quit IRC | 19:39 | |
clarkb | ok back from all the things | 19:40 |
*** ldnunes_ has joined #openstack-infra | 19:40 | |
zaro | anteaya: snowball sounds better than avalanche :) | 19:40 |
pabelanger | fungi, thanks, ubuntu dib looks happier | 19:41 |
*** ominakov has joined #openstack-infra | 19:41 | |
anteaya | zaro: it does | 19:42 |
greghaynes | mordred: yea, I think that doesnt actually apply to ubuntu even | 19:42 |
greghaynes | mordred: IIRC its a rhel thing | 19:42 |
clarkb | rlucio: yes, use the branch modifier on the job in the jobs list to specify only master | 19:42 |
*** ajo has joined #openstack-infra | 19:43 | |
clarkb | rlucio: if you look at our layout.yaml file you should find man examples of this | 19:43 |
greghaynes | the dib fix probably to just check tar version and if theres no xattrs dont specify it and let it blow up if it needs to | 19:43 |
clarkb | rlucio: just grep for 'branch' | 19:43 |
*** marun has quit IRC | 19:43 | |
rlucio | clarkb: cool, thanks | 19:44 |
clarkb | can I get reviewers for https://review.openstack.org/#/c/166960/ it should finally clean up the os-loganalyze issues with console.html files getting log levels | 19:44 |
*** yfried has joined #openstack-infra | 19:45 | |
*** sdake has joined #openstack-infra | 19:46 | |
*** MarkAtwood has quit IRC | 19:46 | |
*** MarkAtwood has joined #openstack-infra | 19:46 | |
*** heyongli has joined #openstack-infra | 19:46 | |
*** soren has quit IRC | 19:47 | |
*** hyakuhei has joined #openstack-infra | 19:47 | |
*** sdake__ has quit IRC | 19:49 | |
JayF | Hey, got a strange tests issue -> https://review.openstack.org/#/c/164931/ is failing pep8 in check/gate but passes locally on a fresh checkout + set of deps | 19:50 |
*** grantbow has joined #openstack-infra | 19:50 | |
*** grantbow has quit IRC | 19:50 | |
*** grantbow has joined #openstack-infra | 19:50 | |
fungi | JayF: did you run it via 'tox -e pep8' or some other way? | 19:52 |
JayF | I included the whole output (I used tox -r) in my comment on that change | 19:52 |
fungi | ahh, you left a comment saying you did | 19:52 |
JayF | although JoshNang just confirmed he was able to reproduce in his new devstack | 19:52 |
JayF | so I'm really confused | 19:52 |
clarkb | two things to try | 19:52 |
clarkb | always grab the zuul ref or rebase on $branch manually and use tox -r | 19:52 |
*** AJaeger_ has quit IRC | 19:53 | |
fungi | JayF: checking the pip freeze from that job log against one in your .tox/pep8 venv would also be a good place to start | 19:53 |
*** dhritishikhar has joined #openstack-infra | 19:53 | |
greghaynes | mordred: pabelanger ah, I see whats going on, youre actually trying to build a redhat image. The possible fix there is to detect that and disable selinux... thats pretty :( though | 19:54 |
*** mrunge_bbl has quit IRC | 19:54 | |
*** marun has joined #openstack-infra | 19:54 | |
greghaynes | mordred: pabelanger is being able to build redhat on ubuntu 12.04 something thats wanted badly? | 19:54 |
JayF | jlvillal: JoshNang ^ a couple of suggestions above. I'm going to finish my lunch but if you guys wanna try a thing, there's a good start | 19:55 |
JayF | otherwise after lunch I'll figure it out and fix it | 19:55 |
*** hodos has quit IRC | 19:55 | |
*** hodos has joined #openstack-infra | 19:56 | |
pabelanger | greghaynes, define badly? Nice to have for me as it avoids some nodepool upgrades | 19:57 |
*** notnownikki has quit IRC | 19:57 | |
SpamapS | I think there's a patch in the queue to make dib not use xattrs if it's not supported. | 19:57 |
greghaynes | pabelanger: ah, well thats good enough for me | 19:57 |
greghaynes | SpamapS: orly | 19:57 |
SpamapS | yeah looking now | 19:57 |
*** sarob has quit IRC | 19:57 | |
SpamapS | it may have even landed | 19:58 |
SpamapS | 8026dda8a019abcd570e25ba3c8bedc9356718c9 | 19:58 |
SpamapS | it was for the centos6 element | 19:58 |
SpamapS | so this is interesting | 19:59 |
greghaynes | hrmm, I wonder if that works | 19:59 |
SpamapS | redhat-common actually _re-explodes_ the tarball with --xattrs if it is supported | 19:59 |
SpamapS | using the tar from inside the chroot | 19:59 |
greghaynes | oh | 20:00 |
greghaynes | wat | 20:00 |
openstackgerrit | Matthew Treinish proposed openstack-infra/devstack-gate: Add flag to disable tempest with admin creds https://review.openstack.org/166993 | 20:00 |
*** ldnunes_ has quit IRC | 20:01 | |
greghaynes | So, what I was going to say before being confused about what is actually going on, if we do like that patch does itll work to make an image but selinux should make it so most everything doesnt work on that box | 20:01 |
mordred | greghaynes: yes | 20:01 |
mordred | greghaynes: at least, we make all of our images on the same build host right now | 20:02 |
SpamapS | greghaynes: unless we make it non-enforcing | 20:02 |
clarkb | anteaya: I have reviewed that os-loganalyze change properly now | 20:02 |
greghaynes | SpamapS: exactly, so I think that part is missing | 20:02 |
SpamapS | well IIRC you have to ask for enforcing explicitly right now | 20:02 |
greghaynes | ah, well then | 20:02 |
clarkb | SpamapS: greghaynes this is why I think dib should just mount the qcow2 | 20:02 |
clarkb | rather than extract a tarball, chroot, then go back to a qcow2 | 20:03 |
greghaynes | clarkb: dont you have to nbd for that? | 20:03 |
clarkb | greghaynes: yes | 20:03 |
clarkb | but it should just work on precise | 20:03 |
clarkb | and elsewhere | 20:03 |
clarkb | or convert to raw and mount that, that shouldn't need nbd right? | 20:04 |
mordred | clarkb: what if you don't start with a qcow | 20:04 |
greghaynes | yes, I prefer that | 20:04 |
mordred | clarkb: but instead start with a chroot? | 20:04 |
SpamapS | clarkb: Yeah I don't know exactly why we repack. It was that way in the initial commit. | 20:04 |
SpamapS | clarkb: nbd has races unfortunately. | 20:04 |
clarkb | mordred: you would mount an empty disk and put the chroot there | 20:04 |
*** marcusvrn has quit IRC | 20:04 | |
clarkb | mordred: you are still chrooting but none of this back and forth taring | 20:04 |
SpamapS | which is why we avoid it in diskimage-builder now | 20:04 |
*** jgrimm is now known as zz_jgrimm | 20:05 | |
mordred | oh good | 20:05 |
clarkb | dib does an incredible amount of unnecessary IO | 20:05 |
clarkb | which really surprised me because I thought it was optimized to not do that | 20:05 |
SpamapS | clarkb: convert to raw works, but has to be done carefully so you don't have a 30G non-sparse file every time. ;) | 20:05 |
*** mfink_ has quit IRC | 20:05 | |
SpamapS | clarkb: simple > fast :) | 20:06 |
SpamapS | because at one point it _was_ simple :) | 20:06 |
*** soren has joined #openstack-infra | 20:06 | |
greghaynes | yea, we should definitely make dealing with raw data smarter, AIUI thats just a matter of effort though | 20:06 |
SpamapS | greghaynes: lvm ftw? | 20:06 |
greghaynes | yes | 20:06 |
SpamapS | aaaaand we've reimplemented schroot | 20:06 |
* greghaynes wants very much so | 20:06 | |
greghaynes | hey, well, it works | 20:06 |
SpamapS | let's just use schroot | 20:07 |
*** tsg_ has quit IRC | 20:07 | |
fungi | sdague: i confirmed that sphinx 1.3.1 is getting pulled in rather than 1.2.3, looks like new tox is ignoring the versions specified in requirements files (confirmed with a requirements.txt listing only sphinx). trying to figure out why now | 20:07 |
greghaynes | maybe make dib-run-parts > schroot | 20:07 |
greghaynes | er, use schroot | 20:07 |
greghaynes | that being said, lvm shouldnt be a hard req | 20:07 |
JayF | clarkb: fungi: looks like a new hacking. .8.1 in passing tests, .10.1 in failing ones | 20:08 |
*** tsg has joined #openstack-infra | 20:08 | |
JayF | clarkb: fungi: And it adds a stack of rules that break things | 20:08 |
mordred | ianw: ah - I see the thing you're talking about | 20:08 |
mordred | ianw: the base element is not really a base element | 20:08 |
*** Swami has quit IRC | 20:08 | |
SpamapS | clarkb: if you ever get a bee in your bonnet about the unnecessary I/O, btw, I'd be quite happy if a bug appeared for us to work on at some point. :) | 20:08 |
mordred | ianw: it shoudl be named "opinionated element that sets things up similar to how some cloud images are set up | 20:08 |
mordred | " | 20:08 |
fungi | JayF: i wonder if this is the same underlying cause as the sphinx problem i'm looking into (tox installing latest versions of things even if we pin requirements to something lower) | 20:09 |
* JayF also confused why tox -r wasn't pulling in .8.1 | 20:09 | |
clarkb | SpamapS: ya one day when I am not already busy doing something else with dib a 45 minute iumage build is likely to make me fix it | 20:09 |
SpamapS | right. :) | 20:09 |
mordred | ianw: I always run disk-image-create with the "-n" option | 20:09 |
JayF | fungi: test-requirements.txt:hacking>=0.8.0,<0.9 | 20:09 |
JayF | fungi: ^ in IPA | 20:09 |
greghaynes | I really dont think the tarring is much of the image build time though | 20:09 |
ianw | mordred: i guess :) how were you starting d-i-b | 20:09 |
greghaynes | I agree its uneccesary | 20:09 |
JayF | fungi: so I suspect yes | 20:09 |
fungi | JayF: can you see if upgrading tox causes it to start failing for you too? can you confirm what version of tox you're passing with right now | 20:09 |
clarkb | greghaynes: it really is | 20:09 |
fungi | ? | 20:09 |
clarkb | greghaynes: as someone with slow local builds the tar then copy later is at least half of my build time | 20:09 |
greghaynes | but I have 30sec image bulids when im not doing much of any extra work | 20:09 |
JayF | fungi: that's the test-requirements for IPA; .10.1 got installed in the gate which is not >=0.8.0,<0.9 | 20:10 |
mordred | SpamapS: the minimal images point out a bug in the base element ... should we fix? | 20:10 |
JayF | http://logs.openstack.org/31/164931/5/check/gate-ironic-python-agent-pep8/1a0efa7/console.html | 20:10 |
clarkb | next is the qemu-img conversion | 20:10 |
greghaynes | clarkb: yea, that stuff takes 10's of seconds usually | 20:10 |
* JayF upgrades tox and tests | 20:10 | |
fungi | JayF: my money says latest tox will show this problem for you too | 20:11 |
clarkb | greghaynes: I think you have ssds | 20:11 |
greghaynes | heh | 20:11 |
*** annegentle has joined #openstack-infra | 20:11 | |
clarkb | greghaynes: and are not building 5GB images | 20:11 |
SpamapS | mordred: yes most definitely | 20:11 |
greghaynes | clarkb: You should hack up dib to time them | 20:11 |
greghaynes | clarkb: curious what it says | 20:11 |
sdague | fungi: interesting | 20:11 |
JayF | fungi: well, pip --upgrade install tox broke my environment, so standby | 20:11 |
JayF | fungi: but completely fresh environments being broken (like JoshNang saw) supports this | 20:11 |
fungi | sdague: confirmed it's not the -U to pip install at fault either | 20:11 |
ianw | mordred: i see, wasn't aware of -n | 20:12 |
sdague | fungi: nice | 20:12 |
JayF | JoshNang: what's the tox version in your devstack with pep8 fails? | 20:12 |
JoshNang | one sec | 20:12 |
mordred | SpamapS: "base" expects cloud-init -which not all root elements installs | 20:12 |
fungi | sdague: tox 1.9 did change some pip install behaviors (at least it no longer passes --pre by default) so maybe other things changed around how it calls pip too | 20:12 |
mordred | SpamapS: so we should really make a "cloud-init" element that "base" can depend on | 20:12 |
sdague | fungi: what level of pain is it to try to pin tox in the gate? | 20:13 |
mordred | SpamapS: or - rename base and make it not included by default | 20:13 |
greghaynes | mordred: yep, thats a bg | 20:13 |
greghaynes | er, bug | 20:13 |
JoshNang | JayF: tox 1.9.1 | 20:13 |
jlvillal | JayF: JoshNang: tox 1.7.1 and 1.9.0 are working for me | 20:13 |
fungi | sdague: image rebuilds at a minimum | 20:13 |
*** smarcet has quit IRC | 20:13 | |
fungi | jlvillal: okay, so sounds like maybe it broke between 1.9.0 and 1.9.1 (i hadn't started bisecting tox just yet) | 20:13 |
clarkb | we pinned it in the past, it took like 6 months to unpin it | 20:14 |
greghaynes | mordred: IMO its probably easier to just fix base, theres a few things in there that need to stay (like updating) | 20:14 |
*** arosen has joined #openstack-infra | 20:14 | |
SpamapS | mordred: -n skip the default inclusion of the 'base' element | 20:14 |
clarkb | and when I worked to unpin it I got a bunch of -2s from projects | 20:14 |
clarkb | so | 20:14 |
clarkb | I am very much against pinning pip if we can avoid it | 20:14 |
mordred | SpamapS: that's how I run | 20:14 |
*** mrda-away is now known as mrda | 20:14 | |
mordred | greghaynes, SpamapS: https://review.openstack.org/166995 | 20:14 |
*** andreykurilin_ has joined #openstack-infra | 20:15 | |
mordred | I added cloud-init to the base package-installs | 20:15 |
* greghaynes liked the remove cloud-init from base and make other elements depend on it idea | 20:15 | |
JayF | fungi: confirmed, it broke for me when I went 1.9.0->1.9.1 | 20:15 |
mordred | yes. I think that's a great idea | 20:15 |
SpamapS | mordred: we may have matured the deps/provides enough to not need base... perhaps we could just let things dep on it if they do need it for real. | 20:15 |
*** mfink_ has joined #openstack-infra | 20:15 | |
mordred | SpamapS: I will defer to your jugement on that one | 20:15 |
*** rlandy|afk is now known as rlandy | 20:16 | |
fungi | i should have looked at tox bugs before i started down this road | 20:16 |
Kiall | Q - Anyone else seeing gate-*-pep8 jobs randomly taking much longer than usual, and timing out after 30 mins? Happened twice on this translation iport review, with a sucess in the middle.. | 20:16 |
Kiall | https://review.openstack.org/#/c/166506/ | 20:16 |
JoshNang | JayF: confirmed as well | 20:16 |
greghaynes | SpamapS: Neat thing about that - maybe rather than check for an OS element to see if a user has a valid list of elements we can check for elements in the deps | 20:16 |
fungi | sdague: JayF: https://bitbucket.org/hpk42/tox/issue/230/tox-191-ignores-package-versions-specified | 20:16 |
fungi | dstufft: you'll probably like that one too | 20:16 |
JayF | fungi: Is there anything I can do to help resolve this to unstick IPA check/gate? | 20:16 |
greghaynes | SpamapS: er, we can check for a base element in the elements | 20:17 |
jeblair | jhesketh: ping re 166596 when you are awake | 20:17 |
clarkb | jeblair: I thought we had plenty of disagreement :) | 20:17 |
* clarkb reads meeting log | 20:17 | |
jeblair | clarkb: i thought no one at the meeting disagreed with that | 20:18 |
clarkb | ya not in that meeting but iirc sdague and others didn't like that method | 20:18 |
clarkb | I am fine with it either way | 20:18 |
SpamapS | greghaynes: right so perhaps we can just let minimal elements provide base and be done w/ it? | 20:18 |
samueldmq | mtreinish, jsut left my first review on infra on your patch (#164397) | 20:18 |
clarkb | I don't think one is necessarily better than the other | 20:18 |
samueldmq | mtreinish, please let me know if you have any question/comment about my review | 20:18 |
SpamapS | greghaynes: but guessing that would violate the actual provides thing, because part of base is having tools you expect all the time | 20:19 |
jeblair | clarkb: yeah, i heard a rumor that sdague wanted it in there, no one could point me to where that conversation actually happened | 20:19 |
*** kgiusti has left #openstack-infra | 20:19 | |
jeblair | clarkb: i also thought jhesketh was going to look into that too | 20:19 |
*** andreykurilin_ has quit IRC | 20:19 | |
*** andreykurilin__ has joined #openstack-infra | 20:19 | |
jeblair | at any rate, sdague was mentioned in the meeting, and he's here now and can speak for himself at some point if he wants :) | 20:20 |
clarkb | yup | 20:20 |
fungi | JayF: if it looks like it was this morning's image updates which caused the bug to emerge, then i'll delete them and switch to yesterday's while we see what tox upstream does with that bug report | 20:20 |
fungi | checkign logstash now to confirm | 20:20 |
*** e0ne has joined #openstack-infra | 20:21 | |
sdague | what's the question? | 20:21 |
JayF | fungi: that fits with the change I approved this morning that hit the bug. It passed CI at 2:50am PST | 20:21 |
JayF | fungi: but then failed the gate ~7h later | 20:21 |
clarkb | fungi: there is an argument there that that code in tox should be completely removed | 20:21 |
jeblair | 19:07:14 <jeblair> for the footer, should we have the d-g jobs generate their own index pages? | 20:21 |
JayF | fungi: Is it possible we have done something like pushed bad releases in post jobs due to this tox failure? | 20:21 |
jeblair | 19:07:38 <jhesketh> My opinion is yes, but sdague and others disagree | 20:21 |
jeblair | sdague: ^ | 20:22 |
JayF | fungi: I don't think so, but just tossing it out there | 20:22 |
jeblair | sdague: from http://eavesdrop.openstack.org/meetings/infra/2015/infra.2015-03-10-19.01.log.html | 20:22 |
clarkb | jeblair: while looking at os-loganalyze any chance for review on https://review.openstack.org/#/c/166960/ ? | 20:22 |
*** crc32 has joined #openstack-infra | 20:22 | |
jeblair | clarkb: ya | 20:22 |
fungi | JayF: it looks like the first occurrence was 14:58 utc today, which is around when our image updates start to show up on our providers. correlating to our mirror logs now to make sure that's not just coincident with the tox release time | 20:23 |
*** crc32 has quit IRC | 20:23 | |
sdague | right, my issue is that typically we figure out well after the fact that we should update that documentation, which is fine in the current system because it becomes relevant for all existing jobs | 20:23 |
JayF | fungi: bah, your obsession with evidence! Shouldn't we just change things and see what happens? /s | 20:23 |
sdague | all existing logs | 20:23 |
sdague | but if this is getting burned in on upload time, it's all going to be wrong all the time | 20:24 |
sdague | at which point it's probably more misleading than useful | 20:24 |
fungi | JayF: yep, bandersnatch saw tox 1.9.1 on pypi as early as 09:30 utc today, so that's many hours after release where we didn't hit this. i think we can safely say an image rollback will stop the bleeding for the moment | 20:25 |
clarkb | greghaynes: I think the ocnfusion on 165582 is that the fail test seems to assert that it succeeded | 20:25 |
jeblair | sdague: true, but we've also seen that the pattern matching gets very complex whenever we make changes, and it's hard to keep it current because it's so far away from what it's actually referencing.... so my suggestion was that we have the job themselves generate some things (like "the nova logs are here") and link to, say, a wiki page or some other documentation that explains overall things that might evolve over time. | 20:25 |
*** tonytan4ever has quit IRC | 20:25 | |
clarkb | greghaynes: you should have one node at the end of the fail test and it is the same node as the other node | 20:26 |
jeblair | sdague: basically, the opposite is true too -- we can't keep a single document current for 6 months of work on separate branches. but if we put the most specific stuff in d-g itself, then it should be much easier to keep it somewhat current and correct for various conditions | 20:26 |
greghaynes | clarkb: ah! | 20:26 |
greghaynes | clarkb: correct | 20:26 |
JayF | fungi: lmk when the recheck hammer will fix a thing and I'll drop it. Thanks! | 20:27 |
fungi | JayF: will do, but expect it to be a few minutes. it's a lot of images | 20:27 |
openstackgerrit | Sean Dague proposed openstack/requirements: add what-broke.py analayzer https://review.openstack.org/166997 | 20:27 |
jogo | anteaya: do you have any further questions about reviewing https://review.openstack.org/#/c/165652/1 ? | 20:27 |
SpamapS | ITP: liblog-any-adapter-screencoloredlevel-perl ... #OhPerl | 20:28 |
anteaya | jogo: no, I just hadn't gotten my head around wether more jobs needed to be removed or not | 20:29 |
anteaya | jogo: I have no argument with the patch | 20:29 |
openstackgerrit | Min Pae proposed openstack-infra/project-config: Add openstack-cue to gerritbot https://review.openstack.org/158011 | 20:29 |
anteaya | let me finish up what I'm doing and look again | 20:29 |
openstackgerrit | Min Pae proposed openstack-infra/system-config: adding openstack-cue to eavesdrop https://review.openstack.org/166033 | 20:30 |
jogo | anteaya: thanks | 20:30 |
jroll | fungi: whenever you have a few, I'd love to check in about the ironic etherpad again... I don't think it's a big deal if we lose history, fwiw (no rush, just following up) | 20:30 |
*** ghostpl_ has quit IRC | 20:30 | |
fungi | jroll: yep, i'm partway into seeing what can be recovered, just got sidetracked by food and a completely broken world courtesy of new tox release | 20:31 |
ianw | clarkb / someone : are we in a position to deploy https://review.openstack.org/164036 (Generate nodepool logging configuration) & https://review.openstack.org/164904 (Add nodepool config check) ? | 20:31 |
jroll | fungi: yep, no worries, thanks :) | 20:31 |
NobodyCam | fungi: awesome thank you :)!!! | 20:32 |
sdague | jeblair: yeh, I see that concern as well. Anyway, I think people are going to be less motivated to fix the docs if they get baked into the jobs instead of can be postdated based on realizing that a particular bug report from a month ago had bad information in there. | 20:32 |
mtreinish | samueldmq: thanks for the review, I responded inline. They were mostly different formatting choices that I did on purpose. | 20:32 |
openstackgerrit | Merged openstack-infra/os-loganalyze: fix path matching redux https://review.openstack.org/166960 | 20:33 |
sdague | I no longer have a hugely strong opinion here. So feel free to move forward however folks are happy with it | 20:33 |
clarkb | ianw: if others think that the hpcloud madness is past then yes I am ok with moving forward with that | 20:34 |
sdague | fungi: https://review.openstack.org/166997 is probably useful down the road | 20:34 |
jeblair | sdague: thanks. you may get your wish anyway, since jhesketh already wrote what you want. the troubling thing for me is that he did so after i thought we agreed otherwise, and that we were unable to effectively discuss this at the meeting. | 20:34 |
fungi | JayF: there are ~40 bare-trusty nodes currently building from the broken (now deleted) image, so in a few minutes those should hopefully get used and then it will be safe to recheck your changes. give it maybe 30 more minutes at the outside just to be safe (but 10 is probably fine) | 20:34 |
jeblair | sdague, clarkb, jhesketh: in the future, i will try to be more clear when i think we have actually agreed to something at a meeting. | 20:35 |
jeblair | sdague: i would find it a great help if you could be available at least for the portion of the infra meeting that relates to the swift log effort | 20:35 |
*** crc32 has joined #openstack-infra | 20:36 | |
*** ociuhandu has quit IRC | 20:36 | |
clarkb | crinkle: you should -1 163540 :) | 20:37 |
*** andreykurilin__ has quit IRC | 20:37 | |
*** andreykurilin_ has joined #openstack-infra | 20:37 | |
sdague | the infra meeting has typically ended up at a pretty bad time for me, given the 2 hr meeting block after. I'd also be really happy to do this via email, because it seems like most of it could be hashed out there async. | 20:37 |
*** Swami has joined #openstack-infra | 20:38 | |
*** Hal1 has quit IRC | 20:38 | |
sdague | fungi: fwiw - here is the output of that command - http://paste.openstack.org/show/195557/ | 20:38 |
*** tsg has quit IRC | 20:38 | |
fungi | sdague: that's checking just what's directly in global-requirements.txt or transitive deps too? | 20:39 |
sdague | fungi: only what's in global-requirements.txt | 20:39 |
clarkb | mtreinish: can ou weigh in on 160830 | 20:39 |
fungi | still, it's a good start | 20:39 |
sdague | the next stage drill down would be harder | 20:39 |
fungi | agreed, you basically have to install those things at the moment to find out | 20:39 |
sdague | right | 20:40 |
sdague | this does it entirely off metadata | 20:40 |
*** fcorral_hp has quit IRC | 20:40 | |
sdague | which makes it pretty quick, at least for narrow 'since' windows | 20:40 |
pabelanger | pew... finally got nodepool doing dib.... even ran into: https://bugs.launchpad.net/diskimage-builder/+bug/1326973 | 20:41 |
openstack | Launchpad bug 1326973 in diskimage-builder "executable bit not preserved when dib is installed via pip on ubuntu precise" [High,Won't fix] - Assigned to Ghe Rivero (ghe.rivero) | 20:41 |
sdague | 15s for -s 7 (the last 7 days) | 20:41 |
mtreinish | clarkb: sure I'll take a look right now | 20:41 |
*** hyakuhei has quit IRC | 20:41 | |
vipuls | hey, as an owner of a channel, how do i provide access to openstackinfra | 20:41 |
openstackgerrit | David Kranz proposed openstack-infra/project-config: Create new periodic job to run tempest without admin creds https://review.openstack.org/166964 | 20:41 |
jeblair | sdague: it's fine if you choose not to attend the meeting. if you would like to provide input on a subject we discuss at the meeting, please do so in a way that we can process in the meeting. and expect that we may sometimes make decisions in the meeting itself. | 20:41 |
vipuls | I ran /msg chanserv access #channel add openstackinfra +AFRefiorstv -- says unauthorized | 20:41 |
samueldmq | mtreinish, k thanks for your replies, I re-reviewed | 20:41 |
*** tonytan4ever has joined #openstack-infra | 20:42 | |
samueldmq | mtreinish, +1ed, that patch should not be hold by those nits | 20:42 |
fungi | vipuls: sounds like you're not the channel owner | 20:42 |
*** soren has quit IRC | 20:42 | |
vipuls | fungi: nvm -- got it working | 20:42 |
vipuls | thx | 20:42 |
sdague | jeblair: sure, will do | 20:42 |
fungi | vipuls: did you literally use "#channel" or did you put in your actual channel name there? | 20:42 |
vipuls | lol ;) | 20:43 |
fungi | got it ;) | 20:43 |
vipuls | fungi: i realized that as soon as i pasted it here | 20:43 |
samueldmq | mtreinish, I see your ascii art diagram (in the build) exactly how you defined at the change, do you expected that to be different? | 20:44 |
*** peristeri has quit IRC | 20:44 | |
samueldmq | mtreinish, I am asking because I saw your comment in there saying it was completely screwed up | 20:44 |
mtreinish | samueldmq: yeah it was supposed to be a nice right angle to the subunit worker | 20:44 |
samueldmq | mtreinish, hmm .. k, will wait for next patchset, thanks | 20:45 |
morganfainberg | fungi, clarkb, you guys LP wizards? I need help | 20:45 |
pleia2 | vipuls: now that you've added the account, you can leave a recheck comment in https://review.openstack.org/#/c/158011/ | 20:45 |
morganfainberg | fungi, clarkb, how do I configure a security team for a project? | 20:45 |
fungi | morganfainberg: we have some admin access at least. whatcha need? | 20:45 |
vipuls | pleia2: yep - will do | 20:46 |
morganfainberg | fungi, keystonemiddleware seems to be misconfigured, which means... we have other issues with other projects i'm sure | 20:46 |
morganfainberg | in keystone that is | 20:46 |
pleia2 | vipuls: actually, still and issue with the change, commenting | 20:46 |
*** tmcpeak has joined #openstack-infra | 20:46 | |
fungi | morganfainberg: as in specify what group can see private security bugs, or to let the vmt know which group to subscribe to them? we usually just do the latter | 20:46 |
tmcpeak | hey all, can somebody take a look at this? https://review.openstack.org/#/c/166999/ we're trying to add permission to annotate a tag, and Gerrit is doing some fresh strangeness | 20:46 |
*** mwagner_lap has quit IRC | 20:47 | |
morganfainberg | fungi, well for example keystone core (keystone-drivers) can't see the security bugs unless explicitly subscribed | 20:47 |
tmcpeak | it says the patch is in merge conflict, but it really shouldn't be | 20:47 |
*** jaypipes has quit IRC | 20:47 | |
*** mrmartin has quit IRC | 20:47 | |
fungi | morganfainberg: i thought that's how it was on most projects | 20:47 |
*** Rockyg has quit IRC | 20:47 | |
morganfainberg | fungi, and i know at least w/ keystone (server) the VMT team can see bugs and are notifified because they always add the OSSA/OSSN incomplete as soon as a security bug is filed | 20:47 |
morganfainberg | fungi, afaik only *I* can see the security bugs for ksm unless i subcribe people | 20:48 |
jeblair | tmcpeak: that's now how me modify acls in gerrit, instead, see http://docs.openstack.org/infra/manual/creators.html#adding-the-repository-to-the-ci-system | 20:48 |
morganfainberg | fungi, i'd like it to mirror keystone proper and keystoneclient | 20:48 |
morganfainberg | fungi, and since i'm going to be adding keystoneauth soon, same there. ;) don't want to be the single point of "OMG SECURITY" for the project... | 20:48 |
fungi | morganfainberg: got it. i haven't dug too deep into the configuration around that in lp, but i'll see what is visible to me that might control it | 20:48 |
tmcpeak | jeblair: ahh ok, thank you | 20:49 |
fungi | morganfainberg: also ttx might know | 20:49 |
sdague | fungi: anyway, calling it a day shortly. You have any thoughts on dealing with the tox thing? | 20:49 |
sdague | I know it bounced a high priority revert in nova today | 20:49 |
morganfainberg | fungi, awesome. i'll bug ttx tomorrow during 1:1 if we don't see things when you take a gander | 20:49 |
fungi | sdague: i've deleted today's nodepool images so it should stop biting us until tomorrow. hoping holger fixes it with a 1.9.2 in the meantime since the bug's been reported to him for hours now | 20:49 |
*** vipuls is now known as vipul | 20:49 | |
clarkb | fungi: it will probably take holger months | 20:49 |
fungi | ugh | 20:50 |
clarkb | we should consider a pin to 1.9.0 | 20:50 |
*** MarkAtwood has quit IRC | 20:50 | |
*** tiswanso has quit IRC | 20:50 | |
fungi | clarkb: well, in this case it's an extremely serious regression introduced today in a point release. i have to expect that he's keeping tabs on that | 20:50 |
fungi | not just a regression, but one which is almost certainly not the behavior even he intended | 20:51 |
sdague | fungi: ok cool | 20:51 |
sdague | thanks | 20:51 |
clarkb | I mean imo the 1.7.2 thing was also an extremely serious regression but they are pretty bad at dealing with them | 20:51 |
sdague | fungi: when a docs job completes, how do we see the logs? | 20:51 |
tmcpeak | jeblair: we won't have one here, we're not in requirements yet | 20:51 |
clarkb | sdague: s/docs-draft/logs/ | 20:51 |
fungi | sdague: you can construct the log url by | 20:51 |
fungi | yeah, what clarkb said | 20:51 |
clarkb | tmcpeak: every project has one... | 20:52 |
tmcpeak | clarkb: we aren' t a legit project yet | 20:52 |
sdague | oh... so pbr freeze doesn't display tox version? | 20:52 |
*** jaypipes has joined #openstack-infra | 20:52 | |
fungi | tmcpeak: http://git.openstack.org/cgit/openstack-infra/project-config/tree/gerrit/acls/stackforge/bandit.config | 20:52 |
sdague | http://logs.openstack.org/61/154761/4/check/gate-nova-docs/205cce9/console.html | 20:52 |
fungi | sdague: not really since tox is installed outside the venv | 20:52 |
tmcpeak | fungi: ahhh stackforge! | 20:53 |
morganfainberg | sdague, it can't know since you don't know wh... what fungi said | 20:53 |
*** MarkAtwood has joined #openstack-infra | 20:53 | |
sdague | fungi: ok, I guess it might be nice for the jobs to record it somewhere | 20:53 |
clarkb | mtreinish: see comment on 166993 | 20:53 |
jeblair | sdague, clarkb, fungi: https://bitbucket.org/hpk42/tox/issue/228/tox-doesnt-work-with-older-pip-versions | 20:53 |
dhellmann | sdague: sorry, stepped out, what was it you wanted to ask me? | 20:53 |
fungi | yep, that's the one i pasted earlier | 20:53 |
jeblair | sdague, clarkb, fungi: see the last comment from hgk there | 20:53 |
morganfainberg | sdague, ++ yes that would be good info, isn't it recored somewhere though? | 20:53 |
sdague | morganfainberg: not that I can see | 20:53 |
morganfainberg | sdague, /me wants to say it is explicitly installed so that should be in the install info | 20:53 |
mtreinish | clarkb: we will be testing it upstream, with a periodic/experimental job | 20:54 |
fungi | jeblair: thanks i'll give that a shot and update the bug | 20:54 |
morganfainberg | i might be totally wrong though | 20:54 |
clarkb | mtreinish: another question for you then | 20:54 |
sdague | ok, thanks all, enjoy the evening | 20:54 |
jeblair | sdague: g'night | 20:54 |
morganfainberg | sdague, have a good one! | 20:54 |
mtreinish | clarkb: sure, shoot | 20:54 |
clarkb | mtreinish: how is that different than tempest just always testing without admin creds when it doesn't need admin rights? | 20:54 |
clarkb | mtreinish: eg shouldn' this just be a default? | 20:54 |
*** baoli has joined #openstack-infra | 20:55 | |
adam_g | console.html has tox version logged at least | 20:55 |
*** fesp has joined #openstack-infra | 20:56 | |
fungi | jeblair: oh, that was issue 228... not our issue since we're running into this _with_ pip 6.x | 20:56 |
JayF | There are two changes (nova-docker and solum) at the top of status.openstack.org/zuul ... both are queued waiting on tempest f20 jobs ... there aren't fedora 20 images anymore though, right? | 20:57 |
fungi | jeblair: issue 230 is the one we just ran into | 20:57 |
mtreinish | clarkb: well for 2 reasons, 1 we still need to default to tenant isolation (which requires admin to create user/tenants) to run in parallel, and 2 the assumption for the gate has been that tempest will be able to run admin things | 20:57 |
clarkb | mtreinish: right, so use admin creds for those things and not admin for the rest | 20:57 |
fungi | jeblair: though maybe it's also fixed in that dev tarball of tox, so worth giving a shot regardless | 20:57 |
clarkb | mtreinish: basically an auth handoff sort of deal so that you cover both at the same time | 20:57 |
*** hichihara has joined #openstack-infra | 20:58 | |
mtreinish | clarkb: tempest does that internally, which is why we want to have a job which runs without admin to verify that we don't regress | 20:58 |
clarkb | mtreinish: doesn't that cover it itself? | 20:58 |
*** _nadya_ has joined #openstack-infra | 20:58 | |
clarkb | mtreinish: I guess I don't understand how what I describe is any different than an admin less job other than an admin less job would have to have an admin set stuff up before hand | 20:58 |
mtreinish | clarkb: except we don't have a job anywhere which runs without admin being set | 20:58 |
mtreinish | so it'll always run everything | 20:58 |
*** MarkAtwood has quit IRC | 20:59 | |
mtreinish | which is all we're trying to add here | 20:59 |
jeblair | fungi, clarkb: oh wow i love that it's 2 issues. but yeah, def worth testing since he _does_ seem poised to release something called 1.3.2 | 20:59 |
*** baoli has quit IRC | 20:59 | |
*** annegentle has quit IRC | 20:59 | |
*** baoli has joined #openstack-infra | 21:00 | |
mtreinish | clarkb: fwiw, if you're curious how you can configure creds with tempest take a look at: http://docs.openstack.org/developer/tempest/configuration.html#auth-credentials | 21:00 |
clarkb | mtreinish: I guess I don't understand how a admin less job is different then | 21:01 |
tmcpeak | thanks all, got it sorted. Infra rocks! | 21:01 |
clarkb | mtreinish: you would be doing the same setup for it that tempest normally does itself? | 21:01 |
clarkb | mtreinish: so the thing to test doesn't regress is what we are already testing which is the setup within tempest | 21:02 |
*** jamielennox|away is now known as jamielennox | 21:02 | |
*** _nadya_ has quit IRC | 21:02 | |
openstackgerrit | Vipul Sabhaya proposed openstack-infra/project-config: Add openstack-cue to gerritbot https://review.openstack.org/158011 | 21:03 |
fungi | JayF: there are still f20 workers being built, though there may not be any available at the moment | 21:03 |
fungi | JayF: and it's possible something has broken our ability to boot them yet again | 21:03 |
JayF | fungi: aha. I don't know/care personally, but remembered seeing that in here + something that looked anomolous on the status page | 21:03 |
clarkb | jogo: did you want to give a go at putting the cpu mapping in 163514 in place? | 21:04 |
mtreinish | clarkb: what setup are you talking about? | 21:04 |
clarkb | mtreinish: the tenant setup and anything else that requires admin creds | 21:04 |
mtreinish | clarkb: the primary motivator for this is to ensure we don't add tests that explicitly require admin but we don't properly skip gracefully if no admin is provided | 21:04 |
*** Slower has left #openstack-infra | 21:04 | |
mtreinish | because right now if you don't give tempest admin it will fail a set of tests which were poorly written | 21:04 |
fungi | JayF: yeah, early last week a bug in fedora caused us to stop being able to boot f20 nodes, and a workaround was implemented | 21:04 |
mtreinish | clarkb: https://review.openstack.org/#/c/166955/ | 21:05 |
clarkb | mtreinish: oh you would skipping tests not expecting setup to be done for them | 21:05 |
*** samueldmq is now known as samueldmq-away | 21:06 | |
mtreinish | clarkb: yeah, it's a valid configuration to run tempest without tenant isolation (the automatic tenant user creation in setup class) | 21:06 |
mtreinish | so we want to make sure that doesn't break | 21:06 |
fungi | jeblair: by the time i was done testing, holger uploaded 1.9.2. i think it's still broken for us though | 21:07 |
nibalizer | pleia2: we got a shoutout https://training.linuxfoundation.org/images/lftc_evolution_sysadmin.jpg :) | 21:07 |
clarkb | fungi: hahahaha | 21:07 |
fungi | testing now to confirm | 21:08 |
* JayF tests tox 1.9.2 as well | 21:08 | |
pleia2 | nibalizer: \o/ | 21:08 |
openstackgerrit | Monty Taylor proposed openstack-infra/project-config: Add elements for Infra servers https://review.openstack.org/140840 | 21:09 |
*** radez is now known as radez_g0n3 | 21:09 | |
*** cdent has quit IRC | 21:09 | |
*** afazekas_ has joined #openstack-infra | 21:09 | |
mordred | greghaynes, SpamapS: ^^ also - that now depends on that other dib change | 21:09 |
*** claudiub has quit IRC | 21:09 | |
morganfainberg | fungi, so, looks like i have keystone-bugs set as the bug supervisor, but it appears to not have an affect for the keystonemiddleware project... it's just weird. *shrug* i'll bug ttx about it tomorrow. | 21:09 |
fungi | jeblair: clarkb: JayF: no, fixed! | 21:09 |
clarkb | mtreinish: right so thats the case I don't see us breaking ever | 21:09 |
JayF | fungi: wfm as well | 21:09 |
clarkb | mtreinish: because we are already testing the code that would regress. the setup code | 21:09 |
fungi | jeblair: clarkb: JayF: also he updated the relevant issue report indicating as much | 21:09 |
clarkb | mtreinish: but if the intent is to catch thats that fail when that setup is not run thats different | 21:10 |
*** aysyd has quit IRC | 21:10 | |
fungi | so when tomorrow's images build, we should be in the clear (or get to discover what else just broke in 1.9.2 i guess?) | 21:10 |
jeblair | fungi: just a regular day then! :) | 21:10 |
mtreinish | clarkb: right, the tenant isolation code has never broken (at least from what I can remember) this is for the case outside of that | 21:10 |
fungi | yep! unwinding my stack of issues by one pop | 21:10 |
fungi | morganfainberg: actually looking at your lp question now | 21:11 |
*** tmcpeak has left #openstack-infra | 21:11 | |
morganfainberg | fungi, yeah figured i'd update you with what I found. but in short, looks like bug-supervisor should allow people to see these things and for ksm it isn't | 21:11 |
* morganfainberg wonders if a typo snuck in somehow. | 21:11 | |
openstackgerrit | Monty Taylor proposed openstack-infra/project-config: Announce diskimage-builder in infra channel https://review.openstack.org/167011 | 21:11 |
clarkb | mordred: ^ why? | 21:12 |
mordred | jeblair, fungi, clarkb: ^^ feel free to veto that - but currently I'd love to not have to paste patch references in here when hacking on dib for infra | 21:12 |
clarkb | mordred: none of us have core on that | 21:12 |
clarkb | and I am pretty sure most of us have never reviewed dib change | 21:13 |
mordred | clarkb: we should work on fixing that -I just moved 100% of the config drive work to an element in dib directly because it makes more sense to live there | 21:13 |
fungi | morganfainberg: are you saying that all 36 members of https://launchpad.net/~keystone-bugs/+members can see private bug reports? | 21:13 |
morganfainberg | fungi, i believe so. | 21:13 |
mordred | clarkb: but, I don't feel strongly about it- it's a strawman patch | 21:13 |
morganfainberg | fungi, well at least with keystone. | 21:13 |
morganfainberg | fungi, keystone-core (keystone-drivers), VMT, and Release-mngmt | 21:13 |
morganfainberg | and people i've never heard of | 21:14 |
morganfainberg | oh and infra | 21:14 |
*** annegentle has joined #openstack-infra | 21:14 | |
morganfainberg | ah that is a service account | 21:15 |
fungi | morganfainberg: if i look at a current private security bug for keystone, the list of "may be notified" subscribers is much smaller | 21:15 |
morganfainberg | right | 21:15 |
morganfainberg | fungi, that is notified of changes (emails) doesn't mean who can access it | 21:15 |
fungi | so i don't think it's anything to do with the bug-supervisor group for the same project | 21:15 |
fungi | oh, perhaps | 21:16 |
fungi | yeah, because you're not in that list | 21:16 |
*** dkranz has quit IRC | 21:16 | |
morganfainberg | yep. | 21:16 |
fungi | yet you've commented on the bug i'm looking at | 21:16 |
morganfainberg | well i have some special set of "i'm PTL" settings somewhere. | 21:16 |
morganfainberg | but i honestly don't know (dolph said he set something for when I became PTL somewhere) | 21:16 |
morganfainberg | but even before i didn't have issues seeing private bugs for keystone (when just core) | 21:17 |
clarkb | mordred: I think my criteria for that is basically post changes for projects that channel denizens are expected to be reviewing | 21:17 |
clarkb | mordred: if you want people to be reviewing dib them cool otherwise -0 | 21:17 |
fungi | morganfainberg: if nothing else, we also have some former lp admins/developers we can bug for details on how that's implemented | 21:17 |
fungi | i'm just _sure_ they love reminiscing about the good old days | 21:18 |
morganfainberg | fungi, right. and we might want to check to see if that keystone-bug list needs to be paired down. | 21:18 |
morganfainberg | fungi, if that is the right group/tool to do that. | 21:18 |
openstackgerrit | Merged openstack-infra/gear: Resolve python3 test failures https://review.openstack.org/158147 | 21:18 |
*** yamamoto_ has joined #openstack-infra | 21:18 | |
jogo | in place? | 21:18 |
fungi | morganfainberg: i didn't previously have the impression that bug-supervisor group members could all see private security bugs, but if it's true it does give pause for thought | 21:19 |
morganfainberg | fungi, well i just don't know what magic setting keystone uses that ksm doesn't | 21:19 |
fungi | morganfainberg: agreed. i'm poking around to see if i can tell | 21:19 |
*** prad has joined #openstack-infra | 21:20 | |
morganfainberg | fungi, so it might be something not keystone-bugs. but even still I don't know a bunch of names on that keystone-bugs list, we might want to pair that down even if they are just able to do normal bug triage | 21:20 |
*** hyakuhei has joined #openstack-infra | 21:20 | |
jeblair | clarkb: see my clarification to your clarification on 165582; and if it's still not clear, perhaps we should have a chat with greghaynes. :) | 21:20 |
morganfainberg | especially if inheritance from say vuln-mngmt and drivers is enough | 21:20 |
jogo | clarkb: did you run a experimental job on your patch? | 21:20 |
clarkb | jogo: my patch doesn't work | 21:21 |
jogo | clarkb: ahh, any ideas why? | 21:21 |
clarkb | jogo: its incompletele. Someone needs to take that cpu mapping and put it in /usr/share/libvirt/cpu_map.xml | 21:21 |
fungi | morganfainberg: i've temporarily added myself to your drivers group in lp to see more options in the configuration ui. i'll remove myself when i'm done | 21:22 |
morganfainberg | fungi, feel free to :) | 21:22 |
jogo | clarkb: ahh any ideas what danpb things of this approach | 21:22 |
clarkb | jogo: no | 21:22 |
clarkb | jeblair: yup I think your comment makes it all clear :) | 21:22 |
*** sdake__ has joined #openstack-infra | 21:22 | |
jeblair | woo i have been useful today! | 21:23 |
*** yamamoto_ has quit IRC | 21:23 | |
clarkb | jogo: I asked in nova channel at one point but got nothing | 21:23 |
fungi | morganfainberg: https://launchpad.net/keystone/+sharing | 21:23 |
morganfainberg | oh god really? | 21:23 |
fungi | morganfainberg: i knew i saw it somewhere before | 21:23 |
* morganfainberg sighs | 21:23 | |
morganfainberg | thanks | 21:23 |
* fungi puts on his ui hater hat | 21:23 | |
* morganfainberg steals said ui hater hat and wears it proudly | 21:23 | |
morganfainberg | thanks | 21:23 |
morganfainberg | that has been bugging me | 21:24 |
jogo | clarkb: he isn't in the nova channel much anymore | 21:24 |
notmyname | I've got a fun* problem (*not fun at all) | 21:24 |
morganfainberg | fungi, really appreciate the help | 21:24 |
anteaya | zaro: so I'm having an issue with the gerrit .gitreview file: http://git.openstack.org/cgit/openstack-infra/gerrit/tree/.gitreview | 21:24 |
*** tkelsey has quit IRC | 21:24 | |
anteaya | zaro: methinks it should be openstack-infra, no? not openstack-ci? | 21:24 |
fungi | morganfainberg: any time. i needed the refresher anyway | 21:24 |
anteaya | zaro: any reason I shouldn't offer a patch to change that? | 21:24 |
notmyname | I'm trying to merge master into a feature branch. requirements.txt has changed, so the requirements job is triggered. but the global reuirements repo doesn't have that feature branch, so it fails every time, preventing the merge commit from landing | 21:25 |
fungi | anteaya: probably but i don't think we ever do anything with the master branch. have a look in some of the openstack branches for it | 21:25 |
notmyname | https://jenkins04.openstack.org/job/gate-swift-requirements/45/console and https://review.openstack.org/#/c/166558/ | 21:25 |
jogo | clarkb: I'll try to take a whack at it later today but if I don't get to it today I can't do it tomorrow as I will be traveling all day | 21:25 |
anteaya | fungi: ah perhaps that is why I was having problems | 21:26 |
clarkb | jogo: unfortunately its an xml file so its less trivial than I would have hopedwhich is why I am curious if you want to give it a go | 21:26 |
*** sdake has quit IRC | 21:26 | |
*** tkelsey has joined #openstack-infra | 21:26 | |
clarkb | notmyname: that job should probabl be updated to treat feature/* as master | 21:26 |
* anteaya tries to remember how to pull in branches locally | 21:26 | |
openstackgerrit | Merged openstack-infra/project-config: Add python3-jobs for gear https://review.openstack.org/158148 | 21:26 |
jogo | clarkb: was hoping there is some magical XML parsing tool to make that easy | 21:26 |
openstackgerrit | Matthew Treinish proposed openstack-infra/system-config: Add docs for subunit2sql https://review.openstack.org/164397 | 21:26 |
*** samueldmq has joined #openstack-infra | 21:26 | |
*** dboik_ has joined #openstack-infra | 21:27 | |
notmyname | clarkb: rigth now it's blocking work. I've done everything I can to sidestep it, but it can't be sidestepped (eg using either master's version or the feature branch version). because they are different, the merge commit sees *something* and triggers the job | 21:27 |
notmyname | clarkb: is there anything I can do to get it unblocked? | 21:27 |
clarkb | notmyname: I am looking to see what needs to change in the job now | 21:28 |
notmyname | thansk :-) | 21:28 |
*** melwitt has quit IRC | 21:28 | |
*** thedodd has quit IRC | 21:28 | |
mtreinish | pleia2: ^^^ that is really weird because it looks like: http://imgur.com/FX9Mh7a in my editor | 21:28 |
fungi | mtreinish: pleia2: inline tabs? | 21:29 |
pleia2 | mtreinish: the other patch actually looked ok to me | 21:29 |
*** dboik has quit IRC | 21:30 | |
anteaya | fungi: that was it thanks | 21:31 |
jeblair | clarkb, fungi: hrmph. we actually talked about how that job should really use zuul-cloner when we patched it to use git.o.o this weekend, but i wanted to unbreak quickly and simply and so did not do that then | 21:31 |
mtreinish | pleia2: the rendered output was all broken though: http://docs-draft.openstack.org/97/164397/3/check/gate-ci-docs/6796b19//doc/build/html/logstash.html#system-architecture | 21:31 |
*** mattfarina has quit IRC | 21:31 | |
mtreinish | fungi: maybe? but I have vim set to auto expand tabs | 21:32 |
pleia2 | mtreinish: that also looks ok to me | 21:32 |
jeblair | pleia2, mtreinish: http://docs-draft.openstack.org/97/164397/3/check/gate-ci-docs/6796b19//doc/build/html/logstash.html#system-architecture lgtm too | 21:32 |
pleia2 | mtreinish: maybe draw on paper and take a picture what it's supposed to look like :D | 21:32 |
clarkb | jeblair: I will push a patch in just a second (also not using zuul cloner) | 21:32 |
fungi | pleia2: damp bar napkin | 21:32 |
clarkb | jeblair: you can look at it and decide if zuul cloner would be better | 21:32 |
openstackgerrit | Clark Boylan proposed openstack-infra/project-config: Treat feature branches as equivalent to master https://review.openstack.org/167014 | 21:33 |
*** afazekas_ has quit IRC | 21:34 | |
clarkb | jeblair: I think in either case we will need images to be rebuilt | 21:34 |
jeblair | clarkb: i believe we need the normal fallback logic. we should test for existence of feature/foo before falling back to master; otherwise projects will not be able to share a feature requirements branch. | 21:34 |
jeblair | clarkb: which, btw, is probably the fastest way to unblock swift (create feature/ec on requirements) | 21:35 |
clarkb | jeblair: I think we want to explicitly never have a feature branch in global requirements though | 21:35 |
clarkb | because again its global and feature == master | 21:35 |
mtreinish | pleia2, jeblair wait really, it looks all kinds of weird for me: http://imgur.com/fRLGsQH | 21:35 |
*** yamahata has quit IRC | 21:35 | |
jeblair | (status quo permits feature/foo on requirements but does not fall back; 167014 falls back but does not permit a shared branch; it inverts the situation) | 21:35 |
clarkb | right, my assertion is a shared branch is wrong in those one case | 21:36 |
pleia2 | mtreinish: your browser seems to not want to render monospaced fonts properly | 21:36 |
*** yamahata has joined #openstack-infra | 21:36 | |
jeblair | clarkb: hrm. it's a thought. :) | 21:36 |
mtreinish | pleia2: oh you know that might be the case. (I barely know how to use a web browser) If you're fine with it looking weird for me I'll revert to the previous rev which looks good in vim | 21:37 |
*** otter768 has joined #openstack-infra | 21:37 | |
pleia2 | mtreinish: wfm | 21:37 |
*** yamahata has quit IRC | 21:37 | |
*** afazekas_ has joined #openstack-infra | 21:37 | |
jeblair | clarkb: however, supporting a shared branch as well as a fallback does not preclude us simply deciding not to create a shared branch | 21:37 |
*** yamahata has joined #openstack-infra | 21:38 | |
*** dustins has quit IRC | 21:38 | |
openstackgerrit | Matthew Treinish proposed openstack-infra/system-config: Add docs for subunit2sql https://review.openstack.org/164397 | 21:38 |
mtreinish | pleia2: ^^^ | 21:38 |
pleia2 | thanks | 21:39 |
*** tsg has joined #openstack-infra | 21:40 | |
*** Swami has quit IRC | 21:40 | |
* mtreinish goes and checks firefox font settings | 21:40 | |
*** melwitt has joined #openstack-infra | 21:41 | |
*** jcoufal has quit IRC | 21:42 | |
mtreinish | pleia2: heh, so it turns out you can tell firefox to use a non monospaced font for monospace text... (which apparently I did) | 21:42 |
*** otter768 has quit IRC | 21:42 | |
*** afazekas_ has quit IRC | 21:43 | |
pleia2 | mtreinish: oh, yeah, don't do that :) | 21:43 |
fungi | mtreinish: wow. i mean, i have a tendency to do the opposite, so i know it's possible, but that's an amusing configuration choice on your part | 21:44 |
openstackgerrit | Clark Boylan proposed openstack-infra/project-config: Fallback to master on requirements check job https://review.openstack.org/167014 | 21:44 |
clarkb | jeblair: ^ is the alternative | 21:44 |
mtreinish | fungi: as far as I can remember I never set that, I'm going to blame arch defaults | 21:44 |
*** rlandy has quit IRC | 21:45 | |
*** weshay has quit IRC | 21:45 | |
openstackgerrit | Merged openstack-infra/devstack-gate: Fix inconsisten Ubuntu version in README.rst https://review.openstack.org/161922 | 21:45 |
fungi | mtreinish: just keep reminding yourself it's a great distro | 21:45 |
*** ldnunes has quit IRC | 21:45 | |
pleia2 | fungi: me too, was unhealthily excited with the ubuntu mono font was released | 21:45 |
*** hodos has quit IRC | 21:46 | |
*** baoli has quit IRC | 21:48 | |
asselin | can I get reviews on this? 5 lines & will help 3rd party folks: https://review.openstack.org/#/c/164889/ | 21:48 |
*** nelsnelson has quit IRC | 21:49 | |
fungi | pleia2: i just wish https://packages.debian.org/sid/fonts-opendyslexic had a proper mono variant | 21:49 |
clarkb | asselin: is that tested on precise as well? | 21:49 |
mtreinish | fungi: that just means you need to make it :) | 21:50 |
fungi | mtreinish: yep. we'll see if i get some free time this weekend | 21:50 |
pleia2 | fungi: ooh, I hadn't seen that before, yeah mono that would help my brain | 21:50 |
asselin | clarkb, yes | 21:50 |
clarkb | droid is a decent font exept for O and 0 looking identical | 21:50 |
*** dprince has quit IRC | 21:50 | |
pleia2 | clarkb: yeah | 21:51 |
*** pc_m has quit IRC | 21:51 | |
JayF | Hey; I have a question about adding a new project: Some projects (in gerrit/projects.yaml) have an "acl-config:" value whereas others don't. Is this something I need and is there docs on what to set it to? | 21:52 |
pleia2 | JayF: the ones that don't have it automatically figure it out based on project name | 21:53 |
fungi | pleia2: right now if i try to use opendyslexic in xterm, it thinks all glyphs are east asian width (two-cell), so i use vl gothic instead (which is not a mono font but works well in an xterm anyway) | 21:53 |
pleia2 | JayF: the ones that do have it, point to an acl other than the name of their project (so, one typically shared by a bunch of projects) | 21:53 |
openstackgerrit | James E. Blair proposed openstack-infra/project-config: Use zuul-cloner in project-requirements-change https://review.openstack.org/167019 | 21:53 |
JayF | pleia2: aha, so if, for isntance, my acl was pointing almost entirely at groups for another project, I might want to drop the acl I add and instead do acl-config: /home/gerrit2/acls/openstack/ironic-python-agent.config ? | 21:54 |
JayF | (where IPA is the project whose acls I want to copy) | 21:54 |
*** sdake__ has quit IRC | 21:54 | |
jeblair | clarkb: i think your PS1 is okay, PS2 has a problem which i also had to address in 167019 | 21:54 |
*** asselin_ has joined #openstack-infra | 21:54 | |
*** jcoufal has joined #openstack-infra | 21:54 | |
*** hyakuhei has quit IRC | 21:55 | |
pleia2 | JayF: yeah, as long as you're ok with any changes that other project may make to their acls, I assume your project is related to ironic? | 21:55 |
openstackgerrit | James E. Blair proposed openstack-infra/project-config: Use zuul-cloner in project-requirements-change https://review.openstack.org/167019 | 21:55 |
JayF | pleia2: very, yes. | 21:55 |
pleia2 | JayF: yeah, so that makes sense :) | 21:55 |
openstackgerrit | Jay Faulkner proposed openstack-infra/project-config: Add coreos-image-builder project https://review.openstack.org/155868 | 21:55 |
JayF | pleia2: ^ mind looking at that to make sure it makes sense? | 21:55 |
jeblair | clarkb: so i think the actual change to use zuul-cloner is equally simple; however if feature branches are supported, i think they should probably have include_dev set. | 21:56 |
clarkb | jeblair: ya | 21:56 |
openstackgerrit | James E. Blair proposed openstack-infra/project-config: Use zuul-cloner in project-requirements-change https://review.openstack.org/167019 | 21:57 |
jeblair | typos | 21:57 |
*** sdake has joined #openstack-infra | 21:57 | |
*** tkelsey has quit IRC | 21:57 | |
jeblair | clarkb: so i think we should do either your PS1 or zuul-cloner | 21:57 |
clarkb | jeblair: lets go ahead and do zuul-cloner since that was a want anyways | 21:57 |
clarkb | also it iwll use the cache | 21:58 |
jeblair | ya | 21:58 |
*** asselin_ has quit IRC | 21:58 | |
fungi | pleia2: oh, looks like they added a mono variant to opendyslexic a few months back, but the package in debian is just a bit outdated | 21:59 |
jeblair | notmyname: can you wait until tomorrow? if not, we can probably kick off an image build and have it in an hour. and if you can't wait that long we could consider adding a feature/ec branch to requirements for a few hours and then delete it. | 21:59 |
clarkb | pleia2: I just use droid anyways. 'ga' as a command in vim is handy at times :) | 21:59 |
openstackgerrit | Merged openstack-infra/system-config: Log server can run on Apache 2.4 / Ubuntu Trusty https://review.openstack.org/164889 | 22:00 |
fungi | clarkb: jeblair: i'm more in favor of the zuul-cloner direction still | 22:00 |
pleia2 | JayF: I'm going to dig up some docs to be sure, but group may imply an acl | 22:00 |
notmyname | jeblair: let me check. gut reaction says tomorrow is ok, but this is a key week to get the last ec stuff done for kilo. every moment counts. let me see if anything is depending on the master changes | 22:00 |
pleia2 | JayF: s/group/groups: | 22:00 |
JayF | pleia2: We have another project that is in group: ironic but has it's own ACL | 22:00 |
JayF | pleia2: see ironic-python-agent in the same file :) | 22:00 |
JayF | pleia2: and I know the correct ACL applies because I'm core on IPA but not Ironic :) | 22:01 |
jeblair | fungi: want to review 167019? | 22:01 |
*** dannywilson has quit IRC | 22:01 | |
fungi | jeblair: just finished actually | 22:01 |
*** timcline has quit IRC | 22:02 | |
notmyname | jeblair: yes. tomorrow is fine. | 22:02 |
jeblair | notmyname: cool, thanks. sorry for the snag. | 22:02 |
notmyname | jeblair: no worries. thanks for fixing it (+ clarkb) | 22:03 |
jeblair | notmyname: btw, if you create a new feature branch to clean up the ec feature branch, i assume you'll call it "feature/ec2" ? | 22:03 |
jhesketh | Morning | 22:03 |
notmyname | jeblair: of course. with 3 numeric milestones called "sN" | 22:04 |
jeblair | makes perfect sense | 22:04 |
jeblair | i hadn't thought about that, but now i really hope the release/milestone cycle stays around at least for the "S" release. | 22:05 |
notmyname | heh | 22:05 |
*** melwitt has quit IRC | 22:05 | |
*** ominakov has quit IRC | 22:06 | |
*** yamamoto_ has joined #openstack-infra | 22:06 | |
*** sigmavirus24 is now known as sigmavirus24_awa | 22:07 | |
*** melwitt has joined #openstack-infra | 22:08 | |
*** armax has quit IRC | 22:08 | |
*** julim has quit IRC | 22:09 | |
jeblair | jhesketh: let me know when you have a sec to talk about 166596 | 22:10 |
openstackgerrit | Merged openstack-infra/project-config: Remove grenade jobs from projects that grenade doesn't test https://review.openstack.org/165652 | 22:14 |
*** andreykurilin_ has quit IRC | 22:15 | |
morganfainberg | fungi, ping - can i ask you for more super powers help w/ LP | 22:15 |
*** bhunter71 has quit IRC | 22:15 | |
openstackgerrit | Merged openstack-infra/project-config: Use zuul-cloner in project-requirements-change https://review.openstack.org/167019 | 22:16 |
morganfainberg | fungi, i'd like to make https://launchpad.net/python-keystoneclient maintained by keystone-drivers if that is ok | 22:16 |
openstackgerrit | Merged openstack-infra/project-config: Add identity v3 jobs to periodic-qa https://review.openstack.org/166593 | 22:16 |
*** baoli has joined #openstack-infra | 22:17 | |
*** dimtruck is now known as zz_dimtruck | 22:17 | |
*** thedodd has joined #openstack-infra | 22:18 | |
*** dkliban is now known as dkliban_afk | 22:18 | |
*** thedodd has quit IRC | 22:18 | |
fungi | morganfainberg: done. since that group is maintained by openstack-admins we can still add/remove ourselves from your drivers group like i did earlier if we need to tweak something on the project itself | 22:19 |
morganfainberg | fungi, exactly | 22:19 |
morganfainberg | fungi, i'm trying to convert security stuff over to a more restrictive group, couldn't change ksc because drivers wasn't the maintainer | 22:19 |
*** amitgandhinz has quit IRC | 22:20 | |
*** unicell1 has joined #openstack-infra | 22:21 | |
*** unicell has quit IRC | 22:21 | |
fungi | morganfainberg: (vmt hat on) appreciated. ideally keep near 0 people on the sharing list besides the vmt. we do sometimes get vulnerability reports obviously misfiled against the wrong projects and the vmt can redirect those without it being obvious to a lot of people on the first project | 22:21 |
morganfainberg | fungi, thats the idea. | 22:21 |
*** ominakov has joined #openstack-infra | 22:22 | |
ianw | fungi: did you check the f20 nodes are booting? as mentioned previously some jobs seem stuck waiting for nodes for a long time | 22:22 |
morganfainberg | fungi, so do you want me to have OS-VMT inherit from the keystone-sec group, or have explicit sharing? | 22:22 |
morganfainberg | fungi, i'm leaning towards the former just for sanity-sake | 22:22 |
morganfainberg | fungi, so it's less groups to remember to share with | 22:22 |
ianw | also maybe something got messed up in the nodepool allocator | 22:23 |
fungi | morganfainberg: then if it's likely a vulnerability against that actual project, we pretty much immediately subscribe keystone-coresec where you should maintain a broader list of trusted security-focused devs and reviewers to help triage it, at which point they can in turn subscribe more specific subject matter experts as needed | 22:23 |
*** gordc has quit IRC | 22:23 | |
morganfainberg | fungi, ok so do you want keystone-coresec always subscribed or just VMT? | 22:24 |
morganfainberg | fungi, since i'm doing the whole "fix this ick atm"... cause it is ick | 22:24 |
fungi | morganfainberg: maybe just vmt and the ptl | 22:24 |
morganfainberg | fungi, ok let me work on fixing these things then. | 22:25 |
fungi | morganfainberg: we don't really have a hard-and-fast rule about it though. projects are to a great extent encouraged to police that and decide for themselves. we're simply here to help | 22:25 |
openstackgerrit | Merged openstack-infra/project-config: Promote lbaasv2 api tempest tests to voting for neutron and neutron-lbaas https://review.openstack.org/166038 | 22:25 |
morganfainberg | fungi, hm.. i'm inclined to say keystone-coresec is always involved, but i expect the list to be PTL, and 1 or two others | 22:26 |
morganfainberg | and we can loop in drivers/etc as needed | 22:26 |
fungi | k | 22:26 |
morganfainberg | oh | 22:27 |
fungi | ianw: looks like we have 46 building, 9 deleting and 1 used. my guess is that a recent image update broke f20 builds yet again, and one of our 8 providers has a stale image which still boots | 22:27 |
morganfainberg | hah, someone already used keystone-coresec | 22:27 |
clarkb | fungi: see comment on 165660 | 22:27 |
morganfainberg | hahaha /me had created another group. | 22:27 |
morganfainberg | ugh | 22:27 |
openstackgerrit | Merged openstack-infra/project-config: For neutron and neutron-lbaas, skip more wasted jobs https://review.openstack.org/166035 | 22:27 |
morganfainberg | ok let me remove that other group... | 22:27 |
fungi | morganfainberg: we renamed the project-core groups in lp to project-coresec a while back to reflect how we'd been using them | 22:27 |
morganfainberg | right | 22:28 |
morganfainberg | ah coresec is already populated | 22:28 |
clarkb | dougwig: 165648 should interst you | 22:28 |
ianw | fungi: well i'm going through image.log and afaics builds are looking ok... | 22:28 |
morganfainberg | i'll fix all the project perms in a few. | 22:28 |
morganfainberg | fungi, thanks. [ah looks like dolph is admin of coresec for keystone[] | 22:29 |
fungi | clarkb: 165660 is the minimum to get the job passing. the real index is in 165199 | 22:29 |
clarkb | fungi: why move the readme then? | 22:29 |
dolphm | morganfainberg: oops, what can i do? | 22:29 |
fungi | clarkb: i renamed the readme for consistency while boilerplating | 22:30 |
*** baoli has quit IRC | 22:30 | |
morganfainberg | dolphm, oh you can change who is in the keystone-coresec group. iirc you were delegated that role | 22:30 |
*** dboik_ has quit IRC | 22:30 | |
morganfainberg | dolphm, i'm just fixing a bunch of wonky/bad security permissions in our projects right now | 22:30 |
clarkb | fungi: ok, its really confusing to have that done because I expect the readme to get rendered by the build but it isn't | 22:30 |
*** dboik has joined #openstack-infra | 22:30 | |
dolphm | morganfainberg: fixed | 22:31 |
fungi | clarkb: smarcet and the rest of the tipit crew wanted to be able to get docs-draft rendering and publication working, but had already written the change with the content. i'm trying to encourage them to rebase onto it (or squash them) | 22:31 |
*** prad has quit IRC | 22:31 | |
clarkb | ok I can remove my -1 | 22:32 |
clarkb | fungi: maybe we can put the readme back? | 22:32 |
*** dboik has quit IRC | 22:32 | |
fungi | i'm happy to not rename the readme until later. just wanted to get their existing files not touched by the other change into rst for consistency | 22:32 |
clarkb | ya I think that would make the change a bit more consistent. Otherwise you just end up with no readme | 22:33 |
fungi | how is there no readme? did i forget to git add it? | 22:33 |
clarkb | anways I will remove the -1 and suggest that | 22:33 |
*** salv-orl_ has joined #openstack-infra | 22:34 | |
*** salv-orlando has quit IRC | 22:34 | |
clarkb | fungi: you deleted readme.md and the README.rst is not rendered | 22:34 |
clarkb | fungi: because your index.rst is empty | 22:34 |
clarkb | which is why I -1'd | 22:34 |
fungi | and readme.md was rendered? i'm not understanding | 22:34 |
clarkb | nothing was rendered | 22:34 |
fungi | and nothing is still rendered | 22:35 |
clarkb | so the readme contents are effectively gone | 22:35 |
fungi | i'm missing the gone part | 22:35 |
clarkb | they are neither at repo top level nor part of the documentation | 22:35 |
fungi | but they are, as README.rst instead of readme.md | 22:35 |
clarkb | fungi: your change deletes readme.md and does not replace it | 22:35 |
clarkb | fungi: but that file is not rendered | 22:35 |
clarkb | fungi: so its a noop | 22:35 |
fungi | uh | 22:35 |
clarkb | the normal user would not find it | 22:35 |
fungi | readme.md wasn't rendered either | 22:35 |
clarkb | fungi: it is on the hosting thouhg | 22:36 |
fungi | on what hosting? | 22:36 |
clarkb | its at the root of the repo and anyone looking for it will find it | 22:36 |
clarkb | with your change no one will find it | 22:36 |
*** mfink_ has quit IRC | 22:36 | |
fungi | i'm clearly missing something | 22:36 |
clarkb | fungi: your index.rst needs to have a toctree and includes to render stuff | 22:36 |
fungi | people find readme.md files but not README.rst files even if they're in the same directory? | 22:36 |
clarkb | OH! you didn't move the file | 22:37 |
clarkb | ok thats what I was msising for some reason I had read that the README.rst went in doc/source | 22:37 |
fungi | i did but since i also changed the content formatting from md to rst git didn't see it as a move | 22:37 |
fungi | it saw it as an add/delete | 22:37 |
fungi | even though i used git mv and then edited it | 22:37 |
clarkb | ya my confusion was that I thought the file was in the doc/source dir with index.rst | 22:37 |
clarkb | so why rename it at all? | 22:38 |
clarkb | maybe thats the question I should be asking | 22:38 |
*** salv-orl_ has quit IRC | 22:38 | |
fungi | because the rest of the project documentation is/is going to be in rst once smarcet's change lands | 22:38 |
greghaynes | clarkb: jeblair is correct re: 165582 (sorry, I hadnt fully paged in the mental context for that change when we talked) | 22:38 |
fungi | but that might just be me getting ocd about having two kinds of markdown | 22:39 |
fungi | in one project | 22:39 |
fungi | i can buy that we don't really care if a project mixes rst and md in different files | 22:40 |
morganfainberg | fungi, ok all fixed permission wise now. VMT is subscribed, as am I. | 22:40 |
fungi | morganfainberg: thanks for your attention to detail! | 22:40 |
morganfainberg | fungi, keystone-coresec is not automatically subscribed. open bugs i just added keystone-drivers to open security bugs as they previously had access. so we didn't cut anyone off suddenlyt | 22:40 |
*** jcoufal has quit IRC | 22:40 | |
fungi | clarkb: optionally, i can just update the commit message to say i moved and reformatted the readme for consistency | 22:41 |
fungi | morganfainberg: okie dokie | 22:41 |
clarkb | fungi: because of the change my brain implicitly assumed you would be rendering that file with sphinx | 22:41 |
clarkb | hence my confusion | 22:41 |
clarkb | if you want to change the format thats fine but doing so in the make sphinx work change implies you want to use it with sphinx | 22:41 |
*** HeOS has quit IRC | 22:41 | |
*** salv-orlando has joined #openstack-infra | 22:42 | |
fungi | clarkb: i mean, i added stuff to .gitignore too even though sphinx will technically work without that, and added sphinx modules that the subsequent change will need even though it will run without them | 22:42 |
clarkb | yes but those are all related to sphinx | 22:43 |
clarkb | README.rst is not related at all | 22:43 |
fungi | since getting our normal sphinx toolchain working there implied some basic python packaging boilerplate (even though the project itself is in php), i did some other simple consistency changes along with that | 22:43 |
fungi | but i'll back out the readme move and propose that as a different change later once stuff is working | 22:44 |
jeblair | mtreinish, dtroyer what's a repo with a good example of an in-tree devstack plugin? | 22:46 |
clarkb | jeblair: ec2-api I think | 22:46 |
*** bhunter71 has joined #openstack-infra | 22:46 | |
fungi | grr, a distro python upgrade has all my existing venvs broken and spewing "ImportError: No module named datetime" | 22:47 |
clarkb | fungi: ya you have to start from scratch iirc | 22:47 |
clarkb | fungi: at least thats what I did after my precise to trusty upgrade | 22:47 |
clarkb | its good fun | 22:47 |
clarkb | greghaynes: reviewed your change to add wheel jobs | 22:47 |
greghaynes | \O/ | 22:47 |
greghaynes | clarkb: That might need to be a new feature in pypi-mirror | 22:48 |
mtreinish | jeblair: yeah the ec2 stackforge, or I think some of the neutron *aas services use it too | 22:48 |
mtreinish | jeblair: I'd ask sdague when he's around too, he was driving a lot of that | 22:48 |
clarkb | greghaynes: if thats the case then nevermind for now | 22:49 |
greghaynes | clarkb: which is :( because there are literally 0 tests | 22:49 |
*** armax has joined #openstack-infra | 22:49 | |
clarkb | greghaynes: but there are a few other comments that should be addressed | 22:49 |
clarkb | greghaynes: fwiw there were tests we just stopped running them when the project died | 22:49 |
greghaynes | yea, others are valid | 22:49 |
clarkb | greghaynes: also I commented again so make sure you look at latest diff | 22:49 |
openstackgerrit | Merged openstack-infra/project-config: Add nodepool config check https://review.openstack.org/164904 | 22:50 |
clarkb | mtreinish: 164451 ok with you? | 22:51 |
*** erlon has quit IRC | 22:51 | |
mtreinish | clarkb: yeah | 22:52 |
mtreinish | I thought I threw a +1 on that at some point | 22:52 |
*** mfink_ has joined #openstack-infra | 22:52 | |
*** crc32 has quit IRC | 22:53 | |
*** annegentle has quit IRC | 22:53 | |
clarkb | mtreinish: also 164314 | 22:54 |
clarkb | morganfainberg: ^ you too | 22:54 |
*** bhunter71 has quit IRC | 22:55 | |
morganfainberg | clarkb, works for me. | 22:55 |
morganfainberg | clarkb, i was more curious of where it was going and what it change(s) if anything | 22:56 |
mtreinish | clarkb: I'll trade you for 164397, 165183, and 164374 | 22:56 |
morganfainberg | clarkb, but not heart set on splitting keystone out - and wouldn't want to make that the only testing w/o functional up and running | 22:56 |
*** ominakov has quit IRC | 22:56 | |
*** sarob has joined #openstack-infra | 22:58 | |
mtreinish | clarkb: eh, I'm on the fence for it, but not opposed to it.(since it's an additional job) I disagree with sdague, I think at least for right now, we should do more than just identity as the regex if we're going to do that because those tests don't really do much of anything | 22:59 |
clarkb | my biggest thing is hammering keystone with a bunch of requests that come from all of the other services is basically keystone functional testing right now | 22:59 |
clarkb | it happens to be integration testing which isn't the best but thats where we get the coverage | 22:59 |
*** sabeen has quit IRC | 23:01 | |
*** timcline has joined #openstack-infra | 23:01 | |
*** markvoelker has quit IRC | 23:01 | |
openstackgerrit | Ian Wienand proposed openstack-infra/system-config: testing nodepool cfg job https://review.openstack.org/167035 | 23:02 |
*** timcline has quit IRC | 23:02 | |
clarkb | I think I made a small dent in the project-config queue | 23:03 |
*** sarob has quit IRC | 23:03 | |
openstackgerrit | Merged openstack-infra/project-config: zuul: add check-grenade-dsvm-neutron to tempest checks https://review.openstack.org/164451 | 23:03 |
*** ivar-laz_ has joined #openstack-infra | 23:04 | |
clarkb | nibalizer: see comment on 166976 | 23:05 |
*** dangers is now known as dangers_away | 23:06 | |
jeblair | mtreinish: i agree with clarkb on 164314. it's only an "additional job" temporarily -- the intent is clearly for it to replace the current job so we should evaluate it in that light. | 23:06 |
openstackgerrit | Merged openstack-infra/system-config: Add docs for subunit2sql https://review.openstack.org/164397 | 23:06 |
jeblair | morganfainberg: ^ | 23:06 |
morganfainberg | jeblair, my statement stands. | 23:07 |
*** ivar-lazzaro has quit IRC | 23:07 | |
morganfainberg | jeblair, not heartset on it or anything. however it shakes out is fine. | 23:07 |
jeblair | morganfainberg: yeah, just pinging you on that so you'd see it | 23:07 |
*** tnovacik has quit IRC | 23:07 | |
*** asselin_ has joined #openstack-infra | 23:07 | |
*** dannywilson has joined #openstack-infra | 23:08 | |
mtreinish | jeblair: sure, and I think to do that it might be too soon (or not, it's not entirely clear because keystone's coverage from tempest is weird so we might not necessarily let bugs slip in) | 23:08 |
*** dannywilson has quit IRC | 23:08 | |
mtreinish | which is why I didn't vote one way or the other | 23:09 |
*** dannywilson has joined #openstack-infra | 23:09 | |
openstackgerrit | Merged openstack-infra/puppet-subunit2sql: Fix db upgrade console redirection to be posix compliant https://review.openstack.org/165183 | 23:10 |
openstackgerrit | James Polley proposed stackforge/gertty: Add change list options to configuration https://review.openstack.org/161032 | 23:10 |
mtreinish | jeblair: in the worst case though if we did move to that and keystone regressed in a way that a full tempest would catch, it would be caught pretty quickly because everything was relying on it | 23:11 |
*** sarob has joined #openstack-infra | 23:13 | |
jeblair | mtreinish: yeah, we have a system to prevent that from happening and it is not clear to me at the moment why we should turn that system off | 23:13 |
jeblair | mtreinish: (i would prefer we have a different system to prevent that from happening in place before we disable the one we have) | 23:14 |
openstackgerrit | James Polley proposed stackforge/gertty: Fix urllib3 InsecurePlatformWarning https://review.openstack.org/164770 | 23:17 |
openstackgerrit | James Polley proposed stackforge/gertty: Right align line numbers https://review.openstack.org/163979 | 23:17 |
mtreinish | jeblair: sure and I tend to agree, I'm just playing devil's advocate (or pretending to be sdague) | 23:17 |
*** spzala has quit IRC | 23:17 | |
*** Swami has joined #openstack-infra | 23:17 | |
anteaya | krotscheck: :( | 23:18 |
clarkb | ianw: will needto wait for zuul to reload its layout.yaml before the nodepool config job will show up on your change | 23:18 |
*** asselin_ has quit IRC | 23:19 | |
clarkb | ianw: on the zuul status page there is a timestamp for when the layout.yaml was last reloaded, once that updates I think you can recheck and expect to see it | 23:19 |
krotscheck | anteaya: Wha? | 23:19 |
ianw | clarkb: yep, just put it in while i was thinking of it | 23:19 |
anteaya | krotscheck: monty's post to the mailing list | 23:20 |
krotscheck | Oh, that finally hit? | 23:20 |
*** sarob has quit IRC | 23:20 | |
krotscheck | anteaya: Thanks for pointing that out, I have a response all prepped. | 23:20 |
anteaya | krotscheck: respond away | 23:20 |
* krotscheck was ready for it. | 23:20 | |
anteaya | you're up | 23:20 |
anteaya | krotscheck: yeah, I didn't figure he would pull the run out from under you without discussing it with you | 23:20 |
anteaya | krotscheck: that would not be his style | 23:21 |
anteaya | rug | 23:21 |
*** ddieterly has quit IRC | 23:23 | |
jhesketh | jeblair: re 166596.. Okay I didn't quite realise we had come to a decision, I should have looked through the logs to check. I do agree though, it makes a lot of sense to store them with the logs and I was initially advocating for that. However, when I went to write it the other day I couldn't think of any easy way to do it and whipped this together as it put us in feature parity with what we're already doing. | 23:23 |
jhesketh | jeblair: My thinking that if we were going to do it at a log level, whenever the job uploads a README to a folder that has an index generated for it, it is then appended to the index | 23:24 |
jhesketh | that way we can have different help messages in sub-folders etc | 23:24 |
jeblair | jhesketh: okay, it's already written and isn't going to kill us. we can always move to another way later | 23:25 |
jeblair | jhesketh: that's a good idea actually, and matches an existing apache pattern | 23:25 |
jeblair | jhesketh: mostly i saw something happen which was the opposite of what i expected, and figured that's at least worth chatting about :) | 23:25 |
jhesketh | jeblair: well I don't think it's time wasted, it's still a feature that could land in os-loganalyze even if we don't use it | 23:26 |
jeblair | jhesketh: in the future in meetings, i'll try to make sure it's more clear when we really do have agreement and on what | 23:26 |
jhesketh | jeblair: yeah the biggest difficulty with putting in the jobs was figuring out how to get the jobs to manage the content (and where to store that) | 23:26 |
jeblair | jhesketh: devstack-gate? | 23:26 |
jhesketh | yeah that probably makes the most sense | 23:27 |
clarkb | devstack gate could just have a couple static file sthat we upload straight out of the repo | 23:27 |
*** sarob has joined #openstack-infra | 23:27 | |
dougwig | clarkb: thank you. | 23:28 |
jhesketh | clarkb: yeah, although I think I'd create a macro (or two) for actually selecting and placing the static file(s) in the right place. Then other jobs that want to do something similar can make new macros | 23:29 |
clarkb | jhesketh: sounds reasonable | 23:29 |
*** chlong has joined #openstack-infra | 23:30 | |
jhesketh | jeblair, clarkb: What would we expect the readme to be appended to be called. For example, README.html, just README or README.md (and render it etc) | 23:30 |
clarkb | jhesketh: I think I would use raw html (I noted that on the change review too) and call it log_footer.html? | 23:31 |
jeblair | mtreinish, sdague, dtroyer: https://review.openstack.org/167042 | 23:32 |
jhesketh | clarkb: I was thinking this would still be listed in the index so maybe something like index_footer.html would be clearer? | 23:32 |
*** asettle has joined #openstack-infra | 23:32 | |
clarkb | jhesketh: ya | 23:32 |
tchaypo | clarkb: I had a look at fungi’s suggestion for https://review.openstack.org/#/c/164770/ and it seems to be equivalent to what you did - https://pypi.python.org/pypi/requests/2.6.0 lists the 3 packages as you had listed as being the [security] extras | 23:33 |
tchaypo | clarkb: but i think that explicitly using requests[security] is more future proof. Do you mind if I update the change to use that? | 23:33 |
clarkb | tchaypo: I think fungi wrote that change | 23:33 |
fungi | tchaypo: clarkb: worth noting though, that depending on those things is a lot heavier than not doing so | 23:33 |
*** mwagner_lap has joined #openstack-infra | 23:33 | |
openstackgerrit | Joe Gordon proposed openstack-infra/project-config: Don't run neutron-large-ops on neutron advanced services https://review.openstack.org/165648 | 23:33 |
clarkb | tchaypo: the reason I wrote it the way I wrote it is the warning message you receive tells ou to do it that way | 23:34 |
clarkb | tchaypo: it does not tell you to use a requests extras install thing | 23:34 |
fungi | yep, i merely got that recommendation from the requests devs | 23:34 |
clarkb | fungi: ya it had been -1'd for that reason iirc | 23:34 |
clarkb | I don't care, I think what urllib3/requests have done is ridiculous and turnnig off the warning completely doesn't make me feel tremendously less secure | 23:34 |
fungi | agreed, i thought 164770 had been abandoned because gertty was silencing it regardless | 23:35 |
tchaypo | I thought that the warning got turned off because the underlying problem was fixed with these libraries | 23:35 |
*** mtanino has quit IRC | 23:35 | |
clarkb | tchaypo: thats one way of turning the warning off | 23:35 |
*** asselin_ has joined #openstack-infra | 23:36 | |
*** baoli has joined #openstack-infra | 23:36 | |
clarkb | tchaypo: unfortunately its also one of the least user friendly | 23:36 |
*** sarob has quit IRC | 23:36 | |
tchaypo | you mean https://review.openstack.org/#/c/165447/ ? | 23:36 |
clarkb | probably the most user unfriendly is telling everyone to upgrade to latest python2 | 23:36 |
clarkb | tchaypo: ya either that or the change zaro wrote are probably better | 23:36 |
tchaypo | I’m not sure who zaro is | 23:37 |
clarkb | tchaypo: https://review.openstack.org/#/c/157259/ | 23:37 |
tchaypo | Oh - change 165448 hides the warning completely. | 23:37 |
jeblair | zaro, tchaypo; tchaypo, zaro | 23:37 |
tchaypo | boo, hiss. | 23:37 |
anteaya | zaro is khai do | 23:37 |
anteaya | stick close to him when traveling you will never get lost | 23:38 |
fungi | tchaypo: honestly, a warning saying "j00 python 2 oldxorz" isn't so helpful to users regardless | 23:38 |
*** otter768 has joined #openstack-infra | 23:38 | |
anteaya | he has an inate sense of direction | 23:38 |
tchaypo | Change 157259 is different - my understanding is that it only disables the warning in cases when we’re not doing the validation anyway | 23:38 |
jeblair | tchaypo: it's a different warning | 23:39 |
jeblair | tchaypo: but yes | 23:39 |
tchaypo | even more better. | 23:39 |
jeblair | tchaypo: it disables both warnings when ssl cert validation is disabled | 23:39 |
jeblair | tchaypo: were you boo-ing my change to not display the insecure platform warning? | 23:40 |
fungi | i think he was saying boo-urns | 23:40 |
*** sarob has joined #openstack-infra | 23:40 | |
tchaypo | no, I was booing the idea of having it in a pop-up | 23:40 |
jeblair | tchaypo: ah yeah, so the change immediately after adding it as a popup is to make sure it never appears as a popup :) | 23:41 |
jeblair | tchaypo: i promise i'll merge them together :) | 23:41 |
tchaypo | it’s a silly warning and seems to keep happening over and over (if I’m thinking of the right warning) and that makes me think the pop-up would be perenially popped-up | 23:41 |
*** wenlock has quit IRC | 23:41 | |
*** zz_dimtruck is now known as dimtruck | 23:41 | |
*** annegentle has joined #openstack-infra | 23:41 | |
fungi | tchaypo: there was also a bug (in urllib3 i think?) where it was throwing that on every instantiation rather than only on import or something. fix may already be proposed | 23:41 |
jeblair | tchaypo: the logic is: if python is going to emit a warning to the user, we should do it within the display framework we have, thus, display all warnings as popups. step 2 is now that the annoying warning is displayed correctly, stop it from ever displaying. | 23:42 |
*** tonytan4ever has quit IRC | 23:42 | |
jeblair | tchaypo: so the warning popup code should almost never be used; however, if there ever is a legitimate warning, it will be. | 23:42 |
*** otter768 has quit IRC | 23:43 | |
jeblair | so yeah, 'boo-urns'. :) | 23:43 |
tchaypo | jeblair: I agree with the logic | 23:43 |
tchaypo | the problem here is that the error message is showing too often, so I agree with hiding it most of the time as well | 23:44 |
jeblair | the end result of my changes is that you'll never see it, and it will be logged, once, to the log file. | 23:44 |
*** ajmiller has quit IRC | 23:44 | |
tchaypo | but the warnings sound to me like they’re warning about something most users will care about | 23:44 |
tchaypo | I think it makes sense to fix it rather than hide it | 23:44 |
*** sarob has quit IRC | 23:45 | |
tchaypo | well, fix rather than hide completely. I’m fine with hiding it all-but-once-per-invokation-of-gertty | 23:45 |
jeblair | tchaypo: the only way to fix it is to upgrade python. | 23:45 |
tchaypo | ah. | 23:45 |
clarkb | or use the pyopenssl stuff | 23:45 |
tchaypo | I thought that installing requests[security] was fixing it | 23:45 |
fungi | an alternate solution, but does drag in dependencies which need local libraries and headers for the same to compile extensions at install time | 23:46 |
jeblair | true | 23:46 |
*** markvoelker has joined #openstack-infra | 23:47 | |
jeblair | yeah, i think the solution is out of proportion to the problem there | 23:47 |
fungi | so also not exactly user-friendly for a lightweight pure python tool | 23:47 |
openstackgerrit | Joshua Hesketh proposed openstack-infra/project-config: zuul_swift_upload add support for appending help https://review.openstack.org/167046 | 23:47 |
jhesketh | clarkb, jeblair: so thinking about it with a clearer mind, the upload side of stuff isn't too complicated ^ | 23:47 |
tchaypo | okay | 23:48 |
tchaypo | this is getting a bit heavy-weight, but | 23:48 |
tchaypo | maybe we could mirror what requests do - add some extra-requires | 23:49 |
tchaypo | if people install gertty[security] they get requests[security]; if they don’t, they get the warnings (unless they have a new enough python in which case it doesn’t matter) | 23:49 |
jeblair | dtroyer: where should a devstack plugin list debs/rpms ? | 23:50 |
openstackgerrit | Merged openstack-infra/project-config: Don't run neutron-large-ops on neutron advanced services https://review.openstack.org/165648 | 23:50 |
zaro | anteaya: wrong branch, master has Gerrit 2.4. we are working on 2.9.4 branch http://git.openstack.org/cgit/openstack-infra/gerrit/tree/.gitreview?h=openstack/2.9.4 | 23:51 |
*** markvoelker has quit IRC | 23:51 | |
*** ibiris is now known as ibiris_away | 23:52 | |
jeblair | tchaypo: i have a litmus test that involves imagining whether i, as a user of gertty, would throw a tomato at the author of gertty given some behavior; i think i would be dodging my own produce with that. :) | 23:52 |
fungi | jeblair: it's called juggling! | 23:52 |
*** ghostpl_ has joined #openstack-infra | 23:57 | |
zaro | tchaypo: hi, we met at tripleO meetup, seattle. | 23:58 |
*** dims_ has joined #openstack-infra | 23:58 | |
ianw | jeblair: since https://review.openstack.org/#/c/161535/ they can list them in the plugi | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!