Thursday, 2020-05-14

openstackgerritPavel Basov proposed airship/airshipctl master: [WIP] Testing polling capabilties  https://review.opendev.org/72642300:02
openstackgerritMatt McEuen proposed airship/airshipctl master: [#177] Add Flux Helm Operator manifests  https://review.opendev.org/72029800:09
openstackgerritNishant Kumar proposed airship/deckhand master: Remove unused code for policy validation as feature not implemented  https://review.opendev.org/68129600:33
openstackgerritNishant Kumar proposed airship/deckhand master: Remove unused code for policy validation as feature not implemented  https://review.opendev.org/68129600:34
openstackgerritNishant Kumar proposed airship/deckhand master: Remove unused code for policy validation as feature not implemented  https://review.opendev.org/68129600:34
openstackgerritJames Gu proposed airship/airshipctl master: [#30] Implement document info command  https://review.opendev.org/70748302:16
*** uzumaki has quit IRC02:58
openstackgerritAlexander Noskov proposed airship/airshipctl master: Fix default checkout options in Airshipctl config  https://review.opendev.org/72747604:35
*** evrardjp has quit IRC04:36
*** evrardjp has joined #airshipit04:36
*** KeithMnemonic has quit IRC05:14
*** dpawlik has joined #airshipit06:06
openstackgerritNikolay Fedorov proposed airship/airshipctl master: [WIP] Add ephemeral function with ReplacementTransformer  https://review.opendev.org/72709507:19
*** uzumaki has joined #airshipit08:10
openstackgerritNikolay Fedorov proposed airship/airshipctl master: [WIP] Move transformer at site level  https://review.opendev.org/72796608:20
openstackgerritArnava proposed airship/airshipctl master: [issue # 188]implement proper file permissions on Airship2 generated files  https://review.opendev.org/72796908:32
openstackgerritNikolay Fedorov proposed airship/airshipctl master: [WIP] Change naming of nodes  https://review.opendev.org/72797008:36
*** dpawlik has quit IRC08:40
*** dpawlik has joined #airshipit08:45
*** born2bake has joined #airshipit08:49
openstackgerritDmitry Ukov proposed airship/airshipctl master: [WIP] Add control plane deployment  https://review.opendev.org/71475808:58
openstackgerritRuslan Aliev proposed airship/airshipctl master: Return an error if there is a problem to access config file  https://review.opendev.org/72582009:23
openstackgerritRuslan Aliev proposed airship/airshipctl master: Remove "cluster initinfra" command  https://review.opendev.org/72427609:29
openstackgerritDmitry Ukov proposed airship/airshipctl master: [WIP] Add control plane deployment  https://review.opendev.org/71475809:36
openstackgerritPavel Basov proposed airship/airshipctl master: [WIP] Testing polling capabilties  https://review.opendev.org/72642309:38
openstackgerritRuslan Aliev proposed airship/airshipctl master: Remove "cluster initinfra" command  https://review.opendev.org/72427609:44
openstackgerritArnava proposed airship/airshipctl master: WIP.....[issue # 188]implement proper file permissions on Airship2 generated files  https://review.opendev.org/72796910:13
openstackgerritArnava proposed airship/airshipctl master: {WIP][issue # 188]proper file permissions on  Airship2 generated files  https://review.opendev.org/72796910:20
openstackgerritDmitry Ukov proposed airship/airshipctl master: [WIP] Add control plane deployment  https://review.opendev.org/71475810:20
openstackgerritArnava proposed airship/airshipctl master: [WIP][issue # 188]proper file permissions on  Airship2 generated files  https://review.opendev.org/72796910:20
openstackgerritRuslan Aliev proposed airship/airshipctl master: Check python3-netaddr is installed within sushy-emulator role  https://review.opendev.org/72801910:32
*** avolkov has joined #airshipit10:37
openstackgerritRuslan Aliev proposed airship/airshipctl master: Add EnsureComplete config check during phase apply call  https://review.opendev.org/72802911:07
openstackgerritDmitry Ukov proposed airship/airshipctl master: [WIP] Add control plane deployment  https://review.opendev.org/71475811:27
*** happyhemant has joined #airshipit11:33
openstackgerritPavel Basov proposed airship/airshipctl master: [WIP] Testing polling capabilties  https://review.opendev.org/72642312:19
openstackgerritPavel Basov proposed airship/airshipctl master: [WIP] Testing polling capabilties  https://review.opendev.org/72642313:01
*** KeithMnemonic has joined #airshipit13:43
airship-irc-bot1<alexander.hughes> lf a review from a deckhand core please.  fixes gate issues blocking other work https://review.opendev.org/#/c/727343/13:47
openstackgerritMerged airship/airshipctl master: Fix default checkout options in Airshipctl config  https://review.opendev.org/72747614:05
*** mikefix has joined #airshipit14:29
*** uzumaki has quit IRC14:35
*** uzumaki has joined #airshipit14:39
*** mikefix has quit IRC14:48
*** evrardjp has quit IRC14:55
*** evrardjp has joined #airshipit14:56
*** jamesgu has quit IRC15:09
*** jamesgu_ has joined #airshipit15:09
openstackgerritStas Egorov proposed airship/airshipctl master: Fixed local gate script error  https://review.opendev.org/72690315:14
openstackgerritTrung Thai proposed airship/porthole master: Create an initial Automation Validation Testing framework  https://review.opendev.org/72011315:20
openstackgerritMerged airship/drydock master: Address PEP8 Failures in Drydock  https://review.opendev.org/72734315:26
openstackgerritVladimir Kozhukalov proposed airship/airshipctl master: Implement document rendering sub-command  https://review.opendev.org/69865815:28
*** jamesgu has joined #airshipit15:49
openstackgerritSirajudeen proposed airship/airshipctl master: Fix document pull with default airship config  https://review.opendev.org/72812416:04
openstackgerritJames Gu proposed airship/airshipctl master: [#24] Fix airshipctl config get-cluster filtering  https://review.opendev.org/70987316:09
mattmceuenPSA since it confused me a bit -- if you want to use the `airshipit.org/deploy-k8s` label to avoid pushing a resource to k8s, you need to put quotes around your "false".  Otherwise it gets interpreted as a boolean I suppose, which defeats our label-based filtering16:13
openstackgerritAlexander Hughes proposed airship/governance master: Clarification of tie breaking method  https://review.opendev.org/72812716:16
*** happyhemant has quit IRC16:22
openstackgerritAlexander Hughes proposed airship/airshipctl master: etcd encryption addition  https://review.opendev.org/72081216:25
*** dpawlik has quit IRC16:27
*** evrardjp has quit IRC16:33
*** evrardjp has joined #airshipit16:33
*** uzumaki has quit IRC16:36
openstackgerritMerged airship/porthole master: Enable Deployment configuration consistency across all UC  https://review.opendev.org/72511716:44
openstackgerritSirajudeen proposed airship/airshipctl master: Fix Lint warnings: Missing comments for functions  https://review.opendev.org/72780417:09
openstackgerritPrateek Dodda proposed airship/drydock master: Enabling Apparmor for drydock init containers  https://review.opendev.org/72476817:12
*** uzumaki has joined #airshipit17:18
openstackgerritAndrew Schiefelbein proposed airship/airshipui master: [WIP] POC pluggable auth method  https://review.opendev.org/72693717:27
openstackgerritStas Egorov proposed airship/airshipctl master: [#116]: added sub-commands to set/get bootstrap info  https://review.opendev.org/72057618:00
openstackgerritRyan Schroder proposed airship/airshipui master: Yaml Visualizer Project  https://review.opendev.org/72488818:16
airship-irc-bot1<kk6740> i’ve opened an issue regarding today’s discussion about manifest metadata, everyone is welcome to participate: https://github.com/airshipit/airshipctl/issues/24018:35
*** dpawlik has joined #airshipit18:51
openstackgerritAndrew Schiefelbein proposed airship/airshipui master: [WIP] POC pluggable auth method  https://review.opendev.org/72693719:13
openstackgerritAndrew Schiefelbein proposed airship/airshipui master: [WIP] POC pluggable auth method  https://review.opendev.org/72693719:27
*** avolkov has quit IRC19:28
openstackgerritAlexander Noskov proposed airship/airshipctl master: Create a directory for `baremetal isogen` artifacts.  https://review.opendev.org/72826419:36
openstackgerritAlexander Noskov proposed airship/airshipctl master: Change default repo from Treasuremap to Airshipctl  https://review.opendev.org/72826619:41
openstackgerritMatt McEuen proposed airship/airshipctl master: [WIP] Add a setting for kustomize plugins  https://review.opendev.org/72826819:43
airship-irc-bot1<mattmceuen> hey @ih616h or other config gurus: I want to add a setting for dropping a kustomize plugin folder underneath `$HOME/.airship` , and I'm going down a rabbit hole of passing around the settings object to everything that transitively calls NewBundle().  Before I complete this fun excercise can you confirm it's the right way to add that setting :slightly_smiling_face:19:46
*** dpawlik has quit IRC19:46
airship-irc-bot1<mattmceuen> The heart of it is here, but the tendrils reach out:  https://review.opendev.org/#/c/728268/1/pkg/document/bundle.go19:46
airship-irc-bot1<an310y> Hi, I see some guys can self-assign issue on a GitHub. Can I get these permissions to assign issues on myself? It will be much convenient to assign the issues prior to starting working on it.19:47
airship-irc-bot1<kk6740> @an310y, i think @alexander.hughes may help with this19:47
openstackgerritDmitry Ukov proposed airship/airshipctl master: [WIP] Add control plane deployment  https://review.opendev.org/71475819:47
airship-irc-bot1<kk6740> btw same problem for me19:47
airship-irc-bot1<alexander.hughes> there's items in flight to make this easier, in mean time I'm happy to assign please let me know what issue you're taking19:48
airship-irc-bot1<ih616h> @an310y https://github.com/ianpittwood/Gerrit-to-Github-Issues/pull/819:48
airship-irc-bot1<ih616h> ^the said items19:48
airship-irc-bot1<dwalt> @kk6740 as an Airship core, you should have permission to assign contributors. Let me make sure you get added :slightly_smiling_face:19:52
airship-irc-bot1<dwalt> @mattmceuen taking a look, but I may not be the config _guru_ you are seeking19:52
airship-irc-bot1<kk6740> thanks19:52
airship-irc-bot1<mattmceuen> dwalt you are also the docs guru, your head is heavy laden with many hats19:53
airship-irc-bot1<alexander.hughes> this tbh ^19:53
airship-irc-bot1<dwalt> lol19:54
uzumakiall these hats, he stoops as he walks, his countenance shading his sage eyes and his low grave voice emanating wisdom.. he's the docs guru xD19:55
airship-irc-bot1<ih616h> @mattmceuen couldn't you simply pass the path as a string?19:55
airship-irc-bot1<ih616h> that way you don't need to pass around the entire settings object19:56
airship-irc-bot1<ih616h> unless you think you're going to need the settings for seomthine else19:56
airship-irc-bot1<mattmceuen> @ih616h I think the blast radius of changes would be the same that way (touch a zillion files to pass around a string instead of a struct), but then also introducing a weird kustomize plugin string all over the code... might be better alternative to pass around the settings box in that case (and then other settings are handy if we need them sometime)19:57
airship-irc-bot1<ih616h> ah, makes sense19:58
airship-irc-bot1<mattmceuen> I mean an alternative would be to make settings global19:58
airship-irc-bot1<dwalt> So the intention is that this value should be specified by an environment variable or a CLI flag? Never by the config file?19:58
openstackgerritSirajudeen proposed airship/airshipctl master: Validate config before document pull  https://review.opendev.org/72827619:58
airship-irc-bot1<mattmceuen> or make the kustomize plugins path global19:58
airship-irc-bot1<mattmceuen> but that would break the conventions we have19:58
airship-irc-bot1<mattmceuen> @dwalt I thought about the config file, but it felt like it fits in better with the other bits that default to files under `.airship` -- the airship config and kube config.  If there's some reason putting it in config would help, though, doesn't seem bad20:00
airship-irc-bot1<mattmceuen> but yeah, as it stands in that PS, only way to tweak it is via env variable (I don't expect it'll be something that needs overriding on the cmd line)20:00
openstackgerritAlexander Noskov proposed airship/airshipctl master: Change default repo from Treasuremap to Airshipctl  https://review.opendev.org/72826620:01
openstackgerritIan Howell proposed airship/airshipctl master: Refactor context management  https://review.opendev.org/72593720:04
airship-irc-bot1<mattmceuen> I'll also let you guys tell me if there's no reason /not/ to make the settings global, though... that would result in less code than we have today :slightly_smiling_face:20:05
openstackgerritAlexander Hughes proposed airship/governance master: Clarification of tie breaking method  https://review.opendev.org/72812720:05
airship-irc-bot1<dwalt> @kk6740 invite sent20:05
openstackgerritAlexander Hughes proposed airship/airshipctl master: etcd encryption addition  https://review.opendev.org/72081220:05
airship-irc-bot1<alexander.hughes> we've got nothing but failures in https://zuul.opendev.org/t/openstack/builds?job_name=airship-airshipctl-gate-test&project=airship/airshipctl what looks like last 4 hours20:06
airship-irc-bot1<alexander.hughes> anyone looking at this?20:06
airship-irc-bot1<dwalt> @mattmceuen That makes sense to me. The only reason we put those paths in the root settings object, I presume, is because those paths cannot be specified in the config file. It sounds like this is in the same category20:08
airship-irc-bot1<alexander.hughes> last merged patch to airshipctl is https://review.opendev.org/#/c/727476/20:08
airship-irc-bot1<se6518> @alexander.hughes I see E: Unable to correct problems, you have held broken packages. during cloud-init install20:08
airship-irc-bot1<dwalt> e.g. how do you know where the airshipctl config file is if you can only get the path from the config itself?20:09
airship-irc-bot1<se6518> we are using unstable repos, so ...20:10
airship-irc-bot1<mattmceuen> yeah exactly.  As long as there's no use case for supporting two different configs within a single airshipctl run (which seems unlikely) I think we'd be ok with making those string global20:10
airship-irc-bot1<se6518> maybe it's time to think about repos?20:10
airship-irc-bot1<se6518> cloud-init : Depends: python3-jsonschema but it is not going to be installed20:12
airship-irc-bot1<kk6740> @dwalt thanks, got it20:12
airship-irc-bot1<mb551n> I am seeing that same cloud-init error in my local gate that I have been trying to get running20:13
airship-irc-bot1<se6518> anybody knows why do we use UNSTABLE_REPO for airshipctl?20:15
airship-irc-bot1<dwalt> it looks like it's in stable now. So we can just change it? https://packages.debian.org/buster/cloud-init20:16
airship-irc-bot1<dwalt> seems it has been removed form unstable20:17
airship-irc-bot1<dwalt> If I am reading it correctly20:17
airship-irc-bot1<alexander.hughes> `/ Packages / buster (stable) / admin / cloud-init`20:18
airship-irc-bot1<alexander.hughes> that's how I'm reading it too20:18
openstackgerritMatt McEuen proposed airship/airshipctl master: Stub out replacement structure  https://review.opendev.org/72486920:23
openstackgerritArvinderpal Wander proposed airship/airshipctl master: Adds support for `clusterctl move` of capi and bmo resources from ephermeral to workload cluster.  https://review.opendev.org/72494020:44
openstackgerritTrung Thai proposed airship/porthole master: Create an initial Automation Validation Testing framework  https://review.opendev.org/72011320:57
openstackgerritAndrew Schiefelbein proposed airship/airshipui master: POC pluggable auth method  https://review.opendev.org/72693721:15
*** born2bake has quit IRC21:24
airship-irc-bot1<pb269f> should move to a single debian repo/source21:37
airship-irc-bot1<pb269f> theres no reason to pull in cloud-init from sid21:37
airship-irc-bot1<pb269f> ive run without - and works fine21:37
openstackgerritDrew Walters proposed airship/images master: debian-isogen: Remove unstable repository  https://review.opendev.org/72830221:49
openstackgerritDrew Walters proposed airship/images master: debian-isogen: Remove unstable repository  https://review.opendev.org/72830221:52
airship-irc-bot1<dwalt> Change to remove sid: https://review.opendev.org/72830222:01
airship-irc-bot1<dwalt> Image builds fine22:01
airship-irc-bot1<kk6740> lets try to merge and see if airshipctl builds stop failing ?22:03
airship-irc-bot1<dwalt> :crossed_fingers: yep22:03
*** uzumaki has quit IRC22:18
*** uzumaki has joined #airshipit22:19
openstackgerritMatt McEuen proposed airship/airshipctl master: Parameterize ApplyOptions() for local apply  https://review.opendev.org/72830723:06
openstackgerritRuslan Aliev proposed airship/airshipctl master: Remove "cluster initinfra" command  https://review.opendev.org/72427623:45
openstackgerritRuslan Aliev proposed airship/airshipctl master: Check python3-netaddr is installed within sushy-emulator role  https://review.opendev.org/72801923:46
openstackgerritRuslan Aliev proposed airship/airshipctl master: Add EnsureComplete config check during phase apply call  https://review.opendev.org/72802923:52

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