| *** michaelbeaver has quit IRC | 00:00 | |
| *** aaronsheffield has quit IRC | 00:25 | |
| *** sreejithp has quit IRC | 00:37 | |
| *** cheng1 has joined #airshipit | 01:21 | |
| *** sthussey has quit IRC | 01:22 | |
| *** Nishant__ has quit IRC | 01:40 | |
| *** debugger_io has quit IRC | 02:30 | |
| *** hemanth_n has joined #airshipit | 02:46 | |
| *** hemanth_n has quit IRC | 02:57 | |
| *** roman_g has quit IRC | 03:17 | |
| *** openstackgerrit has joined #airshipit | 03:36 | |
| openstackgerrit | Dan Crank proposed openstack/airship-deckhand master: Log client-id in UCP API endpoints https://review.openstack.org/634068 | 03:36 |
|---|---|---|
| *** cheng1 has quit IRC | 03:37 | |
| openstackgerrit | jacky06 proposed openstack/airship-armada master: Update json module to jsonutils https://review.openstack.org/640476 | 04:43 |
| openstackgerrit | jacky06 proposed openstack/airship-armada master: Update json module to jsonutils https://review.openstack.org/640476 | 04:48 |
| openstackgerrit | jacky06 proposed openstack/airship-armada master: Update json module to jsonutils https://review.openstack.org/640476 | 04:58 |
| *** cheng1 has joined #airshipit | 05:31 | |
| *** jhesketh has quit IRC | 05:47 | |
| *** jhesketh has joined #airshipit | 05:48 | |
| *** cheng1 has quit IRC | 05:48 | |
| *** cheng1 has joined #airshipit | 05:58 | |
| *** c06 has joined #airshipit | 06:03 | |
| c06 | hi all.. is there any cleanup scripts available for airskiff...! | 06:04 |
| openstackgerrit | Smruti Soumitra Khuntia proposed openstack/airship-armada master: End user logging for audit traceabilty https://review.openstack.org/638124 | 06:09 |
| *** dims has quit IRC | 06:24 | |
| *** juhak has quit IRC | 06:31 | |
| *** juhak_ has joined #airshipit | 06:31 | |
| *** juhak_ is now known as juhak | 06:37 | |
| *** cheng1 has quit IRC | 06:40 | |
| c06 | hi, i tried this patch for airship in a bottle: https://review.openstack.org/#/c/615387/ | 06:48 |
| c06 | facing error while running airship-in-a-bottle.sh script | 06:51 |
| c06 | http://paste.openstack.org/show/747269/ - any suggestions.? | 06:51 |
| *** cheng1 has joined #airshipit | 07:06 | |
| *** c06 has quit IRC | 07:35 | |
| *** c06 has joined #airshipit | 07:44 | |
| *** pkaralis has quit IRC | 07:51 | |
| *** roman_g has joined #airshipit | 08:43 | |
| *** mbeierl has quit IRC | 09:02 | |
| *** mbeierl has joined #airshipit | 09:04 | |
| *** c06 has quit IRC | 09:19 | |
| *** roman_g has quit IRC | 09:20 | |
| *** lemko has joined #airshipit | 09:23 | |
| *** cheng1 has quit IRC | 09:34 | |
| *** roman_g has joined #airshipit | 09:41 | |
| *** cheng1 has joined #airshipit | 09:49 | |
| *** cheng1 has quit IRC | 10:08 | |
| *** hemanth_n has joined #airshipit | 10:29 | |
| *** dimitris_ has quit IRC | 10:58 | |
| *** dims has joined #airshipit | 11:01 | |
| *** hemanth_n has quit IRC | 11:15 | |
| *** dimitris_ has joined #airshipit | 11:23 | |
| *** jesusaur has quit IRC | 13:22 | |
| *** jesusaur has joined #airshipit | 13:25 | |
| *** hemanth_n has joined #airshipit | 13:31 | |
| mattmceuen | Hey c06: I'll take a look today :) | 13:44 |
| mattmceuen | Here's the agenda for our team meeting coming up in 15min folks! https://etherpad.openstack.org/p/airship-meeting-2019-03-05 | 13:44 |
| *** levmorgan has joined #airshipit | 13:51 | |
| *** aaronsheffield has joined #airshipit | 13:55 | |
| *** Nishant_ has joined #airshipit | 13:57 | |
| *** sgrasley has joined #airshipit | 13:59 | |
| evrardjp | o/ | 13:59 |
| *** juhak has quit IRC | 13:59 | |
| mattmceuen | #startmeeting airship | 14:00 |
| openstack | Meeting started Tue Mar 5 14:00:10 2019 UTC and is due to finish in 60 minutes. The chair is mattmceuen. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| *** openstack changes topic to " (Meeting topic: airship)" | 14:00 | |
| mattmceuen | #topic Rollcall | 14:00 |
| openstack | The meeting name has been set to 'airship' | 14:00 |
| mattmceuen | GM / GE everyone! | 14:00 |
| *** openstack changes topic to "Rollcall (Meeting topic: airship)" | 14:00 | |
| Nishant_ | o/ | 14:00 |
| *** juhak has joined #airshipit | 14:00 | |
| dwalt | GM! o/ | 14:00 |
| mattmceuen | Agenda for today: https://etherpad.openstack.org/p/airship-meeting-2019-03-05 | 14:00 |
| hemanth_n | GM | 14:00 |
| mattmceuen | We'll wait a few for folks to trickle in; please add anything you'd like to discuss to the agenda | 14:00 |
| seaneagan | o/ | 14:00 |
| georgk | i | 14:01 |
| georgk | hi | 14:01 |
| levmorgan | o/ | 14:01 |
| mattmceuen | Ok - let's get started | 14:02 |
| mattmceuen | #topic Armada Helm test timeout value change | 14:02 |
| *** openstack changes topic to "Armada Helm test timeout value change (Meeting topic: airship)" | 14:02 | |
| mattmceuen | dwalt: all yours | 14:02 |
| dwalt | Thanks mattmceuen | 14:02 |
| dwalt | Just a heads up: there is an Armada change that went in recently that changes the behavior of how Helm tests are run in Armada | 14:03 |
| dwalt | #link | 14:03 |
| dwalt | #link https://review.openstack.org/618585 | 14:03 |
| dwalt | woops :) | 14:03 |
| dwalt | The timeout value for Helm tests is no longer derived from the wait timeout value and is a separate test.timeout key | 14:04 |
| mattmceuen | dwalt, wasn't it also the case that the helm test was actually being included in the overall wait timeout? | 14:04 |
| mattmceuen | Such that, if timeout was 5 min, and the actual deploy took 5 min, there wouldn't be any time left for test? | 14:05 |
| *** sthussey has joined #airshipit | 14:05 | |
| dwalt | mattmceuen: yes, it was derived from the remaining time left from the wait timeout after an upgrade was completed | 14:05 |
| dwalt | exactly | 14:05 |
| dwalt | Now that the new default timeout value is 300, some longer tests may unexpectedly timeout. For instance, we saw this happen with Neutron in the Airship-Seaworthy site | 14:06 |
| dwalt | #link https://review.openstack.org/639462 | 14:06 |
| dwalt | That's all, just more of a heads up | 14:06 |
| mattmceuen | Got it - good one, thanks for bring it up | 14:06 |
| dwalt | sure thing | 14:07 |
| mattmceuen | #topic Airship comminity IRC meeting time change | 14:07 |
| *** openstack changes topic to "Airship comminity IRC meeting time change (Meeting topic: airship)" | 14:07 | |
| mattmceuen | Hey roman_g do we have you in yet perchance? | 14:07 |
| mattmceuen | I have a handle on this one anyway: https://review.openstack.org/#/c/640359/ | 14:07 |
| mattmceuen | As discussed here & on the mailing list, there's a majority desire to shift the meeting time a bit - this formally moves it 2 hours later on the same days as now | 14:08 |
| mattmceuen | So heads up -- today's meeting may be the last one at the current time! However, I think it just depends on when the PS merges. | 14:08 |
| mattmceuen | So reviews on that PS are appreciated - thanks | 14:09 |
| mattmceuen | #topic Onboarding Environments / Dev Documentation | 14:10 |
| *** openstack changes topic to "Onboarding Environments / Dev Documentation (Meeting topic: airship)" | 14:10 | |
| mattmceuen | Hey sthussey, you were out last week but we talked about something you added to the agenda - looping back: | 14:10 |
| mattmceuen | There is a variety of different tooling/yamls for standing up an Airship enviroment | 14:10 |
| mattmceuen | I put in a storyboard item to try to align those a bit based on our discussion last week: https://storyboard.openstack.org/#!/story/2005123 | 14:11 |
| mattmceuen | The storyboard description got a little verbose -- if anyone would like to discuss / nitpick it, I can copy/paste into a spec; but I'll only do that if needed or helpful :D | 14:11 |
| mattmceuen | In any case - feedback welcome from all on that (as is implementation!) | 14:12 |
| mattmceuen | I think we brought up last week dwalt that you're working on aligning airskiff site to the treasuremap globals -- how's that going? | 14:12 |
| sthussey | So we still want to maintain the automation for standing up 3 different flavors of dev/testing environments? | 14:13 |
| mattmceuen | so dwalt just lost his connection, he may not be answering the question :) | 14:13 |
| *** dustinspecker has joined #airshipit | 14:13 | |
| mattmceuen | the spec calls out aligning aiab single node tooling to the aiab multinode tooling to remove one of those | 14:13 |
| mattmceuen | not spec, storyboard | 14:13 |
| mattmceuen | as single node is a special case of multinode | 14:14 |
| sthussey | Okay, I misread that as only the demo environment | 14:14 |
| *** stingerpk has joined #airshipit | 14:14 | |
| roman_g | o/ | 14:15 |
| roman_g | Hi. Just came. Sorry for being late. | 14:15 |
| mattmceuen | I think (anyone correct me if wrong) that the single-node demo env is really the one that gets a lot of use, and the single-node dev site doesn't get a lot of use (since devs often use multinode aiab) - anyone know otherwise? | 14:15 |
| mattmceuen | o/ roman_g! no worries | 14:15 |
| sthussey | Can we note in there deleting the dev-minimal then? | 14:16 |
| mattmceuen | sure thing - if anyone has any concerns between now & impl time, please bring it up; if so we can just migrate the dev-minimal site at the same time | 14:17 |
| *** michael-beaver has joined #airshipit | 14:17 | |
| dwalt | Sorry for the connection issues. The patch to align Airskiff with the UCP global documents in Treasuremap is here: https://review.openstack.org/635218 | 14:17 |
| mattmceuen | As part of the discussion last week, also came up that it would be valuable to have some additional developer-centric documentation. I created a story / tasks for that as well: https://storyboard.openstack.org/#!/story/2005122 | 14:18 |
| dwalt | I haven't been able to revisit it this week, but it's very close. There are just a few secret substitutions that need to be cleaned up | 14:18 |
| mattmceuen | awesome - ty dwalt | 14:18 |
| *** stingerpk has quit IRC | 14:19 | |
| mattmceuen | Interested on feedback on the content of the dev doc story. Especially from folks who need it the most! If it doesn't answer the right questions etc, please let me know and we can add more tasks | 14:19 |
| roman_g | Will have a look | 14:19 |
| levmorgan | Will do. | 14:20 |
| mattmceuen | I made the tasks a bit granular because different authors have experience in different areas. If anyone would like to author some content that is not called out as a task yet, though, please go ahead and add a task | 14:20 |
| mattmceuen | I plan to tackle some of the content and would appreciate as much collaboration as possible :D | 14:20 |
| mattmceuen | thanks guys | 14:20 |
| mattmceuen | #topic Divingbell gates are broken in upstream | 14:20 |
| *** openstack changes topic to "Divingbell gates are broken in upstream (Meeting topic: airship)" | 14:20 | |
| mattmceuen | roman_g: I know you discussed this a bit with the OSH folks; where does this sit now - still trying to root cause the specific issue, right? | 14:21 |
| roman_g | Roman is here. Slowly progressing. | 14:21 |
| mattmceuen | Is there anything other folks can do to assist you in that? | 14:22 |
| roman_g | I know that behavior changes between 2.11.0 and 2.12.0.rc1 | 14:22 |
| roman_g | Wel,, may be a quick Zoom session, to show me how i can quickly test different versions of helm/tiller | 14:22 |
| *** lemko has quit IRC | 14:22 | |
| roman_g | Would appreciate a lot. | 14:22 |
| sthussey | is this the issue of how maps are handled? | 14:23 |
| roman_g | yes. | 14:23 |
| roman_g | patch from cheng1 did not help | 14:23 |
| sthussey | Might also want to check with what version of Go tiller was built. May be a underlying change in Go's behavior | 14:24 |
| roman_g | I can compile/recompile helm/tiller very fast, but I did not get to a point where I could easily use newly compiled helm/tiller for the chart renderring | 14:24 |
| roman_g | I have go version go1.10.4 linux/amd64 | 14:25 |
| mattmceuen | roman_g I'll set up a troubleshooting session & send out on the ML. seaneagan if you can join I think that would be helpful too. I'm thinking it'll probably be Thurs or Fri | 14:25 |
| roman_g | latest on Ubuntu 16.04.5 LTS | 14:25 |
| mattmceuen | #action mattmceuen: set up troubleshooting session for helm mapping issue impacting divingbell gates | 14:26 |
| roman_g | thank you, mattmceuen | 14:26 |
| mattmceuen | thank you roman_g for tackling this | 14:26 |
| roman_g | sthussey: good advice. Thanks. | 14:26 |
| mattmceuen | #topic Deckhand & Shipyard - OpenSUSE builds | 14:27 |
| *** openstack changes topic to "Deckhand & Shipyard - OpenSUSE builds (Meeting topic: airship)" | 14:27 | |
| roman_g | arunkant: | 14:27 |
| roman_g | james is not here | 14:27 |
| roman_g | Arun, are you here? | 14:27 |
| mattmceuen | This one may be a good one for the ML :) | 14:28 |
| roman_g | #action followup in ML for OpenSUSE builds of Airship | 14:29 |
| mattmceuen | ty roman_g | 14:29 |
| mattmceuen | #topic Minimirror: move spec and review on the fly patches | 14:29 |
| *** openstack changes topic to "Minimirror: move spec and review on the fly patches (Meeting topic: airship)" | 14:29 | |
| dwalt | thanks for adding this again roman_g | 14:29 |
| roman_g | :D | 14:30 |
| roman_g | would you have a look? | 14:30 |
| dwalt | There has been some new developments in this area, as it looks like we will soon be able to publish a mini-mirror image from openstack-helm-images. That means we can integrate this in AIAB and seaworthy | 14:30 |
| dwalt | as for the spec, I still need to remove that. Thanks for the reminder | 14:31 |
| mattmceuen | oh, that's awesome | 14:31 |
| roman_g | it's piblished in openstack-helm-images already, as far as I can see | 14:31 |
| dwalt | thank you evrardjp as well for your work in osh-images so that this can be possible! | 14:31 |
| mattmceuen | +1 | 14:31 |
| roman_g | but there are some patches from Alex Volkov, which are non-merged, and will not be merged (because minimirror is in different repo now). For example, there is a good manual for the minimirrod. | 14:32 |
| roman_g | *or | 14:32 |
| mattmceuen | yeah, those will need to be abandoned or reworked against the new repo as appropriate | 14:33 |
| mattmceuen | will take a look roman_g | 14:33 |
| roman_g | thank you | 14:33 |
| mattmceuen | alrighty, moving on: | 14:33 |
| mattmceuen | #topic Airship talks accepted into the Denver Summit | 14:33 |
| *** openstack changes topic to "Airship talks accepted into the Denver Summit (Meeting topic: airship)" | 14:33 | |
| roman_g | #link https://review.openstack.org/#/q/minimirror - some of them seem to be good to be integrated into the openstack-helm-images repo | 14:33 |
| mattmceuen | oops | 14:33 |
| roman_g | sorry | 14:34 |
| mattmceuen | my bad roman_g, pulled the trigger too early | 14:34 |
| mattmceuen | https://etherpad.openstack.org/p/airship-meeting-2019-03-05 | 14:34 |
| mattmceuen | ^ that has an extract of the Airship-related sessions at the upcoming summit, FYI | 14:35 |
| mattmceuen | Some of them by us in the meeting, some by other folks, and some are cross-project presentations that have tagged #airship | 14:35 |
| mattmceuen | But good to see a lot of content and variety planned for the open infra summit around Airship! | 14:35 |
| *** c06 has joined #airshipit | 14:36 | |
| mattmceuen | Any other thoughts on this topic? | 14:36 |
| roman_g | Congrats to winners :) | 14:37 |
| roman_g | Look forward to watch the recording on youtube. | 14:37 |
| mattmceuen | +1 | 14:37 |
| mattmceuen | #topic Ironic driver spec | 14:37 |
| *** openstack changes topic to "Ironic driver spec (Meeting topic: airship)" | 14:37 | |
| mattmceuen | we have hemanth_n here - pas-ha I believe we wanted to sync up a bit or have some Q&A on the Ironic spec, right? | 14:38 |
| mattmceuen | I think we may be missing pas-ha here today | 14:39 |
| mattmceuen | I believe the open questions were around Ironic functions not being exactly 1:1 with MaaS functions, and how to bridge the gap from a plugin perspective | 14:41 |
| mattmceuen | There is discussion around that in the Spec, but I'm not certain what the outstanding questions are that needed to be discussed here | 14:41 |
| hemanth_n | I will take a look at spec and will try to answer them ... and probably will reach to pas-ha | 14:42 |
| mattmceuen | thanks hemanth_n - that would be awesome | 14:42 |
| mattmceuen | Spec, for anyone who's interested: https://review.openstack.org/#/c/613358/ | 14:42 |
| sthussey | It looks like someone from Dell also has some input on that spec | 14:42 |
| mattmceuen | yep Prakash | 14:43 |
| mattmceuen | He has been providing feedback on the spec as well and has expressed interest in collaborating on Ironic - Airship integration | 14:44 |
| mattmceuen | We'll continue this on the spec review proper - thx guys | 14:44 |
| mattmceuen | #topioc 1.0 release | 14:45 |
| mattmceuen | This is your hogepodge! | 14:45 |
| hogepodge | I added the topic | 14:45 |
| roman_g | #topic 1.0 release | 14:45 |
| mattmceuen | sigh | 14:45 |
| mattmceuen | #topic 1.0 release | 14:45 |
| hogepodge | We're looking at scheduling for the time between now and the summit, and the 1.0 release is supposed to happen in there | 14:45 |
| *** openstack changes topic to "1.0 release (Meeting topic: airship)" | 14:45 | |
| hogepodge | I wanted to check on the status of the release, how it might bump up against the OpenStack Stein release, and other issues around it. | 14:46 |
| hogepodge | We also have things like Foundation Newsletter and demos and work on the bare metal whitepaper that will also need to take the release into accound. | 14:47 |
| hogepodge | account. | 14:47 |
| mattmceuen | Good add. Time is definitely flying between now and the summit. | 14:47 |
| mattmceuen | Initial Ironic integration is the biggest thing we wanted to have in place for the 1.0 release | 14:48 |
| mattmceuen | So that is one we need to stay on top of | 14:48 |
| hogepodge | Will Ironic cause it to slip past the summit? | 14:49 |
| mattmceuen | Remains to be seen, I am optimistic. We don't need the integration to be 100% full-featured for the release IMO, but we did express it as something we should have in place. | 14:51 |
| mattmceuen | I will sync up with pas-ha and get a better answer | 14:51 |
| mattmceuen | Another area we wanted to firm up a bit is the "ease of use" around YAMLs. I think we've come a long way since Berlin in a couple areas: | 14:52 |
| mattmceuen | 1) the treasuremap repo has continued to mature, and we're publishing monthly releases that can be used as a basis for deployments | 14:53 |
| mattmceuen | 2) The airship-spyglass project can be used to generate site configurations from xls files | 14:53 |
| hogepodge | ok | 14:54 |
| hogepodge | I think the preference would be to have a release with fewer features than a late release, mostly because of the summit platform | 14:55 |
| hogepodge | but we can't have a broken release | 14:55 |
| mattmceuen | Documentation was an additional area to firm up for 1.0. The dev documentation brought up earlier is a good foundation for that. | 14:55 |
| hogepodge | we can talk more at the community meeting this Friday | 14:55 |
| hogepodge | (which is another topic to slip in under the wire) | 14:55 |
| mattmceuen | Roger hogepodge - will proceed with that in mind | 14:55 |
| mattmceuen | :D | 14:55 |
| mattmceuen | go for it | 14:55 |
| hogepodge | There's and OSF/Airship community meeting this Friday. I'll send details to the mailing list. | 14:56 |
| hogepodge | For now we'll start weekly, but will move to biweekly if it makes more sense. | 14:56 |
| mattmceuen | Sounds like a plan - ty for setting that up | 14:56 |
| *** c06 has quit IRC | 14:56 | |
| mattmceuen | #topic Roundtable | 14:57 |
| *** openstack changes topic to "Roundtable (Meeting topic: airship)" | 14:57 | |
| mattmceuen | Just a couple mins left -- | 14:57 |
| mattmceuen | https://review.openstack.org/#/c/615387/ | 14:57 |
| mattmceuen | https://review.openstack.org/#/c/635507/ - openstack/airship-treasuremap - Add cache for results of requests to quay.io in Updater tool | 14:57 |
| mattmceuen | ^ if we can get some review on these that would be awesome | 14:57 |
| mattmceuen | Anything else to discuss today? | 14:57 |
| levmorgan | Well | 14:58 |
| levmorgan | Does anyone know about the coverage threshold in Zuul being raised to 86 percent (at least for Pegleg?) | 14:58 |
| sthussey | I saw a PS that did that | 14:59 |
| dwalt | #link https://review.openstack.org/634593/ | 14:59 |
| Nishant_ | I see it 86 percent here - https://github.com/openstack/airship-pegleg/blob/master/tox.ini#L63 | 15:00 |
| levmorgan | Thanks! | 15:00 |
| mattmceuen | Looks like tests were added as part of that change so that is good :) | 15:01 |
| dwalt | +1 | 15:01 |
| mattmceuen | thanks lamt! | 15:01 |
| sthussey | https://review.openstack.org/#/c/634593/6/tox.ini | 15:01 |
| mattmceuen | we our out of time! | 15:01 |
| mattmceuen | *are | 15:01 |
| mattmceuen | #endmeeting | 15:01 |
| *** openstack changes topic to "airshipit.org || General Review Dashboard: https://review.openstack.org/#/q/project:%255Eopenstack/airship.*+status:open,n,z" | 15:01 | |
| openstack | Meeting ended Tue Mar 5 15:01:48 2019 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:01 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/airship/2019/airship.2019-03-05-14.00.html | 15:01 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/airship/2019/airship.2019-03-05-14.00.txt | 15:01 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/airship/2019/airship.2019-03-05-14.00.log.html | 15:01 |
| mattmceuen | Thanks everybody! | 15:01 |
| roman_g | Thanks. | 15:02 |
| *** michael-beaver has quit IRC | 15:07 | |
| *** juhak has quit IRC | 15:14 | |
| *** juhak has joined #airshipit | 15:15 | |
| *** juhak has quit IRC | 15:16 | |
| *** juhak has joined #airshipit | 15:16 | |
| *** levmorgan has quit IRC | 15:27 | |
| *** michael-beaver has joined #airshipit | 15:27 | |
| *** stingerpk has joined #airshipit | 15:28 | |
| *** openstackgerrit has quit IRC | 15:28 | |
| *** stingerpk has quit IRC | 15:30 | |
| *** juhak has quit IRC | 15:55 | |
| *** juhak_ has joined #airshipit | 15:56 | |
| *** michael-beaver has quit IRC | 15:56 | |
| *** openstackgerrit has joined #airshipit | 16:00 | |
| openstackgerrit | Alexander Hughes proposed openstack/airship-pegleg master: Update tox coverage to 85 percent https://review.openstack.org/641063 | 16:00 |
| *** michael-beaver has joined #airshipit | 16:18 | |
| *** michael-beaver has quit IRC | 16:22 | |
| *** michaelbeaver has joined #airshipit | 16:22 | |
| *** michaelbeaver has quit IRC | 16:26 | |
| *** hemanth_n has quit IRC | 16:27 | |
| *** hemanth_n has joined #airshipit | 16:29 | |
| *** hemanth_n has quit IRC | 16:31 | |
| openstackgerrit | Merged openstack/airship-pegleg master: Update tox coverage to 85 percent https://review.openstack.org/641063 | 16:35 |
| roman_g | https://review.openstack.org/#/c/635507/ - please, review. openstack/airship-treasuremap - Add cache for results of requests to quay.io in Updater tool | 16:41 |
| roman_g | Thank you. | 16:41 |
| roman_g | https://review.openstack.org/#/c/640359/ - please, review. openstack-infra/irc-meetings - Update Airship meeting time, change chair | 16:43 |
| roman_g | Especially cores. Thank you. | 16:43 |
| openstackgerrit | Lev Morgan proposed openstack/airship-pegleg master: Search all scopes for catalogs https://review.openstack.org/634821 | 16:48 |
| *** roman_g has quit IRC | 16:51 | |
| *** roman_g has joined #airshipit | 16:53 | |
| -openstackstatus- NOTICE: Gerrit is being restarted for a configuration change, it will be briefly offline. | 17:09 | |
| *** michael-beaver has joined #airshipit | 17:10 | |
| *** peyunco has joined #airshipit | 17:50 | |
| roman_g | Ou, yeah. I got to understanding on how to quickly test helm/tiller. | 18:11 |
| openstackgerrit | Nishant Kumar proposed openstack/airship-maas master: Support for password rotation https://review.openstack.org/634981 | 18:46 |
| openstackgerrit | Nishant Kumar proposed openstack/airship-shipyard master: [Database] Shipyard DB changes https://review.openstack.org/636147 | 18:49 |
| openstackgerrit | Alexander Hughes proposed openstack/airship-pegleg master: PKI Cert generation and check updates https://review.openstack.org/639414 | 18:51 |
| openstackgerrit | Merged openstack/airship-deckhand master: Use helm-toolkit for DB initialization https://review.openstack.org/639218 | 19:21 |
| openstackgerrit | Merged openstack/airship-drydock master: Use helm toolkit for DB initialization https://review.openstack.org/634625 | 20:46 |
| *** georgk has quit IRC | 20:52 | |
| *** georgk has joined #airshipit | 20:53 | |
| roman_g | commit which broke overrides functionality (used in divingbell, used in openstack-helm-infra, used in gates) is this one: https://github.com/helm/helm/commit/de5a6a931bfc47c1fa3a72db038002cbf68315ff | 21:01 |
| roman_g | changeing github.com/imdario/mergo from v0.3.5 back to 6633656539c1639d9d78127b7d47c622b5d7b6dc in glide.yaml and rebuilding latest helm does not make anything better (sugestion from Pete Birley) | 21:02 |
| roman_g | I'm tracking issue here https://storyboard.openstack.org/#!/story/2005095 (there are some basic scripts to get to tests). | 21:04 |
| openstackgerrit | Michael Beaver proposed openstack/airship-armada master: Enable Armada to acquire Tiller IP from config file https://review.openstack.org/633237 | 21:11 |
| openstackgerrit | Michael Beaver proposed openstack/airship-armada master: Move Tiller into Armada pod https://review.openstack.org/632788 | 21:11 |
| openstackgerrit | Michael Beaver proposed openstack/airship-armada master: Normalize assignment of Tiller defaults https://review.openstack.org/641126 | 21:11 |
| openstackgerrit | Anthony Bellino proposed openstack/airship-divingbell master: [WIP] Initial commit for Ansible framework https://review.openstack.org/639186 | 21:11 |
| openstackgerrit | Anthony Bellino proposed openstack/airship-divingbell master: [WIP] Initial Ansible Daemonset https://review.openstack.org/640539 | 21:18 |
| *** peyunco has quit IRC | 21:36 | |
| dustinspecker | Hello. Using Shipyard's test_site action. When the helm test fails, Armada logs have the fact it failed, but Shipyard reports the step armada_test_releases as success. It used to report this as fail/error. Is this an intentional change? | 21:55 |
| openstackgerrit | Rahul Khiyani proposed openstack/airship-maas master: UCP:readOnlyRootFilesystem: true for Maas chart https://review.openstack.org/639200 | 22:00 |
| openstackgerrit | Rahul Khiyani proposed openstack/airship-promenade master: UCP:readOnlyRootFilesystem: true for promenade, etcd and K8s charts https://review.openstack.org/639189 | 22:01 |
| openstackgerrit | Rahul Khiyani proposed openstack/airship-shipyard master: UCP:readOnlyRootFilesystem: true for shipyard and airflow charts https://review.openstack.org/639195 | 22:01 |
| openstackgerrit | Rahul Khiyani proposed openstack/airship-drydock master: UCP:readOnlyRootFilesystem: true for drydock chart https://review.openstack.org/639197 | 22:02 |
| dustinspecker | Is it possible this could happen when using a newer Shipyard CLI (docker image) than the shipyard-api pod and armada-api pods? Not sure if there was an update between the two that I'm just out of sync on. | 22:04 |
| dwalt | dustinspecker: I believe it should always be a success. That step is supposed to report whether or not Helm tests were executed successfully, not whether the tests passed | 22:20 |
| dwalt | It also logs the direct Armada API response, which *should* match the correct result of the test | 22:21 |
| dustinspecker | Interesting. I thought at one point if the test pod existed, it came back as fail/error. Now it's coming back as success. | 22:21 |
| dustinspecker | Thank you! | 22:21 |
| dwalt | do you have the exact output you can share? | 22:22 |
| dwalt | Here is the code in question: https://github.com/openstack/airship-shipyard/blob/master/src/bin/shipyard_airflow/shipyard_airflow/plugins/armada_test_releases.py#L67 | 22:22 |
| dustinspecker | Hmm. Thank you. I really thought it came back with fail/error. Thanks for info - I'm probably misremembering. | 22:23 |
| dwalt | dustinspecker: np! It's definitely possible it could report a failure, but I think that's indicative of a failure to run the test rather than a failure of the test itself | 22:24 |
| dustinspecker | Yeah, in this case it was because the test pod existed, so helm status says failed. Just surprised Shipyard/Armada came back as success. Was convinced it didn't do that. | 22:25 |
| Nishant_ | does the shipyard step - `armada_build` also shows as success, ideally in this case it should say failed | 22:33 |
| Nishant_ | dustinspecker: ^^ | 22:35 |
| dustinspecker | nishant: There is not an `armada_build` step. Every step is success. | 22:37 |
| dustinspecker | nishant: this is for test_site action. This is an action to trigger a helm test on demand. | 22:38 |
| *** dustinspecker has quit IRC | 22:44 | |
| Nishant_ | dustinspecker: ok I have not had a chance to playaround with the `test_site` action but during a `deploy_site` or an `update_site` a similar issue does result in the `armada_build` step marked as failed which runs the test. | 22:51 |
| Nishant_ | ok i did run a `test_site` action and I see same behaviour as seen by dustinspecker i.e. some tests fail stating the test already exist but the step indicates to be a success and as dwalt mentioned it is the expected behaviour. | 23:07 |
| dwalt | NISHANT_: thanks for looking into that! | 23:07 |
| openstackgerrit | Dmitrii Kabanov proposed openstack/airship-maas master: Add package repositories https://review.openstack.org/636848 | 23:10 |
| Nishant_ | dwalt: Do you think it can make more sense to mark the step as a failure or add another step which would validate if all the tests passed or not? If not passed then step should be marked as failed causing the `test_site` action to fail? | 23:12 |
| dwalt | That would probably be more in-line with the behavior of armada_build | 23:12 |
| dwalt | Maybe we can propose a storyboard item for this? The test_site action is due for changes soon since Armada will be changing its testing interface | 23:13 |
| Nishant_ | ok sounds like it can be a point of discussion | 23:16 |
| *** sgrasley has quit IRC | 23:32 | |
| *** michael-beaver has quit IRC | 23:33 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!