Monday, 2020-03-16

openstackgerritJagan Mohan Kavva proposed airship/maas master: Enable Docker default AppArmor profile to maas  https://review.opendev.org/70513600:03
openstackgerritJagan Mohan Kavva proposed airship/maas master: Enable Docker default AppArmor profile to maas  https://review.opendev.org/70513600:11
*** mfuller has joined #airshipit00:20
*** mfuller has quit IRC00:26
*** rezroo has quit IRC01:36
*** rezroo has joined #airshipit02:40
*** mfuller has joined #airshipit04:09
*** segorov has joined #airshipit04:31
*** mfuller has quit IRC04:57
*** evrardjp has quit IRC05:35
*** evrardjp has joined #airshipit05:36
openstackgerritVladimir Maliaev proposed airship/promenade master: Adjust plugin cri parameters  https://review.opendev.org/71316405:44
*** irclogbot_3 has quit IRC06:29
*** mfuller has joined #airshipit06:49
*** rezroo has quit IRC06:49
*** dpawlik has joined #airshipit06:58
*** irclogbot_1 has joined #airshipit07:28
openstackgerritMerged airship/airshipctl master: Add single source of Seletors  https://review.opendev.org/71280207:45
*** dpawlik has quit IRC08:07
*** dpawlik has joined #airshipit08:07
*** dpawlik has quit IRC08:18
*** dpawlik has joined #airshipit08:28
*** mfuller has quit IRC08:31
openstackgerritDmitry Ukov proposed airship/airshipctl master: [WIP]Ensure node power state ON/OFF for Reset command  https://review.opendev.org/71320111:46
openstackgerritDmitry Ukov proposed airship/airshipctl master: [WIP]Improve test coverage  https://review.opendev.org/71320712:14
openstackgerritJagan Mohan Kavva proposed airship/maas master: Enable Docker default AppArmor profile to maas  https://review.opendev.org/70513612:35
openstackgerritSirajudeen proposed airship/airshipctl master: Increase test coverage  https://review.opendev.org/71312513:01
*** rezroo has joined #airshipit13:21
openstackgerritSreejith Punnapuzha proposed airship/airshipctl master: [#106] Add docker install role  https://review.opendev.org/71154013:24
openstackgerritMerged airship/images master: Fix publish job  https://review.opendev.org/71306613:26
*** dpawlik has quit IRC13:29
openstackgerritMerged airship/airshipctl master: Fix network and cloud data for ephemeral host  https://review.opendev.org/71282913:39
openstackgerritMerged airship/airshipctl master: Show isogen container logs if debug is enabled  https://review.opendev.org/70659413:41
openstackgerritSirajudeen proposed airship/airshipctl master: Increase test coverage  https://review.opendev.org/71312514:00
openstackgerritAndrew Schiefelbein proposed airship/airshipui master: [WIP] This adds a plugin for OpenStack to the Airship UI  https://review.opendev.org/71278814:04
openstackgerritAndrew Schiefelbein proposed airship/airshipui master: [WIP] This adds a plugin for OpenStack to the Airship UI  https://review.opendev.org/71278814:13
openstackgerritJagan Mohan Kavva proposed airship/maas master: Enable Docker default AppArmor profile to maas  https://review.opendev.org/70513614:22
openstackgerritSirajudeen proposed airship/airshipctl master: Increase test coverage  https://review.opendev.org/71312514:22
openstackgerritAndrew Schiefelbein proposed airship/airshipui master: [WIP] This adds a plugin for OpenStack to the Airship UI  https://review.opendev.org/71278814:45
airship-irc-bot<dwalt> From Friday- does anyone know the answer to this? Posting again for more visibility :slightly_smiling_face:  >  Is the proper command to fix goimports `goimports -local "opendev.org/airship/airshipctl" -w .`?14:47
airship-irc-bot<ih616h> @dwalt that should work14:47
airship-irc-bot<ih616h> I have seen issues with it sometimes hanging though14:47
airship-irc-bot<dwalt> @ih616h my surprise was that it modifies several files that were not originally modified in my commit. That causes linting failures14:48
airship-irc-bot<dwalt> This change: https://review.opendev.org/71199214:48
airship-irc-bot<ih616h> "opendev.org/airship/go-redfish/client" isn't an `airshipctl` package14:52
airship-irc-bot<ih616h> i'm not sure why `goimports` would reorder it that way though14:52
airship-irc-bot<ih616h> maybe move that package to its own block above?14:52
airship-irc-bot<dwalt> I'll try that. Thanks :slightly_smiling_face:14:53
openstackgerritBlake Dittmer proposed airship/pegleg master: mend  https://review.opendev.org/71325014:53
openstackgerritDrew Walters proposed airship/airshipctl master: Provide Redfish feedback in RemoteDirect errors  https://review.opendev.org/71199214:55
*** segorov has quit IRC15:05
*** segorov has joined #airshipit15:07
portdirectfor the convo going on atm in the sig-yaml call: https://raw.githubusercontent.com/kubernetes/kubernetes/master/api/openapi-spec/swagger.json15:17
openstackgerritJagan Mohan Kavva proposed airship/maas master: Enable Docker default AppArmor profile to maas  https://review.opendev.org/70513615:46
openstackgerritIan Howell proposed airship/airshipctl master: WIP: Add the dynamic kubernetes client  https://review.opendev.org/71296316:01
openstackgerritDmitry Ukov proposed airship/airshipctl master: Ensure node power state ON/OFF for Reset command  https://review.opendev.org/71320116:31
airship-irc-bot<an310y> Hi guys, please merge https://review.opendev.org/#/q/topic:vosh+(status:open+OR+status:merged), changes for vosh deployment. Already have CR+2 and Verified+116:46
airship-irc-bot<dwalt> Hey @dukov: I have a question on https://review.opendev.org/711992 when you have the chance. I haven't been able to figure out how re-generating our redfish code will provide the error feedback this patch supplies, but if it does, I'm happy to move this patch towards that integration. Whether it be fixing the unit tests or moving my logic elsewhere, it's no trouble.16:51
*** brtknr_ has quit IRC16:57
*** brtknr has joined #airshipit16:57
openstackgerritIan Howell proposed airship/airshipctl master: Add the dynamic kubernetes client  https://review.opendev.org/71296317:02
openstackgerritDrew Walters proposed airship/airshipctl master: [#33] Convert secret names to underscores  https://review.opendev.org/71329417:19
openstackgerritDrew Walters proposed airship/airshipctl master: [#33] Convert secret names to underscores  https://review.opendev.org/71329417:20
*** mfuller has joined #airshipit17:23
*** mcriswell has joined #airshipit17:28
openstackgerritMerged airship/treasuremap master: Add support for modifying genesis kernel parameters for OVS-DPDK  https://review.opendev.org/70963817:34
*** evrardjp has quit IRC17:35
*** evrardjp has joined #airshipit17:36
openstackgerritMatt McEuen proposed airship/images master: [WIP] Fix image build  https://review.opendev.org/71329917:41
openstackgerritJordan Jensen proposed airship/airshipui master: [WIP] Use .Get when obtaining argo-ui endpoint  https://review.opendev.org/71330017:42
openstackgerritMatt McEuen proposed airship/images master: [WIP] Fix image build  https://review.opendev.org/71329917:42
openstackgerritMatt McEuen proposed airship/images master: [WIP] Fix image build  https://review.opendev.org/71329917:45
airship-irc-bot<dwalt> Looking for reviews on this change to (hopefully) fix the airshipctl image publish job https://review.opendev.org/71329417:48
openstackgerritMatt McEuen proposed airship/images master: Zuul: Use underscores for ansible vars  https://review.opendev.org/71330417:57
openstackgerritChris Wedgwood proposed airship/promenade master: [uwsgi] remove extraneous argument  https://review.opendev.org/71330617:59
openstackgerritMatt McEuen proposed airship/images master: [WIP] Fix image build  https://review.opendev.org/71329918:01
openstackgerritAlexander Noskov proposed airship/treasuremap master: Install the same kernel version for Genesis node  https://review.opendev.org/68049518:03
openstackgerritMichael Beaver proposed airship/divingbell master: Update dpkg commands to be non-interactive  https://review.opendev.org/71330918:09
openstackgerritMerged airship/airshipctl master: [#33] Convert secret names to underscores  https://review.opendev.org/71329418:18
openstackgerritMerged airship/images master: Zuul: Use underscores for ansible vars  https://review.opendev.org/71330418:39
openstackgerritDrew Walters proposed airship/airshipctl master: [#33] Fix secret name  https://review.opendev.org/71331118:42
openstackgerritdiwakar thyagaraj proposed airship/maas master: [WIP]Enable Logs Output in Zuul for Deployment Scripts  https://review.opendev.org/70976319:00
openstackgerritIan Howell proposed airship/airshipctl master: WIP: Cluster Status library tools  https://review.opendev.org/71331919:28
airship-irc-bot<dwalt> @mattmceuen @ih616h Thanks for getting the last image publish attempt in. There was an oversight in my patch. If anyone has a chance to take a look at the second attempt, I'd appreciate it https://review.opendev.org/71331119:29
airship-irc-bot<sirajudeen.yasin> Code review pls..https://review.opendev.org/#/c/713125/  , updated as per review comments19:34
*** segorov has quit IRC19:41
openstackgerritStas Egorov proposed airship/airshipctl master: [#103]: replaced external methods  https://review.opendev.org/71332119:50
openstackgerritStas Egorov proposed airship/airshipctl master: Use 16GB flavor for gate job  https://review.opendev.org/71216019:50
openstackgerritMatt McEuen proposed airship/images master: [WIP] Fix image build  https://review.opendev.org/71329920:01
openstackgerritMerged airship/airshipctl master: [#33] Fix secret name  https://review.opendev.org/71331120:21
openstackgerritMatt McEuen proposed airship/images master: [WIP] Fix image build  https://review.opendev.org/71329920:33
openstackgerritdiwakar thyagaraj proposed airship/maas master: [WIP]Enable Logs Output in Zuul for Deployment Scripts  https://review.opendev.org/70976320:39
openstackgerritJagan Mohan Kavva proposed airship/maas master: Enable Docker default AppArmor profile to maas  https://review.opendev.org/70513620:42
openstackgerritMatt McEuen proposed airship/images master: [WIP] Fix image build  https://review.opendev.org/71329920:45
airship-irc-bot<an310y> @dwalt Hi Drew, can you give WF+1 again for this one https://review.opendev.org/#/c/680495/. I’ve rebased it.20:59
airship-irc-bot<dwalt> np @an310y20:59
airship-irc-bot<an310y> thx20:59
openstackgerritMatt McEuen proposed airship/images master: [WIP] Fix image build  https://review.opendev.org/71329921:04
*** mcriswell has quit IRC21:06
openstackgerritMerged airship/treasuremap master: Install the same kernel version for Genesis node  https://review.opendev.org/68049521:08
openstackgerritMatt McEuen proposed airship/images master: Fix image build  https://review.opendev.org/71329921:17
airship-irc-bot<an310y> @dwalt can you give WF+1 for these as well https://review.opendev.org/#/q/topic:vosh+(status:open)21:30
openstackgerritDrew Walters proposed airship/airshipctl master: [#52] Provide Redfish feedback in RemoteDirect  https://review.opendev.org/71199221:31
*** rezroo has quit IRC21:35
*** rezroo has joined #airshipit21:36
airship-irc-bot<dwalt> @an310y done :slightly_smiling_face:21:36
airship-irc-bot<an310y> great! thanks!21:36
openstackgerritAndrew Schiefelbein proposed airship/airshipui master: [WIP] This adds a plugin for OpenStack to the Airship UI  https://review.opendev.org/71278821:37
openstackgerritJagan Mohan Kavva proposed airship/maas master: Enable Docker default AppArmor profile to maas  https://review.opendev.org/70513621:41
openstackgerritMerged airship/treasuremap master: Add unsafe profile for VM's disk  https://review.opendev.org/71048021:43
openstackgerritMerged airship/treasuremap master: Parametrize timeout parameter for shipyard action  https://review.opendev.org/71156621:46
openstackgerritMerged airship/images master: Fix image build  https://review.opendev.org/71329922:05
openstackgerritStas Egorov proposed airship/airshipctl master: [#103]: replaced external methods for document interface  https://review.opendev.org/71332122:09
openstackgerritStas Egorov proposed airship/airshipctl master: [#102]: removed methods for bundle interface  https://review.opendev.org/71333622:09
*** rezroo has quit IRC23:59
*** rezroo has joined #airshipit23:59

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