14:59:59 #startmeeting airship 15:00:01 Meeting started Tue Dec 3 14:59:59 2019 UTC and is due to finish in 60 minutes. The chair is mattmceuen. Information about MeetBot at http://wiki.debian.org/MeetBot. 15:00:02 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 15:00:03 #topic Rollcall 15:00:04 The meeting name has been set to 'airship' 15:00:11 Hello everyone - time for our weekly IRC meeting! 15:00:11 o/ 15:00:13 o/ 15:00:15 o/ 15:00:16 o/ 15:00:20 Here's the agenda: https://etherpad.openstack.org/p/airship-meeting-2019-12-03 15:00:21 o/ 15:00:24 o/ 15:00:29 please add any additional items, we'll wait a couple minutes to get giong 15:00:47 o/ 15:00:55 o/ 15:02:17 o/ 15:02:42 o/ 15:02:50 Thanks for joining everyone 15:02:58 #topic Some interfaces between airshipctl and kustomize are not avaialble yet 15:03:13 We have several items left from last week which we didn't have time to get to 15:03:26 And I'm not certain now who added them to the agenda :) 15:04:06 re: interface between airshipctl and kustomize, I know Alan had been working to get a patchset in for that before the holidays, and I'm not sure where that landed 15:04:12 treasuremap 2.0 thing, I added it, it got answered last week, so, we can skip that 15:04:20 ok awesome - thanks uzumaki 15:04:50 I think we covered the Treasuremap 2.0 general topic last well - any other discussion you'd like to have on that uzumaki? 15:05:23 Not yet, so we're good 15:05:51 ok cool 15:05:58 #topic Image and Userdata parameters are missing in baremetal.yaml in airshipctl under spec Baremetalhost 15:06:53 Are those missing fields due to old/incomplete generated code in airshipctl, do you know? Or, are they not in the upstream Metal3 spec yet? 15:07:52 diwakar thyagaraj proposed airship/porthole master: Mysqlclient UC Python and Ubuntu upgrade. https://review.opendev.org/696184 15:08:46 Hmm I'm also not finding a baremetal.yaml in airshipctl, so I'm unclear on this topic. Is anyone familiar with it? 15:09:19 mattmceuen: the file is testdata. It's located at pkg/document/testdata/baremetal.yaml 15:09:34 I'm not too familiar with the issue though.. 15:11:13 alright, let's hold off on that till the owner is here and we can talk through it 15:11:39 * ildikov is lurking :) 15:11:43 #topic what is equivalent to openshift-machine-api 15:11:47 o/ ildikov :) 15:11:59 mattmceuen: hi :) 15:12:15 Do we have the owner of this openshift-machine-api topic here this week? 15:12:31 " what is equivalent to openshift-machine-api (https://github.com/metal3-io/baremetal-operator/blob/master/examples/worker-0.yaml#L27) in metal3" 15:13:17 the question seems to be around the last line in there, defining the namespace 15:14:19 I think next time we carry over a bunch of topics to the next meeting, we can start by having the topic owners signal they are here and still want to discuss -- lesson learned :) 15:15:06 alright, moving on to new topics this week: 15:15:09 #topic Revisit use of root user in Airship containers - Pegleg, Spyglass, others? Can these be changed to either generic "airship" project or to project specific user? 15:15:24 alexanderhughes, is this one your shade of lavender? 15:15:38 lavender is the best color 15:16:03 +1 15:16:04 yes, so I was looking at projects again after reading some articles over the break. I'd like to change containers to run as a non-root user in Airship where possible 15:16:31 sounds good to me 15:16:47 currently Pegleg and Spyglass are root. I don't see a reason for this so I'd like to change them if there aren't any concerns from community. in doing so if we plan to continue accepting documents in Pegleg container from Promenade should we consider changing both Promenade and Pegleg to a generic "airship" user to avoid chowns? 15:17:04 or let operators use chowns between containers as has been in the case up til this point 15:17:41 id advocate for a generic user 15:17:54 this is what was done in the loci project to address similar concerns 15:17:57 I don't have any concerns with that - sounds good to me 15:18:01 I like the idea of them having a generic 'airship' user 15:18:24 42424 is what loci used 15:18:37 so 42425 sounds good to me ;) 15:18:42 lol 15:18:47 Don't most projects currently use the 'nobody' user? 15:19:08 nobody has some baggage around it 15:19:22 and where we are, we can make use of a 'real' user 15:19:36 which makes it much easier to set up ownership etc 15:20:01 Right. Then I think an airship user would also be a good thing to move towards 15:20:03 I'm all for "airship" user across all the projects 15:20:06 if we made airship use `nobody` then everyone who uses `nobody` would have the same rights for files 15:20:21 I would still think they should be overridable though 15:20:22 this went to design call a few weeks ago, and we couldn't get consensus 15:20:23 +1 for "airship" 15:20:33 mattmceuen: ++ this is how we did it for loci 15:20:38 Merged airship/porthole master: Enable runtime-default Apparmor Profile to Openstack-Utility Container. https://review.opendev.org/696186 15:20:39 And I don't think we should add dependencies between containers 15:20:57 I.e., don't want one container to assume another container is running as a particular user 15:21:15 w.r.t your promenade + pegleg concern alexanderhughes- can you please dive into that issue? 15:21:47 the end goal is that pegleg is the access point for promenade commands - render, generate bundle, gen certs etc. but we haven't finished that transition yet 15:22:12 as a result operators are rendering in promenade/collecting. then sending those documents to pegleg, but they have different owners "root" "promenade" and causes issues without chown 15:22:36 in the case of pegleg->promenade in particular the files pegleg generates for promenade to consume can't be used because of the 640 permissions on root owned files 15:23:01 standardizing the users or chown documents shared between containers are only paths forward I see until the transition to all pegleg is complete 15:23:41 I am less concerned about the promenade CLI portion (since it's deprecated) 15:23:47 diwakar thyagaraj proposed airship/porthole master: Enable runtime-default Apparmor Profile to Calicoctl Utility Container. https://review.opendev.org/694793 15:24:10 The promenade API portion is already configurable, so we should avoid tying it down to a uid along with the CLI: https://opendev.org/airship/promenade/src/branch/master/charts/promenade/values.yaml#L180 15:24:19 how does that sound? 15:26:04 i.e., I'm ok with locking down the prom CLI to a specific UID (temporarily, till transitioned to pegleg), provided we keep the API uid configurable 15:26:17 sure 15:26:26 ok awesome 15:27:19 #topic Moving issues from Jira to Github 15:27:29 uzumaki I think this one is yours 15:27:54 yup, we had a discussion last week about the issue movement. Any updates from the WC/TC? 15:27:54 From my perspective, no progress since last week, when everyone seemed to be pretty positive around a change to github issues 15:28:32 The holiday got in the way last week sorry :) I will send out a note on the ML to make sure there aren't any concerns 15:28:47 well, how do we plan to pursue it, the actual movement itself? 15:28:53 mattmceuen, that's alright, no worries 15:29:12 I guess we have three choices 15:29:23 1. manual copy and paste to move items 15:29:37 2. "drain" jira and finish out work there while adding new work in github issues 15:29:56 3. put work into an automated job to migrate stuff into github 15:30:28 any preference from your side uzumaki? 15:30:48 Prateek Dodda proposed airship/porthole master: [WIP] https://review.opendev.org/697113 15:31:04 My concern is, if people are joining in the community, during the migration, is there a way to make it easier for them to understand what is what? 15:31:18 +1 15:31:40 +1 to acknowledge that's a very good concern 15:31:59 Otherwise, if it's only for us, the current members, any option out of 2/3 is fine (1 is too much work) 15:32:32 I suspect 3 may also be too much work ;-) 15:32:47 haha 15:33:04 Rodolfo has a weekly meeting on JIra scope tracking, I think it would be a good place to pitch the "drain" idea 15:33:09 There may be a good "cut" point 15:33:27 Escalate this, along with this concern to the other WC/TC members, this needs to be thought through 15:33:31 i.e., maybe starting with e.g. Airship 2.0 beta, scope moves to github, or something like this 15:33:39 mattmceuen, yeah, drain idea seems fine, least work 15:33:59 for sure, we have a WC meeting next week, I will bring this up there as well 15:34:11 that'd be great! 15:34:15 o/ 15:34:39 o/ dwalt - just to confirm (since I had to miss it) - you didn't discuss this yesterday, right? 15:35:09 we did not. As a WC member, it would be really helpful to get input from those already working scope in Jira 15:35:38 Since we want to cause as little disruption as possible 15:36:04 indeed 15:37:00 as I am catching up, I see you said option 2 or option 3 :) 15:37:19 we're open to suggestions! 15:37:24 We will discuss this next Monday (12/9) then. The meeting is open to anyone to join 15:37:42 i'll be there, great! 15:37:50 great! Thanks uzumaki 15:37:51 can I ask a dumb question around the decision to use GitHub issues? 15:37:59 sure can ildikov 15:38:03 go ahead! 15:38:03 additionally, the etherpad is here. You can also leave comments if you cannot attend: https://wiki.openstack.org/wiki/Airship/Airship-WC 15:38:12 if I know correctly you use Git/Gerrit for code development 15:38:22 yup 15:38:34 so I wonder if it doesn't get confusing to use GitHUb for issues but not for the rest of the flow? 15:38:48 ildikov asking the good questions 15:38:57 I think that's a fair point 15:39:08 I assume you talked this through so if you have meeting logs somewhere for my understanding I can catch up on that as opposed to keep up the meeting on this :) 15:39:30 we would associate the issues with the github mirror of our opendev projects, so it wouldn't be "completely divorced" 15:40:00 it just looked confusing to me, who's not deeply involved at this point 15:40:17 that part makes sense, however the mirror is read only 15:40:19 it'd be difficult to fight off the urge to open a pull request via github, i'll admit 15:40:38 we've had a couple brief conversations, but haven't gotten deep - I'll capture some of the potential gotchas/concerns in an etherpad, along with some feedback from the kubecon meetup 15:40:40 Samuel Pilla proposed airship/promenade master: Upgrade Tiller version for k8s 1.16 https://review.opendev.org/693395 15:40:41 o/ srwilkers 15:40:44 o/ 15:40:45 and my memories from OpenStack is that people get confused by the mirror even when you don't use any GitHub feature besides browsing the code 15:42:04 I admit it's not my favorite approach. However the pros/cons need to be weighed against overwhelming developer preference (so far) to using github issues for tracking scope, seems to hit a sweet spot of ease of use + functionality 15:42:31 mattmceuen: I'm happy to add this to a pro/con etherpad if/when you create one 15:42:45 I think in the wash it'll be "what's the least annoying of N evils" :) 15:42:54 so pros/cons lists will help with that 15:43:17 lol, I'm with you on the least annoying note 15:43:35 thanks much ildikov - will share w/ you as soon as I get it up, and will post it to the ML 15:43:40 I brought this up more from forward looking perspective and with new comers in mind 15:43:41 Samuel Pilla proposed airship/treasuremap master: Upgrade Tiller version for k8s 1.16 https://review.opendev.org/694604 15:43:55 mattmceuen: sounds great, thank you! 15:44:02 I'll keep an eye on the ML 15:44:17 #topic Roundtable 15:44:35 That's it as far as the agenda goes, team - any additional topics, feedback, or requests for code review? 15:45:25 nope, looking forward to the pros/cons list on the ML 15:45:55 awesome - I will prioritize that to the top of my overdue todo items :D 15:46:03 I'm sure you will :D 15:46:08 hoping to catch up before xmas holidays 15:46:24 Samuel Pilla proposed airship/treasuremap master: Upgrade Tiller version for k8s 1.16 https://review.opendev.org/694604 15:46:25 on that note, then - I'll give us 14 minutes back! 15:46:33 Thanks everyone for your time and the great discussion 15:46:48 have a great week, and see you here in IRC/ML in the meantime 15:46:50 thanks all :) 15:47:01 #endmeeting