| jeblair | clarkb, mordred: ^ | 00:00 |
|---|---|---|
| *** Sukhdev has joined #openstack-infra | 00:00 | |
| *** thorst has quit IRC | 00:00 | |
| clarkb | jeblair: maybe update the commit title too ? | 00:00 |
| jeblair | clarkb: picky! ;) | 00:00 |
| anteaya | yeah for me there is a difference between a job in the experimental queue (experimental) and a non-voting job in check (non-voting) | 00:01 |
| jeblair | hrm, gerrit weirdness... | 00:02 |
| clarkb | anteaya: the big difference here is it runs on every commit so people will know the job exists | 00:02 |
| anteaya | sure a non-voting job | 00:02 |
| clarkb | anteaya: one big problem with experimental is that it isn't really visible so jobs ar eleft unrun | 00:02 |
| jeblair | yes it was a type | 00:02 |
| jeblair | a typo | 00:02 |
| openstackgerrit | James E. Blair proposed openstack-infra/project-config: Make nodepool dsvm non-voting. https://review.openstack.org/226090 | 00:02 |
| anteaya | the commit message says experimental | 00:02 |
| clarkb | anteaya: oh right I asked for it to be updated and there it is | 00:02 |
| anteaya | clarkb: no argument here | 00:02 |
| openstackgerrit | James E. Blair proposed openstack-infra/project-config: Make nodepool dsvm non-voting https://review.openstack.org/226090 | 00:02 |
| jeblair | yes, i'm trying to fix it | 00:02 |
| *** achanda has quit IRC | 00:03 | |
| anteaya | thanks | 00:03 |
| anteaya | :) | 00:03 |
| jeblair | gerrit is being very weird about it | 00:03 |
| jeblair | notice particularly patchset #3. | 00:03 |
| anteaya | how so? | 00:03 |
| clarkb | could be db backups | 00:03 |
| anteaya | it isn't acknowledged | 00:03 |
| jeblair | ya | 00:03 |
| jeblair | anyway, let's move on :) | 00:04 |
| *** achanda_ has quit IRC | 00:04 | |
| anteaya | code and commit match | 00:04 |
| anteaya | yay | 00:04 |
| jeblair | clarkb: this is needed for ovh: https://review.openstack.org/225344 | 00:06 |
| clarkb | looking | 00:06 |
| jeblair | clarkb: it failed dsvm, but erroneously afaict | 00:06 |
| *** hodos has quit IRC | 00:07 | |
| *** DericHorn-HP has joined #openstack-infra | 00:07 | |
| *** ashleighfarnham has joined #openstack-infra | 00:08 | |
| *** Sukhdev has quit IRC | 00:10 | |
| *** gyee has quit IRC | 00:10 | |
| clarkb | jeblair: the curious thing about that fail is according to the nodepool log the ubuntu-dib VM is ready when the list command for ubuntu-dib runs | 00:10 |
| *** glauco_ has joined #openstack-infra | 00:10 | |
| clarkb | though that may be a case of the logs having two different meanings for ready | 00:11 |
| *** glauco has quit IRC | 00:12 | |
| *** jswarren_ has joined #openstack-infra | 00:12 | |
| *** jswarren has quit IRC | 00:12 | |
| clarkb | ya looks like allocator ready means nova says its built | 00:12 |
| clarkb | which is different than nodepool proper reporting ready | 00:12 |
| *** gildub has quit IRC | 00:13 | |
| jeblair | clarkb: so was probably sshing/etc at timeout | 00:13 |
| *** ashleighfarnham has quit IRC | 00:13 | |
| *** gildub has joined #openstack-infra | 00:13 | |
| *** diazjf has joined #openstack-infra | 00:15 | |
| *** timfreund has quit IRC | 00:15 | |
| clarkb | ya looking at logs nova was still saying BUILD but allocator says 1 ready | 00:15 |
| *** mkj_wk has quit IRC | 00:15 | |
| *** Zara has quit IRC | 00:15 | |
| diazjf | ianw, thanks for all the help on the patch. I added another note | 00:15 |
| clarkb | so BUILD too must count as ready for calculating how many nodes to build. So we hadn't even gotten to ssh yet | 00:15 |
| *** gothicmindfood has quit IRC | 00:15 | |
| clarkb | which must mean that the dib build was slower than normal, usually dib build takes a minut eor two longer but booting off of dib image is much faster | 00:16 |
| clarkb | so ya if we can get the log stuff in the better wait change that should help | 00:16 |
| openstackgerrit | James E. Blair proposed openstack-infra/nodepool: Wait for resources in devstack-plugin check https://review.openstack.org/225262 | 00:16 |
| clarkb | since the logging is what is failing there and possibly it needs a rebase? there it s | 00:16 |
| diazjf | ianw, I'm thinking editting the pre_test_hook to install from git should work. I'm not sure if adding the function directly to https://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/jobs/python-barbicanclient.yaml | 00:17 |
| diazjf | would be a better approach | 00:17 |
| ianw | diazjf: no, not really. we don't clone stuff like that during CI jobs | 00:17 |
| jeblair | clarkb: working on that :) | 00:17 |
| *** Zara has joined #openstack-infra | 00:17 | |
| *** mkj_wk has joined #openstack-infra | 00:17 | |
| *** gothicmindfood has joined #openstack-infra | 00:17 | |
| clarkb | jeblair: both mv's in each function should sudo right? | 00:17 |
| diazjf | ianw, so I should just add another function similar to the cinder one? | 00:17 |
| jeblair | clarkb: yeah, sorry | 00:17 |
| *** ddieterly has joined #openstack-infra | 00:18 | |
| openstackgerrit | James E. Blair proposed openstack-infra/nodepool: Wait for resources in devstack-plugin check https://review.openstack.org/225262 | 00:18 |
| ianw | diazjf: let's just get the context right | 00:19 |
| *** yuanying has quit IRC | 00:19 | |
| openstackgerrit | James E. Blair proposed openstack-infra/project-config: Set nodepool dsvm timeout to 1h https://review.openstack.org/226105 | 00:19 |
| jeblair | clarkb: also ^ | 00:19 |
| ianw | so https://review.openstack.org/#/c/208343/ is what's failing | 00:19 |
| *** sflanigan has joined #openstack-infra | 00:20 | |
| diazjf | ianw, yeah pretty much any job running gate-tempest-dsvm-neutron-src-python-barbicanclient | 00:20 |
| *** cbader has joined #openstack-infra | 00:21 | |
| ianw | diazjf: ah, right, ok, this is from lib-forward-testing | 00:21 |
| openstackgerrit | Merged openstack-infra/project-config: Make nodepool dsvm non-voting https://review.openstack.org/226090 | 00:21 |
| greghaynes | jeblair: clarkb for timeouts we can use http://git.openstack.org/cgit/openstack/tripleo-incubator/tree/scripts/wait_for | 00:22 |
| *** DericHorn-HP has quit IRC | 00:23 | |
| diazjf | ianw, I'm new to this gate so I'm not sure exactly whats going on I just know it doesn't find a git repo just a version when it looks in pip freeze | 00:23 |
| jeblair | greghaynes: i don't think the problem with the timeouts was mechanism, i think it was that they were there at all | 00:23 |
| jeblair | greghaynes: and therefore, we hit them | 00:23 |
| *** timfreund has joined #openstack-infra | 00:23 | |
| *** kzaitsev_ws has quit IRC | 00:23 | |
| *** yuanying_ has joined #openstack-infra | 00:24 | |
| clarkb | right the old timeout was simple and should be improved | 00:24 |
| clarkb | the chnage above should do that | 00:24 |
| *** salv-orlando has quit IRC | 00:24 | |
| clarkb | I +2'd | 00:24 |
| greghaynes | ah, removing timeouts and polling works for me | 00:25 |
| *** kzaitsev_ws has joined #openstack-infra | 00:25 | |
| *** angdraug has quit IRC | 00:26 | |
| ianw | diazjf: right, you need to add it to devstack, in lib/cinder | 00:26 |
| openstackgerrit | Darragh Bailey proposed openstack-infra/jenkins-job-builder: Capture logs with FakeLogging Fixture https://review.openstack.org/226107 | 00:26 |
| *** stevemar has joined #openstack-infra | 00:27 | |
| ianw | diazjf: by default, we test libraries as installed by pip, following the standard requirements | 00:27 |
| *** yuanying_ is now known as yuanying | 00:29 | |
| diazjf | ianw, I am just confused as to why I needs to be added to cinder? | 00:32 |
| diazjf | ianw, cinder libs** | 00:33 |
| *** nelsnelson has quit IRC | 00:33 | |
| dims_ | mordred: don't we need a python-openstackclient release for the "Add image create support for image v2"? | 00:33 |
| *** nelsnelson has joined #openstack-infra | 00:34 | |
| *** pvaneck has quit IRC | 00:34 | |
| dims_ | mordred: i was looking at https://review.openstack.org/#/c/225363/ and ended up asking ^^ | 00:35 |
| openstackgerrit | Victor Ryzhenkin proposed openstack-infra/project-config: Add mistral back to murano job definitions file https://review.openstack.org/226108 | 00:36 |
| ianw | diazjf: hmm, good point. it's just that cinder is bringing in python-barbicanclient | 00:36 |
| openstackgerrit | James E. Blair proposed openstack-infra/nodepool: devstack: supply a logging config file https://review.openstack.org/226109 | 00:36 |
| jeblair | greghaynes, clarkb: ^ further debugability of the plugin output | 00:37 |
| ianw | diazjf: however, it's not going to do anything, right? barbican isn't running | 00:37 |
| jeblair | greghaynes, clarkb: (might be nice to avoid screen and switch to files and split out the image logs too; that can build on this) | 00:38 |
| diazjf | ianw, I'm not sure what that test does since none of my patches involve anything with it. Barbican should be running, but not cinder | 00:38 |
| clarkb | jeblair: any reason to not make that sort of config a default? | 00:38 |
| clarkb | jeblair: I think we can load in a docstring of the logfile config into the logging setup function during daemon startup | 00:38 |
| diazjf | ianw, Adding it to cinder libs would be a quick fix since its the only service that currently uses it, but in the future there should be a barbican service | 00:40 |
| diazjf | ianw, a barbican lib | 00:40 |
| jeblair | clarkb: it could not easily be a default under current circumstances. | 00:40 |
| jeblair | clarkb: the images need a different logging mechanism, and actual paths are deployment specific. | 00:40 |
| stevemar | crinkle: btw - i finally figured out why the image API was defaulting to 2 | 00:41 |
| stevemar | crinkle: https://github.com/openstack/os-client-config/blob/265abb2bf6e050d9c7bf9b3e0a307dca32899c1f/os_client_config/defaults.yaml#L11 | 00:41 |
| *** dims_ has quit IRC | 00:42 | |
| crinkle | stevemar: aha it was mordred's fault | 00:43 |
| crinkle | EmilienM: ^ | 00:43 |
| *** ybathia has quit IRC | 00:45 | |
| jeblair | clarkb, greghaynes: i think we need a "delete building images on start" feature similar to our "delete building instances" one... | 00:46 |
| clarkb | jeblair: oh ya probably | 00:46 |
| jeblair | (i just wasted 1.5 hours _not_ uploading an ovh image because of that) | 00:46 |
| diazjf | ianw, can cinder be removed from that job? any suggestions | 00:47 |
| *** DericHorn-HP has joined #openstack-infra | 00:47 | |
| clarkb | jeblair: there was a building image in the db with no upload/build thread behind it? | 00:47 |
| ianw | diazjf: ok, i think this is a bit of a grey area. you could remove the lib-forward-testing template, and not run this test. what are we going to find out from running a git version of python-barbicanclient in an environment where barbican itself isn't running? | 00:47 |
| jeblair | clarkb: after i restarted | 00:48 |
| *** MarkAtwood has joined #openstack-infra | 00:48 | |
| jeblair | clarkb: (there was one running when i stopped, because ovh was in a loop because of the glanceclient/shade bug) | 00:49 |
| *** ashleighfarnham has joined #openstack-infra | 00:49 | |
| *** DericHorn-HP has quit IRC | 00:50 | |
| diazjf | ianw, barbicanclient will have no functionality without barbican. I'm guessing you can just set the endpoint to a different VM with Barbican for it to be able to do anything | 00:50 |
| *** ashleighfarnham has quit IRC | 00:54 | |
| *** geoffarnold has quit IRC | 00:54 | |
| *** ondergetekende has quit IRC | 00:56 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/project-config: Remove lib-forward-testing from python-barbicanclient https://review.openstack.org/226113 | 00:57 |
| *** ondergetekende has joined #openstack-infra | 00:57 | |
| diazjf | ianw, awesome^^ | 00:57 |
| openstackgerrit | James E. Blair proposed openstack-infra/nodepool: Delete building images on startup https://review.openstack.org/226114 | 00:58 |
| jeblair | clarkb, greghaynes: ^ | 00:58 |
| ianw | diazjf: ^ i think this is the way to go. just double-check with sdague / jhesketh | 00:58 |
| *** ok_delta has joined #openstack-infra | 00:58 | |
| diazjf | ianw, thanks, can you add them to the patch | 00:59 |
| ianw | diazjf: they're more likely to notice the irc mention :) | 01:00 |
| stevemar | aww diazjf is knee deep into infra | 01:02 |
| stevemar | they grow up so fast! | 01:02 |
| *** camunoz has quit IRC | 01:02 | |
| diazjf | ianw, thanks!! :-D sdague, jhesketh, can you guys take a look at https://review.openstack.org/#/c/226113/ stevemar, I remember when I was on my first documentation patch, feels like so long ago! | 01:02 |
| *** aeng has quit IRC | 01:03 | |
| *** ddieterly has quit IRC | 01:03 | |
| *** sdake_ has joined #openstack-infra | 01:04 | |
| *** sdake has quit IRC | 01:06 | |
| *** sdake_ has quit IRC | 01:07 | |
| *** sdake has joined #openstack-infra | 01:07 | |
| *** jyuso1 has joined #openstack-infra | 01:08 | |
| *** DericHorn-HP has joined #openstack-infra | 01:08 | |
| *** cbader has quit IRC | 01:09 | |
| *** MarkAtwood has quit IRC | 01:10 | |
| *** DericHorn-HP has quit IRC | 01:11 | |
| *** Gareth has left #openstack-infra | 01:11 | |
| *** tiswanso_ has quit IRC | 01:11 | |
| *** DericHorn-HP has joined #openstack-infra | 01:11 | |
| openstackgerrit | Merged openstack-infra/nodepool: doc: FLUSH PRIVILEGES; is unneeded https://review.openstack.org/219324 | 01:11 |
| *** shashank_hegde has quit IRC | 01:12 | |
| *** tiswanso has joined #openstack-infra | 01:12 | |
| *** weshay has quit IRC | 01:13 | |
| *** alexsyip has quit IRC | 01:15 | |
| *** lawrancejing has joined #openstack-infra | 01:16 | |
| openstackgerrit | Merged openstack-infra/system-config: Update groups maint script https://review.openstack.org/223629 | 01:17 |
| *** claudiub has quit IRC | 01:19 | |
| *** camunoz has joined #openstack-infra | 01:19 | |
| *** aeng has joined #openstack-infra | 01:20 | |
| *** abregman|afk has quit IRC | 01:23 | |
| *** angdraug has joined #openstack-infra | 01:25 | |
| *** dims has joined #openstack-infra | 01:26 | |
| *** angdraug has quit IRC | 01:27 | |
| openstackgerrit | Ramy Asselin proposed openstack-infra/puppet-openstackci: Create a single-node 3rd party ci system https://review.openstack.org/200330 | 01:27 |
| *** DericHorn-HP has quit IRC | 01:28 | |
| *** nadya has joined #openstack-infra | 01:29 | |
| fungi | freerunner: it looks like that failure is probably because you stopped adding openstack/mistral to the PROJECTS variable (which tells devstack-gate which repos to copy into the cache) but are then trying to enable a plugin from it. i suspect you need to add back the line: export PROJECTS="openstack/mistral $PROJECTS" | 01:30 |
| *** otter768 has joined #openstack-infra | 01:30 | |
| *** fitoduarte has quit IRC | 01:30 | |
| *** DericHorn-HP has joined #openstack-infra | 01:31 | |
| *** nadya has quit IRC | 01:33 | |
| *** otter768 has quit IRC | 01:35 | |
| *** bpokorny has quit IRC | 01:41 | |
| *** sridhar_ram has quit IRC | 01:41 | |
| *** tlian2 has quit IRC | 01:44 | |
| openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Add db api method to get time series of runs by key value metadata https://review.openstack.org/223860 | 01:45 |
| *** stevemar has quit IRC | 01:46 | |
| *** ddieterly has joined #openstack-infra | 01:48 | |
| *** dramalho has joined #openstack-infra | 01:48 | |
| *** thorst has joined #openstack-infra | 01:49 | |
| *** asettle has joined #openstack-infra | 01:50 | |
| *** mwagner_lap has quit IRC | 01:52 | |
| *** thorst has quit IRC | 01:52 | |
| *** dramalho has quit IRC | 01:53 | |
| *** mwagner_lap has joined #openstack-infra | 01:53 | |
| *** woodster_ has quit IRC | 01:59 | |
| *** diazjf has left #openstack-infra | 02:01 | |
| *** ddieterl_ has joined #openstack-infra | 02:03 | |
| *** ddieterly has quit IRC | 02:03 | |
| *** ivar-laz_ has joined #openstack-infra | 02:06 | |
| *** camunoz has quit IRC | 02:07 | |
| *** nelsnels_ has joined #openstack-infra | 02:07 | |
| *** aeng_ has joined #openstack-infra | 02:08 | |
| *** nelsnelson has quit IRC | 02:08 | |
| *** ivar-lazzaro has quit IRC | 02:10 | |
| *** ivar-laz_ has quit IRC | 02:11 | |
| *** aeng has quit IRC | 02:11 | |
| *** ok_delta has quit IRC | 02:11 | |
| *** camunoz has joined #openstack-infra | 02:12 | |
| *** amuller has quit IRC | 02:12 | |
| *** DericHorn-HP has quit IRC | 02:12 | |
| *** otter768 has joined #openstack-infra | 02:17 | |
| *** shashank_hegde has joined #openstack-infra | 02:20 | |
| *** otter768 has quit IRC | 02:22 | |
| *** otter768 has joined #openstack-infra | 02:23 | |
| openstackgerrit | Ian Wienand proposed openstack/diskimage-builder: [WIP] Global element defaults https://review.openstack.org/226137 | 02:24 |
| *** rossella_s has quit IRC | 02:25 | |
| *** rossella_s has joined #openstack-infra | 02:25 | |
| *** zz_dimtruck is now known as dimtruck | 02:27 | |
| *** salv-orlando has joined #openstack-infra | 02:28 | |
| *** mtreinish has quit IRC | 02:28 | |
| openstackgerrit | Merged openstack/requirements: Update python-zaqarclient for new release 0.2.0 https://review.openstack.org/225892 | 02:29 |
| *** larainema has joined #openstack-infra | 02:29 | |
| *** jerryz has quit IRC | 02:29 | |
| *** salv-orlando has quit IRC | 02:32 | |
| *** achanda has joined #openstack-infra | 02:33 | |
| *** dims has quit IRC | 02:33 | |
| *** tiswanso has quit IRC | 02:34 | |
| *** unicell has quit IRC | 02:36 | |
| *** sdake has quit IRC | 02:39 | |
| *** bgmccollum has joined #openstack-infra | 02:39 | |
| *** david-ly_ has joined #openstack-infra | 02:39 | |
| *** _ody_ has joined #openstack-infra | 02:40 | |
| *** unicell has joined #openstack-infra | 02:40 | |
| *** david-lyle has quit IRC | 02:40 | |
| *** _ody has quit IRC | 02:40 | |
| *** dhellmann has quit IRC | 02:40 | |
| *** dhellmann has joined #openstack-infra | 02:41 | |
| *** dansmith has quit IRC | 02:41 | |
| *** dansmith has joined #openstack-infra | 02:42 | |
| *** dansmith is now known as Guest24378 | 02:42 | |
| *** unicell has quit IRC | 02:44 | |
| *** Kiall has quit IRC | 02:45 | |
| *** Kiall has joined #openstack-infra | 02:45 | |
| *** DericHorn-HP has joined #openstack-infra | 02:46 | |
| greghaynes | ianw: SpamapS has a spec about making us use something a lot more sane than env vars which alsonaddresses defaults | 02:49 |
| greghaynes | ianw: it needs some work, but you should have a look | 02:50 |
| *** shashank_hegde has quit IRC | 02:50 | |
| *** DericHorn-HP has quit IRC | 02:54 | |
| *** stevemar has joined #openstack-infra | 02:56 | |
| *** DericHorn-HP has joined #openstack-infra | 02:58 | |
| *** mtreinish has joined #openstack-infra | 02:59 | |
| openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Add db api method to get time series of runs by key value metadata https://review.openstack.org/223860 | 02:59 |
| *** DericHorn-HP has quit IRC | 03:03 | |
| *** hdd has quit IRC | 03:05 | |
| ianw | greghaynes: i'm not finding it? where do i look? | 03:12 |
| *** david-ly_ is now known as david-lyle | 03:17 | |
| greghaynes | hrm, it was a patch, probably wip'd | 03:19 |
| *** harlowja_at_home has joined #openstack-infra | 03:20 | |
| *** cburroughs has joined #openstack-infra | 03:23 | |
| openstackgerrit | Douglas Mendizábal proposed openstack-infra/project-config: Add PyPI and doc publish jobs to Castellan https://review.openstack.org/226150 | 03:26 |
| *** nadya has joined #openstack-infra | 03:30 | |
| *** ayoung has quit IRC | 03:30 | |
| *** hichihara has joined #openstack-infra | 03:30 | |
| *** hichihara has quit IRC | 03:31 | |
| *** dims has joined #openstack-infra | 03:34 | |
| *** nadya has quit IRC | 03:35 | |
| *** cburroughs has quit IRC | 03:37 | |
| *** hdd has joined #openstack-infra | 03:38 | |
| *** bpokorny has joined #openstack-infra | 03:38 | |
| *** cburroughs has joined #openstack-infra | 03:40 | |
| *** yamahata has quit IRC | 03:41 | |
| *** ddieterl_ has quit IRC | 03:42 | |
| *** DericHorn-HP has joined #openstack-infra | 03:45 | |
| *** kushal has joined #openstack-infra | 03:48 | |
| *** cdoron has joined #openstack-infra | 03:48 | |
| cdoron | ianw: could I ask you to re-review https://review.openstack.org/#/c/220139/ ? | 03:54 |
| *** armax has quit IRC | 03:55 | |
| *** cburroughs has quit IRC | 03:58 | |
| *** fedexo has joined #openstack-infra | 03:59 | |
| *** dims has quit IRC | 04:09 | |
| *** dimtruck is now known as zz_dimtruck | 04:09 | |
| *** stevemar has quit IRC | 04:09 | |
| *** stevemar has joined #openstack-infra | 04:10 | |
| *** otter768 has quit IRC | 04:10 | |
| *** darrenc is now known as darrenc_afk | 04:14 | |
| *** lawrancejing has quit IRC | 04:16 | |
| *** mtreinish has quit IRC | 04:18 | |
| *** asselin_ has joined #openstack-infra | 04:20 | |
| *** mtreinish has joined #openstack-infra | 04:23 | |
| openstackgerrit | Colleen Murphy proposed openstack-infra/system-config: (WIP) Add compute class for Infra Cloud https://review.openstack.org/213980 | 04:23 |
| openstackgerrit | Colleen Murphy proposed openstack-infra/system-config: (WIP) Add controller class for Infra Cloud https://review.openstack.org/209698 | 04:23 |
| *** lawrancejing has joined #openstack-infra | 04:28 | |
| *** Sree has joined #openstack-infra | 04:29 | |
| *** nadya has joined #openstack-infra | 04:30 | |
| *** lawrancejing has quit IRC | 04:34 | |
| *** nadya has quit IRC | 04:35 | |
| *** Sree_ has joined #openstack-infra | 04:37 | |
| *** Sree_ is now known as Guest31757 | 04:38 | |
| *** Sree has quit IRC | 04:39 | |
| *** sergmelikyan has joined #openstack-infra | 04:40 | |
| *** harlowja_at_home has quit IRC | 04:40 | |
| *** hdd has quit IRC | 04:43 | |
| *** ddieterly has joined #openstack-infra | 04:43 | |
| *** sergmelikyan has quit IRC | 04:45 | |
| *** ddieterly has quit IRC | 04:47 | |
| *** rameshg87 has joined #openstack-infra | 04:51 | |
| rameshg87 | hello folks | 04:52 |
| *** mrmartin has joined #openstack-infra | 04:52 | |
| rameshg87 | is it possible to provide +2/+A permissions for a different group also for a subset of a repository | 04:52 |
| *** Sree has joined #openstack-infra | 04:53 | |
| rameshg87 | for example, currently ironic-core has +2/+A permissions on ironic repository | 04:54 |
| rameshg87 | is it possible to provide +2/+A permissions for ironic/docs/source/drivers/ilo for another group proliantutils-core | 04:54 |
| openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Add db api methods to get keys from metadata tables https://review.openstack.org/226162 | 04:55 |
| *** Sree has quit IRC | 04:55 | |
| *** Guest31757 has quit IRC | 04:55 | |
| *** Sree has joined #openstack-infra | 04:56 | |
| openstackgerrit | Steve Kowalik proposed openstack-infra/project-config: No longer read an existing zanata.xml https://review.openstack.org/226163 | 04:59 |
| *** bpokorny has quit IRC | 05:00 | |
| *** abitha has joined #openstack-infra | 05:01 | |
| openstackgerrit | Merged stackforge/python-jenkins: Adds Cloudbees folder plugin support https://review.openstack.org/180185 | 05:01 |
| *** shardy_ has joined #openstack-infra | 05:04 | |
| *** deepakcs has joined #openstack-infra | 05:05 | |
| *** shardy has quit IRC | 05:05 | |
| *** mwagner_lap has quit IRC | 05:08 | |
| *** fedexo has quit IRC | 05:08 | |
| *** darrenc_afk is now known as darrenc | 05:09 | |
| *** shardy_ has quit IRC | 05:09 | |
| *** shardy has joined #openstack-infra | 05:10 | |
| *** DericHorn-HP has quit IRC | 05:10 | |
| *** mwagner_lap has joined #openstack-infra | 05:11 | |
| *** changbl has quit IRC | 05:11 | |
| *** smcginnis has quit IRC | 05:12 | |
| *** DericHorn-HP has joined #openstack-infra | 05:13 | |
| *** changbl has joined #openstack-infra | 05:15 | |
| openstackgerrit | Steve Kowalik proposed openstack-infra/project-config: Revert "Run propose-translation-update-zanata jobs" https://review.openstack.org/226164 | 05:15 |
| *** ildikov has quit IRC | 05:18 | |
| *** DericHorn-HP has quit IRC | 05:18 | |
| *** DericHorn-HP has joined #openstack-infra | 05:25 | |
| *** salv-orlando has joined #openstack-infra | 05:26 | |
| *** yfried_ has quit IRC | 05:30 | |
| *** salv-orlando has quit IRC | 05:33 | |
| *** abitha has quit IRC | 05:34 | |
| *** DericHorn-HP has quit IRC | 05:41 | |
| *** abregman has joined #openstack-infra | 05:43 | |
| *** ddieterly has joined #openstack-infra | 05:44 | |
| *** austin81 has joined #openstack-infra | 05:47 | |
| *** ddieterly has quit IRC | 05:48 | |
| *** Sukhdev has joined #openstack-infra | 05:49 | |
| *** nadya has joined #openstack-infra | 05:51 | |
| *** kushal has quit IRC | 05:51 | |
| *** stevemar has quit IRC | 05:53 | |
| *** jerryz has joined #openstack-infra | 05:58 | |
| *** BharatK has joined #openstack-infra | 05:59 | |
| *** BharatK_ has joined #openstack-infra | 05:59 | |
| *** BharatK_ has quit IRC | 06:00 | |
| *** BharatK has quit IRC | 06:03 | |
| openstackgerrit | Steve Kowalik proposed openstack-infra/project-config: Collapse two cases in upstream_translation_update https://review.openstack.org/226169 | 06:08 |
| *** cdoron has quit IRC | 06:09 | |
| ianw | cdoron: if nobody gets to it, i'd like to discuss in tomorrows infra meeting | 06:10 |
| *** otter768 has joined #openstack-infra | 06:11 | |
| *** lawrancejing has joined #openstack-infra | 06:12 | |
| *** BharatK has joined #openstack-infra | 06:12 | |
| *** ParsectiX has joined #openstack-infra | 06:15 | |
| *** otter768 has quit IRC | 06:16 | |
| *** Sukhdev has quit IRC | 06:17 | |
| *** nadya has quit IRC | 06:17 | |
| *** asettle has quit IRC | 06:19 | |
| *** kushal has joined #openstack-infra | 06:23 | |
| *** nadya has joined #openstack-infra | 06:24 | |
| *** rossella_s has quit IRC | 06:25 | |
| *** rossella_s has joined #openstack-infra | 06:25 | |
| *** k4n0 has joined #openstack-infra | 06:26 | |
| *** sabari has left #openstack-infra | 06:29 | |
| *** mrmartin has quit IRC | 06:33 | |
| openstackgerrit | Brian Torres proposed openstack-infra/jenkins-job-builder: story: 2000357 Use an xml attribute tree injected into the job to dtermine if a job is managed. Old method relied on modifying the description with a magic string which followed a job on copy. The xml attribute does not follow the job on copy making for a https://review.openstack.org/225562 | 06:38 |
| openstackgerrit | Brian Torres proposed openstack-infra/jenkins-job-builder: story: 2000357 Use an xml attribute tree injected into the job to determine if a job is managed. Old method relied on modifying the description with a magic string which followed a job on copy. The xml attribute does not follow the job on copy making for https://review.openstack.org/225562 | 06:39 |
| openstackgerrit | Nikolay Mahotkin proposed openstack/requirements: Fixing code-blocks in README.rst https://review.openstack.org/225616 | 06:40 |
| *** dizquierdo has joined #openstack-infra | 06:43 | |
| *** matbu has quit IRC | 06:44 | |
| *** ddieterly has joined #openstack-infra | 06:44 | |
| *** matbu has joined #openstack-infra | 06:45 | |
| *** ddieterly has quit IRC | 06:49 | |
| *** asettle has joined #openstack-infra | 06:50 | |
| *** cdoron has joined #openstack-infra | 06:51 | |
| *** smcginnis has joined #openstack-infra | 06:53 | |
| *** Ala has joined #openstack-infra | 06:56 | |
| *** Sree has quit IRC | 06:56 | |
| *** mwagner_lap has quit IRC | 06:56 | |
| *** Sree has joined #openstack-infra | 06:57 | |
| *** mwagner_lap has joined #openstack-infra | 06:57 | |
| *** david-lyle has quit IRC | 06:58 | |
| *** jlanoux has joined #openstack-infra | 07:00 | |
| *** sergmelikyan has joined #openstack-infra | 07:00 | |
| openstackgerrit | Brian Torres proposed openstack-infra/jenkins-job-builder: Use xml instead of job description for is_managed https://review.openstack.org/225562 | 07:01 |
| *** peristeri has joined #openstack-infra | 07:03 | |
| *** salv-orlando has joined #openstack-infra | 07:03 | |
| *** ParsectiX has quit IRC | 07:03 | |
| *** sergmelikyan has quit IRC | 07:05 | |
| *** ildikov has joined #openstack-infra | 07:05 | |
| *** kiran-r has joined #openstack-infra | 07:07 | |
| *** Hal has joined #openstack-infra | 07:07 | |
| *** matrohon has joined #openstack-infra | 07:12 | |
| *** david-lyle has joined #openstack-infra | 07:14 | |
| *** scheuran has joined #openstack-infra | 07:15 | |
| *** jamielennox is now known as jamielennox|away | 07:15 | |
| *** dtantsur|afk is now known as dtantsur | 07:16 | |
| *** achanda has quit IRC | 07:17 | |
| *** kiran-r has quit IRC | 07:18 | |
| openstackgerrit | Brian Torres proposed openstack-infra/jenkins-job-builder: Use xml instead of job description for is_managed https://review.openstack.org/225562 | 07:21 |
| *** kiran-r has joined #openstack-infra | 07:23 | |
| *** arxcruz has joined #openstack-infra | 07:23 | |
| *** ifarkas has joined #openstack-infra | 07:24 | |
| *** abregman has quit IRC | 07:25 | |
| *** abregman has joined #openstack-infra | 07:25 | |
| *** nadya has quit IRC | 07:27 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements: Updated from generate-constraints https://review.openstack.org/226182 | 07:28 |
| *** cdoron has quit IRC | 07:36 | |
| *** nadya has joined #openstack-infra | 07:37 | |
| *** yolanda has joined #openstack-infra | 07:40 | |
| *** sweston has quit IRC | 07:41 | |
| *** ams_ has quit IRC | 07:41 | |
| *** edwarnicke has quit IRC | 07:41 | |
| *** teran has quit IRC | 07:41 | |
| *** patrickeast has quit IRC | 07:41 | |
| *** warpc has quit IRC | 07:41 | |
| *** fdegir has quit IRC | 07:41 | |
| *** othermordred has quit IRC | 07:41 | |
| *** j^2 has quit IRC | 07:41 | |
| *** DuncanT has quit IRC | 07:41 | |
| *** nzeer has quit IRC | 07:41 | |
| *** austin81 has quit IRC | 07:41 | |
| *** dougwig has quit IRC | 07:41 | |
| *** erlon has quit IRC | 07:41 | |
| *** mgkwill has quit IRC | 07:41 | |
| *** electrical has quit IRC | 07:41 | |
| *** xgerman has quit IRC | 07:41 | |
| *** zxiiro has quit IRC | 07:41 | |
| *** simonmcc has quit IRC | 07:41 | |
| *** GheRivero has quit IRC | 07:41 | |
| *** ramishra has quit IRC | 07:41 | |
| *** skath has quit IRC | 07:41 | |
| *** serverascode has quit IRC | 07:41 | |
| *** zhenguo has quit IRC | 07:41 | |
| *** marcusvrn_ has quit IRC | 07:41 | |
| *** Ng has quit IRC | 07:41 | |
| *** Krinkle has quit IRC | 07:41 | |
| *** boris-42 has quit IRC | 07:41 | |
| *** ctracey has quit IRC | 07:41 | |
| *** tchaypo has quit IRC | 07:41 | |
| *** thingee has quit IRC | 07:41 | |
| *** zhiyan has quit IRC | 07:41 | |
| *** rackertom has quit IRC | 07:41 | |
| *** ameade has quit IRC | 07:41 | |
| *** Piet has quit IRC | 07:41 | |
| *** briancurtin has quit IRC | 07:41 | |
| *** jraim has quit IRC | 07:41 | |
| *** jordanP has joined #openstack-infra | 07:42 | |
| *** notnownikki has joined #openstack-infra | 07:43 | |
| *** briancurtin has joined #openstack-infra | 07:43 | |
| *** GheRivero has joined #openstack-infra | 07:43 | |
| *** rackertom has joined #openstack-infra | 07:43 | |
| *** zhenguo has joined #openstack-infra | 07:45 | |
| *** mrmartin has joined #openstack-infra | 07:45 | |
| *** ddieterly has joined #openstack-infra | 07:45 | |
| *** ramishra has joined #openstack-infra | 07:49 | |
| *** ddieterly has quit IRC | 07:50 | |
| *** larainema has quit IRC | 07:50 | |
| *** yolanda has quit IRC | 07:52 | |
| ianw | rameshg87: i feel like we've had this discussion before, and i think the result was mostly about just trusting people | 07:53 |
| *** rlandy has joined #openstack-infra | 07:56 | |
| *** markus_z has joined #openstack-infra | 07:56 | |
| *** nihilifer has quit IRC | 07:57 | |
| *** samuelBartel has joined #openstack-infra | 07:58 | |
| *** Ng has joined #openstack-infra | 08:02 | |
| *** arxcruz has quit IRC | 08:04 | |
| *** arxcruz has joined #openstack-infra | 08:04 | |
| *** gildub has quit IRC | 08:04 | |
| *** shardy_ has joined #openstack-infra | 08:04 | |
| *** HeOS has quit IRC | 08:04 | |
| openstackgerrit | Merged openstack/requirements: Updated from generate-constraints https://review.openstack.org/221157 | 08:06 |
| *** shardy has quit IRC | 08:06 | |
| *** derekh has joined #openstack-infra | 08:08 | |
| *** ok_delta has joined #openstack-infra | 08:08 | |
| *** shardy_ has quit IRC | 08:10 | |
| *** shardy has joined #openstack-infra | 08:10 | |
| *** asettle has quit IRC | 08:11 | |
| *** fhubik has joined #openstack-infra | 08:12 | |
| *** ok_delta has quit IRC | 08:12 | |
| *** otter768 has joined #openstack-infra | 08:12 | |
| *** fhubik has quit IRC | 08:15 | |
| *** fhubik has joined #openstack-infra | 08:15 | |
| rameshg87 | ianw: hi | 08:15 |
| rameshg87 | ianw: okay, so there is no way to actuall do it in acl in infra ? | 08:16 |
| *** otter768 has quit IRC | 08:17 | |
| *** DuncanT has joined #openstack-infra | 08:19 | |
| *** hashar has joined #openstack-infra | 08:22 | |
| *** ams_ has joined #openstack-infra | 08:25 | |
| *** cdoron has joined #openstack-infra | 08:29 | |
| *** fhubik has quit IRC | 08:29 | |
| *** fhubik has joined #openstack-infra | 08:29 | |
| openstackgerrit | Victor Stinner proposed openstack-infra/project-config: Make gate-barbican-python34 voting, add gate job https://review.openstack.org/226202 | 08:31 |
| *** rossella_s has quit IRC | 08:32 | |
| *** alexpilotti has joined #openstack-infra | 08:33 | |
| *** cdoron has quit IRC | 08:35 | |
| *** sergmelikyan has joined #openstack-infra | 08:36 | |
| *** sergmelikyan has quit IRC | 08:36 | |
| dtantsur | ianw, hi! could you please approve https://review.openstack.org/#/c/224476 ? | 08:38 |
| *** sergmelikyan has joined #openstack-infra | 08:40 | |
| *** sergmelikyan has quit IRC | 08:41 | |
| openstackgerrit | Derek Higgins proposed openstack-infra/tripleo-ci: Restore setting of origin/master https://review.openstack.org/226205 | 08:42 |
| *** sergmelikyan has joined #openstack-infra | 08:44 | |
| *** rossella_s has joined #openstack-infra | 08:44 | |
| *** sergmelikyan has quit IRC | 08:45 | |
| *** ddieterly has joined #openstack-infra | 08:46 | |
| *** alexpilotti has quit IRC | 08:47 | |
| *** ddieterly has quit IRC | 08:50 | |
| openstackgerrit | Merged openstack/diskimage-builder: Add option to set EPEL mirror https://review.openstack.org/225921 | 08:51 |
| *** HeOS has joined #openstack-infra | 08:52 | |
| *** sergmelikyan has joined #openstack-infra | 08:55 | |
| *** rguillebert has joined #openstack-infra | 08:55 | |
| *** sergmelikyan has quit IRC | 08:56 | |
| *** niska has quit IRC | 08:56 | |
| Zara | jeblair: thanks! | 08:58 |
| openstackgerrit | Thierry Carrez proposed openstack/requirements: Updated from generate-constraints https://review.openstack.org/226182 | 08:58 |
| openstackgerrit | Merged openstack-infra/tripleo-ci: Use a EPEL mirror https://review.openstack.org/225937 | 08:58 |
| *** dtantsur is now known as dtantsur|brb | 08:59 | |
| *** lawrance_ has joined #openstack-infra | 08:59 | |
| *** hichihara has joined #openstack-infra | 09:01 | |
| *** lawrancejing has quit IRC | 09:01 | |
| *** niska has joined #openstack-infra | 09:01 | |
| openstackgerrit | Merged openstack-infra/python-storyboardclient: Fix oslo imports https://review.openstack.org/226064 | 09:01 |
| openstackgerrit | Marton Kiss proposed openstack-infra/puppet-askbot: Allow custom settings.py template https://review.openstack.org/226213 | 09:02 |
| *** BharatK has quit IRC | 09:05 | |
| *** hichihara has quit IRC | 09:06 | |
| *** pelix has joined #openstack-infra | 09:06 | |
| *** hashar has quit IRC | 09:11 | |
| *** sergmelikyan has joined #openstack-infra | 09:12 | |
| *** sergmelikyan has quit IRC | 09:13 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/diskimage-builder: Updated from global requirements https://review.openstack.org/224606 | 09:17 |
| *** cdent has joined #openstack-infra | 09:19 | |
| *** BharatK has joined #openstack-infra | 09:19 | |
| *** ociuhandu has quit IRC | 09:21 | |
| *** sergmelikyan has joined #openstack-infra | 09:23 | |
| *** yfried_ has joined #openstack-infra | 09:24 | |
| *** sergmelikyan has quit IRC | 09:24 | |
| *** boris-42 has joined #openstack-infra | 09:26 | |
| *** e0ne has joined #openstack-infra | 09:26 | |
| *** jerryz has quit IRC | 09:39 | |
| openstackgerrit | Valeriy Ponomaryov proposed openstack/requirements: Add manila-ui project to project list https://review.openstack.org/226223 | 09:43 |
| *** sergmelikyan has joined #openstack-infra | 09:43 | |
| *** ociuhandu has joined #openstack-infra | 09:43 | |
| *** sergmelikyan has quit IRC | 09:44 | |
| *** ociuhandu_ has joined #openstack-infra | 09:45 | |
| *** othermordred has joined #openstack-infra | 09:45 | |
| *** tchaypo has joined #openstack-infra | 09:46 | |
| *** ddieterly has joined #openstack-infra | 09:47 | |
| *** dims has joined #openstack-infra | 09:47 | |
| *** ociuhandu has quit IRC | 09:47 | |
| *** ociuhandu_ is now known as ociuhandu | 09:47 | |
| *** BharatK has quit IRC | 09:48 | |
| *** yolanda has joined #openstack-infra | 09:48 | |
| *** marcusvrn_ has joined #openstack-infra | 09:49 | |
| *** erlon has joined #openstack-infra | 09:49 | |
| *** sergmelikyan has joined #openstack-infra | 09:51 | |
| *** ddieterly has quit IRC | 09:51 | |
| *** xgerman has joined #openstack-infra | 09:52 | |
| *** sergmelikyan has quit IRC | 09:52 | |
| *** warpc has joined #openstack-infra | 09:52 | |
| *** zxiiro has joined #openstack-infra | 09:52 | |
| *** ameade has joined #openstack-infra | 09:53 | |
| *** edwarnicke has joined #openstack-infra | 09:53 | |
| *** teran has joined #openstack-infra | 09:53 | |
| *** electrical has joined #openstack-infra | 09:53 | |
| *** fhubik is now known as fhubik_brb | 09:53 | |
| *** austin81 has joined #openstack-infra | 09:53 | |
| *** skath has joined #openstack-infra | 09:53 | |
| *** sweston has joined #openstack-infra | 09:53 | |
| *** dougwig has joined #openstack-infra | 09:54 | |
| *** jraim has joined #openstack-infra | 09:54 | |
| *** serverascode has joined #openstack-infra | 09:54 | |
| *** simonmcc has joined #openstack-infra | 09:54 | |
| *** bharathm has quit IRC | 09:54 | |
| *** fdegir has joined #openstack-infra | 09:54 | |
| *** zhiyan has joined #openstack-infra | 09:54 | |
| *** ctracey has joined #openstack-infra | 09:54 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/requirements: Updated from generate-constraints https://review.openstack.org/226182 | 09:54 |
| *** nadya has quit IRC | 09:55 | |
| *** thingee has joined #openstack-infra | 09:55 | |
| *** j^2 has joined #openstack-infra | 09:55 | |
| *** nzeer has joined #openstack-infra | 09:55 | |
| *** abregman is now known as abregman|afk | 09:56 | |
| *** yolanda has quit IRC | 09:56 | |
| *** salv-orlando has quit IRC | 09:56 | |
| *** markvoelker has quit IRC | 09:57 | |
| *** ParsectiX has joined #openstack-infra | 09:59 | |
| *** sdague has joined #openstack-infra | 10:00 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/requirements: Update python-novaclient for new release 2.30.0 https://review.openstack.org/225993 | 10:00 |
| *** BharatK has joined #openstack-infra | 10:02 | |
| *** fhubik_brb is now known as fhubik | 10:02 | |
| *** ParsectiX has quit IRC | 10:03 | |
| *** Krinkle has joined #openstack-infra | 10:04 | |
| *** ParsectiX has joined #openstack-infra | 10:04 | |
| *** mgkwill has joined #openstack-infra | 10:04 | |
| openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient: Render descriptions and comments as Markdown https://review.openstack.org/222709 | 10:04 |
| *** patrickeast has joined #openstack-infra | 10:07 | |
| *** dtantsur|brb is now known as dtantsur | 10:08 | |
| freerunner | fungi: It's here: https://review.openstack.org/#/c/226108/ . Hope, it will work. =) | 10:08 |
| *** niedbalski has quit IRC | 10:09 | |
| *** sergmelikyan has joined #openstack-infra | 10:10 | |
| *** Sree has quit IRC | 10:10 | |
| *** rameshg87 is now known as rameshg87-afk | 10:11 | |
| *** ParsectiX has quit IRC | 10:12 | |
| *** otter768 has joined #openstack-infra | 10:13 | |
| *** sergmelikyan has quit IRC | 10:17 | |
| *** otter768 has quit IRC | 10:17 | |
| *** mpaolino has joined #openstack-infra | 10:21 | |
| *** pc_m has joined #openstack-infra | 10:25 | |
| *** ParsectiX has joined #openstack-infra | 10:28 | |
| *** e0ne has quit IRC | 10:30 | |
| *** shardy has quit IRC | 10:32 | |
| *** e0ne has joined #openstack-infra | 10:35 | |
| *** ihrachys has joined #openstack-infra | 10:35 | |
| *** lawrance_ has quit IRC | 10:37 | |
| *** shardy has joined #openstack-infra | 10:37 | |
| openstackgerrit | Merged openstack/requirements: Update falcon requirement https://review.openstack.org/225886 | 10:38 |
| *** cdoron has joined #openstack-infra | 10:43 | |
| *** cdoron has quit IRC | 10:44 | |
| *** salv-orlando has joined #openstack-infra | 10:47 | |
| *** ddieterly has joined #openstack-infra | 10:47 | |
| *** peristeri has quit IRC | 10:48 | |
| *** fhubik has quit IRC | 10:49 | |
| *** trown|outttypeww is now known as trown | 10:51 | |
| *** ddieterly has quit IRC | 10:52 | |
| *** abregman|afk is now known as abregman | 10:53 | |
| *** fhubik has joined #openstack-infra | 10:54 | |
| *** samueldmq has quit IRC | 10:58 | |
| openstackgerrit | Bob Ball proposed openstack/diskimage-builder: Add support for Xen PV disks https://review.openstack.org/226249 | 10:59 |
| *** samueldmq has joined #openstack-infra | 11:01 | |
| *** jlanoux has quit IRC | 11:04 | |
| *** peristeri has joined #openstack-infra | 11:05 | |
| *** doug-fish has joined #openstack-infra | 11:09 | |
| *** mpaolino has quit IRC | 11:13 | |
| *** baoli has joined #openstack-infra | 11:17 | |
| d0ugal | Where is the best place to ask for help with gerrit? | 11:17 |
| d0ugal | as in, I think something is wrong with my account :) | 11:17 |
| *** ihrachys has quit IRC | 11:18 | |
| *** baoli_ has joined #openstack-infra | 11:19 | |
| openstackgerrit | Merged openstack-infra/tripleo-ci: Restore setting of origin/master https://review.openstack.org/226205 | 11:19 |
| *** ihrachys has joined #openstack-infra | 11:20 | |
| *** rameshg87-afk is now known as rameshg87 | 11:21 | |
| *** rameshg87 has quit IRC | 11:21 | |
| *** abregman is now known as abregman|afk | 11:22 | |
| *** baoli has quit IRC | 11:22 | |
| *** thorst has joined #openstack-infra | 11:23 | |
| *** akscram1 is now known as akscram | 11:24 | |
| *** abregman|afk has quit IRC | 11:28 | |
| openstackgerrit | Merged openstack-infra/release-tools: make version comparison more robust https://review.openstack.org/221408 | 11:28 |
| *** markvoelker has joined #openstack-infra | 11:28 | |
| openstackgerrit | Merged openstack-infra/release-tools: If the release request includes highlights, add them to the release notes https://review.openstack.org/222662 | 11:28 |
| openstackgerrit | Merged openstack-infra/release-tools: if no version is provided, default to most recent entry https://review.openstack.org/222663 | 11:30 |
| openstackgerrit | Merged openstack-infra/release-tools: print the actual release command being run in case we have to retry https://review.openstack.org/222664 | 11:30 |
| openstackgerrit | Merged openstack-infra/release-tools: Support stable branches in repos not under openstack/ https://review.openstack.org/223289 | 11:30 |
| *** gordc has joined #openstack-infra | 11:31 | |
| openstackgerrit | John Trowbridge proposed openstack-infra/tripleo-ci: Turn introspection back on https://review.openstack.org/225934 | 11:31 |
| *** markvoelker has quit IRC | 11:33 | |
| *** pblaho has joined #openstack-infra | 11:33 | |
| *** fhubik is now known as fhubik_brb | 11:33 | |
| *** ldnunes has joined #openstack-infra | 11:34 | |
| *** julim has joined #openstack-infra | 11:37 | |
| openstackgerrit | Valeriy Ponomaryov proposed openstack-infra/project-config: Add check-requirements job for manila-ui project https://review.openstack.org/226267 | 11:42 |
| *** fhubik_brb is now known as fhubik | 11:42 | |
| openstackgerrit | Valeriy Ponomaryov proposed openstack/requirements: Add manila-ui project to project list https://review.openstack.org/226223 | 11:42 |
| *** fhubik is now known as fhubik_brb | 11:43 | |
| *** akrivoka has joined #openstack-infra | 11:45 | |
| openstackgerrit | Marton Kiss proposed openstack-infra/system-config: Add custom settings.py template for ask-staging.o.o https://review.openstack.org/226269 | 11:46 |
| *** ddieterly has joined #openstack-infra | 11:48 | |
| *** jyuso1 has quit IRC | 11:53 | |
| *** ddieterly has quit IRC | 11:53 | |
| *** moravec has quit IRC | 11:53 | |
| *** claudiub has joined #openstack-infra | 11:53 | |
| d0ugal | I've checked the email addresses etc. all match up | 11:56 |
| d0ugal | but despite being listed in the tripleo group, I don't see any changes | 11:56 |
| *** shardy_ has joined #openstack-infra | 11:56 | |
| d0ugal | Nothing tripleo related when I view https://review.openstack.org/#/settings/group-memberships either | 11:57 |
| *** ociuhandu has quit IRC | 11:58 | |
| *** shardy has quit IRC | 11:58 | |
| *** zz_dimtruck is now known as dimtruck | 11:59 | |
| *** dosaboy has quit IRC | 12:00 | |
| *** jlanoux has joined #openstack-infra | 12:00 | |
| *** claudiub|2 has joined #openstack-infra | 12:01 | |
| *** dimtruck is now known as zz_dimtruck | 12:02 | |
| *** shardy_ has quit IRC | 12:02 | |
| *** dprince has joined #openstack-infra | 12:02 | |
| *** shardy has joined #openstack-infra | 12:03 | |
| *** claudiub has quit IRC | 12:03 | |
| *** e0ne has quit IRC | 12:03 | |
| *** ociuhandu has joined #openstack-infra | 12:03 | |
| *** Sree has joined #openstack-infra | 12:04 | |
| *** fhubik_brb is now known as fhubik | 12:04 | |
| *** bdemers has quit IRC | 12:06 | |
| openstackgerrit | Merged openstack-infra/python-storyboardclient: Change ignore-errors to ignore_errors https://review.openstack.org/225716 | 12:07 |
| *** dosaboy has joined #openstack-infra | 12:07 | |
| *** alexpilotti has joined #openstack-infra | 12:09 | |
| *** otter768 has joined #openstack-infra | 12:14 | |
| *** bexelbie has joined #openstack-infra | 12:18 | |
| *** markvoelker has joined #openstack-infra | 12:18 | |
| *** otter768 has quit IRC | 12:19 | |
| *** rfolco has joined #openstack-infra | 12:21 | |
| *** e0ne has joined #openstack-infra | 12:23 | |
| *** doug-fish has quit IRC | 12:23 | |
| *** cdoron has joined #openstack-infra | 12:24 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/diskimage-builder: Add functional test for ironic-agent on Fedora https://review.openstack.org/226293 | 12:24 |
| *** cdoron has quit IRC | 12:24 | |
| *** samueldmq has quit IRC | 12:25 | |
| *** doug-fish has joined #openstack-infra | 12:25 | |
| *** bswartz has quit IRC | 12:26 | |
| *** akrivoka has left #openstack-infra | 12:26 | |
| *** dizquierdo has quit IRC | 12:27 | |
| *** kgiusti has joined #openstack-infra | 12:29 | |
| *** doug-fish has quit IRC | 12:30 | |
| *** zns has joined #openstack-infra | 12:30 | |
| *** burgerk has joined #openstack-infra | 12:31 | |
| *** nadya has joined #openstack-infra | 12:32 | |
| *** edmondsw has joined #openstack-infra | 12:33 | |
| *** weshay_xchat has joined #openstack-infra | 12:33 | |
| *** zz_dimtruck is now known as dimtruck | 12:34 | |
| *** weshay_xchat is now known as weshay | 12:35 | |
| *** bcourt has joined #openstack-infra | 12:36 | |
| *** ddieterly has joined #openstack-infra | 12:38 | |
| *** sean-k-mooney has joined #openstack-infra | 12:38 | |
| *** achanda has joined #openstack-infra | 12:41 | |
| openstackgerrit | Chris Dent proposed openstack-infra/project-config: Adjust rally jobs to use ceilometer devstack plugin https://review.openstack.org/220122 | 12:41 |
| openstackgerrit | Chris Dent proposed openstack-infra/project-config: Use ceilometer devstack plugin in gate https://review.openstack.org/196446 | 12:41 |
| *** nelsnels_ has quit IRC | 12:41 | |
| *** achanda has quit IRC | 12:42 | |
| *** nelsnelson has joined #openstack-infra | 12:42 | |
| sean-k-mooney | hi not sure if any infra cores are about but just wanted to ask if someone could take a look at a review | 12:42 |
| sean-k-mooney | https://review.openstack.org/#/c/222763/ | 12:42 |
| sean-k-mooney | i would like to create a repo to house a collectd plugin to send telemetry to ceilometer | 12:43 |
| *** kiran-r has quit IRC | 12:45 | |
| *** doug-fish has joined #openstack-infra | 12:45 | |
| *** _kiran_ has joined #openstack-infra | 12:45 | |
| *** _kiran_ is now known as kiran-r | 12:45 | |
| *** aysyd has joined #openstack-infra | 12:45 | |
| *** regXboi has joined #openstack-infra | 12:46 | |
| *** hichihara has joined #openstack-infra | 12:47 | |
| *** bltavares has joined #openstack-infra | 12:48 | |
| *** samueldmq has joined #openstack-infra | 12:48 | |
| *** boris-42 has quit IRC | 12:49 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/diskimage-builder: Add functional test for ironic-agent on Fedora https://review.openstack.org/226293 | 12:50 |
| *** amuller has joined #openstack-infra | 12:51 | |
| *** fhubik is now known as fhubik_brb | 12:51 | |
| *** xyang1 has joined #openstack-infra | 12:53 | |
| *** eharney has joined #openstack-infra | 12:54 | |
| *** fhubik_brb is now known as fhubik | 12:57 | |
| *** tongli has joined #openstack-infra | 12:58 | |
| regXboi | anteaya: gm, where would I find the code that runs when gate-neutron-docs does its thing? | 12:58 |
| rcarrillocruz | regXboi: that should be in project-config repo | 12:59 |
| rcarrillocruz | gate-*-docs is a template tho | 12:59 |
| regXboi | rcarrillocruz: I have that repo - let me relook - I probably missed it the first time through | 13:00 |
| rcarrillocruz | check python-jobs | 13:00 |
| rcarrillocruz | should be there | 13:00 |
| rcarrillocruz | regXboi: python-jobs.yaml: name: 'gate-{name}-docs' | 13:02 |
| rcarrillocruz | that from a grep output | 13:02 |
| rcarrillocruz | therefore | 13:02 |
| rcarrillocruz | project-config/jenkins/jobs/python-jobs.yaml | 13:02 |
| regXboi | yeah - found that and am going a bit further down the rabbit hole | 13:02 |
| *** sean-k-mooney has left #openstack-infra | 13:02 | |
| *** bswartz has joined #openstack-infra | 13:03 | |
| *** tlian has joined #openstack-infra | 13:03 | |
| regXboi | rcarrillocruz: do you know what is used as the RST renderer? | 13:03 |
| *** dramalho has joined #openstack-infra | 13:04 | |
| rcarrillocruz | sphynx? not sure... | 13:04 |
| *** jsavak has joined #openstack-infra | 13:05 | |
| openstackgerrit | Chris Dent proposed openstack-infra/project-config: Adjust rally jobs to use ceilometer devstack plugin https://review.openstack.org/220122 | 13:05 |
| *** ddieterly has quit IRC | 13:11 | |
| *** lawrancejing has joined #openstack-infra | 13:11 | |
| *** fhubik_ has joined #openstack-infra | 13:13 | |
| *** fhubik_ is now known as fhubik_brb | 13:13 | |
| *** rlandy has quit IRC | 13:13 | |
| *** dims has quit IRC | 13:14 | |
| *** dims has joined #openstack-infra | 13:15 | |
| *** baoli_ has quit IRC | 13:16 | |
| *** fhubik has quit IRC | 13:16 | |
| *** baoli has joined #openstack-infra | 13:16 | |
| *** _kiran_ has joined #openstack-infra | 13:16 | |
| *** kiran-r has quit IRC | 13:17 | |
| *** _kiran_ is now known as kiran-r | 13:17 | |
| *** tongli has quit IRC | 13:17 | |
| *** salv-orlando has quit IRC | 13:18 | |
| *** k4n0 has quit IRC | 13:19 | |
| *** dkranz has joined #openstack-infra | 13:19 | |
| *** tiswanso has joined #openstack-infra | 13:20 | |
| *** baoli_ has joined #openstack-infra | 13:21 | |
| *** smarcet has joined #openstack-infra | 13:22 | |
| openstackgerrit | sebastian marcet proposed openstack-infra/openstackid: OIDC - OpenId Connect Implementation https://review.openstack.org/198458 | 13:23 |
| *** baoli has quit IRC | 13:24 | |
| *** BharatK has quit IRC | 13:24 | |
| *** alivigni has joined #openstack-infra | 13:25 | |
| *** moravec has joined #openstack-infra | 13:25 | |
| *** diazjf has joined #openstack-infra | 13:27 | |
| *** thiagop has joined #openstack-infra | 13:28 | |
| *** glauco_ has quit IRC | 13:29 | |
| *** glauco has joined #openstack-infra | 13:29 | |
| *** burgerk has quit IRC | 13:31 | |
| *** mriedem_away is now known as mriedem | 13:31 | |
| *** eantyshev has left #openstack-infra | 13:34 | |
| diazjf | Good Morning, sdague, jhesketh, can you please take a look at https://review.openstack.org/#/c/226113/ it is needed for one of our gatetests to pass | 13:34 |
| *** bltavares has quit IRC | 13:36 | |
| *** dramalho has quit IRC | 13:36 | |
| *** e0ne has quit IRC | 13:37 | |
| *** dramalho has joined #openstack-infra | 13:38 | |
| *** ccarrara has joined #openstack-infra | 13:39 | |
| *** e0ne has joined #openstack-infra | 13:39 | |
| *** pradk has joined #openstack-infra | 13:39 | |
| *** e0ne has quit IRC | 13:39 | |
| smarcet | morning , jeblair, fungi , can you guys please take a look at https://review.openstack.org/#/c/214335/, we need to add app key as a parameter for openstackid class, also e need to make app.php a template | 13:40 |
| *** e0ne has joined #openstack-infra | 13:40 | |
| *** kiran-r has quit IRC | 13:40 | |
| *** ddieterly has joined #openstack-infra | 13:41 | |
| *** BharatK has joined #openstack-infra | 13:41 | |
| *** Daisy has joined #openstack-infra | 13:42 | |
| *** claudiub|2 has quit IRC | 13:43 | |
| *** deepakcs has quit IRC | 13:43 | |
| *** fawadkhaliq has joined #openstack-infra | 13:44 | |
| *** peristeri has quit IRC | 13:44 | |
| *** dizquierdo has joined #openstack-infra | 13:44 | |
| openstackgerrit | Merged openstack-infra/project-config: Add mistral back to murano job definitions file https://review.openstack.org/226108 | 13:45 |
| *** lawrancejing has quit IRC | 13:45 | |
| *** jaypipes has joined #openstack-infra | 13:46 | |
| smarcet | jeblair, fungi : i will need some variables to hiera for openstackid config , should i send out an email with variables names/values 2 add ? | 13:47 |
| *** doug-fis_ has joined #openstack-infra | 13:47 | |
| *** bdemers has joined #openstack-infra | 13:47 | |
| fungi | smarcet: where is the change to openstack-infra/system-config making use of those variables? | 13:48 |
| *** cbader has joined #openstack-infra | 13:49 | |
| *** jaypipes has quit IRC | 13:49 | |
| smarcet | here https://review.openstack.org/#/c/214344/ | 13:49 |
| *** peristeri has joined #openstack-infra | 13:50 | |
| *** jaypipes has joined #openstack-infra | 13:50 | |
| *** doug-fish has quit IRC | 13:50 | |
| openstackgerrit | Merged openstack-infra/puppet-openstackid: Updated configuration for OIDC https://review.openstack.org/214335 | 13:51 |
| *** doug-fish has joined #openstack-infra | 13:51 | |
| *** amitgandhinz has joined #openstack-infra | 13:51 | |
| *** Piet has joined #openstack-infra | 13:52 | |
| *** doug-fis_ has quit IRC | 13:52 | |
| *** hodos has joined #openstack-infra | 13:52 | |
| *** mriedem is now known as mriedem_meeting | 13:53 | |
| *** amotoki has joined #openstack-infra | 13:53 | |
| fungi | smarcet: just the openstackid_recaptcha_public_key and openstackid_recaptcha_private_key variables, yeah? you can go ahead and send those to fungi@yuggoth.org and i'll add them into hiera in preparation for that change | 13:56 |
| smarcet | yeah and the app key also | 13:56 |
| smarcet | ok cool | 13:56 |
| smarcet | on it | 13:56 |
| smarcet | thx a lot :) | 13:56 |
| fungi | oh, yeah openstackid_app_key too. got it | 13:56 |
| fungi | does openstackid_app_key need to be synchronized with a value somewhere else, or do i just need to generate something random for that one? | 13:57 |
| smarcet | openstacid app key follows a format | 13:57 |
| fungi | okay, thanks | 13:58 |
| smarcet | its indeed a ramdon value, but its generated by laravel artisan tool | 13:58 |
| smarcet | so i will generete 2 values and will send u out those ones :) | 13:58 |
| *** fawadkhaliq has quit IRC | 13:59 | |
| *** ihrachys has quit IRC | 13:59 | |
| *** ihrachys has joined #openstack-infra | 13:59 | |
| openstackgerrit | Darragh Bailey proposed openstack-infra/jenkins-job-builder: Capture logs with FakeLogging Fixture https://review.openstack.org/226107 | 14:00 |
| openstackgerrit | Darragh Bailey proposed openstack-infra/jenkins-job-builder: Allow save to be called more than once https://review.openstack.org/226331 | 14:00 |
| *** fawadkhaliq has joined #openstack-infra | 14:01 | |
| *** DericHorn-HP has joined #openstack-infra | 14:02 | |
| *** fawadk has joined #openstack-infra | 14:02 | |
| fungi | smarcet: also you'll want to adjust 214344 to set explicit fallback values for those new entries, so that they're properly tested | 14:02 |
| openstackgerrit | Merged openstack-infra/elastic-recheck: Add query for ansible build timeout bug 1498009 https://review.openstack.org/225702 | 14:03 |
| openstack | bug 1498009 in openstack-ansible "gate-openstack-ansible-dsvm-commit build times out in the gate" [Undecided,New] https://launchpad.net/bugs/1498009 | 14:03 |
| *** stevemar has joined #openstack-infra | 14:03 | |
| *** amuller_ has joined #openstack-infra | 14:04 | |
| *** burgerk has joined #openstack-infra | 14:05 | |
| *** moravec has quit IRC | 14:05 | |
| *** fawadkhaliq has quit IRC | 14:05 | |
| fungi | smarcet: and the tests caught an omission too. i've added a review comment about passing those new variables through the openstack_project::openstackid_prod class | 14:06 |
| smarcet | fungi: i saw it | 14:06 |
| smarcet | working on it :) | 14:06 |
| *** moravec1 has joined #openstack-infra | 14:08 | |
| *** amuller has quit IRC | 14:08 | |
| *** fawadkhaliq has joined #openstack-infra | 14:09 | |
| openstackgerrit | sebastian marcet proposed openstack-infra/system-config: Updated OpenStackId configuration https://review.openstack.org/214344 | 14:09 |
| *** fawadk has quit IRC | 14:11 | |
| *** hdd has joined #openstack-infra | 14:11 | |
| openstackgerrit | sebastian marcet proposed openstack-infra/system-config: Updated OpenStackId configuration https://review.openstack.org/214344 | 14:14 |
| *** doug-fish has quit IRC | 14:14 | |
| *** otter768 has joined #openstack-infra | 14:15 | |
| *** doug-fish has joined #openstack-infra | 14:15 | |
| *** bexelbie has quit IRC | 14:15 | |
| pabelanger | timrc: Ya. Guess we could try and patch grafana to fix it. | 14:16 |
| pabelanger | ianw: fungi | 14:16 |
| pabelanger | ianw: fungi: err | 14:16 |
| *** ayoung has joined #openstack-infra | 14:16 | |
| pabelanger | ianw: fungi: I think the --downloadonly flag for dnf finally landed | 14:16 |
| *** olaph has quit IRC | 14:16 | |
| pabelanger | ianw: fungi: will be testing this morning | 14:16 |
| *** nadya has quit IRC | 14:17 | |
| *** nadya has joined #openstack-infra | 14:18 | |
| *** bgmccollum has left #openstack-infra | 14:18 | |
| *** nadya has quit IRC | 14:20 | |
| *** otter768 has quit IRC | 14:20 | |
| *** glauco has quit IRC | 14:20 | |
| *** nadya has joined #openstack-infra | 14:20 | |
| *** MarkAtwood has joined #openstack-infra | 14:20 | |
| *** dangers_away is now known as dangers | 14:23 | |
| *** rguillebert has quit IRC | 14:25 | |
| *** jhesketh has quit IRC | 14:25 | |
| *** glauco has joined #openstack-infra | 14:26 | |
| *** DericHorn-HP has quit IRC | 14:26 | |
| *** dramalho has quit IRC | 14:27 | |
| *** ccarrara has quit IRC | 14:27 | |
| *** glauco has quit IRC | 14:27 | |
| *** ccarrara has joined #openstack-infra | 14:27 | |
| *** wshao has joined #openstack-infra | 14:28 | |
| *** rakhmerov has quit IRC | 14:28 | |
| *** jhesketh has joined #openstack-infra | 14:28 | |
| openstackgerrit | sebastian marcet proposed openstack-infra/system-config: Updated OpenStackId configuration https://review.openstack.org/214344 | 14:28 |
| *** shardy_ has joined #openstack-infra | 14:28 | |
| jhesketh | Morning | 14:29 |
| *** alexpilotti has quit IRC | 14:30 | |
| *** boris-42 has joined #openstack-infra | 14:30 | |
| *** rguillebert has joined #openstack-infra | 14:30 | |
| *** shardy has quit IRC | 14:30 | |
| *** rakhmerov has joined #openstack-infra | 14:31 | |
| *** bexelbie has joined #openstack-infra | 14:31 | |
| *** ayoung has quit IRC | 14:32 | |
| *** amuller_ is now known as amuller | 14:33 | |
| *** shardy_ has quit IRC | 14:34 | |
| *** shardy has joined #openstack-infra | 14:34 | |
| *** bdemers has quit IRC | 14:37 | |
| *** ccarrara has quit IRC | 14:37 | |
| *** bdemers has joined #openstack-infra | 14:38 | |
| *** bharathm has joined #openstack-infra | 14:38 | |
| *** ccarrara has joined #openstack-infra | 14:39 | |
| *** AJaeger has joined #openstack-infra | 14:39 | |
| *** fawadkhaliq has quit IRC | 14:39 | |
| *** mtanino has joined #openstack-infra | 14:41 | |
| *** cburroughs has joined #openstack-infra | 14:41 | |
| openstackgerrit | sebastian marcet proposed openstack-infra/system-config: Updated OpenStackId configuration https://review.openstack.org/214344 | 14:42 |
| openstackgerrit | Merged openstack-infra/shade: Make inner_exception a private member https://review.openstack.org/226085 | 14:43 |
| *** sabeen1 has joined #openstack-infra | 14:45 | |
| *** hashar has joined #openstack-infra | 14:45 | |
| *** oomichi has joined #openstack-infra | 14:46 | |
| *** dramalho has joined #openstack-infra | 14:46 | |
| *** glauco has joined #openstack-infra | 14:47 | |
| *** cloudtrainme has joined #openstack-infra | 14:48 | |
| *** bdemers_ has joined #openstack-infra | 14:49 | |
| *** mdbooth has quit IRC | 14:49 | |
| *** bdemers has quit IRC | 14:49 | |
| *** bdemers_ is now known as bdemers | 14:49 | |
| *** AJaeger has quit IRC | 14:50 | |
| *** adduarte has joined #openstack-infra | 14:51 | |
| *** tsekiyama has joined #openstack-infra | 14:51 | |
| *** tsekiyam_ has joined #openstack-infra | 14:52 | |
| *** rossella_s has quit IRC | 14:53 | |
| *** ayoung has joined #openstack-infra | 14:55 | |
| openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Add db api methods to get keys from metadata tables https://review.openstack.org/226162 | 14:55 |
| openstackgerrit | Matthew Treinish proposed openstack-infra/subunit2sql: Add migration to fix wrong column name in test_metadata https://review.openstack.org/226363 | 14:55 |
| *** mdbooth has joined #openstack-infra | 14:55 | |
| openstackgerrit | svasheka proposed openstack-infra/project-config: Alter horizon.yaml to get screenshots copied https://review.openstack.org/226364 | 14:55 |
| *** tsekiyama has quit IRC | 14:56 | |
| *** flepied has quit IRC | 14:56 | |
| *** Guest24378 is now known as dansmith | 14:56 | |
| *** angdraug has joined #openstack-infra | 14:56 | |
| *** cburroughs has quit IRC | 14:58 | |
| *** dangers is now known as dangers_away | 14:58 | |
| openstackgerrit | svasheka proposed openstack-infra/project-config: Alter horizon.yaml to get screenshots copied https://review.openstack.org/226364 | 14:58 |
| *** eharney has quit IRC | 14:59 | |
| *** sabeen2 has joined #openstack-infra | 14:59 | |
| *** spzala has joined #openstack-infra | 14:59 | |
| fungi | smarcet: i've added the 6 supplied hiera keys, and your latest patchset on 214344 looks right to me | 15:01 |
| fungi | pabelanger: that's excellent news! | 15:01 |
| *** fhubik_brb is now known as fhubik_ | 15:01 | |
| pabelanger | fungi: Ya, just waiting for the build 1.1.2 to be released into repos | 15:02 |
| *** sabeen1 has quit IRC | 15:02 | |
| smarcet | fungi: ok | 15:02 |
| anteaya | regXboi: did you find all you need to understand the docs job? | 15:02 |
| *** moravec1 has quit IRC | 15:02 | |
| smarcet | next step will be to decide what to do with https://review.openstack.org/#/c/198458/ | 15:02 |
| smarcet | i know that is a massive patch | 15:02 |
| smarcet | but was tested against php 5.3.10 and php 5.4/5.5 | 15:03 |
| regXboi | anteaya: yes - enough | 15:03 |
| openstackgerrit | svasheka proposed openstack-infra/project-config: Alter horizon.yaml to get screenshots copied https://review.openstack.org/226364 | 15:03 |
| openstackgerrit | Paul Belanger proposed openstack-infra/grafyaml: The apikey is now optional https://review.openstack.org/224796 | 15:03 |
| openstackgerrit | Paul Belanger proposed openstack-infra/grafyaml: [WIP] Add delete command to CLI https://review.openstack.org/225228 | 15:03 |
| smarcet | also passed all test from http://openid.net/certification/testing/ | 15:03 |
| *** doug-fish has quit IRC | 15:03 | |
| anteaya | regXboi: glad to hear it | 15:03 |
| *** doug-fish has joined #openstack-infra | 15:03 | |
| regXboi | anteaya: yeah, now trying to cast what I have in terms that will work with it | 15:04 |
| *** BharatK has quit IRC | 15:04 | |
| pabelanger | smarcet: why are there libraries sourced in the patchset? | 15:04 |
| *** doug-fish has quit IRC | 15:04 | |
| pabelanger | eg: bower libs | 15:04 |
| anteaya | regXboi: care to share what your goal is? | 15:05 |
| smarcet | bc we dont have a way to run bower on the deployment script yet | 15:05 |
| *** doug-fish has joined #openstack-infra | 15:05 | |
| regXboi | anteaya: linking the graphite dashboards and a few other things into the auto generated docs on each neutron build | 15:05 |
| regXboi | anteaya: https://review.openstack.org/#/c/226265/ | 15:06 |
| *** thorst has quit IRC | 15:06 | |
| *** mriedem_meeting is now known as mriedem | 15:06 | |
| *** doug-fish has quit IRC | 15:06 | |
| *** zns has quit IRC | 15:06 | |
| pabelanger | smarcet: Is anybody working on it? Embedding libraries into your patchset doesn't feel like the right approach. And also adds a complexity into the review process | 15:06 |
| *** doug-fish has joined #openstack-infra | 15:06 | |
| *** zns_ has joined #openstack-infra | 15:06 | |
| *** armax has joined #openstack-infra | 15:07 | |
| smarcet | pabelanger | 15:07 |
| smarcet | i will propose a change on deployment script to support bower | 15:07 |
| anteaya | regXboi: so even with all of the data collected on mtreinish's dashboard this work is still necessary for neutron? | 15:07 |
| smarcet | and will remove those libs from patch :) | 15:07 |
| openstackgerrit | svasheka proposed openstack-infra/project-config: Alter horizon.yaml to get screenshots copied. https://review.openstack.org/226364 | 15:07 |
| openstackgerrit | svasheka proposed openstack-infra/project-config: Alter horizon.yaml to get screenshots copied. https://review.openstack.org/226364 | 15:08 |
| regXboi | anteaya: for now - yes - I'll happily pull it down once the health dashboard is fully working | 15:08 |
| pabelanger | smarcet: Ya, I think that is best. And will help reduce the size | 15:08 |
| smarcet | sure np :) | 15:08 |
| *** bswartz has quit IRC | 15:08 | |
| regXboi | anteaya: there are also links in there to gerrit review dashboards, which is outside the scope of the health dashboard | 15:08 |
| pabelanger | I'd have to confirm, but you'd likely want to drop everything in the public directory. Since that is dependencies / compiled code | 15:09 |
| *** rossella_s has joined #openstack-infra | 15:09 | |
| anteaya | mtreinish: is anyone currently working on the health dashboard? | 15:09 |
| *** ZZelle has quit IRC | 15:09 | |
| regXboi | anteaya: there are folks - I'm one of them :) | 15:09 |
| *** ZZelle has joined #openstack-infra | 15:10 | |
| *** moravec1 has joined #openstack-infra | 15:10 | |
| openstackgerrit | sebastian marcet proposed openstack-infra/openstackid: OIDC - OpenId Connect Implementation https://review.openstack.org/198458 | 15:10 |
| mtreinish | anteaya: yes, lots of people are | 15:10 |
| mtreinish | myself included | 15:10 |
| mtreinish | anteaya: https://review.openstack.org/#/q/status:open+project:openstack/openstack-health,n,z | 15:11 |
| mtreinish | for current open reviews | 15:11 |
| mtreinish | there are also a bunch on the subunit2sql side | 15:11 |
| *** kiran-r has joined #openstack-infra | 15:12 | |
| anteaya | looks like you and tim | 15:12 |
| anteaya | but okay thanks | 15:12 |
| anteaya | regXboi: okay just wanted to ensure you still had effort there | 15:12 |
| *** annegentle has joined #openstack-infra | 15:14 | |
| regXboi | :) | 15:14 |
| *** bswartz has joined #openstack-infra | 15:15 | |
| *** bpokorny has joined #openstack-infra | 15:16 | |
| *** glauco has quit IRC | 15:18 | |
| *** peristeri has quit IRC | 15:18 | |
| *** rossella_s has quit IRC | 15:19 | |
| *** dims has quit IRC | 15:20 | |
| *** btully has quit IRC | 15:21 | |
| *** btully has joined #openstack-infra | 15:22 | |
| *** tsekiyam_ has quit IRC | 15:22 | |
| *** dramalho has quit IRC | 15:22 | |
| *** tsekiyama has joined #openstack-infra | 15:23 | |
| *** yfried_ has quit IRC | 15:24 | |
| clarkb | Nakato: https://review.openstack.org/#/c/219582/ is in merge conflict might be good to get that fixed and merged so that M cycle can really push on constraints for !devstack | 15:25 |
| *** eharney has joined #openstack-infra | 15:26 | |
| *** Sree has quit IRC | 15:26 | |
| *** fhubik_ is now known as fhubik_brb | 15:26 | |
| *** puranamr has joined #openstack-infra | 15:26 | |
| *** annegent_ has joined #openstack-infra | 15:27 | |
| *** puranamr has quit IRC | 15:27 | |
| *** e0ne has quit IRC | 15:28 | |
| *** dims has joined #openstack-infra | 15:28 | |
| *** e0ne has joined #openstack-infra | 15:30 | |
| *** annegentle has quit IRC | 15:30 | |
| openstackgerrit | Clint Adams proposed openstack-infra/infra-specs: tracking of priority efforts in YAML https://review.openstack.org/219372 | 15:31 |
| *** rossella_s has joined #openstack-infra | 15:31 | |
| *** ifarkas has quit IRC | 15:31 | |
| *** weshay is now known as weshay|bbiab | 15:32 | |
| openstackgerrit | sebastian marcet proposed openstack-infra/openstackid: OIDC - OpenId Connect Implementation https://review.openstack.org/198458 | 15:32 |
| *** _ody_ is now known as _ody | 15:34 | |
| *** thorst has joined #openstack-infra | 15:34 | |
| *** sergmelikyan has joined #openstack-infra | 15:36 | |
| *** annegent_ has quit IRC | 15:36 | |
| *** annegentle has joined #openstack-infra | 15:36 | |
| *** geoffarnold has joined #openstack-infra | 15:39 | |
| openstackgerrit | sebastian marcet proposed openstack-infra/puppet-openstackid: Added bower support https://review.openstack.org/226377 | 15:40 |
| *** e0ne has quit IRC | 15:41 | |
| smarcet | fungi, jeblair , pabelanger : patch https://review.openstack.org/#/c/226377/1 adds bower support for OpestackId deployment | 15:43 |
| smarcet | please review at will :) | 15:43 |
| *** fhubik_brb is now known as fhubik_ | 15:45 | |
| *** flepied has joined #openstack-infra | 15:45 | |
| *** dtantsur is now known as dtantsur|afk | 15:45 | |
| *** ivar-lazzaro has joined #openstack-infra | 15:47 | |
| *** ivar-lazzaro has quit IRC | 15:47 | |
| clarkb | pleia2: when you wake can you see the recent thread traffic on the stable release versions in zanata thread? | 15:47 |
| *** ivar-lazzaro has joined #openstack-infra | 15:47 | |
| clarkb | pleia2: I am going to continue on that today as it appears to be important | 15:47 |
| *** amuller is now known as amuller_afk | 15:49 | |
| *** hichihara has quit IRC | 15:50 | |
| clarkb | pleia2: also if you have time https://review.openstack.org/#/c/225966/5 will simplify things a bit for the version stuff and I can rebase onto that | 15:51 |
| *** flepied has quit IRC | 15:51 | |
| clarkb | aeng_: awake? do you know if zanata client pushing a version that does not exist in zanata will create that version or error? I suppose this is why we have a dev zanata instance and I should just go test it :) | 15:51 |
| *** oomichi has quit IRC | 15:53 | |
| openstackgerrit | sebastian marcet proposed openstack-infra/puppet-openstackid: Added bower support https://review.openstack.org/226377 | 15:54 |
| docaedo | What's the process for me to request an SSL cert for apps.openstack.org? | 15:55 |
| fungi | docaedo: you just followed it | 15:55 |
| *** Ala has quit IRC | 15:55 | |
| docaedo | fungi: haha awesome, thanks :) | 15:55 |
| *** monester has joined #openstack-infra | 15:55 | |
| *** woodster_ has joined #openstack-infra | 15:56 | |
| docaedo | we would like one in order to better support the horizon plugin for the app-catalog, which will throw errors if your horizon is behind SSL and then the catalog panel tries loading stuff from non-SSL apps.openstack.org | 15:56 |
| *** dizquierdo has quit IRC | 15:57 | |
| fungi | docaedo: i'll add it to my to do list, but may not get to it until tomorrow. also you should submit a change to openstack-infra/system-config to link the keypair from hiera in the global site manifest (and if necessary plumb the corresponding variables through the necessary puppet classes and add support in the puppet module for the site too) | 15:57 |
| docaedo | fungi: thank you, no rush as I won't be able to submit that patch until later tonight or tomorrow anyway (in transit today so going to be mostly working offline) | 15:58 |
| docaedo | fungi: I'll look at some of the other infra-hosted sites that have SSL and will do my best to match up stuff for app catalog | 15:58 |
| fungi | yeah, today is day-o-meetings for me so little free time to do much more than reply to a few e-mails | 15:58 |
| openstackgerrit | Kevin L. Mitchell proposed stackforge/python-jenkins: Allow setting the next build number https://review.openstack.org/224275 | 15:59 |
| docaedo | fungi: thanks a bunch, fully understand on the timing :) will ping back likely tomorrow then when all my ducks are in a row | 15:59 |
| *** AJaeger has joined #openstack-infra | 16:00 | |
| *** mrmartin has quit IRC | 16:01 | |
| *** bexelbie has quit IRC | 16:01 | |
| teran | folks, we've found an issue in our instance of lodgeit with replying on private pastes. Issue was caused by improper paste_id fetching. But strange thing is you don't have that bug on paste.openstack.org, so do you have any patches not reflected in https://github.com/openstack-infra/lodgeit ? | 16:01 |
| mordred | teran: not to my knowledge, no | 16:02 |
| *** shashank_hegde has joined #openstack-infra | 16:02 | |
| *** BharatK has joined #openstack-infra | 16:02 | |
| *** dzamboni has joined #openstack-infra | 16:03 | |
| *** jlanoux has quit IRC | 16:04 | |
| *** ashleighfarnham has joined #openstack-infra | 16:05 | |
| *** ddieterly has quit IRC | 16:05 | |
| *** ashleighfarnham has quit IRC | 16:05 | |
| *** ddieterly has joined #openstack-infra | 16:05 | |
| *** ashleighfarnham has joined #openstack-infra | 16:05 | |
| *** rguillebert has quit IRC | 16:05 | |
| *** Sree has joined #openstack-infra | 16:06 | |
| *** ddieterl_ has joined #openstack-infra | 16:06 | |
| *** annegent_ has joined #openstack-infra | 16:06 | |
| *** sergmelikyan has quit IRC | 16:06 | |
| *** annegentle has quit IRC | 16:07 | |
| *** fhubik_ has quit IRC | 16:08 | |
| *** fhubik_ has joined #openstack-infra | 16:09 | |
| *** ddieterly has quit IRC | 16:10 | |
| *** scheuran has quit IRC | 16:11 | |
| *** bpokorny has quit IRC | 16:11 | |
| *** skolekonov is now known as _skolekonov | 16:11 | |
| *** gyee has joined #openstack-infra | 16:11 | |
| *** bpokorny has joined #openstack-infra | 16:12 | |
| *** yamahata has joined #openstack-infra | 16:13 | |
| *** tonytan4ever has joined #openstack-infra | 16:14 | |
| *** flepied has joined #openstack-infra | 16:14 | |
| *** ildikov has quit IRC | 16:14 | |
| *** fhubik_ is now known as fhubik_brb | 16:14 | |
| openstackgerrit | sebastian marcet proposed openstack-infra/puppet-openstackid: Added bower support https://review.openstack.org/226377 | 16:14 |
| *** bswartz has quit IRC | 16:14 | |
| *** fhubik_brb is now known as fhubik_ | 16:15 | |
| diazjf | can I get a +A on https://review.openstack.org/#/c/226113/ need a fix to get some Barbican code in | 16:15 |
| *** otter768 has joined #openstack-infra | 16:16 | |
| zaro | morning | 16:16 |
| *** erikwilson has joined #openstack-infra | 16:17 | |
| *** puranamr has joined #openstack-infra | 16:17 | |
| zaro | pelix: folder support is merged! | 16:18 |
| *** glauco has joined #openstack-infra | 16:19 | |
| *** baoli_ has quit IRC | 16:19 | |
| fungi | heading out to grab lunch, back in a while | 16:19 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config: WIP: Merge translation proposal scripts https://review.openstack.org/226398 | 16:19 |
| *** sridhar_ram has joined #openstack-infra | 16:19 | |
| *** bswartz has joined #openstack-infra | 16:20 | |
| anteaya | morning zaro | 16:20 |
| anteaya | fungi: enjoy lunch | 16:20 |
| *** fhubik_ has quit IRC | 16:20 | |
| *** otter768 has quit IRC | 16:20 | |
| diazjf | thanks fungi | 16:20 |
| *** rguillebert has joined #openstack-infra | 16:20 | |
| openstackgerrit | Merged openstack-infra/project-config: Remove lib-forward-testing from python-barbicanclient https://review.openstack.org/226113 | 16:21 |
| *** matrohon has quit IRC | 16:21 | |
| pelix | zaro: Yay! | 16:21 |
| pelix | nice to finally have it there, now just need to work out how to convert to Requests and we're completely finished :p | 16:22 |
| *** moravec1 has quit IRC | 16:23 | |
| *** moravec has joined #openstack-infra | 16:23 | |
| *** olaph has joined #openstack-infra | 16:24 | |
| teran | mordred: got it, thanks | 16:25 |
| *** erikmwilson_ has joined #openstack-infra | 16:25 | |
| *** dosaboy_ has joined #openstack-infra | 16:25 | |
| *** erikmwilson_ has quit IRC | 16:25 | |
| zaro | pelix: that's what i was just thinking. probably not much more features for python-jenkins ;) | 16:26 |
| *** dosaboy_ has quit IRC | 16:26 | |
| *** enikanorov_ has quit IRC | 16:26 | |
| *** erikmwilson_ has joined #openstack-infra | 16:26 | |
| *** enikanorov_ has joined #openstack-infra | 16:26 | |
| *** erikwilson has quit IRC | 16:26 | |
| zaro | hashar: i think this should work now: https://review.openstack.org/#/c/207616/ | 16:26 |
| *** weshay|bbiab is now known as weshay | 16:26 | |
| *** erikwilson has joined #openstack-infra | 16:27 | |
| zaro | pelix: maybe there's a miscommunication on this one, https://review.openstack.org/#/c/185037 | 16:27 |
| waynr | zaro, pelix: oooo, gonna implement requests library in python-jenkins? i just used that library for the first time recently, i like :) | 16:27 |
| *** dramalho has joined #openstack-infra | 16:28 | |
| zaro | waynr: you should review: https://review.openstack.org/#/c/205637/ | 16:28 |
| waynr | okay, i've queued it in my TODO for the day | 16:29 |
| *** dramalho has quit IRC | 16:29 | |
| zaro | cool | 16:29 |
| *** nadya has quit IRC | 16:31 | |
| *** erikmwilson_ has quit IRC | 16:31 | |
| *** dramalho has joined #openstack-infra | 16:32 | |
| *** dosaboy has quit IRC | 16:32 | |
| pelix | waynr: yeah, it's partially in place but there are problems with the POSTs. I've got some fixes locally, but I haven't quite worked out all the little details. See https://review.openstack.org/205637 for current progress. | 16:33 |
| zaro | pelix: so i have a problem with 'exception_project001.yaml' test. that seems to be testing that what's in it fails when run with jjb it's not checking what's failing. | 16:33 |
| *** dosaboy has joined #openstack-infra | 16:33 | |
| zaro | pelix: that test would always fail so it might fail for some other reason that's not expected? | 16:34 |
| *** alexsyip has joined #openstack-infra | 16:34 | |
| dhellmann | fungi, clarkb, pleia2 : if I need to delete and recreate a branch for a project, how messy is that? | 16:35 |
| *** kiran-r is now known as _kiran_ | 16:35 | |
| pelix | zaro: don' think so, that looks to the test duplicates which has the following exception check: 'with ExpectedException(JenkinsJobsException, "^Duplicate .*")' | 16:35 |
| pelix | so although it's any JenkinsJobsException, the message must start with 'Duplicate ' | 16:36 |
| clarkb | dhellmann: its not that messy, you just have to abandon all open changes on the branch, delete the branch in the gerrit ui, create the branch in the gerrit ui | 16:36 |
| dhellmann | clarkb: ok, that's not too bad | 16:36 |
| *** adduarte has quit IRC | 16:36 | |
| pleia2 | clarkb: approved 225966 | 16:36 |
| *** fitoduarte has joined #openstack-infra | 16:37 | |
| pelix | we could always subclass JenkinsJobException with a specific exception for DuplicateDefinitionFoundError and update the tests to match based on the exception type. | 16:37 |
| * pleia2 catches up on other translations discussions | 16:37 | |
| pelix | But unless we have another exception there that also sets the message such that it'll match '^Duplicate .*' it's definitely checking that it gets raised from the correct location | 16:38 |
| clarkb | dhellmann: doing the delete create via git cli requires force push perms which we don't hand out so it does require using the UI, but for one branch its not bad | 16:38 |
| dhellmann | clarkb: yeah, this would just be one branch and I don't know if they actually need it or not | 16:38 |
| dhellmann | clarkb: but knowing I can do it in the UI is good for now | 16:38 |
| *** mrmartin has joined #openstack-infra | 16:39 | |
| openstackgerrit | Merged openstack-infra/project-config: Merge upstream_translation scripts https://review.openstack.org/225966 | 16:39 |
| *** annegent_ has quit IRC | 16:40 | |
| *** _kiran_ has quit IRC | 16:40 | |
| *** kiran-r has joined #openstack-infra | 16:40 | |
| *** erikmwilson_ has joined #openstack-infra | 16:40 | |
| markus_z | dtroyer: Hi dean, do you think that a "bug-report cmd utility" would be in the scope of the python-openstackclient? https://review.openstack.org/#/c/209847/ | 16:40 |
| *** kiran-r has quit IRC | 16:40 | |
| *** kiranr_ has joined #openstack-infra | 16:40 | |
| *** bltavares has joined #openstack-infra | 16:40 | |
| *** kiranr_ has quit IRC | 16:40 | |
| *** erikmwilson_ is now known as erikwilson_ | 16:40 | |
| *** erikwilson has quit IRC | 16:41 | |
| markus_z | dtroyer: I'm currently looking for a central place where all projects (Nova, Cinder, Neutron, ...) could benefit from that. | 16:41 |
| openstackgerrit | Sergey Lukjanov proposed openstack-infra/project-config: Enable Ceilo plugin for Sahara dsvm jobs https://review.openstack.org/217751 | 16:41 |
| zaro | pelix: duplicate definition? i thought it was checking yaml structure? | 16:42 |
| *** annegentle has joined #openstack-infra | 16:42 | |
| openstackgerrit | Merged openstack-infra/project-config: Collapse two cases in upstream_translation_update https://review.openstack.org/226169 | 16:42 |
| *** annegent_ has joined #openstack-infra | 16:42 | |
| *** annegentle is now known as Guest36775 | 16:43 | |
| openstackgerrit | James E. Blair proposed openstack-infra/nodepool: Add more logging to tests and fakes https://review.openstack.org/226412 | 16:43 |
| openstackgerrit | James E. Blair proposed openstack-infra/nodepool: Bust cache in leaked node test https://review.openstack.org/226413 | 16:43 |
| jeblair | clarkb, greghaynes, mordred: ^ fixes a unit test flake we've been seeing | 16:43 |
| clarkb | jeblair: looking | 16:43 |
| clarkb | pleia2: thank you for approving that change I will rebase things soon and keep rolling the ball on version support | 16:44 |
| *** zns_ has quit IRC | 16:44 | |
| openstackgerrit | Merged openstack-infra/project-config: Fix docstring & args when fetching the Zanata XML https://review.openstack.org/225566 | 16:44 |
| *** zns has joined #openstack-infra | 16:44 | |
| mordred | jeblair: +2 from me | 16:46 |
| pleia2 | clarkb: great, thanks | 16:46 |
| *** david-lyle has quit IRC | 16:46 | |
| openstackgerrit | Merged openstack-infra/grafyaml: Remove Babel bits https://review.openstack.org/224855 | 16:46 |
| stevemar | markus_z: thats a bit of a weird one, i'm not sure osc is the right spot for it | 16:47 |
| jeblair | clarkb: also, when you have a sec; i know you looked at this yesterday -- do you have further thoughts on it? I checked the output and it lgtm. https://review.openstack.org/226109 | 16:47 |
| *** achanda has joined #openstack-infra | 16:47 | |
| *** Guest36775 has quit IRC | 16:48 | |
| clarkb | jeblair: I think I need to wrap my head around why using a config like that by default isn't doable | 16:48 |
| openstackgerrit | Merged openstack-infra/glean: Change ignore-errors to ignore_errors https://review.openstack.org/225712 | 16:48 |
| stevemar | markus_z: seems like it would fit better with oslo | 16:48 |
| clarkb | jeblair: or if it is doable go that route | 16:48 |
| dhellmann | clarkb: hmm, I either do not have permission to delete a branch or can't figure it out in the UI. I need to remove stable/liberty from neutron-client https://review.openstack.org/#/admin/projects/openstack/python-neutronclient,branches | 16:48 |
| clarkb | dhellmann: hrm, I thought release-managers could delete but maybe not | 16:48 |
| *** annegent_ has quit IRC | 16:48 | |
| *** ivar-lazzaro has quit IRC | 16:48 | |
| dhellmann | clarkb: I know I can create branches, but I don't think I've ever tried to remove one | 16:48 |
| clarkb | dhellmann: you don't have little check boxes with a grayed out Delete button below the branch list? | 16:48 |
| dhellmann | no, no check boxes or delete button | 16:49 |
| clarkb | dhellmann: you tick the check box then click delete, but if you don't have it I am happy to delete it for you | 16:49 |
| jeblair | clarkb: i refer you to literally weeks of discussions we all had with ianw when image logging was added. | 16:49 |
| openstackgerrit | Merged openstack-infra/grafyaml: Fix scopping error with panel validation https://review.openstack.org/225276 | 16:49 |
| jeblair | clarkb: the logging configuration must be created based on site specific information including actual image names | 16:49 |
| clarkb | jeblair: but not necessarily for the default built in logger | 16:49 |
| dhellmann | clarkb: I would appreciate it if you would remove it, I can recreate it. thanks! | 16:49 |
| clarkb | jeblair: and using a default built in logger doesn't preclude you from doing that | 16:50 |
| openstackgerrit | Merged openstack-infra/project-config: No longer read an existing zanata.xml https://review.openstack.org/226163 | 16:50 |
| clarkb | jeblair: I thought the trouble with ianw's thing was that ou didn't want puppet to be opinionated in that wa | 16:50 |
| *** yamahata has quit IRC | 16:50 | |
| clarkb | in this case its just nodepool deciding by default you don't need stevedore and iso8601 debug logs | 16:50 |
| jeblair | clarkb: no, i did not want _nodepool itself_ to have openstack's logging configuration. | 16:50 |
| *** cody-somerville has joined #openstack-infra | 16:50 | |
| clarkb | jeblair: and I am not saing it should, but couldn't it remove the extra debug logging by default and we would be good? | 16:51 |
| *** trown is now known as trown|lunch | 16:51 | |
| clarkb | dhellmann: done | 16:51 |
| dhellmann | clarkb: ty | 16:51 |
| jeblair | clarkb: i see the value in what you are suggesting, however, you are still suggesting a logging configuration that i would never expect anyone to use in production. i think that "no default; you must configure" is better than "oh, yeah, there's a default but you'll be unhappy if you use it". | 16:51 |
| pelix | zaro: which test are you looking at? thought it was tests/duplicates/fixtures/exception_projects001.yaml | 16:51 |
| *** ivar-lazzaro has joined #openstack-infra | 16:52 | |
| clarkb | jeblair: ok, I don't feel that strongly about it. I just thnik its odd that we would complain about the default logs and the fix ins't to change to default logs | 16:52 |
| markus_z | stevemar: OK, thanks for feedback. I'll talk to the oslo folks. | 16:52 |
| jeblair | clarkb: i would like to make major changes to logs, but just being able to read the devstack logs ranks higher on my list :| | 16:53 |
| *** markus_z has quit IRC | 16:54 | |
| *** Hal has quit IRC | 16:54 | |
| openstackgerrit | Merged openstack-infra/tripleo-ci: Allow a empty ci repository for puppet modules https://review.openstack.org/225630 | 16:54 |
| *** baoli has joined #openstack-infra | 16:55 | |
| *** HeOS has quit IRC | 16:55 | |
| *** fawadkhaliq has joined #openstack-infra | 16:56 | |
| *** xyang1 has quit IRC | 16:57 | |
| *** ybathia has joined #openstack-infra | 16:57 | |
| *** sergmelikyan has joined #openstack-infra | 16:57 | |
| *** xyang1 has joined #openstack-infra | 16:57 | |
| *** mrmartin has quit IRC | 16:58 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config: Disable translation copy https://review.openstack.org/226427 | 16:58 |
| clarkb | jeblair: in https://review.openstack.org/#/c/226412/1/nodepool/fakeprovider.py was it intentional to use dunderscores there? trying to determing if we need _Dummy__kind and _Dummy__kw | 16:59 |
| clarkb | I don't think it breaks anything ut trying to understand if that was a deliberate decision | 16:59 |
| jeblair | clarkb: yes, since we're also setting arbitrary attributes on the objects (since they stand in for *client things) | 17:00 |
| jeblair | clarkb: that way we don't collide with a potential novaclient.server._kind attribute | 17:01 |
| zaro | pelix: i'm talking about change https://review.openstack.org/#/c/185037 a different test with same name. | 17:01 |
| jeblair | clarkb: which probably doesn't exist, but hey. | 17:01 |
| clarkb | oh in the non diff portion of it | 17:01 |
| clarkb | gotcha | 17:01 |
| *** mrmartin has joined #openstack-infra | 17:02 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Use the get_auth function from occ https://review.openstack.org/221126 | 17:02 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Remove many redundant debug logs https://review.openstack.org/225365 | 17:02 |
| *** hashar has quit IRC | 17:02 | |
| *** sergmelikyan has quit IRC | 17:02 | |
| mordred | Shrews: ^^ fixed the error message (and also account for the fact that in normal operation occ returns None on some problems there) | 17:02 |
| *** amuller_afk is now known as amuller | 17:02 | |
| mordred | jeblair: ^^ I rebased on top of your debug log removal patch. it's such a doozy I'd like to land it so we're not rebase-hell-ing | 17:03 |
| *** sergmelikyan has joined #openstack-infra | 17:03 | |
| *** fedexo has joined #openstack-infra | 17:03 | |
| *** ZZelle_ has joined #openstack-infra | 17:03 | |
| jeblair | mordred: it was failing pep8+tests last i saw; you pushed ps3, was that a fix or rebase? | 17:04 |
| openstackgerrit | Bob Ball proposed openstack/diskimage-builder: Add support for Xen PV disks https://review.openstack.org/226249 | 17:04 |
| clarkb | ooh ^ is probably useful for us in rax | 17:05 |
| *** bpokorny has quit IRC | 17:05 | |
| *** derekh has quit IRC | 17:06 | |
| * BobBall grumbles | 17:06 | |
| *** bpokorny has joined #openstack-infra | 17:06 | |
| BobBall | I lost two days on that one | 17:06 |
| *** hemnafk is now known as hemna | 17:06 | |
| jeblair | mordred: NotFound: Floating ip pool not found. (HTTP 404) (Request-ID: req-d38bcbc4-3dbd-4566-866c-76428b54ccef) | 17:07 |
| jeblair | mordred: i think that's because nodepool still has the same rax+hpcloud floating ip logic we had in the launch script | 17:08 |
| *** davideagnello has quit IRC | 17:08 | |
| jeblair | mordred: so i believe that means that further work on ovh is stopped until we either move nodepool to shade or update nodepool with better floating-ip logic | 17:08 |
| *** mrmartin has quit IRC | 17:08 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config: Merge translation proposal scripts https://review.openstack.org/226398 | 17:08 |
| clarkb | BobBall: fwiw our images boot fine in rax without that | 17:09 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Trap exceptions in helper modules https://review.openstack.org/224713 | 17:09 |
| clarkb | BobBall: was that preventing boots for you? | 17:09 |
| BobBall | Yup | 17:09 |
| mordred | jeblair: fix/rebase yes | 17:09 |
| mordred | jeblair: and yes, I believ eyou are right - and I will write those patches next | 17:10 |
| *** davideagnello has joined #openstack-infra | 17:10 | |
| mordred | jeblair: because it turns out the logic to handle all the clouds is ... large | 17:10 |
| BobBall | You might not be getting PV disks if you don't have a kernel without support in? | 17:10 |
| openstackgerrit | Merged openstack-infra/nodepool: Wait for resources in devstack-plugin check https://review.openstack.org/225262 | 17:10 |
| *** puranamr has quit IRC | 17:10 | |
| pelix | zaro: looks like it's checking for YAMLStructureError as the exception raised. | 17:10 |
| *** ildikov has joined #openstack-infra | 17:11 | |
| BobBall | clarkb: My issue was with ironic though: /dev/xvda was being ignored so we couldn't open an iSCSI target on the host to push the root disk down | 17:11 |
| openstackgerrit | Merged openstack-infra/nodepool: devstack: supply a logging config file https://review.openstack.org/226109 | 17:11 |
| jeblair | mordred: er fix or rebase? | 17:11 |
| BobBall | clarkb: I guess booting in RAX may avoid the use of those functions if the initramfs doesn't need to get a handle on the disk itself? | 17:11 |
| pelix | So it could be anyone one of the 3 sources of that Exception, but it doesn't look like it could pass if a different one is thrown | 17:11 |
| *** kushal has quit IRC | 17:12 | |
| *** BharatK has quit IRC | 17:12 | |
| *** dimtruck is now known as zz_dimtruck | 17:12 | |
| *** dramalho has quit IRC | 17:12 | |
| mordred | jeblair: rebase+fix | 17:12 |
| *** abitha has joined #openstack-infra | 17:12 | |
| jeblair | k | 17:12 |
| *** sabeen2 has quit IRC | 17:13 | |
| *** cdent has quit IRC | 17:13 | |
| pelix | zaro: unless you want to do more careful checking on which particular YamlStructureError was being thrown? suggests either adding different patterns for the exception test files and adding a search to the 'ExpectedException' context manager | 17:14 |
| pelix | or creating more specific exceptions for the different types of structure errors | 17:14 |
| *** unicell has joined #openstack-infra | 17:14 | |
| clarkb | mordred: I thought OVH didn't do floating IPs | 17:15 |
| clarkb | runabove certainly didny | 17:15 |
| *** notnownikki has quit IRC | 17:15 | |
| jeblair | clarkb: they don't | 17:15 |
| jeblair | clarkb: that's the problem | 17:15 |
| pelix | zaro waynr: https://review.openstack.org/226331 should hopefully fix the odd random error seen in JJB around testing the cache storage | 17:16 |
| jeblair | clarkb: nodepool doesn't correctly detect that it doesn't need to assign a floating ip | 17:16 |
| *** notnownikki has joined #openstack-infra | 17:16 | |
| clarkb | ah | 17:16 |
| *** fedexo has quit IRC | 17:16 | |
| pelix | zaro waynr: and https://review.openstack.org/226107 helps associate logging with the errors that occurred | 17:16 |
| jeblair | clarkb: the logic is basically "if the extension exists, use it; otherwise, assume rackspace". we'd need to update nodepool to do "if it looks like it has a public ip via some heuristic, use it, otherwise, use floating ips" | 17:17 |
| pelix | when tests do fail it should help make debugging easier | 17:17 |
| *** Sukhdev has joined #openstack-infra | 17:17 | |
| *** tongli has joined #openstack-infra | 17:17 | |
| pelix | as well as also allowing for logging output to be checked | 17:17 |
| jeblair | clarkb: so it's a "test on 3 actual clouds" change to nodepool :/ | 17:17 |
| zaro | pelix: i guess i'm just suggesting that the only error in the test should be a yaml structure error. if it's not then the test should not fail when used with jjb. | 17:17 |
| *** achanda_ has joined #openstack-infra | 17:17 | |
| *** achanda has quit IRC | 17:17 | |
| *** kushal has joined #openstack-infra | 17:18 | |
| *** kushal has quit IRC | 17:18 | |
| *** kushal has joined #openstack-infra | 17:18 | |
| zaro | if the yaml structure were correct in the test it would still fail. | 17:18 |
| jeblair | greghaynes: what's the _gearman_worker_lock for in 197722? | 17:18 |
| clarkb | jeblair: I wonder if we can just compare against known private IP blocks | 17:18 |
| clarkb | that would break for internal clouds though :/ | 17:19 |
| clarkb | hrm | 17:19 |
| *** sridhar_ram has quit IRC | 17:19 | |
| pelix | zaro: yes, so what you mean that there is no need for the file name matching since the only tests should be ones that trigger the exception | 17:19 |
| zaro | pelix: how how can you tell if what failed is because of the yaml structure? | 17:19 |
| jeblair | clarkb: there's definitely something we can do; the question is, do we want to do it or let shade do it | 17:19 |
| openstackgerrit | Merged openstack-infra/project-config: Disable translation copy https://review.openstack.org/226427 | 17:19 |
| openstackgerrit | sebastian marcet proposed openstack-infra/puppet-openstackid: Added bower support https://review.openstack.org/226377 | 17:20 |
| pelix | zaro: any of the yaml files starting with 'exeception_*' are required to result in JJB throwing a with YAMLStructureError, otherwise it will result in a test case filter | 17:20 |
| *** jerryz has joined #openstack-infra | 17:20 | |
| *** dramalho has joined #openstack-infra | 17:20 | |
| pelix | if any other exception or no exception is thrown, the test will fail | 17:20 |
| clarkb | jeblair ya, probably best to use shade since we use it for image uploads | 17:20 |
| clarkb | so adding it to problematic areas first is already a thing | 17:21 |
| *** Sukhdev has quit IRC | 17:21 | |
| *** BobBall is now known as BobBall_AWOL | 17:22 | |
| *** rguillebert has quit IRC | 17:23 | |
| *** cody-somerville has quit IRC | 17:23 | |
| zaro | pelix: so i think the test case should have 2 outcomes, either YAMLStructureError or no exception. | 17:24 |
| *** diazjf has quit IRC | 17:25 | |
| *** mrmartin has joined #openstack-infra | 17:25 | |
| *** wenlock has joined #openstack-infra | 17:26 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config: Merge translation proposal scripts https://review.openstack.org/226398 | 17:26 |
| pelix | ok, so isn't the no exception side tested by all the other yaml files in yamlparser not throwing such an exception? | 17:26 |
| zaro | pelix: maybe other exception is fine but IMO would make it confusing to debug. | 17:26 |
| *** baoli has quit IRC | 17:27 | |
| openstackgerrit | Danilo Ramalho proposed openstack-infra/puppet-gerrit: Setup preconditions for acceptance tests. https://review.openstack.org/219400 | 17:27 |
| *** jordanP has quit IRC | 17:27 | |
| zaro | pelix: you have a point, but i'm not sure what the purpose is for testing other exceptions there. | 17:27 |
| mordred | clarkb, jeblair: ++ | 17:28 |
| mordred | jeblair: crappit. I missed a pep8 on the ugly patch | 17:28 |
| *** mrmartin has quit IRC | 17:28 | |
| *** shardy has quit IRC | 17:28 | |
| pelix | indeed, so until we need to test a 'no exception' case explicitly that is not caught elsewhere, we can remove the else and matching on the yaml filename | 17:28 |
| pelix | so within that test, the only thing that needs to be checked is that the yaml files read will result in JJB throwing YAMLStructureError? | 17:29 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Use the get_auth function from occ https://review.openstack.org/221126 | 17:30 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Remove many redundant debug logs https://review.openstack.org/225365 | 17:30 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Get defaults for image type from occ https://review.openstack.org/225507 | 17:30 |
| mordred | jeblair, Shrews: ^^ sorry for missing it the first time - I could have sworn I ran pep8 locally before pushing | 17:30 |
| *** dims has quit IRC | 17:31 | |
| *** puranamr has joined #openstack-infra | 17:31 | |
| *** Daisy has quit IRC | 17:31 | |
| *** dims has joined #openstack-infra | 17:32 | |
| *** Daisy has joined #openstack-infra | 17:32 | |
| openstackgerrit | Doug Hellmann proposed openstack/requirements: Update neutron and openstack client constraints for liberty https://review.openstack.org/226439 | 17:32 |
| *** kiran-r has joined #openstack-infra | 17:32 | |
| *** puranamr has quit IRC | 17:33 | |
| zaro | pelix: yeah, i think that's fine. maybe just add a little blurb to the test and commit msg that all other tests are already validating valid yaml structure. | 17:34 |
| *** kiranr has joined #openstack-infra | 17:35 | |
| openstackgerrit | Paul Belanger proposed openstack-infra/grafyaml: Add caching support https://review.openstack.org/226002 | 17:36 |
| *** claudiub has joined #openstack-infra | 17:36 | |
| *** Daisy has quit IRC | 17:36 | |
| pelix | zaro: that works for me :-) | 17:37 |
| pelix | no point in adding code where YAGNI applies | 17:37 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Remove many redundant debug logs https://review.openstack.org/225365 | 17:37 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Add more info to some exceptions https://review.openstack.org/225374 | 17:37 |
| *** puranamr has joined #openstack-infra | 17:37 | |
| mordred | jeblair: o m g. I have no idea why .. I really have fixed the exact same pep8 error in that file 4 times. I think I'm losing my mind | 17:38 |
| *** flepied has quit IRC | 17:38 | |
| jeblair | mordred: pep8 does this to you | 17:39 |
| anteaya | just to mordred? | 17:39 |
| *** cdoron has joined #openstack-infra | 17:39 | |
| *** cdoron has quit IRC | 17:39 | |
| *** monester has quit IRC | 17:39 | |
| mordred | anteaya: touché | 17:39 |
| *** Sree has quit IRC | 17:41 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/requirements: Bump python-openstackclient upper-constraint to 1.7.0 https://review.openstack.org/226444 | 17:42 |
| dhellmann | dims: see https://review.openstack.org/226439 | 17:42 |
| dims | ah ok | 17:42 |
| openstackgerrit | Merged openstack-infra/nodepool: Add more logging to tests and fakes https://review.openstack.org/226412 | 17:43 |
| dhellmann | dims: my script for auto-submitting them broke somehow so I did it by hand | 17:43 |
| openstackgerrit | Merged openstack-infra/nodepool: Bust cache in leaked node test https://review.openstack.org/226413 | 17:43 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/requirements: Bump os-client-config upper-constraint to 1.7.4 https://review.openstack.org/225363 | 17:43 |
| *** zz_dimtruck is now known as dimtruck | 17:44 | |
| openstackgerrit | Doug Hellmann proposed openstack-infra/release-tools: provide a better error message when a team is not found https://review.openstack.org/226051 | 17:48 |
| openstackgerrit | Doug Hellmann proposed openstack-infra/release-tools: Report all unreleased library changes https://review.openstack.org/191220 | 17:48 |
| openstackgerrit | Doug Hellmann proposed openstack-infra/release-tools: change Oslo to oslo https://review.openstack.org/226052 | 17:48 |
| openstackgerrit | Doug Hellmann proposed openstack-infra/release-tools: add a tool to compare the constraints file with existing releases https://review.openstack.org/225230 | 17:48 |
| openstackgerrit | Doug Hellmann proposed openstack-infra/release-tools: remove duplicate function title https://review.openstack.org/226018 | 17:48 |
| openstackgerrit | Doug Hellmann proposed openstack-infra/release-tools: add a consistent topic to all .gitreview updates https://review.openstack.org/226017 | 17:48 |
| *** sergmelikyan has quit IRC | 17:51 | |
| *** nadya has joined #openstack-infra | 17:51 | |
| *** trown|lunch is now known as trown | 17:52 | |
| *** mriedem has quit IRC | 17:52 | |
| *** mriedem1 has joined #openstack-infra | 17:53 | |
| *** sergmelikyan has joined #openstack-infra | 17:53 | |
| openstackgerrit | sebastian marcet proposed openstack-infra/openstackid: OIDC - OpenId Connect Implementation https://review.openstack.org/198458 | 17:54 |
| jeblair | greghaynes, clarkb, jhesketh: i've reviewed much of the gearman dib stack; i have some questions and a few things that probably amount to -1s, but need more context first. | 17:55 |
| *** sergmelikyan has quit IRC | 17:58 | |
| *** andersonvom has joined #openstack-infra | 17:58 | |
| *** fawadkhaliq has quit IRC | 17:58 | |
| *** sergmelikyan has joined #openstack-infra | 17:59 | |
| *** andrey-mp has joined #openstack-infra | 17:59 | |
| *** sergmelikyan has quit IRC | 18:00 | |
| openstackgerrit | Doug Hellmann proposed openstack/requirements: update designateclient to 1.5.0 https://review.openstack.org/226465 | 18:00 |
| *** tiswanso has quit IRC | 18:00 | |
| *** tiswanso has joined #openstack-infra | 18:01 | |
| *** adduarte has joined #openstack-infra | 18:02 | |
| *** fitoduarte has quit IRC | 18:03 | |
| waynr | pelix: by the way, I've been experimenting with the use of jinja2 templates to generate groovy scripts to run on Jenkins (for example, to delete jobs that match some given regex) similar to the new delete_all_jobs | 18:04 |
| gordc | random question, is there a place to check status of gate (ie. any known active issues) | 18:04 |
| waynr | i'm definitely in favor of a move to jinja2 templates for JJB :) | 18:04 |
| openstackgerrit | James E. Blair proposed openstack-infra/nodepool: Use OCC cloud specifier in devstack plugin https://review.openstack.org/208114 | 18:04 |
| *** arxcruz has quit IRC | 18:04 | |
| jeblair | clarkb: you may want to WIP that; i rebased/fixed conflict. | 18:05 |
| jeblair | clarkb: (i really wanted to see the results) | 18:05 |
| openstackgerrit | Bruno Tavares proposed openstack-infra/puppet-httpd: Fix proxy and redirect configuration file names. https://review.openstack.org/225331 | 18:05 |
| clarkb | jeblair ok | 18:05 |
| openstackgerrit | Bruno Tavares proposed openstack-infra/puppet-httpd: Fix httpd::dev. https://review.openstack.org/222694 | 18:05 |
| openstackgerrit | Bruno Tavares proposed openstack-infra/puppet-httpd: Add note of support on Python mods for CentOS 7. https://review.openstack.org/224098 | 18:05 |
| openstackgerrit | Bruno Tavares proposed openstack-infra/puppet-httpd: Add acceptance tests for puppet-httpd https://review.openstack.org/222721 | 18:05 |
| openstackgerrit | Bruno Tavares proposed openstack-infra/puppet-httpd: Use httpd::mod when adding Apache modules. https://review.openstack.org/223578 | 18:05 |
| openstackgerrit | Bruno Tavares proposed openstack-infra/puppet-httpd: Fix httpd::python. https://review.openstack.org/226474 | 18:05 |
| clarkb | jeblair and ya seeing results is good | 18:05 |
| *** diazjf has joined #openstack-infra | 18:06 | |
| *** shashank_hegde has quit IRC | 18:06 | |
| clarkb | jeblair I think the issue is devstack writes the file where stack user doesnt find it by default | 18:06 |
| clarkb | which is argyably a devstack bug if that is why it breaks | 18:06 |
| *** esmute has joined #openstack-infra | 18:07 | |
| tristanC | gordc: http://status.openstack.org/elastic-recheck/ ? | 18:08 |
| jeblair | clarkb: hrm; the last output i saw looked like it might actually be working but timed out. | 18:08 |
| pelix | waynr: sounds powerful, though we may need to make it configurable between using the script console and explicitly calling delete on jobs since not all jenkins admins may want to grant access to users to use the console but may allow them to manage jobs via JJB | 18:08 |
| gordc | tristanC: cool stuff. thanks! | 18:08 |
| openstackgerrit | Paul Michali proposed openstack-infra/project-config: Move VPN Functional tests to check queue https://review.openstack.org/223049 | 18:08 |
| jeblair | clarkb: the devstack plugin test stuff has merged, so new results should be clearer | 18:08 |
| andrey-mp | hello all. this commit (https://review.openstack.org/#/c/225424/4) breaks creation of user rc file. now devstack doesn't create certificate for user and doesn't react to changes of EC2/S3 urls if they recreated by plugins | 18:09 |
| andrey-mp | so this commit totally broke ec2-api gating | 18:09 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config: Merge translation proposal scripts https://review.openstack.org/226398 | 18:09 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config: Translations: Refaktor https://review.openstack.org/226477 | 18:09 |
| *** puranamr has quit IRC | 18:09 | |
| *** kgiusti has left #openstack-infra | 18:10 | |
| andrey-mp | what I can do with this commit? | 18:10 |
| *** amitgandhinz has quit IRC | 18:10 | |
| *** HeOS has joined #openstack-infra | 18:11 | |
| waynr | pelix: ah, i hadn't thought about that--will keep in mind whenever I am ready to port this behavior into JJB (it is currently part of a standalone series of Jenkins management scripts I have been working on using python-jenkins) | 18:11 |
| *** ihrachys has quit IRC | 18:11 | |
| dmsimard | Hey, this is no big deal but the github repository description of https://github.com/openstack/python-neutronclient still mentions Quantum :) | 18:12 |
| *** Sukhdev has joined #openstack-infra | 18:12 | |
| AJaeger | pleia2: I missed a line in https://review.openstack.org/226398, hope it's fine now - noticed during work on https://review.openstack.org/226477 | 18:12 |
| pleia2 | AJaeger: whoops, thanks | 18:12 |
| pleia2 | AJaeger: which file? | 18:12 |
| pleia2 | nm, propose_translation_update.sh | 18:13 |
| AJaeger | pleia2: https://review.openstack.org/#/c/226398/3..4/jenkins/scripts/propose_translation_update.sh | 18:13 |
| pleia2 | not surprised I missed that | 18:13 |
| mordred | dmsimard: yeah, we don't pay much attention to those. I have fix that one. thanks! | 18:13 |
| AJaeger | pleia2: I wouldn't have noticed if I wouldn't have done the refactor patch ;( | 18:13 |
| *** ivar-lazzaro has quit IRC | 18:14 | |
| dmsimard | mordred: np :) | 18:14 |
| *** kgiusti has joined #openstack-infra | 18:15 | |
| openstackgerrit | Paul Belanger proposed openstack-infra/nodepool: Add dib success or failues to statsd https://review.openstack.org/226481 | 18:15 |
| *** ivar-lazzaro has joined #openstack-infra | 18:15 | |
| *** ociuhandu has quit IRC | 18:16 | |
| *** krtaylor has quit IRC | 18:16 | |
| *** otter768 has joined #openstack-infra | 18:16 | |
| *** sabeen1 has joined #openstack-infra | 18:18 | |
| *** sergmelikyan has joined #openstack-infra | 18:18 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/nodepool: Add comment field to node and reason to hold https://review.openstack.org/225428 | 18:19 |
| *** shashank_hegde has joined #openstack-infra | 18:20 | |
| *** otter768 has quit IRC | 18:22 | |
| openstackgerrit | Sean Dague proposed openstack-infra/project-config: make nova v2.0 compat jobs voting https://review.openstack.org/222573 | 18:23 |
| *** achanda has joined #openstack-infra | 18:23 | |
| clarkb | mordred maybe you wsnt to add a cli test for list:) | 18:23 |
| *** achanda_ has quit IRC | 18:24 | |
| andrey-mp | hm, i'm invisible! | 18:24 |
| *** andrey-mp has quit IRC | 18:24 | |
| clarkb | jeblair https://review.openstack.org/#/c/195175/19 aiui yes, since commands need to instantiate nodepool to get the config (we can make that better in general too) | 18:26 |
| openstackgerrit | Monty Taylor proposed openstack-infra/nodepool: Add tests for shade cloud config parameter https://review.openstack.org/169339 | 18:26 |
| *** maiteb has joined #openstack-infra | 18:27 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config: Translations: Merge pull_from_zanata https://review.openstack.org/226485 | 18:28 |
| *** sergmelikyan has quit IRC | 18:29 | |
| *** sabeen3 has joined #openstack-infra | 18:29 | |
| *** kiran-r has quit IRC | 18:29 | |
| *** krtaylor has joined #openstack-infra | 18:30 | |
| *** sergmelikyan has joined #openstack-infra | 18:31 | |
| *** sabeen1 has quit IRC | 18:32 | |
| *** hashar has joined #openstack-infra | 18:32 | |
| *** Piet has quit IRC | 18:34 | |
| ccarrara | hi guys. I'm trying to submit a review to openstack/openstack-health, but I'm getting an error telling that I need to approve ICLA, what I already did. Anyone could help me? | 18:35 |
| *** pvaneck has joined #openstack-infra | 18:35 | |
| clarkb | jeblair responded to your comments on the child of the earlier change | 18:35 |
| clarkb | ccarrara can you paste the output of git review to a paste service and link it here? | 18:35 |
| *** Sukhdev has quit IRC | 18:35 | |
| *** jsavak has quit IRC | 18:36 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Use the get_auth function from occ https://review.openstack.org/221126 | 18:36 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Get defaults for image type from occ https://review.openstack.org/225507 | 18:37 |
| ccarrara | clarkb: http://cdn.pasteraw.com/pem6yfui2ey1hzjimxnxgfh6drryp59 | 18:37 |
| clarkb | ccarrara https://ask.openstack.org/en/question/56720/cannot-store-contact-information-when-updating-info-in-openstack-gerrit/ see if that gets it sorted out | 18:38 |
| clarkb | if not let us know | 18:38 |
| *** mrmartin has joined #openstack-infra | 18:38 | |
| *** jsavak has joined #openstack-infra | 18:38 | |
| openstackgerrit | Eric Harney proposed openstack-infra/project-config: Add ceph to os-brick check jobs https://review.openstack.org/223257 | 18:39 |
| *** ayoung has quit IRC | 18:39 | |
| clarkb | the key part of that error message us that contact info needs to be updated | 18:39 |
| *** moravec has quit IRC | 18:40 | |
| *** achanda has quit IRC | 18:42 | |
| mrmartin | hi | 18:42 |
| ccarrara | clarkb: thank you! This worked gracefully. :) | 18:43 |
| mrmartin | fungi, clarkb: what's the plan with the vcsrepo detached head refresh bug? We have three options, each required some care from our side. | 18:43 |
| openstackgerrit | Elizabeth K. Joseph proposed openstack-infra/infra-specs: Move Zanata from priority to implemented spec https://review.openstack.org/225945 | 18:43 |
| openstackgerrit | Michael Krotscheck proposed openstack-infra/project-config: Added npm publish job. https://review.openstack.org/199715 | 18:43 |
| openstackgerrit | Michael Krotscheck proposed openstack-infra/project-config: Added npm-publish-tarball https://review.openstack.org/212153 | 18:43 |
| clarkb | I would just update the ruby to not refresh if HEAD does not change | 18:44 |
| *** moravec has joined #openstack-infra | 18:44 | |
| *** sergmelikyan has quit IRC | 18:44 | |
| mrmartin | fungi, clarkb: I can backport the fix if we decide to skip switching to upstream puppet vcsrepo | 18:44 |
| clarkb | well its less about skipping and more that doing that will take time and be slow | 18:45 |
| clarkb | so do the quick fix now and ibdependently work on switching | 18:45 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Handle glance v1 and v2 difference with is_public https://review.openstack.org/226492 | 18:46 |
| mrmartin | each option have different risks. the puppet-git repo seems promising, but not ready, backport takes time. What are the risks of switching to upstream puppet? | 18:46 |
| clarkb | mrmartin that all of our git will stop working | 18:46 |
| clarkb | and we use it everywhere | 18:46 |
| clarkb | thats a huge risk | 18:47 |
| mrmartin | the same is true for backport :) | 18:47 |
| clarkb | the surface area is mich smaller | 18:47 |
| *** moravec1 has joined #openstack-infra | 18:47 | |
| clarkb | so its risky but less so | 18:47 |
| *** achanda has joined #openstack-infra | 18:48 | |
| mordred | jeblair, Shrews: that shade patch is instead of https://review.openstack.org/#/c/225344/ | 18:48 |
| mrmartin | the less risky option is to finish puppet-git and we can deploy it next to our existing modules, without harming existing infrastructure. | 18:48 |
| *** sergmelikyan has joined #openstack-infra | 18:48 | |
| mordred | clarkb: you too | 18:48 |
| *** moravec has quit IRC | 18:48 | |
| mrmartin | the changeset for upstream backport is quite large. | 18:49 |
| openstackgerrit | Bruno Tavares proposed openstack-infra/puppet-gerrit: Setup preconditions for acceptance tests. https://review.openstack.org/219400 | 18:49 |
| fungi | mrmartin: the even less risky option was clarkb's original suggestion, work around it in the askbot module by using onlyif directives | 18:49 |
| *** Sukhdev has joined #openstack-infra | 18:49 | |
| mrmartin | replace vcsrepo with a git exec ? | 18:50 |
| *** kushal has quit IRC | 18:50 | |
| openstackgerrit | Mike Perez proposed openstack/requirements: Bumping os-brick to 0.4.1 https://review.openstack.org/226500 | 18:51 |
| *** sridhar_ram has joined #openstack-infra | 18:52 | |
| *** moravec1 has quit IRC | 18:52 | |
| clarkb | ya only if sha1 doesnt match update the repo | 18:53 |
| *** ociuhandu has joined #openstack-infra | 18:53 | |
| *** moravec has joined #openstack-infra | 18:53 | |
| *** angela-s has joined #openstack-infra | 18:53 | |
| fungi | mrmartin: i didn't read it as suggesting to replace the vcsrepo resource, just have an onlyif in the existing exec to check whether the git sha changed http://eavesdrop.openstack.org/irclogs/%23openstack-infra/%23openstack-infra.2015-09-11.log.html#t2015-09-11T14:41:21 | 18:54 |
| *** ayoung has joined #openstack-infra | 18:54 | |
| mrmartin | oh, so update all of the depending execs, so refresh trigger won't launch them if the sha is the same. | 18:54 |
| fungi | that's what it seems like he suggested, yes | 18:55 |
| clarkb | I would have to reread thibgs to remember | 18:55 |
| fungi | and also, it's almost meeting time | 18:56 |
| mrmartin | ok. I'm still fighting with openstackid integration of the askbot, the openid implementation seems to be hell broken, we need this vcsrepo when the openid integration works well. | 18:56 |
| openstackgerrit | Bruno Tavares proposed openstack-infra/puppet-httpd: Fix proxy and redirect configuration file names. https://review.openstack.org/225331 | 18:57 |
| openstackgerrit | Bruno Tavares proposed openstack-infra/puppet-httpd: Add note of support on Python mods for CentOS 7. https://review.openstack.org/224098 | 18:57 |
| openstackgerrit | Bruno Tavares proposed openstack-infra/puppet-httpd: Add acceptance tests for puppet-httpd https://review.openstack.org/222721 | 18:57 |
| openstackgerrit | Bruno Tavares proposed openstack-infra/puppet-httpd: Use httpd::mod when adding Apache modules. https://review.openstack.org/223578 | 18:57 |
| openstackgerrit | Bruno Tavares proposed openstack-infra/puppet-httpd: Fix httpd::python. https://review.openstack.org/226474 | 18:57 |
| lifeless | fungi: (btw, you migh twant to add openstack-relmgr-office to your channels) | 18:57 |
| *** Swami has joined #openstack-infra | 18:57 | |
| lifeless | fungi: when does juno get EOL'd ? | 18:57 |
| fungi | lifeless: thanks, adding and i would need to check the etherpad from vancouver to see if we decided. usually any time after the last scheduled point release is fair game though | 18:58 |
| openstackgerrit | Bruno Tavares proposed openstack-infra/puppet-elasticsearch: Add acceptance tests for puppet-elasticsearch. https://review.openstack.org/224301 | 18:59 |
| lifeless | so now then ? :) | 18:59 |
| fungi | lifeless: i'd be cool with that, but will check what available record we may have after the infra meeting concludes | 19:00 |
| pleia2 | meeting time | 19:00 |
| lifeless | just thinking we could get rid of a bunch of maintenance pain | 19:01 |
| *** tiswanso has quit IRC | 19:01 | |
| fungi | yep, for infra too | 19:01 |
| *** hodos has quit IRC | 19:02 | |
| *** hodos has joined #openstack-infra | 19:02 | |
| *** cloudtra_ has joined #openstack-infra | 19:03 | |
| *** tiswanso has joined #openstack-infra | 19:04 | |
| *** cloudtrainme has quit IRC | 19:06 | |
| *** sergmelikyan has quit IRC | 19:07 | |
| *** andersonvom has quit IRC | 19:07 | |
| *** andersonvom has joined #openstack-infra | 19:08 | |
| *** jerryz has quit IRC | 19:08 | |
| *** earlephilhower has joined #openstack-infra | 19:09 | |
| *** Sukhdev has quit IRC | 19:09 | |
| *** e0ne has joined #openstack-infra | 19:10 | |
| *** andersonvom has quit IRC | 19:10 | |
| *** Sukhdev has joined #openstack-infra | 19:10 | |
| *** cloudtra_ has quit IRC | 19:11 | |
| *** Sukhdev has quit IRC | 19:12 | |
| *** Sukhdev has joined #openstack-infra | 19:12 | |
| openstackgerrit | Bruno Tavares proposed openstack-infra/puppet-logstash: Add acceptance tests for puppet-logstash. https://review.openstack.org/224858 | 19:13 |
| openstackgerrit | Bruno Tavares proposed openstack-infra/puppet-logstash: Fix puppet manifests to be able to apply. https://review.openstack.org/226041 | 19:13 |
| *** DericHorn-HP has joined #openstack-infra | 19:14 | |
| *** shashank_hegde has quit IRC | 19:14 | |
| *** Rockyg has joined #openstack-infra | 19:15 | |
| *** cloudtrainme has joined #openstack-infra | 19:16 | |
| *** kushal has joined #openstack-infra | 19:16 | |
| *** pelix has quit IRC | 19:17 | |
| openstackgerrit | Merged openstack-infra/project-config: Set docimpact group for VMWare NSX https://review.openstack.org/225529 | 19:17 |
| openstackgerrit | Merged openstack-infra/puppet-httpd: Add logrotate class to Apache https://review.openstack.org/206582 | 19:17 |
| *** sridhar_ram has quit IRC | 19:18 | |
| *** Sukhdev has quit IRC | 19:18 | |
| openstackgerrit | Paul Michali proposed openstack-infra/project-config: Move VPN Functional tests to check queue https://review.openstack.org/223049 | 19:19 |
| openstackgerrit | Mike Perez proposed openstack/requirements: Bumping os-brick to 0.5.0 https://review.openstack.org/226500 | 19:19 |
| *** bharathm has quit IRC | 19:20 | |
| *** gyee has quit IRC | 19:20 | |
| *** angdraug has quit IRC | 19:24 | |
| *** dangers_away is now known as dangers | 19:26 | |
| *** Sukhdev has joined #openstack-infra | 19:27 | |
| *** yolanda has joined #openstack-infra | 19:27 | |
| openstackgerrit | Bruno Tavares proposed openstack-infra/puppet-graphite: Add acceptance tests for puppet-graphite. https://review.openstack.org/224252 | 19:27 |
| *** puranamr has joined #openstack-infra | 19:27 | |
| *** puranamr has quit IRC | 19:28 | |
| openstackgerrit | Doug Hellmann proposed openstack/requirements: Update os-brick for new release 0.5.0 https://review.openstack.org/226514 | 19:28 |
| *** bdemers has quit IRC | 19:29 | |
| *** yolanda has quit IRC | 19:29 | |
| *** matbu has quit IRC | 19:29 | |
| openstackgerrit | Doug Hellmann proposed openstack/requirements: Update os-brick for new release 0.5.0 https://review.openstack.org/226514 | 19:29 |
| *** yolanda has joined #openstack-infra | 19:29 | |
| *** sridhar_ram has joined #openstack-infra | 19:29 | |
| *** bdemers has joined #openstack-infra | 19:30 | |
| *** wshao has quit IRC | 19:30 | |
| *** matbu has joined #openstack-infra | 19:31 | |
| *** yolanda has quit IRC | 19:31 | |
| *** yolanda has joined #openstack-infra | 19:32 | |
| *** rossella_s has quit IRC | 19:34 | |
| AJaeger | clarkb: thanks for the reviews, will update... | 19:34 |
| *** Sukhdev has quit IRC | 19:35 | |
| *** kushal has quit IRC | 19:35 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config: Translations: Merge pull_from_zanata https://review.openstack.org/226485 | 19:36 |
| openstackgerrit | Bruno Tavares proposed openstack-infra/puppet-logstash: Add acceptance tests for puppet-logstash. https://review.openstack.org/224858 | 19:36 |
| openstackgerrit | Bruno Tavares proposed openstack-infra/puppet-logstash: Fix puppet manifests to be able to apply. https://review.openstack.org/226041 | 19:36 |
| *** jsavak has quit IRC | 19:36 | |
| *** jsavak has joined #openstack-infra | 19:37 | |
| *** achanda has quit IRC | 19:37 | |
| openstackgerrit | Doug Hellmann proposed openstack-infra/release-tools: handle unicode characters in commit messages https://review.openstack.org/226518 | 19:38 |
| openstackgerrit | Doug Hellmann proposed openstack-infra/release-tools: show the command used to build the release notes https://review.openstack.org/226519 | 19:38 |
| *** amitgandhinz has joined #openstack-infra | 19:40 | |
| *** yolanda has quit IRC | 19:41 | |
| *** kushal has joined #openstack-infra | 19:41 | |
| *** yolanda has joined #openstack-infra | 19:42 | |
| *** ddieterl_ has quit IRC | 19:43 | |
| *** ddieterly has joined #openstack-infra | 19:44 | |
| *** jsavak has quit IRC | 19:44 | |
| *** jsavak has joined #openstack-infra | 19:44 | |
| *** flepied has joined #openstack-infra | 19:45 | |
| *** ddieterl_ has joined #openstack-infra | 19:45 | |
| *** bradjones has joined #openstack-infra | 19:46 | |
| *** bradjones has quit IRC | 19:46 | |
| *** bradjones has joined #openstack-infra | 19:46 | |
| *** yolanda has quit IRC | 19:46 | |
| asselin | yolanda, congrats! perhaps you'd like a s imple patch to merge! https://review.openstack.org/#/c/218462/ | 19:46 |
| *** yolanda has joined #openstack-infra | 19:48 | |
| *** ddieterly has quit IRC | 19:48 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/requirements: Bump os-client-config upper-constraint to 1.7.5 https://review.openstack.org/225363 | 19:48 |
| *** Swami has quit IRC | 19:49 | |
| dims | lifeless: dhellmann: can you please bless this again? ^^^ | 19:49 |
| *** Sukhdev has joined #openstack-infra | 19:49 | |
| *** Sukhdev has quit IRC | 19:50 | |
| *** yolanda has quit IRC | 19:50 | |
| *** cody-somerville has joined #openstack-infra | 19:50 | |
| *** cody-somerville has quit IRC | 19:50 | |
| *** cody-somerville has joined #openstack-infra | 19:50 | |
| anteaya | asselin: the pile on begins | 19:50 |
| dhellmann | dims: +2 | 19:50 |
| asselin | anteaya, yeah..will find something to congratulate nibalizer with too! :) | 19:50 |
| *** rossella_s has joined #openstack-infra | 19:51 | |
| nibalizer | hah | 19:51 |
| openstackgerrit | Bruno Tavares proposed openstack-infra/puppet-elasticsearch: Add acceptance tests for puppet-elasticsearch. https://review.openstack.org/224301 | 19:51 |
| anteaya | asselin: ha ha ha | 19:51 |
| dims | thanks dhellmann | 19:52 |
| *** dprince has quit IRC | 19:53 | |
| *** yolanda has joined #openstack-infra | 19:53 | |
| *** nadya has quit IRC | 19:55 | |
| openstackgerrit | matthew wagoner proposed openstack-infra/gitdm: Update group membership for olaph https://review.openstack.org/226525 | 19:55 |
| *** duartea has joined #openstack-infra | 19:56 | |
| lifeless | dims: no point until the depends-on are landed | 19:57 |
| lifeless | dims: (AIUI) | 19:57 |
| *** shashank_hegde has joined #openstack-infra | 19:57 | |
| *** bharathm has joined #openstack-infra | 19:58 | |
| openstackgerrit | Spencer Krum proposed openstack-infra/system-config: Add nibalizer to infra-root https://review.openstack.org/226526 | 19:58 |
| *** sergmelikyan has joined #openstack-infra | 19:59 | |
| *** adduarte has quit IRC | 19:59 | |
| *** yolanda_ has joined #openstack-infra | 20:01 | |
| ianw | jeblair / fungi / clarkb : we didn't get to it, but can we please add fedora22 nodes? https://review.openstack.org/#/c/186619/ <- we're ready to support it and start transitions | 20:01 |
| clarkb | AJaeger: ok reviewing the big one now that the meeting is over | 20:01 |
| *** eantyshev2 has joined #openstack-infra | 20:01 | |
| clarkb | ianw: might be good to check with jeblair and make sure that doesn't interfere with the ovh work somehow (I don't expect it to) | 20:01 |
| *** DericHorn-HP has quit IRC | 20:02 | |
| clarkb | otherwise I am happy to review and +2/approve assuming the config is good | 20:02 |
| *** annegentle has joined #openstack-infra | 20:02 | |
| AJaeger | clarkb: thanks. | 20:02 |
| dims | lifeless: the devstack changes are in, the python-osc has a +W | 20:02 |
| mmedvede | anteaya: our CI also tried to use RHEL-based distro with devstack-gate, but had to manually change d-g to make it work | 20:02 |
| anteaya | mmedvede: hello, thank you for speaking up | 20:02 |
| clarkb | ianw: any reason to not put it in rax as well? | 20:02 |
| *** yolanda_ has quit IRC | 20:03 | |
| *** jamielennox|away is now known as jamielennox | 20:03 | |
| ianw | clarkb: is dib to rax ok? | 20:03 |
| eantyshev2 | anteaya: we had the conversation in a third-party meeting, and some people admitted they had to inline patch the devstack-gate | 20:03 |
| anteaya | eantyshev2: there you are thank you | 20:03 |
| * asselin is one of those | 20:03 | |
| clarkb | mordred: ^ should new fedora's be using some rpm minimal install now or are you ok with using the old fedora element? | 20:03 |
| *** dkranz has quit IRC | 20:03 | |
| openstackgerrit | Bruno Tavares proposed openstack-infra/puppet-logstash: Add acceptance tests for puppet-logstash. https://review.openstack.org/224858 | 20:03 |
| clarkb | ianw: ya, we haven't across the board yet because it requires glean, so my question to mordred is related to using rax | 20:03 |
| anteaya | asselin: ah thank you for speaking up | 20:03 |
| clarkb | ianw: you can't use the base fedora element to do it iirc, ou have to use some minimal thing then add in glean | 20:03 |
| eantyshev2 | anteaya: and this is one of the cases when inline patching could be avoided | 20:03 |
| mordred | clarkb: they should use a fedora-minimal element | 20:03 |
| *** yolanda__ has joined #openstack-infra | 20:04 | |
| mordred | clarkb: the should definitely not use the old fedora element | 20:04 |
| *** yolanda_ has joined #openstack-infra | 20:04 | |
| clarkb | ianw: ^ not sure if that is just terrible or not | 20:04 |
| *** yolanda__ has quit IRC | 20:04 | |
| *** DericHorn-HP has joined #openstack-infra | 20:04 | |
| anteaya | eantyshev2: well glad to know others are affected, I had not known anyone except you was affected by this | 20:05 |
| clarkb | ianw: I think you just s/fedora/fedora-minimal/ and add in simple-init to the elements list to set it up that way then you can rax | 20:08 |
| yolanda_ | hi, so i arrived a bit late to the meeting, just arriving home from the airport. But i saw the announcement of my membership on twitter. So i'm very happy for that, and i'm willing to do a great work on this role, thanks for the confidence! | 20:08 |
| *** bdemers has quit IRC | 20:09 | |
| *** bdemers has joined #openstack-infra | 20:09 | |
| pleia2 | yolanda_: congrats :) | 20:09 |
| clarkb | so I guess the real question is does simple-init work on fedora | 20:10 |
| clarkb | TheJulia: ^ you were doing centos7 at one point I think? | 20:10 |
| eantyshev2 | ianw, I'll vote +1 on 224989 as soon as I check you solution against our distro, just in case | 20:10 |
| TheJulia | clarkb: last time I tried fedora it was working | 20:10 |
| openstackgerrit | Bruno Tavares proposed openstack-infra/puppet-lodgeit: Add acceptance tests for puppet-lodgeit. https://review.openstack.org/225279 | 20:10 |
| clarkb | TheJulia: woot thanks | 20:11 |
| TheJulia | clarkb: your milage, may vary, of course :) | 20:11 |
| *** eantyshev2 has quit IRC | 20:12 | |
| ianw | clarkb: ok, i'll look at that ... | 20:12 |
| *** sergmelikyan has quit IRC | 20:13 | |
| *** sergmelikyan has joined #openstack-infra | 20:13 | |
| greghaynes | dib gates on fedora working | 20:13 |
| greghaynes | (i am not actually here) | 20:14 |
| ianw | mordred: so that change to move the credential writing earlier broke things, because the "create_userrc" actually depends on swift for s3 certificates, etc | 20:14 |
| greghaynes | oh, simple-init + fedora, we dont gate on that nor are there many users of that ATM | 20:15 |
| ianw | mordred: https://review.openstack.org/#/c/226484/2 | 20:15 |
| *** achanda has joined #openstack-infra | 20:15 | |
| mordred | ianw: thanks - I'll go dig in to that then | 20:15 |
| ianw | mordred: i think that change above will fix it | 20:16 |
| mordred | ianw: k. works for me | 20:17 |
| clarkb | greghaynes: ya it would be required for us to rax though | 20:17 |
| greghaynes | yep | 20:17 |
| *** gordc has quit IRC | 20:17 | |
| mordred | ianw: it is, after all, the thing I was mostly concerned with | 20:17 |
| *** otter768 has joined #openstack-infra | 20:17 | |
| greghaynes | clarkb: ubuntu-trusty is still just waiting on more reliable uploading? | 20:18 |
| *** cody-somerville has quit IRC | 20:18 | |
| *** claudiub has quit IRC | 20:19 | |
| clarkb | greghaynes: ya I havne't really looked at it recently | 20:19 |
| clarkb | the zanata stuff has been time sensitive so has been priority for me | 20:19 |
| greghaynes | yea | 20:19 |
| clarkb | speaking of I have more reviews to do on that now and changes to rebase | 20:20 |
| asselin | anyone seen this before? OpenStackCloudException: Image creation failed: 403 Forbidden: Attribute 'is_public' is reserved. (HTTP 403) | 20:20 |
| asselin | http://paste.openstack.org/show/473606/ | 20:20 |
| clarkb | asselin: yes mordred is working on it right now | 20:20 |
| clarkb | its funny that is_public as an attribute is forbidden even when you set it to false | 20:20 |
| asselin | clarkb, great, thanks! | 20:20 |
| asselin | clarkb, interesting :) | 20:21 |
| *** sergmelikyan has quit IRC | 20:21 | |
| mordred | clarkb: glanceclient v2 has strict schema enforcement | 20:21 |
| mordred | clarkb: and yes | 20:21 |
| greghaynes | clarkb: opencloud - where your images must be public | 20:21 |
| *** otter768 has quit IRC | 20:22 | |
| *** adduarte has joined #openstack-infra | 20:22 | |
| openstackgerrit | Bruno Tavares proposed openstack-infra/puppet-diskimage_builder: Add acceptance tests for puppet-diskimage_builder. https://review.openstack.org/220262 | 20:23 |
| clarkb | mordred: then its strict broken enforcement | 20:23 |
| clarkb | mordred: why have a boolean value if only one is valid? | 20:23 |
| mordred | clarkb: oh. no - the key is invalid | 20:23 |
| mordred | clarkb: is_public is not part of the glance v2 api | 20:23 |
| clarkb | mordred: oh I thought it was valid | 20:23 |
| clarkb | mordred: and it depends on policy | 20:23 |
| mordred | it is in glance v1 | 20:23 |
| clarkb | mordred: how do you upload a public image then? | 20:23 |
| mordred | I misunderstood the error the first time | 20:23 |
| clarkb | oh | 20:23 |
| mordred | clarkb: visibility=public | 20:24 |
| mordred | in v2 | 20:24 |
| clarkb | I see | 20:24 |
| *** yolanda has quit IRC | 20:25 | |
| *** yolanda_ is now known as yolanda | 20:25 | |
| *** duartea has quit IRC | 20:25 | |
| clarkb | mordred: so shade's api should probably be pass in visibility, if that matches public rewrite to be is_public=True for v1 else False | 20:27 |
| clarkb | and I am sure if I look for it I will find it | 20:27 |
| clarkb | where it is the change that does this | 20:27 |
| mordred | clarkb: https://review.openstack.org/#/c/226492/ | 20:27 |
| *** wshao has joined #openstack-infra | 20:28 | |
| mordred | clarkb: it keeps visibility as a passthrough, but translates is_public to visibility if you pass it in | 20:28 |
| clarkb | mordred: should we also translate the other way for v1 | 20:28 |
| clarkb | mordred: I think so otherwise your consumption of shade can't be the same for multiple clouds | 20:28 |
| clarkb | dpeending on how you wrote it and end up getting confused | 20:29 |
| mordred | clarkb: ++ | 20:29 |
| clarkb | I suppose if you rtfs then you would discover this and find you can is_public everwhere | 20:29 |
| openstackgerrit | Merged openstack-infra/project-config: Merge translation proposal scripts https://review.openstack.org/226398 | 20:29 |
| mordred | clarkb: I was more aiming to unbreak us for v2 PUT - but doing the other way seems fine too | 20:30 |
| mordred | clarkb: mind if we do that as a second patch? | 20:30 |
| clarkb | mordred: ya thats fine | 20:30 |
| *** moravec1 has joined #openstack-infra | 20:30 | |
| *** moravec1 has quit IRC | 20:30 | |
| mordred | cool | 20:31 |
| openstackgerrit | Merged openstack-infra/project-config: Translations: Refaktor https://review.openstack.org/226477 | 20:31 |
| openstackgerrit | Bruno Tavares proposed openstack-infra/puppet-zuul: Add acceptance tests for puppet-zuul https://review.openstack.org/217732 | 20:31 |
| clarkb | mordred: however you don't have test coverage for that change from what I can see | 20:31 |
| clarkb | mordred: are ou ok with a -1 please test this :P | 20:31 |
| mordred | clarkb: I so added tests | 20:31 |
| clarkb | mordred: there is no v2 with is_public that I see | 20:32 |
| *** kiranr has quit IRC | 20:32 | |
| mordred | there is | 20:32 |
| clarkb | there is v1 with is_public and v2 with visibility | 20:32 |
| mordred | nope. look again | 20:32 |
| *** moravec has quit IRC | 20:32 | |
| mordred | there is an is_public passed to all incarnations | 20:32 |
| mordred | and we test that it gets passed to v1 as is_public and v2 as visibility | 20:32 |
| clarkb | mordred: oh args aren't the args | 20:32 |
| clarkb | thats not confusing | 20:32 |
| *** wshao has quit IRC | 20:32 | |
| clarkb | ok so a followup with need to do it both ways but this does it right for the single direction mapping | 20:33 |
| *** hashar has quit IRC | 20:33 | |
| clarkb | now to review the rest of the stack so that can merge | 20:33 |
| clarkb | out of curiousity why is it stacked like that? I don't see the dep | 20:33 |
| *** baoli has joined #openstack-infra | 20:35 | |
| mordred | clarkb: because the previous one was also touching the exact same tests and I didn't wnat to play a git rebase dance | 20:35 |
| mordred | clarkb: turns out I tink it could be unstacked at this point - but the rest of the stack should be ready to go anyway | 20:35 |
| *** AJaeger has quit IRC | 20:35 | |
| Shrews | jeblair: hola | 20:36 |
| clarkb | jeblair: what is _get_auth_plugin_class() doing in https://review.openstack.org/#/c/225365/5/shade/__init__.py ? was that just a rebase mixup? | 20:36 |
| clarkb | mordred: ^ maybe you know | 20:37 |
| mordred | looking | 20:37 |
| mordred | clarkb: yeah. that might be a rebase derp | 20:37 |
| mordred | clarkb: yup. that functoin should be in the next patch. WERID | 20:38 |
| mordred | clarkb: oh - actually, it shouldn't be there at all | 20:39 |
| openstackgerrit | Ramy Asselin proposed openstack-infra/puppet-openstackci: Create a single-node 3rd party ci system https://review.openstack.org/200330 | 20:39 |
| *** doug-fish has quit IRC | 20:39 | |
| *** gyee has joined #openstack-infra | 20:39 | |
| Shrews | jeblair: So, your shade exception changes... log.error() is called in the base class, but if i'm using shade and haven't configured logging, i get: No handlers could be found for logger "shade.exc" | 20:39 |
| mordred | clarkb: that one was removed in the patch this got rebased on | 20:39 |
| Shrews | also mordred ^^^^ | 20:39 |
| *** tonytan4ever has quit IRC | 20:40 | |
| Shrews | I think that needs some rethinking | 20:40 |
| asselin | anteaya, krtaylor, mmedvede patrickeast first working common-ci patch for 3party folks ^^ | 20:40 |
| clarkb | mordred: so -1, please remove? | 20:40 |
| *** moravec has joined #openstack-infra | 20:40 | |
| clarkb | mordred: you want to leave that comment or should I? | 20:40 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Remove many redundant debug logs https://review.openstack.org/225365 | 20:40 |
| mordred | clarkb: nah. I just fixed it | 20:40 |
| clarkb | mordred: did you rebase all the things on that? | 20:41 |
| clarkb | thats how I got there, I went about 5 deep in the stack | 20:41 |
| mordred | clarkb: I did not - but it's not a conflict, so they should merge properly on it | 20:41 |
| mordred | Shrews: hrm | 20:41 |
| clarkb | mordred: but hold on | 20:41 |
| anteaya | asselin: nice work | 20:41 |
| clarkb | mordred: found one more bug you may want to fix | 20:42 |
| mordred | clarkb: neat! | 20:42 |
| Shrews | mordred: oh, that was actually your change, not jeblair's | 20:42 |
| patrickeast | asselin: oo nice | 20:42 |
| clarkb | mordred: posted | 20:42 |
| openstackgerrit | Glauco Oliveira proposed openstack-infra/project-config: Create new project 'puppet-openstack-health' https://review.openstack.org/226545 | 20:42 |
| mordred | Shrews: why do we get that for log.error and not for log.debug? | 20:42 |
| Shrews | mordred: logging is new from within the exc base class. i assume it happens at all levels | 20:43 |
| openstackgerrit | Bruno Tavares proposed openstack-infra/puppet-logstash: Add acceptance tests for puppet-logstash. https://review.openstack.org/224858 | 20:43 |
| *** claudiub has joined #openstack-infra | 20:43 | |
| clarkb | Shrews: shouldn't you get the default logger in that case? | 20:43 |
| mordred | Shrews: but we've never seen it with the log.deug calls | 20:44 |
| openstackgerrit | Bruno Tavares proposed openstack-infra/puppet-gerrit: Setup preconditions for acceptance tests. https://review.openstack.org/219400 | 20:44 |
| mordred | clarkb: perhaps there is a default setup step we're not performing? | 20:44 |
| mmedvede | asselin: cool! I'll test when I get a moment | 20:45 |
| Shrews | mordred: ah, that message does NOT appear with debug | 20:45 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Remove many redundant debug logs https://review.openstack.org/225365 | 20:46 |
| Shrews | so my confusion grows | 20:46 |
| mordred | clarkb: thnaks. fixed | 20:46 |
| mordred | Shrews: I think we should definitely figure out what we're doing wrong | 20:46 |
| mordred | Shrews: but I'm pretty sure we should be able to call logging things without handlers being set up - otherwise python is lame | 20:47 |
| openstackgerrit | Bruno Tavares proposed openstack-infra/puppet-zuul: Add acceptance tests for puppet-zuul https://review.openstack.org/217732 | 20:47 |
| Shrews | mordred: otherwise??? | 20:47 |
| clarkb | ya it should just null route | 20:47 |
| Shrews | mordred: this wouldn't happen in rust | 20:47 |
| Shrews | just sayin | 20:47 |
| clarkb | unless do you have a logger that says use handler foo and foo doesnt exist? | 20:48 |
| clarkb | Shrews you cant configure rust logging at runtiem? | 20:48 |
| mordred | are we setting up a logging fixture in our tests? | 20:48 |
| clarkb | you should be if testring but I dunno if shade dies | 20:49 |
| clarkb | *does | 20:49 |
| jeblair | https://docs.python.org/3.1/library/logging.html#configuring-logging-for-a-library | 20:49 |
| *** bswartz has quit IRC | 20:49 | |
| jeblair | (new in version 3.1 ?) | 20:50 |
| *** Swami has joined #openstack-infra | 20:50 | |
| *** arithx has joined #openstack-infra | 20:50 | |
| Shrews | mordred: so, default logging level is "warning", so anything below that (debug, info) wouldn't be processed by the logger, thus no message | 20:51 |
| mordred | ah | 20:51 |
| *** e0ne has quit IRC | 20:51 | |
| mordred | ok- so we can just add a nullhandler - and then if the calling app also adds a handler, neat | 20:51 |
| mordred | jeblair, Shrews: does that work for y'all? | 20:52 |
| Shrews | mordred: i think that is the correct solution | 20:52 |
| *** cfarquhar has joined #openstack-infra | 20:53 | |
| *** cfarquhar has quit IRC | 20:53 | |
| *** cfarquhar has joined #openstack-infra | 20:53 | |
| jeblair | mordred: that _seems_ consistent with how i think libs should handle logging; i'm just not sure about python 2.7 though; maybe we need to implement a nullhandler? | 20:53 |
| mordred | jeblair: yes. I believe we need to implement a nullhandlers | 20:53 |
| Shrews | oh, yeah. new in 3.1 | 20:53 |
| Shrews | boo | 20:53 |
| *** annegentle has quit IRC | 20:54 | |
| *** diazjf has left #openstack-infra | 20:54 | |
| mordred | so, in 3.1 there is a nullhandler. in 2.7 we have to define one ourselves | 20:54 |
| mordred | I say just define it ourselves and use it in nomatter what | 20:54 |
| mordred | it is, after all, a null class | 20:54 |
| jeblair | ya, it _sounds_ like it should be simple :) | 20:55 |
| *** DericHorn-HP has quit IRC | 20:55 | |
| *** DericHorn-HP has joined #openstack-infra | 20:55 | |
| clarkb | pleia2: going to try and get the changes to support versions psuhed today so you can look before your day off | 20:57 |
| clarkb | pleia2: out of curiousity do you know if zanata cli pushing a version that does not exist in zanata results inthat version existing or a nerror? | 20:57 |
| openstackgerrit | Bruno Tavares proposed openstack-infra/puppet-gerrit: Setup preconditions for acceptance tests. https://review.openstack.org/219400 | 20:57 |
| pleia2 | clarkb: I don't know, sorry | 20:58 |
| *** notnownikki has quit IRC | 20:59 | |
| *** eharney has quit IRC | 21:00 | |
| *** baoli has quit IRC | 21:01 | |
| *** DericHorn-HP has quit IRC | 21:02 | |
| *** baoli has joined #openstack-infra | 21:02 | |
| *** alivigni has quit IRC | 21:02 | |
| *** dolpher has joined #openstack-infra | 21:02 | |
| openstackgerrit | sebastian marcet proposed openstack-infra/openstackid-resources: Summit Application API https://review.openstack.org/221964 | 21:04 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Add a NullHandler to all of our loggers https://review.openstack.org/226552 | 21:04 |
| mordred | jeblair, Shrews, clarkb: ^^ | 21:04 |
| openstackgerrit | Victor Stinner proposed openstack-infra/project-config: Add non-voting gate-designate-python34 check job https://review.openstack.org/226553 | 21:05 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Add a NullHandler to all of our loggers https://review.openstack.org/226552 | 21:05 |
| *** ccarrara has quit IRC | 21:05 | |
| mordred | or, rather, that | 21:05 |
| *** trown is now known as trown|outttypeww | 21:05 | |
| Shrews | mordred: that message is no longer emitted with that change. | 21:07 |
| *** tiswanso has quit IRC | 21:07 | |
| *** ccarrara has joined #openstack-infra | 21:07 | |
| *** cdent has joined #openstack-infra | 21:07 | |
| Shrews | so, yay | 21:07 |
| jeblair | mordred: oh, i wonder if that could end up attaching a bunch of nullhandlers | 21:08 |
| jeblair | mordred: maybe you should 'getlogger', then see if there are any handlers, and if none, add nullhandler? | 21:08 |
| openstackgerrit | Paul Belanger proposed openstack-infra/grafyaml: [WIP] Add validate command https://review.openstack.org/226554 | 21:08 |
| *** aysyd has quit IRC | 21:08 | |
| mordred | jeblair, clarkb, fungi: since you don't make the tweets: http://inaugust.com/posts/multi-cloud-with-python-openstackclient.html | 21:09 |
| *** claudiub has quit IRC | 21:09 | |
| mordred | jeblair: can do | 21:09 |
| *** ayoung has quit IRC | 21:09 | |
| *** dolpher has quit IRC | 21:09 | |
| fungi | mordred: thanks for acting as a twitter to real conversation gateway ;) | 21:10 |
| pleia2 | hah | 21:10 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade: Add a NullHandler to all of our loggers https://review.openstack.org/226552 | 21:10 |
| jeblair | mordred: cool. can you set that up on puppetmaster? | 21:10 |
| fungi | mordred is how i consume all the twitternews | 21:10 |
| mordred | jeblair: there's with a check to see if there's anything on the handlers | 21:10 |
| mordred | jeblair: yes | 21:10 |
| mordred | jeblair: it's on my TDL | 21:11 |
| *** thorst has quit IRC | 21:11 | |
| pleia2 | speaking of which, from the twitterverse (and this was on planet.o.o): http://hackhackblog.blogspot.com/2015/09/my-first-openstack-infra-patch.html | 21:11 |
| pleia2 | thanks for writing this mmmpork :) | 21:11 |
| *** baoli has quit IRC | 21:12 | |
| *** baoli has joined #openstack-infra | 21:12 | |
| *** mrmartin has quit IRC | 21:12 | |
| *** tlian2 has joined #openstack-infra | 21:13 | |
| mordred | pleia2: heh. my blog no-longer planets, since it's now actually just a collection of static html | 21:13 |
| pleia2 | mordred: sad story | 21:13 |
| *** glauco has quit IRC | 21:13 | |
| pleia2 | mordred: should write a script to generate static rss feed | 21:13 |
| pleia2 | in your free time | 21:13 |
| *** glauco has joined #openstack-infra | 21:13 | |
| *** Sukhdev has joined #openstack-infra | 21:15 | |
| *** ldnunes has quit IRC | 21:15 | |
| *** tlian has quit IRC | 21:15 | |
| mordred | pleia2: well, I'm actually using gulp to generate the content from some templates already | 21:15 |
| fungi | my blog never has planeted, mostly because i've never gotten around to starting a blog. i hear it's some new hipster contraction of "web log" and i should care, so i'll probably eventually get around to it | 21:15 |
| mordred | pleia2: so, in theory, it's either easy or already done by some plugin | 21:15 |
| *** bltavares has quit IRC | 21:15 | |
| jeblair | fungi: i'd give it more time and see if it has legs | 21:16 |
| fungi | sold | 21:16 |
| anteaya | fungi: I so admire the hipster in you | 21:16 |
| mordred | speaking of - hey krotscheck - when I moved to my new computer, something on my laptop was not set up completely properly, so npm run *blah* is not working in my gulp repo | 21:16 |
| fungi | anteaya: i' | 21:16 |
| fungi | m working on the handlebar mustache next | 21:16 |
| mordred | krotscheck: which I'm too dumb to figure out- so I went to just running individual gulp comnmands | 21:16 |
| anteaya | ha ha ha | 21:16 |
| pleia2 | fungi: I think you missed the boat, no one really blogs anymore except for companies and a few people | 21:16 |
| pleia2 | only twitters now | 21:16 |
| anteaya | I was wondering if you were going for the skin tight cargo shorts next | 21:17 |
| mordred | pleia2: http://git.inaugust.com/cgit/inaugust.com/tree/src/posts/multi-cloud-with-python-openstackclient.hbs this is how I'm doing blog posts now | 21:17 |
| fungi | pleia2: thanks. perhaps if i wait long enough then twitter will only be companies and a few people too | 21:17 |
| pleia2 | mordred: wow :) | 21:17 |
| anteaya | oldsters sitting at the next table in tasty harmony in fort collins were arguing about instagram | 21:17 |
| anteaya | I'd say its the way to go | 21:17 |
| *** cdent has quit IRC | 21:17 | |
| anteaya | based on my eavesdropping | 21:18 |
| mordred | pleia2: which lets me just write plain HTML AND not do a ton of copy paste | 21:18 |
| *** smarcet has quit IRC | 21:18 | |
| pleia2 | mordred: and not upgrade wordpress every 2 weeks \o/ | 21:18 |
| anteaya | nicely done mordred | 21:19 |
| anteaya | no obstacles | 21:19 |
| *** jaypipes has quit IRC | 21:19 | |
| mordred | pleia2: yup | 21:19 |
| fungi | yeah, i have a wordpress installation on one domain i'm really just using as a basically static cms. i need to rip it out and replace it with a templating engine which will let me just use restructuredtext files | 21:19 |
| *** moravec has quit IRC | 21:19 | |
| mordred | pleia2: just static files serving from a static web server | 21:19 |
| *** david-lyle has joined #openstack-infra | 21:19 | |
| pleia2 | I have 13+ years of blog in wordpress, likely not going anywhere | 21:20 |
| fungi | and then i have another site where i'm using moinmoin as a basically static cms. same sad story there | 21:20 |
| anteaya | ha ha ha | 21:21 |
| anteaya | little sprinklings | 21:21 |
| *** thiagop has quit IRC | 21:21 | |
| *** DericHorn-HP has joined #openstack-infra | 21:21 | |
| *** moravec has joined #openstack-infra | 21:21 | |
| *** achanda has quit IRC | 21:21 | |
| *** ayoung has joined #openstack-infra | 21:22 | |
| jeblair | speaking of which, randall just wrote more words on his computer for other people to read today. i thought he stopped doing that too, but it looks like i was wrong. | 21:22 |
| jeblair | http://blog.xkcd.com/ | 21:22 |
| *** pc_m has quit IRC | 21:23 | |
| jeblair | also, http://xkcd.com/simplewriter/ should be used for all announcements. henceforth. er, i mean, "after this one". | 21:24 |
| *** annegentle has joined #openstack-infra | 21:24 | |
| *** jsavak has quit IRC | 21:24 | |
| pleia2 | haha | 21:24 |
| anteaya | ha ha ha | 21:24 |
| anteaya | yay randall | 21:24 |
| openstackgerrit | Doug Hellmann proposed openstack/requirements: Add reno for release notes https://review.openstack.org/226560 | 21:25 |
| anteaya | and I will continue to use henceforth forthwith | 21:25 |
| anteaya | and am willing to explain them | 21:25 |
| anteaya | couch and sofa need explaining apparently | 21:25 |
| anteaya | as does furniture | 21:25 |
| anteaya | but not bed | 21:25 |
| *** DericHorn-HP has quit IRC | 21:27 | |
| *** ayoung has quit IRC | 21:29 | |
| fungi | i love that his article dogfoods his tool | 21:30 |
| anteaya | :) | 21:31 |
| openstackgerrit | Stephen Lowrie proposed openstack-infra/project-config: Add libvirt-lxc non-voting gate for nova https://review.openstack.org/226564 | 21:32 |
| openstackgerrit | Stephen Lowrie proposed openstack-infra/project-config: Add libvirt-lxc non-voting gate for nova https://review.openstack.org/226565 | 21:36 |
| *** david-lyle has quit IRC | 21:37 | |
| *** erikwilson_ has quit IRC | 21:38 | |
| *** bdemers has quit IRC | 21:39 | |
| *** madhu_ak has joined #openstack-infra | 21:39 | |
| *** moravec has quit IRC | 21:42 | |
| *** tiswanso has joined #openstack-infra | 21:42 | |
| krotscheck | mordred: Did you do an npm install first? | 21:42 |
| krotscheck | I have a broken macro that's behaving oddly. | 21:42 |
| krotscheck | http://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/jobs/macros.yaml#n176 | 21:42 |
| krotscheck | So, that one _seems_ to work. | 21:42 |
| krotscheck | But then I see http://logs.openstack.org/58/217958/2/check/gate-eslint-config-openstack-npm-run-lint/a85b5f6/console.html#_2015-09-22_17_26_34_851 | 21:42 |
| krotscheck | And, well, it's not executing the || | 21:42 |
| krotscheck | And it's also returning a 0 exit code. | 21:42 |
| krotscheck | Does the shell builder understand ||? | 21:42 |
| *** burgerk has quit IRC | 21:42 | |
| mordred | krotscheck: I did? | 21:42 |
| krotscheck | mordred: Ok, what error are you getting when you do npm run lint or something like that? | 21:43 |
| mordred | krotscheck: no error - it just doesn't do anything | 21:43 |
| mordred | krotscheck: npm run serve | 21:43 |
| sridhar_ram | clarkb: fungi: folks - I managed to write an ice-breaker devstack gate-job for tacker (thanks for the help last week). Please help to review / guide https://review.openstack.org/#/c/226059/ | 21:43 |
| krotscheck | mordred: Which project? | 21:43 |
| *** gondoi is now known as zz_gondoi | 21:43 | |
| mordred | krotscheck: http://git.inaugust.com/cgit/inaugust.com/ | 21:44 |
| nibalizer | krotscheck: I htink think he's working on his blog | 21:44 |
| mordred | krotscheck: gulp serve works | 21:44 |
| mordred | krotscheck: (this is mostly me trying to learn all these tools) | 21:44 |
| krotscheck | mordred: You don't have a serve command in package.json. Try `npm start` | 21:45 |
| *** tlian2 has quit IRC | 21:45 | |
| krotscheck | mordred: Your list of valid commands is http://git.inaugust.com/cgit/inaugust.com/tree/package.json#n7 | 21:45 |
| krotscheck | "start" is also a lifecycle command, so you can just go `npm start` | 21:45 |
| *** esmute has quit IRC | 21:46 | |
| mordred | krotscheck: ah! neat | 21:46 |
| mordred | krotscheck: that's less typing | 21:46 |
| krotscheck | Or you can add a 'serve' command that just passes through to gulp. | 21:46 |
| krotscheck | Anyway. | 21:46 |
| *** tiswanso has quit IRC | 21:46 | |
| krotscheck | Anyone know why the shell builder ignores ||? | 21:46 |
| mordred | krotscheck: I do not - I thought we used those in other places too | 21:47 |
| krotscheck | mordred: We don't really. There's one sudo `things || otherthings` | 21:47 |
| mmmpork | pleia2: thanks for the shout :D | 21:47 |
| mordred | oh. well. then I really don't know | 21:47 |
| *** austin811 has joined #openstack-infra | 21:47 | |
| *** cody-somerville has joined #openstack-infra | 21:48 | |
| *** mriedem1 has quit IRC | 21:48 | |
| *** salv-orlando has joined #openstack-infra | 21:48 | |
| *** dahoo has joined #openstack-infra | 21:49 | |
| *** dahoo is now known as jerryz | 21:49 | |
| *** moravec has joined #openstack-infra | 21:50 | |
| *** tlian has joined #openstack-infra | 21:50 | |
| *** arithx has quit IRC | 21:50 | |
| *** erikwilson_ has joined #openstack-infra | 21:52 | |
| *** erikwilson_ has quit IRC | 21:52 | |
| fungi | now that meetings have mostly wrapped for the current planetary rotation, i need to get back to assembling furniture | 21:53 |
| fungi | but will be around if there is anything urgent requiring my attention | 21:53 |
| *** mriedem has joined #openstack-infra | 21:53 | |
| anteaya | fungi: happy bookshelves | 21:53 |
| fungi | thanks | 21:53 |
| *** annegentle has quit IRC | 21:54 | |
| *** annegentle has joined #openstack-infra | 21:54 | |
| mordred | jeblair: just +A'd your remove-debugging-logs patch - when you get a sec, could use a second set of eyes on https://review.openstack.org/#/c/221126/11 | 21:55 |
| openstackgerrit | Merged openstack-infra/puppet-lodgeit: Add optional, enabled by default, robots.txt https://review.openstack.org/226091 | 21:56 |
| *** ivar-lazzaro has quit IRC | 21:57 | |
| nibalizer | pleia2: ^^ | 21:57 |
| pleia2 | nibalizer: ta | 21:57 |
| *** asettle has joined #openstack-infra | 21:58 | |
| *** ivar-lazzaro has joined #openstack-infra | 21:58 | |
| pleia2 | anteaya: have a moment to look at the commit message on this change? https://review.openstack.org/#/c/201312/ | 21:58 |
| *** wshao has joined #openstack-infra | 21:58 | |
| pleia2 | anteaya: should be an easy enough update, then I think it's fine to land | 21:59 |
| *** cody-somerville has quit IRC | 21:59 | |
| krotscheck | zaro: You around? You know jjb things, right? I think I identified a bug with the script builder. | 21:59 |
| mordred | zaro knows all the things | 22:00 |
| openstackgerrit | Clark Boylan proposed openstack-infra/project-config: Support different translations versions https://review.openstack.org/225951 | 22:00 |
| openstackgerrit | Clark Boylan proposed openstack-infra/project-config: Translate stable/liberty on horizon, DOA, and Nova https://review.openstack.org/225954 | 22:00 |
| openstackgerrit | Clark Boylan proposed openstack-infra/project-config: Make translation pushes to gerrit branch aware https://review.openstack.org/225953 | 22:00 |
| openstackgerrit | Clark Boylan proposed openstack-infra/project-config: Push version specific .pots to Zanata https://review.openstack.org/225952 | 22:00 |
| clarkb | pleia2: ^ there I think I got things sorted out | 22:00 |
| clarkb | now I just have to test what happens when you push a version that doesn't exist in the server | 22:00 |
| *** niedbalski_ has quit IRC | 22:00 | |
| pleia2 | clarkb: huzzah, I'll start going through them momentarily | 22:00 |
| clarkb | pleia2: that rebase was "fun" | 22:00 |
| *** weshay has quit IRC | 22:00 | |
| pleia2 | :) | 22:01 |
| clarkb | but the resulting patches should be easier to review so win on that side | 22:01 |
| *** baoli has quit IRC | 22:01 | |
| *** baoli has joined #openstack-infra | 22:01 | |
| *** wenlock has quit IRC | 22:01 | |
| anteaya | pleia2: yeah I agree that is a very poor commit message | 22:02 |
| anteaya | I hope all my griping about having detailed commit messages gets taken to heart | 22:02 |
| anteaya | as one sentence commit messages are next to useless | 22:02 |
| *** baoli has quit IRC | 22:02 | |
| anteaya | pleia2: and yes I would be my pleasure to create a useful commit message for that patch | 22:02 |
| clarkb | pleia2: so in theory if I go to proposal host, go into a workspace for the job that pushes to zanata, edit the zanata config to change the version and the server then run the command that should be a sufficient test ya? | 22:02 |
| * clarkb attempts this | 22:02 | |
| *** dzamboni has quit IRC | 22:02 | |
| *** niedbalski has joined #openstack-infra | 22:03 | |
| pleia2 | anteaya: thank you! | 22:03 |
| krotscheck | Actually, mordred, I think the issue is that the script block in yaml starts with | - how does one escape | in yaml? | 22:03 |
| *** ivar-lazzaro has quit IRC | 22:03 | |
| pleia2 | clarkb: as the jenkins user, right | 22:03 |
| *** asettle has quit IRC | 22:03 | |
| *** wshao has quit IRC | 22:04 | |
| *** baoli has joined #openstack-infra | 22:04 | |
| *** asettle has joined #openstack-infra | 22:04 | |
| pleia2 | clarkb: jenkins user has all magic credentials, just make sure it goes to -dev | 22:04 |
| clarkb | pleia2: yup Iwill edit that | 22:04 |
| clarkb | I am taking that slave offline temporarily in jenkins to avoid a job coming through and updating things after I edit them | 22:05 |
| *** ivar-lazzaro has joined #openstack-infra | 22:05 | |
| *** asettle has quit IRC | 22:06 | |
| *** timburke has quit IRC | 22:06 | |
| *** asettle has joined #openstack-infra | 22:06 | |
| *** dimtruck is now known as zz_dimtruck | 22:06 | |
| *** btully has quit IRC | 22:07 | |
| *** timburke has joined #openstack-infra | 22:07 | |
| krotscheck | ISO Person who knows YAML really well | 22:07 |
| *** jerrygb has joined #openstack-infra | 22:08 | |
| *** davidlenwell has quit IRC | 22:08 | |
| *** btully has joined #openstack-infra | 22:08 | |
| *** markmcclain has quit IRC | 22:08 | |
| nibalizer | krotscheck: that seems somewhat unlikely to me | 22:08 |
| nibalizer | but its easy to test | 22:08 |
| clarkb | pleia2: oh wow I just realized that the config for zanata is a global user config not a per job config | 22:09 |
| *** davidlenwell has joined #openstack-infra | 22:09 | |
| krotscheck | nibalizer: What, that yaml is interpreting | as a script block start rather than a bash command? | 22:09 |
| clarkb | StevenK: pleia2: we should change that if we can otherwise we have to be very very carefuly | 22:09 |
| pleia2 | clarkb: which config? | 22:09 |
| *** legoktm has quit IRC | 22:09 | |
| clarkb | pleia2: the zanata.ini | 22:10 |
| pleia2 | ah, yeah | 22:10 |
| *** Rockyg has quit IRC | 22:10 | |
| clarkb | which I thought had the version and project in it too but now I don't see it | 22:10 |
| *** edmondsw has quit IRC | 22:10 | |
| *** mwagner_lap has quit IRC | 22:10 | |
| openstackgerrit | Anita Kuno proposed openstack-infra/puppet-gerrit: Set apache to retain one month of logs https://review.openstack.org/201312 | 22:11 |
| *** wshao has joined #openstack-infra | 22:11 | |
| krotscheck | nibalizer: You're right, at least according to this online yaml parser that I found | 22:11 |
| *** legoktm has joined #openstack-infra | 22:11 | |
| *** jvrbanac has quit IRC | 22:11 | |
| nibalizer | ya i just verified with yaml | 22:12 |
| pleia2 | anteaya: perfect, thank you | 22:12 |
| nibalizer | er pyhton | 22:12 |
| clarkb | pleia2: so zanata.xml goes in the project somewhere but zanata.ini is global? I wonder if we can just shove all of this into zanata.xml | 22:12 |
| *** legoktm is now known as Guest5675 | 22:12 | |
| *** zhenguo has quit IRC | 22:12 | |
| anteaya | pleia2: thank you | 22:12 |
| pleia2 | clarkb: the zanata.xml is generated on the fly by our translations scripts (it's not in git) but yeah, zanata.ini is global | 22:12 |
| *** gus has quit IRC | 22:12 | |
| *** tiswanso has joined #openstack-infra | 22:12 | |
| *** zhenguo has joined #openstack-infra | 22:13 | |
| clarkb | pleia2: but then why isn't it in the workspace? | 22:13 |
| clarkb | maybe that job has never run hrm off to go look at other projects | 22:13 |
| *** rackertom has quit IRC | 22:13 | |
| *** alazarev has quit IRC | 22:13 | |
| clarkb | oh the url is in zanata.xml | 22:13 |
| *** jvrbanac has joined #openstack-infra | 22:13 | |
| clarkb | so now I wonder why we have it in zanata.ini at all | 22:13 |
| *** gus has joined #openstack-infra | 22:14 | |
| pleia2 | zanata.ini has credentials that the jenkins user uses to run everything | 22:14 |
| *** ashleigh_ has joined #openstack-infra | 22:14 | |
| clarkb | right and that part is fine | 22:14 |
| *** Guest5675 has quit IRC | 22:14 | |
| *** Guest5675 has joined #openstack-infra | 22:14 | |
| *** ashleighfarnham has quit IRC | 22:14 | |
| clarkb | but it also specifies what server and then we specify that again in zanata.xml | 22:14 |
| pleia2 | ah, right | 22:15 |
| *** rackertom has joined #openstack-infra | 22:15 | |
| pleia2 | I guess it's not needed in both | 22:15 |
| nibalizer | krotscheck: the install-distro-packages builder also uses || | 22:15 |
| *** alazarev has joined #openstack-infra | 22:15 | |
| openstackgerrit | Merged openstack-infra/shade: Remove many redundant debug logs https://review.openstack.org/225365 | 22:16 |
| *** markmcclain has joined #openstack-infra | 22:16 | |
| *** Piet has joined #openstack-infra | 22:16 | |
| *** GheRivero has quit IRC | 22:16 | |
| clarkb | pleia2: and the ini file is confusing, it seems to explicitly specify that the server is translate.o.o in every key as well | 22:16 |
| clarkb | pleia2: so now I am not sure how I can safely update this and need to read docs | 22:16 |
| *** tiswanso has quit IRC | 22:17 | |
| pleia2 | ah, hm | 22:17 |
| clarkb | wow they basically say go to the server and copy pasta | 22:17 |
| * clarkb does this | 22:17 | |
| *** GheRivero has joined #openstack-infra | 22:18 | |
| *** baoli has quit IRC | 22:18 | |
| *** otter768 has joined #openstack-infra | 22:18 | |
| * clarkb decides to do this as his own user and not as the proposal user | 22:19 | |
| *** maiteb has quit IRC | 22:19 | |
| pleia2 | heh, does seem like the better plan :) | 22:19 |
| *** Swami has quit IRC | 22:19 | |
| *** kgiusti has left #openstack-infra | 22:19 | |
| *** dramalho has quit IRC | 22:19 | |
| *** baoli has joined #openstack-infra | 22:20 | |
| *** boris-42 has quit IRC | 22:22 | |
| *** achanda has joined #openstack-infra | 22:22 | |
| *** StevenK has quit IRC | 22:22 | |
| *** StevenK has joined #openstack-infra | 22:22 | |
| *** otter768 has quit IRC | 22:23 | |
| tchaypo | krotscheck: it sounds like you got your YAML answers? | 22:23 |
| *** boris-42 has joined #openstack-infra | 22:24 | |
| clarkb | pleia2: org.zanata.client.exceptions.ConfigException: Username must be specified even though I copied the ini file from translate-dev :/ | 22:24 |
| *** roaet_ has joined #openstack-infra | 22:25 | |
| *** rossella_s has quit IRC | 22:25 | |
| *** roaet- has quit IRC | 22:25 | |
| *** rossella_s has joined #openstack-infra | 22:25 | |
| pleia2 | clarkb: grr | 22:25 |
| clarkb | pleia2: I think put-version is required to create a version so I expect this all to fail the way we want it to, I just really want to confirm | 22:25 |
| * pleia2 nods | 22:25 | |
| clarkb | pleia2: have you seen that error before? | 22:25 |
| pleia2 | I have not | 22:25 |
| *** electrical has quit IRC | 22:26 | |
| clarkb | pleia2: tring with cinder with a cpoied and updated zanata.xml from cinder and the ini from translate-dev | 22:26 |
| *** briancurtin has quit IRC | 22:26 | |
| *** puranamr has joined #openstack-infra | 22:26 | |
| *** darrenc has quit IRC | 22:27 | |
| *** achanda has quit IRC | 22:27 | |
| clarkb | pleia2: looks like the :443 in the url was possibly at fault | 22:27 |
| pleia2 | hm | 22:28 |
| *** cloudtrainme has quit IRC | 22:28 | |
| clarkb | so I removed it and its failing on ssl trust now | 22:28 |
| *** mrda has joined #openstack-infra | 22:28 | |
| *** briancurtin has joined #openstack-infra | 22:28 | |
| *** electrical has joined #openstack-infra | 22:28 | |
| *** frickler has quit IRC | 22:29 | |
| *** frickler has joined #openstack-infra | 22:29 | |
| mrda | Just wondering, now we're in a post-stackforge world, is https://github.com/stackforge/gerrit-dash-creator the canonical location for gerrit-dash-creator? | 22:29 |
| *** cloudtrainme has joined #openstack-infra | 22:29 | |
| pleia2 | mrda: next month we're moving everything on stackforge to the openstack/ namespace | 22:29 |
| *** puranamr has quit IRC | 22:29 | |
| anteaya | pleia2: except for the read only repos | 22:30 |
| pleia2 | and strictly speaking github is never canonical ;) we have https://git.openstack.org/openstack/ etc | 22:30 |
| pleia2 | anteaya: right! | 22:30 |
| mrda | cool, thanks. I'll put a review up then :) | 22:30 |
| mrda | pleia2: I understand, that was a typo :) | 22:30 |
| *** amitgandhinz has quit IRC | 22:30 | |
| *** darrenc has joined #openstack-infra | 22:30 | |
| anteaya | mrda: https://wiki.openstack.org/wiki/Stackforge_Namespace_Retirement | 22:30 |
| anteaya | do you see it listed on the wikipage? | 22:31 |
| pleia2 | mrda: we're aiming to do the move on October 17th, so the urls will be locked in then | 22:31 |
| mrda | thanks anteaya | 22:31 |
| pleia2 | anteaya: thanks! | 22:31 |
| pleia2 | need to step out for a moment, brb | 22:31 |
| anteaya | mrda: welcome, if it isnt' there do say so | 22:31 |
| mrda | gerrit-dash-creator isn't there :) | 22:31 |
| anteaya | mrda: do you own the repo? | 22:31 |
| mrda | umm, no. I just wanted to make a *little* change to a thing | 22:32 |
| *** moravec has quit IRC | 22:32 | |
| anteaya | mrda: yep, one of these two folks needs to add it to that wikipage: https://review.openstack.org/#/admin/groups/362,members | 22:32 |
| mrda | cool, thanks anteaya | 22:33 |
| anteaya | mrda: happy change, if it doesn't merge before oct. 17 it will have to be reprosed to the new namespace | 22:33 |
| anteaya | mrda: welcome | 22:33 |
| clarkb | com.sun.jersey.api.client.UniformInterfaceException: GET https://translate-dev.openstack.org/rest/projects/p/cinder/iterations/i/clarkb-test/locales returned a response status of 404 Not Found | 22:33 |
| *** Guest5675 is now known as legoktm | 22:33 | |
| clarkb | WOOT I think that means its safe to just run that job after every merge then expect some jobs to fail | 22:33 |
| anteaya | yay? | 22:34 |
| clarkb | that way Daisy et al can control this from within zanata rather than the jobs needing to be super smart | 22:34 |
| *** spzala has quit IRC | 22:34 | |
| openstackgerrit | Khai Do proposed stackforge/python-jenkins: Support ability to install plugins https://review.openstack.org/210529 | 22:34 |
| anteaya | yay dumb jobs | 22:34 |
| clarkb | I think its the way Daisy wants to do it as well | 22:34 |
| *** Sukhdev has quit IRC | 22:36 | |
| anteaya | if Daisy is happy, that is great | 22:36 |
| anteaya | I couldn't do her job | 22:37 |
| *** rguillebert has joined #openstack-infra | 22:37 | |
| *** ddieterl_ has quit IRC | 22:38 | |
| krotscheck | tchaypo: Sortof. Still don't know what the bug is | 22:38 |
| *** pradk has quit IRC | 22:40 | |
| tchaypo | krotscheck: it looks like http://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/jobs/macros.yaml#n176 is your issue? | 22:40 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/shade: Use the get_auth function from occ https://review.openstack.org/221126 | 22:41 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/shade: Get defaults for image type from occ https://review.openstack.org/225507 | 22:41 |
| openstackgerrit | Clark Boylan proposed openstack-infra/project-config: Translate stable/liberty on horizon, DOA, and Nova https://review.openstack.org/225954 | 22:42 |
| krotscheck | tchaypo: Yes, evertying after the || is lost, and the script returns error code 0 even if the first half fials. | 22:43 |
| *** ayoung has joined #openstack-infra | 22:44 | |
| *** amuller has quit IRC | 22:44 | |
| pleia2 | clarkb: back and reviewing those changes | 22:44 |
| *** regXboi has quit IRC | 22:45 | |
| clarkb | pleia2: thansk I just fixed a bug in the last one too | 22:45 |
| pleia2 | clarkb: re: common_translation_update.sh on 225951, my brain is totally failing on what ${2:-master} is | 22:45 |
| *** kushal has quit IRC | 22:46 | |
| clarkb | pleia2: it is saying take $2 and if unset use master as the default | 22:46 |
| clarkb | that way we don't have to updte everything all at once | 22:47 |
| *** baoli has quit IRC | 22:47 | |
| *** bpokorny has quit IRC | 22:47 | |
| pleia2 | clarkb: I see, so we'd only pass a version number if we're specifically trying to do this on non-master | 22:47 |
| *** bpokorny has joined #openstack-infra | 22:47 | |
| clarkb | pleia2: only in that one change | 22:48 |
| * pleia2 nods | 22:48 | |
| pleia2 | thanks | 22:48 |
| crinkle | nibalizer: want to talk about the apply-test re: https://review.openstack.org/#/c/209617/ ? | 22:48 |
| clarkb | pleia2: I think the next one sets it across the board, it just made it easier to udpate the "library" scripts before updating the "script" scripts | 22:48 |
| pleia2 | gotcha | 22:48 |
| clarkb | pleia2: I can do another follow up to clean all that up | 22:48 |
| crinkle | nibalizer: I'm thinking maybe we can do something similar to the Node-OS comments to have it look for the MODULE_FILE to set | 22:48 |
| anteaya | beautiful 8 pointer on my lawn eating my psuedo ginseng tree | 22:49 |
| nibalizer | crinkle: what about just having a separate infracloud.pp in the manifests directory? | 22:50 |
| nibalizer | I'd think that'd be better than being clever | 22:50 |
| clarkb | anteaya: now you shoot it? | 22:50 |
| *** zz_dimtruck is now known as dimtruck | 22:50 | |
| crinkle | nibalizer: but you said the import function doesn't work | 22:50 |
| * clarkb wonders what the rules are around that in canada | 22:50 | |
| crinkle | in puppet 4 | 22:50 |
| anteaya | clarkb: too close to the house, out of season, I don't have my gun license | 22:50 |
| crinkle | nibalizer: i don't think the comments are too clever since we're already doing it | 22:50 |
| anteaya | clarkb: not even bow season yet, illegal to shoot from a house | 22:50 |
| nibalizer | crinkle: well we don't really need to import that file actually, since we cna just tell puppet to apply that as the manifest to apply | 22:51 |
| anteaya | and I don't eat meat | 22:51 |
| clarkb | crinkle: nibalizer I don't understand how you will install conflicting puppet modules | 22:51 |
| nibalizer | so we'd have a site.pp next to a infracloud.pp | 22:51 |
| nibalizer | clarkb: infracloud machines get openstack-modules.env and everything else gets modules.env | 22:51 |
| nibalizer | also maybe we should call it infracloud-modules.env | 22:52 |
| crinkle | clarkb: we want to avoid that problem entirely by using puppet apply instead of puppetmaster | 22:52 |
| clarkb | oh I see | 22:52 |
| clarkb | nibalizer: right buit you can't do that if you master | 22:52 |
| clarkb | so puppet apply is the magic | 22:52 |
| nibalizer | ya | 22:52 |
| clarkb | pleia2: whats the best way to poke zanata about this ini file thing? | 22:52 |
| crinkle | nibalizer: okay well what would change about the apply-test script or jjb to get it to read the new infracloud.pp? | 22:52 |
| pleia2 | hmm, lodgeit seems to own the docroot, so dumping robots.txt in there didn't quite do it for paste.o.o (still serves up a 404, even though it's on the filesystem) | 22:52 |
| jeblair | clarkb: i'd say you know the rules about that now. :) | 22:52 |
| * clarkb worries someone is going to say get a jira account and file a bug | 22:53 | |
| *** dangers is now known as dangers_away | 22:53 | |
| openstackgerrit | Merged openstack-infra/project-config: Adding a functional test to the storlets project https://review.openstack.org/220139 | 22:53 |
| clarkb | jeblair: yup, need gun license, season to start, and sit in front yard not in house | 22:53 |
| nibalizer | crinkle: I'd make a second test, infracloud-apply | 22:53 |
| pleia2 | clarkb: the team? #zanata or just ping directly here | 22:53 |
| *** shashank_hegde has quit IRC | 22:53 | |
| crinkle | nibalizer: but the first test works fine, it just needs a different MODULE_FILE set | 22:53 |
| pleia2 | they should be waking up by now :) | 22:53 |
| *** Sukhdev has joined #openstack-infra | 22:54 | |
| ianw | ^^^ that storlets test actually goes ahead and installs docker and does stuff. i had it on the agenda for today but we didn't get to it. my opinion is "all care, no responsibility" ... it's likely we'll make changes underneath that will break such stuff | 22:54 |
| *** Sukhdev has quit IRC | 22:54 | |
| crinkle | nibalizer: it would be copypaste with just a couple of lines changed | 22:54 |
| clarkb | aeng_: camunoz on translate-dev.openstack.org I went into settings and generated an api key and grabbed the resulting ini file for zanata-cli. That included :443 in the host url so it looked like this translate-dev_openstack_org.url=https://translate-dev.openstack.org:443/ when I sued that it said I needed to specify a username | 22:55 |
| tchaypo | krotscheck: what parser are you using? | 22:55 |
| nibalizer | crinkle: ya I'd probably parameterize the apply-test into something that takes some arguments then have two tests defined in jjb/zuul to run the apply-test with different argument | 22:55 |
| crinkle | nibalizer: and we'd need to add logic in jjb to have it run the right apply-test script | 22:55 |
| clarkb | aeng_: camunoz removing the :443 made the username error go away | 22:55 |
| *** rm_you has quit IRC | 22:55 | |
| clarkb | aeng_: camunoz is that a known bug? I think that the ini parser may be breaking on the : ? | 22:55 |
| tchaypo | or wait | 22:55 |
| aeng_ | clarkb, let me have a look | 22:55 |
| nibalizer | this is also just how I'd solve it, if yall want to do it a different way I'm okay with that | 22:55 |
| krotscheck | tchaypo: From what I can gather...hudson.tasks.Shell | 22:55 |
| crinkle | nibalizer: that sounds more clever than having it read comments in site.pp to me :/ | 22:55 |
| jeblair | ianw: yeah, there's also a grey line around when our CI system starts to be abusive to the projects we consume; so maybe that's okay for a small stackforge project but not okay for an openstack project | 22:55 |
| tchaypo | krotscheck: to put it differently - where do you see the error? I was assuming that it was failing to parse the yaml file, but it’s just occurred to me that perhaps the result is that it’s being read in fine but isn’t being output okay when it’s running the shell command? | 22:56 |
| *** wshao has quit IRC | 22:56 | |
| *** annegentle has quit IRC | 22:56 | |
| openstackgerrit | Merged openstack-infra/project-config: Change bifrost release group to bifrost-release https://review.openstack.org/225636 | 22:56 |
| *** sabeen3 has quit IRC | 22:56 | |
| krotscheck | tchaypo: It's being read in fine. When executed, only the part before the || seems to be executed, which errors, but the overall script returns an exit code 0 | 22:57 |
| pleia2 | clarkb: hmm, seems we ran into something like this before https://zanata.atlassian.net/browse/ZNTA-111 | 22:57 |
| tchaypo | Are you able to do anything to capture what’s being passed to the shell? | 22:57 |
| nibalizer | crinkle: yea :/ | 22:57 |
| clarkb | pleia2: I wonder if that is the same bug | 22:57 |
| nibalizer | but two different kinds of control comments in site.pp makes me :( | 22:57 |
| clarkb | aeng_: see https://zanata.atlassian.net/browse/ZNTA-111 though it is presented differently there | 22:57 |
| Clint | krotscheck: is it relying on set -e or something? | 22:57 |
| pleia2 | clarkb: seems to be | 22:57 |
| openstackgerrit | Merged openstack-infra/puppet-gerrit: Set apache to retain one month of logs https://review.openstack.org/201312 | 22:58 |
| *** zns has quit IRC | 22:58 | |
| *** tongli has quit IRC | 22:58 | |
| aeng_ | pleia2, clarkb, this might due to the host.url in your zanata.ini is different to url in your zanata.xml | 22:58 |
| crinkle | nibalizer: why? | 22:58 |
| aeng_ | can you check those url in the 2 files? | 22:58 |
| clarkb | aeng_: it is the same | 22:59 |
| clarkb | oh wait | 22:59 |
| krotscheck | Clint: Not familiar with that flag. | 22:59 |
| clarkb | the one in the xml didn't have the :443 | 22:59 |
| aeng_ | yup | 22:59 |
| clarkb | ok thats not confusing at all | 22:59 |
| jeblair | krotscheck, tchaypo: "npm lint --silent || npm run lint --silent" appears in the jenkins job config for gate-app-catalog-npm-run-lint | 22:59 |
| aeng_ | thats why | 22:59 |
| openstackgerrit | Merged openstack-infra/shade: Add a NullHandler to all of our loggers https://review.openstack.org/226552 | 22:59 |
| krotscheck | tchaypo: At that point it disappears in the bowels of java | 22:59 |
| clarkb | aeng_: ok thanks | 22:59 |
| krotscheck | jeblair: wat. | 22:59 |
| pleia2 | /o\ | 22:59 |
| crinkle | nibalizer: copypasting the script and changing two lines makes me :( :P | 22:59 |
| nibalizer | crinkle: to me it says we're doing the wrong thing | 22:59 |
| krotscheck | jeblair: Yeah, that makes sense. | 22:59 |
| nibalizer | crinkle: I don't want to copypasta the script | 22:59 |
| lifeless | mordred: oh btw, having read your blog post reminded me - have I shown you shellvars? | 23:00 |
| nibalizer | I would make the script a little more generalized | 23:00 |
| nibalizer | then it could be used by both I think | 23:00 |
| ianw | jeblair: yeah, mostly just a heads up -- i think they're going to find the job pretty unreliable, especially if their containers are trying to grab data from mirrors, etc. all stuff we know; i just hope we don't end up having everybody reinvent the wheel | 23:00 |
| krotscheck | jeblair: So, npm lint --silent should fail. and it does (http://logs.openstack.org/58/217958/2/check/gate-eslint-config-openstack-npm-run-lint/a85b5f6/console.html#_2015-09-22_17_26_34_851) | 23:00 |
| tchaypo | This sounds to me as though it’s not a yaml issue | 23:00 |
| aeng_ | clarkb, pleia2, i think the zanata.xml was manually created. If you download from the project-version page, it would have the port in the url | 23:00 |
| krotscheck | But then it never picks up the `npm run lint` | 23:00 |
| jeblair | krotscheck, tchaypo, Clint: 2015-09-22 17:26:16.056 | [gate-eslint-config-openstack-npm-run-lint] $ /bin/bash -xe /tmp/hudson1148032827898161033.sh | 23:00 |
| crinkle | nibalizer: oh okay | 23:01 |
| *** angela-s has quit IRC | 23:01 | |
| Clint | krotscheck: what's the exit code when npm lint --silent fails? | 23:01 |
| jeblair | krotscheck: that's jenkins invoking the shell snippet in question; so i think Clint is on the right track | 23:01 |
| crinkle | nibalizer: then the mysterious part to me is where in jjb to pass the different parameters | 23:01 |
| krotscheck | Clint: 1 | 23:01 |
| openstackgerrit | Merged openstack-infra/project-config: change verify group to fuel-plugins-ci https://review.openstack.org/224159 | 23:02 |
| Clint | krotscheck: then i'm out of ideas | 23:02 |
| *** angdraug has joined #openstack-infra | 23:02 | |
| krotscheck | What does -e do? | 23:02 |
| nibalizer | crinkle: so like this https://review.openstack.org/#/c/175227/1/tools/apply-test.sh is another expmale of where that script is going | 23:02 |
| openstackgerrit | Merged openstack-infra/project-config: Add check-requirements job for manila-ui project https://review.openstack.org/226267 | 23:02 |
| anteaya | SergeyLukjanov: thank you | 23:02 |
| openstackgerrit | Merged openstack-infra/project-config: Set docimpact group for networking-ovn https://review.openstack.org/226014 | 23:02 |
| *** cloudtrainme has quit IRC | 23:02 | |
| nibalizer | crinkle: oh I think we just have two jobs | 23:02 |
| Clint | krotscheck: it chokes on failures, but with an || in the line it won't matter | 23:02 |
| anteaya | so how long will it take before gerrit removes 48 weeks of apache logs now that https://review.openstack.org/#/c/201312/ has merged? | 23:03 |
| nibalizer | crinkle: so here http://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/jobs/infra.yaml#n91 | 23:03 |
| anteaya | because gerrit could use the space | 23:03 |
| nibalizer | we'd probably have two builders in http://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/jobs/infra.yaml#n91 one with the infracloud stuff one with the normal stuff | 23:04 |
| jeblair | anteaya: within 24 hours i hope? | 23:04 |
| anteaya | jeblair: thank you | 23:04 |
| nibalizer | at least thats my general idea, i've not implemented it | 23:04 |
| anteaya | I'll check tomorrow | 23:04 |
| krotscheck | Clint, jeblair: Well, that's a puzzle. Any way for us to see what that huson###.sh script looks like? | 23:04 |
| crinkle | nibalizer: okay so just add another one like that but with another env variable set? | 23:04 |
| *** spzala has joined #openstack-infra | 23:05 | |
| krotscheck | Short of writing a jjb unit test? | 23:05 |
| nibalizer | crinkle: those are the details, I don't really care how it ends up working | 23:05 |
| nibalizer | but yea positional arguments, or env vars, both can be handled by jjb | 23:05 |
| anteaya | jeblair: are there other ways we can make/save space on the gerrit server? | 23:05 |
| crinkle | nibalizer: okay, ty | 23:05 |
| nibalizer | there is already a fair bit of duplication in this shell stuff right now | 23:06 |
| nibalizer | so we could make that better all around while we're in there changing stuff | 23:06 |
| jeblair | krotscheck: i checked gate-eslint-config-openstack-npm-run-lint on jenkins02 and it has "npm lint --silent || npm run lint --silent" in the job config | 23:07 |
| jeblair | that's about as close as it gets | 23:07 |
| nibalizer | specifically install_modules_accptance.sh and apply-test.sh have copied code | 23:07 |
| krotscheck | jeblair: That's odd. | 23:07 |
| krotscheck | jeblair: The project that found this is actually refstack, they have a build with lots of errors that just pass. | 23:08 |
| krotscheck | jeblair: I'll probably just rewrite the whole thing checking for the exit code manually. Less confusion that way | 23:08 |
| anteaya | that isn't reassuring | 23:08 |
| tchaypo | krotscheck: so you believe that given the commandline "npm lint --silent || npm run lint --silent” the first part is being executed? | 23:10 |
| *** asettle has quit IRC | 23:10 | |
| *** ZZelle_ has quit IRC | 23:10 | |
| Clint | tchaypo: it spews help output in the log | 23:10 |
| jeblair | this is strongly suggesting that the exit code is actually 0. | 23:10 |
| krotscheck | tchaypo: Yes. When that's executed, you get the "Usage: npm <command>" output telling you you used something that's not there. It's in the logs. | 23:10 |
| tchaypo | and that returns 1? | 23:11 |
| Clint | it does suggest that | 23:11 |
| krotscheck | tchaypo: Yes | 23:11 |
| tchaypo | which should trigger the part after the || | 23:11 |
| tchaypo | which doesn’t seem to be being run? | 23:11 |
| krotscheck | jeblair, Clint: I agree. | 23:11 |
| jroll | how do we know 'npm run lint --silent' isn't executing? | 23:11 |
| jroll | since it's silent and all | 23:11 |
| Clint | 'cuz the set -x should show it? | 23:11 |
| jeblair | krotscheck: can you trigger a job for the change and point me at the change you triggered it for? | 23:12 |
| jroll | mmm, right | 23:12 |
| krotscheck | jeblair, Clint: In the case of this build, both sides should return 1: https://review.openstack.org/#/c/226050 | 23:12 |
| jeblair | er, "for a change" | 23:12 |
| krotscheck | jeblair: Will ^^ work? | 23:12 |
| krotscheck | Lemme recheck | 23:12 |
| *** achanda has joined #openstack-infra | 23:12 | |
| jeblair | krotscheck: yeah, thx | 23:12 |
| * tchaypo has to run away | 23:13 | |
| Clint | run away! | 23:13 |
| openstackgerrit | Merged openstack-infra/release-tools: automatically submit constraints updates for new releases https://review.openstack.org/201682 | 23:16 |
| openstackgerrit | Merged openstack-infra/release-tools: add special case for django-openstack-auth https://review.openstack.org/225180 | 23:17 |
| openstackgerrit | Merged openstack-infra/release-tools: Show output of zuul-cloner https://review.openstack.org/225229 | 23:17 |
| openstackgerrit | Merged openstack-infra/grafyaml: Bump pbr requirement to >= 1.3 https://review.openstack.org/224860 | 23:17 |
| tonyb | I've seen links to a ci status page that shows how often a given CI is -1'ing does anyone have the URL handy? | 23:17 |
| tonyb | it's not an infra thing but .... | 23:18 |
| asselin | tonyb, http://ci-watch.tintri.com/project?project=cinder&time=24+hours | 23:18 |
| *** spzala has quit IRC | 23:18 | |
| *** dims has quit IRC | 23:18 | |
| tonyb | asselin: Thank you. That's the one. | 23:18 |
| anteaya | we are hoping it will become an infra thing | 23:18 |
| *** dimtruck is now known as zz_dimtruck | 23:18 | |
| pleia2 | anteaya: /etc/logrotate.d/apache2 has been updated on review.o.o, might just want to nudge someone tomorrow to see where we are with gerrit logs | 23:18 |
| pleia2 | anteaya: to compare, right now: 6.9G /var/log/apache2/ | 23:19 |
| asselin | yes, mmedvede's making progress: https://review.openstack.org/#/q/topic:ci-dashboard+status:open,n,z | 23:19 |
| pleia2 | anteaya: that should drop a few gigs :) | 23:19 |
| anteaya | pleia2: woooo, thank you | 23:19 |
| anteaya | will nudge someone tomorrow, unless they beat me to it | 23:19 |
| * pleia2 nods | 23:19 | |
| anteaya | asselin: yay progress! | 23:19 |
| pleia2 | that'll give us some decent breathing room there | 23:20 |
| anteaya | pleia2: and none too soon | 23:20 |
| anteaya | I think we have less than 4 G to spare right now | 23:20 |
| *** dramalho has joined #openstack-infra | 23:20 | |
| pleia2 | yeah, we're at 94% full, 2.5G free | 23:20 |
| openstackgerrit | Merged openstack-infra/project-config: Add ceph to os-brick check jobs https://review.openstack.org/223257 | 23:20 |
| anteaya | ooooo | 23:20 |
| anteaya | pleia2: thanks for pushing on that patch | 23:20 |
| tonyb | anteaya: that would be very cool, if it did. | 23:21 |
| anteaya | tonyb: do weigh in: https://review.openstack.org/#/c/194437/ | 23:21 |
| openstackgerrit | Clark Boylan proposed openstack-infra/nodepool: Add a test for the list command https://review.openstack.org/226591 | 23:22 |
| * krotscheck grumbles about trying to repro this :/ | 23:22 | |
| clarkb | mordred: ^ wrote that test | 23:22 |
| anteaya | mmedvede: let's respin the spec to include asselin's comments please | 23:22 |
| *** stevemar has quit IRC | 23:23 | |
| openstackgerrit | Merged openstack-infra/project-config: move 'gate-{name}-chef-rake-integration' job https://review.openstack.org/194181 | 23:23 |
| openstackgerrit | Merged openstack-infra/infra-manual: Refresh 'Put Your New Project in the Correct Project Group' section https://review.openstack.org/189468 | 23:23 |
| anteaya | mmedvede: I don't want to -1 it but would really like to see asselin's suggestions included | 23:23 |
| jeblair | jenkins@template-bare-trusty-1442931268:~/workspace/gate-refstack-npm-run-lint$ npm lint --silent | 23:23 |
| jeblair | jenkins@template-bare-trusty-1442931268:~/workspace/gate-refstack-npm-run-lint$ echo $? | 23:23 |
| jeblair | 0 | 23:23 |
| jeblair | krotscheck: ^ | 23:24 |
| openstackgerrit | Merged openstack-infra/project-config: Make gate-python-ironic-inspector-client-dsvm voting https://review.openstack.org/224476 | 23:24 |
| krotscheck | jeblair: .... `npm --version`? | 23:24 |
| jeblair | 1.3.10 | 23:24 |
| krotscheck | hookay | 23:24 |
| openstackgerrit | Merged openstack-infra/project-config: Update ACLs for ooi project https://review.openstack.org/216596 | 23:24 |
| krotscheck | lemme go test that one | 23:24 |
| openstackgerrit | Merged openstack-infra/project-config: Update ooi job definitions https://review.openstack.org/216597 | 23:24 |
| jeblair | i think that's in the log too: | 23:24 |
| jeblair | 2015-09-22 23:22:45.568 | npm@1.3.10 /usr/share/npm | 23:24 |
| *** dramalho has quit IRC | 23:24 | |
| *** salv-orlando has quit IRC | 23:25 | |
| *** gildub has joined #openstack-infra | 23:25 | |
| jeblair | krotscheck: anythinge else before i release the node? | 23:25 |
| krotscheck | jeblair: Nothing I can think of, thanks | 23:26 |
| jeblair | krotscheck: np, gl. | 23:26 |
| openstackgerrit | Merged openstack-infra/project-config: Add experimental devstack gate job for tacker https://review.openstack.org/226059 | 23:27 |
| openstackgerrit | Merged openstack-infra/project-config: Enable barbican plugin to Magnum local config https://review.openstack.org/223923 | 23:27 |
| *** Sukhdev has joined #openstack-infra | 23:28 | |
| *** zz_dimtruck is now known as dimtruck | 23:29 | |
| openstackgerrit | David Shrewsbury proposed openstack-infra/shade: Handle glance v1 and v2 difference with is_public https://review.openstack.org/226492 | 23:30 |
| *** darrenc is now known as darrenc_afk | 23:30 | |
| *** Sukhdev has quit IRC | 23:32 | |
| *** stevemar has joined #openstack-infra | 23:32 | |
| *** Sukhdev has joined #openstack-infra | 23:33 | |
| *** ashleigh_ has quit IRC | 23:34 | |
| *** stevemar has quit IRC | 23:36 | |
| *** markvoelker has quit IRC | 23:40 | |
| krotscheck | jeblair, Clint: Turns out npm@1.x returns zero when incorrect commandline parameters are passed. 2.x+ (correctly) returns 1. HELPFUL. | 23:41 |
| *** larainema has joined #openstack-infra | 23:41 | |
| jeblair | 1.x is so friendly | 23:41 |
| pvaneck | krotscheck: was looking as well. seems that https://github.com/npm/npm/releases/tag/v1.4.8 is where it started returning exit code 1 | 23:42 |
| jeblair | krotscheck: maybe you can tell them apart with "npm invalidcommand"? | 23:42 |
| jeblair | "npm invalidcommand && npm lint || npm run lint" | 23:43 |
| krotscheck | jeblair: Wouldn't that just result in 0 && 0 ? | 23:43 |
| krotscheck | (lint is not a valid lifecycle command) | 23:44 |
| *** ivar-lazzaro has quit IRC | 23:44 | |
| openstackgerrit | Michael Krotscheck proposed openstack-infra/project-config: Updated npm-run macro to support npm 1.x https://review.openstack.org/226596 | 23:45 |
| krotscheck | jeblair, pvaneck: Well, I'm opting for this approach instead ^^ | 23:45 |
| jeblair | krotscheck: oh i see | 23:45 |
| *** darrenc_afk is now known as darrenc | 23:46 | |
| krotscheck | oops. Zombie shebang | 23:46 |
| Clint | krotscheck: fantastic | 23:46 |
| openstackgerrit | Michael Krotscheck proposed openstack-infra/project-config: Updated npm-run macro to support npm 1.x https://review.openstack.org/226596 | 23:47 |
| *** otter768 has joined #openstack-infra | 23:47 | |
| krotscheck | Clint: Just think of it like this: With code like that, javascript's still got a ways to go before it takes over the world (thus making you have to learn it ;) ) | 23:48 |
| krotscheck | You're saved! :) | 23:48 |
| Clint | krotscheck: :P | 23:53 |
| *** ivar-lazzaro has joined #openstack-infra | 23:54 | |
| crinkle | nibalizer: wait I'm confused again - we can have two builders but where would they get called? the new one needs to be run in system-config but system-config can't run both jobs, one would always fail http://git.openstack.org/cgit/openstack-infra/project-config/tree/jenkins/jobs/projects.yaml#n4894 | 23:54 |
| *** dims has joined #openstack-infra | 23:54 | |
| *** dims has quit IRC | 23:55 | |
| *** Guest18166 has joined #openstack-infra | 23:55 | |
| nibalizer | i assumed we'd have a new job called 'gate-infra-cloud-puppet-apply' or somethign that would be in that list | 23:55 |
| nibalizer | since infra cloud apply test doesn't have to play games with multiple operatingsystems | 23:57 |
| crinkle | nibalizer: so what would happen then on the other nodes? they would fail that job | 23:57 |
| nibalizer | gate-infra-cloud-puppet-apply should only have to run on trusty right? | 23:58 |
| openstackgerrit | James Slagle proposed openstack-infra/tripleo-ci: Use tripleo.sh to drive ci https://review.openstack.org/225776 | 23:58 |
| crinkle | nibalizer: oh - I think I'm overcomplicating it | 23:58 |
| crinkle | nibalizer: nm, ty | 23:59 |
| nibalizer | sure | 23:59 |
| *** tonytan4ever has joined #openstack-infra | 23:59 | |
| nibalizer | I can spin up an exapmle project-config change if you'd like me to show you what I mean | 23:59 |
| nibalizer | then we can change the apply-test.sh file until it can do what my change needs it to haha | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!