Thursday, 2020-03-12

« Wednesday, 2020-03-11 Index Friday, 2020-03-13 »
openstackgerritKostyantyn Kalynovskyi proposed airship/airshipctl master: [WIP] Split document model, add entrypoints for repos  https://review.opendev.org/71157100:07
openstackgerritKostyantyn Kalynovskyi proposed airship/airshipctl master: Split document model, add entrypoints for repos  https://review.opendev.org/71157100:09
*** openstackstatus has joined #airshipit00:43
*** ChanServ sets mode: +v openstackstatus00:43
*** dwalt has quit IRC00:53
*** segorov has joined #airshipit01:04
*** AlexNoskov has quit IRC01:34
*** rezroo has quit IRC01:57
*** rezroo has joined #airshipit01:57
*** KeithMnemonic1 has joined #airshipit03:28
*** KeithMnemonic has quit IRC03:31
openstackgerritSirajudeen proposed airship/airshipctl master: [#21] - prompt for config init options  https://review.opendev.org/71018004:00
*** KeithMnemonic1 has quit IRC04:06
*** rezroo has quit IRC04:54
*** rezroo has joined #airshipit04:55
*** evrardjp has quit IRC05:35
*** evrardjp has joined #airshipit05:36
*** rezroo has quit IRC06:30
*** rezroo has joined #airshipit06:46
*** dpawlik has joined #airshipit07:04
*** roman_g has quit IRC07:11
*** rezroo has quit IRC07:56
*** SRao has joined #airshipit09:22
*** SRao__ has joined #airshipit09:51
*** SRao__ has quit IRC09:53
*** SRao__ has joined #airshipit09:54
*** SRao has quit IRC09:54
*** SRao__ has quit IRC09:57
openstackgerritAlexander Hughes proposed airship/airshipctl master: [#91] refactor secrets as top level command  https://review.opendev.org/71168412:17
openstackgerritAlexander Hughes proposed airship/airshipctl master: [#70] Remove unnecessary kubectl subcommand  https://review.opendev.org/71008512:17
openstackgerritJagan Mohan Kavva proposed airship/maas master: Enable Docker default AppArmor profile to maas  https://review.opendev.org/70513612:20
openstackgerritNikolay Fedorov proposed airship/images master: Example network-config network_data.json format  https://review.opendev.org/71266712:22
openstackgerritJagan Mohan Kavva proposed airship/maas master: Enable Docker default AppArmor profile to maas  https://review.opendev.org/70513612:30
openstackgerritNikolay Fedorov proposed airship/images master: Example network-config network_data.json format  https://review.opendev.org/71266712:37
openstackgerritMerged airship/airshipctl master: Properly locate tests within a config_test pkg for pkg/config and cmd/config  https://review.opendev.org/71258412:39
openstackgerritAlexander Hughes proposed airship/airshipctl master: [#88] add golang  naming convention documentation  https://review.opendev.org/71152912:43
*** ianychoi has quit IRC13:13
*** rezroo has joined #airshipit13:14
openstackgerritJagan Mohan Kavva proposed airship/maas master: Enable Docker default AppArmor profile to maas  https://review.opendev.org/70513613:16
*** dwalt has joined #airshipit13:56
openstackgerritSreejith Punnapuzha proposed airship/airshipctl master: [#106] Add Gather logs role  https://review.opendev.org/71269314:17
openstackgerritKostyantyn Kalynovskyi proposed airship/airshipctl master: Add repository to airship config template, ansible  https://review.opendev.org/71182214:29
openstackgerritKostyantyn Kalynovskyi proposed airship/airshipctl master: Split document model, add entrypoints for repos  https://review.opendev.org/71157114:29
openstackgerritdiwakar thyagaraj proposed airship/maas master: Enable Docker default AppArmor profile to maas  https://review.opendev.org/70513614:29
*** jamesgu has joined #airshipit14:50
openstackgerritKostyantyn Kalynovskyi proposed airship/airshipctl master: Split document model, add entrypoints for repos  https://review.opendev.org/71157114:55
*** uzumaki has joined #airshipit14:59
*** jamesgu has quit IRC15:15
*** segorov has quit IRC15:17
airshipbot<se6518> I will check new commit which is implementing possibility to use local image and abandon mine if it's not needed anymore.15:19
*** segorov has joined #airshipit15:20
airshipbot<dwalt> That sounds like a plan. Thanks @se651815:23
openstackgerritDan Crank proposed airship/divingbell master: [ad-hoc] Fix test case exit conditions  https://review.opendev.org/70388015:25
openstackgerritdiwakar thyagaraj proposed airship/maas master: [WIP]Enable Logs Output in Zuul for Deployment Scripts  https://review.opendev.org/70976315:30
*** rezroo has quit IRC15:31
openstackgerritdiwakar thyagaraj proposed airship/maas master: Enable Docker default AppArmor profile to maas  https://review.opendev.org/70513615:33
openstackgerritFrancis Bacon Yi proposed airship/airshipctl master: [WIP] Increase config pkg test coverage  https://review.opendev.org/71250315:37
*** mfuller has joined #airshipit15:42
*** thansen has quit IRC15:50
*** thansen has joined #airshipit15:50
*** roman_g has joined #airshipit16:07
howellLF some reviews/suggestions: https://review.opendev.org/#/c/71081416:11
howellstill very much a WIP16:11
*** uzumaki has quit IRC16:11
*** jamesgu has joined #airshipit16:22
openstackgerritMerged airship/airshipctl master: Correct ansible task name for kubectl version  https://review.opendev.org/71216816:25
openstackgerritMerged airship/airshipctl master: [#26] - Skip Image download if already downloaded  https://review.opendev.org/71138316:26
*** dpawlik has quit IRC16:28
*** dpawlik has joined #airshipit16:28
*** jamesgu has quit IRC16:32
openstackgerritFrancis Bacon Yi proposed airship/airshipctl master: [WIP] Increase config pkg test coverage  https://review.opendev.org/71250316:33
*** jamesgu has joined #airshipit16:57
*** jamesgu has quit IRC17:11
openstackgerritMerged airship/airshipctl master: [#70] Remove unnecessary kubectl subcommand  https://review.opendev.org/71008517:13
*** evrardjp has quit IRC17:35
*** evrardjp has joined #airshipit17:36
openstackgerritFrancis Bacon Yi proposed airship/airshipctl master: [WIP] Increase config pkg test coverage  https://review.opendev.org/71250318:06
openstackgerritFrancis Bacon Yi proposed airship/airshipctl master: Increase config pkg test coverage  https://review.opendev.org/71250318:22
*** AlexNoskov has joined #airshipit18:28
AlexNoskov Alexander Noskov Hello, can I get review/merge https://review.opendev.org/#/c/710480/ https://review.opendev.org/#/c/680495/ https://review.opendev.org/#/c/711566/ https://review.opendev.org/#/c/709638/ Thanks in advance!18:29
openstackgerritdiwakar thyagaraj proposed airship/maas master: [WIP]Enable Logs Output in Zuul for Deployment Scripts  https://review.opendev.org/70976318:29
openstackgerritAlexander Hughes proposed airship/airshipctl master: [#91] refactor secrets as top level command  https://review.opendev.org/71168418:30
openstackgerritAhmad Mahmoudi proposed airship/pegleg master: Updated shipyard bionic dependencies  https://review.opendev.org/71184218:38
openstackgerritdiwakar thyagaraj proposed airship/maas master: [WIP]Enable Logs Output in Zuul for Deployment Scripts  https://review.opendev.org/70976319:10
openstackgerritAlexander Hughes proposed airship/airshipctl master: Increase config pkg test coverage  https://review.opendev.org/71250319:17
airshipbot<alexander.hughes> @ih616h https://review.opendev.org/#/c/712503/ any idea why in the gate golangci-lint is calling out errors on the newly added config.InitConfig(t) declarations in this patchset?  `/home/ubuntu/projects/airshipctl/pkg/config/config_test.go:375:26: InitConfig not declared by package config]`19:19
airshipbot<alexander.hughes> the additional tests where this is used are structured similarly to previous instances of config.InitConfig(t) within config_test.go19:20
*** zogger has left #airshipit19:22
airshipbot<ih616h> @alexander.hughes InitConfig was moved to the `testutil` package19:33
openstackgerritDan Crank proposed airship/divingbell master: [ad-hoc] Add new parameter for daemonset_overrides  https://review.opendev.org/71277619:33
airshipbot<ih616h> https://review.opendev.org/#/c/712584/19:33
airshipbot<alexander.hughes> yeah I see it now after rebase19:33
airshipbot<alexander.hughes> thanks19:34
openstackgerritAlexander Hughes proposed airship/airshipctl master: Increase config pkg test coverage  https://review.opendev.org/71250319:34
airshipbot<alexander.hughes> thanks Ian19:39
openstackgerritSreejith Punnapuzha proposed airship/airshipctl master: [#106] Add docker install role  https://review.opendev.org/71154019:43
openstackgerritSreejith Punnapuzha proposed airship/airshipctl master: [#106] Add Gather logs role  https://review.opendev.org/71269319:45
airshipbot<alexander.hughes> getting following error with airshipctl-build-ephemeral-iso, is this task not working currently?  `TASK [airshipctl-build-ephemeral-iso : build ephemeral node iso] *******************************************************` `fatal: [primary]: FAILED! => {"changed": true, "cmd": ["airshipctl", "bootstrap", "isogen"], "delta": "0:01:24.958713", "end": "2020-03-12 18:27:09.299779", "msg": "non-zero retur19:45
airshipbot"2020-03-12 18:25:44.341066", "stderr": "", "stderr_lines": [], "stdout": "2020/03/12 18:25:44 Creating ISO builder container\n2020/03/12 18:27:09 Creating cloud-init for ephemeral K8s\nDocument filtered by selector \"~G_~V_Secret\" found no documents", "stdout_lines": ["2020/03/12 18:25:44 Creating ISO builder container", "2020/03/12 18:27:09 Creating cloud-init for ephemeral K8s", "Document filtered by selector \"~19:45
airshipbot<kk6740> @alexander.hughes yes, it was broken by recent commits19:45
airshipbot<kk6740> i am working currently on implementing one source for all selectors19:46
airshipbot<kk6740> which should fix the issue19:46
airshipbot<kk6740> and in addition to that, i would like to have small discussion here: anyone else has a feeling that we need to source data for ephemeral host from BaremetalHost + Name + Namespace selector, rather then LabelSelector? In kubenetes world, that is what identifies objects uniquely GVK + NAME + NAMESPACE. And since we need only 1 host for that, and we specifically check if its only 1 host, i think it would be mor19:47
airshipbotbut rather NAME +KIND+NAMESPACE https://github.com/airshipit/airshipctl/blob/master/pkg/bootstrap/cloudinit/cloud-init.go#L6519:47
openstackgerritAhmad Mahmoudi proposed airship/pegleg master: Updated shipyard bionic dependencies  https://review.opendev.org/71184219:48
airshipbot<kk6740> please review https://review.opendev.org/#/c/711571/19:52
airshipbot<kk6740> and this one, https://review.opendev.org/#/c/711822/ these are really blocking issues19:54
openstackgerritAhmad Mahmoudi proposed airship/pegleg master: Updated shipyard bionic dependencies  https://review.opendev.org/71184219:56
openstackgerritAhmad Mahmoudi proposed airship/pegleg master: Updated shipyard bionic dependencies  https://review.opendev.org/71184220:01
openstackgerritdiwakar thyagaraj proposed airship/maas master: [WIP]Enable Logs Output in Zuul for Deployment Scripts  https://review.opendev.org/70976320:01
openstackgerritDan Crank proposed airship/divingbell master: [ad-hoc] Add new parameter for daemonset_overrides  https://review.opendev.org/71277620:14
openstackgerritAndrew Schiefelbein proposed airship/airshipui master: [WIP] This adds a plugin for OpenStack to the Airship UI  https://review.opendev.org/71278820:19
openstackgerritMerged airship/airshipctl master: Add repository to airship config template, ansible  https://review.opendev.org/71182220:23
openstackgerritAndrew Schiefelbein proposed airship/airshipui master: [WIP] This adds a plugin for OpenStack to the Airship UI  https://review.opendev.org/71278820:35
openstackgerritAhmad Mahmoudi proposed airship/pegleg master: Updated shipyard bionic dependencies  https://review.opendev.org/71184220:41
openstackgerritAhmad Mahmoudi proposed airship/pegleg master: Updated shipyard bionic dependencies  https://review.opendev.org/71184220:44
openstackgerritDan Crank proposed airship/divingbell master: [ad-hoc] Updates for new daemonset_overrides  https://review.opendev.org/71277620:52
openstackgerritAndrew Schiefelbein proposed airship/airshipui master: [WIP] This adds a plugin for OpenStack to the Airship UI  https://review.opendev.org/71278820:53
openstackgerritMerged airship/airshipctl master: Split document model, add entrypoints for repos  https://review.opendev.org/71157121:10
*** rezroo has joined #airshipit21:33
openstackgerritKostyantyn Kalynovskyi proposed airship/airshipctl master: Add single source of Seletors  https://review.opendev.org/71280221:44
openstackgerritRyan Schroder proposed airship/pegleg master: Updated shipyard bionic dependencies  https://review.opendev.org/71184221:45
openstackgerritKostyantyn Kalynovskyi proposed airship/airshipctl master: Add single source of Seletors  https://review.opendev.org/71280221:51
openstackgerritRyan Schroder proposed airship/pegleg master: Updated shipyard bionic dependencies  https://review.opendev.org/71184222:22
*** mfuller has quit IRC22:24
openstackgerritRyan Schroder proposed airship/pegleg master: Updated shipyard bionic dependencies  https://review.opendev.org/71184222:31
openstackgerritKostyantyn Kalynovskyi proposed airship/airshipctl master: Add single source of Seletors  https://review.opendev.org/71280222:31
*** segorov has quit IRC22:54
*** segorov has joined #airshipit22:54
openstackgerritVladimir Maliaev proposed airship/promenade master: Pull hyperkube image before kubelet service  https://review.opendev.org/65216222:56
openstackgerritRyan Schroder proposed airship/pegleg master: Updated shipyard bionic dependencies  https://review.opendev.org/71184223:21
*** dwalt has quit IRC23:36
openstackgerritRyan Schroder proposed airship/pegleg master: Updated shipyard bionic dependencies  https://review.opendev.org/71184223:49
« Wednesday, 2020-03-11 Index Friday, 2020-03-13 »

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!