| *** sthussey has quit IRC | 00:06 | |
| *** kranthikirang has joined #airshipit | 00:09 | |
| *** kranthikirang has quit IRC | 00:13 | |
| *** lemko has quit IRC | 01:04 | |
| *** cheng1 has joined #airshipit | 01:36 | |
| *** kranthikirang has joined #airshipit | 01:38 | |
| *** kranthikirang has quit IRC | 01:42 | |
| *** kranthikirang has joined #airshipit | 03:26 | |
| *** kranthikirang has quit IRC | 03:31 | |
| *** roman_g has quit IRC | 03:51 | |
| *** cheng1 has quit IRC | 04:13 | |
| *** cheng1 has joined #airshipit | 04:52 | |
| *** cheng1 has quit IRC | 05:11 | |
| *** kranthikirang has joined #airshipit | 05:14 | |
| *** kranthikirang has quit IRC | 05:18 | |
| *** cheng1 has joined #airshipit | 05:26 | |
| *** jamesgu has quit IRC | 06:05 | |
| *** lemko has joined #airshipit | 06:15 | |
| openstackgerrit | Dmitrii Kabanov proposed openstack/airship-divingbell master: Add possibility to add repository and GPG key https://review.openstack.org/644495 | 07:08 |
|---|---|---|
| *** roman_g has joined #airshipit | 08:17 | |
| *** cheng1_ has joined #airshipit | 08:17 | |
| *** cheng1 has quit IRC | 08:20 | |
| *** kukacz has quit IRC | 09:11 | |
| *** jamesgu__ has quit IRC | 09:11 | |
| *** jamesgu__ has joined #airshipit | 09:12 | |
| *** kukacz has joined #airshipit | 09:28 | |
| *** cheng1_ has quit IRC | 09:28 | |
| *** kukacz has quit IRC | 09:29 | |
| *** kukacz has joined #airshipit | 09:30 | |
| *** kukacz has quit IRC | 09:39 | |
| *** kukacz has joined #airshipit | 09:39 | |
| *** cheng1_ has joined #airshipit | 09:56 | |
| *** roman_g has quit IRC | 10:49 | |
| *** roman_g has joined #airshipit | 10:49 | |
| *** mbeierl has quit IRC | 11:50 | |
| *** roman_g has quit IRC | 12:36 | |
| *** roman_g has joined #airshipit | 12:42 | |
| *** roman_g has quit IRC | 12:44 | |
| *** roman_g has joined #airshipit | 12:52 | |
| *** levmorgan has joined #airshipit | 12:54 | |
| *** mbeierl has joined #airshipit | 13:05 | |
| *** aaronsheffield has joined #airshipit | 13:16 | |
| *** levmorgan has quit IRC | 13:21 | |
| *** cheng1_ has quit IRC | 13:26 | |
| *** sthussey has joined #airshipit | 13:26 | |
| *** kranthikirang has joined #airshipit | 13:30 | |
| *** ab2434_ has joined #airshipit | 13:46 | |
| *** michael-beaver has joined #airshipit | 14:20 | |
| *** michaelbeaver has joined #airshipit | 14:23 | |
| *** roman_g has quit IRC | 14:23 | |
| *** michael-beaver has quit IRC | 14:26 | |
| *** jamesgu has joined #airshipit | 14:26 | |
| sthussey | @dwalt do you have the link for today's meeting agenda? | 14:34 |
| dwalt | sthussey: yep! https://etherpad.openstack.org/p/airship-meeting-2019-03-19 | 14:35 |
| *** arunkant has joined #airshipit | 14:41 | |
| *** levmorgan has joined #airshipit | 14:55 | |
| *** roman_g has joined #airshipit | 14:56 | |
| openstackgerrit | Merged openstack/airship-maas master: Run maas-rack and maas-region containers as non-privileged https://review.openstack.org/633269 | 15:05 |
| openstackgerrit | Merged openstack/airship-in-a-bottle master: Mount ~/.ssh to allow Pegleg to clone repos https://review.openstack.org/644444 | 15:07 |
| openstackgerrit | Merged openstack/airship-promenade master: UCP: Enable Audit Logging feature gate in K8s https://review.openstack.org/635568 | 15:15 |
| *** michaelbeaver has quit IRC | 15:48 | |
| *** michaelbeaver has joined #airshipit | 15:49 | |
| mattmceuen | meeting time in 10 min! Agenda: https://etherpad.openstack.org/p/airship-meeting-2019-03-19 | 15:50 |
| mattmceuen | #startmeeting airship | 16:00 |
| openstack | Meeting started Tue Mar 19 16:00:22 2019 UTC and is due to finish in 60 minutes. The chair is mattmceuen. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:00 |
| *** openstack changes topic to " (Meeting topic: airship)" | 16:00 | |
| openstack | The meeting name has been set to 'airship' | 16:00 |
| mattmceuen | #topic Rollcall | 16:00 |
| mattmceuen | GM everyone! | 16:00 |
| *** openstack changes topic to "Rollcall (Meeting topic: airship)" | 16:00 | |
| michaelbeaver | o/ GM | 16:00 |
| mattmceuen | agenda for today: https://etherpad.openstack.org/p/airship-meeting-2019-03-19 | 16:00 |
| levmorgan | o/ sup! | 16:00 |
| evgenyl | Hi! | 16:00 |
| mattmceuen | Let's give it a few mins for folks to join | 16:00 |
| roman_g | o/ | 16:01 |
| dwalt | o/ | 16:01 |
| *** michaelbeaver has quit IRC | 16:01 | |
| mattmceuen | Please add anything you'd like to discuss today to the etherpad | 16:01 |
| seaneagan | o/ | 16:01 |
| *** michael-beaver has joined #airshipit | 16:01 | |
| mattmceuen | #topic Multi-OS image builds | 16:02 |
| *** openstack changes topic to "Multi-OS image builds (Meeting topic: airship)" | 16:02 | |
| roman_g | #action roman_g to move discussion to the mailing list to reach wider audience | 16:02 |
| roman_g | There are controversial opinions, and we possibly need to involve RH/CentOS folks (if any) | 16:03 |
| mattmceuen | Thanks for discussing this last week when I was out | 16:03 |
| mattmceuen | The spec is now merged, so that's great. | 16:03 |
| dwalt | is the spec update still open? roman_g | 16:03 |
| roman_g | sure, it is | 16:03 |
| mattmceuen | Roman has also put in a PS to update the spec -- thanks for bringing it to the ML Roman, I agree that's a good idea | 16:03 |
| mattmceuen | Anything else we want to touch on for this topic here now? | 16:04 |
| mattmceuen | (otherwise we can just use the ML) | 16:04 |
| jamesgu__ | a quick question | 16:04 |
| mattmceuen | sure | 16:04 |
| jamesgu__ | how long do we keep those images on quay.io, with the commit id as the tag? | 16:05 |
| *** sgrasley has quit IRC | 16:05 | |
| roman_g | forever | 16:05 |
| jamesgu__ | I like that :-) | 16:05 |
| sthussey | I believe quay.io has a 3 month TTL | 16:05 |
| sthussey | if an image isn't pulled for 3 months, they remove it | 16:05 |
| *** sgrasley has joined #airshipit | 16:05 | |
| mattmceuen | aha | 16:05 |
| mattmceuen | good compromise that | 16:05 |
| roman_g | quay.io is owned by CoreOS, CoreOS is owned by Red Hat. As long as they sponsor quay.io - we would be there. | 16:05 |
| mattmceuen | does quay impose any kind of space quota? | 16:06 |
| jamesgu__ | yeah we literally going to double or triple if the number of images | 16:06 |
| sthussey | not that I know of | 16:07 |
| mattmceuen | cool | 16:07 |
| sthussey | Looks like it is longer than 3 months | 16:07 |
| sthussey | The oldest Promenade image there is 9 months old | 16:07 |
| mattmceuen | maybe someone's pulling it still? Can you tell? | 16:07 |
| mattmceuen | or does "9 mo old" mean "since last pull" | 16:07 |
| sthussey | No, 9 months since tag creation | 16:08 |
| sthussey | I can't imagine folks are still using it | 16:08 |
| sthussey | Looks like most of the repos have the oldest tag at 9 months | 16:08 |
| arunkant | related question...how frequently images are built and published on quay.io . | 16:09 |
| sthussey | On every merge | 16:09 |
| mattmceuen | it's part of the merge gating | 16:09 |
| arunkant | Okay..so every merge of that repo or only when there is image related change? | 16:10 |
| sthussey | At this point I believe every merge. There may be some filtering for docs | 16:11 |
| sthussey | I'd have to look | 16:11 |
| mattmceuen | https://github.com/openstack/airship-armada/blob/master/.zuul.yaml#L31-L33 | 16:12 |
| arunkant | Okay..thanks..so looks like commit-id will keep changing then as we only have one version of it | 16:12 |
| mattmceuen | Every merge | 16:12 |
| mattmceuen | Ok - one more coffee sip's worth of time before moving on | 16:12 |
| mattmceuen | #topic 1.0 release status | 16:13 |
| *** openstack changes topic to "1.0 release status (Meeting topic: airship)" | 16:13 | |
| mattmceuen | So it's been quiet on this front, but gears are turning | 16:13 |
| mattmceuen | alanmeadows was getting this out to the community, but then he had a happy addition to his family and is out for a bit | 16:14 |
| mattmceuen | I should clarify: this ^ is specifically about governance (one of the subpoints in the agenda) | 16:14 |
| mattmceuen | We'll hopefully have some draft governance doc material out soon which we can discuss and iterate on | 16:15 |
| mattmceuen | Re: 1.0 release status proper -- | 16:15 |
| mattmceuen | The biggest outstanding item for 1.0 is Ironic integration | 16:16 |
| mattmceuen | And there's been some pivoting around what Airship's Ironic integration will look like | 16:16 |
| mattmceuen | Short version -- since K8s cluster API integration with Ironic is going to happen anyway, we'd like to throw our weight behind that and help it succeed | 16:16 |
| mattmceuen | Rather than a Drydock->Ironic native integration (which is what the spec calls for now) | 16:17 |
| mattmceuen | This extends out the Ironic integration with Airship a bit, but it's definitely the right long-term direction as it will be aligned to the k8s community and the Ironic community | 16:17 |
| mattmceuen | So by 1.0 we're planning to have progress and alignment on Ironic integration, but it likely won't be complete or prod ready, as it'll take time to build something reusable by Airship and others | 16:18 |
| mattmceuen | my fingers are tired, any addn'l thoughts / questions? | 16:19 |
| mattmceuen | alright - moving on | 16:19 |
| mattmceuen | #topic Participation in Season of Docs | 16:20 |
| *** openstack changes topic to "Participation in Season of Docs (Meeting topic: airship)" | 16:20 | |
| mattmceuen | #action mattmceuen to look into this deeper | 16:20 |
| mattmceuen | Thanks romang for bringing this up, I think it's potentially a really good thing (and worst case, it can't hurt to try it out) | 16:20 |
| mattmceuen | I saw the convo last week, let me spend some time this week digging in if you don't mind | 16:21 |
| roman_g | no problem. thank you | 16:22 |
| mattmceuen | #topic Participation in Summer of Code 2020 | 16:22 |
| *** openstack changes topic to "Participation in Summer of Code 2020 (Meeting topic: airship)" | 16:22 | |
| mattmceuen | I suppose this falls into the same boat huh :) | 16:22 |
| roman_g | yep | 16:22 |
| mattmceuen | #action mattmceuen to investigate Airship participation in summer of code 2020 | 16:22 |
| roman_g | at least someone accepted the meeting | 16:23 |
| mattmceuen | #topic AIAB is still broken (from mailing list) | 16:23 |
| *** openstack changes topic to "AIAB is still broken (from mailing list) (Meeting topic: airship)" | 16:23 | |
| roman_g | you will have to find time and discuss ) | 16:23 |
| mattmceuen | yes! | 16:23 |
| mattmceuen | So AIAB - thanks to everyone who's put in patches to get this working again | 16:23 |
| roman_g | Didn't try it, but gates show that it's still broken | 16:24 |
| evgenyl | Yeah, here is an example https://review.openstack.org/#/c/643430/ | 16:24 |
| mattmceuen | Are the logs for that getting published publicly yet evgenyl? I don't see them in that PS | 16:25 |
| openstackgerrit | Kaspars Skels proposed openstack/airship-in-a-bottle master: [bug] Fix missing sudo for setting up virtmgr SSH keys https://review.openstack.org/644610 | 16:25 |
| openstackgerrit | Kaspars Skels proposed openstack/airship-in-a-bottle master: [bug] Fix missing sudo for setting up virtmgr SSH keys https://review.openstack.org/644610 | 16:26 |
| evgenyl | They are being published e.g. `https://artifacts.atlantafoundry.com/artifactory/cicd/logs/integration/airship/airship-in-a-bottle/100 : FAILURE` | 16:26 |
| mattmceuen | whansinn had referenced a commit that worked; is that a recent commit (meaning he may be running it differently than the gate) or is that an older commit before something got broken? | 16:27 |
| mattmceuen | that's awesome - thanks evgenyl | 16:27 |
| roman_g | mattmceuen: that's very old commit | 16:27 |
| roman_g | Feb 21, 2019 | 16:28 |
| sthussey | I mean, that isn't that old | 16:28 |
| openstackgerrit | Evgeniy L proposed openstack/airship-in-a-bottle master: [TEST] Do not merge https://review.openstack.org/643430 | 16:29 |
| roman_g | >10 commits ago | 16:29 |
| mattmceuen | looks like the mariadb ingress was crashlooping, but cursory look at the logs didn't show anything that was a clear smoking gun to me | 16:30 |
| mattmceuen | was looking at the wrong one | 16:31 |
| mattmceuen | I0315 00:14:53.238599 6 controller.go:177] Configuration changes detected, backend reload required. | 16:31 |
| mattmceuen | E0315 00:14:53.239610 6 controller.go:189] Unexpected failure reloading the backend: | 16:31 |
| mattmceuen | template: nginx.tmpl:60:14: executing "nginx.tmpl" at <$cfg.EnableVtsStatus>: can't evaluate field EnableVtsStatus in type config.Configuration | 16:31 |
| mattmceuen | W0315 00:14:53.239646 6 queue.go:130] requeuing configmap-change, err template: nginx.tmpl:60:14: executing "nginx.tmpl" at <$cfg.EnableVtsStatus>: can't evaluate field EnableVtsStatus in type | 16:31 |
| mattmceuen | openstack mariadb ingress ^ | 16:31 |
| mattmceuen | dumb question, why do we have an ingress for mariadb? Shouldn't service be enough? | 16:32 |
| mattmceuen | Well I think we'd better take this offline and keep going | 16:33 |
| roman_g | ¯\_(ツ)_/¯ | 16:33 |
| mattmceuen | but let's try to figure this out in the channel today | 16:33 |
| roman_g | I didn't work on that yet | 16:33 |
| sthussey | I believe they need an ingress for multi-master support | 16:33 |
| mattmceuen | ok | 16:34 |
| mattmceuen | #topic Who will be able to attend the Open Infra summit? | 16:34 |
| *** openstack changes topic to "Who will be able to attend the Open Infra summit? (Meeting topic: airship)" | 16:34 | |
| mattmceuen | If you're going to the summit, please add your name to the list in the etherpad! | 16:34 |
| mattmceuen | #topic OpenStack to OpenDev transition | 16:35 |
| *** openstack changes topic to "OpenStack to OpenDev transition (Meeting topic: airship)" | 16:35 | |
| mattmceuen | This is an important one -- roman_g has been our rep with the openstack infra team, can you please give us an overview roman_g? | 16:35 |
| roman_g | Transition date is (so far) set to 4/19 | 16:35 |
| roman_g | Closest change to expect: massive change of git:// URLs to https:// (similar to https://review.openstack.org/#/c/642652/ ) | 16:35 |
| roman_g | #action: roman_g announce in mailing list | 16:35 |
| roman_g | Need to discuss & decide whether we move openstack/airship-* to airship/airship-* or openstack/airship-* to airship/* | 16:36 |
| mattmceuen | My understanding is that there will also be redirects between the old URLs and the new ones, to prevent a breaking interruption | 16:36 |
| evgenyl | Will this be an alias or they are planning to remove the old one? :) | 16:36 |
| roman_g | HTTP/HTTPS redirects would be supported fr reasonable future, of course | 16:37 |
| sthussey | it seems we should transition public references to git.airshipit.org | 16:37 |
| roman_g | Old tenant and repo will be removed | 16:37 |
| roman_g | sthussey: that is an option | 16:37 |
| mattmceuen | Agree sthussey | 16:37 |
| roman_g | but it's not only git repo, but also e.g. zuul tenant, gerrit, and may be something else | 16:38 |
| sthussey | git.airshipit.org points at gerrit I believe | 16:38 |
| roman_g | git.airshipit.org would stay as-is (UI will change to Gitea) | 16:38 |
| sthussey | I guess not | 16:39 |
| roman_g | yes, it points at Gerrit. And will point at Gitea in the future | 16:39 |
| mattmceuen | the `airship` namespace will be a really good thing. When we were added to the `openstack` namespace we used `airship-` prefixes to be a poor-man's namespacing, so like roman_g says we'll need to figure out whether to retain `airship` in our project names, or not. | 16:39 |
| mattmceuen | please mull it over and anticipate roman's email :) | 16:39 |
| roman_g | Finally. Who makes this decision? | 16:39 |
| mattmceuen | back to governance! ha | 16:40 |
| sthussey | as part of all this work, should consider if we move image hosting away from quay.io and into whatever OSF uses | 16:40 |
| roman_g | OSF -> ODF | 16:40 |
| sthussey | whatever | 16:40 |
| roman_g | Do they use any? I haven't seen anything like that. | 16:40 |
| mattmceuen | not sure | 16:41 |
| mattmceuen | I don't think there's one yet | 16:41 |
| mattmceuen | Not sure if any are planned though | 16:41 |
| sthussey | It seems like it will be the artifact publication mechanism of the future | 16:41 |
| roman_g | Who makes the decision? :) Otherwise I'll make life easy, and would select ``openstack/airship-* to airship/airship-*`` option | 16:42 |
| sthussey | So even if they aren't hosting a private registry, it may be worth exploring alignment with anyone else producing OCI images | 16:42 |
| mattmceuen | Agree. I guess the cost/benefit of ODF hosting its own registry is something that needs to be explored. | 16:42 |
| sthussey | I'd prefer dropping duplicate namespaces | 16:42 |
| sthussey | So if they are going to namespace it under an `airship` umbrella, then drop the `airship-` prefix | 16:43 |
| mattmceuen | To answer your question roman_g - I think the core team should vote on it after getting all input | 16:44 |
| roman_g | sthussey and Rodolfo are for dropping the prefix, I'm for leaving the prefix. Other votes? Is it legitimate to decide here? Should I move discussion to the ML? | 16:44 |
| mattmceuen | Hoping there will be consensus across the team after arguments are heard; if not we can split hairs about how to determine | 16:44 |
| mattmceuen | let's let it marinate on the ML first | 16:44 |
| mattmceuen | and let's share our rationales on the ML as well | 16:45 |
| sthussey | I think it is worth putting spec-like in the ML with the full picture | 16:45 |
| roman_g | OK. I have an action item on this already | 16:45 |
| mattmceuen | I don't have a strong opinion either way... yet | 16:45 |
| mattmceuen | thanks | 16:45 |
| mattmceuen | anything else before we move on guys? | 16:45 |
| roman_g | no | 16:45 |
| mattmceuen | #topic Airship approach to various OpenStack efforts that really bring no value - e.g. Python 2 to 3 effort | 16:45 |
| *** openstack changes topic to "Airship approach to various OpenStack efforts that really bring no value - e.g. Python 2 to 3 effort (Meeting topic: airship)" | 16:45 | |
| mattmceuen | Take it away sthussey | 16:45 |
| sthussey | I think the description explains it mostly | 16:46 |
| sthussey | but there have been several expeditions of instituting a formulaic code change across all of OpenStack | 16:46 |
| sthussey | The most recent was moving from using the general `json` library to some oslo variant of it | 16:46 |
| mattmceuen | For cross-project openstack initiatives -- I think this is where the delineation of "openstack proper" vs "open infra" / "open dev" will be helpful | 16:46 |
| sthussey | In some cases, I think it is silly and a waste of time, but it isn't my time so I ignore it | 16:47 |
| sthussey | In this case, it is actually detrimental because it introduces additional nodes the in dependency graph | 16:47 |
| roman_g | Have we been approached to do some work on this kind of changes? Or are we expected to comply? | 16:48 |
| sthussey | Someone else has opened the PS to do the work | 16:48 |
| mattmceuen | In general I think openstack cross-project initiatives are limited to "openstack proper" | 16:48 |
| mattmceuen | and can become "airship" cross-project initiatives too with the right discussion / rationale behind them | 16:48 |
| mattmceuen | but today, all the projects are in the same namespace so the discussion was missed | 16:49 |
| sthussey | I've resisted a hard line on this as it seems it should be a general approach agreeded upon across the airship community | 16:49 |
| mattmceuen | my thought is, "all else equal" I'm happy to adhere to openstack norms and initiatives, since we're one larger community | 16:50 |
| roman_g | can I have a link to that PS? | 16:50 |
| mattmceuen | However, "all else equal" is key, and there needs to be discussion | 16:50 |
| sthussey | @roman_g look for jacky06 as the owner | 16:50 |
| roman_g | thanks | 16:50 |
| roman_g | Are we expected to comply? | 16:50 |
| roman_g | By rules. | 16:50 |
| sthussey | I don't know. | 16:50 |
| roman_g | mattmceuen? | 16:51 |
| mattmceuen | Probably depends who you ask roman_g, but the answer is "no" formally | 16:51 |
| mattmceuen | Some openstack cross-project things could be really valuable to us - like internationalization | 16:51 |
| mattmceuen | and some are probably arguable value, and need to be discussed | 16:51 |
| mattmceuen | Let's bring them up on a case by case basis here or in the ML | 16:51 |
| sthussey | I think it is better to come up with a general viewpoint on it | 16:52 |
| roman_g | again, governance question? | 16:52 |
| sthussey | And I'm fine with "If the work standing on its own is valuable, then treat it as such" | 16:52 |
| mattmceuen | ^ that | 16:52 |
| sthussey | But not so much "This webpage so everyone should do X, so do X" | 16:53 |
| sthussey | webpage says* | 16:53 |
| mattmceuen | There should be no assumption that an openstack cross-project item impacts airship, since many of the items will be irrelevant to airship (e.g. release-schedule related things) | 16:53 |
| openstackgerrit | Merged openstack/airship-in-a-bottle master: [bug] Fix missing sudo for setting up virtmgr SSH keys https://review.openstack.org/644610 | 16:54 |
| mattmceuen | but I for one am certainly open to adoption openstack initiatives when they do stand on their own merit for airship | 16:54 |
| sthussey | good enough | 16:54 |
| * sthussey steps off his soapbox | 16:54 | |
| mattmceuen | I don't think we actually came to consensus on the json thing; let's take a look @ that offline and weigh in on the PS? | 16:55 |
| mattmceuen | thanks for bringing this up sthussey :) | 16:55 |
| roman_g | thanks. | 16:55 |
| mattmceuen | #topic External gates for AIAB are enabled (archived logs are being published) | 16:55 |
| *** openstack changes topic to "External gates for AIAB are enabled (archived logs are being published) (Meeting topic: airship)" | 16:55 | |
| mattmceuen | We touched on this already! nice! | 16:55 |
| roman_g | I also have idea to change YAML library to the one which persists comments in YAML's and order of keys | 16:55 |
| roman_g | at least for versions.conf utility | 16:56 |
| roman_g | * versions.yaml | 16:56 |
| mattmceuen | Sorry roman_g, I missing it - is that related to the AIAB publishing? | 16:57 |
| roman_g | no, that | 16:57 |
| roman_g | 's for previous topic | 16:57 |
| roman_g | json library change request | 16:57 |
| mattmceuen | ahh I see | 16:57 |
| evgenyl | So currently AIAB gates are failing, people should be able to see the logs in the comments. | 16:57 |
| mattmceuen | I'm not following the discussion between roman_g and evgenyl on the etherpad 100%; where did we land? | 16:58 |
| dwalt | evgenyl: The comments keep disappearing for me. Do you know if there is a Gerrit preference to show them? | 16:58 |
| roman_g | evgenyl: thank you! | 16:58 |
| roman_g | dwalt: probably Toggle CI button, lower left corner. | 16:59 |
| mattmceuen | yup that works for me | 16:59 |
| dwalt | perfect. Thanks! | 16:59 |
| mattmceuen | Is this a step toward becoming a third-party gate? Or is this "instead of" a third party gate? | 17:00 |
| roman_g | mattmceuen: we have logs published, but they need to be unarchived. I've pushed a PS for that | 17:00 |
| evgenyl | I completely forgot about this button, and assumed everybody could see these comments.. | 17:00 |
| mattmceuen | ahh thanks roman_g | 17:00 |
| mattmceuen | d'oh we're out of time folks | 17:00 |
| mattmceuen | A couple of patches requesting review: | 17:01 |
| mattmceuen | https://review.openstack.org/#/c/635507/ - openstack/airship-treasuremap - Add cache for results of requests to quay.io in Updater tool | 17:01 |
| mattmceuen | https://review.openstack.org/#/c/638144/ - openstack/airship-divingbell - Enhance docs rendering; update documentation | 17:01 |
| mattmceuen | Thanks very much to you all for your time and effort today, and on Airship in general | 17:01 |
| mattmceuen | have a good one | 17:01 |
| mattmceuen | #endmeeting | 17:01 |
| *** openstack changes topic to "airshipit.org || General Review Dashboard: https://review.openstack.org/#/q/project:%255Eopenstack/airship.*+status:open,n,z" | 17:01 | |
| openstack | Meeting ended Tue Mar 19 17:01:44 2019 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 17:01 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/airship/2019/airship.2019-03-19-16.00.html | 17:01 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/airship/2019/airship.2019-03-19-16.00.txt | 17:01 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/airship/2019/airship.2019-03-19-16.00.log.html | 17:01 |
| levmorgan | You too! See you next week. | 17:02 |
| *** levmorgan has quit IRC | 17:02 | |
| *** mbeierl has quit IRC | 17:10 | |
| roman_g | levmorgan quit :) | 17:10 |
| *** kaspars__ has joined #airshipit | 17:38 | |
| kaspars__ | hey there Airship folks, my test if I setup IRC correctly - feel free to have a friendly reply. | 17:40 |
| roman_g | kaspars__: nice to meet you here | 17:41 |
| mattmceuen | kaspars__ howdy! | 17:41 |
| evgenyl | kaspars__: hey :) | 17:47 |
| *** roman_g has quit IRC | 17:57 | |
| *** JosW has joined #airshipit | 18:18 | |
| *** JosW has left #airshipit | 18:18 | |
| openstackgerrit | Merged openstack/airship-divingbell master: Update helm-toolkit pin https://review.openstack.org/643057 | 18:29 |
| mattmceuen | hey dwalt, for jamesgu's PS here: https://review.openstack.org/#/c/636240/ | 18:34 |
| mattmceuen | What's the driver for making the document name end in "-global"? | 18:34 |
| mattmceuen | I understand the label ending in "-global" | 18:35 |
| mattmceuen | but for the document name seems like just leaving the document name itself agnostic to the layer lends itself better to overriding and inheritance | 18:35 |
| dwalt | I think what jamesgu has there is in-line with the existing standards | 18:36 |
| dwalt | I don't think we want the document name to end in global, right? | 18:36 |
| mattmceuen | I think that's what your story says, right? https://storyboard.openstack.org/#!/story/2004354 | 18:37 |
| dwalt | woops, that sent out of order | 18:37 |
| jamesgu | we don't have -global suffix in the document name, just the label. | 18:37 |
| dwalt | Ah, I didn't realize you were looking at the story item. I didn't intend to suggest he should do it like that, just that we should decide on the standard here | 18:37 |
| mattmceuen | ah I see | 18:38 |
| dwalt | For example, we don't use component labels for everything | 18:38 |
| dwalt | should we just remove them where they are used? | 18:38 |
| mattmceuen | what's an example of a component label that we might remove? | 18:38 |
| dwalt | There's a few in the patch. For example, memcached: https://review.openstack.org/#/c/636240/9/global/software/charts/osh/openstack-memcached/memcached.yaml | 18:39 |
| dwalt | Here is one that is merged: https://github.com/openstack/airship-treasuremap/blob/bdfcc8af409d0cbfd5a4e4bccfddf4f9bfdb06c3/global/software/charts/ucp/keystone/keystone.yaml | 18:40 |
| mattmceuen | Isn't the point of jamegu's patch to add those in though? Is there any harm in putting the hooks in? | 18:40 |
| mattmceuen | I.e., why would we want to remove them | 18:41 |
| dwalt | No harm AFAIK. It would just be for consistency. We haven't talked about this in a while, so maybe it's not as much of a concern as when we were authoring the airskiff site | 18:42 |
| openstackgerrit | Merged openstack/airship-divingbell master: Enhance docs rendering; update documentation https://review.openstack.org/638144 | 18:42 |
| dwalt | Either way, it doesn't have to happen in this patch. I was just hoping to re-surface the discussion is jamesgu was willing to address it in his patch | 18:43 |
| mattmceuen | How bout I add this to next week's meeting agenda (if we can remember the details for one whole week) | 18:43 |
| mattmceuen | Yes, agree and it's successfully re-surfaced :D I missed your story from before | 18:44 |
| jamesgu | I think I've seen many charts have both name and component labels and some only have name label, and the rest doesn't have any in the global. | 18:44 |
| dwalt | Sounds like a plan! | 18:44 |
| mattmceuen | Yeah, that bugs me too jamesgu | 18:44 |
| mattmceuen | If we were driving for consistency, I'd lean more toward including the labels everywhere | 18:44 |
| jamesgu | I agree with that | 18:44 |
| dwalt | Agree. I don't mind if it's present or not, just that it's consistent across all documents | 18:45 |
| mattmceuen | Cool - I'll give you a +2 for now, and we can get more opinions next week | 18:45 |
| mattmceuen | yup | 18:45 |
| dwalt | ++ | 18:45 |
| jamesgu | thanks mattmceuen and dwalt | 18:45 |
| evgenyl | A quick question about summit PTGs, I've checked the schedule, and I'm wondering when the sessions are going to end on Saturday? Trying to figure out what tickets to get :) | 18:47 |
| openstackgerrit | Merged openstack/airship-pegleg master: pki: Add pki_utility unit tests https://review.openstack.org/640620 | 18:47 |
| mattmceuen | I like jamesgu :) one small nit in the PS | 18:50 |
| mattmceuen | *lied | 18:50 |
| mattmceuen | evgenyl afaik sessions will formally end "end of day" (like 5pm), however I expect many folks to be cutting out a bit early to make flights | 18:51 |
| mattmceuen | and discussion always winds down a bit on the last day anyway | 18:51 |
| openstackgerrit | Anthony Bellino proposed openstack/airship-divingbell master: [WIP] Initial Ansible Daemonset https://review.openstack.org/640539 | 18:51 |
| openstackgerrit | James Gu proposed openstack/airship-treasuremap master: Add missing labels to osh charts and ucp chart groups https://review.openstack.org/636240 | 18:53 |
| openstackgerrit | Evgeniy L proposed openstack/airship-in-a-bottle master: Downgrade ingress controller for MariaDB https://review.openstack.org/644634 | 18:54 |
| evgenyl | mattmceuen: thanks! | 18:55 |
| jamesgu | mattmceuen: ugh... I should feel shamed :-) ... pushed up a new PS to fix that... didn't mean to introduce another convention :-) | 18:55 |
| mattmceuen | lol, those double dashes are gonna be catching on like wildfire jamesgu! | 18:55 |
| mattmceuen | all the rage | 18:55 |
| evgenyl | Here is a patch for AIAB that I hope would solve the problem with a failing gate https://review.openstack.org/#/c/644634/ early reviews are welcome! | 18:55 |
| mattmceuen | that's awesome - ty evgenyl | 18:56 |
| openstackgerrit | Lev Morgan proposed openstack/airship-pegleg master: Added document wrapping command https://review.openstack.org/644637 | 19:08 |
| openstackgerrit | Merged openstack/airship-divingbell master: (perm) Fix CL for reverting missing file https://review.openstack.org/642051 | 19:12 |
| *** mbeierl has joined #airshipit | 19:27 | |
| openstackgerrit | Merged openstack/airship-drydock master: Disable tgtd service on maas-rack-controller https://review.openstack.org/643660 | 19:29 |
| *** roman_g has joined #airshipit | 19:32 | |
| *** ab2434_ has quit IRC | 19:35 | |
| openstackgerrit | Merged openstack/airship-treasuremap master: Uplift HAProxy to address CVEs https://review.openstack.org/644245 | 19:43 |
| evgenyl | I'm wondering if there are any deckhand experts, I'm trying to understand this paragraph from the docs https://airship-deckhand.readthedocs.io/en/latest/users/replacement.html#layering-without-replacement isn't a replacement of a single key can be done with just layering & merge strategy? | 19:46 |
| sthussey | @evgenyl I do believe the docs have a poor illustration of `replace` | 19:48 |
| evgenyl | sthussey: So do I understand it correctly, that the main use-case of `replace` is to replace an entire doc from the layer below? | 19:51 |
| sthussey | https://www.irccloud.com/pastebin/tCZ6wfiq/ | 19:53 |
| sthussey | I think the above is a bit better illustration | 19:53 |
| sthussey | it is where you want to snip out a branch of YAML and replace the entire branch | 19:54 |
| sthussey | rather than merging them | 19:54 |
| sthussey | for simple values like `debug: true`, a standard merge cares for it | 19:54 |
| evgenyl | sthussey: Ok, got it, thank you! | 19:54 |
| sthussey | if you just want to snip out the branch w/o replacing it you can use `method: delet` | 19:55 |
| sthussey | delete* | 19:55 |
| openstackgerrit | Kaspars Skels proposed openstack/airship-in-a-bottle master: Minor fixes as follow-up for new Pegleg https://review.openstack.org/644664 | 20:08 |
| kaspars__ | ha! IRC is tracking everything I do :) | 20:09 |
| evgenyl | Yep, we see everything :) | 20:19 |
| *** roman_g has quit IRC | 20:25 | |
| openstackgerrit | Lev Morgan proposed openstack/airship-pegleg master: Added document wrapping command https://review.openstack.org/644637 | 20:42 |
| openstackgerrit | Scott Hussey proposed openstack/airship-in-a-bottle master: (multinode) Make disk layout flexible https://review.openstack.org/638040 | 21:52 |
| *** kranthikirang has quit IRC | 22:05 | |
| *** michael-beaver has quit IRC | 22:16 | |
| openstackgerrit | Merged openstack/airship-treasuremap master: Add missing labels to osh charts and ucp chart groups https://review.openstack.org/636240 | 22:24 |
| openstackgerrit | Hemanth Nakkina proposed openstack/airship-in-a-bottle master: Add proxy support for demo environment https://review.openstack.org/615387 | 22:29 |
| *** kranthikirang has joined #airshipit | 22:54 | |
| *** kranthikirang has quit IRC | 22:59 | |
| *** nishant__ has quit IRC | 23:09 | |
| *** aaronsheffield has quit IRC | 23:43 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!