Monday, 2020-05-11

*** uzumaki has quit IRC03:35
*** uzumaki has joined #airshipit03:37
*** evrardjp has quit IRC04:36
*** evrardjp has joined #airshipit04:36
*** GoldenBear has quit IRC06:06
*** GoldenBear has joined #airshipit06:07
*** dpawlik has joined #airshipit06:10
*** dpawlik has quit IRC06:13
*** dpawlik has joined #airshipit06:18
openstackgerritAhmad Mahmoudi proposed airship/treasuremap master: (fix) Added retries for genesis deploy  https://review.opendev.org/72570707:24
*** uzumaki has quit IRC07:37
*** kukacz_ has joined #airshipit08:11
*** roman_g has joined #airshipit08:11
*** kukacz has quit IRC08:15
*** happyhemant has joined #airshipit10:15
*** toabctl has quit IRC11:43
openstackgerritAlexander Hughes proposed airship/airshipctl master: etcd encryption addition  https://review.opendev.org/72081212:15
*** SRao has joined #airshipit13:15
*** mikefix has joined #airshipit13:23
*** dpawlik has quit IRC13:38
*** dpawlik has joined #airshipit13:42
openstackgerritDrew Walters proposed airship/airshipctl master: Update stale README and reformat docs  https://review.opendev.org/72646613:43
*** uzumaki has joined #airshipit14:01
airship-irc-bot1<rp2723> All dig-airship-yaml call is canceled today. Cheers..14:03
airship-irc-bot1<rp2723> Meant sig of course :-)14:03
openstackgerritTrung Thai proposed airship/porthole master: Create an initial Automation Validation Testing framework  https://review.opendev.org/72011314:32
*** jhesketh has quit IRC14:35
*** uzumaki has quit IRC14:37
*** uzumaki has joined #airshipit14:37
openstackgerritsai battina proposed airship/airshipctl master: WIP: Adds support for `clusterctl move` of capi and bmo resources from ephermeral to workload cluster.  https://review.opendev.org/72494014:41
*** aostapenko has joined #airshipit14:45
*** sreejithp has joined #airshipit14:45
uzumakioh, alright, thanks for the heads-up rodolpho!14:57
*** uzumaki has quit IRC15:03
openstackgerritStas Egorov proposed airship/airshipctl master: [#176] Fixed config loader bug  https://review.opendev.org/72440515:05
-openstackstatus- NOTICE: Our CI mirrors in OVH BHS1 and GRA1 regions were offline between 12:55 and 14:35 UTC, any failures there due to unreachable mirrors can safely be rechecked15:06
openstackgerritMerged airship/airshipctl master: Reusing apache-server instead of httpd  https://review.opendev.org/72064315:39
openstackgerritMerged airship/airshipctl master: Add pluralized aliases to config get commands  https://review.opendev.org/72596815:39
*** uzumaki has joined #airshipit15:43
openstackgerritDrew Walters proposed airship/airshipctl master: Update stale README and reformat docs  https://review.opendev.org/72646615:49
openstackgerritIan Howell proposed airship/airshipctl master: WIP: Refactor context management  https://review.opendev.org/72593715:50
openstackgerritIan Howell proposed airship/airshipctl master: WIP: Refactor context management  https://review.opendev.org/72593715:58
*** ianychoi_ is now known as ianychoi16:09
airship-irc-bot1<ih616h> I'm refactoring contexts to separate out ephemeral vs target - would it make sense to move `Bootstrap` info to the Context at this point? https://github.com/airshipit/airshipctl/blob/master/pkg/config/cluster.go#L3916:09
airship-irc-bot1<ih616h> my thinking there is that it doesn't make sense for a cluster to contain bootstrap info (since a cluster can be ephemeral)16:10
openstackgerritMerged airship/airshipctl master: Fix image push zuul job  https://review.opendev.org/72647316:14
openstackgerritIan Howell proposed airship/airshipctl master: WIP: Refactor context management  https://review.opendev.org/72593716:14
airship-irc-bot1<rp2723> I agree the  notion of bootstrap at the moment appears to be contextual to the site or cluster but only applies to one cluster type , however I am wondering if the image build notion that Pete is working on would leverage any of this bootstrap info and therefore we might want to wait for that to shape up before refactoring this.16:20
aostapenkoHello. I'm working on implementation on probe functionality for airship components allowing to monitor execution time for whole requests, functions and parts of code. This intends to allow devs to easily identify slow parts to make their code faster. Here's the first example for deckhand - https://review.opendev.org/#/c/725738/, example of logs16:20
aostapenkohttps://761cde46fbe79b9b70c4-8c9d0d00628f80269f5780063c8a8810.ssl.cf5.rackcdn.com/725740/26/check/deckhand-airskiff-deployment/04bdcf1/primary/debug/debug-ubuntu-bionic-rax-ord-0016478245/pod-logs/ucp/deckhand-api-7968f94d4-2hpx2/deckhand-api.txtHowever it has sense to me to make this code common across airship components putting it to new library.16:20
aostapenkoI suggest to create new utils (or whatever) project under the airship namespace and, either package it as airship-utils or, preferrably, install via git+http from static commit or tag (as using master will require extended heavy ci for new component itself) and toggle on new functionality in airskiff job.For security reasons removed blind logging16:20
aostapenkoof arguments for decorated functions. Another mechanism allowing to log predefined named arguments is introduced.16:20
airship-irc-bot1<ih616h> @rp2723 so for the moment, we should simply make `CurrentContextBootstrapInfo` default to using the target cluster? https://github.com/airshipit/airshipctl/blob/master/pkg/config/config.go#L86916:23
*** uzumaki has quit IRC16:32
*** evrardjp has quit IRC16:36
*** evrardjp has joined #airshipit16:36
*** SRao has quit IRC16:39
openstackgerritStas Egorov proposed airship/airshipctl master: [#116]: added sub-commands to set/get bootstrap info  https://review.opendev.org/72057616:44
airship-irc-bot1<rp2723> @ih616h , the bootstrap info is for the ephemeral cluster , for the image etc .. so it should not be target by default .16:47
airship-irc-bot1<kk6740> what about power management commands ?16:48
airship-irc-bot1<kk6740> does bootstrap info contain anything for remotedirect?16:48
airship-irc-bot1<rp2723> It’s has a section but this is a question for drew did his changes now rely on that management config instead16:49
openstackgerritStas Egorov proposed airship/airshipctl master: Fixed local gate script deps  https://review.opendev.org/72690316:51
airship-irc-bot1<kk6740> so bootstrap seems to related only to ephemeral host16:51
airship-irc-bot1<mattmceuen> @aostapenko which code are you looking to share across airship 1 projects -- probe.py and probe_filter.py from your deckhand patchset above?  Or is there additional code?17:00
aostapenkoprobe.py and probe_filter.py only17:00
airship-irc-bot1<mattmceuen> got it, thanks.  I can bring that up as a discussion topic in the team meeting tomorrow and see what people think17:01
airship-irc-bot1<mattmceuen> I'm not sure whether it makes sense to add another project just for that (since it's not a whole lot of code, and won't change frequently), but you could make an argument either way17:02
*** mikefix has quit IRC17:45
*** jamesgu has joined #airshipit18:01
*** happyhemant has quit IRC18:05
aostapenkomattmceuen it's all true if we don't expect frequent changes and don't expect creating other common functionality, that is probably true at this stage of development. My code management heart is still bleeding, but lets continue without - library may be created at any time if needed18:29
openstackgerritDmitry Ukov proposed airship/airshipctl master: [WIP] Add control plane deployment  https://review.opendev.org/71475818:31
*** roman_g has quit IRC18:32
openstackgerritSirajudeen proposed airship/airshipctl master: [WIP] Fix job update-github-issues  https://review.opendev.org/72692318:32
*** roman_g has joined #airshipit18:33
openstackgerritDmitry Ukov proposed airship/airshipctl master: Add CAPI initialization steps to gating  https://review.opendev.org/72564318:35
openstackgerritDmitry Ukov proposed airship/airshipctl master: TST logs  https://review.opendev.org/72650218:35
openstackgerritDmitry Ukov proposed airship/airshipctl master: [WIP] Add control plane deployment  https://review.opendev.org/71475818:35
openstackgerritDmitry Ukov proposed airship/airshipctl master: [WIP] Add control plane deployment  https://review.opendev.org/71475818:42
openstackgerritMatt McEuen proposed airship/airshipctl master: Write kustomize plugin errors to stderr  https://review.opendev.org/72692618:46
airship-irc-bot1<mattmceuen> Sure, sounds good to me @aostapenko - we can always decide to deduplicate those in the future too, without much additional work.   Thanks for focusing on code quality and reuse :)18:47
*** toabctl has joined #airshipit18:57
openstackgerritMatt McEuen proposed airship/airshipctl master: Write cli errors to stderr  https://review.opendev.org/72692619:00
openstackgerritMatt McEuen proposed airship/airshipctl master: Write cli errors to stderr  https://review.opendev.org/72692619:03
openstackgerritSirajudeen proposed airship/airshipctl master: [WIP] Fix job update-github-issues  https://review.opendev.org/72692319:24
openstackgerritAndrew Schiefelbein proposed airship/airshipui master: [WIP] POC pluggable auth method  https://review.opendev.org/72693719:38
openstackgerritAndrew Schiefelbein proposed airship/airshipui master: [WIP] POC pluggable auth method  https://review.opendev.org/72693719:47
openstackgerritSirajudeen proposed airship/airshipctl master: [WIP] Fix job update-github-issues  https://review.opendev.org/72692319:48
openstackgerritAndrew Schiefelbein proposed airship/airshipui master: [WIP] POC pluggable auth method  https://review.opendev.org/72693719:57
openstackgerritMerged airship/airshipctl master: Fix pull documents for non-master branch  https://review.opendev.org/72493520:02
openstackgerritMerged airship/airshipctl master: Write cli errors to stderr  https://review.opendev.org/72692620:07
openstackgerritAndrew Schiefelbein proposed airship/airshipui master: [WIP] POC pluggable auth method  https://review.opendev.org/72693720:32
openstackgerritDrew Walters proposed airship/airshipctl master: Fix system power regressions  https://review.opendev.org/72694720:34
*** timClicks has joined #airshipit20:37
*** sreejithp has quit IRC20:41
*** dpawlik has quit IRC20:46
openstackgerritDmitry Ukov proposed airship/airshipctl master: [WIP] Add control plane deployment  https://review.opendev.org/71475821:01
openstackgerritDrew Walters proposed airship/airshipui master: Publish documentation  https://review.opendev.org/72695021:07
*** GoldenBear_ has joined #airshipit21:07
*** GoldenBear has quit IRC21:08
openstackgerritDrew Walters proposed airship/airshipui master: Publish documentation  https://review.opendev.org/72695021:09
openstackgerritDrew Walters proposed airship/airshipui master: Publish documentation  https://review.opendev.org/72695021:22
airship-irc-bot1<dwalt> Hey team. Our airshipctl documentation has fallen out of alignment, and I've made an attempt to resurrect some of it post-alpha. I'd appreciate your reviews on it https://review.opendev.org/72646621:27
openstackgerritSirajudeen proposed airship/airshipctl master: Moving update-github-issues from post to promote  https://review.opendev.org/72692321:33
openstackgerritIan Howell proposed airship/airshipctl master: WIP: Refactor context management  https://review.opendev.org/72593721:34
openstackgerritMerged airship/treasuremap master: Remove treasuremap from airship docs main page  https://review.opendev.org/72641821:36
openstackgerritDmitry Ukov proposed airship/airshipctl master: [WIP] Add control plane deployment  https://review.opendev.org/71475821:40
openstackgerritSophie Huang proposed airship/porthole master: Add curl to mysql utility  https://review.opendev.org/72695621:43
openstackgerritAndrew Schiefelbein proposed airship/airshipui master: [WIP] POC pluggable auth method  https://review.opendev.org/72693721:48
openstackgerritMerged airship/airshipui master: Publish documentation  https://review.opendev.org/72695022:00
*** aostapenko has quit IRC22:07
openstackgerritdiwakar thyagaraj proposed airship/porthole master: Enable Deployment consistency across all UC  https://review.opendev.org/72511722:13
openstackgerritDrew Walters proposed airship/docs master: WIP: Add development guide  https://review.opendev.org/72696022:14
*** KeithMnemonic has joined #airshipit22:14
openstackgerritDrew Walters proposed airship/docs master: WIP: Add development guide  https://review.opendev.org/72696022:25
openstackgerritPrateek Dodda proposed airship/drydock master: Enabling Apparmor for drydock init containers  https://review.opendev.org/72476822:35
openstackgerritPrateek Dodda proposed airship/drydock master: Enabling Apparmor for drydock init containers  https://review.opendev.org/72476822:36
openstackgerritMatt McEuen proposed airship/airshipctl master: RT: add error handling for missing target docs  https://review.opendev.org/72696822:46
openstackgerritDrew Walters proposed airship/docs master: Add development guide  https://review.opendev.org/72696022:47
*** GoldenBear_ has quit IRC23:02
*** GoldenBear has joined #airshipit23:02
*** timClicks has quit IRC23:05
openstackgerritAnthony Bellino proposed airship/porthole master: Add host path and openstack/swift clients to etcdctl utility  https://review.opendev.org/72645723:29
openstackgerritPrateek Dodda proposed airship/drydock master: Enabling Apparmor for drydock init containers  https://review.opendev.org/72476823:52

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!