Tuesday, 2020-02-18

*** nishantkr has quit IRC00:46
openstackgerritJordan Jensen proposed airship/airshipui master: Remove execute permissions for Developers Guide  https://review.opendev.org/70828400:58
*** dwalt has quit IRC01:16
openstackgerritAhmad Mahmoudi proposed airship/shipyard master: WIP: (fix) Addressing file not found issue  https://review.opendev.org/70770902:28
openstackgerritAhmad Mahmoudi proposed airship/shipyard master: WIP: (fix) Addressing file not found issue  https://review.opendev.org/70770903:10
openstackgerritPrateek Dodda proposed airship/shipyard master: Implement Security Context for Airflow_Worker  https://review.opendev.org/69222405:21
openstackgerritPrateek Dodda proposed airship/shipyard master: Implement Security Context for Airflow_Worker  https://review.opendev.org/69222405:21
*** evrardjp has quit IRC05:34
*** evrardjp has joined #airshipit05:34
*** pgaxatte has joined #airshipit07:23
*** dpawlik has joined #airshipit07:38
*** roman_g has quit IRC08:18
openstackgerritDmitry Ukov proposed airship/airshipctl master: Refactor document Bundle interface  https://review.opendev.org/70813208:38
openstackgerritDmitry Ukov proposed airship/airshipctl master: Add filtering methods for document bundle  https://review.opendev.org/70412309:27
openstackgerritDmitry Ukov proposed airship/airshipctl master: Add expression based filter for document bundle  https://review.opendev.org/70501609:27
openstackgerritDmitry Ukov proposed airship/airshipctl master: Implement document rendering sub-command  https://review.opendev.org/69865809:27
openstackgerritDmitry Ukov proposed airship/airshipctl master: Refactor document Bundle interface  https://review.opendev.org/70813209:29
*** georgk has quit IRC10:34
*** rihabb2527 has quit IRC10:34
*** georgk has joined #airshipit10:35
*** rihabb2527 has joined #airshipit10:35
*** rihabb2527 has quit IRC12:29
*** georgk has quit IRC12:29
*** rihabb2527 has joined #airshipit12:31
*** georgk has joined #airshipit12:31
openstackgerritJagan Mohan Kavva proposed airship/maas master: Enable Docker default AppArmor profile to maas  https://review.opendev.org/70513612:41
*** SRao has joined #airshipit13:59
*** nishantkr has joined #airshipit14:03
*** zogger has joined #airshipit14:04
openstackgerritJagan Mohan Kavva proposed airship/maas master: Enable Docker default AppArmor profile to maas  https://review.opendev.org/70513614:06
*** dwalt has joined #airshipit14:06
*** aaronsheffield has joined #airshipit14:07
openstackgerritJagan Mohan Kavva proposed airship/maas master: Enable Docker default AppArmor profile to maas  https://review.opendev.org/70513614:08
*** ab2434_ has joined #airshipit14:36
*** souradage has joined #airshipit14:45
*** michael-beaver has joined #airshipit14:59
ian-pittwood#startmeeting15:00
openstackian-pittwood: Error: A meeting name is required, e.g., '#startmeeting Marketing Committee'15:00
ian-pittwood#startmeeting airship15:00
openstackMeeting started Tue Feb 18 15:00:34 2020 UTC and is due to finish in 60 minutes.  The chair is ian-pittwood. Information about MeetBot at http://wiki.debian.org/MeetBot.15:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
*** openstack changes topic to " (Meeting topic: airship)"15:00
openstackThe meeting name has been set to 'airship'15:00
ian-pittwoodThere we go, still learning the ropes. Hello, everyone!15:00
ian-pittwood#topic rollcall15:01
*** openstack changes topic to "rollcall (Meeting topic: airship)"15:01
mattmceueno/15:01
mattmceuenGM ian-pittwood!15:01
ian-pittwoodHere's today's agenda https://etherpad.openstack.org/p/airship-meeting-2020-02-1815:01
dwalto/15:01
*** ashferg has joined #airshipit15:02
aaronsheffieldo/15:02
ian-pittwoodLooks pretty empty minus the review requests so please feel free to add anything for discussion today!15:02
michael-beavero/15:02
*** ab2434_ has quit IRC15:02
ian-pittwoodI'll wait a couple minutes and then we'll start15:02
*** roman_g has joined #airshipit15:02
souradageo/15:03
nishantkro/15:03
*** LoicL35 has joined #airshipit15:03
alexanderhugheso/15:03
ian-pittwoodAlright, let's get things started.15:04
ian-pittwood#topic Reminder: core team nominations15:04
*** openstack changes topic to "Reminder: core team nominations (Meeting topic: airship)"15:04
roman_go/15:04
ian-pittwoodLooks like this one is from mattmceuen15:04
jamesguo/15:04
mattmceuenHey, not a big one:15:05
mattmceuenJust an occasional reminder around our core reviewer nomination process15:05
*** madhumati has joined #airshipit15:05
mattmceuenWe have a number of different corners of the airship community working on different things -- not exactly 1:1 with projects, but there are definitely different teams doing different things15:06
mattmceuenan important responsibility of the core reviewers on a project to be aware of the status/health of the core reviewer team they're a part of15:06
*** siraj_ has joined #airshipit15:07
mattmceuenif the team doesn't have enough cores -- then please keep your eye out for potential cores and help groom them to readiness15:07
mattmceuenif there's someone qualified and ready -- any member of that project's core team can nominate them on the mailing list15:07
*** pramchan has joined #airshipit15:08
ian-pittwoodWhat are the official rules again on voting for that? Do all the cores specific to that project need to approve or is it a certain percentage of cores?15:08
mattmceuenif a core member has "moved on" and is no longer active in the community -- then there are some friendly conventions around proposing de-core-reviewering15:08
mattmceuenI just double-checked our governance, and don't see anything formal written down (that's what I thought)15:09
mattmceuenthe convention is that there's a one-week period after the core nomination goes out (by an existing core)15:10
mattmceuenand then at the end of the week, +1s / -1s are tallied up15:10
mattmceuenthe only other thing I have is a reminder of my personal guidelines for core reviewer selection -- two checkboxes in my mind:15:11
mattmceuen1. a history of meaningful reviews in the project (meaningful meaning of course, textual feedback that's helpful, as opposed to flyby +1s/-1s on the other end)15:11
mattmceuen2. meaningful, quality, non-trivial contributions to that project.  This person will help steer the project, so it's important they have a strong understanding of it!15:12
mattmceuenThere's no set number of cores for a team -- active projects benefit from a bit more; mature projects don't necessarily need as many15:12
mattmceuenThat's all I had on this - any questions/thoughts?15:13
roman_gstackalitycs provides stats on voting, but lacks stats on comments15:13
roman_gStas if I'm not mistaken submitted a patch to count comments with +0 vote15:14
roman_gNot yet merged, afaik.15:14
roman_gOr me.. I don't remember )15:14
mattmceuenOh that's interesting - hadn't seen that one15:14
mattmceuen:)15:14
ian-pittwoodI think the comments are a little too subjective for that kind of tracking, but I'm not opposed to more statistics15:14
roman_gAnyway. Good way to see who is doing reviews at all.15:14
ian-pittwoodThat's true15:14
pramchanDoes this refer to Airship 1.0 , 2.0, metal3.io, Ironic all or some select ones?15:15
roman_gAnd then filter those who do meaningful reviews15:15
mattmceuenThe most important reason for the "current cores" on a project to nominate "new cores" is that they're the team that's already very familiar with someone's work15:15
roman_gpramchan: Airship 1.0, 2.015:15
pramchanOK thx15:15
mattmceuenso at that granularity, if you have to look them up in stackalytics, it's probably just to validate your existing gut check15:15
roman_gTrue.15:16
ian-pittwoodAny further questions/comments for mattmceuen?15:16
pramchanIs there any link to Treasuremap 2.0?15:16
pramchanAre all docs submitted there in for 2.0?15:16
mattmceuenhttps://opendev.org/airship/treasuremap/src/branch/v215:17
pramchanThanks15:17
ian-pittwoodAlright we're going to move back into announcements since I see something there now15:17
mattmceuenThe goal is to get YAML manifests into the v2 branch of treasuremap (although we're cutting our teeth on the airshipctl project in that area)15:17
ian-pittwood#topic 16GB RAM and 32GB RAM flavors are available for Airship 2.0 initiative (as of now only for jobs in airshipctl repo)15:18
*** openstack changes topic to "16GB RAM and 32GB RAM flavors are available for Airship 2.0 initiative (as of now only for jobs in airshipctl repo) (Meeting topic: airship)"15:18
mattmceuenFor documentation proper, it should be targeted to the airship/docs project for both 1.0 and 2.0 docs15:18
mattmceuen(as a rule of thumb at least)15:18
ian-pittwoodmattmceuen this one is yours15:18
roman_gmine.15:18
roman_git's just an announcement15:18
ian-pittwoodoh my gosh you're the same color on my etherpad15:18
ian-pittwoodmy bad15:18
ian-pittwood#link https://opendev.org/airship/airshipctl/src/branch/master/zuul.d/nodesets.yaml#L18-L4415:19
mattmceuenhahaha that confused the heck out of me -- was like, it looks like I added it but sure don't remember15:19
ian-pittwoodTake it away roman_g15:19
roman_gWell, just an announcement that we can now use this two new flavors and a new nodeset consisting of 1x32GB + 3x16GB RAM VMs. Hardware virtualization is enabled.15:20
portdirectroman_g: are these only for airship jobs atm?15:20
mattmceuenthat's awesome15:20
roman_gKostiantyn Kalynovsky is informed (he is primary user, atm).15:20
portdirectOr projects that ask nicely?15:20
roman_gportdirect: no technical restriction to be used in other projects. But other projects don't really know this flavors exist.15:21
roman_gportdirect: want to utilize for OSH? ;)15:21
portdirectYes15:22
roman_gportdirect: technically you can. Ericsson agreed to share resources for all projects.15:23
mattmceuenbut you set yourself up portdirect, still need to ask nicely :p15:23
roman_gI.e. practically you can, as well.15:23
ian-pittwoodAny more discussion on this announcement?15:24
roman_gportdirect: also, running some other jos on this new nodesets would provide with early feedback in case something fails on provider's side15:25
ian-pittwoodAlright seems like we're good to move forward.15:26
mattmceuenthanks for working with infra team to get us this, roman_g15:26
ian-pittwoodYeah thank you15:26
ian-pittwood#topic Lots of issues to peruse in https://github.com/airshipit/airshipctl/issues15:26
*** openstack changes topic to "Lots of issues to peruse in https://github.com/airshipit/airshipctl/issues (Meeting topic: airship)"15:26
roman_gYou are most welcome.15:26
ian-pittwoodNow back to mattmceuen15:26
mattmceuenoh yeah - this is even quicker15:26
ian-pittwood#link https://github.com/airshipit/airshipctl/issues15:27
mattmceuenas has been brought up a few times, dwalt, howell, Rodolfo, many other folks have been defining more scope in our github issue backlog15:27
mattmceuenso just wanted to add a reminder so interested folks could take a look15:27
mattmceuenthat's all ian-pittwood15:27
ian-pittwoodAlright, cool.15:28
ian-pittwoodNothing more on the agenda so let's move on to roundtable15:28
ian-pittwood#topic roundtable15:28
*** openstack changes topic to "roundtable (Meeting topic: airship)"15:28
dwaltI have an item. With regards to Zuul jobs, do we have a preference for ansible modules or shell commands?15:29
ian-pittwoodThe one benefit I think with shell commands is that they're generally usable for devs locally too15:30
alexanderhughesJust a quick note about reviews request coming up. Airship development isn’t limited to just airshipctl. There’s plenty of metal3 issues I added to help get Prakash’s team more eyes on their patches15:30
dwaltian-pitwood: these aren't shell scripts, just in-line shell commands15:30
mattmceuenGood call alexanderhughes, can you please drop a link to that backlog too?  Is it metal3 or some airship tracking backlog?15:31
dwaltthey are embedded since we are taking the approach that devs should be able to run our playbooks locally15:31
ian-pittwoodDo we have a couple examples dwalt? For the less Zuul savvy among us (me)15:31
pramchanso you want me to add those #PRs to tomorrows meeting?15:31
dwaltThis particular playbook just publishes the airshipctl image to quay, so it wouldn't be used by developers. I'm mostly speaking about the most general case15:31
mattmceuenOne thing to factor in is, a lot of the ansible playbooks themselves are being designed for use on a dev machine, I believe, right?15:32
roman_gdwalt: as long as we use it only for zuul - it doesn't matter. Once we try to reuse those playbooks for deployment & management (e.g. Ansible Tower or alternatives), then it makes sense to use native modules, as it usually provides with better idempotency.15:32
alexanderhughespramchan: they’re in today’s agenda. We can also discuss tomorrow if needed. In mean time can you link the backlog you’re working out of? Is it from old JIRA board or in github issues etc15:32
pramchanOK like https://github.com/metal3-io/metal3-docs/pull/6015:32
roman_gdwalt: in your particular pathc set - it's fine to use shell commands.15:32
roman_g*patch set15:33
ian-pittwoodpramchan: I'll link all the review requests after this15:33
dwaltportdirect: was your comment in reference to all playbooks, or just within the playbook I wrote?15:33
pramchanI need some education on Jira to github transition, is it what Airship Blog shows or any other link I can use to undertsnad that?15:34
portdirectdwalt: just that one, as I've not looked extensively through others15:35
portdirectBut in general it's best to stick to one approach or the other15:35
mattmceuenfor airshipctl in particular -- since it's already a straightforward CLI, I wonder if adding wrapper shell scripts would be overkill?15:35
*** madhumati has quit IRC15:35
portdirectNo15:36
mattmceuentesting often goes in the shell scripts; that's valuable stuff we want to retain on one place or another15:36
portdirectAs it makes it much easier to consume outside of the playbook15:36
*** pgaxatte has quit IRC15:36
ian-pittwoodpramchan: most of the projects are just switching to GitHub Issues. for airshipctl the link is https://github.com/airshipit/airshipctl/issues15:36
ian-pittwoodmost issues that looked active should be ported over already15:37
roman_ghttps://review.opendev.org/#/c/707478/ patch set in question, shell vs. ansible native module15:37
mattmceuenpramchan:  here's an intro guide to github issues in general: https://guides.github.com/features/issues/15:37
alexanderhughespramchan: we can discuss more at flight plan tomorrow, but for now all that’s been migrated is airshipctl linked in the blog. If there’s items getting worked out of JIRA still we need to discuss tomorrow and make sure they get migrated as well as story requirements may have changed.15:37
pramchanSo do  I need to add all Jira issues which I deal in to trasfer to link above?15:38
ian-pittwoodPersonally, I'm fine with working through what's left in Jira and just not making any new issues in there15:39
pramchanOK thaks appreciate that15:39
ian-pittwoodBut I think it's something to bring up in flight plan as alexanderhughes said15:39
mattmceuenok I think we got off track15:39
mattmceuenback to scripts vs ansibles:15:39
dwaltSorry, stepped away for a second. I can translate to the shell wrapper if that's appropriate for image publishing. This seems like a specialized procedure that we don't want devs to run, though15:39
dwaltAnother approach: it looks like the airship-images repo job has merged. We could genericize that and use it in airshipctl15:40
mattmceuenI guess first thing, yeah Pete just to make sure I understand the context:  are you thinking in terms of e.g. tagging images for downstream build/consume?15:40
mattmceuenportdirect^15:40
pramchanDoes Script iclude golang too or you just refer to shell?15:40
mattmceuenjust shell vs ansible15:40
portdirectOh man, you guys hate on the guy in a car ;)15:40
pramchanok15:40
mattmceuenlol15:40
portdirectBut yes15:41
mattmceuenthat makes sense15:41
portdirectWe are installing multiple packages15:41
*** souradage has quit IRC15:41
portdirectJust to write out a tiny config file15:41
portdirectEither template it15:41
portdirectOr use docker cli to generate it15:41
portdirectOr if you want to use docker module15:41
portdirectUse that everywhere15:41
portdirectEither is fine, its just a mix atm that gives us the worst of both worlds15:42
roman_gwhich config file?15:42
portdirectDocker auth file15:42
roman_gah15:42
portdirectMake sense?15:43
roman_gkinda. yes. for me.15:43
dwaltYep. Thanks for the explanation portdirect15:44
roman_gThanks.15:44
mattmceuensounds good to me15:44
ian-pittwoodAlright, anything else for roundtable?15:44
dwaltUnless anyone is strongly opposed, Ill give the docker module a try for this specialized job15:44
ian-pittwood+115:44
roman_g+1 to this15:44
ian-pittwood#topic reviews15:45
*** openstack changes topic to "reviews (Meeting topic: airship)"15:45
ian-pittwood#link https://review.opendev.org/#/c/69087015:45
ian-pittwood#link https://github.com/digambar15/metal3-docs/commit/023b6eb772d20162bf0712e6debd27dcf92325fb15:45
ian-pittwood#link https://github.com/metal3-io/metal3-docs/pull/6415:45
ian-pittwood#link https://github.com/metal3-io/metal3-docs/pull/6015:46
ian-pittwood#link https://github.com/metal3-io/metal3-docs/pull/6315:46
ian-pittwood#link https://github.com/metal3-io/metal3-docs/pull/6615:46
openstackgerritSirajudeen proposed airship/airshipctl master: [#22-WIP] - use table test for set context tests  https://review.opendev.org/70776815:46
ian-pittwood#link https://github.com/metal3-io/baremetal-operator/pull/40715:46
ian-pittwood#link https://github.com/metal3-io/baremetal-operator/issues/35115:46
ian-pittwood#link https://github.com/metal3-io/baremetal-operator/pull/29215:46
ian-pittwoodAlright I believe that's the list15:46
ian-pittwoodThere's further details in the agenda for each of these links15:47
ian-pittwoodDoes anyone have any other review requests or any last minute topics?15:47
ian-pittwoodOk, well with that I think we're all done here. Thanks for your time, everyone!15:49
ian-pittwood#endmeeting15:49
*** openstack changes topic to "https://opendev.org/airship || https://wiki.openstack.org/wiki/Airship || https://review.opendev.org/#/q/projects:airship+status:open+NOT+label:Verified%253D-1+NOT+label:Workflow%253D-1+NOT+message:DNM+NOT+message:WIP"15:49
openstackMeeting ended Tue Feb 18 15:49:27 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:49
openstackMinutes:        http://eavesdrop.openstack.org/meetings/airship/2020/airship.2020-02-18-15.00.html15:49
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/airship/2020/airship.2020-02-18-15.00.txt15:49
openstackLog:            http://eavesdrop.openstack.org/meetings/airship/2020/airship.2020-02-18-15.00.log.html15:49
alexanderhughesThanks all15:49
dwaltThanks!15:49
roman_gThanks to everyone.15:51
*** SRao has quit IRC16:06
*** LoicL35 has quit IRC16:17
openstackgerritSirajudeen proposed airship/airshipctl master: [#22] - using table driven test  https://review.opendev.org/70843816:18
openstackgerritSirajudeen proposed airship/airshipctl master: [#22] - use table test for set context tests  https://review.opendev.org/70776816:29
openstackgerritAndrew Schiefelbein proposed airship/airshipui master: [14] this adds in all command line options from octant.  https://review.opendev.org/70771416:33
openstackgerritMerged airship/airshipctl master: Use RemoveAll to cleanup test directories  https://review.opendev.org/70826816:52
openstackgerritSirajudeen proposed airship/airshipctl master: [#22] - use table driven test for set auth  https://review.opendev.org/70843817:01
*** pramchan has quit IRC17:03
openstackgerritDan Crank proposed airship/divingbell master: [ad-hoc] Fix test case exit conditions  https://review.opendev.org/70388017:17
openstackgerritIan Howell proposed airship/airshipctl master: WIP [#50] Cleanup temp files from unit tests  https://review.opendev.org/70825117:24
openstackgerritIan Howell proposed airship/airshipctl master: WIP [#50] Cleanup temp files from unit tests  https://review.opendev.org/70825117:32
*** evrardjp has quit IRC17:34
*** evrardjp has joined #airshipit17:34
openstackgerritIan Howell proposed airship/airshipctl master: [#50] Clean up temp files from unit tests  https://review.opendev.org/70825117:58
openstackgerritMerged airship/promenade master:  Add  Docker default AppArmor profile to promenade  https://review.opendev.org/70434618:08
openstackgerritMerged airship/promenade master: [proxy] robustness tweak for liveness probe  https://review.opendev.org/70082518:08
openstackgerritDrew Walters proposed airship/airshipctl master: [#51] Set context unless current-context is false  https://review.opendev.org/70847918:28
openstackgerritDrew Walters proposed airship/airshipctl master: [#51] Set context when no flags are provided  https://review.opendev.org/70847918:29
openstackgerritMerged airship/airshipctl master: [#41] Update `make godoc` target  https://review.opendev.org/70743118:32
dwalthowell: Just saw your comments on airshipctl #51. The change above is a band-aid fix for now if there's value to putting it in. I'll start working on switching the --current-context flag to --use-context18:35
howelldwalt: ah, my comment may have been a bit unclear. By use-context, I meant a whole new subcommand as opposed to a new flag18:36
howellis #51 ready for review?18:37
dwaltYeah, #51 is ready for reviews18:39
dwaltI see what you mean. I'll start looking into that today18:39
openstackgerritIan Howell proposed airship/airshipctl master: [#42] Refactor flag operations to be uniform  https://review.opendev.org/70663018:52
openstackgerritDrew Walters proposed airship/airshipctl master: [#51] Remove current-context flag  https://review.opendev.org/70847919:08
openstackgerritMerged airship/airshipctl master: [#6] Add config init subcommand  https://review.opendev.org/70725219:10
openstackgerritAndrew Schiefelbein proposed airship/airshipui master: [14] this adds in all command line options from octant.  https://review.opendev.org/70771419:33
openstackgerritAhmad Mahmoudi proposed airship/shipyard master: WIP: (fix) Addressing file not found issue  https://review.opendev.org/70770919:37
openstackgerritMerged airship/airshipctl master: Add render sub-command frame  https://review.opendev.org/69732719:46
openstackgerritDan Crank proposed airship/divingbell master: [ad-hoc] Fix test case exit conditions  https://review.opendev.org/70388019:53
openstackgerritKostyantyn Kalynovskyi proposed airship/images master: [WIP] Enabling post job to push images  https://review.opendev.org/70850320:10
openstackgerritIan Howell proposed airship/airshipctl master: [#50] Clean up temp files from unit tests  https://review.opendev.org/70825120:15
openstackgerritIan Howell proposed airship/airshipctl master: [#42] Refactor flag operations to be uniform  https://review.opendev.org/70663020:20
openstackgerritDan Crank proposed airship/divingbell master: [ad-hoc] Fix test case exit conditions  https://review.opendev.org/70388020:28
openstackgerritAhmad Mahmoudi proposed airship/shipyard master: WIP: (fix) Addressing file not found issue  https://review.opendev.org/70770920:44
*** michael-beaver has quit IRC20:49
openstackgerritAhmad Mahmoudi proposed airship/shipyard master: WIP: (fix) Addressing file not found issue  https://review.opendev.org/70770921:10
*** high-rez has quit IRC21:21
openstackgerritStas Egorov proposed airship/airshipctl master: [WIP]: new CLI for getting module config  https://review.opendev.org/70728421:52
*** aaronsheffield has quit IRC22:03
openstackgerritPrateek Dodda proposed airship/shipyard master: [WIP] Implement Security Context for Airflow_Worker  https://review.opendev.org/69222422:08
*** timClicks has joined #airshipit22:08
openstackgerritJordan Jensen proposed airship/airshipui master: [14] Add command line options from octant  https://review.opendev.org/70771422:39
openstackgerritPrateek Dodda proposed airship/shipyard master: [WIP] Implement Security Context for Airflow_Worker  https://review.opendev.org/69222422:45
openstackgerritMerged airship/airshipui master: [14] Add command line options from octant  https://review.opendev.org/70771423:06
openstackgerritMerged airship/divingbell master: Add Docker default AppArmor profile to divingbell  https://review.opendev.org/70546923:10
openstackgerritPhil Sphicas proposed airship/promenade master: etcd chart: anchor pre_stop remove bashism  https://review.opendev.org/70853323:15
openstackgerritJordan Kramer proposed airship/armada master: [WIP] Added Password Rotation Gate  https://review.opendev.org/70853823:51
openstackgerritKostyantyn Kalynovskyi proposed airship/airshipctl master: Change document labeling  https://review.opendev.org/70853923:55

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!