| *** msabramo has quit IRC | 00:00 | |
| fungi | jesusaurus: i already have it rebased locally and ready to push up if you don't care | 00:01 |
|---|---|---|
| *** msabramo has joined #openstack-infra | 00:01 | |
| openstackgerrit | K Jonathan Harker proposed a change to openstack-infra/config: Run the puppet apply test (requires sudo) https://review.openstack.org/93950 | 00:02 |
| jesusaurus | oh, meh | 00:02 |
| fungi | wfm | 00:02 |
| *** mriedem has joined #openstack-infra | 00:02 | |
| fungi | just the couple of additional hiera entries on the zuul server in the global site manifest was all i saw conflicting | 00:03 |
| jesusaurus | yeah, thats what i saw too | 00:03 |
| jesusaurus | the new swift entries | 00:03 |
| notmyname | something I need to pay attention to? | 00:04 |
| notmyname | /nocontext | 00:04 |
| *** unicell has joined #openstack-infra | 00:04 | |
| *** jp_at_hp has quit IRC | 00:04 | |
| *** jp_at_hp1 has joined #openstack-infra | 00:04 | |
| fungi | notmyname: nah, just rebasing things on top of recent additions to zuul so it can start interfacing with swift containers | 00:04 |
| notmyname | cool | 00:05 |
| fungi | very cool ;) | 00:05 |
| *** jp_at_hp1 has quit IRC | 00:05 | |
| notmyname | now that y'all are using swift (which is totally awesome!!), I can't just assume I need to pay attention when you mention swift ;-) | 00:05 |
| fungi | jhesketh will be swiftifying all the things now | 00:05 |
| *** jp_at_hp has joined #openstack-infra | 00:06 | |
| jhesketh | heh, over time :-) | 00:06 |
| notmyname | yay | 00:06 |
| jhesketh | jeblair: with the ironic non-voting job stuff... what if the reporter could be configured to take a user argument. That way we'd just create a new pipeline for 'probational jobs' or something | 00:06 |
| Alex_Gaynor | I assume we're moving log files to swift, or something else? | 00:07 |
| fungi | jesusaurus: i'm cool approving 93950 (since clarkb already +2'd) if you have time to keep an eye on http://puppetboard.openstack.org/ for any serious resulting fallout. if not, it can survive another day molding in review | 00:07 |
| jesusaurus | fungi: lets do it | 00:07 |
| fungi | Alex_Gaynor: that's the idea. job artifacts landing in swift containers so we don't have to maintain a vm with a 12+tb cinder aggregate under lvm | 00:07 |
| jeblair | notmyname: you should totally pay attention even more when we say swift! you'll be able to help us solve all the problems! | 00:07 |
| notmyname | jeblair: I do what I can :-) | 00:08 |
| Alex_Gaynor | fungi: 12tb of cinder under lvm definitely sounds worse than swift :-) | 00:08 |
| notmyname | trust me. I notice every time "swift" is mentioned in -infra | 00:08 |
| fungi | Alex_Gaynor: i'm going to quote you on that | 00:08 |
| Alex_Gaynor | :-) | 00:08 |
| jeblair | jhesketh: yeah, that's along the general lines of what i was thinking | 00:08 |
| Alex_Gaynor | fungi: is this going to result in hate mail from the cinder developers ;-) | 00:08 |
| notmyname | jeblair: I want you to be happy swift users :-) | 00:08 |
| jhesketh | jeblair: I can poke at that for you if you'd like | 00:09 |
| *** julim has quit IRC | 00:09 | |
| jeblair | jhesketh: cool, i think devananda would appreciate that! may want to follow up on the thread and also give it another day of exposure in case anyone else chimes in (since part of the idea was to make sure the idea was sufficiently exposed). | 00:10 |
| fungi | Alex_Gaynor: i'm sure the cinder developers never quite intended it to be consumed in the way we have been ;) | 00:10 |
| jeblair | fungi: i think the way we're consuming it is awesome, actually... | 00:11 |
| jhesketh | jeblair: yep, will do | 00:11 |
| jeblair | fungi: the only downside is the per-server volume max | 00:11 |
| fungi | jeblair: true, but there must be scalability/cost issues at the provider end for that to become a factor | 00:11 |
| Alex_Gaynor | this is totally what blob storage is for | 00:11 |
| fungi | jeblair: well, and the outdated xen virtual block device count limit | 00:12 |
| jeblair | fungi: yeah, that's what i was referring to | 00:12 |
| fungi | jeblair: perhaps the cinder devs expect providers to run not quite such limited hypervisors ;) | 00:12 |
| openstackgerrit | A change was merged to openstack-infra/config: Run the puppet apply test (requires sudo) https://review.openstack.org/93950 | 00:13 |
| fungi | jesusaurus: ^ let the puppetboard watching commence | 00:13 |
| jeblair | fungi: but other than that, we're able to scale out filesystems to a significant degree, mostly automated, and quickly | 00:13 |
| *** msabramo has quit IRC | 00:13 | |
| clarkb | whats with ironic job reporters? | 00:13 |
| fungi | jeblair: very true. and reattach them to new systems in theory | 00:13 |
| fungi | clarkb: jobs run upstream not voting as "strongly" ans usual upstream gate jobs | 00:14 |
| jeblair | clarkb: see thread on -dev list | 00:14 |
| fungi | s/and/as/ | 00:14 |
| clarkb | oh third party with zuul | 00:14 |
| fungi | "second party" | 00:14 |
| jeblair | clarkb: yeah, more or less. i'm proposing we call it 'non-binding' | 00:14 |
| fungi | non-binding sounds better | 00:15 |
| fungi | and still plenty vague | 00:15 |
| jeblair | (also better than 'voting non-voting') | 00:15 |
| fungi | "footgun voting" | 00:15 |
| jeblair | heh | 00:16 |
| *** arunkant has quit IRC | 00:16 | |
| fungi | to borrow a lifeless term | 00:16 |
| *** lcheng_ has quit IRC | 00:19 | |
| *** lcheng_ has joined #openstack-infra | 00:19 | |
| *** isviridov has quit IRC | 00:20 | |
| *** msabramo has joined #openstack-infra | 00:22 | |
| *** matsuhashi has joined #openstack-infra | 00:22 | |
| *** isviridov has joined #openstack-infra | 00:22 | |
| *** rpwwpr has joined #openstack-infra | 00:23 | |
| *** praneshp has quit IRC | 00:24 | |
| *** sunrenjie6 has joined #openstack-infra | 00:30 | |
| *** rpwwpr has quit IRC | 00:30 | |
| openstackgerrit | Ian Wienand proposed a change to openstack-infra/config: Add experimental ceilometer/f20/mongodb test https://review.openstack.org/96316 | 00:30 |
| openstackgerrit | Ian Wienand proposed a change to openstack-infra/devstack-gate: Add option for ceilometer mongodb back-end https://review.openstack.org/96317 | 00:32 |
| openstackgerrit | Ian Wienand proposed a change to openstack-infra/devstack-gate: [WIP] Add option for ceilometer mongodb back-end https://review.openstack.org/96317 | 00:33 |
| openstackgerrit | Ian Wienand proposed a change to openstack-infra/config: [WIP] Add experimental ceilometer/f20/mongodb test https://review.openstack.org/96316 | 00:33 |
| notmyname | oh, fun. git 2.0 | 00:34 |
| fungi | twice the fun of git one! | 00:34 |
| *** melwitt has quit IRC | 00:34 | |
| fungi | the release notes are compelling | 00:35 |
| jesusaurus | yay, first run of the new -infra/config test | 00:35 |
| fungi | jesusaurus: no surprises? | 00:35 |
| jesusaurus | fungi: nope, just an expected failure | 00:36 |
| *** wenlock has quit IRC | 00:37 | |
| *** chuckC has quit IRC | 00:38 | |
| *** gokrokve has joined #openstack-infra | 00:38 | |
| fungi | i see nothing in the git 2.0 release notes which will hinder (for for that matter probably significantly benefit) any of my current use cases | 00:39 |
| fungi | s/for for/or for/ | 00:40 |
| *** yamahata has joined #openstack-infra | 00:41 | |
| *** chuckC has joined #openstack-infra | 00:42 | |
| *** gokrokve has quit IRC | 00:42 | |
| *** lcheng_ has quit IRC | 00:43 | |
| *** lcostantino has quit IRC | 00:44 | |
| openstackgerrit | K Jonathan Harker proposed a change to openstack-infra/config: Only include Class[selinux] on rhel-based systems https://review.openstack.org/96319 | 00:45 |
| fungi | jesusaurus: was that ^ the only obvious gap? | 00:45 |
| fungi | i guess the test results will tell us | 00:46 |
| *** crc32 has quit IRC | 00:47 | |
| *** chuckC has quit IRC | 00:47 | |
| *** lttrl has joined #openstack-infra | 00:48 | |
| jesusaurus | fungi: well, i also realized we are only testing on bare-precise | 00:50 |
| jesusaurus | so i'm goind to expand it to a second job on bare-centos6 | 00:50 |
| *** nati_ueno has quit IRC | 00:50 | |
| fungi | jesusaurus: good idea | 00:51 |
| fungi | jesusaurus: though the gap analysis there will turn up... lots | 00:51 |
| jesusaurus | im sure it will :) | 00:51 |
| fungi | there are almost certainly still numerous assumptive ubuntuisms in our puppetry | 00:52 |
| fungi | Could not find value for 'jenkins_dev_api_user' at 38:/home/jenkins/workspace/gate-config-puppet-apply/modules/openstack_project/templates/nodepool/nodepool-dev.yaml.erb | 00:53 |
| fungi | guess there's still work to do there too | 00:53 |
| openstackgerrit | K Jonathan Harker proposed a change to openstack-infra/config: Also run puppet-apply test on bare-centos6 https://review.openstack.org/96320 | 00:55 |
| jesusaurus | fungi: it doesn't look like anything caught fire, so I'm going to wander home and get some food | 00:56 |
| fungi | jesusaurus: yep, seems safe. have a great evening! | 00:56 |
| jesusaurus | thanks, you too | 00:56 |
| *** JayF has quit IRC | 00:58 | |
| *** fifieldt has joined #openstack-infra | 01:00 | |
| *** JayF has joined #openstack-infra | 01:03 | |
| *** BadCub has joined #openstack-infra | 01:05 | |
| *** mburned is now known as mburned_out | 01:06 | |
| pleia2 | clarkb: thanks re: 89928 | 01:07 |
| *** BadCub has quit IRC | 01:09 | |
| *** doug-fish has left #openstack-infra | 01:09 | |
| *** wenlock has joined #openstack-infra | 01:13 | |
| *** doug-fish1 has joined #openstack-infra | 01:14 | |
| *** doug-fish1 has left #openstack-infra | 01:14 | |
| *** wenlock has quit IRC | 01:17 | |
| *** otter768 has quit IRC | 01:18 | |
| *** otter768 has joined #openstack-infra | 01:19 | |
| *** otter768 has quit IRC | 01:20 | |
| *** otter768 has joined #openstack-infra | 01:20 | |
| *** jhesketh has quit IRC | 01:21 | |
| *** nati_ueno has joined #openstack-infra | 01:21 | |
| *** signed8bit has quit IRC | 01:22 | |
| clarkb | zaro: were you planning on responding to spearce's question? | 01:22 |
| clarkb | zaro: because either they are only talking about making google openid not work anymore or they have gone off the deep end | 01:23 |
| *** marcoemorais has quit IRC | 01:23 | |
| clarkb | zaro: and I really don't want to have to deal with deep end no openid in gerrit gerrit when we next upgrade | 01:23 |
| *** jp_at_hp has quit IRC | 01:24 | |
| *** otter768 has quit IRC | 01:27 | |
| *** yjiang has joined #openstack-infra | 01:27 | |
| *** signed8bit has joined #openstack-infra | 01:27 | |
| *** otter768 has joined #openstack-infra | 01:29 | |
| *** nati_ueno has quit IRC | 01:32 | |
| *** Ryan_Lane has quit IRC | 01:33 | |
| *** chuckC has joined #openstack-infra | 01:34 | |
| *** jhesketh has joined #openstack-infra | 01:37 | |
| *** mmaglana has joined #openstack-infra | 01:40 | |
| *** gyee has quit IRC | 01:41 | |
| *** nati_ueno has joined #openstack-infra | 01:42 | |
| *** nati_ueno has quit IRC | 01:42 | |
| *** aconrad has quit IRC | 01:42 | |
| *** yamahata has quit IRC | 01:43 | |
| *** yamahata has joined #openstack-infra | 01:43 | |
| *** nosnos has joined #openstack-infra | 01:45 | |
| *** zz_gondoi is now known as gondoi | 01:45 | |
| *** gokrokve has joined #openstack-infra | 01:51 | |
| *** mmaglana has quit IRC | 01:53 | |
| *** mmaglana has joined #openstack-infra | 01:54 | |
| *** gyee has joined #openstack-infra | 01:54 | |
| *** weshay has joined #openstack-infra | 01:58 | |
| *** mmaglana has quit IRC | 01:58 | |
| *** yaguang has joined #openstack-infra | 02:00 | |
| *** weshay has quit IRC | 02:00 | |
| *** wenlock has joined #openstack-infra | 02:02 | |
| *** bhuvan has quit IRC | 02:07 | |
| openstackgerrit | Matt Ray proposed a change to openstack-infra/config: New Chef cookbooks for client services, Sahara and Trove. https://review.openstack.org/96332 | 02:07 |
| *** signed8bit has quit IRC | 02:12 | |
| *** mbacchi has quit IRC | 02:16 | |
| *** arnaud has quit IRC | 02:16 | |
| *** arnaud has joined #openstack-infra | 02:16 | |
| *** signed8bit has joined #openstack-infra | 02:22 | |
| *** signed8bit has quit IRC | 02:26 | |
| *** arnaud has quit IRC | 02:28 | |
| *** dims has quit IRC | 02:29 | |
| openstackgerrit | Tom Fifield proposed a change to openstack-infra/config: Add Stackforge project for training-guides https://review.openstack.org/96334 | 02:33 |
| *** zhiyan_ is now known as zhiyan | 02:34 | |
| *** Ryan_Lane has joined #openstack-infra | 02:36 | |
| *** face has joined #openstack-infra | 02:37 | |
| *** face is now known as alexandra | 02:38 | |
| *** gondoi is now known as zz_gondoi | 02:41 | |
| *** gokrokve_ has joined #openstack-infra | 02:42 | |
| *** gokrokve has quit IRC | 02:45 | |
| openstackgerrit | Steve Kowalik proposed a change to openstack-infra/config: Add docs job for os-cloud-config https://review.openstack.org/96335 | 02:46 |
| openstackgerrit | Dan Bode proposed a change to openstack-infra/config: Allow modules to be installed from source https://review.openstack.org/96336 | 02:50 |
| bodepd | nibalizer: your puppetboard module is installing version 2.3.0, but no tag exists for it. | 02:52 |
| bodepd | nibalizer: can you add one? | 02:52 |
| nibalizer | 'tag'? | 02:56 |
| nibalizer | git tag? | 02:56 |
| *** marcoemorais has joined #openstack-infra | 02:57 | |
| *** matsuhashi has quit IRC | 03:00 | |
| openstackgerrit | Tom Fifield proposed a change to openstack-infra/config: Add Stackforge project for training-guides https://review.openstack.org/96334 | 03:01 |
| *** matsuhashi has joined #openstack-infra | 03:04 | |
| *** BadCub has joined #openstack-infra | 03:06 | |
| *** W00dy__ has joined #openstack-infra | 03:09 | |
| *** BadCub has quit IRC | 03:10 | |
| openstackgerrit | Tom Fifield proposed a change to openstack-infra/config: Add Stackforge project for training-guides https://review.openstack.org/96334 | 03:10 |
| *** talluri has joined #openstack-infra | 03:14 | |
| openstackgerrit | Tom Fifield proposed a change to openstack-infra/config: Add Stackforge project for training-guides https://review.openstack.org/96334 | 03:14 |
| *** esker has quit IRC | 03:15 | |
| *** esker has joined #openstack-infra | 03:15 | |
| *** mriedem has quit IRC | 03:21 | |
| *** wenlock_ has joined #openstack-infra | 03:23 | |
| *** gokrokve_ has quit IRC | 03:25 | |
| *** Ryan_Lane has quit IRC | 03:25 | |
| *** plars has quit IRC | 03:27 | |
| *** plars has joined #openstack-infra | 03:29 | |
| mordred | clarkb: k | 03:30 |
| cody-somerville | ttx: Where there are lines and where there isn't is confusing in the table on https://wiki.openstack.org/wiki/Releases | 03:31 |
| * StevenK tries to work how jobs are defined when grepping for the job name returns nothing | 03:32 | |
| *** nosnos has quit IRC | 03:33 | |
| *** medieval1 has quit IRC | 03:33 | |
| mordred | StevenK: there are some templated job names | 03:34 |
| *** matsuhashi has quit IRC | 03:34 | |
| mordred | StevenK: what sorts of job are you looking for? | 03:34 |
| cody-somerville | https://wiki.openstack.org/wiki/StableBranch says stable releases happen about every eight weeks. Is that actually true? | 03:35 |
| mordred | cody-somerville: yah | 03:35 |
| *** matsuhashi has joined #openstack-infra | 03:35 | |
| StevenK | mordred: https://review.openstack.org/#/c/96335/ and the gate-config-layout failure | 03:36 |
| mordred | StevenK: you want to look in : modules/openstack_project/files/jenkins_job_builder/config/openstack-publish-jobs.yaml | 03:38 |
| mordred | StevenK: '{name}-docs' | 03:38 |
| mordred | StevenK: openstack-publish-jobs | 03:38 |
| *** matsuhashi has quit IRC | 03:39 | |
| mordred | StevenK: you need to make sure that - openstack-publish-jobs is in your jobs: section in modules/openstack_project/files/jenkins_job_builder/config/projects.yaml | 03:40 |
| StevenK | mordred: Yeah, I just found that | 03:40 |
| openstackgerrit | Steve Kowalik proposed a change to openstack-infra/config: Add docs job for os-cloud-config https://review.openstack.org/96335 | 03:42 |
| *** zhiyan is now known as zhiyan_ | 03:47 | |
| *** marcoemorais has quit IRC | 03:50 | |
| openstackgerrit | Steve Kowalik proposed a change to openstack-infra/config: Add docs job for os-cloud-config https://review.openstack.org/96335 | 03:50 |
| *** matsuhashi has joined #openstack-infra | 03:56 | |
| *** pcrews has quit IRC | 03:58 | |
| *** medieval1 has joined #openstack-infra | 04:03 | |
| *** yfried has quit IRC | 04:05 | |
| *** zhiyan_ is now known as zhiyan | 04:10 | |
| *** lcheng_ has joined #openstack-infra | 04:16 | |
| *** renlt has joined #openstack-infra | 04:18 | |
| *** ctracey has joined #openstack-infra | 04:31 | |
| ctracey | hello all. Quick question here. I know I had seen this before, but I cant seem to find it again. Somewhere I saw a list of pip dependencies pulled in during each build...they were posted (i think) on review.openstack.org somewhere. Would anyone happen to know where this is? | 04:33 |
| *** Ryan_Lane has joined #openstack-infra | 04:36 | |
| *** harlowja_ is now known as harlowja_away | 04:38 | |
| ianw | ctracey: do you mean https://git.openstack.org/cgit/openstack/requirements/tree/ | 04:40 |
| *** Ryan_Lane has quit IRC | 04:40 | |
| ctracey | ianw: no unfortunately | 04:40 |
| ctracey | after a build pip freeze of the hard list of requirements from a build | 04:41 |
| ctracey | with specific versions | 04:41 |
| zaro | clarkb: i'm not real familiar with all the OAuth implementations out there. looks like there were some additional feedback to the thread. luca created a plugin for github OAuth.. https://gerrit-review.googlesource.com/#/c/57570 | 04:43 |
| zaro | clarkb: i'm not sure which one open openstack is setup with so i'm not sure if that would work or not. | 04:44 |
| *** nosnos has joined #openstack-infra | 04:45 | |
| *** dangers is now known as dangers_away | 04:49 | |
| clarkb | I don't think it will work as we use openid | 04:49 |
| clarkb | zaro: https://git.openstack.org/cgit/openstack-infra/config/tree/modules/gerrit/templates/gerrit.config.erb#n24 is what we need to continue working | 04:49 |
| *** alexandra has quit IRC | 04:50 | |
| wenlock_ | clarkb zaro: are you guys planning on switching auth providers soon? | 04:50 |
| clarkb | wenlock_: nope, but upstream gerrit is talking about removing oepnid support | 04:50 |
| clarkb | because google no longer supports openid | 04:50 |
| clarkb | but gerrit forgets that google isn't the only fish in the pond | 04:51 |
| wenlock_ | ahhh, wow, that would stink up a big one | 04:51 |
| clarkb | its a bit annoying. The comment from spearce was "Why shouldn't we remove openid support, google doesn't support it anymore." or some such | 04:52 |
| clarkb | its a very narrow world view | 04:52 |
| wenlock_ | it's sill tbh, why don't they just add the oauth support they broke back, and move on | 04:52 |
| *** zhiyan is now known as zhiyan_ | 04:52 | |
| wenlock_ | clarkb, yikes | 04:52 |
| wenlock_ | so if they drop openid, does that mean they drop other auth mechanisms they would have? like ldap | 04:53 |
| *** arnaud has joined #openstack-infra | 04:53 | |
| clarkb | I don't think ldap is going away | 04:54 |
| mordred | clarkb: uhm. | 04:56 |
| mordred | WHAT? | 04:56 |
| *** aconrad has joined #openstack-infra | 04:56 | |
| *** sweston has joined #openstack-infra | 04:56 | |
| *** sweston has quit IRC | 04:57 | |
| mordred | why would they remove the auth that's used by the largest and most active gerrit installation in the world? | 04:57 |
| openstackgerrit | lifeless proposed a change to openstack-infra/elastic-recheck: Add a recheck rue for ug 1324359 https://review.openstack.org/96351 | 04:57 |
| clarkb | mordred: beacuse they hate you | 04:57 |
| clarkb | but really more beacuse they can't seem to udnersdtand that there is a bigger world outside of google | 04:58 |
| lifeless | google, the company that removed jabber support from their chat system... detect a theme ? | 04:58 |
| clarkb | mordred: I will find you a link to the thread | 04:58 |
| lifeless | </bitter> | 04:58 |
| clarkb | https://groups.google.com/forum/#!topic/repo-discuss/Hjn-6BV3KBU there you go | 04:58 |
| ctracey | mordred or clarkb would you happen to know about my question above? I know mordred knows quite a bit about the pip dependencies issue :) | 04:59 |
| StevenK | "Yes, maybe. The GitHub auth stuff may be interesting to get into Gerrit core. Everyone's got a GitHub account." | 04:59 |
| StevenK | GRRRRRR | 04:59 |
| clarkb | ctracey: they are posted in the build log for that specific test run | 05:00 |
| clarkb | ctracey: so if you find a change on review.openstack.org you can follow the test result links to the listings | 05:00 |
| ctracey | clarkb: yeah i know that, but i swear i saw them broken out from the build logs someplace | 05:00 |
| clarkb | ctracey: each project has a requirements.txt and test-requirements.txt | 05:01 |
| clarkb | which won't be as specific as the pip freeze output as transitive deps won't be included | 05:01 |
| ctracey | clarkb: yeah, i am specifically looking for the =='s :) | 05:01 |
| clarkb | and they don't always list specific versions | 05:01 |
| openstackgerrit | lifeless proposed a change to openstack-infra/elastic-recheck: Add a recheck rule for bug 1324359 https://review.openstack.org/96351 | 05:01 |
| uvirtbot | Launchpad bug 1324359 in tripleo "Ironic seed sometimes disappears mid-initialisation" [Medium,Triaged] https://launchpad.net/bugs/1324359 | 05:01 |
| ctracey | need a better way to build packages and seems logical to pull specific "known goods" for a specific git sha | 05:02 |
| *** mmaglana has joined #openstack-infra | 05:03 | |
| *** oomichi has quit IRC | 05:03 | |
| ctracey | or perhaps I am thinking of something else I may have seen...is there a place where we link openstack components by SHA? like nova 0xdeadbeef works with keystone 0xbaadf00d | 05:04 |
| ctracey | that may be what I had seen | 05:04 |
| clarkb | ctracey: we do have that. https://git.openstack.org/cgit/openstack/openstack | 05:05 |
| lifeless | there is a meta-repo | 05:05 |
| lifeless | that one | 05:05 |
| clarkb | the openstack/openstack repo is full of submodules automagically updated by gerrit when things merge | 05:05 |
| clarkb | so you get an order or tested merged changes | 05:05 |
| *** wenlock has quit IRC | 05:06 | |
| ctracey | clarkb: thanks! | 05:06 |
| *** oomichi has joined #openstack-infra | 05:06 | |
| *** BadCub has joined #openstack-infra | 05:07 | |
| ctracey | this isnt what i had seen back in the day, but this is very good info | 05:07 |
| wenlock_ | we had alot of problems this last week with google openid...:< it simply stopped working because they cut off new domain registrations. we switched back to defaulting to launchpad quickly after. | 05:07 |
| wenlock_ | we had ended it with asking them to support the new oauth, but im surprised they'd take it a step forward to just drop openid | 05:10 |
| *** BadCub has quit IRC | 05:11 | |
| *** wenlock has joined #openstack-infra | 05:14 | |
| *** wenlock has quit IRC | 05:16 | |
| *** wenlock_ has quit IRC | 05:17 | |
| *** wenlock has joined #openstack-infra | 05:17 | |
| *** aconrad has quit IRC | 05:18 | |
| nibalizer | bodepd: producing a 2.3.0 release proved difficult | 05:19 |
| *** mmaglana has quit IRC | 05:19 | |
| nibalizer | since i dunno when exactly I cut that | 05:19 |
| nibalizer | but i made a 2.4.0 for you which is a super minor list of stuff and we can just pop in | 05:19 |
| nibalizer | and github wouldn't tell you, but i gpg signed my tag, so you *know* its legit | 05:20 |
| *** lcheng_ has quit IRC | 05:25 | |
| *** yfried has joined #openstack-infra | 05:28 | |
| *** zhiyan_ is now known as zhiyan | 05:29 | |
| openstackgerrit | lifeless proposed a change to openstack-infra/elastic-recheck: Add a recheck rule for TripleO bug 1324359 https://review.openstack.org/96351 | 05:30 |
| uvirtbot | Launchpad bug 1324359 in tripleo "Ironic seed sometimes disappears mid-initialisation" [Medium,Triaged] https://launchpad.net/bugs/1324359 | 05:30 |
| yjiang | hi,did anyone have met this error "jenkins.JenkinsException: create[dsvm-tempest-full] failed" when use JJB to update a project? | 05:30 |
| bodepd | nibalizer: yeah, the forge sucks like that | 05:31 |
| *** esker has quit IRC | 05:31 | |
| bodepd | cool, I'll just make a note in my patch about that | 05:31 |
| bodepd | only one thing I did not like about my patch is the pushd/popd | 05:31 |
| *** esker has joined #openstack-infra | 05:32 | |
| bodepd | I tried to have git refer to a project for rev-parse where it is not the $cwd/.git | 05:32 |
| bodepd | to no avail | 05:32 |
| mordred | clarkb, zaro: ok. I responded to the list saying that it would hurt us if they removed openid | 05:33 |
| nibalizer | bodepd: so github makes tarballs directly available for all tags | 05:34 |
| nibalizer | but i think the point is that we're moving stuff to stackforge | 05:34 |
| nibalizer | i think i got a puppet-storyboard repo created so we can pull that one out first | 05:34 |
| mordred | nibalizer: we make tarballs for all tags in stackforge too :) | 05:35 |
| *** Longgeek has joined #openstack-infra | 05:35 | |
| nibalizer | well then heyyyyyy | 05:35 |
| *** W00dy__ has quit IRC | 05:35 | |
| *** esker has quit IRC | 05:36 | |
| openstackgerrit | lifeless proposed a change to openstack-infra/config: Make the HP TripleO region clearly that. https://review.openstack.org/96354 | 05:36 |
| *** marcoemorais has joined #openstack-infra | 05:37 | |
| *** marcoemorais1 has joined #openstack-infra | 05:39 | |
| openstackgerrit | Joshua Hesketh proposed a change to openstack-infra/nodepool: Make template and node hostnames configurable https://review.openstack.org/90883 | 05:39 |
| *** face has joined #openstack-infra | 05:39 | |
| *** face has quit IRC | 05:40 | |
| *** marcoemorais has quit IRC | 05:41 | |
| bodepd | nibalizer: it's so much easier to programatically use git over downloading tarballs | 05:46 |
| bodepd | nibalizer: that way, you can support branches, tags, or 'master' | 05:46 |
| bodepd | nibalizer mordred is there a preference over downloaded tarballs vs. tags/revisions in github? | 05:47 |
| *** lcheng_ has joined #openstack-infra | 05:48 | |
| *** pcrews has joined #openstack-infra | 05:49 | |
| mordred | bodepd: no | 05:53 |
| * jesusaurus prefers tags | 05:53 | |
| mordred | bodepd: we drive all tarball creation aroudn here from git tags | 05:53 |
| mordred | so we'll _always_ have a tag | 05:53 |
| mordred | if tarballs work better, they'll be there - if not, the git refs will be ther | 05:53 |
| *** dstanek is now known as dstanek_zzz | 05:58 | |
| krotscheck | nibalizer: It’s more-or-less a new one. | 06:00 |
| krotscheck | nibalizer: I’ve learned a LOT about puppet since. | 06:00 |
| nibalizer | krotscheck: AWESMOE | 06:00 |
| nibalizer | i luv puppet | 06:00 |
| nibalizer | bodepd: i was suggesting wget tarballs as a way forward that might not involve a git clone followed by shenanigans | 06:01 |
| nibalizer | since you were bemoaning pushd/popd | 06:01 |
| nibalizer | but i also didn't get that | 06:01 |
| krotscheck | nibalizer: There are bits that need to go in there for infra- load projects, load superusers, etc. | 06:01 |
| nibalizer | since i <3 pushd/popd | 06:01 |
| krotscheck | nibalizer: But I can work on those as we go. | 06:02 |
| nibalizer | krotscheck: so, what I'd like to do in general, and this is a big conversation, is to generally direct copy the module out of infra/config and into openstack-infra/modulename | 06:02 |
| nibalizer | that way we know it works | 06:02 |
| nibalizer | and can move quickly to bust out modules | 06:02 |
| krotscheck | nibalizer: That makes sense. Was that communicated anywhere? | 06:03 |
| krotscheck | nibalizer: I.e. It would have been nice to know that before I did all that work. | 06:03 |
| nibalizer | what you've done, which is make a new one, a) destroys the git history of the module in the openstack-infra/config and b) means we dont know how it will play with the other modules | 06:03 |
| nibalizer | krotscheck: i didn't communicate that | 06:03 |
| krotscheck | nibalizer: Ok, so, what, I should abandon my patch? | 06:03 |
| * nibalizer is still learning how to communicate with this project and, you should be clear, that I am not an authority 'round these parts | 06:03 | |
| nibalizer | i am meerly an enthusiastic contributor | 06:04 |
| nibalizer | so me sayin 'what id like to do' doesn't mean 'we are doing this' | 06:04 |
| mordred | nibalizer: we COULD port krotscheck's changes into the config repo, and then split the results of that back out | 06:04 |
| nibalizer | ya! | 06:04 |
| mordred | which might get us both krotscheck's work, but also the stuff you were talking about too | 06:04 |
| mordred | and then we can do better about communicating a general plan/approach for the module splits moving forward? | 06:05 |
| krotscheck | I’ll take a look at what’s in /config tomorrow and see how hard a reconciliation would be. | 06:05 |
| mordred | krotscheck: where's your current work? | 06:05 |
| krotscheck | mordred: https://review.openstack.org/#/c/96290/ | 06:06 |
| nibalizer | ya sounds like we should cook up an etherpad or a spec | 06:06 |
| mordred | neat | 06:06 |
| nibalizer | also very soon it will be easier for me to attend the -infra meeting | 06:07 |
| nibalizer | which will help i think | 06:07 |
| krotscheck | mordred: What timezone are you in right now? | 06:07 |
| *** arnaud has quit IRC | 06:07 | |
| *** morganfainberg is now known as stevemar2 | 06:09 | |
| jesusaurus | nibalizer: ++ to a spec | 06:09 |
| mordred | krotscheck: pacific - I have no idea why I'm hacking on things | 06:10 |
| *** stevemar2 is now known as morganfainberg | 06:10 | |
| krotscheck | mordred: Because your days are filled with meetings? | 06:10 |
| krotscheck | nibalizer: What’s changing to make your -infra easier? | 06:11 |
| nibalizer | krotscheck: life changes | 06:11 |
| openstackgerrit | Monty Taylor proposed a change to openstack-infra/config: Use ansible instead of direct ssh calls https://review.openstack.org/87821 | 06:11 |
| openstackgerrit | Monty Taylor proposed a change to openstack-infra/config: Remove salt https://review.openstack.org/96361 | 06:11 |
| openstackgerrit | Monty Taylor proposed a change to openstack-infra/config: Remove old salt references from the puppet https://review.openstack.org/96362 | 06:11 |
| krotscheck | nibalizer: Neat | 06:12 |
| *** pcrews has quit IRC | 06:12 | |
| jesusaurus | mordred: infra has a specs repo now, right? | 06:13 |
| *** ildikov has quit IRC | 06:13 | |
| jesusaurus | does it have anything in it yet? | 06:13 |
| mordred | jesusaurus: yup! | 06:13 |
| mordred | krotscheck: so - looking at your stuff, I think it looks great - I think I'm going to suggest for now that we take application.pp and port changes in it into storyboard/manifests/init.pp | 06:15 |
| mordred | since we don't need the mysql management in the config repo | 06:15 |
| krotscheck | mordred: Right, that’s why I split them out. | 06:15 |
| mordred | and then do teh application + init + mysql as an add-on change post split | 06:15 |
| krotscheck | The goal being: If you want to custom config SB, do it. Otherwise, here’s a neat standalone. | 06:16 |
| mordred | yup | 06:16 |
| krotscheck | That allows us to add new components if necessary. Say, gearman/rabbit as per the subscription spec. | 06:16 |
| krotscheck | I was hoping I could pull apart the webclient and the API, but vhost things made that hard. | 06:17 |
| krotscheck | Or rather, too complicated a switch. | 06:17 |
| *** rdopiera has quit IRC | 06:17 | |
| *** rdopiera has joined #openstack-infra | 06:17 | |
| *** rdopiera has joined #openstack-infra | 06:18 | |
| *** _nadya_ has joined #openstack-infra | 06:20 | |
| krotscheck | mordred: I did have one question. What’s the policy on using puppetforge things? Like, say, puppi::netinstall? | 06:21 |
| nibalizer | krotscheck: as long as the module is apache2, go for it! | 06:21 |
| nibalizer | im not sure what the tolerance is for other licenses ... mordred ? | 06:21 |
| *** praneshp has joined #openstack-infra | 06:21 | |
| *** afazekas has quit IRC | 06:22 | |
| krotscheck | Hrm. | 06:22 |
| krotscheck | https://github.com/example42/puppi/blob/master/LICENSE | 06:22 |
| *** afazekas has joined #openstack-infra | 06:22 | |
| krotscheck | That first header feels weird | 06:22 |
| mordred | puppi seems fine - and also a nice little module! | 06:23 |
| openstackgerrit | Radomir Dopieralski proposed a change to openstack-infra/config: Add gerritbot to the #tuskar channel https://review.openstack.org/96160 | 06:23 |
| *** lcheng_ has quit IRC | 06:24 | |
| krotscheck | Yeah, the whole download-as-a-state-expression is nice | 06:24 |
| krotscheck | I was experimenting with it on latest/jenkins.war, and it faithfully kept my jenkins up to date. | 06:25 |
| *** _nadya_ has quit IRC | 06:25 | |
| mordred | yeah - I think I'd like to look in to using that for the various places where we use that pattern | 06:25 |
| openstackgerrit | K Jonathan Harker proposed a change to openstack-infra/infra-specs: Copy over spec template from nova-specs https://review.openstack.org/96367 | 06:25 |
| krotscheck | Is that a frequent thing? | 06:25 |
| mordred | yeah. we have a bunch of stuff that uses the pattern | 06:26 |
| *** andreykurilin has joined #openstack-infra | 06:27 | |
| mordred | krotscheck: any reason to move the config file from /etc/storyboard/storyboard.conf to /etc/storyboard.conf? | 06:27 |
| *** HenryG has quit IRC | 06:27 | |
| krotscheck | mordred: It didn’t seem to work in /etc/storyboard/storyboard.conf | 06:28 |
| mordred | weird | 06:28 |
| mordred | krotscheck: storyboard didn't? or puppet? | 06:28 |
| krotscheck | storyboard. nibalizer can tell you more, he’s the one who was bashing his head against it. | 06:29 |
| *** praneshp_ has joined #openstack-infra | 06:29 | |
| bodepd | nibalizer: gotcha, I think I'll stick with the git implementation | 06:30 |
| mordred | oh- sadness is going to ensue | 06:32 |
| *** praneshp has quit IRC | 06:32 | |
| *** praneshp_ is now known as praneshp | 06:32 | |
| krotscheck | mordred: Oh? | 06:32 |
| mordred | krotscheck: see, you've gone and been sensible and used modern versions of puppetlabs modules | 06:32 |
| krotscheck | Oh dear. | 06:33 |
| nibalizer | oh no | 06:33 |
| *** HenryG has joined #openstack-infra | 06:33 | |
| mordred | nibalizer: we should really work on sorting that out in general | 06:33 |
| nibalizer | o7 | 06:33 |
| mordred | puppetlabs/apache and puppetlabs/vcsrepo being the main problem children | 06:34 |
| * nibalizer nods emphatically | 06:34 | |
| mordred | I dont' think there is anything in krotscheck's code that will matter with vcsrepo | 06:34 |
| krotscheck | Hrm. That _should_ be fixable though | 06:34 |
| mordred | the usage looks fine with openstackci/vcsrepo | 06:34 |
| krotscheck | At worst I have to provide my own vhost, no? | 06:34 |
| mordred | yeah. it's not a huge deal - but ultimately across the board I'd love to modernize the config codebase | 06:35 |
| nibalizer | ya but we should get onto plabs/apache | 06:36 |
| krotscheck | Well, I have to backport the module anyway. I’ll go dig into it tomorrow. | 06:36 |
| *** pcrews has joined #openstack-infra | 06:36 | |
| nibalizer | and since the puppet module team is fully supporting the module now, i think it won't be as squirly and should work for a long time | 06:36 |
| bodepd | I ran into one weird thing when I tested that install_module.sh scipt | 06:37 |
| bodepd | if I ran it a second time, it explodes b/c of conflicting stdlib versions | 06:37 |
| bodepd | does that sound familiar? | 06:37 |
| *** rickerc has quit IRC | 06:39 | |
| *** ildikov has joined #openstack-infra | 06:40 | |
| openstackgerrit | Dan Bode proposed a change to openstack-infra/config: Allow modules to be installed from source https://review.openstack.org/96336 | 06:40 |
| mordred | bodepd: no - we run that script over and over again all the time - we expect it to be idempotent | 06:43 |
| bodepd | damn. lemme see what I did then :) | 06:43 |
| mordred | bodepd: script looks great | 06:44 |
| bodepd | mordred: I have a high level of confidence that the install_module script currently produces an error if run | 06:50 |
| bodepd | it still runs to completion, so perhaps it has not been noticed | 06:50 |
| bodepd | mordred: s/run/re-run | 06:50 |
| bodepd | postgres pull in 'puppetlabs/concat', '>= 1.0.0 <2.0.0', which results in v1.1.0, which pulls in stdlib', '>= 4.0.0, which conflicts with the version of stdlib specified in install_modules.sh | 06:53 |
| bodepd | which results in: https://gist.github.com/bodepd/78f822a8a3ff4ea07a5a | 06:53 |
| nibalizer | oh noes | 06:54 |
| bodepd | seriously, figgen forget using the forge and deps | 06:54 |
| bodepd | I just specify the version of everything I want to use | 06:55 |
| bodepd | easiest fix is to bump stdlib to something greater than 4 | 06:55 |
| bodepd | it's usually pretty stable, so it probably won't break anything | 06:56 |
| bodepd | except you guys use a pretty ancient version of puppet :) | 06:56 |
| *** pcrews has quit IRC | 06:56 | |
| openstackgerrit | Frederic Lepied proposed a change to openstack-infra/config: glanceclient: make the Python 3.3 gate voting https://review.openstack.org/96370 | 06:56 |
| nibalizer | i think we run install_modules.sh with >/dev/null 2>&1 so thats probably why it hasn't been noticed | 06:56 |
| *** andreykurilin has quit IRC | 06:57 | |
| bodepd | well, it doesn't install the version of stdlib you think it does :) | 06:59 |
| bodepd | maybe it is reasonble to jump bump the version | 06:59 |
| *** dstanek_zzz is now known as dstanek | 07:00 | |
| *** _nadya_ has joined #openstack-infra | 07:01 | |
| *** rickerc has joined #openstack-infra | 07:03 | |
| *** flaper87|afk is now known as flaper87 | 07:06 | |
| mordred | awesome | 07:08 |
| *** dstanek is now known as dstanek_zzz | 07:09 | |
| *** kashyap is now known as kashyap_lunch | 07:10 | |
| *** gyee has quit IRC | 07:10 | |
| mordred | bodepd: I don't think your script handles a module dir existing already | 07:10 |
| mordred | oh - I get it | 07:11 |
| bodepd | that will show up as a module | 07:11 |
| bodepd | in the list call | 07:11 |
| *** yaguang has quit IRC | 07:11 | |
| *** salv-orlando has joined #openstack-infra | 07:13 | |
| mordred | yeah. so ... I think we may need to get cleverer to support the upgrade - since we'll have modules installed (like nibalizer's) from previous forge installs | 07:14 |
| mordred | which we'll want to replace with the new git repo version | 07:14 |
| *** jlibosva has joined #openstack-infra | 07:14 | |
| mordred | but I think we're really close here | 07:14 |
| mordred | krotscheck: gah. puppi is ALMOST exactly what I want | 07:15 |
| mordred | krotscheck: but I can't seem to get it to download http://tarballs.openstack.org/ci/gerrit/gerrit-v2.8.4.15.6dc8444.war and put it into a dir without unpacking it as 'gerrit.war' | 07:15 |
| mordred | the war deploy command wants to do very weird things | 07:16 |
| mordred | and netinstall doesn't seem to want to rename it for me | 07:16 |
| mordred | it DOES quite happily download it to gerrit-v2.8.4.15.6dc8444.war successfully | 07:16 |
| *** _nadya_ has quit IRC | 07:18 | |
| *** yaguang has joined #openstack-infra | 07:19 | |
| *** pblaho has joined #openstack-infra | 07:22 | |
| *** bogdando has quit IRC | 07:26 | |
| *** bogdando has joined #openstack-infra | 07:26 | |
| openstackgerrit | Steven Hardy proposed a change to openstack/requirements: bump heatclient to latest version https://review.openstack.org/96377 | 07:29 |
| *** kashyap_lunch is now known as kashyap | 07:29 | |
| openstackgerrit | Steven Hardy proposed a change to openstack/requirements: bump heatclient to latest version https://review.openstack.org/96377 | 07:30 |
| ttx | cody-somerville: could you expand on that ? There is one line per release... | 07:30 |
| *** jp_at_hp has joined #openstack-infra | 07:31 | |
| *** zzelle has joined #openstack-infra | 07:31 | |
| *** zzelle has quit IRC | 07:32 | |
| *** zzelle has joined #openstack-infra | 07:32 | |
| *** zzelle has quit IRC | 07:34 | |
| *** zzelle has joined #openstack-infra | 07:34 | |
| *** zzelle has quit IRC | 07:36 | |
| *** zzelle has joined #openstack-infra | 07:36 | |
| YorikSar | Hello, everyone. | 07:37 |
| YorikSar | Smth strange happened to Zuul. | 07:37 |
| YorikSar | Status page won't update because status.json has a change with id == null | 07:37 |
| *** mrda is now known as mrda-away | 07:39 | |
| *** pelix has joined #openstack-infra | 07:41 | |
| mattoliverau | That's it for me, time to call it a night! Night all. | 07:44 |
| *** wenlock has quit IRC | 07:45 | |
| openstackgerrit | Yuriy Taraday proposed a change to openstack-infra/config: Allow Zuul status page to update with periodic tasks present https://review.openstack.org/96384 | 07:47 |
| YorikSar | mordred, SergeyLukjanov: ^ - as long as you're up... | 07:50 |
| *** morganfainberg is now known as morganfainberg_Z | 07:54 | |
| *** matsuhashi has quit IRC | 07:58 | |
| *** matsuhashi has joined #openstack-infra | 07:59 | |
| *** marcoemorais1 has quit IRC | 08:00 | |
| *** matsuhas_ has joined #openstack-infra | 08:02 | |
| *** mrmartin has joined #openstack-infra | 08:02 | |
| *** matsuhashi has quit IRC | 08:04 | |
| *** ihrachyshka has joined #openstack-infra | 08:04 | |
| *** e0ne has joined #openstack-infra | 08:12 | |
| *** morganfainberg_Z is now known as morganfainberg | 08:12 | |
| *** ociuhandu has quit IRC | 08:18 | |
| *** ociuhandu has joined #openstack-infra | 08:20 | |
| *** jpich has joined #openstack-infra | 08:21 | |
| *** ociuhandu has quit IRC | 08:24 | |
| *** derekh_ has joined #openstack-infra | 08:25 | |
| *** jooools has joined #openstack-infra | 08:27 | |
| *** dizquierdo has joined #openstack-infra | 08:29 | |
| renlt | clarkb ping | 08:31 |
| *** oomichi has quit IRC | 08:31 | |
| *** Ivo has quit IRC | 08:35 | |
| kashyap | renlt, He's in PDT timezone, you might just want to leave a msg here | 08:36 |
| renlt | appreciate, kashyap, thanks for telling me. | 08:37 |
| renlt | kashyap, I just tried to push a new branch to the remote(both origin and gerrit) of our stackforge project, both failed. Do you know if there is a guide on how to create a new branch for stackforge poroject? Thanks! | 08:38 |
| renlt | now there is only a master branch, we want to create a release branch like stable/icehouse | 08:40 |
| kashyap | renlt, I'm afraid, I'm not an admin here, just a lurker. You may have to wait someone more knowledgeable on that aspect. | 08:40 |
| kashyap | renlt, There's some details here that might help you - http://ci.openstack.org/stackforge.html | 08:41 |
| *** mrmartin has quit IRC | 08:42 | |
| renlt | kashyap, thanks all the same, I read that guide, I think it doesn't cover this topic in detail yet. Then I'll wait. | 08:45 |
| kashyap | renlt, Or you could mail the infra list where there are a wider set of eyes - http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-infra | 08:46 |
| *** jcoufal has joined #openstack-infra | 08:46 | |
| *** sunrenjie6 has quit IRC | 08:52 | |
| *** sunrenjie6 has joined #openstack-infra | 08:52 | |
| *** Ivo has joined #openstack-infra | 08:53 | |
| *** salv-orlando_ has joined #openstack-infra | 08:53 | |
| *** matsuhas_ has quit IRC | 08:53 | |
| *** salv-orlando has quit IRC | 08:54 | |
| *** salv-orlando_ is now known as salv-orlando | 08:54 | |
| *** mrmartin has joined #openstack-infra | 08:56 | |
| *** sunrenjie6 has quit IRC | 08:57 | |
| *** matsuhashi has joined #openstack-infra | 08:58 | |
| *** ihrachyshka has quit IRC | 09:01 | |
| *** salv-orlando has quit IRC | 09:04 | |
| renlt | kashyap, appreciate, just sent a note to the infra list and post a question on ask.openstack.org :) | 09:12 |
| *** renlt has quit IRC | 09:12 | |
| *** ominakov has joined #openstack-infra | 09:19 | |
| openstackgerrit | Christian Berendt proposed a change to openstack-infra/os-loganalyze: cleaning up index.rst file https://review.openstack.org/96405 | 09:19 |
| openstackgerrit | Christian Berendt proposed a change to openstack-infra/reviewstats: cleaning up index.rst file https://review.openstack.org/96406 | 09:20 |
| *** e0ne has quit IRC | 09:20 | |
| *** e0ne has joined #openstack-infra | 09:21 | |
| openstackgerrit | Christian Berendt proposed a change to openstack-infra/zuul: cleaning up index.rst file https://review.openstack.org/96407 | 09:21 |
| *** dguerri is now known as _dguerri | 09:22 | |
| *** skolekonov has joined #openstack-infra | 09:23 | |
| mika | zaro: yeah, a jjb release would be awesome :) | 09:23 |
| *** _dguerri is now known as dguerri | 09:26 | |
| openstackgerrit | A change was merged to openstack-infra/config: Allow Zuul status page to update with periodic tasks present https://review.openstack.org/96384 | 09:29 |
| *** zhiyan is now known as zhiyan_ | 09:30 | |
| *** matsuhashi has quit IRC | 09:33 | |
| *** matsuhashi has joined #openstack-infra | 09:33 | |
| openstackgerrit | Sergey Skripnick proposed a change to openstack-infra/config: Make rally-install and rally-dsvm jobs voting https://review.openstack.org/96087 | 09:35 |
| *** matsuhashi has quit IRC | 09:35 | |
| *** matsuhashi has joined #openstack-infra | 09:36 | |
| openstackgerrit | yolanda.robla proposed a change to openstack-infra/storyboard: Added new priority changed event for tasks https://review.openstack.org/96151 | 09:41 |
| *** praneshp has quit IRC | 09:42 | |
| *** oomichi has joined #openstack-infra | 09:49 | |
| *** otter768_ has joined #openstack-infra | 10:00 | |
| *** ominakov has quit IRC | 10:01 | |
| *** otter768 has quit IRC | 10:02 | |
| *** nosnos has quit IRC | 10:02 | |
| *** ominakov has joined #openstack-infra | 10:03 | |
| *** unicell has quit IRC | 10:04 | |
| *** nosnos has joined #openstack-infra | 10:05 | |
| *** isviridov has quit IRC | 10:10 | |
| *** rcarrillocruz has quit IRC | 10:10 | |
| *** jesusaurus has quit IRC | 10:10 | |
| *** rcarrillocruz has joined #openstack-infra | 10:10 | |
| *** jesusaurus has joined #openstack-infra | 10:11 | |
| openstackgerrit | yolanda.robla proposed a change to openstack-infra/storyboard-webclient: Added priority on task row https://review.openstack.org/96156 | 10:13 |
| *** ociuhandu has joined #openstack-infra | 10:13 | |
| openstackgerrit | Sergey Skripnick proposed a change to openstack-infra/config: Add rally to PROJECTS in rally-cli job https://review.openstack.org/96414 | 10:15 |
| *** isviridov has joined #openstack-infra | 10:18 | |
| *** ominakov has quit IRC | 10:28 | |
| *** markmc has joined #openstack-infra | 10:29 | |
| *** e0ne has quit IRC | 10:34 | |
| *** e0ne has joined #openstack-infra | 10:34 | |
| *** e0ne has quit IRC | 10:39 | |
| *** matsuhashi has quit IRC | 10:41 | |
| *** HenryG has quit IRC | 10:41 | |
| *** nosnos has quit IRC | 10:41 | |
| *** lttrl has quit IRC | 10:42 | |
| *** afazekas is now known as afazekas_sick_do | 10:43 | |
| *** nosnos has joined #openstack-infra | 10:45 | |
| *** nosnos has quit IRC | 10:46 | |
| *** nosnos has joined #openstack-infra | 10:47 | |
| *** nosnos has quit IRC | 10:48 | |
| *** Longgeek has quit IRC | 10:51 | |
| *** afazekas_sick_do has quit IRC | 10:53 | |
| rcarrillocruz | can anyone please confirm if we are indeed disabling draft reviews in gerrit? are we performing the change tomorrow? | 10:54 |
| *** Longgeek has joined #openstack-infra | 10:55 | |
| *** HenryG has joined #openstack-infra | 10:56 | |
| *** LurkingFrog has quit IRC | 10:58 | |
| *** rlandy has joined #openstack-infra | 11:00 | |
| *** e0ne has joined #openstack-infra | 11:02 | |
| *** yaguang has quit IRC | 11:06 | |
| sdague | rcarrillocruz: I know the intent was always to disable drafts, I don't know what the current schedule for it is | 11:06 |
| *** yaguang has joined #openstack-infra | 11:07 | |
| *** yjiang has quit IRC | 11:07 | |
| *** yaguang has quit IRC | 11:07 | |
| openstackgerrit | Marton Kiss proposed a change to openstack-infra/config: Fix groups-dev drush dependency problem https://review.openstack.org/96421 | 11:07 |
| *** ominakov has joined #openstack-infra | 11:08 | |
| *** IvanBerezovskiy has joined #openstack-infra | 11:12 | |
| *** ihrachyshka has joined #openstack-infra | 11:13 | |
| *** dims has joined #openstack-infra | 11:16 | |
| *** prad_ has joined #openstack-infra | 11:17 | |
| zzelle | rcarrillocruz, " We will take advantage of this | 11:20 |
| zzelle | and disable pushes of new Drafts on Friday May 30, 2014. | 11:20 |
| zzelle | " | 11:20 |
| *** dguerri is now known as _dguerri | 11:20 | |
| mrmartin | sdague: hi, can you check why this gate job failed: http://logs.openstack.org/21/96421/1/check/gate-config-puppet-apply/4f6d0ba/console.html | 11:21 |
| mrmartin | sudo: unable to resolve host bare-precise-hpcloud-b3-4409943 | 11:21 |
| mrmartin | Unsupported OS at /etc/puppet/modules/selinux/manifests/params.pp:23 on node bare-precise-1401200060.openstacklocal | 11:21 |
| *** prad__ has joined #openstack-infra | 11:22 | |
| *** prad_ has quit IRC | 11:22 | |
| mrmartin | and jenkins says it is succeeded anyway:. | 11:22 |
| *** _dguerri is now known as dguerri | 11:24 | |
| *** Longgeek has quit IRC | 11:29 | |
| *** Longgeek has joined #openstack-infra | 11:30 | |
| *** Longgeek has quit IRC | 11:31 | |
| YorikSar | mrmartin: puppet-apply jobs are non-voting, so you hange Verified+1 | 11:31 |
| YorikSar | mrmartin: *hange=>have | 11:32 |
| mrmartin | ok, but is it normal, that fails? | 11:32 |
| YorikSar | mrmartin: I guess so. Why would they mark it as non-voting otherwise? | 11:32 |
| mrmartin | ok. | 11:33 |
| *** mburned_out is now known as mburned | 11:33 | |
| *** Longgeek has joined #openstack-infra | 11:35 | |
| *** Longgeek has quit IRC | 11:36 | |
| *** prad__ has quit IRC | 11:38 | |
| *** Longgeek has joined #openstack-infra | 11:38 | |
| sdague | mrmartin: it looks like the selinux issue is the real issue | 11:43 |
| sdague | that's a non voting job, so because it failed doesn't make the whole jenkins run fail | 11:43 |
| *** salv-orlando has joined #openstack-infra | 11:45 | |
| *** dguerri is now known as _dguerri | 11:47 | |
| *** bcrochet|g0ne is now known as bcrochet | 11:54 | |
| *** pdmars has joined #openstack-infra | 11:56 | |
| *** ominakov has quit IRC | 11:59 | |
| *** dims has quit IRC | 12:00 | |
| *** prad_ has joined #openstack-infra | 12:00 | |
| *** mwagner_lap has quit IRC | 12:02 | |
| *** ominakov has joined #openstack-infra | 12:02 | |
| *** weshay has joined #openstack-infra | 12:05 | |
| *** mbacchi has joined #openstack-infra | 12:07 | |
| *** fanhe has joined #openstack-infra | 12:09 | |
| *** salv-orlando has quit IRC | 12:10 | |
| rcarrillocruz | k, thx zzelle and sdague | 12:10 |
| *** ArxCruz has joined #openstack-infra | 12:14 | |
| *** prad_ has quit IRC | 12:14 | |
| *** prad_ has joined #openstack-infra | 12:15 | |
| *** samuelmz has joined #openstack-infra | 12:21 | |
| *** salv-orlando has joined #openstack-infra | 12:22 | |
| *** _dguerri is now known as dguerri | 12:23 | |
| *** smarcet has joined #openstack-infra | 12:23 | |
| *** dprince has joined #openstack-infra | 12:23 | |
| *** dizquierdo has quit IRC | 12:24 | |
| *** rfolco has joined #openstack-infra | 12:24 | |
| *** weshay has quit IRC | 12:27 | |
| *** weshay has joined #openstack-infra | 12:27 | |
| *** hashar has joined #openstack-infra | 12:29 | |
| *** eharney has quit IRC | 12:29 | |
| *** exz has left #openstack-infra | 12:30 | |
| *** exz has joined #openstack-infra | 12:30 | |
| *** exz has left #openstack-infra | 12:31 | |
| *** talluri has quit IRC | 12:34 | |
| *** talluri has joined #openstack-infra | 12:35 | |
| *** lcostantino has joined #openstack-infra | 12:37 | |
| *** oomichi has quit IRC | 12:37 | |
| *** talluri has quit IRC | 12:39 | |
| *** aysyd has joined #openstack-infra | 12:40 | |
| *** dstanek_zzz is now known as dstanek | 12:41 | |
| samuelmz | hey guys, I just submitted a patch to Tempest .. it fixes a failing test due to a nova patch | 12:41 |
| samuelmz | talking on #openstack-infra | 12:41 |
| samuelmz | * Topic for #openstack-infra is: Discussion of OpenStack Developer Infrastructure | docs http://ci.openstack.org | bugs https://launchpad.net/openstack-ci/ | https://git.openstack.org/cgit/openstack-infra/config/tree/ | 12:41 |
| samuelmz | * Topic for #openstack-infra set by openstackstatus at Tue May 27 19:09:55 2014 | 12:41 |
| samuelmz | * salv-orlando (~salv-orla@151.70.195.112) has joined #openstack-infra | 12:41 |
| samuelmz | * _dguerri is now known as dguerri | 12:41 |
| samuelmz | * smarcet (~sebastian@186.22.94.44) has joined #openstack-infra | 12:41 |
| samuelmz | * dprince (~dan.princ@fowl.fesnel.com) has joined #openstack-infra | 12:41 |
| samuelmz | * dizquierdo has quit (Quit: Ex-Chat) | 12:41 |
| samuelmz | * rfolco (rfolco@nat/ibm/x-dubfjvxpnaswjmkz) has joined #openstack-infra | 12:41 |
| samuelmz | * weshay has quit (Quit: Leaving) | 12:41 |
| samuelmz | * weshay (whayutin@nat/redhat/x-lmqoppjmurrhrcoq) has joined #openstack-infra | 12:41 |
| samuelmz | * hashar (~sempitern@mediawiki/hashar) has joined #openstack-infra | 12:41 |
| samuelmz | * eharney has quit (Ping timeout: 252 seconds) | 12:41 |
| samuelmz | * exz (~exz@2a01:e35:242c:eae0:96de:80ff:fed7:6108) has left #openstack-infra | 12:41 |
| samuelmz | * exz (~exz@2a01:e35:242c:eae0:96de:80ff:fed7:6108) has joined #openstack-infra | 12:41 |
| samuelmz | * exz (~exz@2a01:e35:242c:eae0:96de:80ff:fed7:6108) has left #openstack-infra | 12:41 |
| samuelmz | * talluri has quit (Remote host closed the connection) | 12:41 |
| samuelmz | * talluri (~talluri@14.141.5.5) has joined #openstack-infra | 12:41 |
| samuelmz | * lcostantino (~lcostanti@host18.190-3-8.static.telmex.net.ar) has joined #openstack-infra | 12:41 |
| samuelmz | * oomichi has quit (Quit: Leaving...) | 12:41 |
| samuelmz | * talluri has quit (Ping timeout: 255 seconds) | 12:41 |
| samuelmz | * aysyd (aysyanne@nat/ibm/x-rmljilplerqybnjt) has joined #openstack-infra | 12:41 |
| samuelmz | I'm sorry, the link https://review.openstack.org/#/c/96441/ | 12:41 |
| hashar | copy paste failure :D | 12:42 |
| *** rodrigods has joined #openstack-infra | 12:42 | |
| samuelmz | hashar, the link to review https://review.openstack.org/#/c/96441/ | 12:42 |
| samuelmz | The error when running tests http://paste.openstack.org/show/82015/ | 12:44 |
| samuelmz | Look at the review, you can find everything there :D | 12:44 |
| sdague | samuelmz: in order to skip an existing test that verifies a piece of nova behavior, we require a nova core +2 on the patch, to ensure the change in behavior is desired | 12:44 |
| samuelmz | sdague, ok ... I'm gonna submit a new patch on Nova taking into account some details and I ll ask for a +2 | 12:47 |
| samuelmz | sdague, thanks for this info | 12:47 |
| *** signed8bit has joined #openstack-infra | 12:48 | |
| *** yamahata has quit IRC | 12:51 | |
| openstackgerrit | Matt Ray proposed a change to openstack-infra/config: New Chef cookbooks for client services, Sahara and Trove. https://review.openstack.org/96332 | 12:54 |
| *** dstanek is now known as dstanek_zzz | 12:56 | |
| *** _nadya_ has joined #openstack-infra | 12:58 | |
| *** miqui has joined #openstack-infra | 12:58 | |
| *** ArxCruz has quit IRC | 13:05 | |
| fungi | mrmartin: that particular job got added about 12 hours ago and is part of an ongoing puppet clean-up effort, which is why it's initially non-voting while the remaining kinks get ironed out | 13:06 |
| mrmartin | fungi: ok thanks, it was a bit strange for me that the build succeeds when we got a fail | 13:06 |
| *** Ajaeger has joined #openstack-infra | 13:07 | |
| *** e0ne_ has joined #openstack-infra | 13:08 | |
| *** ArxCruz has joined #openstack-infra | 13:08 | |
| *** dstanek_zzz is now known as dstanek | 13:08 | |
| sdague | mrmartin: we do that all the time when we have non stable test jobs | 13:09 |
| *** jaypipes has quit IRC | 13:10 | |
| *** mriedem has joined #openstack-infra | 13:10 | |
| *** dims has joined #openstack-infra | 13:10 | |
| *** e0ne has quit IRC | 13:10 | |
| *** zhiyan_ is now known as zhiyan | 13:11 | |
| *** prad_ has quit IRC | 13:15 | |
| *** jaypipes has joined #openstack-infra | 13:16 | |
| *** julim has joined #openstack-infra | 13:21 | |
| *** eharney has joined #openstack-infra | 13:25 | |
| *** samuelmz is now known as samuelmz-afk | 13:28 | |
| *** HenryG has quit IRC | 13:30 | |
| *** bknudson has joined #openstack-infra | 13:30 | |
| *** HenryG has joined #openstack-infra | 13:31 | |
| *** thomasbiege has joined #openstack-infra | 13:33 | |
| openstackgerrit | A change was merged to openstack/requirements: Remove hplefthandclient from global-requirements https://review.openstack.org/95974 | 13:38 |
| *** zehicle_at_dell has joined #openstack-infra | 13:38 | |
| *** homeless has joined #openstack-infra | 13:42 | |
| *** unicell has joined #openstack-infra | 13:43 | |
| *** yamahata has joined #openstack-infra | 13:44 | |
| *** nealph has joined #openstack-infra | 13:45 | |
| *** gokrokve has joined #openstack-infra | 13:46 | |
| *** lxsli has joined #openstack-infra | 13:46 | |
| *** weshay has quit IRC | 13:46 | |
| *** Longgeek has quit IRC | 13:48 | |
| nealph | Am seeing repeated failure to start gate on https://review.openstack.org/#/c/94633/ after a bunch of recheck attempts....can someone give some insight? | 13:48 |
| *** wenlock has joined #openstack-infra | 13:49 | |
| *** prad_ has joined #openstack-infra | 13:49 | |
| *** Longgeek has joined #openstack-infra | 13:49 | |
| dims | nealph, are you watching here? http://status.openstack.org/zuul/ | 13:50 |
| *** unicell has quit IRC | 13:50 | |
| lxsli | this is check-tripleo queue | 13:50 |
| mestery | anteaya: I have a couple of Neutron 3rd party CI accounts which would like to be able to vote +1/-1 on patches. They have been running for a while. How do we make that happen? | 13:50 |
| dims | nealph, looks like it's running now - https://jenkins01.openstack.org/job/gate-tripleo-incubator-docs/916/console | 13:51 |
| fungi | mestery: i can put them in the voting group. which accounts? | 13:51 |
| mestery | fungi: Thanks! It's the Big Switch CI, and the OpenDaylight CI. | 13:52 |
| * mestery notices the Big Switch CI is already voting. | 13:52 | |
| mestery | fungi: I guess that change took place recently, which is good! | 13:52 |
| fungi | mestery: Big Switch CI is already in the group allowed to vote | 13:53 |
| fungi | yep | 13:53 |
| mestery | fungi: Thanks! | 13:53 |
| *** prad_ has quit IRC | 13:53 | |
| fungi | mestery: OpenDaylight Jenkins has been moved to teh voting group now as well | 13:53 |
| fungi | they should be all set | 13:53 |
| mestery | fungi: Great, much appreciated! | 13:53 |
| fungi | my pleasure | 13:53 |
| nealph | dims: yep, jp just kicked it again, but if you look at the patch history you'll see that it's gone through this cycle a few times. This might be the one that gets it through though... | 13:54 |
| *** achuprin has joined #openstack-infra | 13:54 | |
| * nealph fingers crossed | 13:54 | |
| fungi | nealph: i don't entirely understand your concern. it looks like it passed every time | 13:55 |
| fungi | nealph: can you explain what you're expecting to happen? | 13:55 |
| fungi | it looks like multiple people keep adding recheck comments to re-run check tests on it, but they always pass | 13:56 |
| *** gibi has left #openstack-infra | 13:56 | |
| *** ccorrigan has joined #openstack-infra | 13:57 | |
| nealph | fungi: expecting it to move to gate change queue...yes? | 13:57 |
| fungi | nealph: someone has to approve it first | 13:57 |
| lxsli | fungi: One, sometimes a 'recheck no bug' seems to be ignored. Two, the tripleo gate often loses its queue (reset?). Three, although the docs job often passes, the four jobs in the main one intermittently fail | 13:57 |
| lxsli | nealph: it has to pass the check-tripleo pipeline before any core will +A. Only then can it enter the merge gate proper | 13:58 |
| lxsli | nealph: JP + I have been playign fast and loose by calling the check the gate | 13:58 |
| *** jasond has joined #openstack-infra | 13:58 | |
| fungi | lxsli: yeah, the main tripleo cloud keeps falling offline due to a variety of issues the their team has been working to resolve, so its reporting has been intermittent as a resuly | 13:58 |
| fungi | result | 13:58 |
| *** dguerri is now known as _dguerri | 13:58 | |
| *** _dguerri is now known as dguerri | 13:58 | |
| jasond | does anybody know why this review is stuck? https://review.openstack.org/#/c/84520/ | 13:58 |
| *** weshay has joined #openstack-infra | 13:58 | |
| achuprin | Hi Infra | 13:58 |
| lxsli | fungi: that'll be it then. So the advice is give that team time + keep on top of rechecks? | 13:59 |
| fungi | lxsli: probably, yes. you should ask in #tripleo | 13:59 |
| lxsli | anyone specific you know who's working on that cloud? | 13:59 |
| *** prad_ has joined #openstack-infra | 14:00 | |
| fungi | jasond: it looks like that one was in the middle of being tested back in april when we took zuul offline for an upgrade, and so no check vote was left. i've added a recheck comment now to get it to run check tests which should then (if they pass) automatically enqueue it into the gate pipeline | 14:00 |
| jasond | fungi: thank you | 14:01 |
| *** ihrachyshka_ has joined #openstack-infra | 14:01 | |
| *** mfer has joined #openstack-infra | 14:01 | |
| fungi | lxsli: http://git.openstack.org/cgit/openstack/tripleo-incubator/tree/tripleo-cloud/tripleo-cd-admins | 14:01 |
| fungi | lxsli: found from https://wiki.openstack.org/wiki/TripleO/TripleOCloud#tripleo-cd-admins_team | 14:01 |
| fungi | achuprin: what's up? | 14:01 |
| lxsli | fungi: fab, thank you | 14:01 |
| *** yfried has quit IRC | 14:01 | |
| fungi | jasond: and it looks like it needs a rebase before it can go in | 14:02 |
| *** jgrimm has joined #openstack-infra | 14:02 | |
| jasond | fungi: yep, working on that now | 14:03 |
| *** prad__ has joined #openstack-infra | 14:03 | |
| achuprin | fungi: I have a question about sync gerrit and github repo | 14:03 |
| achuprin | fungi: what do you use for this? | 14:03 |
| nealph | fungi:lxsli:thanks for the insight...will keep a close eye on check-tripleo progress and ping the folks in #tripleo if needed. | 14:03 |
| jeblair | sdague: anything i can do to help with the depends on thing? | 14:03 |
| *** prad_ has quit IRC | 14:03 | |
| *** ihrachyshka has quit IRC | 14:04 | |
| *** prad__ is now known as prad_ | 14:04 | |
| sdague | jeblair: well I still don't understand why the current patch fails - https://review.openstack.org/#/c/94365/ - realistically I haven't looked at it this week, as I've been trying to rebuild javelin | 14:05 |
| *** ildikov_ has joined #openstack-infra | 14:05 | |
| *** ildikov has quit IRC | 14:06 | |
| *** wenlock has quit IRC | 14:06 | |
| *** aysyd has quit IRC | 14:06 | |
| sdague | if you have cycles to look at things, it might be good to look at the jeepyb patches, as those are older but simpler, and I'm trying to reduce the contexts I've got outstanding in my patch queue | 14:06 |
| *** aysyd has joined #openstack-infra | 14:07 | |
| *** talluri has joined #openstack-infra | 14:07 | |
| *** doug-fish has joined #openstack-infra | 14:11 | |
| jeblair | sdague: okay, that sounds like a plan. | 14:12 |
| zzelle | YorikSar, fungi, hi | 14:13 |
| YorikSar | zzelle: Hello | 14:13 |
| *** gokrokve has quit IRC | 14:14 | |
| fungi | achuprin: there's a plug-in for gerrit which we use for that... https://gerrit-review.googlesource.com/#/admin/projects/plugins/replication | 14:15 |
| *** dizquierdo has joined #openstack-infra | 14:15 | |
| fungi | YorikSar: hi there | 14:15 |
| openstackgerrit | Peter Belanyi proposed a change to openstack-infra/config: Add jshint job for tuskar-ui https://review.openstack.org/96473 | 14:16 |
| zzelle | fungi, YorikSar , everyone there, cool | 14:16 |
| zzelle | fungi, YorikSar and I have questions about https://bugs.launchpad.net/git-review/+bug/1322365 | 14:16 |
| uvirtbot | Launchpad bug 1322365 in git-review "git-review 1.24 (or newer) needs to be released" [High,In progress] | 14:16 |
| zzelle | fungi, do you have a roadmap for the 1.24 release ? | 14:17 |
| fungi | zzelle: yep, in fact i just updated my local install to run the tip of master just so i can smoke out any potential uncaught regressions in it before we tag it | 14:18 |
| YorikSar | fungi, zzelle: It seems we might want https://review.openstack.org/93952 to land before the release. | 14:18 |
| YorikSar | fungi: I've done develop install to ~/.local and have been working like that for ages :) | 14:19 |
| fungi | YorikSar: zzelle: i agree. i left it open so it could collect some reviews from prospective users of the feature (since i am not likely to be one of them) | 14:19 |
| *** talluri has quit IRC | 14:19 | |
| *** ihrachyshka_ is now known as ihrachyshka | 14:19 | |
| *** pcrews has joined #openstack-infra | 14:20 | |
| achuprin | fungi: thanks | 14:20 |
| mtreinish | SergeyLukjanov: quick question is there a reason that all my tempest review emails from gerrit show you as the 'To:' address? | 14:20 |
| zzelle | fungi, YorikSar, from my point of view, it's not a "must have" feature more a "may have", so this review can wait for the next release | 14:21 |
| YorikSar | fungi, zzelle: I wonder where did the need for that option come from. | 14:21 |
| SergeyLukjanov | mtreinish, O_O | 14:21 |
| zzelle | fungi, YorikSar: the (my) need for that option is not for review.o.o | 14:22 |
| SergeyLukjanov | mtreinish, IIRC all reviewers are in the "to" list | 14:22 |
| YorikSar | zzelle: Oh, so you're one of users, not just implementing it for someone. Good. I wonder if you're the only one :) | 14:23 |
| SergeyLukjanov | mtreinish, but I'm not reviewing tempest a lot, so, it seems strange | 14:23 |
| fungi | zzelle: understandable. i can imagine where it might be useful in other environments, which is why i wanted to see some additional feedback from prospective users | 14:23 |
| *** thomasbiege has quit IRC | 14:24 | |
| zzelle | YorikSar, fungi, perhaps it can be useful for the infra team but the need has been explicited | 14:24 |
| *** tnurlygayanov has quit IRC | 14:24 | |
| zzelle | fungi, i share your point of view | 14:24 |
| mtreinish | SergeyLukjanov: it's also on new uploaded reviews | 14:24 |
| zzelle | fungi, YorikSar, that' why we can delay it for next release | 14:25 |
| fungi | zzelle: yeah, we usually still want to review the change before hitting submit on it, so while we do use submit to bypass gating sometimes (when we're fixing the gate more often than not), submitting a patchset immediately on upload would be rare for us | 14:25 |
| *** rickerc has quit IRC | 14:25 | |
| mtreinish | but only tempest | 14:25 |
| jeblair | zzelle: sorry if this has been asked already, but why not just 'git push' in that case? | 14:25 |
| *** skolekonov has quit IRC | 14:26 | |
| *** ildikov_ has quit IRC | 14:26 | |
| jeblair | zzelle: (if it has, feel free to just point me at the conversation) | 14:26 |
| *** ildikov has joined #openstack-infra | 14:26 | |
| *** tnurlygayanov has joined #openstack-infra | 14:26 | |
| fungi | zzelle: the -S option there basically expresses the sentiment, "i want this sent through the code review system, but not pre-merge tested nor actually reviewed by anyone, just merged straight away" | 14:26 |
| fungi | and yes, as jeblair points out, git push to the branch is something you can set in the acl for a project (and you can even still disallow --force or merge commits) | 14:27 |
| YorikSar | fungi, zzelle: Do I understand correctly that it won't submit existing patchset? Like if I've just downloaded someone's patchset, verified it and want to submit it. | 14:27 |
| *** reed has joined #openstack-infra | 14:27 | |
| zzelle | jeblair, my (small) team is used to git push, we introduce gerrit review as an option : you can choose between git push/review (expect for new devs) | 14:28 |
| mtreinish | SergeyLukjanov: for example: http://paste.openstack.org/show/4Bxu4Xub43QTcOZVGSVD/ | 14:28 |
| reed | hello channel | 14:28 |
| zzelle | jeblair, return on experience ... everyone push and remember the git review option | 14:28 |
| *** david-lyle has joined #openstack-infra | 14:28 | |
| fungi | YorikSar: i haven't tested it, but it looked like it wanted to push the new patchset for review and immediately submit. i may certainly be misreading the code though | 14:28 |
| *** rickerc has joined #openstack-infra | 14:29 | |
| fungi | hello reed | 14:29 |
| *** yaguang has joined #openstack-infra | 14:29 | |
| *** samuelmz-afk is now known as samuelmz | 14:29 | |
| zzelle | jeblair, so we reverse the behaviour, everyone must use git review and can bypass code review using submit | 14:29 |
| jeblair | zzelle: got it, thx | 14:30 |
| zzelle | jeblair, i was surprised by the result but it works ! | 14:30 |
| reed | zzelle, what's going on with your irc client? so many blank spaces and empty lines | 14:30 |
| reed | zzelle, guess what, nothing is wrong with your client | 14:30 |
| reed | it's my display going crazy | 14:30 |
| SergeyLukjanov | mtreinish, honestly - no ideas | 14:31 |
| *** ominakov has quit IRC | 14:31 | |
| reed | ok, reboot time | 14:31 |
| *** reed has quit IRC | 14:31 | |
| Ajaeger | infra team, FYI, the post-requirements job is still failing - now because of climate rename and not yet merged .gitreview update. Patch https://review.openstack.org/#/c/95287/ hasn't passed the gate ;( | 14:32 |
| zzelle | fungi, exactly --submit-immediately permits to push a change and try to merge it without waiting for approval | 14:32 |
| *** dims has quit IRC | 14:33 | |
| sdague | mtreinish: I have that same mail with the correct to | 14:33 |
| sdague | any chance you've got something locally that's munging? | 14:34 |
| *** yaguang has quit IRC | 14:34 | |
| fungi | Ajaeger: are you sure? https://review.openstack.org/96144 should have fixed that for proposals to master branches | 14:34 |
| Ajaeger | fungi: the proposal job checks out python-blazarclient and cannot access gerrit | 14:34 |
| Ajaeger | let me grab the failure... | 14:34 |
| mtreinish | sdague: maybe, but that'd be really weird | 14:34 |
| fungi | Ajaeger: ohhh...! | 14:34 |
| openstackgerrit | Peter Belanyi proposed a change to openstack-infra/config: Add jshint job for tuskar-ui https://review.openstack.org/96473 | 14:34 |
| fungi | Ajaeger: yes, i get what you're saying now | 14:35 |
| *** paul-- has quit IRC | 14:35 | |
| Ajaeger | https://jenkins.openstack.org/job/propose-requirements-updates/lastBuild/consoleText | 14:35 |
| mtreinish | sdague: and it's only the tempest new uploads, the other projects are normal | 14:35 |
| fungi | Ajaeger: we should probably make that a non-terminating failure in the job | 14:35 |
| *** otter768_ has quit IRC | 14:35 | |
| sdague | oh, maybe I don't have that. I don't seem to be getting emails on newuploads | 14:36 |
| sdague | just comments | 14:36 |
| fungi | Ajaeger: we have quite a few projects who want to receive requirements proposals, but who may from time to time break their .gitreview files and i'm not sure we want to fail to update all projects because of something like that on one particular project | 14:36 |
| fungi | Ajaeger: though that is definitely what happens at the moment | 14:36 |
| *** paul-- has joined #openstack-infra | 14:37 | |
| zzelle | fungi, YorikSar, based on current discussion we should perhaps differ the decision on https://review.openstack.org/93952 after release | 14:37 |
| *** nati_ueno has joined #openstack-infra | 14:37 | |
| fungi | zzelle: sounds good. i agree it's not a crucial bug fix | 14:37 |
| *** ihrachyshka has quit IRC | 14:37 | |
| YorikSar | zzelle, fungi: Agreed. I guess you'll have to live off master for some time if you'll need it. | 14:37 |
| zzelle | jeblair, reed, don't hesitate to share your viewpoint on https://review.openstack.org/93952 | 14:38 |
| Ajaeger | fungi: let me cook up a patch... | 14:38 |
| YorikSar | fungi: By the way, is there any concrete reason why git-review releases are so rare? | 14:38 |
| *** flaper87 is now known as flaper87|afk | 14:38 | |
| *** otter768 has joined #openstack-infra | 14:38 | |
| *** mriedem has quit IRC | 14:39 | |
| YorikSar | fungi: Or it's just that noone asked for one? | 14:39 |
| zzelle | YorikSar, fungi, don't worry, currently git review -t topic%submit provides a workaround | 14:39 |
| fungi | YorikSar: it's relatively stable, so the urgency for releases hasn't been there | 14:39 |
| *** zehicle_at_dell has quit IRC | 14:39 | |
| fungi | YorikSar: though i agree we could do a better job of releasing more often anyway | 14:39 |
| *** crc32 has joined #openstack-infra | 14:40 | |
| *** doug-fish has quit IRC | 14:40 | |
| fungi | YorikSar: i also waited longer than typical on this one because this release introduces some pretty radical packaging changes (switch to pbr, adding python dependencies, entrypoints wrapper, et cetera) | 14:40 |
| *** doug-fish has joined #openstack-infra | 14:40 | |
| *** mmaglana has joined #openstack-infra | 14:40 | |
| fungi | so i wanted to make sure everyone had a chance to weigh in with "i can't install this on mac os x" or whatever | 14:41 |
| *** crc32 has quit IRC | 14:41 | |
| *** doug-fish has quit IRC | 14:41 | |
| *** doug-fish has joined #openstack-infra | 14:41 | |
| YorikSar | fungi: In fact there were not too much development until recently. | 14:41 |
| *** crc32 has joined #openstack-infra | 14:41 | |
| *** mriedem has joined #openstack-infra | 14:41 | |
| *** wenlock has joined #openstack-infra | 14:42 | |
| fungi | YorikSar: agreed. it basically does the things it's intended to do, so a lot of the recent development has been code clean-up, cosmetic fixes, et cetera | 14:42 |
| zzelle | fungi, about pbr, pbr is supported by python setup.py develop ? | 14:42 |
| YorikSar | fungi: I use master on Mac OSX as well :) | 14:42 |
| fungi | zzelle: yes | 14:42 |
| *** doug-fish has quit IRC | 14:42 | |
| *** wenlock has quit IRC | 14:42 | |
| YorikSar | zzelle: It works with "pip install -e ./" | 14:42 |
| fungi | zzelle: if you look in setup.py, it's declared as a setup_requires | 14:43 |
| YorikSar | zzelle: (which is better) | 14:43 |
| *** doug-fish has joined #openstack-infra | 14:43 | |
| *** wenlock has joined #openstack-infra | 14:43 | |
| fungi | YorikSar: in fact, i install/upgrade it into a virtualenv with 'pip install -U .' and then symlink ~/bin/git-review to the entrypoints wrapper in the venv. works fine | 14:44 |
| *** doug-fish has quit IRC | 14:44 | |
| *** doug-fish has joined #openstack-infra | 14:44 | |
| *** weshay has quit IRC | 14:45 | |
| fungi | though i should still get around to adding ~/man to my manpath and linking the manpage for it so i can test git review --help and whatnot | 14:45 |
| YorikSar | fungi: Wouldn't "pip install --user -e ./" be easier? I collect all such tools in ~/.local, add ~/.local/bin to PATH and ~/.local/man to MANPATH | 14:46 |
| *** yaguang has joined #openstack-infra | 14:47 | |
| fungi | YorikSar: i like to be able to keep per-tool virtualenvs since i may switch between multiple versions of some when testing or run different things with conflicting requirements | 14:47 |
| *** mmaglana has quit IRC | 14:47 | |
| *** gokrokve has joined #openstack-infra | 14:48 | |
| *** lcostantino has quit IRC | 14:48 | |
| *** mmaglana has joined #openstack-infra | 14:48 | |
| openstackgerrit | Andreas Jaeger proposed a change to openstack-infra/config: Ignore failure of git review -s in propose_requirements_update https://review.openstack.org/96482 | 14:49 |
| Ajaeger | fungi, I hope the patch does its job ^ | 14:49 |
| *** enikanorov_ has joined #openstack-infra | 14:50 | |
| *** thedodd has joined #openstack-infra | 14:51 | |
| *** rdopiera has quit IRC | 14:51 | |
| *** zhiyan is now known as zhiyan_ | 14:52 | |
| *** praneshp has joined #openstack-infra | 14:53 | |
| *** mmaglana has quit IRC | 14:53 | |
| *** unicell has joined #openstack-infra | 14:55 | |
| YorikSar | fungi, zzelle: It looks like everything we want to make it to release has already landed. I guess, some testing and some cool ML anouncement is all that's needed :) | 14:56 |
| openstackgerrit | Dan Prince proposed a change to openstack-infra/config: Install the openstackci-yum module. https://review.openstack.org/95787 | 14:57 |
| openstackgerrit | Dan Prince proposed a change to openstack-infra/config: Add yum.openstack.org lightweight Fedora 20 mirror https://review.openstack.org/90875 | 14:57 |
| *** gokrokve has quit IRC | 14:57 | |
| openstackgerrit | Dan Prince proposed a change to openstack-infra/config: Import puppet-yum project https://review.openstack.org/90874 | 14:57 |
| *** weshay has joined #openstack-infra | 14:58 | |
| *** gokrokve has joined #openstack-infra | 14:58 | |
| *** rpwwpr has joined #openstack-infra | 14:58 | |
| *** doug-fish has quit IRC | 14:58 | |
| *** praneshp_ has joined #openstack-infra | 15:00 | |
| *** ominakov has joined #openstack-infra | 15:01 | |
| *** gokrokve has quit IRC | 15:02 | |
| fungi | Ajaeger: close i think. i've commented on a couple of issues | 15:03 |
| *** praneshp has quit IRC | 15:03 | |
| *** praneshp_ is now known as praneshp | 15:03 | |
| Ajaeger | fungi: I have been thinking about the pushd as well and was convinced I don't need to move it. But looking again with your comments I agree ;) | 15:04 |
| fungi | YorikSar: agreed. but with openstack disabling drafts on its gerrit this week (thus breaking -D for our users) i wanted to make one last call for whether anyone wanted to reintroduce the -W (set to work-in-progress) option as a suitable alternative | 15:04 |
| fungi | Ajaeger: yeah, i had to test it myself to be sure | 15:05 |
| fungi | Ajaeger: for f in foo bar baz ; do mkdir -p $f ; pushd $f ; pwd ; if test "$f" = "bar" ; then continue ; fi ; popd ; pwd ; rm -fr $f ; done | 15:05 |
| openstackgerrit | Andreas Jaeger proposed a change to openstack-infra/config: Ignore failure of git review -s in propose_requirements_update https://review.openstack.org/96482 | 15:05 |
| Ajaeger | fungi: I added also a missing "set -e" | 15:06 |
| fungi | Ajaeger: ooh! good catch. i completely missed that | 15:06 |
| signed8bit | So, I ended up with a launchpad username that is different than my gerrit username. Stacklytics links out to launchpad with my gerrit username which is a 404. Is there any safe recourse to reconciling my username foulup? | 15:06 |
| Ajaeger | fungi: so did I when writing it initially ;) | 15:06 |
| *** doug-fish has joined #openstack-infra | 15:06 | |
| fungi | signed8bit: i believe there is a file in stackforge/stackalytics to which you can submit a change mapping those up. SergeyLukjanov may be able to tell you more | 15:07 |
| Ajaeger | fungi: once you're in review mood, please give https://review.openstack.org/#/c/95377/ a good review as well... | 15:07 |
| Ajaeger | I'll be back later... | 15:07 |
| * anteaya would still like to see if the infra-manual commit can get in this week: https://review.openstack.org/#/c/92475/ | 15:07 | |
| signed8bit | thanks fungi | 15:07 |
| SergeyLukjanov | signed8bit, it's /etc/default_data.json in stackalytics | 15:08 |
| signed8bit | great thanks SergeyLukjanov | 15:08 |
| SergeyLukjanov | signed8bit, np | 15:09 |
| YorikSar | fungi: Oh, right. Totally forgot about that one. | 15:09 |
| *** esker has joined #openstack-infra | 15:11 | |
| *** esker has quit IRC | 15:12 | |
| *** esker has joined #openstack-infra | 15:12 | |
| *** samuelmz is now known as samuelmz-hungry | 15:14 | |
| *** weshay has quit IRC | 15:16 | |
| bodepd | mordred: I read your comments from last night, and I'm not sure what the solution looks like. I could add logic that says: if the directory exists, and it is not a git repo, wipe it out and then resume | 15:16 |
| mordred | yeah. I think that would work. | 15:17 |
| *** dstanek has left #openstack-infra | 15:18 | |
| sdague | hmmmm.... changing the commit message doesn't reset the vote any more? | 15:18 |
| bodepd | mordred: I could even do one better, by checking to see if the remote matches the key from SOURCE_MODULES | 15:18 |
| bodepd | (to also handle the case where the remote might change) | 15:19 |
| mordred | ooh. fancy! | 15:19 |
| *** gokrokve has joined #openstack-infra | 15:19 | |
| bodepd | yep. I've had my coffee :) | 15:19 |
| *** esker has quit IRC | 15:19 | |
| *** reed has joined #openstack-infra | 15:20 | |
| bodepd | mordred: good to see you guys are thinking about ansible for ad-hoc orchestration. I'm leaning towards it for the same | 15:20 |
| bodepd | mordred: so I'll likely be looking at things like how to kick off Puppet from ansible | 15:21 |
| bodepd | I'm a little nervous about scale, but maybe it will work across a message queue by the time that I care about that kind of scale | 15:22 |
| mordred | bodepd: well, we're using ssh to kick of puppet right now - it's working really well | 15:22 |
| *** weshay has joined #openstack-infra | 15:22 | |
| bodepd | mordred: with a bash script? | 15:22 |
| mordred | bodepd: yup | 15:22 |
| mordred | like, that's how puppet runs in prod for us now | 15:22 |
| mordred | we don't run agent daemons anywhere | 15:22 |
| mordred | it's WAY more stable | 15:23 |
| anteaya | reed: hey there, in austin? | 15:23 |
| *** jergerber has joined #openstack-infra | 15:23 | |
| bodepd | mordred: how do I do a patch to create new repos in openstack-infra? | 15:25 |
| bodepd | mordred: doc link would be appreciated! | 15:25 |
| mordred | bodepd: lemme find you a link ... | 15:25 |
| mordred | bodepd: https://review.openstack.org/#/c/92231/ <-- here is a change you can cargo cult | 15:27 |
| *** nati_ueno has quit IRC | 15:27 | |
| mordred | there is a narrative here: http://ci.openstack.org/stackforge.html | 15:27 |
| mordred | although there are things about that narrative that you'd want to change for it to be an infra repo - thus the link | 15:27 |
| mordred | bodepd: ^^ | 15:27 |
| *** mwagner_lap has joined #openstack-infra | 15:28 | |
| *** lcostantino has joined #openstack-infra | 15:28 | |
| *** blamar has quit IRC | 15:29 | |
| *** rpwwpr has quit IRC | 15:31 | |
| mordred | krotscheck: as you look at porting your puppet changes in - make sure you add: | 15:31 |
| mordred | class { 'puppi': | 15:31 |
| mordred | install_dependencies => false, | 15:32 |
| mordred | } | 15:32 |
| mordred | otherwise puppi decides it's going to install a bunch of things we don't want (/me learned this on laptop last night) | 15:32 |
| anteaya | ttx work on gerrit-powered-agenda seems to have stopped on May 7th | 15:32 |
| *** BadCub has joined #openstack-infra | 15:32 | |
| *** BadCub has quit IRC | 15:32 | |
| *** lcheng_ has joined #openstack-infra | 15:34 | |
| *** BadCub has joined #openstack-infra | 15:35 | |
| *** mmaglana has joined #openstack-infra | 15:41 | |
| *** esker has joined #openstack-infra | 15:44 | |
| *** weshay has quit IRC | 15:47 | |
| *** zhiyan_ is now known as zhiyan | 15:47 | |
| *** weshay has joined #openstack-infra | 15:48 | |
| *** jcoufal has quit IRC | 15:50 | |
| *** esker has quit IRC | 15:50 | |
| morganfainberg | mordred, i know you asked about the memcache dep for keystone client @summit: https://review.openstack.org/#/c/95987/ here is the spec that will be used to resolve that (split of middleware to it's own repo/package) | 15:51 |
| *** esker has joined #openstack-infra | 15:51 | |
| bodepd | mordred: thanks, I could probably find some stackforge commits I've done in the past as well for review :) | 15:51 |
| bodepd | mordred: ah, nm, that example is perfect! | 15:52 |
| mordred | morganfainberg: ++ | 15:53 |
| mordred | morganfainberg: also, aren't you looking at using the caching thing too? | 15:54 |
| *** rpwwpr has joined #openstack-infra | 15:54 | |
| *** gokrokve has quit IRC | 15:54 | |
| morganfainberg | mordred, yep, looking to move everything to dogpile, but that (likely) will take more effort/time to do right | 15:54 |
| *** gokrokve has joined #openstack-infra | 15:54 | |
| mordred | ++ | 15:54 |
| *** yaguang has quit IRC | 15:54 | |
| *** mrodden has joined #openstack-infra | 15:55 | |
| *** gokrokve has quit IRC | 15:55 | |
| openstackgerrit | Monty Taylor proposed a change to openstack-infra/config: Use ansible instead of direct ssh calls https://review.openstack.org/87821 | 15:56 |
| openstackgerrit | Monty Taylor proposed a change to openstack-infra/config: Remove salt https://review.openstack.org/96361 | 15:56 |
| openstackgerrit | Monty Taylor proposed a change to openstack-infra/config: Remove old salt references from the puppet https://review.openstack.org/96362 | 15:56 |
| jeblair | dhellmann_: mordred: i left some comments along with my +2 of 56760 | 15:56 |
| mordred | jeblair: cool - I will go read them | 15:56 |
| *** crc32 has quit IRC | 15:56 | |
| mordred | it's gonna be weird when we go 6 digits soon | 15:57 |
| *** zhiyan is now known as zhiyan_ | 15:57 | |
| *** lttrl has joined #openstack-infra | 15:57 | |
| anteaya | so salt is going away? | 15:57 |
| openstackgerrit | A change was merged to openstack/requirements: bump heatclient to latest version https://review.openstack.org/96377 | 15:57 |
| *** ihrachyshka has joined #openstack-infra | 15:57 | |
| mordred | jeblair: I like your suggestions | 15:58 |
| mordred | anteaya: I pushed those up to start the conversation | 15:58 |
| anteaya | you succeeded | 15:59 |
| anteaya | I'm listening | 16:00 |
| mordred | it's a complexity that we don't actually use - and while 2 years ago I thought that maybe using not-puppet would be nice, at this point we've actually gained a bunch of people with good puppet background who are excited about improving our puppet | 16:00 |
| mordred | so the only thing we'd be using it for would be ad-hoc remote execution | 16:00 |
| anteaya | it being ansible | 16:01 |
| mordred | salt | 16:01 |
| anteaya | where does "Use ansible instead of direct ssh calls" come in? | 16:01 |
| mordred | the reason I'm proposing ansible instead is that ansible doesn't need as much supporting stuff since it operates via ssh | 16:01 |
| anteaya | right, so ansible would be doing the ad-hoc remote execution | 16:02 |
| mordred | yah. but that stems from the fact that we're triggering puppet over ssh right now | 16:02 |
| mordred | but not able to use that setup for remote execution needs we have when doing maintenance | 16:02 |
| anteaya | okay | 16:03 |
| mordred | I've been keeping an ansible hosts list on my laptop and using it to do tasks - which is fine -but I can't really share any of those scripts with the team since we dont have the keys set up for it | 16:03 |
| anteaya | I'm just sad because I like the salt guys | 16:03 |
| anteaya | they are great people | 16:03 |
| anteaya | and I feel a part of teh salt failure is my inability to learn fast enough | 16:03 |
| anteaya | which I can't change | 16:03 |
| anteaya | it makes logical sense and it seems to be the momentum, so I don't plan on standing in the way | 16:04 |
| anteaya | but I am sad | 16:04 |
| anteaya | so that's all I can say | 16:04 |
| mordred | well, I agree - I'm sad too to be honest | 16:04 |
| mordred | I also like the salt guys | 16:05 |
| mordred | and I also did not learn salt fast enough | 16:05 |
| mordred | (so don't think that's all about you not learning it fast enough) | 16:05 |
| *** dougwig has joined #openstack-infra | 16:05 | |
| mordred | however - I think that's one of the things - I did not learn salt fast enough, but I got proficient in ansible in like a day | 16:05 |
| mordred | might have just been a function of trying to do different things | 16:05 |
| dougwig | hi all. i'm looking to get #neutron-lbaas added to eavesdrop for logging/archiving. it was recommended that i reach out here. | 16:06 |
| mordred | but the fact that I can just use ansible from my laptop was really helpful :) | 16:06 |
| anteaya | yeah, it all makes logical sense | 16:06 |
| mordred | dougwig: yes! this is a thing you can do via making a patch ... anteaya do we have docs for that? | 16:06 |
| anteaya | http://ci.openstack.org/irc.html | 16:07 |
| pleia2 | good morning | 16:07 |
| *** shakamunyi has joined #openstack-infra | 16:07 | |
| *** dguerri is now known as _dguerri | 16:07 | |
| mgagne | ansible is super cool, I was able to replace my lame loop over ssh in ~15m, no agent to install. =) | 16:07 |
| anteaya | hey pleia2 | 16:08 |
| mordred | mgagne: yup. see https://review.openstack.org/87821 | 16:08 |
| mgagne | mordred: yep, I saw you change. the summit was a big eye opened on lot of things, ansible being one of them. | 16:10 |
| *** _dguerri is now known as dguerri | 16:10 | |
| anteaya | mtreinish jeblair oh god I didn't know the infra/qa meeting was including third party CI | 16:10 |
| mordred | mgagne: it's really helpful in some ways | 16:10 |
| anteaya | s/meeting/meetup | 16:10 |
| mtreinish | anteaya: not really, it was just we thought the bootstrapping would be useful for people running thirdparty ci | 16:11 |
| mtreinish | it's not about running third party ci | 16:11 |
| anteaya | mtreinish: be prepared to have the thread highjacked | 16:11 |
| anteaya | mtreinish: I am all out of armour for this week, I had no idea that would be a part of this | 16:11 |
| mordred | mgagne: I especially think the combo of ansible for remote ex and playbooks of tasks + puppet for config management is really nice | 16:12 |
| anteaya | gear up | 16:12 |
| *** nati_ueno has joined #openstack-infra | 16:12 | |
| mgagne | mordred: ++ | 16:12 |
| *** zhiyan_ is now known as zhiyan | 16:12 | |
| mordred | speaking of - fungi, jeblair: ^^ when I Was making the ansible patch and looking through the docs, I was noticing some of the places in our docs where we talk about doing things on hosts - like the clearing out of stale workspaces | 16:12 |
| jeblair | anteaya: i believe at the summit we identified 3rd-party ci, bootstrapping, and the qa/infra overlap as one of the more useful things to do at a meetup | 16:13 |
| mordred | fungi, jeblair: and I think we could start to make playbooks for those sorts of thigns - not for auto-execution - but so that there is a vetted script in a known location on ci-puppetmaster that one of us can use | 16:13 |
| jeblair | mordred: yeah, that still applies to the very few static hosts we have; not so much to the unit test hosts anymore | 16:14 |
| *** marcoemorais has joined #openstack-infra | 16:14 | |
| *** terryw has joined #openstack-infra | 16:14 | |
| anteaya | jeblair: we mentioned it, and I had thought we had agreed that focusing on the non-third party items was a better use of our time | 16:14 |
| anteaya | either way it is done now | 16:14 |
| anteaya | oh and yay germany | 16:15 |
| mordred | jeblair: yah. I don't think it's a huge win for that task - but certainly if we find any sorts of things like that we need to do, encoding it is a nice option | 16:15 |
| zaro | morning | 16:16 |
| anteaya | morning zaro | 16:16 |
| jeblair | oops | 16:16 |
| *** blamar has joined #openstack-infra | 16:16 | |
| jeblair | there's a bug in my 'select patchset' gertty patch. i can't use the letter 'p' in an inline comment anymore. | 16:17 |
| mordred | jeblair: hehehe | 16:17 |
| pleia2 | http://www.elasticsearch.org/blog/openstack-elastic-recheck-powered-elk-stack/ | 16:17 |
| pleia2 | go us :) | 16:17 |
| *** otherwiseguy has quit IRC | 16:17 | |
| zaro | mika: jjb was released just about a week ago. is there anything specifically that you wanted in the release? | 16:17 |
| mtreinish | pleia2: cool | 16:17 |
| jeblair | strangely, it is not an issue very often | 16:18 |
| *** marcoemorais has quit IRC | 16:18 | |
| *** rpwwpr has quit IRC | 16:19 | |
| jeblair | sdague: 94196 reviewed | 16:19 |
| *** IvanBerezovskiy has left #openstack-infra | 16:20 | |
| *** zhiyan is now known as zhiyan_ | 16:21 | |
| *** hashar has quit IRC | 16:22 | |
| *** esker has quit IRC | 16:22 | |
| *** esker has joined #openstack-infra | 16:22 | |
| *** praneshp has quit IRC | 16:25 | |
| *** marcoemorais has joined #openstack-infra | 16:26 | |
| *** nati_ueno has quit IRC | 16:26 | |
| *** isviridov is now known as isviridov|away | 16:26 | |
| anteaya | reed: you around? | 16:27 |
| dougwig | mordred and anteaya: thank you, on it. | 16:27 |
| *** esker has quit IRC | 16:27 | |
| openstackgerrit | A change was merged to openstack-infra/jeepyb: Stop writing project configs https://review.openstack.org/90865 | 16:28 |
| sdague | jeblair: ok, what about the one underneath it? | 16:29 |
| *** e0ne_ has quit IRC | 16:30 | |
| openstackgerrit | Khai Do proposed a change to openstack-infra/jenkins-job-builder: reference tests as examples https://review.openstack.org/96279 | 16:30 |
| *** e0ne has joined #openstack-infra | 16:31 | |
| jeblair | sdague: did that too, minor note there | 16:31 |
| openstackgerrit | A change was merged to openstack-infra/jeepyb: Add PostgreSQL database support for gerritdb https://review.openstack.org/92686 | 16:31 |
| *** marcoemorais has quit IRC | 16:32 | |
| sdague | jeblair: ok, so here is my dilema. Part of using the exceptions here was to remove the super function nesting that we were doing previously. | 16:32 |
| sdague | but it seems that the function nesting is what's wanted? | 16:32 |
| *** marcoemorais has joined #openstack-infra | 16:32 | |
| anteaya | darmstadt was a centre for art nouveau: https://en.wikipedia.org/wiki/Darmstadt#Art_Nouveau | 16:33 |
| sdague | because we'll be back to 3 level nesting of functions if I change back 94196 | 16:33 |
| jeblair | sdague: well, in each case i'm only suggesting one conditional -- and that's based on a "something to do"/"nothing to do" decision that one of the methods makes. so there should only be one level of nesting. | 16:33 |
| jeblair | sdague: i admit, that's a personal preference because i don't like "raise EverythingIsNormalException()". | 16:34 |
| jeblair | sdague: however, i don't feel a super strong ownership stake in that part of the code | 16:34 |
| *** jooools has quit IRC | 16:34 | |
| jeblair | sdague: so if fungi and mordred are okay with "raise EverythingIsNormalException()" i can go along with that | 16:34 |
| jeblair | sdague: however, in that case, i would suggest catching it specifically and not logging it, because, after all, everything is normal. | 16:35 |
| *** e0ne has quit IRC | 16:35 | |
| sdague | yeh, I can change the exceptions to a "NothingToDo" exception | 16:35 |
| sdague | for when we're just saying short circuit | 16:35 |
| openstackgerrit | Dan Bode proposed a change to openstack-infra/config: Allow modules to be installed from source https://review.openstack.org/96336 | 16:36 |
| jeblair | sdague: but your last comment suggests that you may still be missing a nuance -- there should only be a single if statement | 16:36 |
| sdague | I mean on the bottom one | 16:36 |
| *** jpich has quit IRC | 16:36 | |
| jeblair | sdague: oh. isn't it the case for that one too, though? | 16:37 |
| sdague | yeh, so the top one would have 1 if | 16:37 |
| sdague | the bottom one would have 2 | 16:37 |
| sdague | the bottom one started with 3 | 16:37 |
| jeblair | sdague: "fetch_dash; if copy_dash: push_dash;" | 16:37 |
| sdague | yep | 16:37 |
| sdague | not talking about that one | 16:38 |
| sdague | talking about 94196 | 16:38 |
| jeblair | sdague: oh, you and i are apparently looking at git branches from opposite directions :) | 16:38 |
| sdague | heh | 16:39 |
| sdague | fair enough :) | 16:39 |
| jeblair | sdague: "fetch_config; if copy_acl_config: create_groups; push_acl_config" | 16:39 |
| *** samuelmz-hungry is now known as samuelmz | 16:39 | |
| jeblair | sdague: so that's still one level, it just pushes create groups and push_acl_config both into the conditional | 16:40 |
| sdague | fetch_config; if copy_acl_config: if create_groups: push_acl_config | 16:40 |
| jeblair | sdague: no if is needed for create_groups | 16:40 |
| sdague | oh, gotcha | 16:41 |
| sdague | ok, right, there was no noop path there before | 16:41 |
| clarkb | wow sb | 16:42 |
| *** gokrokve has joined #openstack-infra | 16:42 | |
| *** _nadya_ has quit IRC | 16:44 | |
| bodepd | mordred: one thing I ddin't see if that example patch. Do new repos need to clone an existing repo, or is that optional? | 16:45 |
| openstackgerrit | Emilien Macchi proposed a change to openstack-infra/devstack-gate: features: do not drop ceilometer when using grenade https://review.openstack.org/96518 | 16:45 |
| jeblair | sdague: i figure i'm permanently to blame for confusion around this for attaching a comment to the wrong function earlier. sorry. :/ but i do like how it looks with exceptions for errors while having the single go/no-go decision expressed as a conditional; i think it illustrates the expected process very well. | 16:46 |
| *** lcheng_ has quit IRC | 16:46 | |
| jeblair | clarkb: the most important thing is that i broke typing the letter 'p' in inline comments in gertty | 16:47 |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/jeepyb: add dashboard support to jeepyb https://review.openstack.org/94260 | 16:47 |
| *** ociuhandu has quit IRC | 16:47 | |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/jeepyb: use exceptions instead of return True/False https://review.openstack.org/94196 | 16:47 |
| jeblair | clarkb: and strangely it took many reviews for me to notice that | 16:47 |
| sdague | so how about that | 16:47 |
| mordred | bodepd: it's optional | 16:48 |
| sdague | doing an early return instead of nesting | 16:48 |
| clarkb | jeblair: huh I didnt notice either | 16:48 |
| sdague | which I'm hoping satisfies all the things | 16:48 |
| mordred | bodepd: if you don't tell it an upstream to clone, you'll just get a blank repo | 16:48 |
| *** amotoki has quit IRC | 16:48 | |
| bodepd | mordred: yay | 16:49 |
| clarkb | jeblair: I would like to push on the zuul test reorg this week to hopefully setup zuul changes for next week | 16:49 |
| clarkb | we keep adding tests for features making it harder to get the test reorg in | 16:50 |
| jeblair | clarkb: ack, will prioritize | 16:50 |
| clarkb | I think it needs a rebase I eill do that asap | 16:51 |
| jeblair | clarkb, fungi, mordred, SergeyLukjanov, jhesketh: i'll be out from the middle of next week for 1-1.5 weeks | 16:51 |
| jesusaurus | clarkb: what is the test reorg? | 16:51 |
| mordred | jesusaurus: okie | 16:51 |
| mordred | gah | 16:51 |
| mordred | jeblair: okie | 16:51 |
| jeblair | i'll be in the wilderness so definitely not accessible | 16:52 |
| SergeyLukjanov | jeblair, ack | 16:52 |
| Ajaeger | 1 - 1.5 weeks? Are you not sure whether you get lost- or whether you like it? ;) | 16:52 |
| *** SumitNaiksatam has quit IRC | 16:54 | |
| *** dkliban_afk is now known as dkliban | 16:55 | |
| fungi | mmmtastyscrollback | 16:56 |
| openstackgerrit | Monty Taylor proposed a change to openstack-infra/config: Notify in IRC when puppet run fails https://review.openstack.org/96519 | 16:57 |
| *** shivharis has joined #openstack-infra | 16:57 | |
| *** yfried has joined #openstack-infra | 16:59 | |
| *** shivharis has left #openstack-infra | 16:59 | |
| *** dangers_away is now known as dangers | 17:00 | |
| fungi | bodepd: i've definitely cleaned up a *ton* of local git clones on servers where vcsrepo didn't update the git remote | 17:01 |
| *** andreykurilin_ has joined #openstack-infra | 17:01 | |
| fungi | bodepd: so would be a welcome feature/fix | 17:01 |
| *** shardy is now known as shardy_afk | 17:01 | |
| clarkb | jesusaurus: it breaks zuul monolithic single test class into pieces so that we can have classes that test with different base configs and so on | 17:01 |
| *** harlowja_away is now known as harlowja_ | 17:02 | |
| *** dangers is now known as dangers_away | 17:02 | |
| jesusaurus | ahh | 17:03 |
| *** dangers_away is now known as dangers | 17:03 | |
| sdague | jeblair: going to head out for a ride, if you take a look at the jeepyb changes again I can address feedback when I'm back. | 17:04 |
| *** sarob has joined #openstack-infra | 17:04 | |
| *** arnaud has joined #openstack-infra | 17:05 | |
| openstackgerrit | Morgan Fainberg proposed a change to openstack/requirements: Bump keystoneclient minimum version https://review.openstack.org/96520 | 17:06 |
| bodepd | fungi: If I understand correctly, the latest version of https://review.openstack.org/#/c/96336 should fix that for things defined as SOURCE_MODULES | 17:07 |
| *** gokrokve_ has joined #openstack-infra | 17:07 | |
| *** sarob_ has joined #openstack-infra | 17:07 | |
| *** ominakov has quit IRC | 17:09 | |
| fungi | bodepd: neat! | 17:10 |
| *** sarob has quit IRC | 17:11 | |
| openstackgerrit | Dan Bode proposed a change to openstack-infra/config: Add puppet-pip as a unique project https://review.openstack.org/96522 | 17:11 |
| *** gokrokve has quit IRC | 17:12 | |
| *** SumitNaiksatam has joined #openstack-infra | 17:12 | |
| *** zhiyan_ is now known as zhiyan | 17:12 | |
| clarkb | jeblair: 85230 is before the test reorg change. It has a +2 from you and +1 from jhesketh. I don't want to rebase it as it seems to be happy | 17:13 |
| clarkb | jeblair: maybe we can approve that change then I can rebase the remaining stack? | 17:13 |
| fungi | jeblair: want a traceback from pressing pgdn in an open review summary in gertty? | 17:13 |
| fungi | jeblair: something about raise ListBoxError, "Widget %r at position %r within listbox calculated %d rows but rendered %d!"% (widget,w_pos,w_rows, canvas.rows()) | 17:13 |
| fungi | urwid.listbox.ListBoxError: Widget <RevisionRow selectable box/flow widget> at position 12 within listbox calculated 11 rows but rendered 3! | 17:13 |
| fungi | could just be because ETINYSCREEN | 17:14 |
| *** _nadya_ has joined #openstack-infra | 17:14 | |
| *** yfried has quit IRC | 17:14 | |
| clarkb | fungi: at some point we should get you a proper screen :) | 17:14 |
| *** crc32 has joined #openstack-infra | 17:15 | |
| Shrews | most phones are bigger than fungi's laptop screen | 17:15 |
| fungi | mmm, also it seems to want to do mouse integration, and so freaks out if i try to select text in my xterm | 17:15 |
| *** fanhe has quit IRC | 17:16 | |
| clarkb | fungi: use shift to select text | 17:16 |
| clarkb | fungi: I run into that a lot too | 17:16 |
| * fungi is one of those luddites who thinks console apps should be unaware of window system pointers | 17:16 | |
| vponomaryov | fungi, clarkb: Hello, if you have some time, could you please review this: https://review.openstack.org/#/c/95207/ | 17:16 |
| jeblair | fungi: oh i think that's fixed in my latest patchsets | 17:16 |
| fungi | jeblair: okay, will try | 17:16 |
| jeblair | fungi: except you won't be able to use 'p' in comments. i'm about to fix that one. | 17:17 |
| *** lttrl has quit IRC | 17:17 | |
| fungi | i did see something in review about "make things work on fungi's teensy screens" | 17:17 |
| * fungi paraphrases | 17:17 | |
| *** derekh_ has quit IRC | 17:18 | |
| *** yfried has joined #openstack-infra | 17:18 | |
| *** duran has joined #openstack-infra | 17:18 | |
| jeblair | fungi: yeah, it's tested down to 3x3. :) | 17:19 |
| *** Longgeek has quit IRC | 17:19 | |
| *** terryw is now known as otherwiseguy | 17:19 | |
| fungi | heehee | 17:20 |
| *** duran has quit IRC | 17:20 | |
| *** markmc has quit IRC | 17:21 | |
| *** zhiyan is now known as zhiyan_ | 17:22 | |
| *** jlibosva has quit IRC | 17:23 | |
| *** jergerber has quit IRC | 17:23 | |
| *** jlibosva has joined #openstack-infra | 17:24 | |
| *** _nadya_ has quit IRC | 17:25 | |
| *** Ryan_Lane has joined #openstack-infra | 17:26 | |
| Shrews | fungi: tonight + meetup + olaph = beer^3, yes? | 17:27 |
| zaro | jeblair: so i've been looking at adding SSL to gearman-plugin but am not getting very far. All i seem to get is that there's no SSL drop in library replacement for SocketChannel, you would need to use the java SSLEngine library to make it work. | 17:28 |
| *** jlibosva has quit IRC | 17:28 | |
| openstackgerrit | Yuriy Taraday proposed a change to openstack-infra/git-review: Add an option to mark uploaded patchsets as WIP https://review.openstack.org/96528 | 17:29 |
| zaro | jeblair: i've read lots of info on SSLEngine but am not understanding how to integrate it with gearman-java. sorry :( | 17:29 |
| krotscheck | mordred: Check | 17:30 |
| jeblair | zaro: thanks. do you have an idea of how much effort that might take? that way we can weigh it against other options (like switching all at once or partitioning the system). | 17:30 |
| YorikSar | fungi, zzelle: ^^ - just uploaded it using "git review -W" | 17:30 |
| clarkb | YorikSar: nice | 17:31 |
| openstackgerrit | James E. Blair proposed a change to stackforge/gertty: Handle (ignore) no-diff renames https://review.openstack.org/95769 | 17:32 |
| openstackgerrit | James E. Blair proposed a change to stackforge/gertty: Add patchset selection in diff https://review.openstack.org/95768 | 17:32 |
| jeblair | clarkb, fungi: ^ 95769 is the tip. | 17:32 |
| mordred | jeblair: would stunnel or apache be an option for ssl term? | 17:32 |
| zaro | jeblair: well i guess that depends on who does the work. from my reading of SSLEngine everyone seems to comments on how 'notoriously' difficult and non-trial it is to use. | 17:32 |
| mordred | jeblair: oh, wait - hangon. I'm stupid - let me rephrase | 17:32 |
| mordred | jeblair: would stunnel be an option for being an ssl client? | 17:32 |
| fungi | Shrews: tonight + meetup + olaph = beer^3^3, ja! | 17:33 |
| jeblair | mordred: that might work, yeah. | 17:33 |
| fungi | YorikSar: awesome! looking | 17:33 |
| clarkb | jeblair: thoughts on approving 85230? will make my rebasing simpler | 17:34 |
| zaro | jeblair: sorry, i mean i have no idea how much effort it would take. i don't even know where to start actually. I could not find enough relavent examples to go on. | 17:34 |
| openstackgerrit | Dan Bode proposed a change to openstack-infra/config: Allow modules to be installed from source https://review.openstack.org/96336 | 17:34 |
| jeblair | clarkb: done | 17:35 |
| clarkb | woot thanks | 17:35 |
| bodepd | jesusaurus: I'm not sure about your recommended error checking. I removed the dead code in the latest revision | 17:35 |
| *** ildikov has quit IRC | 17:35 | |
| *** arnaud has quit IRC | 17:35 | |
| jeblair | zaro, mordred: so we'll go with "quite a lot of effort" then. :) | 17:36 |
| bodepd | jesusaurus: I can do it, but it will be a lot of code and processing. basically, iterate through every source key, then iterate through every forge module and see if they match | 17:36 |
| bodepd | jesusaurus: is there something else that you had in mind? | 17:36 |
| *** nati_ueno has joined #openstack-infra | 17:37 | |
| *** gyee has joined #openstack-infra | 17:37 | |
| zzelle | YorikSar, fungi: i was about to code it :) | 17:38 |
| bodepd | jesusaurus: it also won't tell us if there are conflicts b/c of deps of forge modules | 17:38 |
| YorikSar | zzelle: Ha! I'm super fast, mkay :) | 17:39 |
| bodepd | jesusaurus: I think a better strategy is to install everything from source and not use the forge | 17:39 |
| jesusaurus | bodepd: oh did you remove that conditional? | 17:39 |
| bodepd | jesusaurus: what conditional? | 17:39 |
| fungi | jeblair: running gertty from 95768 now and previously discovered crashes are gone. awesome! | 17:39 |
| YorikSar | I've added my concerns there. | 17:39 |
| bodepd | maybe I misunderstood.. hold on | 17:39 |
| jesusaurus | bodepd: i don't disagree, but if a module ends up in both vars, I think we should alert the user which is actually being used | 17:39 |
| fungi | jeblair: also, i notice that home and end don't jump to top/bottom. is that just a missing feature? if so, i'll try to remember to write a patch | 17:39 |
| *** ihrachyshka has quit IRC | 17:40 | |
| *** melwitt has joined #openstack-infra | 17:40 | |
| YorikSar | zzelle, clarkb, fungi: Now I'll disapear. Looking forward to comments on WIP patch. | 17:40 |
| bodepd | jesusaurus: I misunderstood your comment | 17:40 |
| bodepd | jesusaurus: I though you meant to check MODULES against SOURCE_MODULES for conflicts | 17:40 |
| jesusaurus | bodepd: oh, what did you think i meant? | 17:40 |
| zzelle | YorikSar, bye | 17:41 |
| jesusaurus | oh, no, you were already checking to see if the module was in MODULES to see if you needed to rm the directory | 17:41 |
| jeblair | fungi: i think that kind of navigation should be handled by the listbox. i don't know right off the top of my head the right way to fix that; if there's a global thing or if we need to add keypress handlers that catch 'home' and return some command code... | 17:41 |
| jeblair | fungi: clarkb might have seen it in his explorations... | 17:42 |
| bodepd | jesusaurus: ok, for that comment. SOURCES has priority. | 17:42 |
| jeblair | fungi: but regardless, that does seem like good idea | 17:42 |
| jesusaurus | bodepd: yes, my comment was that we shouldnt SILENTLY make one have priority over the other | 17:42 |
| bodepd | jesusaurus: if a directory is already there, it deletes it. it could be there for several different reasons though | 17:42 |
| *** dizquierdo has quit IRC | 17:43 | |
| bodepd | jesusaurus: it could have been there b/c: 1. it had previously been installed via the forge and now it's installed by source, it could be a conflict that could be based on either a duplicate forge install or a dep of a forge module | 17:44 |
| *** praneshp has joined #openstack-infra | 17:44 | |
| bodepd | jesusaurus: ok, so if I print out a message before I delete, does that work? | 17:44 |
| openstackgerrit | A change was merged to openstack-infra/zuul: Fix races in timer trigger tests. https://review.openstack.org/85230 | 17:44 |
| *** ociuhandu has joined #openstack-infra | 17:45 | |
| jesusaurus | bodepd: youve changed the logic on the conditional, so addressing my previous comment isnt as simple as it was before | 17:45 |
| jesusaurus | oh whoops, nevermind, i was looking at the wrong line | 17:46 |
| samuelmz | sdague, ping | 17:46 |
| openstackgerrit | Dan Bode proposed a change to openstack-infra/config: Allow modules to be installed from source https://review.openstack.org/96336 | 17:47 |
| *** pblaho has quit IRC | 17:47 | |
| bodepd | jesusaurus: check out that latest commit and see if it resolves your concern | 17:48 |
| jesusaurus | bodepd: wfm | 17:48 |
| clarkb | jeblair: fungi I haven't tried home and end | 17:48 |
| clarkb | I do keep trying to gg and G though | 17:48 |
| bodepd | jesusaurus: now I just need to re-run my annoying manual tests :) | 17:48 |
| *** atiwari has joined #openstack-infra | 17:51 | |
| samuelmz | hi guys, I'm getting strange failures on jenkins execution on my patch (https://review.openstack.org/#/c/96441/) | 17:52 |
| openstackgerrit | Ramy Asselin proposed a change to openstack-infra/config: Allow choice of GIT protocol used. https://review.openstack.org/96539 | 17:52 |
| samuelmz | I just modified a test and put a @test.skip_because begore it | 17:52 |
| samuelmz | s/begore/before | 17:52 |
| Ajaeger | samuelmz: it fails in pep8, doesn't it? | 17:53 |
| Ajaeger | click on the pep8 line and follow it to the console.html file, it will tell you details... | 17:53 |
| samuelmz | Ajaeger, yes.. but failures on pep8 may reflects on other failures? | 17:53 |
| Ajaeger | samuelmz: one problem at a time ;) Fix the pep8 failures - looking at them, they are valid failures. | 17:54 |
| Ajaeger | You can run flake8 locally to test... | 17:54 |
| samuelmz | Ajaeger, for check-tempest-dsvm-neutron-heat-slow-icehouse I'm getting "Stack heat-313879233 failed to reach CREATE_COMPLETE status within the required time (900 s)." | 17:54 |
| samuelmz | Ajaeger, ok.. I'm gonna fix it first ..ç | 17:55 |
| Ajaeger | samuelmz: might indeed be some unrelated problem | 17:55 |
| *** _nadya_ has joined #openstack-infra | 17:55 | |
| samuelmz | Ajaeger, I'm using run_tempest.sh, how can I run flake8? | 17:56 |
| *** openstackgerrit has quit IRC | 17:57 | |
| Ajaeger | samuelmz: "tox -e pep8" | 17:57 |
| Ajaeger | Or just run "flake8" on the console | 17:58 |
| bodepd | I opened a ticket on that stdlib bug. Tickets don't get printed to irc? Is that something you guys really use? | 17:58 |
| *** yamahata has quit IRC | 17:59 | |
| mordred | bodepd: when you say tickets ... what do you mean? | 18:00 |
| clarkb | jeblair: this rebase is fun :) it may take a bit before I am happy with it | 18:00 |
| clarkb | jeblair: trying to make sure I don't miss anything little | 18:00 |
| mordred | bodepd: (and no, we don't have new bugs announce themselves here) | 18:00 |
| *** rfolco has quit IRC | 18:00 | |
| jeblair | clarkb: :/ | 18:00 |
| *** angdraug has joined #openstack-infra | 18:01 | |
| *** sarob_ has quit IRC | 18:01 | |
| jeblair | oh look, here's an example of diffing images: https://review.openstack.org/#/c/89865/2 | 18:02 |
| *** esker has joined #openstack-infra | 18:02 | |
| *** esker has quit IRC | 18:03 | |
| jeblair | (they don't actually visually change, they're just being compressed) | 18:03 |
| jeblair | mordred, krotscheck, fungi: ^ | 18:03 |
| jeblair | (now that the gerrit config setting has been changed to display those) | 18:03 |
| *** tsv has joined #openstack-infra | 18:04 | |
| * krotscheck +1’d that review | 18:04 | |
| clarkb | jeblair: well I am running tox on it now so maybe it wasn't too bad | 18:04 |
| clarkb | jeblair: mostly just a few new features added to the support classes that I needed to make sure didn't break | 18:04 |
| jeblair | krotscheck: yeah, i wasn't sure if that had been enabled when you originally looked at it, it was a recent change | 18:04 |
| krotscheck | Well, I downloaded them at the time. | 18:04 |
| jeblair | yeah, now gerrit displays them inline | 18:04 |
| *** rpwwpr has joined #openstack-infra | 18:05 | |
| krotscheck | Neat :) | 18:05 |
| krotscheck | I’m just a little worried about putting Large Binary Files (tm) into Git | 18:05 |
| krotscheck | Small things like this are neat. | 18:05 |
| krotscheck | Hey, does it show fonts? | 18:05 |
| krotscheck | :D | 18:05 |
| clarkb | as long as we don't put isos in there we should be fine | 18:05 |
| jeblair | yeah, it's something to be cognizant of | 18:05 |
| clarkb | and I think AaronGr knows how to deal with that now | 18:05 |
| mordred | krotscheck: are you saying I shouldn't put all my warez into infra/config ? | 18:05 |
| jeblair | i think screenshots/illustrations/etc in specs repos would probably be okay too. | 18:06 |
| jeblair | giant pdfs, or huge bitdepth tiff files, maybe not so much | 18:06 |
| krotscheck | mordred: Are we still in an age of warez? | 18:07 |
| nibalizer | /dcc krotscheck | 18:08 |
| tsv | Hi there. I pushed a patch to a barbican client review (https://review.openstack.org/#/c/80124) and do not see Jenkins job starting. This project has recently moved from stackforge to openstack. Could that be the reason ? What has to be done now to get Jenkins to check the patch ? | 18:08 |
| mordred | krotscheck: how could we not be? | 18:08 |
| krotscheck | mordred: How many not-open-source things do you use? | 18:09 |
| fungi | clarkb: funny, when home and end didn't work i too instinctively tried gg and G ;) | 18:10 |
| mordred | krotscheck: wait - I don't need to use warez to get emacs? | 18:11 |
| samuelmz | Ajaeger, I fixed pep8 failures, should I submit a new patch set and see what happens ? :) | 18:12 |
| bodepd | mordred: yeah, I meant launchpad. I filed a ticket! | 18:12 |
| rcarrillocruz | lulz at warez :D | 18:12 |
| krotscheck | mordred: Well, maybe emacs. | 18:12 |
| krotscheck | mordred: Who knows what’s in that. | 18:13 |
| Ajaeger | samuelmz: if you're convinced your patch is fine, then send it in | 18:13 |
| *** arnaud has joined #openstack-infra | 18:13 | |
| * Ajaeger would push it ;) | 18:13 | |
| *** zhiyan_ is now known as zhiyan | 18:13 | |
| *** afazekas has joined #openstack-infra | 18:14 | |
| fungi | tsv: that does seem odd... uploaded a new patchset at 2014-05-28 22:40 utc, and it never started testing. i'll check zuul's log | 18:14 |
| fungi | bodepd: parking ticket | 18:15 |
| *** dkehn has quit IRC | 18:15 | |
| Ajaeger | fungi, tsv: The patch still has "stackforge/..." as project - is that really correct? | 18:15 |
| fungi | Ajaeger: oh, wow, right. that got renamed. i wonder why it still shows up with the old project name? | 18:16 |
| fungi | it definitely predates "the great renaming" so maybe we missed something new-gerrit-wise | 18:16 |
| rcarrillocruz | guys, where's handled tarballs.openstack.org? is it a server managed by infra? | 18:17 |
| clarkb | rcarrillocruz: it is a server managed by infra | 18:17 |
| fungi | Ajaeger: https://review.openstack.org/#/admin/projects/stackforge/python-barbicanclient still exists too | 18:17 |
| rcarrillocruz | and it's managed by puppet or... ? | 18:17 |
| samuelmz | Ajaeger, there are other errors like for check-tempest-dsvm-neutron-heat-slow-icehouse I'm getting "Stack heat-313879233 failed to reach CREATE_COMPLETE status within the required time (900 s)." | 18:17 |
| *** dkehn has joined #openstack-infra | 18:17 | |
| clarkb | rcarrillocruz: yup its managed by puppet. It is really just a vhost on static.openstack.org | 18:17 |
| Ajaeger | samuelmz: sorry, can't help further with those ;( | 18:17 |
| rcarrillocruz | ahh | 18:17 |
| fungi | rcarrillocruz: specifically it's an apache vhost on the static.openstack.org instance in rackspace. see the openstack-infra/config:modules/openstack_project/manifests/static.pp file | 18:18 |
| rcarrillocruz | ok, thx for pointing me clarkb | 18:18 |
| samuelmz | Ajaeger, but this has nothing to do with what I've modified, then I gonna resubmit | 18:18 |
| rcarrillocruz | just wondering what's the process when , let's say, we create a new tarball for gerrit... | 18:18 |
| samuelmz | Ajaeger, np, you already helped me a lot | 18:18 |
| samuelmz | Ajaeger, thnaks | 18:18 |
| Ajaeger | samuelmz: unless there's some strange side effect or implicit dependency... | 18:18 |
| clarkb | we lost openstackgerrit | 18:19 |
| clarkb | jeblair: I have rebased the zuul changes and the test refactor should be good now | 18:19 |
| Ajaeger | fungi: that looks strange. barbican itself is only there once, but the client is listed twice | 18:19 |
| fungi | something is definitely wrong... both https://review.openstack.org/#/admin/projects/openstack/python-barbicanclient and https://review.openstack.org/#/admin/projects/stackforge/python-barbicanclient seem to exist in gerrit | 18:19 |
| jeblair | clarkb: is gerritbot dead? | 18:19 |
| clarkb | jeblair: looks like it timedout recently | 18:19 |
| clarkb | jeblair: I was going to look at it in a minute | 18:19 |
| clarkb | rcarrillocruz: a jenkins job creates an artifact then SCPs it to tarballs.openstack.org | 18:20 |
| *** _nadya_ has quit IRC | 18:20 | |
| Ajaeger | fungi: it seems fine for other projects - I don't see climate and neither stackforge/barbican | 18:20 |
| rcarrillocruz | excellent, thx | 18:20 |
| *** _nadya_ has joined #openstack-infra | 18:20 | |
| clarkb | looks like gerritbot is running | 18:20 |
| clarkb | and has been since the 25th so maybe freenode derpage? I will give it a few minutes to correct itself before I hit it with fire | 18:20 |
| *** pelix has quit IRC | 18:21 | |
| fungi | whoops... i see why. https://review.openstack.org/#/c/94997/5/modules/openstack_project/files/review.projects.yaml renamed both barbican and python-barbicanclient but our lists on https://etherpad.openstack.org/p/repo-renaming-2014-05-23 for stuff to rename only included barbican, not python-barbicanclient | 18:22 |
| fungi | so when we merged 94997 it created a new openstack/python-barbicanclient project | 18:22 |
| fungi | ugh | 18:22 |
| *** zhiyan is now known as zhiyan_ | 18:23 | |
| anteaya | so the patch to create xstatic repos for horizon creates 16 new repos, we know that? https://review.openstack.org/#/c/95716/4 | 18:23 |
| jeblair | fungi: and of course it populated it with the upstream. | 18:24 |
| angdraug | http://logs.openstack.org/95/94295/9/check/check-tempest-dsvm-full/c0d8a40/console.html#_2014-05-29_07_46_23_873 | 18:24 |
| angdraug | is ^ a known problem? | 18:24 |
| jeblair | that was bound to happen eventually :/ | 18:24 |
| fungi | jeblair: the old, original upstream, yep | 18:24 |
| clarkb | wow :( | 18:24 |
| * fungi checks to see what can be untangled | 18:24 | |
| jeblair | fungi: i just made openstack/python-barbicanclient | 18:24 |
| jeblair | read-only using the gui | 18:24 |
| clarkb | angdraug: I would ask the qa team | 18:24 |
| fungi | jeblair: thanks. i can confirm it's had no changed merged since before the rename, thankfully | 18:25 |
| jeblair | fungi: it looks like there are no changes for it in gerrit, so i think we may be able to untangle it | 18:25 |
| clarkb | oh good | 18:25 |
| fungi | jeblair: yep, agreed, just checked that too | 18:25 |
| Ajaeger | fungi: ;( | 18:26 |
| jeblair | (that's why i made it read-only; to hopefully keep it in that state) | 18:26 |
| fungi | so now we just need to determine how to un-create the openstack/python-barbicanclient project in gerrit cleanly, so we can rename the old one properly | 18:26 |
| *** zehicle_at_dell has joined #openstack-infra | 18:27 | |
| jeblair | fungi: yeah, it's basically a full project delete (which one to delete doesn't really matter). i think we did that once (a similar oops) | 18:28 |
| fungi | this looks like a fairly recent account of someone going through it: http://blogs.collab.net/teamforge/here-at-last-how-to-delete-projects-in-gerrit#.U4d8IN3rJGo | 18:28 |
| *** hashar has joined #openstack-infra | 18:28 | |
| fungi | nevermind... "the Gerrit project itself remains and all the code reviews associated with the repository will be still shown on Gerrit WebUI" | 18:28 |
| clarkb | pleia2: oh hey http://www.elasticsearch.org/blog/openstack-elastic-recheck-powered-elk-stack/ | 18:28 |
| fungi | i may just go hunting for project ids in gerrit's database | 18:29 |
| *** arnaud has quit IRC | 18:29 | |
| jeblair | fungi: that refers to a 'delete project plugin'. | 18:29 |
| clarkb | we did it with puplications | 18:29 |
| fungi | oh, didn't read far enough into the article, but it talks about using the delete-projeck plug-in which we may or may not want to | 18:29 |
| fungi | yeah | 18:29 |
| fungi | so the question is, do we want to install and have that plug-in available for the future? in which case maybe we should use it for this too | 18:30 |
| pleia2 | clarkb: yes! went up this morning :) | 18:31 |
| clarkb | fungi: it can't hurt. I would only be worried about the ACLs around it | 18:31 |
| clarkb | eg who can delete things | 18:31 |
| fungi | seems to imply it's an administrators-only command | 18:31 |
| *** radez_g0n3 is now known as radez | 18:32 | |
| *** timrc is now known as timrc-afk | 18:32 | |
| tsv | @fungi thanks for checking. What do you suggest I should do now to get Jenkins building ? | 18:32 |
| *** bodepd has quit IRC | 18:33 | |
| Ajaeger | tsv: Wait ;) | 18:33 |
| tsv | will do :) | 18:33 |
| jeblair | fungi: the code in git master suggests that there are new capabilities for it | 18:33 |
| *** pcm_ has joined #openstack-infra | 18:33 | |
| *** morganfainberg is now known as morganfainberg_Z | 18:33 | |
| clarkb | we will need to build the plugin for our version of gerrit iirc | 18:33 |
| clarkb | we should have a template for doing that in JJB already | 18:34 |
| jeblair | "delete project" and "delete own project" | 18:34 |
| Ajaeger | tsv: fungi needs to do his magic to fix it and then it should work on its own - and if not, run "recheck no bug". But first let fungi fix everything and give the green light... | 18:34 |
| *** atiwari has quit IRC | 18:34 | |
| jeblair | // Don't let people delete All-Projects, that's stupid | 18:35 |
| jeblair | throw new UnloggedFailure("Perhaps you meant to rm -fR " + site.site_path); | 18:35 |
| fungi | tsv: yeah, you've caught us in a rather unfortunate mistake. we didn't actually rename the barbicanclient project when we should have. i'm going to revert the half of https://review.openstack.org/94997 which dealt with the client so we can take another shot at it | 18:35 |
| jeblair | it apparently has a sense of humor ^ | 18:35 |
| *** ildikov has joined #openstack-infra | 18:35 | |
| jeblair | fungi: why revert? | 18:36 |
| *** ihrachyshka has joined #openstack-infra | 18:36 | |
| fungi | jeblair: i guess it depends on how quickly we expect to be able to take a gerrit outage to add the plug-in, delete the miscreated new project and retry the rename steps for it | 18:36 |
| *** morganfainberg_Z is now known as morganfainberg | 18:37 | |
| jeblair | fungi: i think it's sort of an emergency situation. i think we should do it asap. | 18:37 |
| fungi | jeblair: fair enough | 18:37 |
| fungi | working on a patch now to add the new plugin | 18:39 |
| *** aconrad has joined #openstack-infra | 18:39 | |
| *** hashar has quit IRC | 18:41 | |
| jeblair | fungi: cool. https://etherpad.openstack.org/p/rename-python-barbicanclient | 18:41 |
| *** ihrachyshka has quit IRC | 18:43 | |
| *** jp_at_hp has quit IRC | 18:45 | |
| tsv | Ajaeger: fungi: no problem. thanks for fixing it | 18:45 |
| jeblair | clarkb: i just obliterated a question you put in the ep | 18:46 |
| clarkb | jeblair: ya I grok now so its ok | 18:46 |
| jeblair | ok | 18:46 |
| clarkb | we would delete openstack/foo then move stackforge/foo to openstack/foo using the normal process | 18:46 |
| jeblair | yup. fungi: i'm assuming that we did neither the git repo move nor the db update, right? | 18:46 |
| fungi | jeblair: correct, the only things which happened were those triggered by merging the config change | 18:47 |
| fungi | jeblair: so i agree we should delete openstack/foo then move stackforge/foo to openstack/foo using the normal process | 18:47 |
| zaro | mordred: do you know what would be the correct fix for this? https://bugs.launchpad.net/openstack-ci/+bug/1322882 | 18:48 |
| fungi | https://review.openstack.org/96553 i think gets us the build job for the plugin | 18:48 |
| uvirtbot | Launchpad bug 1322882 in pbr "jenkins-job-builder 0.8.0 tarball with the switch to pbr is unusable for packagers" [Undecided,New] | 18:48 |
| fungi | oh, wait, i need to edit modules/gerrit/templates/gerrit.config.erb to enable the plugin too | 18:48 |
| fungi | maybe | 18:48 |
| *** hashar has joined #openstack-infra | 18:49 | |
| *** talluri has joined #openstack-infra | 18:49 | |
| anteaya | are we going to try this on review-dev first? | 18:49 |
| anteaya | I realize we are in a hurry | 18:50 |
| anteaya | but I do advocate for a dry run on -dev | 18:50 |
| fungi | zaro: do you happen to know what else we need to do to add a plugin besides the build job for it? https://review.openstack.org/96553 | 18:50 |
| *** pcm_ has quit IRC | 18:51 | |
| *** praneshp has quit IRC | 18:51 | |
| jeblair | i think i broke the etherpad | 18:52 |
| jeblair | better, needed to reload | 18:52 |
| jeblair | apparently i managed to paste something that broke it | 18:52 |
| *** praneshp has joined #openstack-infra | 18:55 | |
| jeblair | fungi, clarkb: which one of you did the cgit mirrors last week? did you do git05? (it's not listed in the instructions) | 18:55 |
| *** bodepd has joined #openstack-infra | 18:55 | |
| mordred | jeblair: I did | 18:55 |
| clarkb | jeblair: it was mordred iirc | 18:55 |
| fungi | yeah, and my notes in the etherpad said 05 but we may need to update our docs i guess | 18:56 |
| fungi | going by https://etherpad.openstack.org/p/repo-renaming-2014-05-23 | 18:56 |
| zaro | fungi: ohh, yeah. i have a chang for that. lemme look for it. | 18:56 |
| zaro | fungi: https://review.openstack.org/#/c/91193/ | 18:57 |
| fungi | zaro: thanks! | 18:57 |
| *** hogepodge has joined #openstack-infra | 18:58 | |
| *** UtahDave has joined #openstack-infra | 18:58 | |
| fungi | zaro: so if i base a new change as a child of 91193 i just need to edit review_dev.pp and/or review.pp to add the plugin name and built version (determined by the results of the periodic job)? | 18:59 |
| zaro | fungi: yes, that is correct. | 19:01 |
| *** rfolco has joined #openstack-infra | 19:01 | |
| *** andreykurilin_ has quit IRC | 19:02 | |
| fungi | oh, and i see my build job addition is wrong... it assumes a stable-2.8 branch which this plugin lacks... only has master | 19:02 |
| *** tsv has quit IRC | 19:02 | |
| jeblair | fungi: do you think we should avoid using it then? | 19:03 |
| clarkb | zaro: fungi I think that puppet change needs an update | 19:03 |
| *** MIDENN_ has quit IRC | 19:03 | |
| jeblair | (though from what i've seen, i think the master version should be okay | 19:03 |
| fungi | jeblair: instructions for the plugin claim it's been working since gerrit 2.7, so... | 19:03 |
| *** KurtMartin has joined #openstack-infra | 19:03 | |
| *** ociuhandu has quit IRC | 19:03 | |
| *** MIDENN_ has joined #openstack-infra | 19:03 | |
| jeblair | the plugin itself doesn't do any work, it just uses the existing db api to do all the work | 19:03 |
| jeblair | it's basically just "db.deletProject(foo)") | 19:04 |
| clarkb | but maybe not, depends on whether or not the file and exec resources implicitly depend on User[gerrit2] | 19:04 |
| clarkb | otherwise we need to require | 19:04 |
| anteaya | mgagne: when you get a chance, this file looks odd to me: https://review.openstack.org/#/c/96334/4/modules/openstack_project/files/zuul/layout.yaml | 19:04 |
| fungi | jeblair: oh, there's an api call for this? | 19:04 |
| anteaya | mgagne: but I can't decide how it looks odd | 19:04 |
| jeblair | fungi: the orm, more or less. | 19:04 |
| *** atiwari has joined #openstack-infra | 19:05 | |
| jeblair | dbHandler.delete(project); | 19:05 |
| jeblair | is the actual call | 19:05 |
| jeblair | and then fsHandler.delete(project, input == null ? false : input.preserve); | 19:05 |
| fungi | jeblair: ahh, this adds a gerrit api call, but there is a stable java method in gerrit which it's calling | 19:05 |
| jeblair | fungi: yep | 19:05 |
| fungi | okay, so we do still need the plugin if we want to do this cleanly | 19:06 |
| jeblair | fungi: yes | 19:06 |
| mgagne | anteaya: only issue reported by Jenkins is that the project was not properly added to respect the alphabetized list | 19:06 |
| mgagne | angdraug: oh, got fixed in latest patch | 19:06 |
| anteaya | mgagne: are the right job names used in that file? | 19:06 |
| mgagne | angdraug: sorry, not you | 19:06 |
| jeblair | fungi: oh wait, no it does do its own work... | 19:07 |
| anteaya | mgagne: that is the part that looks odd to me | 19:07 |
| jeblair | fungi: https://gerrit.googlesource.com/plugins/delete-project/+/master/src/main/java/com/googlesource/gerrit/plugins/deleteproject/database/Schema73DatabaseDeleteHandler.java | 19:07 |
| jeblair | https://gerrit.googlesource.com/plugins/delete-project/+/master/src/main/java/com/googlesource/gerrit/plugins/deleteproject/database/Schema77DatabaseDeleteHandler.java | 19:07 |
| *** e0ne has joined #openstack-infra | 19:07 | |
| jeblair | fungi: ^ | 19:07 |
| *** kmartin has quit IRC | 19:07 | |
| jeblair | there's two different methods for diffirent db schemas | 19:07 |
| Ajaeger | anteaya: the job names look fine to me - scroll up to openstack-manuals or operations-guide | 19:07 |
| jeblair | so presumably as long as we have a schema >=73 we should be okay | 19:08 |
| *** openstackgerrit has joined #openstack-infra | 19:08 | |
| anteaya | Ajaeger: okay thanks | 19:08 |
| fungi | clarkb: i restarted gerritbot, and openstackgerrit has returned | 19:08 |
| openstackgerrit | Jeremy Stanley proposed a change to openstack-infra/config: Add the delete-project Gerrit plugin https://review.openstack.org/96553 | 19:08 |
| fungi | so that's ^ the plugin build job | 19:08 |
| jeblair | we are on schema 86 | 19:08 |
| Ajaeger | anteaya: let me review the whole thing... | 19:08 |
| fungi | corrected for the only existing upstream branch | 19:08 |
| fungi | schema 86 sounds dire (and also later than 73 or 77) | 19:09 |
| SergeyLukjanov | fungi, oh, please, share me the knowledge of how to restart the gerritnot ;) | 19:09 |
| fungi | SergeyLukjanov: i tried to stop it with its initscript, but it complained there was no pidfile, then i looked in the process list, found its pid and sent it a sigterm with kill, but that didn't seem to help, so then i tried a sighup via kill -1 and that got it to die, then i started it with its initscript and waited | 19:10 |
| jeblair | "gerrit-plugin" sure shows up a lot in those job names, but we can change that later. :) | 19:10 |
| mordred | fungi, jeblair: so, to anteaya's point - do we want to test the plugin on review-dev before installing it in to prod? | 19:10 |
| fungi | mordred: i can split my change which enables the plugin to do review-dev first, then both | 19:11 |
| jeblair | mordred, anteaya: oh, sorry i didn't see that. sounds like a good idea. | 19:11 |
| fungi | but first we need the 96553 change merged and the periodic jobs for it trigegred | 19:11 |
| mgagne | anteaya: looks "ok" to me, can't say for sure | 19:11 |
| SergeyLukjanov | fungi, okay, thank you, so, init script should work in mid case | 19:11 |
| jeblair | +2 | 19:11 |
| clarkb | fungi: thanks | 19:12 |
| anteaya | mgagne: thanks for looking, Ajaeger made a good point and I scrolled up, so I can only see the error in the jjb file on that patch | 19:12 |
| fungi | since i don't see how the actual "version" is calculated (comparing to the javamelody plugin build), i probably need the build completed before i can draft the changes to add it to review-dev and review | 19:13 |
| *** openstackgerrit has quit IRC | 19:13 | |
| fungi | which will also depend on zaro's 91193 change merging and working | 19:13 |
| *** zhiyan_ is now known as zhiyan | 19:14 | |
| mordred | jeblair: /me has to AFK :( - I've got a lunch meeting in SF at 1pm | 19:14 |
| Ajaeger | anteaya: It is wrong - I'll comment on the review... | 19:15 |
| jeblair | fungi: do you want to go down that route, or use https://gerrit.googlesource.com/plugins/delete-project/+/master/src/main/java/com/googlesource/gerrit/plugins/deleteproject/database/Schema73DatabaseDeleteHandler.java as a basis to create our own sql commands? | 19:15 |
| clarkb | fungi: zaro mordred jeblair SergeyLukjanov can you see my comment on 91193? | 19:15 |
| clarkb | I think puppet may be smart enough to do an implicit require when you have users involved but I never remember | 19:16 |
| Ajaeger | anteaya: suggested fixes added | 19:16 |
| clarkb | jeblair: fungi: doing our own thing for this one project may not be so bad | 19:16 |
| *** nati_ueno has quit IRC | 19:16 | |
| SergeyLukjanov | clarkb, looking | 19:17 |
| *** arnaud has joined #openstack-infra | 19:17 | |
| anteaya | Ajaeger: thanks, will read again | 19:17 |
| pleia2 | running out to a lunch meeting, bbiab | 19:18 |
| zaro | clarkb: i guess it doesn't hurt to add that requirement. | 19:18 |
| clarkb | zaro: yeah I am just worried that they will run out of order and fail because the user doesn't exist | 19:18 |
| nealph | fungi:circling back with some more questions related to https://review.openstack.org/#/c/94633/....looks like there's a gate setup failure occuring (if I'm interpreting http://logs.openstack.org/33/94633/1/check-tripleo/check-tripleo-overcloud-f20/ae5ba5d/devstack-gate-setup-host.txt correctly) | 19:18 |
| anteaya | Ajaeger: awesome thanks, something just felt wrong, thanks for making the corrections | 19:19 |
| *** dprince has quit IRC | 19:19 | |
| clarkb | nealph: where is the failure? it looks ok to me | 19:19 |
| Ajaeger | anteaya: thanks for pointing me in the direction - after seeing the jjb change, I knew something was broken ;) | 19:19 |
| anteaya | :D | 19:19 |
| *** gokrokve_ has quit IRC | 19:20 | |
| fungi | jeblair: doing the db queries myself this time may not be so bad. reading the source now | 19:20 |
| *** hashar has quit IRC | 19:21 | |
| nealph | clarkb:I'm grabbing at straws a bit here (I don't see a clear failure either), but looks like .pydistutils.cfg is missing at 15:24:25.574 | 19:21 |
| *** dims has joined #openstack-infra | 19:21 | |
| nealph | clarkb:fungi:really, the only clear message is the build timeout at https://jenkins07.openstack.org/job/check-tripleo-overcloud-f20/514/console | 19:22 |
| Ajaeger | anteaya: I shouldn't have reviewed this - found a couple of "cosmetic" bugs that I need to cleanup now with the operations-guide ;( | 19:22 |
| clarkb | nealph: so the setup did fail but I don't think it failed in that log | 19:22 |
| clarkb | nealph: the timeout is the failure | 19:24 |
| *** zhiyan is now known as zhiyan_ | 19:24 | |
| clarkb | nealph: and apparently ./toci_test doesn't write a log file | 19:24 |
| fungi | it seems it was executing tripleo-ci/toci_gate_test.sh: line 13 when it was killed for running too long | 19:24 |
| clarkb | yup | 19:24 |
| clarkb | and that didn't log anything | 19:24 |
| fungi | toci_devtest.sh started at 15:24:17.011 and was killed at 17:44:25.115 | 19:25 |
| nealph | okay, I see that...any other option than to kick it off again? Is there somewhere else I could dig for clues? | 19:25 |
| *** gokrokve has joined #openstack-infra | 19:25 | |
| *** arnaud has quit IRC | 19:25 | |
| zaro | gerritbot down? | 19:26 |
| anteaya | Ajaeger: yeah, that will happen | 19:26 |
| fungi | 1913 <-- openstackgerrit (~openstack@review.openstack.org) has quit (Ping timeout: 245 seconds) | 19:26 |
| zaro | clarkb: i've updated 91193 | 19:26 |
| fungi | le sigh | 19:26 |
| SergeyLukjanov | fungi, I can try to restart it :) | 19:27 |
| *** derekh_ has joined #openstack-infra | 19:27 | |
| fungi | SergeyLukjanov: be my guest. it also may have logged some reason for breaking in /var/log/*gerritbot* | 19:27 |
| SergeyLukjanov | fungi, sshing | 19:28 |
| zaro | anybody knowledgale about pbr? | 19:28 |
| fungi | seems it logs in /var/log/gerritbot/gerritbot_irc.log and /var/log/gerritbot/gerritbot_debug.log | 19:28 |
| *** nati_ueno has joined #openstack-infra | 19:28 | |
| fungi | and also /var/log/gerritbot/gerritbot.log | 19:29 |
| SergeyLukjanov | fungi, yup | 19:29 |
| SergeyLukjanov | fungi, I see | 19:29 |
| SergeyLukjanov | 2014-05-29 19:28:29,536 ERROR gerritbot: Exception encountered in event loop | 19:29 |
| SergeyLukjanov | raise ServerNotConnectedError("Not connected.") | 19:29 |
| jeblair | fungi: clarkb: grabbing a quick lunch, biab | 19:30 |
| clarkb | hrm I should probably do that too | 19:30 |
| nealph | clarkb:fungi: any ideas on https://review.openstack.org/#/c/94633/ (or "the eternally damned patch")? | 19:30 |
| fungi | i'm working out sql queries for the delete, though i need to disappear in about 45 minutes for a user group meeting next town over | 19:30 |
| nealph | it seems futile to kick it off again... | 19:31 |
| clarkb | nealph: I think you need to ask the tripleo folks | 19:31 |
| fungi | nealph: did the people in #tripleo have any suggestions for what could be going wrong? | 19:31 |
| SergeyLukjanov | fungi, I'll try to start gerritbot again and take a look on new errors | 19:31 |
| clarkb | nealph: test_devtest isn't logging anything and timing out | 19:31 |
| derekh_ | nealph: The region your job ran on doesn't have any running testenvs, your job timeouted out waiting on a testenv | 19:31 |
| derekh_ | nealph: I tried to fix this today but couldn't, so I've propsed a patch to remove the region | 19:31 |
| clarkb | that was easy | 19:31 |
| nealph | help came to me, it seems. :) | 19:32 |
| nealph | derekh_:thanks for the update. | 19:32 |
| derekh_ | nealph: no prob | 19:32 |
| nealph | I assume your patch will pass tripleo-check? | 19:33 |
| *** rlandy has quit IRC | 19:33 | |
| nealph | :D | 19:33 |
| clarkb | fungi: anything you would like me to do before grabbing lunch? | 19:33 |
| Ajaeger | anteaya: https://review.openstack.org/96562 is the cleanup if you're interested - shouldn't make any real change, so no priority | 19:33 |
| fungi | clarkb: nah | 19:33 |
| clarkb | I am looking at 91193 real quick | 19:33 |
| clarkb | zaro: I don't think that will parse. you need to do [File[foo], Class[bar]] when listing requirements | 19:34 |
| clarkb | zaro: there isn't an implicit iterable type there | 19:34 |
| *** signed8b_ has joined #openstack-infra | 19:35 | |
| *** signed8bit has quit IRC | 19:35 | |
| *** openstackgerrit has joined #openstack-infra | 19:35 | |
| zaro | ugh! will do | 19:36 |
| *** nati_ueno has quit IRC | 19:36 | |
| SergeyLukjanov | openstackgerrit, welcome back | 19:36 |
| *** nati_ueno has joined #openstack-infra | 19:36 | |
| fungi | hopefully it sticks around for more than 5 minutes this time | 19:36 |
| nealph | derekh_:removing that region will force it to run on a working box, correct? Any ETA, by chance? | 19:36 |
| SergeyLukjanov | fungi, yup | 19:36 |
| SergeyLukjanov | fungi, I've stoped it by kill -1 due the same issue with pid file | 19:37 |
| clarkb | ok finding food back in a bit | 19:38 |
| openstackgerrit | Khai Do proposed a change to openstack-infra/config: puppetize installation of gerrit third party plugins https://review.openstack.org/91193 | 19:38 |
| derekh_ | nealph: thats the plan, should be soon after these get merged https://review.openstack.org/#/c/96546/ and https://review.openstack.org/#/c/96547/ | 19:38 |
| *** zehicle_at_dell has quit IRC | 19:38 | |
| derekh_ | infra prople when ye have a chance can you take a look at ^^ please, should reduce the constant tripleo pestering while we get something stable | 19:39 |
| derekh_ | fungi: clarkb ^ | 19:39 |
| *** signed8b_ is now known as signed8bit | 19:39 | |
| lifeless | mordred: is there a bug open for | 19:39 |
| lifeless | 2014-05-29 00:55:52.694 | Error <urlopen error timed out> while getting https://pypi.python.org/packages/source/r/requests/requests-2.3.0.tar.gz#md5=7449ffdc8ec9ac37bbcd286003c80f00 (from https://pypi.python.org/simple/requests/) | 19:39 |
| lifeless | mordred: errors in the requirements-integration-dsvm job ? | 19:40 |
| lifeless | mordred: I know we run against pypi deliberately, but using e.g. bandersnatch would let us avoid network issues | 19:40 |
| *** gokrokve has quit IRC | 19:40 | |
| *** gokrokve has joined #openstack-infra | 19:41 | |
| lifeless | argh stab python26 still hating on me | 19:42 |
| *** markwash has joined #openstack-infra | 19:43 | |
| *** flaper87|afk is now known as flaper87 | 19:44 | |
| *** signed8bit has quit IRC | 19:44 | |
| *** gokrokve has quit IRC | 19:45 | |
| *** signed8bit has joined #openstack-infra | 19:45 | |
| SergeyLukjanov | fungi, openstackgerrit is still with us :) | 19:47 |
| *** _nadya_ has quit IRC | 19:48 | |
| anteaya | zaro: I'm just looking at the commit message for https://review.openstack.org/#/c/91193/ | 19:48 |
| anteaya | the term third party is fairly overloaded in the openstack space | 19:48 |
| openstackgerrit | Derek Higgins proposed a change to openstack-infra/config: Remove the tripleo-test-cloud provider https://review.openstack.org/96547 | 19:49 |
| anteaya | a quick google doesn't give me any returns that tell me that gerrit uses the term third party to categorize its plugins | 19:49 |
| anteaya | it may, I just haven't found any google reference | 19:49 |
| anteaya | zaro: are you open to using a phrase other than third party for gerrit plugins you are puppetizing | 19:50 |
| anteaya | ? | 19:50 |
| jeblair | anteaya: seems clear to me | 19:50 |
| anteaya | you don't see any potential for confusion? | 19:50 |
| anteaya | I'm feeling like such a snowflake today | 19:50 |
| zaro | msabramo: i was wondering if this issue has been fixed? https://bugs.launchpad.net/openstack-ci/+bug/1322882 | 19:51 |
| uvirtbot | Launchpad bug 1322882 in pbr "jenkins-job-builder 0.8.0 tarball with the switch to pbr is unusable for packagers" [Undecided,New] | 19:51 |
| jeblair | i don't think of testing network gear when i read "puppetize installation of gerrit third party plugins". | 19:51 |
| zaro | anteaya: do you have a suggestion? | 19:52 |
| *** arnaud has joined #openstack-infra | 19:52 | |
| openstackgerrit | Brian Elliott proposed a change to openstack-infra/config: Project configuration for cachemonkey https://review.openstack.org/96570 | 19:52 |
| jeblair | fungi: try "sudo -H mysql reviewdb" | 19:53 |
| *** nati_ueno has quit IRC | 19:54 | |
| anteaya | I guess I just have a filter for the phrase third party | 19:54 |
| zaro | anteaya: in gerrit world, it's etheir a *core plugin* or *plugin* | 19:54 |
| anteaya | zaro: how about gerrit plugin | 19:54 |
| openstackgerrit | K Jonathan Harker proposed a change to openstack-infra/config: Also run puppet-apply test on bare-centos6 https://review.openstack.org/96320 | 19:54 |
| jeblair | anteaya: no, _that_ would be confusing. | 19:55 |
| fungi | jeblair: indeed, that does work and is far shorter than me adding a defaults file option | 19:55 |
| anteaya | the body of the commit message explains the difference between core plugin and plugin | 19:55 |
| clarkb | turkey cranberry sandwich aquired | 19:55 |
| clarkb | and look fungi is putting out security things too | 19:55 |
| fungi | third-party turkey cranberry? | 19:55 |
| *** markmc has joined #openstack-infra | 19:55 | |
| *** markmc has quit IRC | 19:55 | |
| jesusaurus | fungi: to answer your question on 96320: i expect the precise test to pass pretty soon, but i dont expect the centos test to pass for a while. there are probably many ubuntu assumptions in there | 19:56 |
| anteaya | gerrit calls the javamelody plugin a gerrit plugin? | 19:56 |
| jeblair | fungi: should we start drafting the delete sql? | 19:56 |
| fungi | jesusaurus: in that case we might want it experimental in its early stages | 19:56 |
| clarkb | jeblair: I think so | 19:57 |
| *** nati_ueno has joined #openstack-infra | 19:57 | |
| fungi | jeblair: that's what i'm working on, just matching up the queries from the plugin source to the existing tables/columns to make sure they're sane | 19:57 |
| jeblair | oh that's what that space is for | 19:57 |
| fungi | ;D | 19:58 |
| jeblair | fungi: would it help if i started copying code from the plugin into the etherpad, or do you have something going on already and i just just go do dishes? :) | 19:58 |
| fungi | it will (hopefully) be filled of teh sqlz before i have to run to the local osug meetup | 19:58 |
| jesusaurus | fungi: i agree that it should be experimental | 19:58 |
| fungi | jeblair: you may be following the javaspaghetti better than i am, so adding some stubs in the etherpad for what you think needs to be run wouldn't hurt | 19:59 |
| jeblair | fungi: ok will do | 19:59 |
| fungi | like, for example, i've determined that the check for subscribed submodules is probably not anything we care about | 19:59 |
| *** gokrokve has joined #openstack-infra | 20:00 | |
| fungi | (checkS plural, for subscribed to and subscribed by... except for things which are part of openstack/openstack) | 20:00 |
| fungi | also, for this particular case, most of these queries are a noop | 20:01 |
| *** nati_ueno has quit IRC | 20:01 | |
| *** rodrigods has quit IRC | 20:01 | |
| lifeless | jeblair: got a few minutes for a focused discussion on burning in tripleo-test-clouds? | 20:01 |
| lifeless | jeblair: we want to eliminate the fragility we've been seeing | 20:02 |
| lifeless | jeblair: the workload we want to be robust for is running tripleo-ci (and if infra wants to regular dvsm) jobs | 20:02 |
| lifeless | jeblair: seems to me that running many such jobs that are testing trunk rather than a commit is an ideal way to see how it performs | 20:02 |
| jeblair | fungi: i think it might be just that one table. | 20:03 |
| *** nati_ueno has joined #openstack-infra | 20:03 | |
| jeblair | fungi: (project watches) | 20:03 |
| fungi | jeblair: yep, and there are no rows where project_name="openstack/python-barbicanclient" | 20:04 |
| jeblair | fungi: so, um, it's deleted! :) | 20:04 |
| fungi | jeblair: so we may just be able to stop gerrit, delete the git repos (in home and opt) and then do our rename steps as usual) | 20:04 |
| fungi | basically anything manage-projects would have done for us which could get in the way. same for needing to delete on the cgit servers and in github i guess | 20:05 |
| jeblair | lifeless: now isn't a great time for a focused discussion, maybe in a few hours? | 20:05 |
| lifeless | jeblair: derekh_ will be asleep then (I hope :)) but sure, he and I will brainstorm now and you and i can chat later | 20:06 |
| derekh_ | lifeless: cool, will try and be back later aswell but can't promis anything | 20:06 |
| jeblair | fungi: i put the delete steps in with the mv steps | 20:07 |
| jeblair | fungi: for the git repos | 20:07 |
| Ajaeger | is zuul currently stopped? | 20:07 |
| openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: query for hairpin fail in libvirt https://review.openstack.org/96574 | 20:07 |
| fungi | jeblair: yep, saw. perfect thanks! | 20:07 |
| Ajaeger | I have a post job that's queued for ages... | 20:07 |
| *** zehicle_at_dell has joined #openstack-infra | 20:07 | |
| fungi | Ajaeger: doesn't look stopped, just busy and failing a bunch of stuff in the gate | 20:07 |
| *** duran has joined #openstack-infra | 20:07 | |
| fungi | Ajaeger: the one that's been there for openstack-manuals for 28 minutes? | 20:08 |
| Ajaeger | fungi: yes | 20:08 |
| jeblair | and i suspect the hpcloud failures are causing us to run at reduced capacity | 20:08 |
| clarkb | yes the failures are currently ugh | 20:08 |
| jeblair | the graphs look like a lot of nodes went straight to delete | 20:08 |
| fungi | jeblair: yeah, looks like it. the building count is waaaay up, which i assume is all discards/retries in hpcloud | 20:09 |
| clarkb | but mordred is talking to people and trying to get them to join us here so that we can be a bit more direct in fixing it | 20:09 |
| openstackgerrit | A change was merged to openstack-infra/config: Reduce tripleo ci jobs https://review.openstack.org/96546 | 20:09 |
| fungi | anyway, i can help with the rename if we want to start it right this moment, but i need to run away in 5 or 10 minutes | 20:10 |
| Ajaeger | thanks, will be patient ;) | 20:10 |
| fungi | otherwise i can help once i get to the conference room at cisco | 20:10 |
| clarkb | my sandwich is done so I am ready whenever | 20:10 |
| jeblair | fungi: do you think the etherpad is all prepared? | 20:10 |
| fungi | jeblair: gave it one last re-read and lgtm | 20:11 |
| jeblair | fungi: i think clarkb and i can handle it; why don't we go ahead and start nowish | 20:12 |
| jeblair | of course, starting means waiting for run_all to finish... so still might be a few | 20:13 |
| jeblair | once that's done, we can do the announcement | 20:13 |
| fungi | jeblair: okay, cool. i'll pitch in when i get settled at the other end, if you're still going | 20:13 |
| * fungi runs away | 20:13 | |
| jeblair | fungi: cool, thanks, and have fun/beer | 20:14 |
| *** zhiyan_ is now known as zhiyan | 20:15 | |
| *** nati_ueno has quit IRC | 20:15 | |
| clarkb | yes have fun giving those openstack people a hard time | 20:16 |
| jeblair | okay, run all ssh is stopped | 20:16 |
| jeblair | clarkb: are you on-deck and ready to proceed? | 20:16 |
| *** ociuhandu has joined #openstack-infra | 20:16 | |
| clarkb | I am | 20:17 |
| clarkb | I can do the mirrors if you want to do gerrit? | 20:17 |
| clarkb | both review.o.o mirror and cgit mirrors | 20:17 |
| jeblair | how about: #status alert Gerrit is going offline to correct an issue with a recent project rename. ETA 20:30 UTC. | 20:18 |
| jeblair | clarkb: ^ | 20:18 |
| clarkb | sounds good, though the reindex took ~5 minutes iirc | 20:18 |
| clarkb | maybe 2045 as ETA? | 20:18 |
| jeblair | ok | 20:18 |
| jeblair | #status alert Gerrit is going offline to correct an issue with a recent project rename. ETA 20:45 UTC. | 20:18 |
| openstackstatus | jeblair: sending alert | 20:18 |
| *** derekh_ has quit IRC | 20:19 | |
| *** zehicle_at_dell has quit IRC | 20:19 | |
| -openstackstatus- NOTICE: Gerrit is going offline to correct an issue with a recent project rename. ETA 20:45 UTC. | 20:20 | |
| *** ChanServ changes topic to "Gerrit is going offline to correct an issue with a recent project rename. ETA 20:45 UTC." | 20:20 | |
| clarkb | jeblair: just let me know when you are ahppy with things being moved on the mirrors | 20:21 |
| jeblair | stopped zuul | 20:21 |
| openstackstatus | jeblair: finished sending alert | 20:22 |
| *** jergerber has joined #openstack-infra | 20:22 | |
| jeblair | stopped gerrit | 20:23 |
| jeblair | clarkb: i think you can start now | 20:23 |
| clarkb | k | 20:23 |
| jeblair | performing mysqldump | 20:24 |
| *** reaperhulk has joined #openstack-infra | 20:25 | |
| *** openstackgerrit has quit IRC | 20:25 | |
| jeblair | complete and db updated | 20:25 |
| *** dkliban is now known as dkliban_afk | 20:26 | |
| *** eharney has quit IRC | 20:26 | |
| *** zhiyan is now known as zhiyan_ | 20:26 | |
| clarkb | jeblair: cgit mirrors mv'd | 20:27 |
| clarkb | going to delete from github now then look at jenkins slaves | 20:27 |
| jeblair | reindexing | 20:28 |
| jeblair | ooh it has a spinning cursor | 20:28 |
| clarkb | jenkins slaves look clean | 20:30 |
| jeblair | 41% | 20:30 |
| *** eglynn has joined #openstack-infra | 20:32 | |
| * eglynn is seeing 503s from gerrit | 20:33 | |
| *** angdraug has quit IRC | 20:33 | |
| jeblair | eglynn: see topic | 20:33 |
| jeblair | eglynn: and announcement that went out to every channel :) | 20:33 |
| jeblair | reindex done | 20:33 |
| eglynn | jeblair: apologies for the noise, /me was off-grid | 20:33 |
| jeblair | eglynn: np | 20:33 |
| SergeyLukjanov | fungi, openstackgerrit is offline again :( | 20:34 |
| SergeyLukjanov | [00:25:16] openstackgerrit (~openstack@review.openstack.org) left IRC. (Ping timeout: 245 seconds) | 20:34 |
| jeblair | clarkb: ready for me to start gerrit? | 20:34 |
| anteaya | fungi is away | 20:34 |
| clarkb | jeblair: I am ready if you are ready | 20:34 |
| jeblair | https://review.openstack.org/#/q/project:openstack/python-barbicanclient,n,z | 20:34 |
| clarkb | woot | 20:35 |
| clarkb | jeblair: ready for me to trigger a replication? | 20:35 |
| jeblair | starting zuul | 20:35 |
| jeblair | clarkb: yup | 20:35 |
| *** gokrokve has quit IRC | 20:35 | |
| eglynn | ... all is well again with the world :) | 20:35 |
| Ajaeger | jeblair: more important: no results for https://review.openstack.org/#/q/project:stackforge/python-barbicanclient,n,z ;) | 20:35 |
| eglynn | ... thanks chaps! :) | 20:35 |
| clarkb | of course the command is different now | 20:36 |
| jeblair | clarkb: naturally! | 20:36 |
| jeblair | restoring zuul's queues | 20:36 |
| *** tsv has joined #openstack-infra | 20:36 | |
| jeblair | #status ok Gerrit is back online | 20:37 |
| openstackstatus | jeblair: sending ok | 20:37 |
| *** zehicle_at_dell has joined #openstack-infra | 20:37 | |
| eglynn | sad face ... eglynn's draft comments on a review-in-flight were lost | 20:38 |
| *** mwagner_lap has quit IRC | 20:38 | |
| clarkb | replication started | 20:38 |
| *** ChanServ changes topic to "Discussion of OpenStack Developer Infrastructure | docs http://ci.openstack.org | bugs https://launchpad.net/openstack-ci/ | https://git.openstack.org/cgit/openstack-infra/config/tree/" | 20:39 | |
| -openstackstatus- NOTICE: Gerrit is back online | 20:39 | |
| *** lakshmiS has joined #openstack-infra | 20:39 | |
| jeblair | eglynn: :( i guess your client didn't get to save them before the shutdown? | 20:39 |
| *** radez is now known as radez_g0n3 | 20:39 | |
| eglynn | jeblair: ... ah well, fresh enough in the mind to quickly re-type, not a prob! | 20:39 |
| clarkb | gerrit task count is falling slowly (I think I may have misread replication docs and double tapped it oh well) | 20:40 |
| jeblair | heh | 20:40 |
| clarkb | the --all is not necessary :) | 20:40 |
| clarkb | and --wait would have made it synchronous | 20:40 |
| openstackstatus | jeblair: finished sending ok | 20:41 |
| *** reaperhulk has left #openstack-infra | 20:41 | |
| jeblair | clarkb: should i run "manage-projects openstack/python-barbicanclient" ? | 20:41 |
| clarkb | jeblair: maybe wait for replication to complete? | 20:42 |
| clarkb | down to 157 tasks now | 20:42 |
| *** melwitt has quit IRC | 20:42 | |
| jeblair | Ajaeger: i'm afraid that post job isn't going to run now. if it's very important, we can try to trigger it manually... | 20:43 |
| *** melwitt has joined #openstack-infra | 20:43 | |
| *** melwitt has quit IRC | 20:43 | |
| *** aconrad has quit IRC | 20:43 | |
| *** melwitt has joined #openstack-infra | 20:44 | |
| *** chuckC has quit IRC | 20:44 | |
| *** melwitt has quit IRC | 20:44 | |
| *** melwitt has joined #openstack-infra | 20:45 | |
| clarkb | jeblair: looking at show-queue I believe replication is done | 20:45 |
| clarkb | all replication tasks are very new | 20:45 |
| *** gokrokve has joined #openstack-infra | 20:47 | |
| Ajaeger | jeblair: no real problem, I'll wait for the next run. We approve often enough ;) | 20:48 |
| *** atiwari has quit IRC | 20:49 | |
| *** mburned is now known as mburned_out | 20:49 | |
| *** gokrokve has quit IRC | 20:50 | |
| *** shakamunyi has quit IRC | 20:51 | |
| *** shakamunyi has joined #openstack-infra | 20:52 | |
| *** olaph has joined #openstack-infra | 20:53 | |
| jeblair | oh, i think that bug where jeepyb overwrites the acl config happened (i just approved mordred's fix for that recently) | 20:53 |
| jeblair | running puppet and now it's running m-p | 20:53 |
| clarkb | jeblair: so this run of m-p should correct the ACL with mordred's fix? | 20:54 |
| clarkb | zaro: see comment on 91193 on how to fix lint | 20:54 |
| *** dizquierdo has joined #openstack-infra | 20:55 | |
| *** atiwari has joined #openstack-infra | 20:55 | |
| jeblair | clarkb: even without mordred's fix i think, because puppet will put the acl file back and jeepyb won't want to touch it again | 20:55 |
| clarkb | oh right | 20:55 |
| jeblair | notice: /Stage[main]/Openstack_project::Gerrit/Exec[manage_projects]/returns: ERROR:manage_projects:Failed to add groups file for project: openstack-attic/melange | 20:56 |
| clarkb | huh | 20:57 |
| jeblair | okay maybe we do need mordred's fix? | 20:57 |
| clarkb | maybe otherwise it will use that default | 20:57 |
| jeblair | because the acl file is empty again | 20:57 |
| *** gokrokve has joined #openstack-infra | 20:58 | |
| *** rcarrill` has joined #openstack-infra | 20:58 | |
| jeblair | clarkb: i'm open to ideas | 20:58 |
| jeblair | it looks like mordred's fix merged | 20:58 |
| *** jooools has joined #openstack-infra | 20:58 | |
| clarkb | did it get installed? | 20:58 |
| clarkb | we have had pip installation problems with jeepyb in the past | 20:58 |
| jeblair | manage-projects: error: unrecognized arguments: --version | 20:59 |
| *** sarob has joined #openstack-infra | 20:59 | |
| clarkb | jeblair: pip freeze says jeepyb==a8bcd43 | 20:59 |
| clarkb | 43c20c85786f4e6ca9d6bc7f71f7f88f403864d0 is most recent commit | 20:59 |
| clarkb | but a8bcd includes mordreds fix | 21:00 |
| jeblair | a8bcd43 is the commit that should fix it | 21:00 |
| *** rcarrillocruz has quit IRC | 21:00 | |
| clarkb | yup so must be a bug /me looks | 21:00 |
| jeblair | clarkb: oh wait, no, it's the commit before then | 21:00 |
| *** hogepodge_ has joined #openstack-infra | 21:01 | |
| jeblair | it's Merge "Care less about github metadata" | 21:01 |
| clarkb | ohhh yup | 21:01 |
| fungi | did my acl file creation busywait get approved? or is it still waiting on sdague's exceptions patch? | 21:01 |
| *** mrmartin has quit IRC | 21:01 | |
| clarkb | jeblair: /opt/jeepyb is the right commit | 21:02 |
| jeblair | fungi: it had an error, i -1'd it | 21:02 |
| *** hogepodge has quit IRC | 21:02 | |
| *** hogepodge_ is now known as hogepodge | 21:02 | |
| clarkb | want me to try a `pip install .` in that dir? | 21:02 |
| jeblair | clarkb: it's because of https://review.openstack.org/#/c/92686/ | 21:02 |
| jeblair | clarkb: i just did that | 21:02 |
| *** hogepodge has quit IRC | 21:02 | |
| jeblair | we need pg_config installed | 21:02 |
| clarkb | oh gah | 21:02 |
| fungi | oh, missed that | 21:02 |
| sdague | fungi: it needs a rebase as well, I fixed some of jeblair's concerns under it | 21:02 |
| *** flaper87 is now known as flaper87|afk | 21:03 | |
| *** fifieldt has quit IRC | 21:03 | |
| *** hogepodge has joined #openstack-infra | 21:03 | |
| * clarkb tries to remember what the dev C lib for postgres is | 21:03 | |
| jeblair | i'm not sure what the point of conditional imports are without conditional requirements | 21:04 |
| jeblair | libpq-dev | 21:04 |
| clarkb | we will have to change that everywhere we use jeepyb | 21:04 |
| clarkb | arg | 21:05 |
| *** bhuvan has joined #openstack-infra | 21:05 | |
| sdague | jeblair: is there a reason not to leave it out of requirements? | 21:05 |
| clarkb | jeblair: maybe we remove it from requirements.txt? | 21:05 |
| clarkb | sdague: jinx | 21:05 |
| *** gokrokve has quit IRC | 21:06 | |
| jeblair | clarkb: want to propose that change? | 21:06 |
| clarkb | sure | 21:07 |
| *** aysyd has quit IRC | 21:07 | |
| jeblair | this would be a good use of extras | 21:08 |
| jeblair | https://pythonhosted.org/setuptools/pkg_resources.html#requirement-objects | 21:08 |
| jeblair | i don't think we've explored how that would work with pbr, etc, though. | 21:09 |
| clarkb | https://review.openstack.org/96588 | 21:09 |
| *** pcrews has quit IRC | 21:09 | |
| lifeless | lol I wrote this "There is no tolling support for picking new (postversioned) version numbers." | 21:09 |
| lifeless | dhellmann_: & | 21:09 |
| lifeless | when I went to correct it my first thought was 'trolling support...' | 21:09 |
| *** pdmars has quit IRC | 21:10 | |
| jeblair | lifeless: we can always count on you for ensuring the highest level of trolling support! ;) | 21:10 |
| *** bhuvan_ has joined #openstack-infra | 21:10 | |
| *** bhuvan has quit IRC | 21:11 | |
| jeblair | clarkb: force-merged | 21:11 |
| jeblair | puppet installed jeepyb, updated the acl file and is running m-p | 21:13 |
| lifeless | hmm, who is around and has hacked on pbr internals ? | 21:14 |
| lifeless | dims: I think you have? ^ | 21:14 |
| *** shakamunyi has quit IRC | 21:14 | |
| *** lcostantino has quit IRC | 21:17 | |
| *** zhiyan_ is now known as zhiyan | 21:18 | |
| *** arnaud has quit IRC | 21:18 | |
| jeblair | clarkb: oh.... puppet was _deleting_ the acl file | 21:18 |
| jeblair | clarkb: because it's not in the config repo | 21:18 |
| jeblair | clarkb: https://review.openstack.org/96593 | 21:19 |
| lifeless | jeblair: as soon as gerrit is back up https://review.openstack.org/#/c/96547/ is fairly important to tripleo - we want to rip it up and really fix it | 21:19 |
| clarkb | jeblair: that lgtm should I force submit? | 21:19 |
| lifeless | jeblair: right now its got reduced testenv resources, which is impacting jobs, but we can scramble to fix it if landing that needs to be delayed | 21:20 |
| jeblair | clarkb: done | 21:20 |
| *** fifieldt has joined #openstack-infra | 21:20 | |
| jeblair | lifeless: lgtm | 21:21 |
| fungi | approved | 21:21 |
| fungi | addressed my concerns | 21:21 |
| lifeless | jeblair: thanks | 21:22 |
| fungi | oh, heh, double-approved i guess ;) | 21:22 |
| jeblair | fungi: oh welcome back | 21:22 |
| jeblair | updated puppet on master and running puppet again | 21:23 |
| *** jooools has quit IRC | 21:23 | |
| *** chianingwang has joined #openstack-infra | 21:23 | |
| *** salv-orlando has quit IRC | 21:24 | |
| *** julim has quit IRC | 21:24 | |
| fungi | so i take it the previous change to rename barbicanclient also missed moving the acl file? | 21:24 |
| clarkb | fungi: ya | 21:25 |
| jeblair | clarkb: https://review.openstack.org/#/admin/projects/openstack/python-barbicanclient,access | 21:26 |
| jeblair | fungi: ^ yay | 21:26 |
| clarkb | \o/ | 21:27 |
| *** zhiyan is now known as zhiyan_ | 21:27 | |
| Ajaeger | looking back at tsv's patch that triggered this change: https://review.openstack.org/#/c/80124 - it looks fine but still not started. | 21:27 |
| fungi | excellent | 21:27 |
| jeblair | re-enabling run-all | 21:27 |
| Ajaeger | I'll do a recheck no bug on it | 21:27 |
| fungi | Ajaeger: that should kick it in | 21:27 |
| fungi | Ajaeger: but yes, any existing open changes which got new patchsets on barbicanclient in the last week will need similar nudging | 21:28 |
| Ajaeger | fungi: "Starting check jobs. http://status.openstack.org/zuul/" ;) | 21:28 |
| *** rpwwpr has quit IRC | 21:28 | |
| Ajaeger | fungi, there are two or three - I trigger them... | 21:29 |
| jeblair | clarkb, fungi: okay, that's everything i think. | 21:29 |
| *** hogepodge has quit IRC | 21:29 | |
| clarkb | jeblair: I can't think of anything else | 21:30 |
| *** andreykurilin_ has joined #openstack-infra | 21:30 | |
| *** marcoemorais has quit IRC | 21:30 | |
| lifeless | fungi: can you let me know when https://review.openstack.org/#/c/96547/ is live? | 21:30 |
| Ajaeger | fungi: two in total... | 21:30 |
| *** nati_ueno has joined #openstack-infra | 21:31 | |
| *** marcoemorais has joined #openstack-infra | 21:31 | |
| fungi | lifeless: sure | 21:31 |
| lifeless | thanks! | 21:31 |
| *** marcoemorais has quit IRC | 21:31 | |
| *** chianingwang has quit IRC | 21:31 | |
| *** marcoemorais has joined #openstack-infra | 21:32 | |
| *** your_nickname has joined #openstack-infra | 21:32 | |
| clarkb | fungi: jeblair https://review.openstack.org/96600 updated docs | 21:33 |
| jeblair | lifeless: you should be able to check at http://puppetboard.openstack.org/node/nodepool.openstack.org | 21:33 |
| clarkb | were there other items we want to update docs on? | 21:33 |
| *** hogepodge has joined #openstack-infra | 21:33 | |
| Ajaeger | fungi, jeblair: I'd like to get https://review.openstack.org/#/c/95377/ in this week (translation setup for log messages) - with enough lead time to my vacation to fix any issues. I don't expect any since I tested on all repos but you never know... So, if you could review this week, I would appreciate it. | 21:33 |
| *** salv-orlando has joined #openstack-infra | 21:34 | |
| *** kinrui has joined #openstack-infra | 21:34 | |
| *** kinrui has quit IRC | 21:34 | |
| *** openstackgerrit has joined #openstack-infra | 21:35 | |
| jeblair | Ajaeger: so we decided that we _do_ want to tranlate log messages now? | 21:35 |
| *** salv-orlando has quit IRC | 21:35 | |
| tsv | Ajaeger: fungi: thanks! i got a Jenkins notification | 21:36 |
| *** your_nickname has quit IRC | 21:36 | |
| Ajaeger | jeblair: dhellmann_ did already all the hard work for Juno | 21:36 |
| Ajaeger | s/Juno/Icehouse/ | 21:36 |
| Ajaeger | jeblair: https://blueprints.launchpad.net/oslo/+spec/log-messages-translation-domain | 21:36 |
| *** chianingwang has joined #openstack-infra | 21:36 | |
| *** gokrokve has joined #openstack-infra | 21:37 | |
| jeblair | Ajaeger: just curious. for a long time we heard from people that they didn't want log messages translated. i guess i missed the summit session where that changed. | 21:37 |
| Ajaeger | missing part now is interaction with transifex - so, log messages with the exception of debug messages get translated | 21:37 |
| clarkb | jeblair: I believe it boiled down to user choice. If you choose to translate them then you incur whatever extra mapping cost that may require | 21:37 |
| *** tsv has quit IRC | 21:37 | |
| jeblair | clarkb: but operators actuall do want this? | 21:38 |
| lifeless | jeblair: ooh shiny | 21:38 |
| clarkb | jeblair: IBM wanted it apparently | 21:38 |
| clarkb | and I want to say it was required for some business standard or other | 21:38 |
| *** chianingwang is now known as your_nickname | 21:38 | |
| Ajaeger | jeblair: sorry, wasn't part of those discussions, only heard about it as fact, I miss the history here that you have... | 21:38 |
| zzelle | jeblair, as a dev and operator i don't want it | 21:38 |
| clarkb | (I personally think it is crazy but what do I know) | 21:38 |
| *** gokrokve_ has joined #openstack-infra | 21:38 | |
| fungi | the upshot was that log messages will now embed a message id which does not change between different translations/localizations so that it doesn't make web-search-troubleshooting impossible | 21:39 |
| Ajaeger | clarkb: I'm glad that these are separate domains - so translators can prioritize the "user visible" strings over log strings. | 21:39 |
| * Ajaeger uses English locales and not German ;) | 21:39 | |
| openstackgerrit | Joshua Harlow proposed a change to openstack/requirements: Add doc8 a documentation style checking package https://review.openstack.org/94061 | 21:40 |
| jeblair | fungi: guru meditation! | 21:40 |
| Ajaeger | jeblair: for log.info, python code uses "_LI", for warning "_LW" - instead of the usual "_" | 21:40 |
| Ajaeger | so, yes, some extra fun ;( | 21:41 |
| *** gokrokve has quit IRC | 21:41 | |
| fungi | jeblair: yep, i think they were even dubbing the error codes "guru meditations" which pleased me in nostalgic ways | 21:41 |
| Ajaeger | thanks, jeblair ! | 21:41 |
| *** your_nickname has quit IRC | 21:42 | |
| *** talluri has quit IRC | 21:42 | |
| *** duran has quit IRC | 21:42 | |
| *** chianingwang has joined #openstack-infra | 21:42 | |
| sdague | fungi: no, that was a different thing in nova | 21:43 |
| sdague | that's the signal handler to dump state | 21:43 |
| fungi | sdague: oh, right. i conflated them | 21:43 |
| clarkb | jhesketh: thanks for updating that nodepool change +2 | 21:43 |
| fungi | still was glad to see that term lives on | 21:43 |
| *** gokrokve_ has quit IRC | 21:43 | |
| clarkb | jeblair: fungi: 90883 is a relatively simple nodepool change that will make downstream users of it very happy | 21:43 |
| *** talluri has joined #openstack-infra | 21:44 | |
| harlowja_ | hi guys, qq, https://review.openstack.org/#/c/94236/ merged yesterday, and i think it was supposed to be prepopulated from another github repo, but it seems like that didn't happen (https://github.com/stackforge/doc8 exists as empty) | 21:44 |
| clarkb | harlowja_: http://git.openstack.org/cgit/stackforge/doc8/tree/ its there | 21:45 |
| fungi | harlowja_: my guess is that the github project creation failed partway in again | 21:45 |
| clarkb | so probably just a github derp | 21:45 |
| harlowja_ | clarkb kk | 21:45 |
| harlowja_ | can i also maybe get myself added to doc8-core :) | 21:45 |
| fungi | last one i saw, m-p got as far as adding the project in github, but not adding the gerrit account as allowed to push into it, so replication never happened | 21:45 |
| harlowja_ | or doc8-ptl | 21:45 |
| harlowja_ | fungi ok, next change then will cause this push i suppose | 21:46 |
| clarkb | though I just did a replication so you would expect that to have updated github | 21:46 |
| lifeless | jeblair: I can't tell from http://puppetboard.openstack.org/node/nodepool.openstack.org | 21:46 |
| lifeless | jeblair: I may just not know how to use it | 21:46 |
| lifeless | jeblair: but there's no resource details | 21:46 |
| fungi | clarkb: i believe m-p doesn't check and retry to add the permission for gerrit to write in it | 21:46 |
| *** mriedem has quit IRC | 21:46 | |
| clarkb | fungi: oh, you think it is a github side ACL problem? I can look at that | 21:47 |
| fungi | so rerunning m-p and retriggering replication won't fix that | 21:47 |
| fungi | clarkb: yeah, that was it for the last one i ran into | 21:47 |
| jeblair | lifeless: http://puppetboard.openstack.org/report/nodepool.openstack.org/30426d03a6cd6df4ecfd38a6fe1b5799d2cdcf26 | 21:47 |
| *** openstackgerrit has quit IRC | 21:47 | |
| fungi | clarkb: gerrit account never ended up in the "team" which had write perms for the created project on github | 21:47 |
| jeblair | lifeless: you can tell that it updated nodepool.yaml at a certain time; assuming that time is > when the change merged, that change should be included | 21:48 |
| clarkb | https://github.com/orgs/stackforge/teams/owners needs our gerrit account? that sounds right and it is missing | 21:48 |
| fungi | and we chalked that up to a github api throttle/derp | 21:48 |
| lifeless | jeblair: ok, *now* it shows when I refresh :P | 21:48 |
| clarkb | I will look at a different project and manually update doc8 | 21:48 |
| lifeless | jeblair: thanks | 21:48 |
| clarkb | actually that is a stackforge wide rule | 21:48 |
| clarkb | so I think I need a second team on doc8? /me looks at a different project | 21:48 |
| *** mrda-away is now known as mrda | 21:48 | |
| harlowja_ | clarkb i created 1324701 just incase this needs to be tracked (for access stuff) | 21:48 |
| *** masayukig has joined #openstack-infra | 21:49 | |
| clarkb | fungi: https://github.com/stackforge/doc8/settings/collaboration can you double check that looks right to you? | 21:49 |
| jeblair | clarkb: openstack-gerrit is part of the gerrit team | 21:50 |
| clarkb | jeblair: ya I just added stackforge/gerrit | 21:50 |
| jeblair | clarkb: only the owner team is org-wide, the gerrit team needs to be added to each project | 21:50 |
| *** tsv has joined #openstack-infra | 21:50 | |
| *** mfer has quit IRC | 21:50 | |
| *** talluri has quit IRC | 21:51 | |
| clarkb | harlowja_: looks good now | 21:51 |
| clarkb | I will close bug 1324701 | 21:51 |
| uvirtbot | Launchpad bug 1324701 in openstack-ci "Add harlowja to doc8-ptl (and doc8-core)" [Undecided,New] https://launchpad.net/bugs/1324701 | 21:51 |
| harlowja_ | clarkb great :) | 21:52 |
| harlowja_ | thx! | 21:52 |
| pleia2 | zaro: it's like you're having your own bug day! :) | 21:52 |
| *** chuckC has joined #openstack-infra | 21:52 | |
| * zaro squashing em all | 21:53 | |
| pleia2 | \o/ | 21:53 |
| fungi | clarkb: looking | 21:55 |
| zaro | pleia2: kinda odd that i'm not getting email from LP for issues i've updated. | 21:56 |
| * pleia2 is | 21:56 | |
| fungi | clarkb: looks right | 21:56 |
| anteaya | I'm looking here and expecting to see the slave_scripts dir: http://git.openstack.org/cgit/openstack-infra/config/tree/modules/jenkins/files | 21:56 |
| anteaya | and I don't see it | 21:56 |
| clarkb | jeblair: jhesketh https://review.openstack.org/#/c/85204/11 should be ready whenever folks settle into doing reviews | 21:56 |
| clarkb | zaro: I am getting a ton of mail about it :) | 21:56 |
| fungi | clarkb: if you retrigger replication for that project, it ought to work now | 21:56 |
| jeblair | clarkb: cool, thx | 21:56 |
| clarkb | zaro: but it makes me happy so keep it up | 21:56 |
| anteaya | I see it in my local repo, I can't see any commit message that mentions moving slave_scripts | 21:57 |
| clarkb | fungi: ya I think harlowja_ pushed something and replication has happened | 21:57 |
| anteaya | has slave_script moved? | 21:57 |
| fungi | clarkb: cool beans | 21:57 |
| clarkb | anteaya: yes, it was moved to openstack_project | 21:57 |
| clarkb | anteaya: as our slave scripts are actually really openstack specific and not part of a generic make jenkins go thing | 21:57 |
| lifeless | why doesn't lists.o.o show up in google when I search for subjects :( | 21:57 |
| anteaya | any rememberance of what the commit message said? | 21:57 |
| anteaya | okay | 21:57 |
| clarkb | anteaya: its part of the work to make our modules reconsumable by others | 21:57 |
| pleia2 | anteaya: http://git.openstack.org/cgit/openstack-infra/config/tree/modules/openstack_project/files/slave_scripts | 21:57 |
| anteaya | thanks | 21:58 |
| anteaya | wondering how I missed the commit message | 21:58 |
| *** dims has quit IRC | 21:58 | |
| *** signed8bit has quit IRC | 21:58 | |
| *** chianingwang has quit IRC | 21:58 | |
| fungi | lifeless: i'm watching the used count for nodes in that region, and i think it's dropping now but will know better once it's had a little time | 21:58 |
| clarkb | lifeless: it shows up for me | 21:58 |
| lifeless | clarkb: odd, I end up on mail-archive.com and crap sites like that | 21:59 |
| pleia2 | anteaya: this looks like the change https://review.openstack.org/#/c/89708/ | 21:59 |
| lifeless | clarkb: e.g. https://www.google.co.nz/search?client=ubuntu&channel=fs&q=Automatic+version+creation+in+PBR&ie=utf-8&oe=utf-8&gfe_rd=cr&ei=PK2HU_rBAqeN8QfnxoC4AQ | 21:59 |
| clarkb | lifeless: I get some of that too but lists.o.o is high in the list | 21:59 |
| clarkb | lifeless: oh I prepend with 'openstack' | 21:59 |
| fungi | i just add site:lists.openstack.org to search terms | 21:59 |
| fungi | or site:bugs.launchpad.net or whatever | 22:00 |
| lifeless | you all suck :) | 22:00 |
| *** hogepodge has quit IRC | 22:00 | |
| anteaya | pleia2: thanks that would be why, the commit message didn't have any mention of script, which was the word I was searching for | 22:00 |
| lifeless | dhellmann_: https://review.openstack.org/96608 | 22:01 |
| clarkb | so infra folks. This mid cycle thing. Do you plan to attend the whole week or just the beginning when infra things happen? | 22:01 |
| * Ajaeger waves good-night | 22:01 | |
| *** gokrokve has joined #openstack-infra | 22:01 | |
| clarkb | Ajaeger: good night | 22:01 |
| *** Ajaeger has quit IRC | 22:01 | |
| *** rfolco has quit IRC | 22:01 | |
| *** jergerber has quit IRC | 22:02 | |
| pleia2 | clarkb: it's a long way to go not to spend the whole time there (and I think it's valuable) | 22:02 |
| clarkb | pleia2: ya thats what I am thinking | 22:03 |
| clarkb | but its only a ~10-11 hour flight if direct | 22:03 |
| jeblair | clarkb: i think the bootstrap bits in the middle are really interesting, so that's basically mon-thur anyway | 22:03 |
| * pleia2 only applies "only" to sub 6 hr flights | 22:04 | |
| clarkb | pleia2: after perth I apply "only" to anything less than half way around the world | 22:04 |
| pleia2 | hehe | 22:04 |
| jeblair | so i'll probably stay the whole week, and maybe do something fun on friday if the qa stuff looks boring. ;) | 22:04 |
| fungi | clarkb: yeah, i am planning to get to frankfurt around lunchtime on sunday, fly out friday night, and get to portland sunday afternoon | 22:05 |
| clarkb | ya I am thinking arrive sunday, get as much sleep as possible then leave saturday | 22:05 |
| bodepd | mordred: I got feedback on those new repo patches that didn't match that example you provided me with | 22:05 |
| clarkb | then probably drive to portland on monday | 22:05 |
| clarkb | or train I may have to train | 22:05 |
| bodepd | clarkb: yay portland | 22:05 |
| pleia2 | oscon hasn't really been on my radar, considering skipping it this time around because I'm conferenced out and someone should stay home ;) | 22:05 |
| clarkb | fungi: we can do the pub track | 22:06 |
| clarkb | pleia2: I won't be at oscon. my mother and brother will be in town that week and with a bunch of people there for oscon I figure I can do the pub track | 22:06 |
| fungi | or the clarkb homebrew track | 22:06 |
| *** gokrokve has quit IRC | 22:06 | |
| pleia2 | clarkb: ah, I see :) | 22:06 |
| clarkb | fungi: indeed. I plan to brew a saison (its too warm for anything else) next month | 22:06 |
| pleia2 | yay, saison is one of my favorites | 22:07 |
| anteaya | so on the shebang line for slave_scripts do you want to use -xe consistently on all the .sh files? | 22:07 |
| anteaya | I see some with -xe some with -e some with -x and some with just the shebang | 22:07 |
| clarkb | anteaya: I think it depends, but making a list and cleaning it up is probably a good idea | 22:08 |
| fungi | -x is generally safe but might be noisy depending on the script | 22:08 |
| fungi | -e will need a lot of care, since a lot of those may not currently be -e safe | 22:08 |
| anteaya | I can draw up a list | 22:09 |
| jeblair | anteaya: is there a problem? | 22:09 |
| anteaya | no, I am trying to be useful | 22:09 |
| jeblair | anteaya: i wouldn't assume that there's a hard and fast rule for those, or that their current values are wrong | 22:09 |
| anteaya | I think I have learned acl scripts enough to give a decently reliable review | 22:09 |
| mattoliverau | Morning | 22:09 |
| anteaya | I thought I would learn what the slave_scripts are doing | 22:09 |
| anteaya | I'm not assuming anything, I'm asking | 22:10 |
| anteaya | morning mattoliverau | 22:10 |
| *** weshay has quit IRC | 22:10 | |
| jeblair | *nod* | 22:10 |
| pleia2 | anteaya: have you read jaypipes' blog posts about our CI system? | 22:10 |
| anteaya | pleia2: the first two I did | 22:10 |
| *** shakamunyi has joined #openstack-infra | 22:11 | |
| anteaya | does he talk about the slave scripts? | 22:11 |
| pleia2 | anteaya: ok cool, http://www.joinfu.com/2014/01/understanding-the-openstack-ci-system/ talks about a few of the slave scripts and helps get a grasp on the general concept | 22:11 |
| anteaya | thanks I'll take another look | 22:11 |
| * jaypipes still needs to complete part 3 on nodepool stuffs... | 22:11 | |
| anteaya | hey jaypipes | 22:11 |
| jaypipes | hey anteaya :) | 22:12 |
| anteaya | as for germany, I don't know where I will be coming in from, I need to hear from reed first | 22:12 |
| pleia2 | anteaya: yeah, just search for "slave_scripts" on the post and you'll get what you want, he shows where they're called from and stuff | 22:12 |
| anteaya | so I don't know when I will be there | 22:12 |
| reed | anteaya, short answer: we're not doing the infra training | 22:12 |
| jeblair | reed: why not? | 22:12 |
| reed | I will send more details from the plane I'm about to board | 22:12 |
| anteaya | I have ancestors that came from bavaria so I am thinking of maybe pricing out a few days round there | 22:13 |
| anteaya | reed: that's a short answer | 22:13 |
| *** mrodden has quit IRC | 22:13 | |
| anteaya | reed: safe travels, I will await details | 22:13 |
| *** aconrad has joined #openstack-infra | 22:13 | |
| pleia2 | anteaya: Krumbach is just 20 minutes from Dusseldorf :) | 22:13 |
| anteaya | we should take a day and visit :D | 22:14 |
| pleia2 | that would be awesome | 22:14 |
| anteaya | let's do it | 22:14 |
| anteaya | I am not going to oscon and apparently there is no prior infra training to hold time open for | 22:14 |
| anteaya | you have to be pain free though | 22:15 |
| anteaya | eat good things | 22:15 |
| anteaya | apples, apples, apples | 22:15 |
| pleia2 | yeah, working on it :) | 22:15 |
| anteaya | go you | 22:15 |
| *** Alexandra_ has joined #openstack-infra | 22:16 | |
| *** andreykurilin_ has quit IRC | 22:18 | |
| *** reed has quit IRC | 22:18 | |
| *** zhiyan_ is now known as zhiyan | 22:18 | |
| clarkb | supposedly Etscheit comes from Etscheid, Germany but that was a long time ago | 22:19 |
| *** shakayumi has joined #openstack-infra | 22:21 | |
| *** shakamunyi has quit IRC | 22:22 | |
| *** nati_ueno has quit IRC | 22:22 | |
| *** markwash has quit IRC | 22:23 | |
| zaro | clarkb: would it make sense to remove your own changes from 'changes need reviewing' in the imporant changes dashboard? | 22:23 |
| clarkb | zaro: it should already | 22:23 |
| *** dims has joined #openstack-infra | 22:24 | |
| clarkb | I think it does -owner:self | 22:24 |
| fungi | lifeless: the used count on that region continues drop off, though there are roughly 10 nodes nodepool has been stuck trying to delete there since yesterday | 22:24 |
| clarkb | if it doesn't then yes we should add that | 22:24 |
| fungi | lifeless: but you're down to 14 still running jobs at the moment | 22:24 |
| *** isviridov|away is now known as isviridov | 22:24 | |
| zaro | clarkb: i'm seeing my own changes. | 22:25 |
| anteaya | what is Etscheit, clarkb? | 22:25 |
| zaro | clarkb: i don't think i have permissions to make that change. | 22:25 |
| dims | lifeless, pong | 22:25 |
| zaro | clarkb: or at least propose it. | 22:25 |
| clarkb | anteaya: its the Micronesian branch of my family | 22:26 |
| clarkb | zaro: I don't see mine. let me fetch the dashboard and see what the query is | 22:26 |
| anteaya | clarkb: cool | 22:26 |
| anteaya | how far from where we will be? | 22:26 |
| anteaya | I'm looking for Reinke's, I don't have a town | 22:27 |
| anteaya | all I have is "Bavaria" | 22:27 |
| clarkb | anteaya: its near koblenz, you can hop on a train and go up there | 22:27 |
| *** zhiyan is now known as zhiyan_ | 22:27 | |
| fungi | lifeless: these are the uuids of instances i don't think will end up being deleted... http://paste.openstack.org/show/82112/ | 22:27 |
| anteaya | sorry no apostrophe, Reinke | 22:27 |
| anteaya | clarkb: cool that might be fun | 22:28 |
| *** yamahata has joined #openstack-infra | 22:28 | |
| fungi | okay, disappearing again for a few hours. back later | 22:29 |
| clarkb | zaro: query = status:open (reviewer:self OR is:watched OR is:starred) label:code-review=0,self NOT is:draft NOT label:workflow=-1 limit:50 <- so maybe that is missing | 22:30 |
| clarkb | zaro: can you test that query and add NOT owner:self? | 22:30 |
| clarkb | see if that fixes it for you | 22:30 |
| lifeless | dims: hey, you might like to comment on https://review.openstack.org/96608 | 22:30 |
| clarkb | zaro: since I can't seem to test it with my current set of changes | 22:30 |
| lifeless | fungi: cool | 22:31 |
| *** bknudson has quit IRC | 22:31 | |
| *** marcoemorais has quit IRC | 22:32 | |
| *** marcoemorais has joined #openstack-infra | 22:32 | |
| *** marcoemorais has quit IRC | 22:32 | |
| *** marcoemorais has joined #openstack-infra | 22:33 | |
| clarkb | fungi: 96620 is handy | 22:33 |
| clarkb | fungi: and you have reviewed that code previously so I am bugging you but feel free to ignore me and drink more osug beer | 22:33 |
| clarkb | zaro: if you can confirm the NOT owner:self works I can update that | 22:33 |
| zaro | clarkb: yep, adding '-owner:self' removes my changes. | 22:33 |
| *** marcoemorais has quit IRC | 22:34 | |
| clarkb | zaro: can you do the NOT owner:self instead so it is consistent with the rest of the query? | 22:34 |
| *** marcoemorais has joined #openstack-infra | 22:34 | |
| zaro | trying now | 22:34 |
| clarkb | it seems to work for me but I can't reproduce the bug | 22:35 |
| *** gokrokve has joined #openstack-infra | 22:35 | |
| zaro | clarkb: yes, either of em seems to work. | 22:35 |
| clarkb | zaro: cool I will push in a sec | 22:36 |
| clarkb | zaro: try it now | 22:37 |
| lifeless | fungi: I'll stab them all in a bit | 22:38 |
| *** jp_at_hp has joined #openstack-infra | 22:39 | |
| clarkb | zaro: it doesn't look like I broke it so hopefully that change fixed what you saw | 22:40 |
| *** thedodd has quit IRC | 22:41 | |
| *** arnaud has joined #openstack-infra | 22:42 | |
| *** dizquierdo has quit IRC | 22:42 | |
| clarkb | zaro: please confirm it either works or doesn't work now | 22:43 |
| dims | lifeless, as i read it. only change in a project that adopts this is additional information/metadata in setup.cfg. right? (no new files) | 22:44 |
| lifeless | no new files | 22:44 |
| *** olaph has quit IRC | 22:46 | |
| *** mbacchi has quit IRC | 22:46 | |
| Alex_Gaynor | If anyone is interested, I've set up a demo instance of livegrep for OpenStack: http://23.253.156.204:1234/ It lets you instantly search across multiple repos (right now I've got nova, glance, cinder, and swift) | 22:46 |
| anteaya | Alex_Gaynor: thanks | 22:47 |
| jeblair | Alex_Gaynor: ohhhh | 22:48 |
| * anteaya claps hands | 22:48 | |
| jeblair | that's fast. | 22:48 |
| anteaya | woohoo | 22:48 |
| * anteaya does a happy dance | 22:48 | |
| anteaya | this is awesome | 22:48 |
| Alex_Gaynor | I guess that means I should set this up properly and get all the repos into it :-) | 22:49 |
| jeblair | Alex_Gaynor: want to puppet it and run it in infra? :) | 22:49 |
| Alex_Gaynor | jeblair: that seems reasonable. | 22:50 |
| jeblair | clarkb, fungi, mordred, SergeyLukjanov, jhesketh: ^ | 22:50 |
| asselin_ | any know of exisitng puppet scripts that install nodepool? | 22:50 |
| sdague | Alex_Gaynor: how stale is it? | 22:50 |
| anteaya | can we put infra repos in it too? | 22:50 |
| Alex_Gaynor | sdague: I cloned the repos a few minutes ago | 22:51 |
| mattoliverau | asselin_: yeah the config repo :P | 22:51 |
| sdague | because the links I'm hitting seem to go to code that isn't there | 22:51 |
| sdague | Alex_Gaynor: http://23.253.156.204:1234/search/livegrep?q=amazon | 22:51 |
| sdague | 1st link definitely doesn't go to that line of code | 22:51 |
| anteaya | oh and can the links resolve to git.o.o instead of github? | 22:52 |
| Alex_Gaynor | sdague: looks like it's off by 4 lines, I wonder why that is | 22:52 |
| morganfainberg | out of curiosity, did we have a netsplit recently? | 22:53 |
| mika | zaro: oh, i didn't notice the 0.8.0 release, my bad - sorry and awesome news! :) | 22:53 |
| Alex_Gaynor | anteaya: probably! this is my first time deploying it, will dig into it | 22:53 |
| anteaya | cool, great first deployment | 22:53 |
| clarkb | jeblair: so that came up when we were switching to cgit | 22:53 |
| * morganfainberg thinks openstackgerrit bot may have ended on the wrong side of a netsplit | 22:53 | |
| clarkb | jeblair: there is that thing wikimedia uses | 22:53 |
| anteaya | morganfainberg: not that I saw | 22:53 |
| clarkb | and as long as live grep isn't that thing ++ from me :) (I don't think it is) | 22:54 |
| anteaya | morganfainberg: oh yeah, it keeps disappearing today | 22:54 |
| morganfainberg | anteaya, ah might be crashes then | 22:54 |
| clarkb | nope thats different ++ from me | 22:54 |
| *** prad_ has quit IRC | 22:54 | |
| clarkb | maybe we can have it live at git.openstack.org/livegrep? anyways I will leave details to Alex_Gaynor and it is cool | 22:54 |
| anteaya | can anyone bring openstackgerrit back? | 22:54 |
| sdague | anteaya: sadly git.o.o doesn't provide line highlighting, and from what I've seen in cgit it's not trivial to add | 22:54 |
| anteaya | :( | 22:54 |
| anteaya | I'm wanting the livegrep but not wanting pointing people back to the githubs | 22:55 |
| morganfainberg | anteaya, [20140529 14:47:43] openstackgerrit (~openstack@review.openstack.org) left IRC. (Ping timeout: 245 seconds) is when i saw it last. | 22:55 |
| clarkb | sdague: line highlighting? | 22:55 |
| jeblair | cgit does support linking to lines, it just doesn't highlight them | 22:55 |
| sdague | that plus syntax highlighting is why I still default to github for web based code links | 22:55 |
| clarkb | ya that | 22:55 |
| sdague | clarkb: highlighting | 22:55 |
| morganfainberg | the line highlighting is nice, but at least we're able to link to the line in cgit | 22:56 |
| sdague | sure | 22:56 |
| *** nati_ueno has joined #openstack-infra | 22:56 | |
| morganfainberg | sdague, ++ syntax highlighting in github is better. | 22:56 |
| jeblair | wow. he hardcoded github urls into it. hopefully he'll take a patch for that. :) | 22:59 |
| jeblair | return "https://github.com/" + repo_map[name] + "/blob/" + shorten(ref) + "/" + this.get('path').path + "#L" + this.get('context').lno; | 22:59 |
| Alex_Gaynor | jeblair: Yeah, I'm sure he will :-) | 23:00 |
| *** asselin_ has quit IRC | 23:00 | |
| jeblair | Alex_Gaynor: is he sitting next to you? ;) | 23:00 |
| *** MIDENN_ has quit IRC | 23:00 | |
| *** asselin_ has joined #openstack-infra | 23:00 | |
| *** kmartin has joined #openstack-infra | 23:00 | |
| *** mdenny has joined #openstack-infra | 23:01 | |
| Alex_Gaynor | jeblair: no, but he's a friend | 23:02 |
| *** jhesketh__ has joined #openstack-infra | 23:04 | |
| *** KurtMartin has quit IRC | 23:04 | |
| *** jhesketh has quit IRC | 23:04 | |
| *** jhesketh__ is now known as jhesketh | 23:07 | |
| jhesketh | Morning | 23:08 |
| *** testmordredbot has joined #openstack-infra | 23:08 | |
| testmordredbot | I ran a command | 23:08 |
| *** testmordredbot has left #openstack-infra | 23:08 | |
| pleia2 | :) | 23:09 |
| mordred | oh. that's rather unexpected | 23:09 |
| *** derekh_ has joined #openstack-infra | 23:09 | |
| *** testmordredbot has joined #openstack-infra | 23:13 | |
| testmordredbot | I ran a command | 23:13 |
| *** testmordredbot has left #openstack-infra | 23:13 | |
| mattoliverau | mordred: have you gotten tired of your human minions and so decided to have some of the bot kind :P | 23:14 |
| pleia2 | I assumed he was trying to replace himself with a small python script | 23:14 |
| mattoliverau | ahh, of course :P | 23:15 |
| clarkb | mordred: did you manage to sit down with people today to look at 1.1? | 23:16 |
| mordred | no | 23:16 |
| mordred | I just back down to palo alto | 23:16 |
| ianw | sdague: i took a closer look at https://review.openstack.org/#/c/93646/ (Exit early on stack/grenade failures) | 23:16 |
| *** W00dy__ has joined #openstack-infra | 23:17 | |
| *** wenlock has quit IRC | 23:17 | |
| *** david-lyle has quit IRC | 23:17 | |
| ianw | sdague: do you know off-hand what the bits in the logs with double timestamps are (2014-05-09 06:09:37.146 | 2014-05-09 06:09:37.117 10343 INFO migrate.versioning.api ) | 23:18 |
| sdague | ianw: so, that should be fixed now | 23:18 |
| sdague | ianw: https://review.openstack.org/#/c/95934/ | 23:18 |
| *** sarob has quit IRC | 23:18 | |
| *** sarob has joined #openstack-infra | 23:19 | |
| *** zhiyan_ is now known as zhiyan | 23:19 | |
| anteaya | for shebang lines we have 6 different types for shell and two for python in slave_scripts: https://etherpad.openstack.org/p/slave_scripts-shebangs | 23:20 |
| *** yamahata has quit IRC | 23:20 | |
| anteaya | morning jhesketh | 23:20 |
| ianw | sdague: ah, ok cool | 23:20 |
| anteaya | yes the mordred bot shows up at irregular intervals and says ossum | 23:21 |
| mordred | :) | 23:21 |
| anteaya | then gives a description of an exotic beverage | 23:21 |
| anteaya | the key is the irregularity of the postings | 23:21 |
| *** doug-fish has left #openstack-infra | 23:21 | |
| * mordred playing with irc notification | 23:21 | |
| clarkb | I am going to afk to walk home a bit early. Apparently there is halibut that needs cooking | 23:22 |
| ianw | sdague: while i have you, https://review.openstack.org/#/c/95073/ (Fix echo_summary missing in lib/infra) | 23:22 |
| pleia2 | clarkb: enjoy | 23:22 |
| ianw | sdague: not sure what dtroyer means by the resync of functions exactly | 23:23 |
| mattoliverau | clarkb: happy walking | 23:23 |
| sdague | ianw: dtroyer already has a -1 on that one. So that should be addressed | 23:23 |
| sdague | ianw: the functions file in grenade is a sync from devstack, but probably old | 23:23 |
| *** sarob has quit IRC | 23:23 | |
| ianw | sdague: echo_summary seems different, it's in stack.sh -- should it be moved to functions in devstack then resynced? | 23:24 |
| sdague | yeh, that's probably the right thing to do | 23:25 |
| ianw | sdague: ok, i'll shuffle that around | 23:28 |
| *** zhiyan is now known as zhiyan_ | 23:28 | |
| ianw | sdague: re https://review.openstack.org/#/c/93377/ (Start nova-conductor before nova-compute) there may be some history i missed. as it stands, this *always* results in a 10-second delay starting nova-compute, which seems sub-optimal | 23:29 |
| ianw | sdague: i will get to fixing https://review.openstack.org/#/c/93375/ (Log when nova-conductor connection established) because it's damn confusing, and i had to jump into the code to figure out if it was an error or not | 23:30 |
| sdague | we really only want to start nova_api in the api chunk. There are rolling upgrade concerns if we don't | 23:30 |
| sdague | and we definitely need to fix that part of conductor | 23:30 |
| sdague | anyway, time to call it a day | 23:31 |
| *** marcoemorais has quit IRC | 23:31 | |
| ianw | sdague: ok, thanks. i'll just abandon that first change then | 23:31 |
| *** marcoemorais has joined #openstack-infra | 23:32 | |
| *** testmordredbot has joined #openstack-infra | 23:34 | |
| testmordredbot | I am a small python script | 23:34 |
| *** testmordredbot has left #openstack-infra | 23:34 | |
| harlowja_ | hi small python script | 23:34 |
| *** tsv has quit IRC | 23:35 | |
| anteaya | it appears to disappear after every statement | 23:35 |
| anteaya | you have that personality characteristic down pat | 23:35 |
| ianw | testmordredbot: os.system("/bin/ls") | 23:37 |
| ianw | always good to try out some xss on such things :) | 23:37 |
| *** atiwari has quit IRC | 23:38 | |
| *** mwagner_lap has joined #openstack-infra | 23:39 | |
| *** chuckC has quit IRC | 23:41 | |
| *** marcoemorais has quit IRC | 23:43 | |
| *** marcoemorais has joined #openstack-infra | 23:43 | |
| *** sarob has joined #openstack-infra | 23:44 | |
| *** derekh_ has quit IRC | 23:44 | |
| *** sarob_ has joined #openstack-infra | 23:48 | |
| *** sarob has quit IRC | 23:48 | |
| lifeless | fungi: how would you feel if I nuked all the instances ? | 23:50 |
| *** sarob has joined #openstack-infra | 23:50 | |
| *** testmordredbot has joined #openstack-infra | 23:50 | |
| testmordredbot | I am a small python script | 23:50 |
| *** testmordredbot has left #openstack-infra | 23:50 | |
| jeblair | mordred: i usually make a different channel for that sort of thing... | 23:51 |
| jeblair | (at least for early testing) | 23:51 |
| *** chuckC has joined #openstack-infra | 23:52 | |
| *** sarob_ has quit IRC | 23:53 | |
| *** gokrokve has quit IRC | 23:54 | |
| *** gokrokve has joined #openstack-infra | 23:55 | |
| *** sarob has quit IRC | 23:55 | |
| *** oomichi has joined #openstack-infra | 23:55 | |
| *** gokrokve has quit IRC | 23:55 | |
| mordred | jeblair: good point | 23:56 |
| *** testmordredbot has joined #openstack-infra | 23:56 | |
| testmordredbot | I am a small python script on git05.openstack.org | 23:56 |
| *** testmordredbot has left #openstack-infra | 23:56 | |
| mordred | jeblair: (was already running before you said that - I'll move to a channel) | 23:56 |
| *** lakshmiS has quit IRC | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!