| openstackgerrit | Craig Anderson proposed openstack/airship-divingbell master: Exec: Use nsenter to enter all host namespaces when running exec https://review.openstack.org/642588 | 00:06 |
|---|---|---|
| *** sthussey has quit IRC | 00:21 | |
| openstackgerrit | Merged openstack/airship-promenade master: Generalize .fiotest ignore https://review.openstack.org/642548 | 00:28 |
| *** aaronsheffield has quit IRC | 00:46 | |
| openstackgerrit | Craig Anderson proposed openstack/airship-divingbell master: Exec: Use nsenter to enter all host namespaces when running exec https://review.openstack.org/642588 | 01:00 |
| *** openstackstatus has quit IRC | 02:22 | |
| *** openstack has joined #airshipit | 02:22 | |
| *** ChanServ sets mode: +o openstack | 02:22 | |
| openstackgerrit | Craig Anderson proposed openstack/airship-divingbell master: Exec: Use nsenter to enter all host namespaces when running exec https://review.openstack.org/642588 | 02:32 |
| openstackgerrit | Merged openstack/airship-divingbell master: Exec: Use nsenter to enter all host namespaces when running exec https://review.openstack.org/642588 | 03:26 |
| *** Nishant__ has quit IRC | 04:15 | |
| openstackgerrit | Smruti Soumitra Khuntia proposed openstack/airship-shipyard master: User context tracing through logging https://review.openstack.org/633873 | 05:03 |
| *** cheng1 has joined #airshipit | 05:48 | |
| *** cheng1 has quit IRC | 05:59 | |
| *** cheng1 has joined #airshipit | 05:59 | |
| *** irclogbot_0 has quit IRC | 06:09 | |
| *** irclogbot_0 has joined #airshipit | 06:09 | |
| *** cheng1 has quit IRC | 06:51 | |
| *** cheng1 has joined #airshipit | 06:52 | |
| *** jamesgu has quit IRC | 06:54 | |
| *** juhak has quit IRC | 07:33 | |
| *** juhak has joined #airshipit | 07:33 | |
| *** dimitris_ has joined #airshipit | 08:13 | |
| *** cheng1 has quit IRC | 09:02 | |
| *** cheng1 has joined #airshipit | 09:02 | |
| *** roman_g has joined #airshipit | 09:04 | |
| *** cheng1 has quit IRC | 09:38 | |
| *** cheng1 has joined #airshipit | 09:50 | |
| *** nick_kar has joined #airshipit | 09:50 | |
| *** cheng1 has quit IRC | 10:20 | |
| *** cheng1 has joined #airshipit | 10:21 | |
| openstackgerrit | Alexander Hughes proposed openstack/airship-pegleg master: PKI Cert generation and check updates https://review.openstack.org/639414 | 11:11 |
| openstackgerrit | Alexander Hughes proposed openstack/airship-pegleg master: PKI Cert generation and check updates https://review.openstack.org/639414 | 11:13 |
| *** cheng1 has quit IRC | 11:56 | |
| openstackgerrit | Roman Gorshunov proposed openstack/airship-divingbell master: Fix docs build on ReadTheDocs, enchance renderring https://review.openstack.org/638144 | 12:28 |
| *** AlexanderHughes has joined #airshipit | 12:47 | |
| openstackgerrit | Roman Gorshunov proposed openstack/airship-divingbell master: Fix docs build on ReadTheDocs, enchance renderring https://review.openstack.org/638144 | 12:53 |
| openstackgerrit | Sean Eagan proposed openstack/airship-armada master: build: support helm-toolkit pinning https://review.openstack.org/642568 | 12:57 |
| *** pkaralis has joined #airshipit | 13:00 | |
| *** aaronsheffield has joined #airshipit | 13:02 | |
| *** AlexanderHughes has quit IRC | 13:21 | |
| *** sthussey has joined #airshipit | 13:23 | |
| *** dustinspecker has joined #airshipit | 13:40 | |
| *** ab2434_ has joined #airshipit | 13:45 | |
| *** juhak has quit IRC | 13:47 | |
| *** juhak has joined #airshipit | 13:48 | |
| *** openstack has joined #airshipit | 15:37 | |
| *** ChanServ sets mode: +o openstack | 15:37 | |
| wahnsinn | roman_g: some of them are not ready (0/1), but I guess that's how it is supposed to be | 15:37 |
| *** openstackstatus has joined #airshipit | 15:38 | |
| *** ChanServ sets mode: +v openstackstatus | 15:38 | |
| *** juhak has quit IRC | 15:39 | |
| *** juhak has joined #airshipit | 15:39 | |
| dwalt | #startmeeting airship | 16:00 |
| openstack | Meeting started Tue Mar 12 16:00:16 2019 UTC and is due to finish in 60 minutes. The chair is dwalt. 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 |
| dwalt | #topic Rollcall | 16:00 |
| aaronsheffield | o/ | 16:00 |
| *** openstack changes topic to "Rollcall (Meeting topic: airship)" | 16:00 | |
| dwalt | hey everyone! mattmceuen is out today, so I'll be hosting | 16:00 |
| dwalt | Here is our agenda: https://etherpad.openstack.org/p/airship-meeting-2019-03-12 | 16:00 |
| dwalt | Let's give it until 5 after for everyone to join and add items | 16:00 |
| evgenyl | Hi | 16:01 |
| michael-beaver | o/ | 16:01 |
| pas-ha | o/ | 16:01 |
| *** nickchase has joined #airshipit | 16:01 | |
| Nishant__ | o/ | 16:02 |
| *** AlexanderHughes has joined #airshipit | 16:03 | |
| roman_g | щ. | 16:04 |
| roman_g | o/ | 16:04 |
| jamesgu | o/ | 16:05 |
| dwalt | o/ to everyone trickling in! It's 5 after, let's go ahead and begin | 16:05 |
| dwalt | #topic Divingbell overrides fixed | 16:05 |
| *** openstack changes topic to "Divingbell overrides fixed (Meeting topic: airship)" | 16:05 | |
| dwalt | roman_g: the floor is yours | 16:05 |
| sthussey | here | 16:06 |
| roman_g | Just wanted to thank everyone who helped with fixing overrides issue | 16:07 |
| roman_g | sthussey srwilkers portdirect * Craig | 16:07 |
| *** peyunco has joined #airshipit | 16:07 | |
| roman_g | & | 16:07 |
| srwilkers | o/ | 16:07 |
| portdirect | we do now need to work out what to do about armada though | 16:08 |
| dwalt | ty for all of the hard word on that | 16:08 |
| portdirect | as its goning to need to be rebuilt against helm 2.13 | 16:08 |
| portdirect | and i think there is an issue there? | 16:08 |
| Nishant__ | there is already a PS out there by Sean to pin armada to a stable HTK | 16:09 |
| roman_g | portdirect: latest gates are green for armada, but yes, it needs to be checked | 16:09 |
| dwalt | portdirect: I saw a patch from seaneagan about pinning helm toolkit today. Should we reconsider moving forward with that approach? | 16:09 |
| Nishant__ | https://review.openstack.org/#/c/642568/ | 16:09 |
| dwalt | thanks NISHANT_ | 16:09 |
| portdirect | im meaning not the armada chart, but the ability for armada to make use of the fix | 16:09 |
| portdirect | in things like divingbell | 16:09 |
| portdirect | as the fix requires helm 2.13 | 16:09 |
| portdirect | and armada talks to an older version of tiller atm | 16:10 |
| portdirect | we need armada to support the `mergeOverwrite` function | 16:10 |
| dwalt | I see. I think there was a patch from michael-beaver to update Armada to 2.13 yesterday as well | 16:10 |
| portdirect | ah nice | 16:10 |
| Nishant__ | oh gotcha ! yep that needs to be thought through | 16:10 |
| portdirect | you have a link | 16:10 |
| dwalt | checking now | 16:10 |
| dwalt | michael-beaver: do you have any updates on that front? | 16:11 |
| michael-beaver | Yeah I threw that up yesterday, if we want to take that as the path forward I can brush it up and make sure it is using all the right functions | 16:11 |
| seaneagan | yeah, armada and promenade both can't use helm 2.13.0 due to: https://github.com/helm/helm/issues/5408 | 16:11 |
| michael-beaver | I initially abandoned it since it looks like we were going to just pin the htk | 16:11 |
| dwalt | #link https://review.openstack.org/642553 | 16:11 |
| seaneagan | will be fixed in helm 2.13.1 | 16:11 |
| portdirect | ok - so we blocked untill then seaneagan ? | 16:12 |
| openstackgerrit | Kaspars Skels proposed openstack/airship-in-a-bottle master: Cert/key related improvements to gate-multinode https://review.openstack.org/642585 | 16:12 |
| portdirect | as hacky as it is - can armada just use a 2.13 tiller and be happy? | 16:12 |
| seaneagan | yes we are blocked from using helm 2.13's `mergeOverwrite` until then | 16:13 |
| seaneagan | no that is the issue that we are using tiller flags that were accidentally removed in 2.13 | 16:14 |
| seaneagan | will be re-added in 2.13.1 | 16:14 |
| portdirect | ok - we probably need to work out how to better co-ordinate these fixes then | 16:14 |
| portdirect | so we dont end up breaking each others projects | 16:14 |
| portdirect | you have any idea when 2.13.1 is out? | 16:14 |
| seaneagan | no, but here's all i know: https://github.com/helm/helm/milestone/62 | 16:15 |
| portdirect | could we use a pinned commit from master untill then to work round this? or we totally dependent on a release from helm? | 16:15 |
| seaneagan | i think it would be possible but that may leave armada in an unreleasable state | 16:16 |
| roman_g | add a gate to HTK with Armada? | 16:16 |
| roman_g | (we already have divingbell gate, non-voting) | 16:17 |
| dwalt | roman_g: We do gate HTK in Armada when there are chart changes, but maybe we should consider running that gate on every patch | 16:17 |
| seaneagan | i think roman_g: meant the other way around? | 16:18 |
| roman_g | vice versa | 16:18 |
| roman_g | yes. add armada gate to HTK | 16:18 |
| portdirect | this would be ok - though would need to be non-voting | 16:18 |
| roman_g | yes, same as divingbell | 16:18 |
| *** wwilley has joined #airshipit | 16:18 | |
| roman_g | #action roman_g check on adding armada non-voting gate to openstack-helm-infra (HTK) | 16:19 |
| dwalt | that would be great. ty roman_g | 16:20 |
| *** mfuller_ has joined #airshipit | 16:20 | |
| dwalt | as far as updating HTK for other project gates, are we content with the temporary pinning approach? | 16:20 |
| dwalt | e.g. promenade, shipyard | 16:21 |
| Nishant__ | yes that could make sense | 16:21 |
| seaneagan | i think ability to pin htk is good, should probably try to keep the pin pointed at master whenever possible | 16:22 |
| dwalt | #action dwalt update remaining Airship gates that utilize HTK | 16:23 |
| dwalt | seaneagan: agree | 16:23 |
| dwalt | anything else on this before we move on? | 16:23 |
| dwalt | #topic Minimirror: move spec and review on the fly patches | 16:24 |
| *** openstack changes topic to "Minimirror: move spec and review on the fly patches (Meeting topic: airship)" | 16:24 | |
| dwalt | I see roman_g added this. I have a patch out right now to remove the Airship spec | 16:24 |
| dwalt | did you have additional thoughts on this roman_g? | 16:24 |
| dwalt | #link https://review.openstack.org/641767 | 16:24 |
| roman_g | yes. there are other patches under https://review.openstack.org/#/q/minimirror which need to be reviewed and moved to openstack-helm-addons | 16:25 |
| roman_g | or -images | 16:25 |
| roman_g | where the minimirror resides | 16:25 |
| dwalt | Is there specific functionality to gain from these patches? My understanding was that most of these are obsolete with the recent additions to OSH-addons and OSH-images | 16:26 |
| roman_g | nobody reviewed them. if they are obsolete - then those patches need to be abandoned | 16:26 |
| roman_g | they might be useful | 16:27 |
| dwalt | agreed. I can take another look and reach out to the author in case I missed over anything | 16:27 |
| dwalt | #action dwalt reach out to author of stale mini-mirror patches | 16:27 |
| roman_g | author (Andrey Volkov) has left the business, I've contacted him already | 16:27 |
| dwalt | ty roman_g. We can revisit this next week after I cross-reference the patches against existing mini-mirror functionality | 16:29 |
| dwalt | #topic Deckhand & Shipyard - OpenSUSE builds | 16:30 |
| *** openstack changes topic to "Deckhand & Shipyard - OpenSUSE builds (Meeting topic: airship)" | 16:30 | |
| roman_g | jamesgu, arunkant - do oyu have any news on the patches? | 16:30 |
| jamesgu | arun, can you provide updates? | 16:31 |
| arunkant | roman_g: yes, those patches are ready for review | 16:31 |
| arunkant | I have added the docs and addressed earlier comments ..so will appreciate more eyes on those reviews | 16:32 |
| roman_g | thanks, arunkant, will have a look | 16:32 |
| dwalt | arunkant: great! I will have another look later today as well. Thanks for all the work on this | 16:32 |
| dwalt | + jamesgu | 16:32 |
| roman_g | >> and agree on naming convention in multi-distro spec document (airship-specs repo), please? | 16:32 |
| arunkant | roman_g: there is armada review for similar change | 16:33 |
| roman_g | nice | 16:33 |
| dwalt | roman_g: Does the spec call out a specific convention for the Dockerfiles? | 16:33 |
| roman_g | no, and it would be good to have it there | 16:34 |
| roman_g | to discuss it there and to have it there | 16:34 |
| sthussey | I haven't looked, but did the docker image tags get adjusted? | 16:34 |
| sthussey | because last time I looked, it seems like the different builds are going to trample on one another | 16:34 |
| dwalt | sthussey: at first glance, that seems to be missing from the Armada change | 16:35 |
| dwalt | #link https://review.openstack.org/#/c/637327/10/Makefile | 16:35 |
| arunkant | So we are using Dockerfile.opensuse_leap15 for file names. And tag names are also updated for opensuse specific images in publish jobs | 16:35 |
| roman_g | So. Can we have it discussed and agreed on in a specs/approved/multi-linux-distros.rst first, and then work on patches? This way we would have a standard across airship-* subprojects nad a standar for other distros to follow. | 16:36 |
| dwalt | arunkant: the tag is adjusted on line 31 | 16:37 |
| dwalt | I believe you just need to add the distro to the tag | 16:37 |
| roman_g | would be also good to check on OSH work, because they do work in this direction too | 16:38 |
| sthussey | I'll check again, first review on that doesn't show the distro getting in the tag | 16:39 |
| arunkant | dwalt: Is this the right place to specify tag for distro image: https://git.openstack.org/cgit/openstack/airship-armada/tree/.zuul.yaml?h=refs/changes/27/637327/10#n108 | 16:39 |
| sthussey | So for a tag of a given commit, the SUSE and the Ubuntu builds will be in a race condition to see which ends up with the tag | 16:39 |
| dwalt | arunkant: it's on line 31 of the Makefile | 16:40 |
| openstackgerrit | Sean Eagan proposed openstack/airship-armada master: Run chart job unconditionally https://review.openstack.org/642817 | 16:40 |
| dwalt | appending the distro will prevent the race condition sthussey is referring to | 16:40 |
| openstackgerrit | Kaspars Skels proposed openstack/airship-in-a-bottle master: Cert/key related improvements to gate-multinode https://review.openstack.org/642585 | 16:41 |
| sthussey | I think it makes sense to change the image name | 16:41 |
| sthussey | So we'd end up with something like quay.io/airshipit/drydock-opensuse:latest | 16:41 |
| sthussey | rather than quay.io/airshipit/drydock:latest-opensuse | 16:41 |
| sthussey | I haven't looked at the spec though | 16:42 |
| dwalt | I like that better. arunkant: would you be able to update the spec to represent these two items? | 16:42 |
| roman_g | looks good from layers squashing perspective | 16:42 |
| dwalt | I'm happy to assist after the meeting if you have additional questions | 16:42 |
| jamesgu | where does the OS version go in the image tag? | 16:43 |
| sthussey | It goes in the image name, not the tag | 16:43 |
| arunkant | dwalt: publish jobs use this https://github.com/openstack/airship-shipyard/blob/master/tools/gate/roles/build-images/tasks/airship-shipyard.yaml#L113 | 16:43 |
| sthussey | because the Ubuntu-based image and the SUSE-based image are basically not related in terms of Docker's view | 16:43 |
| roman_g | The spec has not been updated yet. This is what I'm asking for - let's first discuss and agree details in a spec, and then go for implementation. Can we? | 16:43 |
| jamesgu | yes agreed I think. so an example would be like this: quay.io/airshipit/drydock-opensuse-15.0:lates | 16:44 |
| sthussey | yep | 16:44 |
| arunkant | roman_g: I thought that we need to add docs for how to add new distro support .. | 16:45 |
| sthussey | @roman_g I have no argument against hammering out this standard in the spec and then following up with updates to the PS | 16:45 |
| roman_g | arunkant: yes. after we get the spec agreed )) | 16:45 |
| sthussey | Let's just be expedient about reviews and approval as I'm sure arunkant and jamesgu would like to get this finished out | 16:45 |
| dwalt | ++ I am happy to look as soon as it's up | 16:46 |
| jamesgu | we can do both at the same time. | 16:46 |
| arunkant | roman_g: okay..then will update the spec first in that case.. | 16:46 |
| dwalt | arunkant: jamesgu: ty both | 16:46 |
| sthussey | We'll need to update the ubuntu build as well | 16:46 |
| dwalt | #action arunkant update multi-distro image spec | 16:46 |
| roman_g | arunkant: thank you. | 16:46 |
| sthussey | to look something like quay.io/airshipit/drydock-ubuntu-16.04:latest | 16:47 |
| roman_g | would be good to see how other projects approach the same problem. | 16:47 |
| *** nick_kar has quit IRC | 16:48 | |
| roman_g | many projects build images based on ubuntu+centos+alpine | 16:48 |
| dwalt | roman_g: I recommend looking at OSH-images. Though, I think arunkant and jamesgu have mostly followed that approach | 16:48 |
| dwalt | arunkant: please also feel free to ping us here once it's up, so we can review it asap :) | 16:49 |
| openstackgerrit | Kaspars Skels proposed openstack/airship-in-a-bottle master: Cert/key related improvements to gate-multinode https://review.openstack.org/642585 | 16:49 |
| dwalt | okay, anything else here? We only have ~10 min remaining | 16:49 |
| roman_g | dwalt: I know how OSH images are built, thanks. I'm thinking of other projects. | 16:49 |
| sthussey | We can iterate the design - let's solve the issue in front of us today, not the issues in front of other projects when they made their decision | 16:50 |
| dwalt | roman_g: Gotcha - we can definitely keep those mind when reviewing :) | 16:51 |
| dwalt | #topic Participation in Google Summer of Code and Season of Docs | 16:51 |
| *** openstack changes topic to "Participation in Google Summer of Code and Season of Docs (Meeting topic: airship)" | 16:51 | |
| dwalt | roman_g: I believe this is yours. Do you mind taking this? | 16:51 |
| roman_g | sthussey: I'm not trying to solve other projects problems | 16:51 |
| roman_g | All right. We have skipped Google summer of code. | 16:52 |
| roman_g | But we have possibility to join season of docs. | 16:52 |
| roman_g | How do we understand if we need that or not? | 16:52 |
| dwalt | roman_g: I'm not familiar with the season of docs. Do you have any information on it? | 16:53 |
| roman_g | Sseason of Docs program in one line: we ask for help on documentation, tech.writer works on documentation, we accept it, and google pays some bonus to the writer and to the mentor from the project | 16:53 |
| sthussey | What is the effort to be accepted by Google? | 16:54 |
| roman_g | 0) decide on work to do 1) fill in an application form 2) dedicate a mentor 3) work with tech.writer | 16:55 |
| roman_g | >> Create Airship Developer Guide - https://storyboard.openstack.org/#!/story/2005122 | 16:55 |
| roman_g | this is a list from Matt | 16:55 |
| roman_g | which fits good | 16:55 |
| roman_g | from my point of view. | 16:55 |
| sthussey | https://developers.google.com/season-of-docs/docs/mentor-responsibilities | 16:56 |
| dwalt | It sounds like a good opportunity, though I'd like to read about it a bit more before discussing | 16:57 |
| dwalt | Does anyone have any experience with the program? | 16:57 |
| roman_g | It's totally new program, opened yesterday. | 16:58 |
| dwalt | ah yeah, just saw that | 16:58 |
| roman_g | *announced | 16:58 |
| dwalt | #link https://developers.google.com/season-of-docs/docs/timeline | 16:58 |
| openstackgerrit | Sean Eagan proposed openstack/airship-promenade master: build: support helm-toolkit pinning https://review.openstack.org/642825 | 16:58 |
| sthussey | May be worth talking w/ Lindsey as he is a technical writer | 16:58 |
| openstackgerrit | Kaspars Skels proposed openstack/airship-in-a-bottle master: Cert/key related improvements to gate-multinode https://review.openstack.org/642585 | 16:58 |
| dwalt | ++ | 16:59 |
| roman_g | sent an e-mail already, pending reply,and Matt & Pete got this email too | 16:59 |
| roman_g | no reply yet | 16:59 |
| dwalt | roman_g: great. Let's revisit this next week if you're okay with that | 16:59 |
| roman_g | okok | 16:59 |
| dwalt | That'll give everyone a chance to read about it, since it's so new | 17:00 |
| dwalt | ty roman_g | 17:00 |
| dwalt | Sorry to cut it short, but we are out of time. Thanks for the good discussion everyone! Please give these patches some extra attention if you have time: | 17:01 |
| dwalt | https://review.openstack.org/#/c/635507/ - openstack/airship-treasuremap - Add cache for results of requests to quay.io in Updater tool | 17:01 |
| dwalt | https://review.openstack.org/#/c/615387/ | 17:01 |
| dwalt | https://review.openstack.org/636229 (Shipyard suse image support only) | 17:01 |
| dwalt | https://review.openstack.org/638301 (Deckhand suse image support only) | 17:01 |
| dwalt | https://review.openstack.org/#/c/637327/ ( Armada suse image support only) | 17:01 |
| dwalt | #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 12 17:01:39 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-12-16.00.html | 17:01 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/airship/2019/airship.2019-03-12-16.00.txt | 17:01 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/airship/2019/airship.2019-03-12-16.00.log.html | 17:01 |
| roman_g | thank you all | 17:02 |
| *** nick_kar has joined #airshipit | 17:03 | |
| wahnsinn | roman_g: I found a pod which is in CrashLoopBackOff state: mariadb-ingress-.... | 17:10 |
| roman_g | wahnsinn: seems you have the same issue as I have. | 17:11 |
| roman_g | I didn't digg into it. | 17:11 |
| roman_g | some problems with port numbers? | 17:11 |
| wahnsinn | roman_g: there seems to be an issue with the config map: "executing "nginx.tmpl" at <$cfg.EnableVtsStatus>: can't evaluate field EnableVtsStatus in type config.Configuration" | 17:11 |
| wahnsinn | furthermore the "airship-openstack-mariadb-airship-openstack-mariadb-mariadb-ingress" is empty | 17:12 |
| wahnsinn | *configmap | 17:12 |
| wahnsinn | what do you mean by problems with port numbers? can't find anything related in the log | 17:14 |
| roman_g | that was just a guess | 17:15 |
| roman_g | because previously there were problems with maas-ingress, and specifically with the port numbers. it's now fixed | 17:15 |
| wahnsinn | ah, okay | 17:18 |
| *** AlexanderHughes has quit IRC | 17:22 | |
| openstackgerrit | Kaspars Skels proposed openstack/airship-in-a-bottle master: Cert/key related improvements to gate-multinode https://review.openstack.org/642585 | 17:35 |
| openstackgerrit | Anthony Bellino proposed openstack/airship-divingbell master: [WIP] Initial Ansible Daemonset https://review.openstack.org/640539 | 17:40 |
| openstackgerrit | James Gu proposed openstack/airship-treasuremap master: Add missing labels to osh charts and ucp chart groups https://review.openstack.org/636240 | 17:47 |
| openstackgerrit | Evgeniy L proposed openstack/airship-treasuremap master: Shipyard now requires db init template from osh-infra https://review.openstack.org/642841 | 18:01 |
| evgenyl | Hi dwalt, wanted to talk you regarding to a broken gate for airship-treasuremap, the failure is related to updated shipyard that now requires new templates from helm-infra (failure example http://logs.openstack.org/71/642571/1/check/airskiff-deploy-gate/bcc708a/job-output.txt.gz) here is a fix attempt https://review.openstack.org/#/c/642841/ , is it ok to just try to uplift osh-infra? Or do you think it would better to pin | 18:04 |
| evgenyl | shipyard? | 18:04 |
| openstackgerrit | Kaspars Skels proposed openstack/airship-in-a-bottle master: Cert/key related improvements to gate-multinode https://review.openstack.org/642585 | 18:10 |
| *** peyunco has quit IRC | 18:10 | |
| openstackgerrit | Kaspars Skels proposed openstack/airship-in-a-bottle master: Cert/key related improvements to gate-multinode https://review.openstack.org/642585 | 18:12 |
| openstackgerrit | Kaspars Skels proposed openstack/airship-in-a-bottle master: Cert/key related improvements to gate-multinode https://review.openstack.org/642585 | 18:14 |
| arunkant | dwalt: do you have few mins to talk about distro image names ? | 18:17 |
| *** levmorgan has joined #airshipit | 18:19 | |
| dwalt | evgenyl: Updating osh-infra there should be fine, but we are waiting on another change to merge in osh-infra that is also breaking the gates https://review.openstack.org/642115 | 18:24 |
| openstackgerrit | Alexander Hughes proposed openstack/airship-pegleg master: Set salt when generating genesis bundle https://review.openstack.org/642848 | 18:24 |
| evgenyl | dwalt: Is this change also related to treasuremap gates? | 18:25 |
| dwalt | arunkant: sure! | 18:25 |
| dwalt | evgenyl: yes, there are issues with PostgreSQL without that patch. Although, postgres looks healthy in your logs | 18:26 |
| dwalt | Thanks for submitting the uplift, I'll watch it through and see what happens | 18:27 |
| *** levmorgan has quit IRC | 18:29 | |
| arunkant | dwalt: So in weekly meeting, you suggested the idea of having distro information in image name. Is there a benefit of that approach instead of just using tags to identify distro and version info? | 18:30 |
| *** nickchase has quit IRC | 18:30 | |
| arunkant | dwalt: I looking docker.io to see samples and rarely see image name is changed ..like for centos httpd, the repo url is different but image name is still kept same | 18:31 |
| sthussey | in my mind, there is little chance that you will fluctuate which distro-base image you want to deploy | 18:32 |
| sthussey | but you will likely want to continually march along commit tags as you update deployments | 18:33 |
| arunkant | dwalt: Trying to see what's the benefit with different image name vs tags to identify it | 18:33 |
| openstackgerrit | Evgeniy L proposed openstack/airship-treasuremap master: Shipyard now requires db init template from osh-infra https://review.openstack.org/642841 | 18:33 |
| sthussey | which seems to align with a static image name and changing tag | 18:33 |
| sthussey | in the end, it doesn't matter a lot and I would support either method if the rest of the airship community decides | 18:34 |
| dwalt | arunkant: Can you send an example of the centos project you are referring to? I think sthussey was referring to the Makefile variable, IMAGE_NAME | 18:34 |
| sthussey | You can look at most of the docker.io library images like 'python' or 'httpd' | 18:34 |
| *** michaelbeaver has joined #airshipit | 18:35 | |
| *** jamesgu has quit IRC | 18:35 | |
| *** jamesgu has joined #airshipit | 18:36 | |
| arunkant | dwalt: So I have images created locally like this .. http://paste.openstack.org/show/747639/ | 18:36 |
| arunkant | dwalt: This is what I found when searching for httpd https://hub.docker.com/r/centos/httpd/tags | 18:37 |
| arunkant | sthussey: ^^^ | 18:37 |
| *** michael-beaver has quit IRC | 18:37 | |
| sthussey | michael-beaver pointed out that with making the name describe the distro base, you end up with managing a lot more Docker repos | 18:38 |
| sthussey | So putting it in the tag makes sense | 18:38 |
| arunkant | sthussey: I see similar pattern for python https://hub.docker.com/_/python/?tab=description .. tag specifies distro information | 18:39 |
| openstackgerrit | Kaspars Skels proposed openstack/airship-in-a-bottle master: Cert/key related improvements to gate-multinode https://review.openstack.org/642585 | 18:44 |
| arunkant | sthussey, dwalt: I am not finding patterns or some best practices around image names so open to ideas. So whatever is recommended and seems better from community perspective, I can propose in spec and follow it in reviews. | 18:46 |
| openstackgerrit | Sirajudeen proposed openstack/airship-in-a-bottle master: [pegleg] - Used the latest pegleg verion. https://review.openstack.org/637993 | 18:49 |
| openstackgerrit | Kaspars Skels proposed openstack/airship-in-a-bottle master: Cert/key related improvements to gate-multinode https://review.openstack.org/642585 | 18:50 |
| openstackgerrit | Scott Hussey proposed openstack/airship-promenade master: [fix] etcd backup support for multiple releases https://review.openstack.org/642856 | 18:51 |
| sthussey | @arunkant then lets go with the tags | 18:55 |
| dwalt | arunkant: Sorry, I had to step away for a few. I agree with sthussey in that the point michael-beaver made limits us to tags | 18:59 |
| arunkant | sthussey : Okay..so does this look okay ? quay.io/airshipit/shipyard:opensuse_15.0 for opensuse image and quay.io/airshipit/shipyard:ubuntu_1604 for ubuntu | 19:02 |
| arunkant | thanks dwalt | 19:04 |
| arunkant | sthussey: Also may be quay.io/airshipit/shipyard:latest (which refers to default ubuntu ) for backward-compatibility and their reference in charts | 19:09 |
| sthussey | I would expect the distro to just be appended as dwalt suggested earlier today | 19:11 |
| sthussey | that way we can pin to a commit | 19:11 |
| openstackgerrit | Jared Miller proposed openstack/airship-promenade master: Do not quote tls arg values https://review.openstack.org/642586 | 19:15 |
| openstackgerrit | Alexander Hughes proposed openstack/airship-pegleg master: Set salt when generating genesis bundle https://review.openstack.org/642848 | 19:16 |
| arunkant | sthussey: So thinking was to have version information than latest as we might be publishing different images for different versions within a distro. | 19:17 |
| *** wahnsinn has quit IRC | 19:17 | |
| arunkant | sthussey: latest by itself does not have any distro version information .. | 19:17 |
| openstackgerrit | Sandeep Reddy Thumma proposed openstack/airship-promenade master: setting .PEM files permissions to 640 https://review.openstack.org/640775 | 19:17 |
| openstackgerrit | Sandeep Reddy Thumma proposed openstack/airship-promenade master: setting .PEM files permissions to 640 https://review.openstack.org/640775 | 19:22 |
| openstackgerrit | Merged openstack/airship-armada master: build: support helm-toolkit pinning https://review.openstack.org/642568 | 19:27 |
| *** michael-beaver has joined #airshipit | 19:28 | |
| openstackgerrit | Merged openstack/airship-promenade master: build: support helm-toolkit pinning https://review.openstack.org/642825 | 19:29 |
| openstackgerrit | Drew Walters proposed openstack/airship-promenade master: [fix] etcd backup support for multiple releases https://review.openstack.org/642856 | 19:30 |
| *** michaelbeaver has quit IRC | 19:32 | |
| dwalt | arunkant: I'm assuming we would just publish something like quay.io/airshipit/armada:ubuntu-latest | 19:34 |
| dwalt | As far as I'm aware, we don't publish any other versioned images ourselves (quay.io moves tags from latest to a unique SHA when an image is published though, so those would not include distro names) | 19:36 |
| openstackgerrit | Merged openstack/airship-armada master: Normalize assignment of Tiller defaults https://review.openstack.org/641126 | 19:37 |
| dwalt | If versioned images was something we wanted to do, I think that would be a larger discussion outside of multi-distro support | 19:38 |
| openstackgerrit | Merged openstack/airship-in-a-bottle master: Removed the hardcoded value and used the env var https://review.openstack.org/642506 | 19:38 |
| dwalt | evgenyl: looks like the postgres issue is not an issue, your patch passed the check https://review.openstack.org/642841 | 19:40 |
| evgenyl | dwalt: Oh, cool, thanks for letting me know. | 19:42 |
| dwalt | evgenyl: np, thanks for submitting! | 19:42 |
| *** mbeierl has quit IRC | 19:43 | |
| sthussey | Every image we publish is tagged with the commit | 19:44 |
| openstackgerrit | Drew Walters proposed openstack/airship-promenade master: Do not quote tls arg values https://review.openstack.org/642586 | 19:45 |
| sthussey | Unfortunately I cannot paste the screenshot from quay.io | 19:45 |
| openstackgerrit | Alexander Hughes proposed openstack/airship-pegleg master: Set salt when generating genesis bundle https://review.openstack.org/642848 | 19:55 |
| openstackgerrit | Alexander Hughes proposed openstack/airship-pegleg master: Set salt when generating genesis bundle https://review.openstack.org/642848 | 19:56 |
| openstackgerrit | Alexander Hughes proposed openstack/airship-pegleg master: Set salt when generating genesis bundle https://review.openstack.org/642848 | 19:57 |
| openstackgerrit | Merged openstack/airship-shipyard master: tools: generalize execute_shipyard_action https://review.openstack.org/640830 | 19:59 |
| openstackgerrit | Michael Beaver proposed openstack/airship-armada master: Support in Armada for locking Tiller https://review.openstack.org/632483 | 20:03 |
| openstackgerrit | Michael Beaver proposed openstack/airship-armada master: Move Tiller into Armada pod https://review.openstack.org/632788 | 20:03 |
| openstackgerrit | Merged openstack/airship-promenade master: Do not quote tls arg values https://review.openstack.org/642586 | 20:07 |
| openstackgerrit | Merged openstack/airship-in-a-bottle master: [pegleg] - Used the latest pegleg verion. https://review.openstack.org/637993 | 20:08 |
| openstackgerrit | Sean Eagan proposed openstack/airship-armada master: Run chart job unconditionally https://review.openstack.org/642817 | 20:10 |
| openstackgerrit | Nishant Kumar proposed openstack/airship-deckhand master: [FIX] Change Helm-toolkit pinning to new commit https://review.openstack.org/642884 | 20:33 |
| openstackgerrit | Merged openstack/airship-pegleg master: [Decrypt] - Added Decrypt before doc upload https://review.openstack.org/635097 | 20:58 |
| *** dustinspecker has quit IRC | 21:10 | |
| openstackgerrit | Evgeniy L proposed openstack/airship-in-a-bottle master: Fix the misspelling of "required" https://review.openstack.org/631133 | 21:15 |
| openstackgerrit | Evgeniy L proposed openstack/airship-in-a-bottle master: Cache registry image source config https://review.openstack.org/633525 | 21:22 |
| openstackgerrit | Rajeshwari Dharwadkar proposed openstack/airship-pegleg master: Support pegleg to run on opensuse leap15 image https://review.openstack.org/639495 | 21:26 |
| openstackgerrit | Jared Miller proposed openstack/airship-treasuremap master: Disable weak tls ciphers for kube-apiserver https://review.openstack.org/636754 | 21:28 |
| openstackgerrit | Kaspars Skels proposed openstack/airship-in-a-bottle master: Cert/key related improvements to gate-multinode https://review.openstack.org/642585 | 21:39 |
| openstackgerrit | Merged openstack/airship-in-a-bottle master: Cache registry image source config https://review.openstack.org/633525 | 21:41 |
| openstackgerrit | Merged openstack/airship-promenade master: [fix] etcd backup support for multiple releases https://review.openstack.org/642856 | 21:49 |
| openstackgerrit | Sean Eagan proposed openstack/airship-promenade master: Pin rest of helm-toolkit references https://review.openstack.org/642906 | 21:59 |
| openstackgerrit | Merged openstack/airship-deckhand master: [FIX] Change Helm-toolkit pinning to new commit https://review.openstack.org/642884 | 21:59 |
| openstackgerrit | Dan Crank proposed openstack/airship-promenade master: Log client-id in UCP API endpoints https://review.openstack.org/634071 | 22:07 |
| openstackgerrit | Anthony Bellino proposed openstack/airship-divingbell master: [WIP] Initial Ansible Daemonset https://review.openstack.org/640539 | 22:22 |
| openstackgerrit | Dan Crank proposed openstack/airship-promenade master: Log client-id in UCP API endpoints https://review.openstack.org/634071 | 22:28 |
| *** ab2434_ has quit IRC | 22:34 | |
| *** ianychoi_ is now known as ianychoi | 22:47 | |
| *** michaelbeaver has joined #airshipit | 23:00 | |
| openstackgerrit | Dan Crank proposed openstack/airship-deckhand master: Log client-id in UCP API endpoints https://review.openstack.org/634068 | 23:01 |
| roman_g | https://review.openstack.org/#/c/636223/ - please, review (we promissed to review it on a meeting). | 23:01 |
| roman_g | Spec: Support OCI image registry with authentication turned on | 23:01 |
| roman_g | by Angie Wang, Windriver | 23:02 |
| *** michael-beaver has quit IRC | 23:04 | |
| *** michaelbeaver has quit IRC | 23:06 | |
| *** aaronsheffield has quit IRC | 23:22 | |
| openstackgerrit | Merged openstack/airship-in-a-bottle master: Fix the misspelling of "required" https://review.openstack.org/631133 | 23:23 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!