Monday, 2020-07-13

openstackgerritdiwakar thyagaraj proposed airship/porthole master: [WIP] Enable for key-generator  https://review.opendev.org/74000201:33
*** evrardjp has quit IRC04:33
*** evrardjp has joined #airshipit04:33
openstackgerritSirisha Gopigiri proposed airship/airshipctl master: Testing certificate insertion with Zuul Gates  https://review.opendev.org/73943106:44
openstackgerritVrushali proposed airship/airshipctl master: Testing docker-gate  https://review.opendev.org/73868210:13
*** born2bake has joined #airshipit10:23
openstackgerritdiwakar thyagaraj proposed airship/porthole master: [WIP] Enable for key-generator  https://review.opendev.org/74000210:27
openstackgerritVrushali proposed airship/airshipctl master: Testing docker-gate  https://review.opendev.org/73868210:40
openstackgerritSirisha Gopigiri proposed airship/airshipctl master: Host Configuration of Kubernetes clusters using Ansible Operator  https://review.opendev.org/74001610:48
*** muhaha has joined #airshipit11:07
openstackgerritdiwakar thyagaraj proposed airship/porthole master: [WIP] Enable for key-generator  https://review.opendev.org/74000211:13
openstackgerritdiwakar thyagaraj proposed airship/porthole master: [WIP] Enable for key-generator  https://review.opendev.org/74000211:44
openstackgerritdiwakar thyagaraj proposed airship/maas master: [WIP] Enable CI for Maas  https://review.opendev.org/70976311:58
openstackgerritdiwakar thyagaraj proposed airship/porthole master: [Fix] Remove the dependency for Ceph-Utility Deployment Jobs  https://review.opendev.org/73929212:01
*** avolkov has joined #airshipit12:03
airship-irc-bot1<nfedorov> gate/20_run_test_runner.sh fails because ```'''ERROR! the role 'airshipctl-systemwide-executable' was not found in /root/airshipctl/playbooks/roles:/root/airshipctl/roles:/root/airshipctl/playbooks  ERROR! the role 'airshipctl-test-configs' was not found in /root/airshipctl/playbooks/roles:/root/airshipctl/roles:/root/airshipctl/playbooks  ERROR! the role 'airshipctl-pull-docs' was not found in12:14
airship-irc-bot1/root/airshipctl/playbooks/roles:/root/airshipctl/roles:/root/airshipctl/playbooks  ERROR! the role 'airshipctl-build-ephemeral-iso' was not found in /root/airshipctl/playbooks/roles:/root/airshipctl/roles:/root/airshipctl/playbooks  ERROR! the role 'install-kubectl' was not found in /root/airshipctl/playbooks/roles:/root/airshipctl/roles:/root/airshipctl/playbooks  ERROR! the role 'airshipctl-deploy-ephemeral-node' was not found in12:14
airship-irc-bot1/root/airshipctl/playbooks/roles:/root/airshipctl/roles:/root/airshipctl/playbooks  ERROR! the role 'airshipctl-cluster-init-ephemeral' was not found in /root/airshipctl/playbooks/roles:/root/airshipctl/roles:/root/airshipctl/playbooks  ERROR! the role 'airshipctl-phase-apply-initinfra' was not found in /root/airshipctl/playbooks/roles:/root/airshipctl/roles:/root/airshipctl/playbooks  ERROR! the role 'airshipctl-phase-controlplane' was not12:14
airship-irc-bot1found in /root/airshipctl/playbooks/roles:/root/airshipctl/roles:/root/airshipctl/playbooks```12:14
airship-irc-bot1<nfedorov> commit 9892e5ea812f175707b5751648377b6c17af5b3d (*HEAD -> master*, *origin/master*, *origin/HEAD*)12:15
openstackgerritMerged airship/election master: Init repo for 2020 WC Election  https://review.opendev.org/74049812:41
*** zombieJulia is now known as TheJulia13:06
*** irclogbot_2 has quit IRC13:27
*** thansen has quit IRC13:27
*** georgk has quit IRC13:27
*** irclogbot_2 has joined #airshipit13:28
*** thansen has joined #airshipit13:28
*** georgk has joined #airshipit13:28
openstackgerritJagan Mohan Kavva proposed airship/porthole master: Update all the utility containers to K8s 1.17.3  https://review.opendev.org/73977313:38
airship-irc-bot1<ih616h> @mattmceuen @raliev Sorry I missed this last week - Here's the change that fixes the chicken/egg problem: https://review.opendev.org/#/c/739618/13:49
airship-irc-bot1<ih616h> it lifts the `airshipctl config init` command to the root `airshipctl init`, a la Helm 213:50
openstackgerritIan Howell proposed airship/airshipctl master: Fix configuration initialization  https://review.opendev.org/73961813:59
airship-irc-bot1<mattmceuen> Thanks @ih616h no worries.  It would be a bit strange to rework the CLI just to work around a technical issue; is there any additional reason we want to go with "airshipctl init"?  "airshipctl config init" makes sense to me since it does exactly what it says on the tin :)14:04
airship-irc-bot1<ih616h> no additional reasoning aside from following prior art. I also think it fits as a top-level command, since it initializes `airshipctl` as a whole, not just the config module14:08
airship-irc-bot1<ih616h> e.g. `helm` and `kubectl` also don't work without there config files14:09
airship-irc-bot1<mattmceuen> Ah, I may have missed it - I thought airshipctl init was only for config files; are we adding more into that command?14:14
openstackgerritJagan Mohan Kavva proposed airship/porthole master: Update all the utility containers to K8s 1.17.3  https://review.opendev.org/73977314:14
airship-irc-bot1<ih616h> airshipctl is more or less unusable without a config file14:15
airship-irc-bot1<mattmceuen> I agree that "helm init" does more than config, and so makes sense at the top level.  I don't see the analogous command for kubectl though, what is that one?14:16
airship-irc-bot1<mattmceuen> I agree with you on more or less unusable without a config file, which is why "config init" makes sense if it's just init'ing the config.  That leaves it side by side with the commands that modify the config.  If we're planning on adding other functionality into that command, though, then it wouldn't make sense to leave it under config14:18
airship-irc-bot1<ih616h> alright, that seems reasonable enough to me. I think the best solution for this problem then would be to go the opposite direction I took by delegating the config loading deeper into the tree rather than lifting the command. I'll adjust the change14:20
airship-irc-bot1<mattmceuen> Did you see @raliev’s change - is that along the lines of what you're thinking?  https://review.opendev.org/#/c/740533/14:21
airship-irc-bot1<ih616h> I don't know if this is the right approach. I think it'd be better if the commands that need a config file call the `LoadConfig` function, and commands that don't need it don't call the function14:27
airship-irc-bot1<ih616h> as opposed to having different behaviors on different calls to the function14:28
airship-irc-bot1<mattmceuen> Ok, gotcha14:29
openstackgerritNikolay Fedorov proposed airship/airshipctl master: [WIP] Install airshipctl as generator plugin  https://review.opendev.org/74070314:32
openstackgerritNikolay Fedorov proposed airship/airshipctl master: [WIP] Install airshipctl as generator plugin  https://review.opendev.org/74070314:40
openstackgerritdiwakar thyagaraj proposed airship/porthole master: [WIP] Enable Socat for Openstack UC  https://review.opendev.org/73849114:44
openstackgerritdiwakar thyagaraj proposed airship/porthole master: [WIP] Enable Socat for Openstack UC  https://review.opendev.org/73849114:44
*** uzumaki has joined #airshipit14:52
openstackgerritNikolay Fedorov proposed airship/airshipctl master: [WIP] Install airshipctl as generator plugin  https://review.opendev.org/74070314:59
openstackgerritJagan Mohan Kavva proposed airship/porthole master: Update all the utility containers to K8s 1.17.3  https://review.opendev.org/73977315:13
openstackgerritNikolay Fedorov proposed airship/airshipctl master: [WIP] Install airshipctl as generator plugin  https://review.opendev.org/74070315:22
openstackgerritSreejith Punnapuzha proposed airship/airshipctl master: Fix aishipctl validate documents job  https://review.opendev.org/74030015:26
openstackgerritSirajudeen proposed airship/airshipctl master: Remove import from airshipctl config  https://review.opendev.org/74071915:41
openstackgerritSreejith Punnapuzha proposed airship/airshipctl master: Enhance target node deployment on gates  https://review.opendev.org/73869615:51
airship-irc-bot1<alexander.hughes> July Airship update now live: https://www.airshipit.org/blog/airship-update-july-2020/15:58
openstackgerritMerged airship/porthole master: [Fix] Remove the dependency for Ceph-Utility Deployment Jobs  https://review.opendev.org/73929216:07
openstackgerritSirajudeen proposed airship/airshipctl master: Remove import from airshipctl config  https://review.opendev.org/74071916:12
openstackgerritdiwakar thyagaraj proposed airship/porthole master: [WIP] Enable for key-generator  https://review.opendev.org/74000216:20
openstackgerritdiwakar thyagaraj proposed airship/porthole master: [WIP] Enable for key-generator  https://review.opendev.org/74000216:20
openstackgerritdiwakar thyagaraj proposed airship/porthole master: [WIP] Enable for key-generator  https://review.opendev.org/74000216:21
airship-irc-bot1<se6518> Hello, core reviewers, would you, please, check https://review.opendev.org/740150 ?16:40
*** uzumaki has quit IRC17:11
openstackgerritJagan Mohan Kavva proposed airship/porthole master: Update all the utility containers to K8s 1.17.3  https://review.opendev.org/73977317:18
openstackgerritJagan Mohan Kavva proposed airship/porthole master: Update all the utility containers to K8s 1.17.3  https://review.opendev.org/73977317:19
openstackgerritMerged airship/airshipctl master: Cleaning for gate scripts  https://review.opendev.org/74015017:32
openstackgerritdiwakar thyagaraj proposed airship/porthole master: [WIP] Enable for key-generator  https://review.opendev.org/74000217:34
openstackgerritdiwakar thyagaraj proposed airship/porthole master: [WIP] Enable for key-generator  https://review.opendev.org/74000217:35
openstackgerritSirajudeen proposed airship/airshipctl master: WIP - Bash script log enhancements  https://review.opendev.org/73998817:37
openstackgerritSirajudeen proposed airship/airshipctl master: WIP - Bash script log enhancements  https://review.opendev.org/73998817:42
openstackgerritdiwakar thyagaraj proposed airship/porthole master: Revert "Removing the Openstack-utility helm test Case."  https://review.opendev.org/74073617:54
openstackgerritdiwakar thyagaraj proposed airship/porthole master: Revert "Removing the Openstack-utility helm test Case."  https://review.opendev.org/74073617:54
openstackgerritdiwakar thyagaraj proposed airship/porthole master: [WIP] Revert "Removing the Openstack-utility helm test Case."  https://review.opendev.org/74073617:54
openstackgerritIan Howell proposed airship/airshipctl master: Add unit tests for missing Bundle functions  https://review.opendev.org/74050317:58
*** GoldenBear has quit IRC18:07
*** GoldenBear_ has joined #airshipit18:08
openstackgerritBijaya Sharma proposed airship/airshipui master: Added unit-tests in Makefile  https://review.opendev.org/73868818:20
openstackgerritNishant Kumar proposed airship/armada master: Handle unknow chart status in more resilient manner  https://review.opendev.org/74073818:29
openstackgerritNishant Kumar proposed airship/armada master: [WIP]Handle unknow chart status in more resilient manner  https://review.opendev.org/74073818:32
openstackgerritdiwakar thyagaraj proposed airship/porthole master: [WIP] Enable for key-generator  https://review.opendev.org/74000218:38
openstackgerritKostyantyn Kalynovskyi proposed airship/airshipctl master: Allow to use variable substitution for cluster-api components  https://review.opendev.org/73985419:03
openstackgerritdiwakar thyagaraj proposed airship/porthole master: [WIP] Revert "Removing the Openstack-utility helm test Case."  https://review.opendev.org/74073619:05
openstackgerritAndrew Schiefelbein proposed airship/airshipui master: Add gomod tidy check to lint gate  https://review.opendev.org/72960219:08
openstackgerritAndrew Schiefelbein proposed airship/airshipui master: Add gomod tidy check to lint gate  https://review.opendev.org/72960219:19
openstackgerritAndrew Schiefelbein proposed airship/airshipui master: Add gomod tidy check to lint gate  https://review.opendev.org/72960219:24
openstackgerritBijaya Sharma proposed airship/airshipui master: Added copyright in Makefile  https://review.opendev.org/74034719:27
openstackgerritBijaya Sharma proposed airship/airshipui master: Added copyright in Makefile  https://review.opendev.org/74034719:38
openstackgerritVrushali proposed airship/airshipctl master: Testing docker-gate  https://review.opendev.org/73868219:40
openstackgerritMichael Beaver proposed airship/airshipctl master: Improve the --help message for isogen command  https://review.opendev.org/73319019:43
openstackgerritMerged airship/porthole master: Update all the utility containers to K8s 1.17.3  https://review.opendev.org/73977320:05
openstackgerritKostyantyn Kalynovskyi proposed airship/airshipctl master: Add manifestreader interface implementation  https://review.opendev.org/73892120:06
openstackgerritKostyantyn Kalynovskyi proposed airship/airshipctl master: Add applier from cli-utils package  https://review.opendev.org/73605220:06
openstackgerritKostyantyn Kalynovskyi proposed airship/airshipctl master: Integrate cli-utils applier with phases.  https://review.opendev.org/73605420:10
openstackgerritAndrew Schiefelbein proposed airship/airshipui master: [WIP] SSO for plugin dashboards  https://review.opendev.org/73846120:22
*** muhaha has quit IRC20:26
openstackgerritRick Bartra proposed airship/porthole master: [WIP] etcdctl-utility container with hostPID: true  https://review.opendev.org/74080320:28
openstackgerritMichael Beaver proposed airship/airshipctl master: Improve the --help message for isogen command  https://review.opendev.org/73319020:31
openstackgerritMerged airship/shipyard master: Implement helm-toolkit snippet to airflow pods/containers  https://review.opendev.org/73961720:42
openstackgerritRuslan Aliev proposed airship/airshipctl master: Add manifestreader interface implementation  https://review.opendev.org/73892121:01
openstackgerritRuslan Aliev proposed airship/airshipctl master: Integrate cli-utils applier with phases.  https://review.opendev.org/73605421:01
openstackgerritRuslan Aliev proposed airship/airshipctl master: Add applier from cli-utils package  https://review.opendev.org/73605221:01
openstackgerritRuslan Aliev proposed airship/airshipctl master: Add poller  https://review.opendev.org/73488221:01
openstackgerritRuslan Aliev proposed airship/airshipctl master: Add applier from cli-utils package  https://review.opendev.org/73605221:11
openstackgerritRuslan Aliev proposed airship/airshipctl master: Add poller  https://review.opendev.org/73488221:28
openstackgerritdiwakar thyagaraj proposed airship/porthole master: [WIP] Revert "Removing the Openstack-utility helm test Case."  https://review.opendev.org/74073621:39
openstackgerritdiwakar thyagaraj proposed airship/porthole master: [WIP] Enable for key-generator  https://review.opendev.org/74000221:40
openstackgerritNishant Kumar proposed airship/armada master: [WIP]Handle unknow chart status in more resilient manner  https://review.opendev.org/74073821:48
*** born2bake has quit IRC21:52
*** fritchie has joined #airshipit22:18
*** fritchie has left #airshipit22:31
*** avolkov has quit IRC22:33
openstackgerritMatt McEuen proposed airship/airshipctl master: Add an m3 host generator function  https://review.opendev.org/73503322:55
openstackgerritMatt McEuen proposed airship/airshipctl master: Add an m3 host generator function  https://review.opendev.org/73503323:31
*** thansen has quit IRC23:32
*** thansen has joined #airshipit23:32

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