openstackgerrit | Pavel Basov proposed airship/airshipctl master: [WIP] Testing polling capabilties https://review.opendev.org/726423 | 00:02 |
---|---|---|
openstackgerrit | Matt McEuen proposed airship/airshipctl master: [#177] Add Flux Helm Operator manifests https://review.opendev.org/720298 | 00:09 |
openstackgerrit | Nishant Kumar proposed airship/deckhand master: Remove unused code for policy validation as feature not implemented https://review.opendev.org/681296 | 00:33 |
openstackgerrit | Nishant Kumar proposed airship/deckhand master: Remove unused code for policy validation as feature not implemented https://review.opendev.org/681296 | 00:34 |
openstackgerrit | Nishant Kumar proposed airship/deckhand master: Remove unused code for policy validation as feature not implemented https://review.opendev.org/681296 | 00:34 |
openstackgerrit | James Gu proposed airship/airshipctl master: [#30] Implement document info command https://review.opendev.org/707483 | 02:16 |
*** uzumaki has quit IRC | 02:58 | |
openstackgerrit | Alexander Noskov proposed airship/airshipctl master: Fix default checkout options in Airshipctl config https://review.opendev.org/727476 | 04:35 |
*** evrardjp has quit IRC | 04:36 | |
*** evrardjp has joined #airshipit | 04:36 | |
*** KeithMnemonic has quit IRC | 05:14 | |
*** dpawlik has joined #airshipit | 06:06 | |
openstackgerrit | Nikolay Fedorov proposed airship/airshipctl master: [WIP] Add ephemeral function with ReplacementTransformer https://review.opendev.org/727095 | 07:19 |
*** uzumaki has joined #airshipit | 08:10 | |
openstackgerrit | Nikolay Fedorov proposed airship/airshipctl master: [WIP] Move transformer at site level https://review.opendev.org/727966 | 08:20 |
openstackgerrit | Arnava proposed airship/airshipctl master: [issue # 188]implement proper file permissions on Airship2 generated files https://review.opendev.org/727969 | 08:32 |
openstackgerrit | Nikolay Fedorov proposed airship/airshipctl master: [WIP] Change naming of nodes https://review.opendev.org/727970 | 08:36 |
*** dpawlik has quit IRC | 08:40 | |
*** dpawlik has joined #airshipit | 08:45 | |
*** born2bake has joined #airshipit | 08:49 | |
openstackgerrit | Dmitry Ukov proposed airship/airshipctl master: [WIP] Add control plane deployment https://review.opendev.org/714758 | 08:58 |
openstackgerrit | Ruslan Aliev proposed airship/airshipctl master: Return an error if there is a problem to access config file https://review.opendev.org/725820 | 09:23 |
openstackgerrit | Ruslan Aliev proposed airship/airshipctl master: Remove "cluster initinfra" command https://review.opendev.org/724276 | 09:29 |
openstackgerrit | Dmitry Ukov proposed airship/airshipctl master: [WIP] Add control plane deployment https://review.opendev.org/714758 | 09:36 |
openstackgerrit | Pavel Basov proposed airship/airshipctl master: [WIP] Testing polling capabilties https://review.opendev.org/726423 | 09:38 |
openstackgerrit | Ruslan Aliev proposed airship/airshipctl master: Remove "cluster initinfra" command https://review.opendev.org/724276 | 09:44 |
openstackgerrit | Arnava proposed airship/airshipctl master: WIP.....[issue # 188]implement proper file permissions on Airship2 generated files https://review.opendev.org/727969 | 10:13 |
openstackgerrit | Arnava proposed airship/airshipctl master: {WIP][issue # 188]proper file permissions on Airship2 generated files https://review.opendev.org/727969 | 10:20 |
openstackgerrit | Dmitry Ukov proposed airship/airshipctl master: [WIP] Add control plane deployment https://review.opendev.org/714758 | 10:20 |
openstackgerrit | Arnava proposed airship/airshipctl master: [WIP][issue # 188]proper file permissions on Airship2 generated files https://review.opendev.org/727969 | 10:20 |
openstackgerrit | Ruslan Aliev proposed airship/airshipctl master: Check python3-netaddr is installed within sushy-emulator role https://review.opendev.org/728019 | 10:32 |
*** avolkov has joined #airshipit | 10:37 | |
openstackgerrit | Ruslan Aliev proposed airship/airshipctl master: Add EnsureComplete config check during phase apply call https://review.opendev.org/728029 | 11:07 |
openstackgerrit | Dmitry Ukov proposed airship/airshipctl master: [WIP] Add control plane deployment https://review.opendev.org/714758 | 11:27 |
*** happyhemant has joined #airshipit | 11:33 | |
openstackgerrit | Pavel Basov proposed airship/airshipctl master: [WIP] Testing polling capabilties https://review.opendev.org/726423 | 12:19 |
openstackgerrit | Pavel Basov proposed airship/airshipctl master: [WIP] Testing polling capabilties https://review.opendev.org/726423 | 13:01 |
*** KeithMnemonic has joined #airshipit | 13: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 |
openstackgerrit | Merged airship/airshipctl master: Fix default checkout options in Airshipctl config https://review.opendev.org/727476 | 14:05 |
*** mikefix has joined #airshipit | 14:29 | |
*** uzumaki has quit IRC | 14:35 | |
*** uzumaki has joined #airshipit | 14:39 | |
*** mikefix has quit IRC | 14:48 | |
*** evrardjp has quit IRC | 14:55 | |
*** evrardjp has joined #airshipit | 14:56 | |
*** jamesgu has quit IRC | 15:09 | |
*** jamesgu_ has joined #airshipit | 15:09 | |
openstackgerrit | Stas Egorov proposed airship/airshipctl master: Fixed local gate script error https://review.opendev.org/726903 | 15:14 |
openstackgerrit | Trung Thai proposed airship/porthole master: Create an initial Automation Validation Testing framework https://review.opendev.org/720113 | 15:20 |
openstackgerrit | Merged airship/drydock master: Address PEP8 Failures in Drydock https://review.opendev.org/727343 | 15:26 |
openstackgerrit | Vladimir Kozhukalov proposed airship/airshipctl master: Implement document rendering sub-command https://review.opendev.org/698658 | 15:28 |
*** jamesgu has joined #airshipit | 15:49 | |
openstackgerrit | Sirajudeen proposed airship/airshipctl master: Fix document pull with default airship config https://review.opendev.org/728124 | 16:04 |
openstackgerrit | James Gu proposed airship/airshipctl master: [#24] Fix airshipctl config get-cluster filtering https://review.opendev.org/709873 | 16:09 |
mattmceuen | PSA 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 filtering | 16:13 |
openstackgerrit | Alexander Hughes proposed airship/governance master: Clarification of tie breaking method https://review.opendev.org/728127 | 16:16 |
*** happyhemant has quit IRC | 16:22 | |
openstackgerrit | Alexander Hughes proposed airship/airshipctl master: etcd encryption addition https://review.opendev.org/720812 | 16:25 |
*** dpawlik has quit IRC | 16:27 | |
*** evrardjp has quit IRC | 16:33 | |
*** evrardjp has joined #airshipit | 16:33 | |
*** uzumaki has quit IRC | 16:36 | |
openstackgerrit | Merged airship/porthole master: Enable Deployment configuration consistency across all UC https://review.opendev.org/725117 | 16:44 |
openstackgerrit | Sirajudeen proposed airship/airshipctl master: Fix Lint warnings: Missing comments for functions https://review.opendev.org/727804 | 17:09 |
openstackgerrit | Prateek Dodda proposed airship/drydock master: Enabling Apparmor for drydock init containers https://review.opendev.org/724768 | 17:12 |
*** uzumaki has joined #airshipit | 17:18 | |
openstackgerrit | Andrew Schiefelbein proposed airship/airshipui master: [WIP] POC pluggable auth method https://review.opendev.org/726937 | 17:27 |
openstackgerrit | Stas Egorov proposed airship/airshipctl master: [#116]: added sub-commands to set/get bootstrap info https://review.opendev.org/720576 | 18:00 |
openstackgerrit | Ryan Schroder proposed airship/airshipui master: Yaml Visualizer Project https://review.opendev.org/724888 | 18: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/240 | 18:35 |
*** dpawlik has joined #airshipit | 18:51 | |
openstackgerrit | Andrew Schiefelbein proposed airship/airshipui master: [WIP] POC pluggable auth method https://review.opendev.org/726937 | 19:13 |
openstackgerrit | Andrew Schiefelbein proposed airship/airshipui master: [WIP] POC pluggable auth method https://review.opendev.org/726937 | 19:27 |
*** avolkov has quit IRC | 19:28 | |
openstackgerrit | Alexander Noskov proposed airship/airshipctl master: Create a directory for `baremetal isogen` artifacts. https://review.opendev.org/728264 | 19:36 |
openstackgerrit | Alexander Noskov proposed airship/airshipctl master: Change default repo from Treasuremap to Airshipctl https://review.opendev.org/728266 | 19:41 |
openstackgerrit | Matt McEuen proposed airship/airshipctl master: [WIP] Add a setting for kustomize plugins https://review.opendev.org/728268 | 19: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 IRC | 19: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.go | 19: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 this | 19:47 |
openstackgerrit | Dmitry Ukov proposed airship/airshipctl master: [WIP] Add control plane deployment https://review.opendev.org/714758 | 19:47 |
airship-irc-bot1 | <kk6740> btw same problem for me | 19: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 taking | 19:48 |
airship-irc-bot1 | <ih616h> @an310y https://github.com/ianpittwood/Gerrit-to-Github-Issues/pull/8 | 19:48 |
airship-irc-bot1 | <ih616h> ^the said items | 19: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 seeking | 19:52 |
airship-irc-bot1 | <kk6740> thanks | 19:52 |
airship-irc-bot1 | <mattmceuen> dwalt you are also the docs guru, your head is heavy laden with many hats | 19:53 |
airship-irc-bot1 | <alexander.hughes> this tbh ^ | 19:53 |
airship-irc-bot1 | <dwalt> lol | 19:54 |
uzumaki | all 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 xD | 19: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 object | 19:56 |
airship-irc-bot1 | <ih616h> unless you think you're going to need the settings for seomthine else | 19: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 sense | 19:58 |
airship-irc-bot1 | <mattmceuen> I mean an alternative would be to make settings global | 19: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 |
openstackgerrit | Sirajudeen proposed airship/airshipctl master: Validate config before document pull https://review.opendev.org/728276 | 19:58 |
airship-irc-bot1 | <mattmceuen> or make the kustomize plugins path global | 19:58 |
airship-irc-bot1 | <mattmceuen> but that would break the conventions we have | 19: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 bad | 20: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 |
openstackgerrit | Alexander Noskov proposed airship/airshipctl master: Change default repo from Treasuremap to Airshipctl https://review.opendev.org/728266 | 20:01 |
openstackgerrit | Ian Howell proposed airship/airshipctl master: Refactor context management https://review.opendev.org/725937 | 20: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 |
openstackgerrit | Alexander Hughes proposed airship/governance master: Clarification of tie breaking method https://review.opendev.org/728127 | 20:05 |
airship-irc-bot1 | <dwalt> @kk6740 invite sent | 20:05 |
openstackgerrit | Alexander Hughes proposed airship/airshipctl master: etcd encryption addition https://review.opendev.org/720812 | 20: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 hours | 20: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 category | 20: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 install | 20: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 global | 20: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 installed | 20:12 |
airship-irc-bot1 | <kk6740> @dwalt thanks, got it | 20:12 |
airship-irc-bot1 | <mb551n> I am seeing that same cloud-init error in my local gate that I have been trying to get running | 20: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-init | 20:16 |
airship-irc-bot1 | <dwalt> seems it has been removed form unstable | 20:17 |
airship-irc-bot1 | <dwalt> If I am reading it correctly | 20: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 too | 20:18 |
openstackgerrit | Matt McEuen proposed airship/airshipctl master: Stub out replacement structure https://review.opendev.org/724869 | 20:23 |
openstackgerrit | Arvinderpal Wander proposed airship/airshipctl master: Adds support for `clusterctl move` of capi and bmo resources from ephermeral to workload cluster. https://review.opendev.org/724940 | 20:44 |
openstackgerrit | Trung Thai proposed airship/porthole master: Create an initial Automation Validation Testing framework https://review.opendev.org/720113 | 20:57 |
openstackgerrit | Andrew Schiefelbein proposed airship/airshipui master: POC pluggable auth method https://review.opendev.org/726937 | 21:15 |
*** born2bake has quit IRC | 21:24 | |
airship-irc-bot1 | <pb269f> should move to a single debian repo/source | 21:37 |
airship-irc-bot1 | <pb269f> theres no reason to pull in cloud-init from sid | 21:37 |
airship-irc-bot1 | <pb269f> ive run without - and works fine | 21:37 |
openstackgerrit | Drew Walters proposed airship/images master: debian-isogen: Remove unstable repository https://review.opendev.org/728302 | 21:49 |
openstackgerrit | Drew Walters proposed airship/images master: debian-isogen: Remove unstable repository https://review.opendev.org/728302 | 21:52 |
airship-irc-bot1 | <dwalt> Change to remove sid: https://review.opendev.org/728302 | 22:01 |
airship-irc-bot1 | <dwalt> Image builds fine | 22:01 |
airship-irc-bot1 | <kk6740> lets try to merge and see if airshipctl builds stop failing ? | 22:03 |
airship-irc-bot1 | <dwalt> :crossed_fingers: yep | 22:03 |
*** uzumaki has quit IRC | 22:18 | |
*** uzumaki has joined #airshipit | 22:19 | |
openstackgerrit | Matt McEuen proposed airship/airshipctl master: Parameterize ApplyOptions() for local apply https://review.opendev.org/728307 | 23:06 |
openstackgerrit | Ruslan Aliev proposed airship/airshipctl master: Remove "cluster initinfra" command https://review.opendev.org/724276 | 23:45 |
openstackgerrit | Ruslan Aliev proposed airship/airshipctl master: Check python3-netaddr is installed within sushy-emulator role https://review.opendev.org/728019 | 23:46 |
openstackgerrit | Ruslan Aliev proposed airship/airshipctl master: Add EnsureComplete config check during phase apply call https://review.opendev.org/728029 | 23:52 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!