19:02:38 #startmeeting infra 19:02:38 o/ 19:02:39 o/ 19:02:40 Meeting started Tue Jun 3 19:02:38 2014 UTC and is due to finish in 60 minutes. The chair is jeblair. Information about MeetBot at http://wiki.debian.org/MeetBot. 19:02:41 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 19:02:44 The meeting name has been set to 'infra' 19:02:52 #link https://wiki.openstack.org/wiki/Meetings/InfraTeamMeeting 19:02:54 agenda ^ 19:03:01 #link http://eavesdrop.openstack.org/meetings/infra/2014/infra.2014-05-27-19.00.html 19:03:04 last meeting ^ 19:03:17 nothing from last meeting so 19:03:23 #topic manage-projects status (fungi) 19:03:53 howdy 19:04:11 fungi: are there any errors we should be looking out for still? 19:04:19 i noticed we created a couple more projects recently 19:04:48 looks like the fix which i had proposed is no longer waiting on https://review.openstack.org/94196 19:05:00 so just needs reviewing/approving 19:05:11 #link https://review.openstack.org/94684 19:05:20 i haven't spotted any new failures though 19:05:34 there was the github failure to set account rights 19:05:42 but other than that I haven't seen anything 19:05:45 clarkb: oh, right. there was 19:06:03 o/ 19:06:12 o/ 19:06:12 that we've so far attributed to github api call failures or throttling, but i don't know if anyone has gone digging in the puppet logs yet 19:06:37 we might see it if we approve all those xstatic changes 19:06:41 it's happened a couple times in the same pattern though, in as many months 19:06:51 quite possibly 19:07:10 16 repos in that xstatic patch 19:08:08 yeah, it'll a good test for manage-projects 19:08:17 okay, so next time we see the github thing, we should probably start digging into the logs 19:08:24 ++ 19:08:32 yes 19:08:33 #topic Review Third Party wiki templates (anteaya) 19:08:46 #link https://wiki.openstack.org/wiki/Template:ThirdPartySystemInfoSubst 19:08:56 #link https://wiki.openstack.org/wiki/Template:ProgramsThirdPartySubst 19:09:13 o/ 19:09:14 so the first one is the one we looked at last week, for third party accounts 19:09:27 the second one is new, meant for openstack programs 19:09:49 if we are happy, I can get krtaylor to fill out one for third party 19:09:58 anteaya: cool. most of that looks good. i'd suggest removing the mission statement as it's sort of duplicative 19:10:01 and perhaps jeblair will fill out one for infra 19:10:06 as a seed for others 19:10:13 anteaya: nope, we're not a third party :) 19:10:17 I can remove the mission statement 19:10:19 yeah, i think we discussed having the mission statement just replaced by a link to the program page 19:10:24 we are an openstack program though 19:10:27 fungi: ++ 19:10:35 which is what the second template is for 19:11:24 anteaya: ah, i see; though we have no third-party testing related to our program... i'm not sure it makes sense to have this for every program, but rather just for those that have 3rd party testing 19:11:28 anteaya: one question though, what is the intended workflow here for third-party systems which are dedicated to only testing stackforge projects (you don't have to know the answer now, just worth thinking about--there are more than a few) 19:11:48 right, I we do have a relationship with thrid party though 19:12:13 in taht we oversee the struture and have a direct say in how that structure is built and managed 19:12:47 fungi: I don't know, I don't know how stackforge testing would differ from testing openstack projects 19:12:58 I would need to talk to some maintainers of such an account 19:12:59 anteaya: sure, but i don't think we have the kind of relationship that i see this page being used for 19:13:05 okay 19:13:16 I'll work with neutron to make a seed page 19:14:02 any other comments/corrections on these templates? 19:14:16 I can work to have the seed pages up for review next week 19:14:26 anteaya: lgtm, and thanks again! 19:14:29 great 19:14:31 thanks 19:14:58 * ttx lurks 19:15:15 #topic Formatting Automated Gerrit Account Names (anteaya) 19:15:25 me again 19:15:30 #link https://etherpad.openstack.org/p/automated-gerrit-account-naming-format 19:15:53 so I have dumped what I have found about gerrit account names for automated accounts 19:16:03 we discussed some keywords last week 19:16:17 I need some more agressive guidance on how to seperate the groups 19:16:28 and then to id which ones need to be renamed 19:16:37 and then what process to rename 19:17:08 anteaya: anything that votes (or is intended to eventually vote) in the verified colum is "{name} CI" 19:17:19 great 19:17:20 anteaya: anything else automated is "{name} Bot" 19:17:25 that helps 19:17:37 okay so when we look at {name} 19:17:41 what are the options? 19:17:45 ++ for CI/Bot naming 19:18:01 anteaya, there are company names and project names 19:18:02 we have a group that needs 4 cinder ci accounts 19:18:10 that can get long 19:18:17 anteaya, we have sahara-ci (honestly it's still savanna-ci) 19:18:24 you do 19:18:25 anteaya: i think most of those pass the sniff test; they seem fairly descriptive 19:18:36 okay so we can grandfather these 19:18:50 anteaya: oh, well, i think many of them should change 19:18:51 seems fine, then anyone wanting to filter all automated systems can just match on " (CI|Bot)$" in names i suppose 19:18:57 fungi: yup 19:19:16 jeblair: just the descriptive name or the username too? 19:19:17 jeblair: many of the existing ones on the list should change? 19:19:20 fungi, it'll be useful I hope 19:19:20 anteaya: i think we should be looking for something like "{company} {product} CI" 19:19:28 jeblair: great 19:19:42 anteaya: or "{product} CI" or "{company} CI" if they are, er, a less diverse company. :) 19:19:45 i also think getting rid of "openstack" in third-party ci/bot names is advisable (i try to trim that out when i'm the one handling requests) 19:19:57 fungi, ++ 19:19:58 fungi: ++ 19:20:00 yes, I agree that only we should be using openstack 19:20:14 but i do see some in the list, so they ought to get cleaned up in this process 19:20:19 let me come up with a list of suggested revisions for the list for next week 19:20:22 and we need to rename it to openstack-ci (OpenStack CI) 19:20:56 do we plan on updating name and username? 19:21:01 or just the descriptive name? 19:21:17 I think everything 19:21:29 part of the issue is that name and username don't match 19:21:32 https://review.openstack.org/Documentation/cmd-set-account.html won't update the username 19:21:34 like brocade 19:21:38 so username will require DB changes 19:21:44 anteaya: ok 19:21:53 we have brocade_jenkins and brocade_tempest for teh same account 19:21:58 that is just confusing 19:22:20 yes, that will be a step during the process of changing the names 19:22:25 editing the db 19:22:26 i don't think the usernames matter 19:22:42 I would like them to be consistent 19:22:57 they aren't displayed in the webui with their votes, and changing them instantly breaks teh remote systems which were logging in with them 19:23:05 so if someone asks me about brocade_jenkins I know what account they are talking about 19:23:07 which is a mighty steep price for consistency 19:23:15 fungi: good point 19:23:36 gerrit already makes sure they're unique, so i think that should be sufficient 19:23:49 do we have any control over how they set their usernames? 19:24:05 personally I'd like to see username-displayedname consistence, but price - break all 3rd party, edit db manually 19:24:07 so that at the very least in future the usernames and account names can match up? 19:24:24 we can assign them in the future 19:24:32 great 19:24:36 I can settle for that 19:24:37 just need to make sure that's part of the process 19:24:41 * anteaya nods 19:24:46 could we always resolve username to display name? 19:25:00 there are gerrit api calls which can do that 19:25:03 i believe 19:25:08 like russellb's reviewstats shows usernames 19:25:14 did 19:25:20 reviewstats is offline 19:25:44 I think I have enough to work on this and come back next week with some suggestions 19:25:48 thanks 19:25:48 not sure it's worth publishing anymore, fine if people want to run locally, but stackalytics has a better UI anyway 19:25:48 https://review.openstack.org/Documentation/rest-api-accounts.html#get-account yup 19:26:05 russellb: I liked yours better 19:26:16 clarkb, thanks 19:26:27 russellb: i use it; i find examining disagreements to be important 19:26:29 so, we should have no issues with it 19:26:36 you guys seen http://stackalytics.com/report/contribution/nova-group/30 ? 19:27:06 russellb: there isn't anythin like that for infra last I looked 19:27:14 (probably does exist it just isn't navigable) 19:27:18 russellb: I hadn't before, I still like yours better, simpler and I like text 19:27:21 clarkb: http://stackalytics.com/report/contribution/infra-group/30 19:27:34 I had chatted with anteaya a couple weeks ago about hosting it somewhere in infra 19:27:35 russellb: nice; though i actually have reviewstats _output_ the disagreements so i can read them; and i also make lots of ad-hoc groups. 19:27:45 ah, cool 19:27:57 then other things came up, I think I was supposed to follow up with russellb to see if he wanted us to ;) 19:28:02 well i can fix the hosted stats if enough people really care ... I think it may be something with the upgrade, i dunno, i haven't messed with it 19:28:12 jeblair: we could likely get stackaltics to do that too 19:28:13 +100000 to hosting it in infra 19:28:15 plz 19:28:21 if people want it 19:28:25 anteaya: so i think that's it for the account name topic 19:28:27 ? 19:28:28 just never got around to doing the work to get it hosted myself 19:28:33 jeblair: yes 19:28:34 russellb: cool 19:28:44 i think the horizon repo topic is left over from last week 19:28:47 so next up 19:28:52 #topic Release git-review 1.24 19:28:55 i don't know who added that 19:29:10 i'm not sure if it was intended as an imperative 19:29:18 jeblair: do it now ! :P 19:29:21 yes! 19:29:21 like, adding it to the wiki page will cause it to happen 19:29:25 I am on board with a release 19:29:39 * ttx hacks up a irc-to-tag script 19:29:45 does the https functionality have feature parity ish with ssh now? 19:29:47 fungi: ? 19:29:48 ha ha ha 19:30:02 ttx: wiki -> irc -> tag 19:30:04 I think that is the one item that may be nice to have in before doing a release but it isn't critical either 19:30:05 yep, i'll go ahead and cut one. did we want to get the -W option reviewed and merged for 1.24 as well? 19:30:24 fungi: there was enough commenting on that chnage that I didn't expect it to merge quickly 19:30:27 i think i added it and maybe forgot to add (fungi) on the line 19:30:31 but I didn't look at it after the first patch 19:30:35 jeblair: as soon as I'm finished with my IRC actions -> RememberTheMilk gateway 19:30:45 i was letting the dust settle on it but need to revisit 19:31:12 * ttx looks forward to not having to ad LANG=C every time he uses git-review 19:31:25 fungi added it: https://wiki.openstack.org/w/index.php?title=Meetings/InfraTeamMeeting&diff=prev&oldid=53841 19:31:28 ttx speak moar english 19:31:46 fungi: is the '-W' option an openstackism? 19:31:56 anteaya: yeah, i remember adding it, was just admonishing myself for forgetting the nick tagline 19:31:58 mordred: I realized in Jerusalem that French should be attempted first 19:32:03 fungi: ah 19:32:20 jeblair: yes 19:32:23 jeblair: yah. 19:32:26 jeblair: good point, it's for setting workflow +1 and its docs mention it only works with a gerrit configured for that label 19:32:27 clarkb: :( 19:32:35 er, workflow -1 19:32:49 would be neat if that could be generalized, but i don't have any _good_ ideas off the top of my head for that 19:32:57 yeah, i'll see if any spring to mind 19:33:01 jeblair: maybe a .gitreview value to set on -W 19:33:24 possibly, followed by a mass patchbomb to all projects in our gerrit 19:33:25 clarkb: heh, that's going to be a fun 297 patches to merge 19:33:33 i can take that up 19:33:43 or make it default to what we do 19:33:49 and let others override that way 19:33:51 that would then be an openstackism 19:34:06 i'm more in favor of having it default to off 19:34:14 anyway, that can be taken up in the review 19:34:18 no need to waste meeting time on it 19:34:22 i mean "git review --label='Work In Progress'" works but isn't exactly convenient. 19:34:30 fungi: true. 19:34:49 config file for WIP label? 19:34:52 fungi: so, maybe cut 1.24 without wip? 19:34:56 to make ttx happy 19:34:58 jeblair: yeah, i think so 19:35:00 jeblair, ++ 19:35:09 not only ttx :) 19:35:17 wip can be considered for 1.25 19:35:23 SergeyLukjanov: ;) 19:35:32 sounds good 19:35:34 #topic Fedora/Centos7 Plans (ianw) 19:35:35 i've been using current master long enough i'm pretty sure it's sane (plus, we do have some testing on it as well) 19:35:51 hi, so the f20 job for devstack has been pretty reliable 19:36:07 so i'm looking at getting it on the path to voting 19:36:16 the first thing is getting it in multiple clouds 19:36:39 was the issue that hpcloud didn't have a base image? 19:36:43 are we fully moved to the new hp cloud that has f20 images? 19:36:55 hpcloud only has f17 iirc 19:36:56 ianw: we are 19:37:02 even in 1.1 19:37:08 oh?!? 19:37:10 eek 19:37:22 | 831fa6a5-1ca5-42ea-bd41-4cbebf01085a | Fedora 20 Server 64-bit 20140407 - Partner Image | ACTIVE | 19:37:28 clarkb: theres a partner image 19:37:30 ah another partner image 19:37:30 * anteaya has lost power once already, it could happen again anytime 19:38:10 so most of our "run on new image" work is waiting on dib/glance in nodepool 19:38:19 but since we already have this working on f20... 19:38:29 i don't see a reason for this work to block on that. 19:38:37 yup the unbound change to make dib work needs approval I had planned on doing that today but we are still underwater on the zuul/nodepool stuff 19:38:43 jeblair: wfm 19:38:47 that's my next question; what is the status of the dib work? 19:39:08 ianw: increasing review priority for me this week, assuming nodepool settles down for us 19:39:14 ianw: there are patches up for review. We need a small change to puppet which is ready for approval we just need time 19:39:23 ianw: then its on to reviewing the changes that actually marry dib and nodepool 19:39:45 is there a nodepool glance change yet? 19:39:55 mordred_phone: ^ 19:39:55 clarkb: ok, i might ping you outside meeting time to get fully up to speed 19:40:31 so the other thing, thinking forward to centos7 release, i think that's going to be the best long-term base for rpm testing 19:40:41 +1 19:40:47 * jeblair assumes mordred's seatback and tray tables are in the upright position 19:40:48 presumably, the dib work will be the way to deploy centos7? 19:40:48 ianw: any eta on that upstream? 19:41:07 ianw: yes I would expect to do trusty and centos7 via dib 19:41:19 fungi: i think "soon" 19:41:31 rsn... got it 19:41:45 fungi: what was your question? 19:42:10 jeblair: i was wondering when centos 7 was due for release 19:42:13 is centos7 going to be used for infra, or just for testing? should i add a puppet-apply test for f20 and/or centos7? 19:42:16 fungi: ah, gotcha 19:42:43 jesusaurus: i think we will be in no hurry to move our centos6 servers to 7 19:42:48 jeblair: we will likely only use fedora for testing never for infra 19:42:55 er jesusaurus ^ 19:42:55 jesusaurus: i expect we would be likely to use it in places where we need it, but yeah, no need to upgrade just for the sake of it 19:43:04 gotcha 19:43:05 so the value of testing on fedora 20 is minimal when it comes to that test 19:43:15 so, in conclusion, i should test out the hp f20 partner image with the idea of bringing it it into nodepool? 19:43:24 ianw: i think so 19:43:36 yup run the build scripts on it and see if it looks happy 19:43:40 then we can add the image to nodepool 19:44:01 centos7 should probably wait for the dib stuff to land (and, also, centos7) 19:44:04 and i will reach out to clarkb about the dib work and that will be the base for centos7 19:44:15 ianw: sounds good 19:44:52 jeblair: yeah, just want to be as ready as possible when it does land :) 19:45:01 ok, thanks, no more on that topic from me 19:45:06 ianw: cool, thanks! 19:45:11 #topic Consistency in acl reviews (anteaya) 19:45:21 #link https://review.openstack.org/#/q/owner:%22Dan+Bode%22+file:%255E.*/acls/.*+NOT+status:abandoned,n,z 19:45:33 apologies in advance if I lose power again 19:45:45 so I would like to review the reviewing of this set of patchs 19:45:53 specifically the acl file reviews 19:46:12 going back of these I realize I was inconsistent, so I need to address that and be more consistent 19:46:18 but the part I wanted to discuss 19:46:29 and I do wish sdague and zaro were here 19:46:59 is that on one of the patches, dan got -1'd for editing the file the way I had asked him to edit 19:47:08 which I find embarassing, personally 19:47:21 he got -1'd twice 19:47:24 here. sorry i forgot about meeting. 19:47:31 I don't think it is embarrasing. It is definitely inefficient. 19:47:45 we can hash out these disagreements without makeing the author go back and forth 19:47:52 agreed 19:47:55 anteaya: it definitely wasn't due to a lack of accurate reviewing on your part; it was because the actual issue is very unclear 19:48:07 if I am reviewing a file poorly, I would like to know and to improve 19:48:14 great 19:48:21 let's see if we can get some clarity 19:48:26 two things I saw 19:48:41 require contributor agreement = true 19:48:50 and the [project] stanza 19:49:00 * anteaya notes the howling wind outside 19:49:09 anyone with any thoughts? 19:49:19 in order to avoid wild swings on the CLA issue, i think we should avoid trying to decide it ourselves early... 19:49:29 jeblair: I agree 19:49:37 it's an unanswered legal and policy question, and will likely be so for quite some time 19:49:37 what should we do for now? 19:49:50 long ago i stopped -1'ing patches for cruft where people are cargo-culting no-op default values 19:50:15 so i think the approach of not changing our existing projects, and also not requiring the cla for projects that are split from projects that don't require the cla is the closest thing we have to keeping the 'status quo' 19:50:28 ++ 19:50:43 okay, so in this case referencing config.config since that is the parent acl of this change series 19:50:47 I can follow that 19:50:48 that's basically what i've done as well 19:51:09 I will do that too, or ask if I can't figure out the parent 19:51:15 is [project] cruft? 19:51:20 I had thought it is 19:51:56 i'm not sure i understand your question 19:52:13 the active = true bit is 19:52:13 in the acl file, the [project] stanza 19:52:23 https://review.openstack.org/#/c/96522/5/modules/openstack_project/files/gerrit/acls/openstack-infra/puppet-pip.config 19:52:23 or is it status = active 19:52:26 whatever that line is 19:52:30 status = active 19:52:34 anteaya: it's not necessary, certainly. i think fungi was saying that it doesn't matter either way, so no use going back and doing another patchset about it 19:52:36 but it doesn't hurt to have it either and is probably not worth a -1 19:52:51 anteaya: what we might want to do is remove all refs from our docs and then merge a patch that removes it from all acl files 19:52:52 clarkb ++ 19:52:54 okay, I will ignore it then if it is in the file 19:52:57 anteaya: then we might stop the cargo culting 19:53:00 right, i need to update my current cleanup patch series, but i think it's something we solve by fixing all the existing acls in one go so people stop copying around unnecessary stuff 19:53:12 fungi: oh you've already started on that :) 19:53:13 fungi: agreed 19:53:21 jeblair: yup there is a series of WIP changes 19:53:22 i've already git it basically scripted 19:53:28 okay, so I will ignore it for now if it is there 19:53:30 I'm still thinking on yaml-based acls, hope to publish spec someday 19:53:44 to make it repeatable and better able to weather rebase hell or race conditions in reviewing existing changes 19:53:50 I don't usually suggest the refs/tags/* line in a file that doesn't have it 19:54:04 I figure if people want tags they will know to include it 19:54:42 I'm trying to suggest tags or smth else when see that folks most probably would like to have it 19:54:51 our documentation should suggest reasonable things, and then we should spot when something is required to be added to the acl by implication from jobs they may be adding or similar evio\dence 19:54:53 evidence 19:55:25 maybe I need to know more about tags then 19:55:27 fungi: I think a large problem is a lot of people doing this have zero experience with our infra and gerrit 19:55:34 clarkb: + 19:55:54 they are told to stackforge or do $thing with openstack and really don't grok what they need 19:56:13 so suggestions may help that 19:56:20 right. if a project is adding release jobs but has no tag section in their acl for example, then it might merit mentioning (to find out if they really meant to add the jobs, or require the acl addition) 19:56:25 maybe a "So you wanna stackforge" doc 19:56:48 clarkb: the existing stackforge doc could be expanded 19:56:56 jeblair: ya 19:57:28 #topic Open discussion 19:57:34 #link https://wiki.openstack.org/wiki/Qa_Infra_Meetup_2014 19:57:45 for anyone going to the meetup, could you please sign up on that page ^ 19:57:55 * SergeyLukjanov remembering the time when I've proposed the sahara addition to stackforge CR 19:58:24 I have the "not quite production" zanata puppet files from Carlos, will be looking through them and hope to have some kind of maintainability report by next meeting (if you've been following the thread on list, it's kind of tricky) 19:59:05 pleia2: i'm sort of surprised. i guess at the summit they didn't realize we run free software here? 19:59:28 jeblair: redhat does all kinds of open source, you just need a license for delivery and updates ;) 19:59:40 sorry, that's probably inapproprite, I'm just frustrated 19:59:52 :( 20:00:02 had a question about private gerrit for security reviews bug 1083101 20:00:03 Launchpad bug 1083101 in openstack-ci "Set up private gerrit for security reviews" [High,In progress] https://launchpad.net/bugs/1083101 20:00:09 pleia2: if we can't run it, pootle still seemed like a good option; the delta between it and zanata was small, and we have people who might actually hack on it. 20:00:21 do we want to continue to make that happen? 20:00:30 jeblair: noted, I'll see how far I get this week we'll go from there 20:00:42 zaro: oh, right, i asked the rest of the vmt. it's still desirable for us 20:00:47 i'm out next week, i'll be completely unreachable 20:00:48 pleia2, your work on a transifex replacement is appreciated. Just heard today some comments that transifex has some problems - we might have lost some translations ;( 20:00:49 +1 20:01:14 and i think we're at time 20:01:16 #endmeeting