*** lifeless has joined #openstack-infra | 00:07 | |
*** otter768 has quit IRC | 00:08 | |
*** medieval1 has quit IRC | 00:09 | |
*** medieval1 has joined #openstack-infra | 00:10 | |
*** otherwiseguy has joined #openstack-infra | 00:34 | |
*** lcheng_ has joined #openstack-infra | 00:34 | |
*** otter768 has joined #openstack-infra | 01:00 | |
*** saschpe has quit IRC | 01:05 | |
*** jhesketh has joined #openstack-infra | 01:06 | |
*** otherwiseguy has quit IRC | 01:07 | |
*** saschpe has joined #openstack-infra | 01:12 | |
*** chyang has joined #openstack-infra | 01:35 | |
*** gilliard_ has joined #openstack-infra | 01:38 | |
*** jhesketh has quit IRC | 01:41 | |
*** nosnos has joined #openstack-infra | 01:41 | |
*** lcheng_ has quit IRC | 01:42 | |
*** nati_ueno has joined #openstack-infra | 01:46 | |
*** Sukhdev has joined #openstack-infra | 01:52 | |
*** gilliard_ has quit IRC | 02:03 | |
*** esker has joined #openstack-infra | 02:06 | |
*** dstanek_zzz is now known as dstanek | 02:08 | |
*** nati_ueno has quit IRC | 02:09 | |
*** changbl has quit IRC | 02:11 | |
*** esker has quit IRC | 02:13 | |
*** nati_ueno has joined #openstack-infra | 02:13 | |
*** esker has joined #openstack-infra | 02:14 | |
*** nati_ueno has quit IRC | 02:16 | |
*** lifeless has quit IRC | 02:16 | |
*** matsuhashi has joined #openstack-infra | 02:17 | |
*** esker has quit IRC | 02:18 | |
*** nati_ueno has joined #openstack-infra | 02:21 | |
*** lifeless has joined #openstack-infra | 02:24 | |
*** nati_ueno has quit IRC | 02:30 | |
*** esker has joined #openstack-infra | 02:31 | |
*** Sukhdev has quit IRC | 02:34 | |
*** timfreun1 is now known as timfreund | 02:54 | |
*** changbl has joined #openstack-infra | 03:00 | |
*** nati_ueno has joined #openstack-infra | 03:06 | |
*** gilliard_ has joined #openstack-infra | 03:09 | |
*** gilliard_ has quit IRC | 03:14 | |
*** dstanek is now known as dstanek_zzz | 03:14 | |
*** malini_afk is now known as malini | 03:19 | |
*** nati_ueno has quit IRC | 03:21 | |
*** lnxnut has joined #openstack-infra | 03:30 | |
*** dstanek_zzz is now known as dstanek | 03:32 | |
*** malini is now known as malini_afk | 03:32 | |
*** chyang has quit IRC | 03:38 | |
*** esker has quit IRC | 03:41 | |
*** esker has joined #openstack-infra | 03:42 | |
*** esker has quit IRC | 03:46 | |
*** otter768 has quit IRC | 03:47 | |
*** Hg has joined #openstack-infra | 03:48 | |
*** Hg is now known as Guest51720 | 03:49 | |
*** nosnos has quit IRC | 03:51 | |
*** Guest51720 has quit IRC | 03:54 | |
*** HenryG has joined #openstack-infra | 03:54 | |
*** lnxnut has quit IRC | 03:57 | |
openstackgerrit | Devananda van der Veen proposed a change to openstack-infra/config: Create ironic-specs git repo https://review.openstack.org/94113 | 04:08 |
---|---|---|
*** HenryG has quit IRC | 04:11 | |
*** HenryG has joined #openstack-infra | 04:12 | |
*** HenryG has quit IRC | 04:16 | |
*** yfried has quit IRC | 04:20 | |
*** HenryG has joined #openstack-infra | 04:22 | |
*** chyang has joined #openstack-infra | 04:26 | |
*** changbl has quit IRC | 04:28 | |
*** nati_ueno has joined #openstack-infra | 04:32 | |
*** nosnos has joined #openstack-infra | 04:32 | |
*** nati_ueno has quit IRC | 04:37 | |
*** isviridov has joined #openstack-infra | 04:42 | |
*** _afazekas has quit IRC | 04:45 | |
*** afazekas is now known as _afazekas | 04:46 | |
*** dstanek is now known as dstanek_zzz | 04:46 | |
*** paul--_ has joined #openstack-infra | 04:47 | |
*** dstanek_zzz is now known as dstanek | 04:50 | |
*** d3nyal has joined #openstack-infra | 04:50 | |
HenryG | Jenkins is going beserk on this: https://review.openstack.org/92477 | 04:51 |
StevenK | HenryG: Comment on a old patchset, known issue | 04:54 |
StevenK | But I have no idea how to stop it | 04:55 |
*** andreykurilin_ has joined #openstack-infra | 05:11 | |
*** isviridov is now known as isviridov_away | 05:12 | |
*** Sukhdev has joined #openstack-infra | 05:14 | |
*** d3nyal has quit IRC | 05:20 | |
*** dstanek is now known as dstanek_zzz | 05:20 | |
*** yfried has joined #openstack-infra | 05:21 | |
*** afazekas_ has joined #openstack-infra | 05:26 | |
*** dstanek_zzz is now known as dstanek | 05:30 | |
*** _nadya_ has joined #openstack-infra | 05:32 | |
*** nati_ueno has joined #openstack-infra | 05:34 | |
*** _nadya_ has quit IRC | 05:38 | |
*** mkoderer has joined #openstack-infra | 05:49 | |
*** dstanek is now known as dstanek_zzz | 05:51 | |
*** dims has quit IRC | 05:54 | |
*** flaper87|afk is now known as flaper87 | 06:10 | |
*** yfried has quit IRC | 06:12 | |
*** nati_ueno has quit IRC | 06:17 | |
*** adam_gone is now known as adam_g | 06:19 | |
*** andreykurilin_ has quit IRC | 06:21 | |
*** yfried has joined #openstack-infra | 06:21 | |
*** matsuhashi has quit IRC | 06:25 | |
*** jlibosva has joined #openstack-infra | 06:27 | |
*** matsuhashi has joined #openstack-infra | 06:28 | |
*** Longgeek has joined #openstack-infra | 06:33 | |
*** nati_ueno has joined #openstack-infra | 06:35 | |
*** paul--_ has quit IRC | 06:38 | |
*** cgoncalves has joined #openstack-infra | 06:38 | |
cgoncalves | hi. someone from the infra team could please check what's happening to https://review.openstack.org/#/c/92477/ please? gerrit is mad :-/ | 06:40 |
*** denis_makogon has joined #openstack-infra | 06:44 | |
*** andreykurilin_ has joined #openstack-infra | 06:44 | |
*** andreaf has joined #openstack-infra | 06:46 | |
*** andreykurilin_ has quit IRC | 06:49 | |
*** vtapia has joined #openstack-infra | 06:50 | |
*** _nadya_ has joined #openstack-infra | 06:50 | |
praneshp_ | cgoncalves: wow | 06:51 |
praneshp_ | cgoncalves: there was discussion about it before you joined the room | 06:52 |
praneshp_ | cgoncalves: HenryG: Jenkins is going beserk on this: https://review.openstack.org/92477 | 06:52 |
praneshp_ | [9:54pm] StevenK: HenryG: Comment on a old patchset, known issue | 06:52 |
praneshp_ | [9:55pm] StevenK: But I have no idea how to stop it | 06:52 |
praneshp_ | [10:11pm] | 06:52 |
*** dstanek_zzz is now known as dstanek | 06:53 | |
cgoncalves | praneshp_: thanks. I also just sent an email to -dev ML letting people know of it | 06:56 |
*** gilliard_ has joined #openstack-infra | 06:57 | |
*** nati_ueno has quit IRC | 07:00 | |
*** dstanek is now known as dstanek_zzz | 07:03 | |
openstackgerrit | Chmouel Boudjnah proposed a change to openstack-infra/config: Add swift specs repository https://review.openstack.org/94140 | 07:05 |
*** _nadya_ has quit IRC | 07:10 | |
*** _nadya_ has joined #openstack-infra | 07:11 | |
*** Sukhdev has quit IRC | 07:14 | |
*** katyafervent_awa is now known as katyafervent | 07:19 | |
*** Ivo has quit IRC | 07:23 | |
*** andreaf has quit IRC | 07:26 | |
*** andreaf has joined #openstack-infra | 07:27 | |
*** pblaho has joined #openstack-infra | 07:27 | |
*** nati_ueno has joined #openstack-infra | 07:29 | |
*** praneshp_ has quit IRC | 07:33 | |
*** nati_ueno has quit IRC | 07:33 | |
*** shardy has joined #openstack-infra | 07:34 | |
*** _nadya_ has quit IRC | 07:40 | |
*** jooools has joined #openstack-infra | 07:42 | |
*** Hal_ has joined #openstack-infra | 07:43 | |
*** e0ne has joined #openstack-infra | 07:45 | |
*** alff has joined #openstack-infra | 07:50 | |
*** andreykurilin_ has joined #openstack-infra | 07:50 | |
*** Hal_ has quit IRC | 07:53 | |
*** Hal_ has joined #openstack-infra | 07:54 | |
*** andreykurilin_ has quit IRC | 07:54 | |
*** gilliard_ has quit IRC | 07:55 | |
*** andreykurilin_ has joined #openstack-infra | 07:55 | |
*** chyang has quit IRC | 08:00 | |
*** medieval1_ has joined #openstack-infra | 08:00 | |
*** medieval1 has quit IRC | 08:02 | |
*** jpich has joined #openstack-infra | 08:04 | |
*** odyssey4me has joined #openstack-infra | 08:04 | |
*** cody-somerville has joined #openstack-infra | 08:06 | |
*** darraghb has joined #openstack-infra | 08:15 | |
*** darraghb has quit IRC | 08:16 | |
*** mrodden has quit IRC | 08:17 | |
*** ominakov has joined #openstack-infra | 08:17 | |
*** locke105 has quit IRC | 08:18 | |
*** darraghb has joined #openstack-infra | 08:18 | |
*** mrodden has joined #openstack-infra | 08:18 | |
*** locke105 has joined #openstack-infra | 08:18 | |
*** darraghb has left #openstack-infra | 08:23 | |
*** pelix has joined #openstack-infra | 08:23 | |
*** vponomaryov has joined #openstack-infra | 08:25 | |
*** hashar has joined #openstack-infra | 08:25 | |
*** _dguerri is now known as dguerri | 08:26 | |
*** dkranz has joined #openstack-infra | 08:37 | |
*** derekh_ has joined #openstack-infra | 08:37 | |
*** Longgeek has quit IRC | 08:40 | |
*** andreykurilin_ has quit IRC | 08:47 | |
*** andreykurilin_ has joined #openstack-infra | 08:47 | |
*** andreykurilin_ has quit IRC | 08:51 | |
*** dguerri is now known as _dguerri | 08:52 | |
*** Hal_ has quit IRC | 08:55 | |
ttx | cgoncalves: maybe try to upload a new patchset with some gratuitous change, might break the infinite loop | 08:59 |
*** _dguerri is now known as dguerri | 09:01 | |
*** dizquierdo has joined #openstack-infra | 09:02 | |
*** rlandy has joined #openstack-infra | 09:05 | |
*** matrohon has joined #openstack-infra | 09:06 | |
*** LinuxJedi has joined #openstack-infra | 09:06 | |
*** Hal_ has joined #openstack-infra | 09:10 | |
*** Longgeek has joined #openstack-infra | 09:10 | |
ttx | mordred: you mentioned that the bug about AUTHORS generation was fixed (https://bugs.launchpad.net/pbr/+bug/1309585) but https://review.openstack.org/#/c/88627/ was just abandoned ? | 09:10 |
uvirtbot | Launchpad bug 1309585 in pbr "AUTHORS file generation is borken" [High,In progress] | 09:10 |
*** jp_at_hp has joined #openstack-infra | 09:11 | |
*** LinuxJedi has quit IRC | 09:11 | |
*** Longgeek has quit IRC | 09:13 | |
*** che-arne has joined #openstack-infra | 09:15 | |
*** LinuxJedi has joined #openstack-infra | 09:19 | |
*** _nadya_ has joined #openstack-infra | 09:23 | |
*** che-arne has quit IRC | 09:36 | |
*** Longgeek has joined #openstack-infra | 09:42 | |
*** ihrachyshka has joined #openstack-infra | 09:43 | |
cgoncalves | ttx: done, it stopped. any chance of cleaning that mess in Gerrit? | 09:43 |
ttx | cgoncalves: maybe infra team can do it | 09:44 |
ttx | they need to investigate the loop first :) | 09:44 |
derekh_ | Has nodepool stopped talking to the tripleo-test-cloud provider for some reason ? | 09:45 |
*** andreykurilin_ has joined #openstack-infra | 09:45 | |
*** andreykurilin_ has quit IRC | 09:47 | |
*** andreykurilin_ has joined #openstack-infra | 09:49 | |
*** matsuhashi has quit IRC | 09:50 | |
*** Hal_ has quit IRC | 09:52 | |
*** Hal_ has joined #openstack-infra | 09:52 | |
*** e0ne has quit IRC | 09:55 | |
*** rcarrillocruz has joined #openstack-infra | 09:56 | |
*** rcarrill` has quit IRC | 09:57 | |
*** paul--_ has joined #openstack-infra | 10:03 | |
*** e0ne has joined #openstack-infra | 10:05 | |
*** yfried_ has joined #openstack-infra | 10:07 | |
openstackgerrit | Ricardo Carrillo Cruz proposed a change to openstack-infra/config: Parameterize SSL certificate attributes on review.pp manifest https://review.openstack.org/93687 | 10:07 |
*** yfried has quit IRC | 10:07 | |
*** Ivo has joined #openstack-infra | 10:08 | |
*** vogxn has joined #openstack-infra | 10:13 | |
*** oomichi has joined #openstack-infra | 10:13 | |
openstackgerrit | Ricardo Carrillo Cruz proposed a change to openstack-infra/config: Parameterize SSL certificate attributes on review.pp manifest https://review.openstack.org/93687 | 10:15 |
openstackgerrit | Ricardo Carrillo Cruz proposed a change to openstack-infra/config: Parameterize SSL certificate attributes on review.pp manifest https://review.openstack.org/93687 | 10:20 |
*** Hal_ has quit IRC | 10:22 | |
*** vogxn has quit IRC | 10:24 | |
*** Ivo has quit IRC | 10:25 | |
*** vogxn has joined #openstack-infra | 10:29 | |
*** e0ne has quit IRC | 10:31 | |
*** mfer has joined #openstack-infra | 10:38 | |
*** vogxn has quit IRC | 10:47 | |
*** mwagner_ has quit IRC | 10:48 | |
*** yfried_ has quit IRC | 10:53 | |
sdague | welcome back folks | 10:55 |
*** _nadya_ has quit IRC | 10:57 | |
hashar | sdague: hello! Someone pushed bash8 to pypi \O/ | 10:58 |
hashar | just sharing my hapiness | 10:58 |
sdague | it was mrodden doing the real work on that, many thanks to him | 10:59 |
*** _nadya_ has joined #openstack-infra | 11:01 | |
openstackgerrit | Antoine Musso proposed a change to openstack-infra/zuul: Pass change and pipeline to the status_url https://review.openstack.org/86900 | 11:03 |
*** rcarrillocruz has quit IRC | 11:04 | |
*** flaper87 is now known as flaper87|afk | 11:04 | |
*** flaper87|afk is now known as flaper87 | 11:05 | |
*** e0ne has joined #openstack-infra | 11:06 | |
*** paul--_ has quit IRC | 11:08 | |
*** fbo_away_away is now known as fbo_away_away_aw | 11:08 | |
*** rcarrillocruz has joined #openstack-infra | 11:08 | |
*** vogxn has joined #openstack-infra | 11:09 | |
*** yfried_ has joined #openstack-infra | 11:10 | |
*** e0ne has quit IRC | 11:10 | |
*** dguerri is now known as _dguerri | 11:12 | |
*** lnxnut has joined #openstack-infra | 11:13 | |
*** mburned_out is now known as mburned | 11:16 | |
openstackgerrit | Antoine Musso proposed a change to openstack-infra/zuul: Pass change and pipeline to the status_url https://review.openstack.org/86900 | 11:16 |
*** wayneeseguin has quit IRC | 11:16 | |
*** e0ne has joined #openstack-infra | 11:18 | |
pelix | Any more feedback available on https://review.openstack.org/#/c/48783? | 11:19 |
*** nosnos has quit IRC | 11:21 | |
*** e0ne has quit IRC | 11:22 | |
*** lnxnut has quit IRC | 11:24 | |
*** _nadya_ has quit IRC | 11:27 | |
*** ominakov has quit IRC | 11:28 | |
*** _nadya_ has joined #openstack-infra | 11:29 | |
*** wayneeseguin has joined #openstack-infra | 11:29 | |
*** ihrachyshka_ has joined #openstack-infra | 11:30 | |
*** mfer has quit IRC | 11:30 | |
*** ominakov has joined #openstack-infra | 11:31 | |
*** ominakov has quit IRC | 11:31 | |
*** ihrachyshka has quit IRC | 11:32 | |
*** ominakov has joined #openstack-infra | 11:33 | |
openstackgerrit | Victor Sergeyev proposed a change to openstack-infra/devstack-gate: Add oslo.db to the devstack gate https://review.openstack.org/94173 | 11:36 |
*** _nadya_ has quit IRC | 11:36 | |
*** che-arne has joined #openstack-infra | 11:42 | |
*** andreykurilin_ has quit IRC | 11:42 | |
*** mbacchi has joined #openstack-infra | 11:48 | |
*** ominakov has quit IRC | 11:52 | |
*** paul--_ has joined #openstack-infra | 11:52 | |
*** ominakov has joined #openstack-infra | 11:53 | |
*** e0ne has joined #openstack-infra | 11:53 | |
*** fbo_away_away_aw has quit IRC | 11:55 | |
*** fbo_away has joined #openstack-infra | 11:56 | |
*** bcrochet has quit IRC | 11:57 | |
*** bcrochet has joined #openstack-infra | 12:00 | |
*** Ivo has joined #openstack-infra | 12:02 | |
*** vogxn has quit IRC | 12:02 | |
*** Hal_ has joined #openstack-infra | 12:03 | |
*** paul--_ has quit IRC | 12:04 | |
*** dims_ has joined #openstack-infra | 12:07 | |
*** bcrochet has quit IRC | 12:07 | |
*** bcrochet has joined #openstack-infra | 12:10 | |
*** pdmars has joined #openstack-infra | 12:15 | |
*** thomasem has joined #openstack-infra | 12:20 | |
*** miqui_ has joined #openstack-infra | 12:21 | |
*** krtaylor has joined #openstack-infra | 12:22 | |
*** dstanek_zzz is now known as dstanek | 12:22 | |
*** niska has quit IRC | 12:29 | |
*** weshay has joined #openstack-infra | 12:29 | |
*** dhellmann is now known as dhellmann_ | 12:31 | |
*** niska has joined #openstack-infra | 12:34 | |
*** jpeeler has joined #openstack-infra | 12:34 | |
*** jpeeler has quit IRC | 12:35 | |
*** jpeeler has joined #openstack-infra | 12:35 | |
*** isviridov_away is now known as isviridov | 12:36 | |
andreaf | jeblair: ping | 12:38 |
*** thomasem has quit IRC | 12:38 | |
*** dstanek is now known as dstanek_zzz | 12:38 | |
*** thomasem has joined #openstack-infra | 12:39 | |
*** thomasem has quit IRC | 12:43 | |
sdague | andreaf: I expect he's back on westcoast time now | 12:45 |
*** rfolco has joined #openstack-infra | 12:45 | |
andreaf | sdague: ah right, will try later, thanks | 12:45 |
anteaya | morning morning | 12:52 |
anteaya | I slept in my own house for that last 2 nights, I am soooooo happy | 12:52 |
*** Ivo has quit IRC | 12:59 | |
*** fbo_away is now known as fbo_away_away | 13:01 | |
*** mfer has joined #openstack-infra | 13:01 | |
*** mfer has quit IRC | 13:02 | |
*** malini_afk is now known as malini | 13:02 | |
*** mfer has joined #openstack-infra | 13:02 | |
*** mfer has quit IRC | 13:03 | |
*** mfer has joined #openstack-infra | 13:03 | |
*** rcarrill` has joined #openstack-infra | 13:03 | |
sdague | anteaya: awesome | 13:04 |
*** BobBall has joined #openstack-infra | 13:04 | |
*** rcarrillocruz has quit IRC | 13:05 | |
*** mfer_ has joined #openstack-infra | 13:05 | |
*** mfer has quit IRC | 13:06 | |
*** gilliard_ has joined #openstack-infra | 13:06 | |
*** dprince has joined #openstack-infra | 13:06 | |
*** smarcet has joined #openstack-infra | 13:07 | |
openstackgerrit | Malini Kamalambal proposed a change to openstack-infra/config: Make Tempest job voting for Marconi https://review.openstack.org/93568 | 13:07 |
*** rcarrill` has quit IRC | 13:09 | |
anteaya | sdague: thanks | 13:10 |
anteaya | I'm drumming up business for this meeting which is happening today in about 5 hours: https://wiki.openstack.org/wiki/Meetings/ThirdParty | 13:11 |
anteaya | if you can think of anyone who should know about this meeting, do pass the link along | 13:12 |
*** dstanek_zzz is now known as dstanek | 13:12 | |
openstackgerrit | Chmouel Boudjnah proposed a change to openstack-infra/config: Add swift specs repository https://review.openstack.org/94140 | 13:12 |
*** Ivoz has joined #openstack-infra | 13:12 | |
chmouel | notmyname: ^ | 13:12 |
*** thomasem has joined #openstack-infra | 13:13 | |
*** sahumada has quit IRC | 13:14 | |
*** rcarrillocruz has joined #openstack-infra | 13:15 | |
*** vogxn has joined #openstack-infra | 13:16 | |
*** paul--_ has joined #openstack-infra | 13:16 | |
*** changbl has joined #openstack-infra | 13:20 | |
*** thomasem has left #openstack-infra | 13:20 | |
*** esker has joined #openstack-infra | 13:21 | |
*** esker has quit IRC | 13:22 | |
*** mriedem has joined #openstack-infra | 13:22 | |
*** andreykurilin_ has joined #openstack-infra | 13:23 | |
*** dkliban_afk is now known as dkliban | 13:25 | |
*** zz_gondoi is now known as gondoi | 13:27 | |
*** mfer_ is now known as mfer | 13:28 | |
*** sahumada has joined #openstack-infra | 13:28 | |
*** andreykurilin_ has quit IRC | 13:29 | |
*** andreykurilin_ has joined #openstack-infra | 13:35 | |
*** maxbit has joined #openstack-infra | 13:35 | |
*** andreykurilin_ has quit IRC | 13:36 | |
ttx | sdague: o/ | 13:36 |
sdague | ttx: what's up? | 13:37 |
*** adalbas has joined #openstack-infra | 13:37 | |
*** homeless has joined #openstack-infra | 13:37 | |
ttx | just waving hello in response to your "welcome back" | 13:38 |
* ttx is not 100% operational yet | 13:38 | |
*** andreykurilin_ has joined #openstack-infra | 13:40 | |
sdague | :) | 13:40 |
sdague | yeh, I just processed my gpg keys | 13:40 |
anteaya | hey ttx | 13:40 |
anteaya | sdague: thank you for that | 13:40 |
anteaya | so why does https://review.openstack.org/#/c/93958/1 have so many dependencies | 13:40 |
ttx | sdague: I upgraded to 14.04 LTS. Only requires one half of my brain | 13:40 |
*** oomichi has quit IRC | 13:40 | |
sdague | heh | 13:41 |
anteaya | dependencies like new repos, that I don't understand | 13:41 |
anteaya | I understand new repos, I don't understand the relationship to 93958 though | 13:41 |
sdague | anteaya: I think it's an artifact of that being the top of master | 13:41 |
*** otherwiseguy has joined #openstack-infra | 13:41 | |
anteaya | interesting | 13:41 |
anteaya | that could create some fun down the road | 13:42 |
sdague | it's the way things always worked before, the visualization might be different | 13:42 |
sdague | 93958 is merged | 13:42 |
sdague | so it's "needed by" all the changes that are proposed | 13:43 |
sdague | because that's the parent git ref | 13:43 |
anteaya | oh okay, so if the only thing that is different is that we can see it now, that if fine | 13:44 |
anteaya | I will get prepared to respond to similar questions from others about this | 13:44 |
anteaya | ttx doing anything at 1800 utc today? https://wiki.openstack.org/wiki/Meetings/ThirdParty | 13:44 |
*** andreykurilin_ has quit IRC | 13:45 | |
anteaya | ttx I'd love to have you lurking to ensure we start off headed in the right direction | 13:45 |
anteaya | if you can | 13:45 |
ttx | anteaya: hmm, not really at that time. I can read logs afterwards though | 13:45 |
ttx | if you send me an email so that i remember it | 13:45 |
*** andreykurilin_ has joined #openstack-infra | 13:46 | |
anteaya | okay | 13:46 |
*** gondoi is now known as zz_gondoi | 13:46 | |
anteaya | chmouel: confirming you want a separate gerrit group for swift-specs | 13:50 |
anteaya | the swift-specs-core group | 13:50 |
pelix | Is there anything wrong with https://review.openstack.org/#/c/48783 ? | 13:50 |
anteaya | pelix: well so far jenkins and zaro are happy with the patch | 13:51 |
anteaya | so of the reviews thus far noone has found anything wrong with it | 13:52 |
pelix | but not enough right with it? :p | 13:53 |
anteaya | not enough reviews on it yet | 13:53 |
anteaya | not enough people have had a chance to look at it | 13:54 |
notmyname | anteaya: chmouel: no. I want swift-specs to have the same review group as swift itself | 13:54 |
chmouel | ah yeah that's an error, | 13:54 |
anteaya | you are welcome to help out and reduce the congnitive load on the reviewers by becoming a reviewer anytime, pelix | 13:54 |
anteaya | if you want some suggestions for getting started, let me know, I will help you | 13:55 |
pelix | anteaya: given it's been unchanged since Apr 21, I'm wondering if people wanted to ask me questions about it before they gave any other feedback | 13:55 |
*** jasond` has joined #openstack-infra | 13:55 | |
anteaya | chmouel notmyname k, thanks, I'll wait for the next patchset | 13:55 |
anteaya | pelix: great, thanks for offering | 13:55 |
pelix | anteaya: have been reviewing off/on on JJB for the last 9 months or so :) | 13:55 |
anteaya | pelix: wonderful, thank you for your reviews | 13:55 |
*** jasond` has left #openstack-infra | 13:56 | |
anteaya | chmouel: and nice catch on the ironic specs patch | 13:57 |
*** annegent^ has joined #openstack-infra | 13:57 | |
*** zz_gondoi is now known as gondoi | 13:58 | |
*** _nadya_ has joined #openstack-infra | 13:58 | |
*** ihrachyshka_ has quit IRC | 13:59 | |
chmouel | anteaya: the ironic patch may want to have the ironic-core to vote for the ironic-specs as well (since i was copying and paste from that patch) | 14:00 |
chmouel | instead of the ironic-specs-group | 14:00 |
openstackgerrit | Chmouel Boudjnah proposed a change to openstack-infra/config: Add swift specs repository https://review.openstack.org/94140 | 14:01 |
*** zns has joined #openstack-infra | 14:01 | |
*** ihrachyshka has joined #openstack-infra | 14:01 | |
anteaya | chmouel: thanks I will ask them about that | 14:02 |
*** esker has joined #openstack-infra | 14:04 | |
anteaya | chmouel: can I request a review on https://review.openstack.org/#/c/93929/ please? | 14:05 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/jeepyb: use exceptions instead of return True/False https://review.openstack.org/94196 | 14:05 |
*** rwsu has joined #openstack-infra | 14:06 | |
*** otherwiseguy has quit IRC | 14:07 | |
*** mattoliverau has quit IRC | 14:09 | |
*** paul--_ has quit IRC | 14:09 | |
sdague | any idea if fungi is back today? | 14:10 |
sdague | I'd like to get some early feedback on that last change | 14:10 |
jeblair | sdague: mostly out today, back tomorrow | 14:10 |
sdague | jeblair: but you're awake :) | 14:11 |
anteaya | hey jeblair | 14:11 |
*** mattoliverau has joined #openstack-infra | 14:11 | |
jeblair | anteaya: mornin'! | 14:12 |
sdague | jeblair: hope it was an uneventful trip home | 14:12 |
jeblair | sdague: yes, except for writing like 8 gertty patches -- including kill-thread ;) | 14:13 |
jeblair | (i haven't merged it yet) | 14:13 |
jeblair | (but they're on gerrit) | 14:14 |
sdague | nice | 14:14 |
rcarrillocruz | i'm cleaning up review.pp manifest that contain site specific info. What I did was to add the review class additional arguments and a hiera lookup at the node level. However, I assume I don't have access to hiera data sources (common.yaml for example) as it's sensitive data. Not sure how to proceed, should I open another bug requesting those changes to add it to hiera? | 14:14 |
*** lnxnut has joined #openstack-infra | 14:14 | |
sdague | jeblair: I decided no code on the way home. Needed to let my brain do something else. | 14:14 |
*** signed8bit has joined #openstack-infra | 14:14 | |
rcarrillocruz | the change is this btw: https://review.openstack.org/#/c/93687/7 | 14:16 |
anteaya | sdague: good call | 14:16 |
jeblair | sdague: that's my usual course of action. i have found hacking on gertty strangely relaxing though. | 14:16 |
jeblair | rcarrillocruz: probably the best thing is to note the required steps in the commit message | 14:16 |
rcarrillocruz | something like "Infra team would need to add values for params foo,bar etc on common.yaml" ? | 14:17 |
krtaylor | morning | 14:18 |
anteaya | morning krtaylor | 14:18 |
anteaya | I'm drumming up business for our meeting later today | 14:18 |
krtaylor | hey anteaya - are we doing the 3rd party meeting today? | 14:18 |
krtaylor | hehheh | 14:18 |
anteaya | krtaylor: yes | 14:18 |
anteaya | :D | 14:18 |
sdague | jeblair: it would be nice to know if you hate this - https://review.openstack.org/#/c/94196/ before I start trying to put dashboard support on top of it | 14:18 |
anteaya | bad call on my part, today is supposed to be a holiday | 14:18 |
anteaya | not the first time I have shot myself in the foot like this | 14:19 |
anteaya | meh, I'll live with it | 14:19 |
openstackgerrit | Devananda van der Veen proposed a change to openstack-infra/config: Create ironic-specs git repo https://review.openstack.org/94113 | 14:19 |
jeblair | sdague: just left comments on it | 14:19 |
sdague | cool | 14:19 |
*** morganfainberg_Z is now known as morganfainberg | 14:19 | |
krtaylor | anteaya, hehheh, well, we could start next week | 14:19 |
* krtaylor is still trying to unpack brain | 14:19 | |
jeblair | okay, my priorities are now: a. breakfast, b. fix zuul comment-loop-on-old-patches, c. other things i expected to do today. :) | 14:20 |
*** olaph has joined #openstack-infra | 14:21 | |
rcarrillocruz | hmm, found 'some' info on how to go forward also on http://ci.openstack.org/sysadmin.html#adding-a-new-server | 14:21 |
clarkb | jeblair you can abandon my patchset for zuul comments if you like but I think the test may be useful | 14:21 |
rcarrillocruz | so I guess I'll ping some of you once the patch lands | 14:21 |
jeblair | clarkb: haven't seen it -- link? | 14:21 |
rcarrillocruz | it's something i wanted to raise tomorrow in the infra meeting, i think our manifests should be more generic and have as much specific as possible at the node level | 14:22 |
jeblair | clarkb: oh, found it. | 14:22 |
rcarrillocruz | this helps use the manifests to lay out 'infra' servers in VM when we work on bugs | 14:22 |
clarkb | https://review.openstack.org/#/c/92942/ | 14:22 |
anteaya | jeblair: sounds like a good list | 14:23 |
clarkb | though the test needs a different layout file due to that bug in requires I mentioned | 14:24 |
openstackgerrit | Graham Hayes proposed a change to openstack-infra/config: Add stackforge/designate-specs https://review.openstack.org/87540 | 14:24 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/jeepyb: use exceptions instead of return True/False https://review.openstack.org/94196 | 14:24 |
krtaylor | anteaya, seems like a good agenda/wrap up from summit | 14:25 |
jeblair | clarkb: so i'm trying to figure out if your change is sufficient or if it should be a pipeline requirement | 14:25 |
anteaya | krtaylor: thanks | 14:25 |
krtaylor | anteaya, next week we should get cinder on the agenda | 14:25 |
anteaya | krtaylor: if we are still all in chill mode, that might be a good beginning for the discussion | 14:25 |
*** dstanek is now known as dstanek_zzz | 14:26 | |
clarkb | jeblair: I think we decided it should be a pipeline thing | 14:26 |
anteaya | krtaylor: do add to the agenda | 14:26 |
jeblair | clarkb: i think since that flag defaults to true, it's safe for non-gerrit-triggered changes and post jobs -- but it would prevent, say, the experimental pipeline from being run on old patchsets | 14:26 |
clarkb | which is why abandoning my change is ok. but the test is mostly useable if you dont want to rewrite that | 14:26 |
krtaylor | anteaya, will do, after the cinder session, I think we have to get that ball rolling | 14:26 |
*** mattoliverau has quit IRC | 14:26 | |
*** gilliard_ has quit IRC | 14:26 | |
anteaya | krtaylor: yes, the cinder folks need to have some time and attention | 14:26 |
anteaya | also sahara has indicated interest and I do expect ironic to show up before too long | 14:27 |
jeblair | clarkb: my memory was fuzzy about this since it happened while i was semi-out. do you want to continue working on it? | 14:27 |
krtaylor | anteaya, I went to the triple-o session and it seemed healthy, at least for the moment | 14:28 |
anteaya | krtaylor: awesome, thank you | 14:28 |
clarkb | jeblair: I can if it helps you but I was hoping for today to be a bit lazy (need to do OS upgrades locally) | 14:28 |
anteaya | and I am so glad to hear that | 14:28 |
anteaya | krtaylor: can you ensure they are aware of the existance of the meeting and feel welcome to attend? | 14:29 |
jeblair | clarkb: okay, i can work on it. it's just that there was another really long loop this morning, so i think it's kinda urgent. | 14:29 |
clarkb | so feel free to jump on it if you prefer | 14:29 |
clarkb | ++ | 14:29 |
clarkb | I can definitely provide reviews with a reasonable understanding of the thing | 14:29 |
*** otherwiseguy has joined #openstack-infra | 14:29 | |
*** ihrachyshka_ has joined #openstack-infra | 14:29 | |
*** mattoliverau has joined #openstack-infra | 14:30 | |
krtaylor | anteaya, yes, I'll make the rounds and spread the news | 14:30 |
openstackgerrit | Morgan Fainberg proposed a change to openstack-infra/config: Create identity-specs git repo https://review.openstack.org/94119 | 14:30 |
dizquierdo | hello there, mm just wondering if at some point during the last weeks the review.openstack.org IP address was changed | 14:31 |
dizquierdo | I received an internal warning when mining gerrit | 14:32 |
dizquierdo | and well... just to be sure :) | 14:32 |
devananda | anteaya: ah, discussing the 3rd party meeting? yes, some ironic folks should show up | 14:32 |
*** msabramo has quit IRC | 14:32 | |
*** ihrachyshka has quit IRC | 14:32 | |
devananda | anteaya: at this point, should I point core devs or third-party contribs your way // to the meeting? | 14:33 |
derekh_ | Has nodepool stopped talking to the tripleo-test-cloud provider for some reason ? | 14:33 |
clarkb | dizquierdo: yes it was changed. there are mail list announcements with all the details if you want them | 14:33 |
clarkb | the ip address and ssh host keys were changed when we upgraded to gerrit 2.8 | 14:34 |
dizquierdo | oh that's ok clarkb, didn't see the email sorry about that | 14:34 |
dizquierdo | thanks a lot! :) | 14:34 |
dizquierdo | sorry for the noise | 14:34 |
krtaylor | devananda, meeting info, we will be framing here-> https://wiki.openstack.org/wiki/Meetings/ThirdParty | 14:35 |
anteaya | devananda: both thanks | 14:39 |
anteaya | I need third party folks to know | 14:39 |
anteaya | mostly today I would really like to hear from core devs to shape the format | 14:39 |
devananda | anteaya: ack. though several ironic-core won't make it today (we are skipping our meeting because brainoverload) | 14:40 |
Shrews | clarkb: do you know if tox 1.7.1 is ok to use? | 14:40 |
Shrews | https://bitbucket.org/hpk42/tox/issue/150/posargs-configerror seems to indicate it is, just wondering if we've tested it | 14:41 |
anteaya | devananda: understood | 14:41 |
*** zns has quit IRC | 14:42 | |
*** wenlock has joined #openstack-infra | 14:42 | |
mordred | Shrews, clarkb: wow. the discussion on that bug is amazing | 14:42 |
anteaya | by way of an fyi, most other specs repos are using the name of their signature project as the name of their specs repo, keystone is wants to call theirs identiy specs: https://review.openstack.org/#/c/94119/ | 14:42 |
*** mwagner_lap has joined #openstack-infra | 14:42 | |
anteaya | personally I have no problem with the name of their specs repo | 14:43 |
anteaya | I don't think the name was part of the discussion around specs repos and consistentcy | 14:43 |
clarkb | shrews: I think you need the next release which hasnt happened yet | 14:43 |
anteaya | so I +1'd the patch as it stands | 14:43 |
clarkb | you can install tox from source to test (which I havent done but should do) | 14:43 |
Shrews | clarkb: ah, yeah. > 1.7.1 needed | 14:44 |
morganfainberg | anteaya, any idea why there was a python27 job for some of the spec repos? dolphm pointed it out and i'm now wondering as well. I was using a previous spec repo as a template | 14:44 |
lxsli | jeblair andreaf: any idea what fixed https://bugs.launchpad.net/nova/+bug/1285288 please? | 14:45 |
uvirtbot | Launchpad bug 1285288 in nova "gate-nova-docs No module named ...." [Undecided,Incomplete] | 14:45 |
anteaya | morganfainberg: no idea | 14:45 |
*** lnxnut is now known as hipster | 14:45 | |
anteaya | since the patches will be text I have no idea why people are running unit tests against the patches | 14:45 |
morganfainberg | anteaya, ok let me go ask around, if it's not needed, I'm going to pull it out (less things to do is good) | 14:45 |
anteaya | morganfainberg: agreed | 14:45 |
clarkb | morganfainberg it does simple checks iirc of format maybe | 14:45 |
yfried_ | clarkb: I am working on a patch for debugging multi-node setup. it's default mode should be to connect to localhost via ssh. for this I will need default user/pass for the machine running tempest, is there some way to get it? | 14:46 |
anteaya | mostly I have been focused on the acl files | 14:46 |
*** james_li has joined #openstack-infra | 14:46 | |
clarkb | yfried_: no you should use ssh keys only | 14:46 |
lxsli | morganfainberg: there's a unit test that checks the section titles (in tripleo-specs at least) | 14:46 |
morganfainberg | lxsli, aha. that is a good answer. | 14:46 |
yfried_ | clarkb: ok - so default user/key? | 14:46 |
*** packet has joined #openstack-infra | 14:47 | |
morganfainberg | lxsli, thanks!! | 14:47 |
lxsli | morganfainberg: I helped!!! \o/ | 14:47 |
morganfainberg | dolphm, ^ lxsli's comment | 14:47 |
clarkb | on our test slaves the password is created but not the same across nodes and not stored anywhere accessible | 14:47 |
clarkb | so no default | 14:47 |
morganfainberg | lxsli, and I think i'm going to steal your test :) | 14:48 |
lxsli | morganfainberg: before you do, check https://review.openstack.org/#/c/91801/ | 14:48 |
*** mattoliverau has quit IRC | 14:48 | |
*** yfried_ is now known as yfried| | 14:48 | |
morganfainberg | lxsli, ++ | 14:48 |
morganfainberg | lxsli, awesome! thanks again!! | 14:49 |
lxsli | glad I could help :) | 14:49 |
yfried| | clarkb: ok, so a way to get the current pass/key for a localhost? | 14:49 |
*** yfried| is now known as yfried|gone | 14:49 | |
clarkb | yfried| no even when they are stored they are hashed. ssh keys is the way to do this | 14:50 |
*** jgrimm has joined #openstack-infra | 14:51 | |
*** mattoliverau has joined #openstack-infra | 14:52 | |
dolphm | morganfainberg: lxsli: ah, thanks! | 14:52 |
*** esker has quit IRC | 14:54 | |
*** yfried|gone has quit IRC | 14:54 | |
*** dstanek_zzz is now known as dstanek | 14:54 | |
*** esker has joined #openstack-infra | 14:54 | |
openstackgerrit | Graham Hayes proposed a change to openstack-infra/config: Add stackforge/designate-specs https://review.openstack.org/87540 | 14:56 |
*** thedodd has joined #openstack-infra | 14:56 | |
mugsie | anteaya: i think ^ fixes what you were looking for | 14:57 |
* anteaya clicks | 14:57 | |
*** zns has joined #openstack-infra | 14:57 | |
*** adalbas has quit IRC | 14:57 | |
*** ominakov has quit IRC | 14:58 | |
anteaya | mugsie: it does, +1 | 14:58 |
anteaya | and thank you | 14:58 |
mugsie | anteaya: np - thanks for the help | 14:59 |
*** fbo_away_away is now known as fbo | 14:59 | |
anteaya | :D | 14:59 |
*** esker has quit IRC | 14:59 | |
krotscheck | Storyboard meeting in #openstack-meeting-3 | 15:01 |
*** atiwari has joined #openstack-infra | 15:02 | |
*** mattoliverau has quit IRC | 15:02 | |
*** andreykurilin_ has quit IRC | 15:03 | |
*** moted has joined #openstack-infra | 15:04 | |
*** mattoliverau has joined #openstack-infra | 15:05 | |
*** esker has joined #openstack-infra | 15:06 | |
*** blamar has joined #openstack-infra | 15:07 | |
*** hipster has quit IRC | 15:08 | |
*** doug-fish has joined #openstack-infra | 15:09 | |
*** adalbas has joined #openstack-infra | 15:09 | |
*** adalbas has quit IRC | 15:12 | |
james_li | clarkb: if you have time today please review https://review.openstack.org/#/c/92655/2 ? | 15:13 |
*** markmcclain has joined #openstack-infra | 15:14 | |
*** mattoliverau has quit IRC | 15:17 | |
*** mattoliverau has joined #openstack-infra | 15:23 | |
*** andreykurilin_ has joined #openstack-infra | 15:23 | |
SergeyLukjanov | anteaya, /me acking sahara interest for 3rd party testing discussions | 15:23 |
clarkb | https://github.com/openstack/openstack/graphs/commit-activity more things get merged during holiday weeks than summit weeks :) | 15:24 |
*** lcostantino has joined #openstack-infra | 15:25 | |
*** vogxn has quit IRC | 15:25 | |
anteaya | SergeyLukjanov: awesome, thank you | 15:26 |
anteaya | SergeyLukjanov: can you make today's meeting? | 15:26 |
anteaya | krtaylor: can you look at this? https://etherpad.openstack.org/p/third-party-meeting-format | 15:27 |
anteaya | jaypipes if you are about ^^ | 15:28 |
anteaya | clarkb: ha ha ha | 15:28 |
anteaya | that is really funny | 15:28 |
*** maxbit has quit IRC | 15:28 | |
anteaya | 10 merges on Monday | 15:28 |
*** maxbit has joined #openstack-infra | 15:29 | |
*** UtahDave has joined #openstack-infra | 15:30 | |
*** mkerrin has joined #openstack-infra | 15:30 | |
openstackgerrit | afazekas proposed a change to openstack-infra/elastic-recheck: Add query for 1320617 https://review.openstack.org/94080 | 15:30 |
*** lcostantino has quit IRC | 15:31 | |
*** ptippett has joined #openstack-infra | 15:32 | |
*** esker has quit IRC | 15:33 | |
devananda | anteaya: fwiw, summary of ironic's 3rd party aims: https://wiki.openstack.org/wiki/Ironic/Testing#Third_party_CI | 15:34 |
*** esker has joined #openstack-infra | 15:34 | |
*** msabramo has joined #openstack-infra | 15:34 | |
anteaya | devananda: awesome | 15:34 |
anteaya | would you be willing to link them to https://wiki.openstack.org/wiki/Meetings/ThirdParty | 15:35 |
devananda | ack | 15:35 |
openstackgerrit | afazekas proposed a change to openstack-infra/elastic-recheck: Add query for 1320617 https://review.openstack.org/94080 | 15:35 |
anteaya | thanks | 15:35 |
*** igor_ has joined #openstack-infra | 15:37 | |
*** morganfainberg is now known as morganfainberg_Z | 15:38 | |
*** duran has joined #openstack-infra | 15:39 | |
*** signed8bit has quit IRC | 15:41 | |
anteaya | hey clarkb what are the chances you might be around between 1800 and 1900 to address https://etherpad.openstack.org/p/juno-infra-improving-3rd-party-testing for the third party meeting? | 15:47 |
*** jaypipes has joined #openstack-infra | 15:48 | |
*** denis_makogon has quit IRC | 15:49 | |
*** dkranz has quit IRC | 15:49 | |
anteaya | hey jaypipes | 15:52 |
anteaya | https://wiki.openstack.org/wiki/Meetings/ThirdParty | 15:53 |
anteaya | https://etherpad.openstack.org/p/third-party-meeting-format | 15:53 |
jaypipes | anteaya: reading... | 15:53 |
*** signed8bit has joined #openstack-infra | 15:53 | |
krtaylor | anteaya, we might want to focus that meeting format a bit ;) | 15:53 |
anteaya | jaypipes: and this was the announcement email: ttp://lists.openstack.org/pipermail/openstack-dev/2014-May/035182.html | 15:54 |
anteaya | krtaylor: I'm open, what do you want to see? | 15:54 |
*** hemnafk is now known as hemna | 15:54 | |
jaypipes | krtaylor, anteaya: did we agree to do the staggered neutron/cinder weeks? | 15:54 |
anteaya | jaypipes: we talked about that | 15:55 |
anteaya | and then I got talking to some other folks, horizon, sahara | 15:55 |
anteaya | and now ironic | 15:55 |
anteaya | the priority for me is for openstack leads to talk to each other | 15:55 |
krtaylor | jaypipes, anteaya I think that makes sense, given we could spend the meeting on any one of those | 15:56 |
anteaya | I had mistakenly thought other programs were aware and following the process infra was setting out, that appears not to be the case | 15:56 |
krtaylor | anteaya, even if we get an initial nod from leads and delegation, I'm happy | 15:56 |
krtaylor | a representative that is | 15:57 |
anteaya | right, so I don't think this is a conversation that will be the bulk of the focus forever, but I do want to focus on it right now | 15:57 |
jaypipes | krtaylor, anteaya: I can see both sides. Perhaps we can make it a firm thing that we will only do deep dives into systems that have posted to the infra list or dev list prior to the meeting? to prevent the free-for-all that happened in previous meetings? | 15:57 |
*** mattoliverau has quit IRC | 15:57 | |
anteaya | then once openstack leads have some sort of uniformity, then focus on the third party folks | 15:57 |
*** msabramo1 has joined #openstack-infra | 15:57 | |
anteaya | jaypipes: I like that as a suggestion | 15:58 |
krtaylor | hm, that sounds good | 15:58 |
anteaya | yes, some sort of indication the folks getting airtime know our process | 15:58 |
anteaya | jaypipes: can you propose some edits to agenda/etherpad to reflect that? | 15:58 |
*** msabramo has quit IRC | 15:58 | |
krtaylor | anteaya, jaypipes but I think that some leading will need to happen, for example, scheduling the teams instead of waiting for them to sign up | 15:59 |
krotscheck | Did I miss the discussion on the infra-specs repo? | 15:59 |
krotscheck | ‘cause I suddenly need it to exist. | 15:59 |
krotscheck | So if someone’s working on that.... | 15:59 |
anteaya | krtaylor: fair enough | 15:59 |
jeblair | krotscheck: we discussed it at the summit; anteaya has a review up to create it | 15:59 |
*** jlibosva has quit IRC | 15:59 | |
anteaya | krotscheck: do review: https://review.openstack.org/#/c/93929/ | 16:00 |
krotscheck | Excellent! | 16:00 |
krotscheck | Just what I needed. | 16:00 |
jeblair | it's at the top of my queue, right after fixing an urgent zuul bug | 16:00 |
anteaya | oops chmouel needs to hear from me | 16:00 |
*** mdenny has joined #openstack-infra | 16:00 | |
krtaylor | anteaya, I will try to dig out enough to spend some quality time with my pre-summit list of third party items for today's meeting | 16:00 |
anteaya | ah chmouel wants tests: https://review.openstack.org/#/c/93929/2/modules/openstack_project/files/zuul/layout.yaml | 16:00 |
*** mattoliverau has joined #openstack-infra | 16:00 | |
anteaya | jeblair: what tests, if any do you want on the repo? | 16:00 |
anteaya | krtaylor: thank you | 16:01 |
jeblair | anteaya: only doc | 16:01 |
anteaya | jeblair: I will add doc tests | 16:01 |
pleia2 | good morning | 16:01 |
jeblair | pleia2: how are you feeling? | 16:02 |
chmouel | anteaya: yes i was assuming this was standard for the *-specs | 16:02 |
*** dkranz has joined #openstack-infra | 16:03 | |
anteaya | chmouel: good instincts, thank you | 16:03 |
anteaya | pleia2: hey there | 16:03 |
jeblair | there's a spelling tox env; is anyone using that? | 16:04 |
jeblair | (in the cookiecutter) | 16:05 |
*** matjazp has joined #openstack-infra | 16:05 | |
lxsli | jeblair: any idea what fixed https://bugs.launchpad.net/nova/+bug/1285288 please? | 16:05 |
uvirtbot | Launchpad bug 1285288 in nova "gate-nova-docs No module named ...." [Undecided,Incomplete] | 16:05 |
*** Hal_ has quit IRC | 16:05 | |
pleia2 | jeblair: hanging in there :) have some diagnostic stuff next tuesday | 16:06 |
jeblair | lxsli: no idea. i only marked it as not affecting openstack-ci | 16:07 |
*** ptippett is now known as ptippett[A] | 16:07 | |
* ptippett[A] is now away - Reason : Auto-Away after 30 minutes | 16:07 | |
lxsli | :( thanks | 16:07 |
anteaya | pleia2: I hope the results point to a way forward | 16:07 |
pleia2 | anteaya: thanks, me too :) | 16:08 |
jeblair | ptippett[A]: would you please disable your away message? thanks. | 16:08 |
*** vogxn has joined #openstack-infra | 16:10 | |
*** e0ne has quit IRC | 16:10 | |
*** vtapia has quit IRC | 16:11 | |
*** dizquierdo has quit IRC | 16:11 | |
openstackgerrit | Anita Kuno proposed a change to openstack-infra/config: Create an infra-specs repo https://review.openstack.org/93929 | 16:11 |
*** melwitt has joined #openstack-infra | 16:13 | |
*** marcoemorais has joined #openstack-infra | 16:13 | |
*** praneshp_ has joined #openstack-infra | 16:14 | |
*** pblaho has quit IRC | 16:15 | |
*** msabramo1 has quit IRC | 16:16 | |
*** marcoemorais has quit IRC | 16:17 | |
*** hipster has joined #openstack-infra | 16:18 | |
*** hashar has quit IRC | 16:21 | |
lxsli | anteaya: would you be interested in copying https://review.openstack.org/#/c/91801/ for infra-specs? | 16:21 |
*** mfer is now known as mattfarina | 16:21 | |
*** jp_at_hp has quit IRC | 16:22 | |
*** msabramo has joined #openstack-infra | 16:22 | |
*** aconrad has joined #openstack-infra | 16:23 | |
*** mattfarina is now known as mfer | 16:23 | |
*** vogxn has quit IRC | 16:23 | |
*** jp_at_hp has joined #openstack-infra | 16:24 | |
*** afazekas_ is now known as afazekas | 16:24 | |
*** yfried|gone has joined #openstack-infra | 16:24 | |
anteaya | lxsli: that looks great | 16:25 |
anteaya | I'm not sure what format jeblair wants from infra-specs | 16:25 |
anteaya | I don't know if we want to follow the bp template that folks would use for a service repo or not | 16:25 |
lxsli | OK, my main interest is avoiding another copy-paste of the base code :) | 16:26 |
anteaya | right | 16:26 |
anteaya | and agreed | 16:26 |
*** asselin has joined #openstack-infra | 16:26 | |
lxsli | It seems only nova, neutron and tripleo-specs have the test so far | 16:27 |
lxsli | They've all diverged already :) | 16:27 |
anteaya | I'm glat they do | 16:27 |
anteaya | perhaps the others don't know about it | 16:27 |
lxsli | Yeah probably | 16:27 |
anteaya | do you feel like posting an email to the ml drawing awareness to its presence? | 16:28 |
*** rcarrill` has joined #openstack-infra | 16:28 | |
lxsli | Yeah that's a great idea | 16:28 |
anteaya | cool | 16:28 |
phschwartz | morning all | 16:29 |
*** david-lyle has joined #openstack-infra | 16:29 | |
*** rcarrillocruz has quit IRC | 16:30 | |
pleia2 | morning phschwartz | 16:30 |
anteaya | morning phschwartz | 16:32 |
*** zns has quit IRC | 16:32 | |
clarkb | trusty upgrade is in progress | 16:32 |
anteaya | anyone know what meetbot will do if fed multiple arguments separated by whitespace to the startmeeting command? | 16:32 |
clarkb | so you guys on the other side | 16:32 |
anteaya | clarkb: for personal use, yeah? | 16:33 |
anteaya | not for infra use, correct? | 16:33 |
clarkb | correct | 16:33 |
anteaya | great | 16:33 |
clarkb | on my main machine though | 16:33 |
anteaya | might some folks exciting lurking in channel | 16:33 |
anteaya | clarkb: see you later | 16:33 |
anteaya | and good luck | 16:33 |
anteaya | clarkb: so you might not be done in time for the third party meeting then, I guess | 16:34 |
pleia2 | anteaya: afaik it's #startmeeting title/team of the meeting | 16:34 |
pleia2 | anteaya: so there aren't arguments, the entire thing is a title with spaces | 16:35 |
anteaya | ah so spaces are allowed, cool | 16:35 |
anteaya | sorry by arguments I meant the title of the meeting | 16:35 |
jeblair | anteaya: is this for third party ci? | 16:35 |
anteaya | yes | 16:35 |
jeblair | anteaya: can you use 'third_party_ci'? | 16:35 |
jeblair | anteaya: http://eavesdrop.openstack.org/meetings/third_party_ci/2014/ | 16:36 |
anteaya | was going to call it third-party | 16:36 |
jeblair | anteaya: the title ends up in the url there ^ | 16:36 |
anteaya | yes, I wanted to drop the ci portion while linking to those logs in the wiki | 16:36 |
jeblair | anteaya: isn't ci important? :) | 16:36 |
*** zns has joined #openstack-infra | 16:36 | |
anteaya | since that format served a specific purpose and I feel the need for a broader purpose | 16:36 |
anteaya | ci is important | 16:36 |
anteaya | but there are other issues as well | 16:36 |
anteaya | so I didn't want people to feel their topic was excluded if it didn't deal directly with ci | 16:37 |
anteaya | as a for instance | 16:37 |
*** cody-somerville has quit IRC | 16:37 | |
anteaya | david_lyle has been asked by ceph for horizon to support ceph | 16:37 |
*** alff has quit IRC | 16:37 | |
anteaya | david talked to me and we agree that is a conversation that fits in this space | 16:37 |
anteaya | but it doesn't directly deal with ci | 16:38 |
jeblair | anteaya: that seems like a significant scope expansion | 16:38 |
*** mrodden has quit IRC | 16:38 | |
anteaya | agreed | 16:38 |
*** andreykurilin_ has quit IRC | 16:39 | |
*** marcoemorais has joined #openstack-infra | 16:39 | |
anteaya | and I don't feel the decision will take place in this space | 16:39 |
anteaya | but it is a place to field the question, get some input and format it for the next place to discuss | 16:39 |
anteaya | in this instance to present to the tc | 16:39 |
*** aconrad has quit IRC | 16:39 | |
jeblair | that very much sounds like a -dev ml discussion | 16:39 |
*** _nadya_ has quit IRC | 16:39 | |
anteaya | fair enough | 16:40 |
jeblair | which is the right (and required) place to have a discussion about things that are presented to the tc | 16:40 |
*** zns has quit IRC | 16:40 | |
anteaya | so this space shouldn't be a place where people can indicate this is what they have been approached about? | 16:40 |
david-lyle | for me the real question is what is the policy regarding third party support, and when does a third party become a non-third party | 16:40 |
*** cody-somerville has joined #openstack-infra | 16:41 | |
*** zns has joined #openstack-infra | 16:41 | |
david-lyle | what's the line, so that I don't make one off decisions that come back to haunt me/us? | 16:41 |
*** yfried|gone has quit IRC | 16:41 | |
anteaya | it is the avoidance of one off decisions that haunt us I am trying to address | 16:41 |
david-lyle | yes | 16:42 |
anteaya | and if the right place to discuss is tc or mailing list, then that can come out in the discussion | 16:42 |
david-lyle | and I am very appreciative of that | 16:42 |
anteaya | thanks | 16:42 |
jeblair | anteaya: i would say the third party ci meeting is not the place i would expect that sort of thing to be discussed. | 16:42 |
*** jpich has quit IRC | 16:43 | |
anteaya | jeblair: where would you like someone like david-lyle to approach as a first step to get teh recommendation to take the discussion to the ml | 16:43 |
anteaya | since neither he or I knew that before you said it | 16:43 |
jeblair | anteaya: the -dev list is the universal discussion forum for the project, that's why the tc requires motions be presented there for discussion several days (nearly a week) before the tc votes on them | 16:44 |
*** morganfainberg_Z is now known as morganfainberg | 16:44 | |
jeblair | so i think going there with this issue is a fine idea | 16:44 |
*** igor_ has quit IRC | 16:45 | |
anteaya | the issue I am looking to address is that david-lyle didn't know that | 16:45 |
anteaya | and neither did I | 16:45 |
anteaya | and I don't know waht other projects are doing regarding questions of this nature | 16:45 |
*** _nadya_ has joined #openstack-infra | 16:45 | |
*** igor_ has joined #openstack-infra | 16:45 | |
anteaya | so having a forum for them to bring questions to the table helps them do the right thing, which is take it to the ml in this instance | 16:46 |
*** tnurlygayanov has joined #openstack-infra | 16:46 | |
anteaya | if this isn't it, and I am overstepping, that is fine | 16:46 |
anteaya | but I do think such a venue needs to exist | 16:46 |
openstackgerrit | Andreas Jaeger proposed a change to openstack-infra/config: Remove usage of docbook-properties.sh https://review.openstack.org/94227 | 16:47 |
*** cody-somerville has quit IRC | 16:48 | |
david-lyle | I guess the larger question for is that while services provide an extension framework for different backend drivers and that effects Horizon directly and presumably other projects to a lesser degree | 16:48 |
*** harlowja has joined #openstack-infra | 16:48 | |
david-lyle | do we not integrate any of that support directly into Horizon? | 16:49 |
jeblair | david-lyle: yes, i think the existing driver support in other projects makes this a very interesting and complicated question. | 16:49 |
david-lyle | yet we support multiple hypervisors in nova and expose hypervisor dependent settings in Horizon | 16:49 |
*** igor_ has quit IRC | 16:50 | |
david-lyle | to me trying to develop the UI, the difference isn't readily apparent, other than something was blessed at one point and other things are less blessed | 16:50 |
*** mrodden has joined #openstack-infra | 16:51 | |
david-lyle | but if the mailing list is the place, I can raise the larger question there | 16:52 |
*** derekh_ has quit IRC | 16:52 | |
jeblair | i think so; i think it's a big issue that all the projects need to think about -- especially since their decisions may affect horizon | 16:52 |
*** sarob has joined #openstack-infra | 16:53 | |
* david-lyle working on said email | 16:54 | |
*** jwiffy is now known as jgriffith | 16:55 | |
anteaya | jeblair: initially I had thought that everyone was aware of infra third party requirements and it turns out they aren't | 16:55 |
anteaya | jeblair: cinder is doing their own thing, ironic is doing their own thing | 16:55 |
*** jerryz has joined #openstack-infra | 16:56 | |
*** ihrachyshka_ has quit IRC | 16:56 | |
*** Ajaeger1 has joined #openstack-infra | 16:56 | |
jgriffith | anteaya: can you clarify what "our own thing" is in this context? | 16:56 |
anteaya | jeblair: so I think the scope needs to be expanded to include discussion as opposed to just informing people what infra has decided and has documentation around | 16:56 |
anteaya | jgriffith: some vendors in the meeting want to test nightly or weekly | 16:57 |
david-lyle | jeblair, I don't think my particular questions cover everything anteaya was concerned about, I'm just a symptom | 16:57 |
jgriffith | anteaya: oh... that | 16:57 |
anteaya | that really isn't the spirt of the format we have | 16:57 |
anteaya | yeah | 16:57 |
anteaya | which to me is an indication there are other things | 16:57 |
jgriffith | anteaya: frequency IMO is a seperate topic from implementation | 16:57 |
salv-orlando | jogo: are you ok if I remove the fingerprint also for bug 1224001? It's capturing all the failures for bug 1253896 | 16:57 |
jeblair | anteaya: what we have decided and have documentation around are our requirements. third party test systems don't get to use gerrit if they don't meet it. our requirements are around very basic things. what actually get tested is up to the projects. | 16:57 |
jgriffith | anteaya: I think the spirit is "unrealistic" | 16:57 |
uvirtbot | Launchpad bug 1224001 in neutron "test_network_basic_ops fails waiting for network to become available (dup-of: 1253896)" [High,In progress] https://launchpad.net/bugs/1224001 | 16:57 |
*** yfried|gone has joined #openstack-infra | 16:57 | |
uvirtbot | Launchpad bug 1253896 in tempest/havana "Attempts to verify guests are running via SSH fails. SSH connection to guest does not work." [Critical,Confirmed] https://launchpad.net/bugs/1253896 | 16:57 |
anteaya | jgriffith: frequency should still be a discussion though | 16:57 |
salv-orlando | I am already working on identifying a new fingerprint | 16:57 |
jgriffith | anteaya: sure | 16:57 |
jgriffith | anteaya: don't disagree | 16:57 |
jgriffith | anteaya: that why we talked about it at the summit :) | 16:58 |
salv-orlando | jogo_: ^^ | 16:58 |
jeblair | anteaya: and no, nightly/weekly is not an option. | 16:58 |
jgriffith | jeblair: not an option? | 16:58 |
jeblair | anteaya: we have a way of doing testing here, it's per-patch. | 16:58 |
jeblair | jgriffith: no, not an option. | 16:58 |
*** yfried|gone is now known as yfried | 16:58 | |
anteaya | so I think we need a space to talk about this | 16:58 |
jgriffith | jeblair: so you're saying all drivers are required to run their local CI on every single patch? | 16:58 |
anteaya | unless we are just going to one at a time it in infra | 16:58 |
anteaya | as we find out about things | 16:58 |
jgriffith | jeblair: no flexibility, no starting point | 16:58 |
jeblair | jgriffith: yes, that's how we test things here. | 16:58 |
jgriffith | jeblair: it's all or nothing right off the bat | 16:59 |
jgriffith | jeblair: that seems a bit... "unfair" | 16:59 |
jeblair | jgriffith: not sure what you mean about starting? are you talking about brining up a new 3rd party ci system? | 16:59 |
yfried | clarkb: still here? sorry I had to leave right after I asked you a question: how do I get the current ssh key/pass to localhost? | 16:59 |
jeblair | jgriffith: we have a process for bootstrapping them gently | 16:59 |
*** maxbit has quit IRC | 16:59 | |
*** zns has quit IRC | 16:59 | |
*** homeless_ has joined #openstack-infra | 17:00 | |
jgriffith | jeblair: I'm confused, how is requiring you to have infrastructure that processes hundreds of patches a day "gentle"? | 17:00 |
jgriffith | jeblair: given the resources that exist in OpenStack infra I think it's completely unrealistic to think that everybody is going to duplicate that | 17:00 |
jgriffith | jeblair: unless I'm completely missing a detail here | 17:01 |
*** gyee has joined #openstack-infra | 17:01 | |
jeblair | jgriffith: lots of systems have done this. a third party systems doesn't have to test nearly as many patches as openstack infra does | 17:01 |
krtaylor | jgriffith, you don't have to turn everything on at once | 17:01 |
jgriffith | jeblair: ummm.... so that's a bit contradictory | 17:01 |
jgriffith | jeblair: you said "test every commit" | 17:01 |
jgriffith | jeblair: then you just said "doesn't have to test every commit" | 17:01 |
jeblair | jgriffith: but yeah, for a test to be effective, it has to reliably vote within a reasonable time on all patches, otherwise reviewers don't have the info they need to review | 17:02 |
jeblair | jgriffith: no i said they don't have to test as many as infra does | 17:02 |
krtaylor | jgriffith, that's what I was discussing at summit, you can start slow , with every merge, not patchset | 17:02 |
jgriffith | jeblair: ok | 17:02 |
jeblair | jgriffith: infra runs LOTS of tests, many of which never even complete. :) | 17:02 |
jgriffith | jeblair: fair | 17:02 |
jgriffith | jeblair: sighhh | 17:02 |
jgriffith | jeblair: I think this sort of changes the spirit of being open and making it easy to commit | 17:02 |
*** homeless has quit IRC | 17:03 | |
jeblair | jgriffith: this is mostly about not waisting developers time. if test results aren't available to reviewers, what's the point? | 17:03 |
jgriffith | jeblair: the point is to make sure a 3'rd party driver actually "work" in OpenStack | 17:03 |
jgriffith | Currently we do nothing but code review | 17:03 |
jgriffith | the reality is 1/2 of the drivers don't work | 17:04 |
*** homeless_ has quit IRC | 17:04 | |
jgriffith | So the point is | 17:04 |
jgriffith | even nightly says "hey" this driver does what it advertises | 17:04 |
jgriffith | CI isn't the right fit for "everything" IMO | 17:04 |
jgriffith | and it becomes a distraction in this case | 17:04 |
*** ramashri has joined #openstack-infra | 17:04 | |
jeblair | jgriffith: what is the proposed feedback channel to developers? | 17:05 |
jgriffith | jeblair: I think the confusion here is the intent of having the testing | 17:05 |
jgriffith | jeblair: I don't care about continuous feedback on every patch | 17:05 |
jgriffith | jeblair: I "DO" care about code quality of drivers | 17:05 |
krtaylor | jgriffith, the only way to fix the driver situation is to get in front of the changes, that means CI | 17:05 |
jgriffith | krtaylor: I completely disagree | 17:06 |
jgriffith | krtaylor: but alas I'm certain I'll never win this debate | 17:06 |
krtaylor | jgriffith, but I think it is less painful than you think | 17:06 |
anteaya | jgriffith: :( | 17:06 |
jgriffith | krtaylor: it's not the setup | 17:06 |
jgriffith | krtaylor: it's the resources and the maintenance | 17:07 |
jgriffith | krtaylor: my expectation is that each driver runs dsvm-full against itself. | 17:07 |
jgriffith | krtaylor: That requires signficant resources, you can't say that it doesn't | 17:07 |
*** bnemec has joined #openstack-infra | 17:07 | |
*** aconrad has joined #openstack-infra | 17:07 | |
jgriffith | krtaylor: jeblair I'd argue that you'll have one of two results in this case: | 17:08 |
*** otherwiseguy has quit IRC | 17:08 | |
jgriffith | 1. People just won't bother | 17:08 |
krtaylor | jgriffith, the big pain point will be getting up to where the drivers are working, then stay in fron of it will be less work | 17:08 |
jgriffith | 2. You'll get bogus reporting back in that won't mean anything | 17:08 |
*** otherwiseguy has joined #openstack-infra | 17:08 | |
jgriffith | krtaylor: I've already started that work | 17:08 |
jgriffith | krtaylor: and I'd argue that you do that by starting with nightlies | 17:08 |
jgriffith | krtaylor: when that's running stable and smooth, you graduate to the next level | 17:08 |
jeblair | jgriffith: 1) hasn't been at all the case with nova or neutron -- people are VERY motivated to have their driver supported | 17:09 |
jeblair | jgriffith: why are you setting up a third party system? | 17:09 |
*** hogepodge has joined #openstack-infra | 17:09 | |
jgriffith | jeblair: to verify that when OpenStack hits it's next milestone my product "works" with the bits being shipped | 17:09 |
jeblair | jgriffith: 2) is a problem, but can/should be addressed by project devs providing feedback to the third party testers about the utility of their systems/tests | 17:09 |
*** _nadya_ has quit IRC | 17:09 | |
jgriffith | jeblair: and so that as a customer tha same can be verified | 17:09 |
*** otherwiseguy has quit IRC | 17:10 | |
jgriffith | jeblair: krtaylor ok, I see this isn't really open for discussion | 17:10 |
jgriffith | jeblair: krtaylor I'll play along | 17:10 |
*** sarob_ has joined #openstack-infra | 17:10 | |
krtaylor | jgriffith, turn on a trickle of tests to start, not a firehose, with existing resources, then as resources become available add more tests and patchsets | 17:11 |
jgriffith | krtaylor: well that's pretty much what I was proposing | 17:11 |
jgriffith | krtaylor: but the opinion here was that it was "all or nothing" | 17:11 |
krtaylor | jgriffith, I think it was frequency, not quantity | 17:12 |
jgriffith | krtaylor: aren't they directly related? | 17:12 |
jgriffith | anyway... I popped over here to ask a completely different question :) | 17:13 |
jgriffith | jeblair: I think I've missed a step in setting up cinder-specs... | 17:13 |
jgriffith | jeblair: https://review.openstack.org/#/c/94035/ | 17:13 |
*** kaisero has joined #openstack-infra | 17:13 | |
*** sarob has quit IRC | 17:13 | |
jeblair | jgriffith: for reference, nova probably has the most complete process around this; there may be some good info in https://etherpad.openstack.org/p/juno-nova-third-party-ci | 17:13 |
*** mbacchi has quit IRC | 17:14 | |
jgriffith | jeblair: approval group for this? | 17:14 |
jgriffith | jeblair: reading the etherpad..... | 17:14 |
jeblair | jgriffith: check again now | 17:14 |
jeblair | jgriffith: i just added you to the group (but there are 2 yous in the db -- make sure i got the right one) | 17:15 |
jgriffith | jeblair: haha | 17:15 |
jgriffith | jeblair: it's updated in my settings | 17:15 |
jgriffith | jeblair: thanks | 17:16 |
jgriffith | jeblair: back to the CI thing :) | 17:16 |
jeblair | jgriffith: cool, you can add others as you see fit at https://review.openstack.org/#/admin/groups/344,members | 17:16 |
*** igor_ has joined #openstack-infra | 17:16 | |
jgriffith | jeblair: awesome.. thanks | 17:16 |
jgriffith | so my purpose here is different than Nova's I think | 17:16 |
zaro | aconrad: you want another jjb release? | 17:16 |
jgriffith | jeblair: In other words I'm at the "comptabilitly" stage | 17:17 |
*** hashar has joined #openstack-infra | 17:17 | |
openstackgerrit | Joshua Harlow proposed a change to openstack-infra/config: Add a doc8 stackforge project https://review.openstack.org/94236 | 17:17 |
jeblair | jgriffith: https://wiki.openstack.org/wiki/HypervisorSupportMatrix is also relevant | 17:17 |
*** sarob has joined #openstack-infra | 17:17 | |
phschwartz | anteaya: The patch that I talked to you about. https://review.openstack.org/#/c/93953/ | 17:17 |
jgriffith | jeblair: But that's what i"m saying | 17:17 |
jgriffith | jeblair: it's not relevant to me/Cinder | 17:17 |
jgriffith | jeblair: Cinder has always said that "everybody" fulfills the entire API | 17:17 |
jgriffith | jeblair: that's been since day 1 | 17:18 |
jgriffith | jeblair: we don't have said matrix with "mix and match" | 17:18 |
jgriffith | jeblair: you either run/work with the tests in Tempest or you're out | 17:18 |
jeblair | jgriffith: i was referring to the 3 groups at the top, not the matrix itself | 17:18 |
jgriffith | jeblair: I know... same thing | 17:18 |
jeblair | jgriffith: 'driver works with cinder' and 'driver works with nova' seem very similar | 17:18 |
jgriffith | jeblair: what I'm saying is there's no "Group A, B, C" etc | 17:18 |
jgriffith | it works or it doesn't | 17:19 |
jgriffith | if it "doesn't" it doesn't belong | 17:19 |
*** Longgeek has quit IRC | 17:19 | |
rcarrill` | zaro: hi there, in regards to https://review.openstack.org/#/c/93773/ , would you be ok if I address the extended scope in other bug/changes? I rather land it as is to get some functionality, since the stuff you mention I will need to dig more, jeepyb does not have logic for that from what I can see... | 17:19 |
*** rcarrill` is now known as rcarrillocruz | 17:19 | |
jgriffith | jeblair: I see what you're saying | 17:19 |
*** signed8bit has quit IRC | 17:19 | |
*** sarob_ has quit IRC | 17:19 | |
jgriffith | jeblair: ok... so there's a good point there | 17:19 |
anteaya | phschwartz: thanks, I'll look at it this afternoon | 17:19 |
jgriffith | but it's different when we're talking about 25-30 drivers vers 3 :) | 17:20 |
*** signed8bit has joined #openstack-infra | 17:20 | |
mordred | btw - /me is a fan of "everyone fulfills the entire API" and wishes that were the case in more places | 17:20 |
jeblair | jgriffith: yeah, that's the situation neutron is in, and their code reviews aren't pretty. i suggested that we could run a zuul that everyone talked to and roll those up into one review, but people didn't like that. | 17:20 |
*** mattoliverau has quit IRC | 17:21 | |
*** igor_ has quit IRC | 17:21 | |
*** maxbit has joined #openstack-infra | 17:21 | |
jgriffith | jeblair: interesting | 17:21 |
*** julim has joined #openstack-infra | 17:21 | |
jgriffith | jeblair: honestly though, I see where you folks are coming from... I just want to express my goal :) | 17:22 |
jeblair | jgriffith: tbh, i only really care about things that affect how developers review changes. if running a test in a lab and telling people on (eg) the mailing list when it breaks works for you, i don't really have an opinion there. | 17:22 |
jgriffith | jeblair: actually have some proof/evidence that a driver included in trunk can run the same tests that the reference driver runs | 17:22 |
*** _nadya_ has joined #openstack-infra | 17:22 | |
jeblair | jgriffith: which third party systems are allowed to leave comments on your project is also entirely up to you. | 17:22 |
jeblair | as long as they meet the minimal requirements infra has | 17:23 |
jgriffith | jeblair: well, I want all of them too :) | 17:23 |
jgriffith | jeblair: I'm just not ready to try and herd everyone together for "every" patch that gets submitted | 17:23 |
jgriffith | eventually I want that | 17:23 |
jgriffith | but not yet | 17:23 |
jgriffith | and I don't want them to be gating | 17:23 |
jgriffith | What I want is a dashboard | 17:24 |
jgriffith | Here's the last time this driver was run and it's results | 17:24 |
jgriffith | and here's a list of drivers with "no" results ever | 17:24 |
*** dprince has quit IRC | 17:24 | |
zaro | rcarrillocruz: i've updated the change. | 17:24 |
mordred | jgriffith: have you seen the dashboard for collating the driver tests that the mirantis guys have been working on? | 17:24 |
anteaya | and I think this is a really good conversation to have | 17:25 |
*** hashar has quit IRC | 17:25 | |
jgriffith | mordred: indeed, I have | 17:25 |
anteaya | and I am glad we are having it | 17:25 |
mordred | jgriffith: it sounds like their goal is very similar to what you just stated | 17:25 |
mordred | k. good | 17:25 |
jgriffith | mordred: and that's an ok directin IMO | 17:25 |
anteaya | and I think there are projects in addition to cinder who need the time and space to have the conversation | 17:25 |
* mordred has no direct opinions on that per se - just wanted to make sure you were aware of the effort | 17:25 | |
jeblair | jgriffith: our experience is that we end up with broken code fast if we don't test each commit | 17:25 |
jgriffith | mordred: but if the infra team doesn't agree with any of this I am certainly not going to just go on my merry way and discard what you all would like to see | 17:26 |
anteaya | and ensuring folks are aware of efforts from different spots is a goal | 17:26 |
*** mbacchi has joined #openstack-infra | 17:26 | |
*** amcrn has joined #openstack-infra | 17:26 | |
jgriffith | anteaya: +1 | 17:26 |
jgriffith | jeblair: meh... fair | 17:26 |
mordred | jgriffith: ++ | 17:26 |
jgriffith | jeblair: but my opinion is that today we don't test at all which sucks | 17:26 |
*** mattoliverau has joined #openstack-infra | 17:26 | |
anteaya | agreed, not testing at all sucks | 17:26 |
jgriffith | jeblair: when I required self-test/cert we found that only about 1/2 the drivers even worked | 17:27 |
jgriffith | that sucks!! | 17:27 |
*** james_li has quit IRC | 17:27 | |
jgriffith | I don't even care if it's "hey, this borked that on Monday" | 17:27 |
jgriffith | I just want to make sure when we release everything is tested/working | 17:27 |
anteaya | so I think different folks are on different places of the spectrum of getting this to not suck for third party | 17:27 |
anteaya | and having a place to see where folks are is a good place to start | 17:28 |
jgriffith | anteaya: really I just want it to NOT SUCK for customers building OpenStack Clouds | 17:28 |
anteaya | jgriffith: agreed | 17:28 |
anteaya | I really want that too | 17:28 |
anteaya | I think we all do | 17:28 |
jgriffith | anteaya: I think so too | 17:28 |
anteaya | and we are all currently in different places working towards that goal | 17:28 |
jgriffith | alright... I'll let you all do some real work for a while.... | 17:28 |
jgriffith | but I'll be back :) | 17:28 |
*** thedodd has quit IRC | 17:29 | |
anteaya | so I would like to create a discussion space to foster that direction | 17:29 |
openstackgerrit | Joshua Harlow proposed a change to openstack-infra/config: Add a doc8 stackforge project https://review.openstack.org/94236 | 17:29 |
anteaya | jgriffith: will you be in the -meeting channel in 31 minutes? | 17:29 |
jgriffith | anteaya: I certainly can be | 17:29 |
anteaya | jgriffith: I do hope so: https://wiki.openstack.org/wiki/Meetings/ThirdParty | 17:30 |
*** _nadya_ has quit IRC | 17:30 | |
anteaya | jgriffith: sorry I posted to -dev and ops and your channel | 17:30 |
anteaya | you might have missed it though | 17:30 |
jgriffith | anteaya: :) | 17:30 |
anteaya | :D | 17:30 |
anteaya | thanks | 17:30 |
anteaya | I really appreicate your understanding | 17:30 |
anteaya | you are awesome | 17:30 |
anteaya | jeblair: I can see how the scope creep for this meeting caught you by surprise, it did for me as well | 17:31 |
anteaya | if you can be available for the discussion of format for the meeting, I do welcome your input | 17:32 |
clarkb | woo only UI things appear to be broken | 17:32 |
anteaya | clarkb: yay | 17:32 |
anteaya | clarkb: can you be available to talk about our infra third party session in the third party meeting? | 17:33 |
anteaya | it starts in about 27 minutes in -meeting | 17:33 |
clarkb | probably | 17:33 |
anteaya | thanks | 17:33 |
clarkb | I am going to keep hitting these issues with a hammer but the machine itself is working | 17:33 |
anteaya | I would be appreciative | 17:33 |
anteaya | yay for a working machine | 17:34 |
clarkb | I just don't have any panels and chromium thinks it should draw window decorations for itself | 17:34 |
anteaya | ha ha ha | 17:34 |
anteaya | that must be enertaining | 17:34 |
clarkb | also dmenu is broken | 17:34 |
anteaya | that is helpful | 17:34 |
mordred | clarkb: what is the thing that you are poking at? | 17:34 |
anteaya | was this a straight saucy to trusty upgrade? | 17:35 |
clarkb | anteaya: yes (xubuntu) | 17:35 |
anteaya | funny all the ui is borked | 17:35 |
clarkb | mordred: the xfce top panel shows up then disappears. I recreated it from scratch and logged out and back in and same thing | 17:35 |
clarkb | also dmenu has no entries in it | 17:35 |
clarkb | and chromium is insistent on drawing window decorations | 17:36 |
andreaf | jeblair: ping | 17:36 |
clarkb | all in all small issues so I imagine I can get them sorted out | 17:36 |
*** marun has joined #openstack-infra | 17:36 | |
*** lcheng has joined #openstack-infra | 17:36 | |
clarkb | oh huh dmenu may be working but the font is the wrong color? I think there is something funny with compositing happening | 17:38 |
pleia2 | following up on https://etherpad.openstack.org/p/translation-platform-discussion - do we have action items yet for who is going to set up zanata? are we waiting on anything from the zanata folks? I can start digging in if needed | 17:38 |
mordred | clarkb: ah - I'd help but I did straight install when I installed trusty | 17:38 |
*** mattoliverau has quit IRC | 17:38 | |
mordred | pleia2: I believe the action item is "pleia2 will do that" :) | 17:38 |
pleia2 | mordred: clarkb uses goofy window manager with xubuntu anyway, I don't have his problems ;) | 17:38 |
* Ajaeger1 agrees with mordred | 17:38 | |
pleia2 | mordred: ok cool | 17:39 |
*** homeless has joined #openstack-infra | 17:39 | |
Ajaeger1 | pleia2: We should have everything but if you run into problems, the zanata guys are committed to help | 17:39 |
mordred | pleia2: and I _think_ we probably want to go the "run jboss ourselves and install war into it" route rather than the "download tarball with bundled deps in it" route | 17:39 |
mordred | but I'm probably crazy wrong about all of those words | 17:39 |
pleia2 | Ajaeger1: yeah, browsing the migration docs linked to the etherpad now | 17:39 |
clarkb | pleia2: ya Inotice xfce has more window manager knobs now and I wonder if I have offended them | 17:39 |
Ajaeger1 | pleia2: the idea was that we (you ;) set it up ourselves so that we know whether we can maintain it. | 17:39 |
clarkb | if I change the size of the panel in the panel settings it resizes my windows but still no panel | 17:40 |
pleia2 | mordred, Ajaeger1 - ok, I'll take notes as I install this | 17:40 |
mordred | pleia2: coolio | 17:40 |
pleia2 | thanks | 17:40 |
Ajaeger1 | thanks, pleia2 ! | 17:40 |
mordred | pleia2: btw - I have discovered that I can now just apt-get install mate - which means I'm now happy again because I have my clock applet | 17:40 |
mordred | without having to run mint | 17:41 |
pleia2 | mordred: I still like having 3 clocks in my panel <3 | 17:41 |
pleia2 | no clicky to see them | 17:41 |
mordred | pleia2: the shadow map and the "here are a list of 8 cities your frequent, their current weather and a button to change timezone to them" is my main draw to the gnome2 applet | 17:41 |
mordred | pleia2: of course, 4 of those features "should not be" features of a clock applet - but it's really nice anyway :) | 17:42 |
*** dims_ has quit IRC | 17:42 | |
pleia2 | mordred: yeah, I add multiple clocks and multiple weathers to my panel so I can always see them all without interaction | 17:42 |
mordred | pleia2: I track too many cities - I'd get confuseled | 17:43 |
*** Ryan_Lane has joined #openstack-infra | 17:43 | |
pleia2 | need to have philly weather so I can troll my friends back east | 17:43 |
pleia2 | ho ho, snowing! it's 70f here, again | 17:43 |
*** sarob has quit IRC | 17:43 | |
*** mattoliverau has joined #openstack-infra | 17:43 | |
mordred | haha | 17:44 |
*** james_li has joined #openstack-infra | 17:45 | |
*** matjazp has quit IRC | 17:46 | |
*** afazekas has quit IRC | 17:46 | |
*** kmartin has joined #openstack-infra | 17:47 | |
*** nati_ueno has joined #openstack-infra | 17:49 | |
*** harlowja has quit IRC | 17:50 | |
*** harlowja has joined #openstack-infra | 17:50 | |
*** rcarrillocruz has quit IRC | 17:55 | |
openstackgerrit | James E. Blair proposed a change to openstack-infra/zuul: Add pipeline requirement for current-patchset. https://review.openstack.org/92942 | 17:57 |
jeblair | clarkb: ^ | 17:57 |
jeblair | mordred: if you're around; a review of that ^ would be good (it's an operational problem) | 17:57 |
mordred | jeblair: on it | 17:58 |
clarkb | arg this is the weirdest thing. I think it may be a race in desktop startup | 17:59 |
*** harlowja has quit IRC | 18:00 | |
*** harlowja has joined #openstack-infra | 18:00 | |
*** rcarrillocruz has joined #openstack-infra | 18:00 | |
openstackgerrit | James E. Blair proposed a change to openstack-infra/config: Use current-patchset pipeline requirement https://review.openstack.org/94243 | 18:00 |
jeblair | and there's the complementary change | 18:00 |
mordred | jeblair: are there contexts in which people would want to set current-patchset to false? I ask because it seems like perhaps true should be the default | 18:01 |
mordred | but it's VERY possible I have not thought it through enough | 18:01 |
jeblair | mordred: i haven't thought of one, but "don't care" is the current default | 18:01 |
*** krotscheck has quit IRC | 18:01 | |
*** krotscheck has joined #openstack-infra | 18:02 | |
jeblair | mordred: setting it to false means "this pipeline will never run on the current patchset". i'd love to see the example of that use case. but it matches "open" which is also tri-state. | 18:02 |
jeblair | mordred: that consistency is why i went ahead with it instead of making it binary. | 18:02 |
mordred | ah. gotcha | 18:02 |
mordred | thanks | 18:02 |
clarkb | woot looks like my test was mostly useful after all | 18:03 |
mordred | jeblair: then in that case, I understand, agree, and the change looks good to me | 18:04 |
*** alff has joined #openstack-infra | 18:06 | |
*** bhuvan has joined #openstack-infra | 18:07 | |
openstackgerrit | James E. Blair proposed a change to openstack-infra/zuul: Add pipeline requirement for current-patchset. https://review.openstack.org/92942 | 18:07 |
jeblair | pep8^ | 18:07 |
*** thedodd has joined #openstack-infra | 18:08 | |
*** jooools has quit IRC | 18:09 | |
*** timrc-afk is now known as timrc | 18:10 | |
zaro | pleia2: need help checking out tag using bzr. | 18:10 |
* mordred afks again | 18:10 | |
*** prad has joined #openstack-infra | 18:10 | |
*** julim has quit IRC | 18:11 | |
zaro | pleia2: how would i get this? https://launchpad.net/gearman-java/trunk/0.06 | 18:11 |
pleia2 | zaro: hey | 18:11 |
jeblair | clarkb: are you reviewing https://review.openstack.org/92942 ? | 18:11 |
*** krotscheck has quit IRC | 18:12 | |
pleia2 | zaro: the syntax is something like bzr branch tag:0.06 lp:gearman-java | 18:12 |
zaro | pleia2: wait that's the wrong one. i want 0.6 from history. it's rev no. 95 | 18:12 |
*** maxbit has quit IRC | 18:12 | |
pleia2 | but let me doublecheck | 18:12 |
*** krotscheck has joined #openstack-infra | 18:13 | |
*** alff has quit IRC | 18:13 | |
clarkb | jeblair: yes | 18:13 |
*** julim has joined #openstack-infra | 18:13 | |
*** maxbit has joined #openstack-infra | 18:13 | |
jeblair | clarkb: ok, no rush, just checking | 18:13 |
*** alff has joined #openstack-infra | 18:13 | |
zaro | pleia2: i have latest checked out, history shows shows trunk is at rev 96. i just want to get revno. 95 | 18:14 |
clarkb | jeblair: yup just got through it | 18:15 |
clarkb | looking at the complementary change now | 18:15 |
*** hipster has quit IRC | 18:15 | |
*** LinuxJedi has left #openstack-infra | 18:16 | |
pleia2 | zaro: ah, that's easier then, sec | 18:16 |
*** igor_ has joined #openstack-infra | 18:16 | |
*** zns has joined #openstack-infra | 18:17 | |
*** nati_ueno has quit IRC | 18:17 | |
*** mwagner_lap has quit IRC | 18:18 | |
*** andreaf has quit IRC | 18:18 | |
pleia2 | zaro: so you can: bzr diff -r 95 | 18:19 |
pleia2 | (I couldn't actually find a 0.6 tag, but maybe I'm doing something wrong, it's been a while) | 18:20 |
pleia2 | zaro: or: bzr revert 9t | 18:20 |
*** marun has quit IRC | 18:20 | |
pleia2 | to actually go back | 18:20 |
pleia2 | bzr revert 95 | 18:20 |
*** igor_ has quit IRC | 18:21 | |
zaro | log still shows 96 as latest. so it's only reverted the change, not checkout from there. can bzr not checkout from a revision? | 18:22 |
openstackgerrit | A change was merged to openstack-infra/zuul: Add pipeline requirement for current-patchset. https://review.openstack.org/92942 | 18:23 |
pleia2 | zaro: oh, then you want: bzr branch -r 95 lp:gearman-java | 18:23 |
pleia2 | zaro: (you had mentioned you already had checked it out, so I thought you wanted to work with that) | 18:23 |
*** krotscheck has quit IRC | 18:24 | |
*** krotscheck has joined #openstack-infra | 18:24 | |
*** maxbit has quit IRC | 18:24 | |
zaro | pleia2: thanks. that was impossible to find on internet. | 18:25 |
pleia2 | zaro: yeah, bzr has great documentation for the basics, but beyond that it can be a bit hairy | 18:26 |
SpamapS | pleia2: thats because bzr only has the basics. :) | 18:26 |
SpamapS | none of those scary knobs that break things and lose data. ;) | 18:26 |
*** sweston has joined #openstack-infra | 18:27 | |
*** e0ne has joined #openstack-infra | 18:28 | |
pleia2 | SpamapS: haha | 18:28 |
*** doude has joined #openstack-infra | 18:32 | |
*** nati_ueno has joined #openstack-infra | 18:32 | |
*** pelix has quit IRC | 18:34 | |
*** briancurtin has joined #openstack-infra | 18:35 | |
*** sweston has quit IRC | 18:35 | |
*** jp_at_hp has quit IRC | 18:35 | |
*** sweston has joined #openstack-infra | 18:35 | |
briancurtin | Anyone familiar with testr together with coverage to understand why "tox -e cover" in https://github.com/stackforge/python-openstacksdk is trying to import a nonexistent "openstacksdk" module? | 18:36 |
*** cody-somerville has joined #openstack-infra | 18:37 | |
*** gokrokve has joined #openstack-infra | 18:38 | |
clarkb | briancurtin: I am going to guess that the distutils hook there is assuming that the python module has the same name as the name at https://github.com/stackforge/python-openstacksdk/blob/master/setup.cfg#L2 less the python- prefix | 18:39 |
*** Ajaeger1 has quit IRC | 18:39 | |
clarkb | briancurtin: it needs to know what to collect coverage for and I think that may be a pbr assumption | 18:40 |
*** maxbit has joined #openstack-infra | 18:40 | |
briancurtin | clarkb: i had figured it was some magic stripping off the python- prefix. i guess i should look at pbr then to find a place to say what the real name is? | 18:40 |
clarkb | briancurtin: ya | 18:41 |
briancurtin | cool, thanks | 18:41 |
krotscheck | phschwartz: Do y’all have a vinz channel yet? | 18:42 |
clarkb | briancurtin: https://git.openstack.org/cgit/openstack-dev/pbr/tree/pbr/testr_command.py#n61 | 18:42 |
clarkb | briancurtin: I think you can just pass that value in after the --coverage in tox.ini | 18:42 |
*** trainoti has joined #openstack-infra | 18:44 | |
*** crc32 has joined #openstack-infra | 18:46 | |
jeblair | krotscheck, phschwartz: this is a great channel to talk about vinz | 18:47 |
*** hipster has joined #openstack-infra | 18:47 | |
krotscheck | jeblair: Should I deprecate #storyboard? | 18:47 |
jeblair | krotscheck: i think that would be a good idea -- i think it will promote more overlap | 18:48 |
*** rcarrill` has joined #openstack-infra | 18:48 | |
openstackgerrit | Sean Dague proposed a change to openstack-infra/jeepyb: use exceptions instead of return True/False https://review.openstack.org/94196 | 18:49 |
krotscheck | jeblair: Not certain I agree there. This channel gets noisy, and trying to pick out individual conversation threads is a bit of a PITA. | 18:49 |
*** whoops has joined #openstack-infra | 18:50 | |
krotscheck | Though I don’t have a better suggestion on how to make sure that client-related discussions happen in the same place. | 18:50 |
*** hashar has joined #openstack-infra | 18:50 | |
*** Ryan_Lane has quit IRC | 18:51 | |
*** rcarrillocruz has quit IRC | 18:51 | |
*** hipster has quit IRC | 18:51 | |
*** Ryan_Lane has joined #openstack-infra | 18:51 | |
pleia2 | sdague: do you have your slides from your ER talk online somewhere? | 18:51 |
*** _nadya_ has joined #openstack-infra | 18:52 | |
*** maxbit has quit IRC | 18:52 | |
sdague | pleia2: https://www.dropbox.com/s/z1qlncj7nrvydcd/Elastic%20Recheck.odp will get you the latest version | 18:53 |
sdague | I haven't posted them proper yet | 18:53 |
krotscheck | one vinz related question: Does the Gerrit API support CORS? | 18:53 |
jeblair | krotscheck: it does, but it's fairly managable with strategic highlighting. i think the key is that there's a good group of people interested in these problems here who can contribute. | 18:53 |
jeblair | krotscheck: (that was response to previous statement, not api question) | 18:54 |
krotscheck | jeblair: Figured. | 18:54 |
jeblair | krotscheck: don't know off hand, haven't tried. docs are here: https://review.openstack.org/Documentation/rest-api.html | 18:54 |
jeblair | krotscheck: and you may have luck in #gerrit | 18:55 |
pleia2 | sdague: thanks, was looking for the awesome flow graphic to include in the ES blog post | 18:55 |
* jeblair afks for lunch | 18:55 | |
sdague | pleia2: yep, that's the source version :) | 18:55 |
*** andreaf has joined #openstack-infra | 18:55 | |
*** krotscheck has quit IRC | 18:56 | |
*** krotscheck has joined #openstack-infra | 18:57 | |
*** nati_uen_ has joined #openstack-infra | 18:59 | |
phschwartz | jeblair: I would agree it would allow for more overlap of people. | 19:00 |
phschwartz | krotscheck: Hadn't created the channel. I planed too, but if jeblair thinks it is better here, I am for that. | 19:00 |
*** Hal_ has joined #openstack-infra | 19:01 | |
krtaylor | thanks anteaya, that was a good kickoff | 19:01 |
*** Ryan_Lane has quit IRC | 19:01 | |
*** Ryan_Lane1 has joined #openstack-infra | 19:01 | |
anteaya | krtaylor: thank you, yes I am pleased with how it went | 19:01 |
anteaya | I gave myself some action items, I have to get used to using the #action tag | 19:01 |
clarkb | pleia2: I haven't responded to your email yet but lgtm | 19:02 |
*** nati_ueno has quit IRC | 19:02 | |
clarkb | pleia2: also its a bit embarrassing that we are still running kibana 2 >_> | 19:02 |
krtaylor | anteaya, hehheh #action use #action more | 19:02 |
pleia2 | clarkb: thanks | 19:03 |
jerryz | anteaya: could you elaborate more about concerns on 3rd party CI in gate pipeline? | 19:03 |
anteaya | krtaylor: yes | 19:03 |
clarkb | also I think I have working xfce and dmenu now | 19:03 |
pleia2 | \o/ | 19:03 |
clarkb | + xmonad | 19:03 |
jerryz | for stackforge, in particular | 19:03 |
anteaya | jerryz: it isn't a concern it is a process | 19:03 |
clarkb | I put a sleep prior to xmonad --replace and that allows xfce to start everything it needs first without being confused by xmonad | 19:04 |
*** mattoliverau has quit IRC | 19:04 | |
anteaya | to be in the gate requires a project to go through the incubation and integration process | 19:04 |
clarkb | so warning hax | 19:04 |
pleia2 | hah :) | 19:04 |
*** marcoemorais has quit IRC | 19:04 | |
krotscheck | phschwartz: So, initial tests on the gerrit API shows that it just doesn’t respond to OPTIONS requests at all. | 19:04 |
anteaya | this is a process outlined and regulated by the tc | 19:04 |
krotscheck | phschwartz: Which means that CORS is likely not supported. | 19:04 |
krtaylor | jerryz, are you referring to your local zuul/jenkins or upstream community's | 19:05 |
anteaya | any project can go through the process, but a project that has not gone through the process doesn't get into the gate | 19:05 |
krotscheck | phschwartz: It might be possible to just have the vhost proxy to an existing gerrit install so we don’t encounter the browser sandbox. | 19:05 |
rcarrill` | could any core reviewers have a look at https://review.openstack.org/#/c/93773/1 ? | 19:06 |
jerryz | krtaylor: adding them in my local zuul is not literally gating because only community's has the privilege to block or merge a patch | 19:06 |
*** andreykurilin_ has joined #openstack-infra | 19:06 | |
phschwartz | krotscheck: I was experiencing the same issue internal here. | 19:07 |
*** mattoliverau has joined #openstack-infra | 19:07 | |
anteaya | jerryz: yes and that priviledge is administered by the tc as I outlined above | 19:08 |
*** maxbit has joined #openstack-infra | 19:09 | |
*** rcarrillocruz has joined #openstack-infra | 19:09 | |
jerryz | anteaya: i was talking about the 3rd party jenkins jobs getting into gate pipeline. not giving 3rd party zuul/gerrit-trigger plugin the same privilege as community's zuul | 19:10 |
*** rcarrill` has quit IRC | 19:10 | |
*** marcoemorais has joined #openstack-infra | 19:10 | |
rcarrillocruz | zaro: thx for the +1 | 19:11 |
jerryz | anteaya: like registering jenkins to zuul.o.o and let zuul distribute 3rd party jobs to 3rd party environments | 19:11 |
*** dstanek is now known as dstanek_zzz | 19:12 | |
*** mkoderer has quit IRC | 19:12 | |
anteaya | jerryz: okay I think we might not be using the same meanings for the words we are typing | 19:12 |
anteaya | what do you mean when you say gate pipeline? | 19:12 |
anteaya | that has a very specific meaning and I don't know what you mean when you use it | 19:13 |
*** dstanek_zzz is now known as dstanek | 19:14 | |
jerryz | anteaya: my point is it is useful to test against several patches merged together within a project or across projects, which is done in gate pipeline | 19:14 |
anteaya | yes, which is why we use it | 19:15 |
jerryz | anteaya: check pipeline is per patch test | 19:15 |
anteaya | mostly for speed | 19:15 |
anteaya | yes it is | 19:15 |
jerryz | anteaya: what if a 3rd party test would fail on patches merged together, we submit a bug later? but that would be hard to investigate | 19:16 |
anteaya | jerryz: have you come across that? | 19:16 |
anteaya | do you have a specific example of the third part test that passed in check and failed as a result of patches merged together in the gate? | 19:17 |
jerryz | anteaya: no. just saying | 19:17 |
anteaya | okay, when you have a specific example come back with the details | 19:17 |
*** igor_ has joined #openstack-infra | 19:17 | |
anteaya | then we have something to look at and can work towards a solution at that time | 19:17 |
anteaya | is that fair? | 19:18 |
jerryz | anteaya: the same reason as other gate- jobs | 19:18 |
*** hipster has joined #openstack-infra | 19:18 | |
*** Hal_ has quit IRC | 19:18 | |
*** cody-somerville has quit IRC | 19:19 | |
anteaya | right now you want to engage in a hypothetical conversation, and I would like to address our actual issues | 19:19 |
phschwartz | krotscheck: Interesting when I hit the api from python it works like a charm on a local instance, fails against reviews.openstack.org | 19:19 |
phschwartz | javascript fails on both | 19:19 |
anteaya | and I have agreed to discuss this again with you, if and when this becomes an actual issue | 19:19 |
jerryz | anteaya: fair | 19:19 |
anteaya | jerryz: great thank you | 19:19 |
krotscheck | phschwartz: What version are you running locally? I think the infra gerrit is still a couple of versions behind. | 19:20 |
phschwartz | 2.8.4 | 19:20 |
* anteaya heads out for a walk | 19:20 | |
krotscheck | phschwartz: So that’s not it. Hrm. | 19:20 |
krotscheck | phschwartz: Well, the JS thing is probably the browser sandbox, that’s a known thing. | 19:21 |
phschwartz | I will have to find a work around for that with vinz | 19:21 |
clarkb | anteaya: jerryz fwiw I think jeblair has been considering using authenticated gearman with ACLs to have our zuul run jobs in other jenkinses | 19:21 |
krotscheck | phschwartz: mod_proxy? | 19:21 |
clarkb | I don't think this completely solves what you are talking about but it may be one mechanism to do that should we get it working | 19:21 |
phschwartz | krotscheck: I haven't tried it, but it should work (fingers crossed) | 19:22 |
clarkb | krotscheck: we are only one minor version behind now and that released during the summit iirc | 19:22 |
jerryz | clarkb: great | 19:22 |
*** igor_ has quit IRC | 19:22 | |
phschwartz | Let me setup mod_proxy and try it | 19:23 |
*** hipster has quit IRC | 19:23 | |
clarkb | ok I *think* I have this machine working the way I want it to | 19:23 |
jerryz | clarkb: after zuul.o.o 's gearman server get an idea what jobs other jenkinses have, we could add 3rd party jobs in check, gate pipelines or just in check pipeline | 19:24 |
*** lcheng has quit IRC | 19:24 | |
clarkb | jerryz: well maybe | 19:24 |
clarkb | there is still a lot of concern around depending on things we have no contro lover | 19:24 |
clarkb | so for example if a third party system were to be turned off for a week and we gate on it there is nothing we could do to merge code other than remove those tests from teh gate | 19:25 |
clarkb | so I think current feeling on that subject is to simply never gate on it in the first place | 19:25 |
jerryz | clarkb: that is a issue about trust , reliability | 19:26 |
*** nati_uen_ has quit IRC | 19:26 | |
clarkb | jerryz: well its deeper than that but that is part of it | 19:26 |
*** nati_ueno has joined #openstack-infra | 19:26 | |
clarkb | there is also concern for making devs depend on something they have no influence over | 19:26 |
*** praneshp_ has quit IRC | 19:29 | |
openstackgerrit | Ricardo Carrillo Cruz proposed a change to openstack-infra/config: Parameterize SSL certificate attributes on review.pp manifest https://review.openstack.org/93687 | 19:29 |
*** marun has joined #openstack-infra | 19:31 | |
*** dims has joined #openstack-infra | 19:34 | |
*** mwagner_lap has joined #openstack-infra | 19:35 | |
clarkb | pleia2: any idea why light locker is distinct from xscreensaver? | 19:38 |
clarkb | pleia2: I only ask because it is possible to get both to lock the screen requiring your password twice | 19:38 |
*** cody-somerville has joined #openstack-infra | 19:40 | |
pleia2 | clarkb: I don't know why, might be because they both want to interface with the suspend system if it's unlocking that, so they fight, I know you simply can't have them both (was written about here: http://xubuntu.org/news/screen-locking-in-xubuntu-14-04/ ) | 19:40 |
*** nati_ueno has quit IRC | 19:41 | |
*** marcoemorais has quit IRC | 19:42 | |
*** marcoemorais has joined #openstack-infra | 19:42 | |
clarkb | ah I can get rid of one or the other apparently. I htink ahve have them both working right now though | 19:42 |
*** Ryan_Lane1 is now known as Ryan_lane | 19:44 | |
*** Ryan_lane has joined #openstack-infra | 19:44 | |
*** Ryan_lane is now known as Ryan_Lane | 19:44 | |
*** _nadya_ has quit IRC | 19:44 | |
*** hipster has joined #openstack-infra | 19:47 | |
*** julim has quit IRC | 19:49 | |
*** markmcclain has quit IRC | 19:51 | |
*** salv-orlando has quit IRC | 19:52 | |
*** duran_ has joined #openstack-infra | 19:52 | |
*** hipster has quit IRC | 19:52 | |
*** simonmcc has quit IRC | 19:52 | |
*** arnaud_ has joined #openstack-infra | 19:52 | |
*** julim has joined #openstack-infra | 19:53 | |
*** duran_ has quit IRC | 19:53 | |
*** sandywalsh_ has joined #openstack-infra | 19:53 | |
*** jaypipes has quit IRC | 19:53 | |
*** greghaynes has quit IRC | 19:53 | |
*** gothicmindfood has quit IRC | 19:53 | |
*** jgriffith has quit IRC | 19:53 | |
*** gothicmindfood has joined #openstack-infra | 19:54 | |
*** amcrn has quit IRC | 19:54 | |
*** zul has quit IRC | 19:54 | |
*** jraim has quit IRC | 19:54 | |
*** duran has quit IRC | 19:54 | |
*** ewindisch has quit IRC | 19:54 | |
*** soren has quit IRC | 19:54 | |
*** sandywalsh has quit IRC | 19:54 | |
*** mdenny has quit IRC | 19:54 | |
*** hughsaunders has quit IRC | 19:54 | |
*** jaypipes has joined #openstack-infra | 19:54 | |
*** sdake has quit IRC | 19:55 | |
*** tteggel has quit IRC | 19:55 | |
*** cody-somerville has quit IRC | 19:55 | |
*** tteggel has joined #openstack-infra | 19:55 | |
*** dangers_away has quit IRC | 19:55 | |
*** jgriffith has joined #openstack-infra | 19:56 | |
*** soren has joined #openstack-infra | 19:56 | |
*** whoops_ has joined #openstack-infra | 19:56 | |
*** cp16net has quit IRC | 19:56 | |
*** sarob has joined #openstack-infra | 19:56 | |
*** jraim has joined #openstack-infra | 19:57 | |
*** dangers_away has joined #openstack-infra | 19:57 | |
*** maxbit has quit IRC | 19:57 | |
*** mdenny has joined #openstack-infra | 19:57 | |
*** packet has quit IRC | 19:58 | |
*** zul has joined #openstack-infra | 19:58 | |
*** nati_ueno has joined #openstack-infra | 19:58 | |
*** whoops has quit IRC | 19:58 | |
*** whoops_ is now known as whoops | 19:58 | |
*** praneshp_ has joined #openstack-infra | 20:00 | |
*** praneshp_ has quit IRC | 20:00 | |
*** amcrn has joined #openstack-infra | 20:00 | |
*** harlowja has quit IRC | 20:01 | |
*** cp16net has joined #openstack-infra | 20:01 | |
*** marcoemorais has quit IRC | 20:01 | |
*** melwitt has quit IRC | 20:02 | |
*** marcoemorais has joined #openstack-infra | 20:02 | |
*** melwitt has joined #openstack-infra | 20:02 | |
*** harlowja has joined #openstack-infra | 20:02 | |
*** rlandy has quit IRC | 20:02 | |
*** dangers_away has quit IRC | 20:02 | |
*** ewindisch has joined #openstack-infra | 20:03 | |
*** dangers_away has joined #openstack-infra | 20:04 | |
*** hughsaunders has joined #openstack-infra | 20:05 | |
*** alff has quit IRC | 20:06 | |
*** sdake has joined #openstack-infra | 20:07 | |
*** simonmcc has joined #openstack-infra | 20:08 | |
jeblair | restarting zuul to pick up the requires change | 20:08 |
*** cody-somerville has joined #openstack-infra | 20:08 | |
clarkb | k | 20:08 |
*** greghaynes has joined #openstack-infra | 20:09 | |
jerryz | jeblair: i see in dev list there is someone reporting dead loop of merge-failure | 20:10 |
clarkb | jerryz: that is the issue jeblair is fixing right now | 20:10 |
*** mattoliverau has quit IRC | 20:10 | |
clarkb | jeblair: I think I am all set on trusty now. Going to review the stack of gertty changes then update gertty to dive into reviews | 20:10 |
clarkb | I suppsoe I should test the tox fix first | 20:10 |
*** praneshp_ has joined #openstack-infra | 20:10 | |
jeblair | jerryz: https://review.openstack.org/#/c/94243/ | 20:11 |
jeblair | zuul is restarted and i approved that ^ change; when it lands and zuul updates, we should be good | 20:11 |
*** dims has quit IRC | 20:11 | |
*** dims has joined #openstack-infra | 20:12 | |
*** Hal_ has joined #openstack-infra | 20:13 | |
*** mattoliverau has joined #openstack-infra | 20:14 | |
*** e0ne has quit IRC | 20:14 | |
*** salv-orlando has joined #openstack-infra | 20:14 | |
openstackgerrit | Sean Dague proposed a change to openstack-infra/jeepyb: add dashboard support to jeepyb https://review.openstack.org/94260 | 20:16 |
*** zns has quit IRC | 20:16 | |
*** sarob has quit IRC | 20:17 | |
*** sarob has joined #openstack-infra | 20:17 | |
*** dims has quit IRC | 20:18 | |
clarkb | I just ran tox -epy27 in zuul's repo with tox 1.7.2.dev1. I think the fix is working \o/ | 20:18 |
*** igor_ has joined #openstack-infra | 20:18 | |
openstackgerrit | A change was merged to openstack-infra/config: Use current-patchset pipeline requirement https://review.openstack.org/94243 | 20:18 |
*** Hal_ has quit IRC | 20:19 | |
clarkb | jeblair: what is upgrade path for gertty with the alembic change? do I need to delete the existing DB? | 20:21 |
clarkb | looks like there is a call to migrate in db.py so it may do it gracefully? | 20:21 |
jeblair | clarkb: no, it should stamp your existing db and then upgrade | 20:21 |
jeblair | yep | 20:21 |
*** mwagner_lap has quit IRC | 20:22 | |
*** sarob has quit IRC | 20:22 | |
aconrad | zaro: yes, another jjb release would be great | 20:22 |
jeblair | clarkb: (just note that if that changes again before merging, you might have to drop your local db or otherwise manually fix it) | 20:23 |
*** igor_ has quit IRC | 20:23 | |
clarkb | ah that is what the stamp() does | 20:23 |
clarkb | jeblair: yeah not too worried about it | 20:23 |
clarkb | jeblair: what does the mako template do? did it generate the initial_schema.py file? | 20:23 |
*** Ryan_Lane1 has joined #openstack-infra | 20:23 | |
*** Ryan_Lane has quit IRC | 20:24 | |
jeblair | clarkb: i believe the mako template is what's used when you autogenerate a new schema (so yes, and will be used in future too) | 20:24 |
jeblair | s/schema/revision/ | 20:25 |
clarkb | jeblair: +1 just one comment in the config file | 20:25 |
jeblair | clarkb: answer is "yes", oddly enough. :) | 20:26 |
anteaya | jeblair: do you want vinz to have channel #vinz or #openstack-vinz? | 20:27 |
jeblair | anteaya: i would like discussion to be in #openstack-infra. see scrollback. | 20:27 |
anteaya | ah sorry | 20:28 |
jeblair | anteaya: (applies to storyboard too) | 20:28 |
anteaya | great thanks | 20:29 |
* anteaya edits her comment on the vinz repo creation patch | 20:29 | |
*** dstanek is now known as dstanek_zzz | 20:30 | |
*** lcheng has joined #openstack-infra | 20:30 | |
jeblair | delete from change_messages where change_id=92477 and author_id=3 and patchset_patch_set_id=3 and date(written_on)="2014-05-19"; | 20:30 |
jeblair | clarkb: i'm going to run that on production gerrit to get rid of 10149 extra rows. | 20:30 |
jeblair | clarkb: look good? | 20:30 |
clarkb | isn't jenkins user 2? | 20:31 |
clarkb | otherwise yes lgtm | 20:31 |
jeblair | clarkb: mordred is 2 | 20:31 |
clarkb | ah ok | 20:31 |
anteaya | so we want vinz-webclient-core and vinz-ptl gerrit groups? | 20:32 |
anteaya | s/so/do | 20:32 |
*** julim has quit IRC | 20:32 | |
*** cody-somerville has quit IRC | 20:32 | |
sdague | jeblair: so I'm starting to look into zuul, and was curious to find this - https://github.com/openstack-infra/zuul/blob/master/zuul/trigger/gerrit.py#L348-L354 | 20:33 |
sdague | where is that dependsOn getting calculated? | 20:33 |
jeblair | sdague: it's supplied by gerrit | 20:34 |
anteaya | phschwartz: what does use-storyboard: true do in review.projects.yaml? | 20:34 |
anteaya | phschwartz: do you know, because I don't | 20:34 |
krotscheck | anteaya: It adds that project to the ones loaded into storyboard. | 20:34 |
*** Hefeweizen has joined #openstack-infra | 20:35 | |
krotscheck | Basically “Hey, I’d rather use storyboard than launchpad, let me use it” | 20:35 |
anteaya | krotscheck: cool, and we want that with vinz? | 20:36 |
sdague | jeblair: it seems that depends on is 1 -> 1, however neededBy code already supports a list. Do you think injecting additional data in that area would be the right approach here? | 20:36 |
anteaya | was that a project we gave the go ahead for using storyboard? | 20:36 |
jeblair | anteaya: yeah, any time we can start a new infra project on storyboard yes; it's mostly the ones with existing bugs that we haven't moved over | 20:36 |
anteaya | there is just a lot of stuff in this patch I have never seen before, hence all the questions | 20:36 |
anteaya | jeblair: good enough | 20:36 |
*** amcrn has quit IRC | 20:36 | |
clarkb | jeblair: can you see my comment on the update column helper script change? I am going to grab lunch now but I have a small concern in that change | 20:37 |
anteaya | so last question we want continuious publication from master on vinz, same as storyboard | 20:37 |
*** ramashri has quit IRC | 20:37 | |
*** rcarrill` has joined #openstack-infra | 20:39 | |
*** rcarrillocruz has quit IRC | 20:41 | |
*** e0ne has joined #openstack-infra | 20:41 | |
anteaya | I'll post my comments thus far on the patch, it needs another patchset anyway | 20:41 |
*** amcrn has joined #openstack-infra | 20:42 | |
jeblair | clarkb: responded | 20:42 |
*** zns has joined #openstack-infra | 20:43 | |
clarkb | jeblair: gotcha that is what the other index code is for | 20:43 |
*** _nadya_ has joined #openstack-infra | 20:45 | |
pleia2 | did the Zanata folks say anything about OS preference? It's supported by Redhat so I wonder if we may have another CentOS candidate on our hands | 20:45 |
jeblair | SergeyLukjanov: if you have a minute to look at https://review.openstack.org/#/c/92745/ that would be nice -- i think i found the problems you were talking about and fixed them. | 20:45 |
*** e0ne_ has joined #openstack-infra | 20:45 | |
clarkb | pleia2: it should run on ubuntu it is very similar to gerrit | 20:46 |
*** e0ne has quit IRC | 20:46 | |
clarkb | and I think they have puppet we can use | 20:46 |
pleia2 | clarkb: ok good, I'll go that route first | 20:46 |
pleia2 | and I'll ask them about puppet | 20:47 |
*** hipster has joined #openstack-infra | 20:47 | |
jeblair | pleia2: they indicated they are very happy to help with this, though they are normally in an AU timezone | 20:47 |
* pleia2 nods | 20:48 | |
pleia2 | I joined their channel and user list | 20:48 |
*** amcrn has quit IRC | 20:51 | |
*** hipster has quit IRC | 20:52 | |
*** e0ne_ has quit IRC | 20:55 | |
phschwartz | anteaya: I will submit a new patchset for you. | 20:57 |
openstackgerrit | Philip Marc Schwartz proposed a change to openstack-infra/config: Creation of vinz project in the openstack-infra scheme. https://review.openstack.org/93953 | 20:58 |
sdague | jeblair: what do you think about turning dependsOn into a collection, that will support the current set of operations that the needed_changes bit supports, but will let us have multiple | 20:59 |
*** SumitNaiksatam has joined #openstack-infra | 20:59 | |
*** yamahata has joined #openstack-infra | 20:59 | |
*** pdmars has quit IRC | 20:59 | |
morganfainberg | clarkb, pleia2, anteaya, anyone know if it is more appopriate to access the approvals['type'] field in gerrit's response or the approval['description'] to know if it's a "Workflow" score? | 21:00 |
clarkb | morganfainberg: 'type' | 21:00 |
* morganfainberg is trying to fix WIP tracking in reviewday | 21:00 | |
morganfainberg | clarkb, thanks! | 21:01 |
*** james_li_ has joined #openstack-infra | 21:01 | |
morganfainberg | clarkb, pleia2, second question, do we want to exclude WIP reviews from reviewday? would be easy to do [i think] vs. just pushing them to the floor | 21:01 |
morganfainberg | clarkb, pleia2, since i'm fixing reviewday up as is | 21:01 |
*** Sukhdev has joined #openstack-infra | 21:02 | |
*** markmcclain has joined #openstack-infra | 21:02 | |
*** james_li has quit IRC | 21:03 | |
*** james_li_ is now known as james_li | 21:03 | |
*** andreykurilin_ has quit IRC | 21:04 | |
pleia2 | morganfainberg: actually, excluding them entirely makes sense to me, I'd say we put up a review and see what dprince things too (he's the other core reviewer) | 21:04 |
pleia2 | s/things/thinks | 21:04 |
sdague | jeblair: https://etherpad.openstack.org/p/zuul-depends-on - at the bottom is my current thinking, would be curious to get your thoughts, so I can go dive on that tomorrow | 21:04 |
morganfainberg | pleia2, great! I'll shuffle this around and instead of flooring the value to 0 exclude them. wont be hard to floor the value if that turns out to be undesireable | 21:05 |
*** rcarrillocruz has joined #openstack-infra | 21:05 | |
*** andreykurilin_ has joined #openstack-infra | 21:05 | |
morganfainberg | pleia2, thanks! | 21:05 |
*** mfer has quit IRC | 21:05 | |
*** rfolco has quit IRC | 21:06 | |
anteaya | phschwartz: thanks | 21:07 |
*** rcarrill` has quit IRC | 21:07 | |
*** sballe has joined #openstack-infra | 21:08 | |
*** rfolco has joined #openstack-infra | 21:08 | |
*** Ryan_Lane1 is now known as Ryan_Lane | 21:09 | |
*** Ryan_Lane has joined #openstack-infra | 21:09 | |
jeblair | sdague: that sounds good | 21:09 |
openstackgerrit | Morgan Fainberg proposed a change to openstack-infra/reviewday: Remove WIP reviews from reviewday report https://review.openstack.org/94266 | 21:09 |
sdague | jeblair: cool | 21:10 |
morganfainberg | pleia2, ^ that _should_ do it. *double checks* | 21:10 |
pleia2 | morganfainberg: thanks, I'll test it here too :) | 21:10 |
*** dstanek_zzz is now known as dstanek | 21:14 | |
*** igor_ has joined #openstack-infra | 21:19 | |
*** igor_ has quit IRC | 21:23 | |
morganfainberg | pleia2, i think i found a bug in reviewday testing it here. | 21:23 |
pleia2 | k | 21:23 |
morganfainberg | pleia2, Idf078705e2f5305539aaff62396ec9699e31e568 doesn't have any patchsets | 21:24 |
*** dstanek is now known as dstanek_zzz | 21:24 | |
morganfainberg | pleia2, my guess is we need some extra error handling in here. i'll propose some fixes to address these key errors | 21:24 |
pleia2 | morganfainberg: hm, ok thanks | 21:24 |
morganfainberg | pleia2, http://pasteraw.com/nac78frdwllblqvjyxg23qctzqu988z - if youre interested | 21:26 |
*** zns has quit IRC | 21:26 | |
morganfainberg | ok the specific error case | 21:26 |
pleia2 | morganfainberg: thanks | 21:26 |
morganfainberg | oh. we already catch the error, we just reraise | 21:27 |
*** lcostantino has joined #openstack-infra | 21:27 | |
morganfainberg | *blink* | 21:27 |
pleia2 | hmm | 21:28 |
morganfainberg | http://git.openstack.org/cgit/openstack-infra/reviewday/tree/bin/reviewday#n77 | 21:28 |
morganfainberg | i'm guessing we don't want to re-raise in these cases. | 21:29 |
*** atiwari has quit IRC | 21:30 | |
andreaf | jeblair: hi, you around? | 21:31 |
*** melwitt has quit IRC | 21:32 | |
*** melwitt has joined #openstack-infra | 21:32 | |
*** harlowja has quit IRC | 21:32 | |
*** mbacchi has quit IRC | 21:33 | |
andreaf | jeblair: I'm trying to find out what was the fix to https://bugs.launchpad.net/nova/+bug/1285288 - and hoping you may know - from you comment on the issue | 21:33 |
uvirtbot | Launchpad bug 1285288 in nova "gate-nova-docs No module named ...." [Undecided,Incomplete] | 21:33 |
*** harlowja has joined #openstack-infra | 21:33 | |
*** trainoti has quit IRC | 21:34 | |
*** aconrad has quit IRC | 21:36 | |
*** prad has quit IRC | 21:36 | |
jeblair | andreaf: i have no idea, i only removed that bug from openstack-ci because i felt it was misfiled. | 21:41 |
andreaf | jeblair: ah ok too bad | 21:41 |
andreaf | jeblair: I'm hitting the issue now so trying to find out where it came from | 21:42 |
*** dims has joined #openstack-infra | 21:44 | |
*** dstanek_zzz is now known as dstanek | 21:45 | |
openstackgerrit | Morgan Fainberg proposed a change to openstack-infra/reviewday: Remove WIP reviews from reviewday report https://review.openstack.org/94266 | 21:45 |
openstackgerrit | Morgan Fainberg proposed a change to openstack-infra/reviewday: Ensure mergeprop instantiation failures are not fatal https://review.openstack.org/94272 | 21:45 |
morganfainberg | pleia2, looks like reviewday data stopped updating a few hours ago due to the mergeprop bug. those fixes should solve both issues. | 21:46 |
andreaf | lcostantino: ping | 21:46 |
*** mwagner_lap has joined #openstack-infra | 21:46 | |
lcostantino | andreaf, hi | 21:46 |
andreaf | lcostantino: I'm hitting a bug you filed https://bugs.launchpad.net/nova/+bug/1285288 and I was wondering if you ever found the cause for it? | 21:47 |
uvirtbot | Launchpad bug 1285288 in nova "gate-nova-docs No module named ...." [Undecided,Incomplete] | 21:47 |
*** hipster has joined #openstack-infra | 21:47 | |
*** prad has joined #openstack-infra | 21:47 | |
lcostantino | andreaf, no, after reissuing another build, it started to work | 21:48 |
*** virmitio has joined #openstack-infra | 21:48 | |
andreaf | lcostantino: ok maybe I've just been unlucky and hit that twice - thanks I'll try again | 21:49 |
lcostantino | andreaf, ok, does it happend on your local env if you try to build docs? | 21:49 |
*** mriedem has quit IRC | 21:49 | |
andreaf | yes it happened in my local env - well it's odd because I had the error in the logs but tox reported ok | 21:50 |
*** _nadya_ has quit IRC | 21:51 | |
*** aconrad has joined #openstack-infra | 21:52 | |
*** hipster has quit IRC | 21:52 | |
pleia2 | morganfainberg: hm, actually this change from last week is failing in the same way https://review.openstack.org/#/c/93901/ not sure yet what's going on | 21:53 |
morganfainberg | yeah looks like a pip error | 21:53 |
morganfainberg | well, pip related issue | 21:53 |
morganfainberg | or pypi | 21:53 |
morganfainberg | or i might be mis-reading it | 21:54 |
*** andreaf has quit IRC | 21:55 | |
*** dstanek is now known as dstanek_zzz | 21:55 | |
*** andreaf has joined #openstack-infra | 21:55 | |
openstackgerrit | A change was merged to openstack-infra/config: Hacking: enable the Python 3 gate. https://review.openstack.org/84690 | 21:55 |
*** hashar has quit IRC | 21:57 | |
*** igor_ has joined #openstack-infra | 21:58 | |
openstackgerrit | Edward Raigosa proposed a change to openstack-infra/config: Implement security class for gerrit base security https://review.openstack.org/94007 | 21:59 |
openstackgerrit | Michael Krotscheck proposed a change to openstack-infra/storyboard-webclient: Added search icon to typeahead fields https://review.openstack.org/94273 | 21:59 |
*** Kiall has quit IRC | 21:59 | |
*** markmcclain has quit IRC | 21:59 | |
*** _afazekas is now known as afzekas | 22:00 | |
*** rfolco has quit IRC | 22:00 | |
*** afzekas is now known as afazekas | 22:01 | |
notmyname | clarkb: were you in the "swift growing pains" talk last week? | 22:01 |
*** mattoliverau has quit IRC | 22:02 | |
*** andreykurilin_ has quit IRC | 22:03 | |
*** igor_ has quit IRC | 22:03 | |
notmyname | jeblair: what's the feasibility of adding a new tag/column to reviews in gerrit? | 22:03 |
notmyname | jeblair: last week we talked about having "core sponsors" for patches. I'm planning on implementing that in a soft way (informal tracking on a wiki page) to start, but it could be interesting to actually have that in gerrit I think | 22:03 |
*** sweston has quit IRC | 22:04 | |
openstackgerrit | Anita Kuno proposed a change to openstack-infra/config: Create an infra-specs repo https://review.openstack.org/93929 | 22:04 |
*** sweston has joined #openstack-infra | 22:05 | |
anteaya | editing the commit message and topic name in the gui can be undone by a subsequent patch submitting via ssh | 22:05 |
anteaya | now I know | 22:06 |
*** sweston has quit IRC | 22:06 | |
anteaya | s/submitting/submitted | 22:06 |
*** homeless has quit IRC | 22:06 | |
clarkb | notmyname: I was | 22:07 |
*** mattoliverau has joined #openstack-infra | 22:08 | |
clarkb | notmyname: I don't understand how a new review column would help "core sponsors" though | 22:08 |
pleia2 | so reviewday checks are failing, seems to be a pep8 thing? maybe something in the tox.ini needs to be updated, I'm stumped http://logs.openstack.org/66/94266/2/check/gate-reviewday-pep8/8d3aec1/console.html | 22:08 |
*** sweston has joined #openstack-infra | 22:09 | |
notmyname | clarkb: just thinking of a central place already in a dev/reviewer workflow that show's patches assigned to you | 22:10 |
notmyname | clarkb: different than "reviewers". more like "of the special reviewers on this patch (core), this person is the most specialist (core sponsor)" | 22:11 |
*** gondoi is now known as zz_gondoi | 22:11 | |
clarkb | notmyname: so you can add a column for that but would have to set that value manually (as the person being the specialist) | 22:11 |
notmyname | clarkb: ya, that's what I'd expect and want | 22:12 |
notmyname | clarkb: this is an experiment to be sure. so we'll be informal to start. but if it works, I'm just thinking of ways to integrate it more tightly with the openstack dev workflow | 22:12 |
clarkb | pleia2: https://git.openstack.org/cgit/openstack-infra/reviewday/tree/setup.py#n18 is the thing breaking | 22:12 |
clarkb | pleia2: mordred may have ideas on how to make that machinery work better | 22:13 |
clarkb | notmyname: ok, just making sure that is what you had in mind | 22:14 |
clarkb | I was thinking it would be more automagic | 22:14 |
jeblair | krotscheck: the best way to ensure that patches land in a particular order is to make them dependent in git; if you don't want to do that but still would like an informal ordering, noting in the _later_ commit message would be a good idea | 22:14 |
*** isviridov is now known as isviridov|away | 22:15 | |
jeblair | krotscheck: (i'm reviewing 93545 after having approved the one it referenced) | 22:15 |
*** doug-fish has left #openstack-infra | 22:15 | |
clarkb | notmyname: https://review.openstack.org/Documentation/config-labels.html you probably want a -1 to 1 label that copies min and max values using the NoOP function | 22:15 |
openstackgerrit | A change was merged to openstack-infra/storyboard: Stories can now be filtered by assignee_id https://review.openstack.org/91912 | 22:15 |
jeblair | krotscheck: i don't think it matters in this case, just noting for future reference | 22:15 |
clarkb | notmyname: actually do 0 to +1 and copy max value | 22:16 |
clarkb | notmyname: doing that would allow reviewers to +1 as a specialist on that change | 22:16 |
clarkb | and set to 0 if that changes | 22:16 |
jeblair | notmyname: sounds interesting! | 22:17 |
openstackgerrit | Salvatore Orlando proposed a change to openstack-infra/elastic-recheck: Delete fingerprint for 1224001 https://review.openstack.org/94276 | 22:17 |
*** _nadya_ has joined #openstack-infra | 22:18 | |
clarkb | notmyname: is that what you had in mind? | 22:18 |
notmyname | clarkb: I'm not sure. | 22:18 |
clarkb | notmyname: I can push a patch really quick to infra/config to give us a basis for the discussion | 22:19 |
notmyname | clarkb: I was thinking of a column that would have eg "Clark Boylan" listed. not that "there exists a core sponsor" | 22:19 |
openstackgerrit | A change was merged to openstack-infra/storyboard: Fix a migration file name https://review.openstack.org/93545 | 22:19 |
notmyname | clarkb: but ya, i think that would be helpful to see | 22:19 |
clarkb | notmyname: so gerrit labels have numeric values assigned by humans | 22:20 |
jeblair | krotscheck, NikitaKonovalov: does storyboard actually support an xml api? the docs have examples for it, but i hope that's just an accidental copy from openstack. | 22:20 |
notmyname | clarkb: ah, ok. heh, I guess I could be "reviewer 23" and then sort by values looking for things assigned to number "23" ;-) | 22:20 |
notmyname | clarkb: but that sounds like a terrible idea | 22:20 |
clarkb | well they aren't arbitrary either. its just like your normal code reviews | 22:21 |
clarkb | so it would say Clark Boylan +1 | 22:22 |
lifeless | mordred_: clarkb: ping | 22:22 |
lifeless | (pbr) | 22:22 |
openstackgerrit | A change was merged to openstack-infra/storyboard: Minor fixes to docs page https://review.openstack.org/93909 | 22:22 |
jeblair | notmyname: and i believe that's searchable now (i think you can search for changes where you have left a vote of a certain value in a particular category; so you could search for changes where you are the core sponsor) | 22:23 |
notmyname | clarkb: ok, I think you've lost me now :-) | 22:23 |
jeblair | clarkb: i think a picture is worth 1k words here | 22:23 |
notmyname | jeblair: clarkb +1 | 22:24 |
notmyname | or +1000 as the case may be | 22:24 |
jeblair | krotscheck: ttx has a question on 91196 | 22:25 |
openstackgerrit | Clark Boylan proposed a change to openstack-infra/config: Add swift sponsors review column https://review.openstack.org/94277 | 22:25 |
clarkb | jeblair: probably | 22:25 |
clarkb | anyways I think that will do the actual config change | 22:26 |
openstackgerrit | Michael Krotscheck proposed a change to openstack-infra/storyboard: Added priorities to tasks https://review.openstack.org/91675 | 22:26 |
krotscheck | jeblair: Checking | 22:26 |
notmyname | clarkb: is there a way to see what that looks like (the result of that config change)? | 22:26 |
jeblair | notmyname: http://paste.openstack.org/show/80961/ | 22:27 |
jeblair | notmyname: that's a quick mockup of the review table | 22:27 |
jeblair | clarkb: ^ | 22:27 |
clarkb | jeblair beat me to it | 22:27 |
jeblair | clarkb: copied from my current review in gertty ;) | 22:28 |
zaro | aconrad: yeah ok, new jjb release tomorrow. | 22:28 |
notmyname | jeblair: clarkb: ah, ok. so that's in the table that is in the patch itself. is that able to be reflected in the general reviews page? | 22:29 |
clarkb | notmyname: yes it will show up in the list of reviews page too | 22:30 |
*** james_li has quit IRC | 22:30 | |
notmyname | jeblair: clarkb: eg on https://review.openstack.org/#/q/status:open+project:openstack/swift,n,z the top one is from tom | 22:30 |
clarkb | notmyname: how it shows up there depends on how you have your gerrit settings set | 22:30 |
clarkb | I will only see it say +1 under sponsors if there is a sponsor. sdague will see notmyname +1 | 22:30 |
notmyname | clarkb: I was imagining another column next to tom's name that eg has mine, if I'm the core sponsor there | 22:30 |
notmyname | ok | 22:30 |
*** dims has quit IRC | 22:30 | |
clarkb | that is a user configurable option | 22:30 |
*** smarcet has quit IRC | 22:30 | |
clarkb | at https://review.openstack.org/#/settings/preferences | 22:31 |
*** aconrad has quit IRC | 22:31 | |
clarkb | "Display Person Name in Review Category" | 22:31 |
notmyname | clarkb: ah, ok! | 22:31 |
notmyname | clarkb: with that system, can I (ie ptl) assign people to that? | 22:33 |
zaro | wow! took me all day to get gearman-java working again. | 22:33 |
clarkb | notmyname: no, individuals must do it themselves | 22:33 |
notmyname | clarkb: ok | 22:34 |
*** mrodden has quit IRC | 22:35 | |
jerryz | clarkb: will publishing a draft trigger check pipeline? | 22:35 |
clarkb | jerryz: no | 22:35 |
clarkb | also please don't use drafts | 22:35 |
sdague | notmyname: what about just actually adding reviewers to reviews? | 22:36 |
jerryz | clarkb: may i ask why? | 22:36 |
clarkb | jerryz: because they are a broken feature in gerrit and we can't disable them | 22:36 |
*** briancurtin has left #openstack-infra | 22:37 | |
*** aconrad has joined #openstack-infra | 22:37 | |
jeblair | clarkb, zaro: did changing acls for drafts not work? | 22:37 |
clarkb | jerryz: if you depend on a change with a current draft patchset it breaks the child change, if you push a draft patchset atop published patchsets it confuses reviewers, and so n | 22:37 |
clarkb | jeblair: correct. drafts super special and ignore acls apparently. This is fixed in 2.9 | 22:37 |
clarkb | jerryz: and you don't gain anything from drafts. It is better to use work in progress | 22:38 |
*** thedodd has quit IRC | 22:38 | |
bknudson | for some reason gerrit table view doesn't show +1 V for a change, even though if you look at the change Jenkins has +1. | 22:38 |
clarkb | jerryz: specifically they aren't secure though some people think they are and use them that way | 22:38 |
clarkb | bknudson: link? | 22:38 |
bknudson | clarkb: https://review.openstack.org/#/q/project:openstack/keystone+status:open+owner:blk-u,n,z | 22:38 |
*** jgrimm has quit IRC | 22:38 | |
bknudson | the top one | 22:39 |
*** ptippett[A] has quit IRC | 22:39 | |
clarkb | bknudson: is there a draft patchset? | 22:39 |
zaro | clarkb: drafts ignore acls? | 22:39 |
clarkb | zaro: enough to make acls disabling drafts not work | 22:39 |
bknudson | clarkb: no, I never had to do a draft for that one. | 22:39 |
zaro | news to me. | 22:39 |
clarkb | bknudson: nevermind I can vote a value so I don't think so | 22:39 |
clarkb | zaro: uh you tested this and _david_ fixed it iirc | 22:40 |
bknudson | not a huge deal but just interesting. | 22:40 |
*** dstanek_zzz is now known as dstanek | 22:40 | |
openstackgerrit | Ben Nemec proposed a change to openstack-infra/config: Add dib-utils project https://review.openstack.org/90281 | 22:40 |
sdague | so I'd be curious how wrong I managed to get this - https://review.openstack.org/#/c/94260/ | 22:40 |
sdague | jeepyb is kind of dark magic to me, so I'm not convinced I got the invocations right | 22:41 |
clarkb | sdague: first thing that jumps out at me is you are using refs/meta/dashboards/site | 22:41 |
clarkb | sdague: any particular reason not to use the existing ref? | 22:41 |
notmyname | sdague: because all reviewers, as currently listed, are seen as equal. the problem that needs to be solved is giving new and non-core contributors a contact point for question and someone who is shepherding the patch through the process of submission->master | 22:42 |
*** hipster has joined #openstack-infra | 22:42 | |
*** james_li has joined #openstack-infra | 22:43 | |
*** hipster_ has joined #openstack-infra | 22:44 | |
notmyname | also, having social pressure on core reviewers to stay more active :-) | 22:44 |
jeblair | krotscheck: what's the process to run a local webclient and point it at the prod server? http://ci.openstack.org/storyboard/install/development.html doesn't seem to cover that case | 22:46 |
sdague | clarkb: dashboards end up being grouped by files under a branch. site seemed a reasonable branch to start with for the generic approach of this, as this will also let projects define dashboards. | 22:46 |
zaro | jeblair, clarkb : disabled drafts is fixed in 2.8 | 22:46 |
sdague | I'm happy to change it to something else | 22:47 |
*** hipster has quit IRC | 22:47 | |
clarkb | zaro: I thought it was 2.9? | 22:47 |
krotscheck | jeblair: tox -egrunt server:prod | 22:47 |
*** SumitNaiksatam has quit IRC | 22:47 | |
clarkb | sdague: reviewed | 22:47 |
*** yamahata has quit IRC | 22:47 | |
sdague | clarkb: thanks! | 22:47 |
*** hipster has joined #openstack-infra | 22:47 | |
zaro | jeblair, clarkb :https://gerrit-review.googlesource.com/#/c/54062/ | 22:47 |
clarkb | sdague: no the name is fine was just curious if there was a reason | 22:47 |
*** SumitNaiksatam has joined #openstack-infra | 22:47 | |
clarkb | oh nice it was backported | 22:48 |
zaro | clarkb: was pushed to 2.9, but got moved to 2.8 | 22:48 |
clarkb | zaro: so we should probably update All-Projects config to block drafts for registered users | 22:48 |
clarkb | I am guessing that will need testing and an announcement to the list so I won't go ahead and just do it | 22:48 |
clarkb | but I will update All-Projects on review-dev really quick | 22:49 |
*** hipster_ has quit IRC | 22:49 | |
zaro | might need to update gerrit though, not sure if this made it to version we have. | 22:49 |
* zaro checks now | 22:49 | |
*** aconrad has quit IRC | 22:49 | |
*** r-daneel has joined #openstack-infra | 22:50 | |
clarkb | ugh review-dev is using a different version that review.o.o | 22:50 |
nibalizer | :( | 22:51 |
*** whoops has quit IRC | 22:52 | |
*** hipster has quit IRC | 22:52 | |
zaro | clarkb: yeah, what's up with that? | 22:52 |
clarkb | don't look at me | 22:52 |
*** gokrokve has quit IRC | 22:52 | |
clarkb | I was conveniently out that week ;) | 22:52 |
clarkb | I am guessing the war for each is different in puppet | 22:53 |
zaro | noticed that review-dev is no longer reporting to puppetboard | 22:53 |
*** flaper87 is now known as flaper87|afk | 22:53 | |
zaro | clarkb: should be the smae | 22:53 |
openstackgerrit | Michael Krotscheck proposed a change to openstack-infra/storyboard: Updated documentation to include instructions for prod-proxy mode. https://review.openstack.org/94279 | 22:53 |
krotscheck | jeblair: ^ | 22:53 |
krotscheck | (Although I have _no_ idea why there’s a big scary red bar in the diff) | 22:54 |
pleia2 | clarkb: continuing on the reviewday thing, any idea why would setup.py + README.md be causing issues now? something in infra changed? | 22:54 |
clarkb | compare https://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/manifests/review.pp#n110 to https://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/manifests/review_dev.pp#n45 | 22:54 |
clarkb | zaro: ^ | 22:54 |
jeblair | krotscheck: mixed tab/spaces | 22:54 |
clarkb | in any case I added the rule to review-dev | 22:54 |
clarkb | about to test it | 22:54 |
clarkb | pleia2: possibly something in pip changed | 22:55 |
jeblair | krotscheck: we're generally all-spaces around here, so at some point we should s/^I/ / in the docs | 22:55 |
jeblair | krotscheck: grount_no_api or grunt? | 22:55 |
*** gokrokve_ has joined #openstack-infra | 22:55 | |
krotscheck | grunt_no_api | 22:55 |
pleia2 | clarkb: aha, I'll investigate | 22:55 |
*** ramashri has joined #openstack-infra | 22:56 | |
zaro | clarkb: argg! | 22:57 |
clarkb | ! [remote rejected] HEAD -> refs/drafts/master/5261 (cannot upload drafts) | 22:58 |
openstack | clarkb: Error: "remote" is not a valid command. | 22:58 |
*** melwitt has quit IRC | 22:59 | |
clarkb | zaro: jeblair ^ so I think that worked as expected | 22:59 |
*** igor_ has joined #openstack-infra | 22:59 | |
*** msabramo has quit IRC | 22:59 | |
clarkb | I added refs/drafts/*\npush= block Registered Users | 22:59 |
clarkb | to All-Projects' config | 22:59 |
jeblair | krotscheck: when i test 91196 locally, clicking on the project page just shows the 'welcome to storyboard' overview page | 22:59 |
*** gokrokve_ has quit IRC | 22:59 | |
clarkb | lifeless: ok pong | 23:00 |
krotscheck | jeblair: Sec, let me get this whitespace fix done and I’ll see what’s going on there. | 23:00 |
openstackgerrit | Michael Krotscheck proposed a change to openstack-infra/storyboard: Removed tabs, changed to four spaces. https://review.openstack.org/94280 | 23:00 |
*** Sukhdev has quit IRC | 23:02 | |
*** lcostantino has quit IRC | 23:02 | |
*** james_li has quit IRC | 23:02 | |
lifeless | clarkb: hi :) | 23:02 |
*** aconrad has joined #openstack-infra | 23:02 | |
clarkb | but now I get a databases error trying to push non draft changes | 23:03 |
clarkb | so it may have broken everything :) | 23:03 |
clarkb | hopefully this is just review-dev being cranky | 23:03 |
*** r-daneel has quit IRC | 23:03 | |
zaro | clarkb: ohh cool! | 23:03 |
*** igor_ has quit IRC | 23:03 | |
*** r-daneel has joined #openstack-infra | 23:04 | |
*** virmitio has quit IRC | 23:04 | |
*** kaisero has quit IRC | 23:04 | |
zaro | rcarrillocruz: we will be turning off drafts soon | 23:04 |
*** dstanek is now known as dstanek_zzz | 23:04 | |
*** sweston has quit IRC | 23:05 | |
clarkb | zaro: well assuming we can get review-dev to work | 23:05 |
openstackgerrit | Michael Krotscheck proposed a change to openstack-infra/storyboard-webclient: Some project ui/ux updates https://review.openstack.org/91196 | 23:05 |
*** dstanek_zzz is now known as dstanek | 23:05 | |
zaro | clarkb: you mean the puppet part? | 23:05 |
krotscheck | jeblair: Give that a shot. | 23:05 |
*** msabramo has joined #openstack-infra | 23:05 | |
clarkb | zaro: no I mean the I can't push anything to review-dev right now part | 23:06 |
*** r-daneel has quit IRC | 23:07 | |
zaro | clarkb: did you set "read = blck group Anonymous Users" and "push = block group Anonymous Users" ? | 23:07 |
clarkb | no just push = block group Registered Users | 23:08 |
zaro | clarkb: section should be [access "refs/drafts/*"] | 23:08 |
clarkb | We don't want to restrict Anonymous users as that is a noop | 23:08 |
zaro | clarkb: yep, makes sense | 23:08 |
clarkb | see https://review-dev.openstack.org/#/admin/projects/All-Projects,access | 23:08 |
jeblair | krotscheck: lgtm; one more thing -- is there a trick to logging in when using server:prod? | 23:09 |
krotscheck | jeblair: Yeah, when you come back to localhost you have to change the protocol from https to http. | 23:10 |
lifeless | clarkb: did you see my pbr query ~ 24 hr ago ? | 23:10 |
krotscheck | Manually. | 23:10 |
clarkb | lifeless: briefly. I didn't really parse it at the time | 23:10 |
lifeless | clarkb: ok; care to re-read? | 23:10 |
*** amcrn has joined #openstack-infra | 23:11 | |
clarkb | sure /me hunts through scrollback | 23:11 |
zaro | clarkb: looks correct to me. | 23:11 |
clarkb | lifeless: is it not possible to use one untagged version and only use that format? | 23:12 |
clarkb | lifeless: eg alpha vs dev? | 23:12 |
lifeless | that is the question ? | 23:12 |
clarkb | I think what confuses me is alpha vs dev is only ever specified by pbr not humans right? so why are we worried about it? | 23:13 |
clarkb | also semver.rst should diaf | 23:13 |
jeblair | krotscheck: thx | 23:13 |
lifeless | humans specify alpha/beta etc today | 23:13 |
*** ramashri has quit IRC | 23:14 | |
lifeless | untagged is always *dev* | 23:14 |
zaro | clarkb: what registered user account are you using to test draft with? | 23:14 |
clarkb | zaro: cboylan | 23:14 |
zaro | clarkb: aren't you an admin? | 23:14 |
clarkb | zaro: I am | 23:14 |
zaro | clarkb: so you shouldn't be blocked from publishing anything. | 23:15 |
clarkb | zaro: thats not true | 23:15 |
clarkb | zaro: Admins don't have super powers | 23:15 |
clarkb | they only have the ability to get super powers | 23:15 |
*** wenlock has quit IRC | 23:16 | |
openstackgerrit | A change was merged to openstack-infra/storyboard-webclient: Added task priority to edit and display of tasks https://review.openstack.org/91693 | 23:16 |
clarkb | lifeless: order is dev < a < b < rc | 23:16 |
clarkb | lifeless: according to semver.rst | 23:16 |
clarkb | lifeless: I think I am still failing to parse your question in a meaningful way | 23:16 |
lifeless | clarkb: yes, and a.dev isn't permitted according to it | 23:16 |
*** jamielennox|away is now known as jamielennox | 23:17 | |
lifeless | clarkb: but this means that a .dev version 5 commits after an alpha tag has a lower precedence than the alpha tag that came before it | 23:17 |
clarkb | zaro: and I want it to prevent me from pushing drafts which it seems to do | 23:17 |
clarkb | lifeless: is that a problem? | 23:18 |
lifeless | clarkb: I don't know pbr's use cases as well as mordred_ or I presume you | 23:19 |
lifeless | clarkb: so it certainly seemed like it could be a problem | 23:19 |
clarkb | lifeless: so the thing that enforces that isn't pbr iirc. It is pip | 23:19 |
zaro | clarkb: i just tried to push to a change to HEAD:refs/drafts gerrit didn't complain. but i also don't see it on review-dev either. | 23:19 |
*** sarob has joined #openstack-infra | 23:19 | |
clarkb | zaro: do you have super powers? | 23:19 |
jeblair | krotscheck: someone left a technical comment that i think only you can respond to on 92639 | 23:20 |
zaro | clarkb: ohh wait, if i do HEAD:refs/drafts/master then it says i can't do that. | 23:20 |
krotscheck | jeblair: Yeah, that’s MaxV. He and I had a really good time discussing various merits of JS tooling vis-a-vis horizon at the summit. | 23:20 |
clarkb | zaro: oh right because the ruele is refs/drafts/* | 23:20 |
clarkb | zaro: are you able to push a normal change/patchset? | 23:21 |
clarkb | zaro: because that is currently very broken for me | 23:21 |
* zaro trying now | 23:21 | |
clarkb | lifeless: I think this would be good to talk about with mordred to see if he actually intended on this gap | 23:21 |
zaro | clarkb: just pushed this one, https://review-dev.openstack.org/#/c/5280/ | 23:22 |
clarkb | lifeless: I don't think it is a huge issue but I am sure someone can get into a painful corner case if they try | 23:22 |
zaro | clarkb: are you using git review? | 23:22 |
jogo_ | salv-orlando: sure, put up a patch to remove 1224001 | 23:22 |
clarkb | zaro: yes | 23:22 |
zaro | clarkb: i'm not | 23:22 |
clarkb | zaro: why not? | 23:22 |
jeblair | krotscheck: ok, i wanted to make sure you didn't miss it, and especially didn't want you to accidentally ignore a potential new js reviewer. would you like me to hold off on approving that change or go ahead and push it through? | 23:22 |
clarkb | I can try it without too, but really git review is the way we tell people to use the thing | 23:22 |
*** jogo_ is now known as jogo | 23:23 | |
zaro | clarkb: don't know, but it seems to work with git push origin HEAD:refs/for/master | 23:23 |
*** david-lyle has quit IRC | 23:23 | |
krotscheck | jeblair: Well, I’m composing a response right now which’ll give more context. | 23:23 |
jeblair | krotscheck: cool, i'll leave it for now then | 23:23 |
krotscheck | kk | 23:23 |
clarkb | zaro: both are working for me now | 23:24 |
clarkb | zaro: so that may have just been the DB going away for a bit | 23:24 |
jeblair | krotscheck: (+2 crvw but not aprvd; feel free to aprv at will) | 23:24 |
krotscheck | jeblair: tx | 23:24 |
zaro | clarkb: yep, me too | 23:25 |
zaro | clarkb: tested with git-review as well. | 23:25 |
clarkb | zaro: cool, so I think that means this works as expected | 23:25 |
clarkb | jeblair: zaro: do we want to make this change soonish? I can draft an announcement tomorrow | 23:26 |
*** ptippett[A] has joined #openstack-infra | 23:26 | |
zaro | clarkb: ok with me. | 23:26 |
*** matsuhashi has joined #openstack-infra | 23:27 | |
jeblair | clarkb: ++ | 23:28 |
*** signed8bit has quit IRC | 23:30 | |
openstackgerrit | A change was merged to openstack-infra/storyboard-webclient: Switched new story modal to use typeahead project completion https://review.openstack.org/92487 | 23:31 |
pleia2 | clarkb: ok, I think I figured it out, it's unpacking /home/jenkins/workspace/gate-reviewday-pep8/.tox/dist/reviewday-0.2.0.zip into the /tmp/pip-blahblah directory but that doesn't include README.md that it's looking for, what is reviewday-0.2.0.zip anyway? | 23:31 |
pleia2 | the repo itself has README.md | 23:31 |
clarkb | I think that is the sdist that it is building | 23:32 |
clarkb | you may need to add README.md to the setup.py as a data file | 23:32 |
pleia2 | alright, I'll give that a shot | 23:32 |
*** weshay has quit IRC | 23:36 | |
praneshp_ | jeblair: clarkb any idea why https://review.openstack.org/#/c/94286/ doesn’t merge? | 23:36 |
praneshp_ | it says cannot be automatically merged, I’m unsure what to change there | 23:36 |
clarkb | praneshp_: you cannot change your refs meta config that way | 23:37 |
praneshp_ | clarkb: ok, I just want to delete a tag, how should I go about doing it | 23:37 |
praneshp_ | i gave a really stupid name to my project’s furst tag, and want to nuke it. | 23:37 |
clarkb | praneshp_: we like to treat tags as permanent | 23:37 |
clarkb | because they cannot be removed from downstream repos | 23:38 |
clarkb | praneshp_: fwiw all config changes should go through the openstack-infra/config repo | 23:38 |
praneshp_ | clarkb: hmm. I don’t have any downstream repos, not sure if anyone is using it formally yet | 23:38 |
praneshp_ | oh, I just folowd the gerrit UI, didn’t even make this commit directly | 23:38 |
clarkb | praneshp_: you have no way of knowing hence the rule | 23:38 |
praneshp_ | Hmm. any suggestions on how to help people not be confused by the bad tag? | 23:39 |
clarkb | zaro: huh is anyone allowed to hit the edit button on ACLs in gerrit? | 23:40 |
clarkb | zaro: do you know if we can disable that too? | 23:41 |
lifeless | clarkb: righto, it just leaves the patch set in limbo :) | 23:42 |
lifeless | clarkb: when you see him point him at me :) | 23:42 |
clarkb | lifeless: when I see him? I am pretty sure he is off to do all the thing sthat don't involve IRC for the next month | 23:42 |
lifeless | he will pop in from time to time I'm sure | 23:43 |
clarkb | jeblair: deleting a tag is just git push gerrit :refs/tags/5.19 right? | 23:44 |
clarkb | I really don't like deleting tags though | 23:44 |
clarkb | yay git for being broken in this way | 23:44 |
*** lcheng has quit IRC | 23:47 | |
*** hipster has joined #openstack-infra | 23:47 | |
morganfainberg | clarkb, https://review.openstack.org/#/c/89804/ seems to be a broken patchset, maybe somehow half draft half not? it has no changeset information returned from the gerrit query. http://pasteraw.com/kcrinap13oy7j6ond47ney7vz22ccbo which is breaking reviewday. I have a fix for reviewday proposed, but this looks... "odd" | 23:48 |
clarkb | morganfainberg: definitely was draft at one point. notice how patchset 6 is the first one | 23:49 |
clarkb | er patchset 5 | 23:49 |
*** msabramo has quit IRC | 23:49 | |
zaro | clarkb: apparently so. not sure how to disabled that in acls settting | 23:49 |
morganfainberg | clarkb, yeah it's super odd. | 23:49 |
*** hemna is now known as hemna_ | 23:49 | |
morganfainberg | clarkb, and the webui returns very different info than the ssh gerrit query | 23:49 |
morganfainberg | i think someone proposed a new draft to it | 23:49 |
*** paul--_ has joined #openstack-infra | 23:49 | |
* morganfainberg thinks drafts are awful in gerrit | 23:50 | |
clarkb | morganfainberg: that may be but I think that prevents you from voting with a number when you review | 23:50 |
clarkb | which I seem to be able to do | 23:50 |
pleia2 | morganfainberg: I think there is consensus on that here | 23:50 |
morganfainberg | clarkb, yeah same, can vote w/ a number | 23:50 |
zaro | clarkb: i think that edit button is a bug. | 23:51 |
*** SumitNaiksatam has quit IRC | 23:51 | |
clarkb | zaro: it is doing it because it proposes it for review if you can't merge the changes | 23:52 |
*** hipster has quit IRC | 23:52 | |
morganfainberg | clarkb, don't think this breaks anything except reviewday... so possibly a minor bug in gerrit's --latest-changeset query argument | 23:52 |
*** gokrokve has joined #openstack-infra | 23:53 | |
clarkb | morganfainberg: yeah, I wonder if it is failing to get latest-changeset on any draft | 23:53 |
morganfainberg | clarkb, i'll keep my eyes open for any other odd breakage, but it's probably nothing to worry about | 23:53 |
*** msabramo has joined #openstack-infra | 23:53 | |
jeblair | clarkb: yes | 23:53 |
clarkb | jeblair: thoughts on deleting a tag in this particular case | 23:54 |
clarkb | jeblair: I know in general we don't do it. And we only move tags when absolutely necessary | 23:54 |
*** dstanek is now known as dstanek_zzz | 23:54 | |
praneshp_ | clarkb: thanks, I thought you forgot about it :) | 23:55 |
* ptippett[A] is now away - Reason : Auto-Away after 30 minutes | 23:56 | |
clarkb | ptippett[A]: please disable your away notice | 23:56 |
jeblair | that's the second notice | 23:57 |
morganfainberg | praneshp_, clarkb doesn't forget things like that :) | 23:57 |
praneshp_ | morganfainberg: lol, correct :) | 23:58 |
morganfainberg | praneshp_, he was just waiting for a an answer to his question :) | 23:58 |
praneshp_ | i didnt have a chance to put a face on clarkb in the summit, saw morganfainberg | 23:58 |
morganfainberg | praneshp_, i had the same problem crossing paths w/ clarkb in hong kong -- come to paris, i'm sure you'll meet clarkb then! | 23:59 |
jeblair | clarkb: i don't object as long as praneshp_ pinky swears he won't do it again ;) | 23:59 |
clarkb | cool | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!