Wednesday, 2020-03-11

*** AlexNoskov has quit IRC00:04
*** ianychoi has quit IRC00:07
*** ianychoi has joined #airshipit00:08
*** diga has joined #airshipit00:10
*** kkalina has quit IRC00:29
*** nishantkr has quit IRC01:11
*** ianychoi has quit IRC01:30
*** ianychoi has joined #airshipit01:32
*** happyhemant has quit IRC02:09
*** ianychoi has quit IRC02:10
*** ianychoi has joined #airshipit02:18
*** segorov has joined #airshipit02:50
openstackgerriteric welch proposed airship/deckhand master: WIP password rotation for deckhand  https://review.opendev.org/71216203:25
*** ianychoi has quit IRC03:35
*** ianychoi has joined #airshipit03:37
*** diga has quit IRC03:47
*** ianychoi has quit IRC05:00
*** ashferg has quit IRC05:02
*** ianychoi has joined #airshipit05:03
*** rezroo has quit IRC05:33
*** evrardjp has quit IRC05:35
*** evrardjp has joined #airshipit05:35
*** ianychoi has quit IRC06:01
*** ianychoi has joined #airshipit06:03
*** ianychoi has quit IRC06:11
*** ianychoi has joined #airshipit06:13
*** ianychoi has quit IRC06:29
*** ianychoi has joined #airshipit06:31
*** ianychoi has quit IRC06:39
*** ianychoi has joined #airshipit06:41
openstackgerritVladimir Maliaev proposed airship/promenade master: Updated resiliency gate  https://review.opendev.org/70854506:46
*** ianychoi has quit IRC06:58
*** rezroo has joined #airshipit07:01
*** ianychoi has joined #airshipit07:05
*** dpawlik has joined #airshipit07:17
*** timClicks has quit IRC08:51
*** rezroo has quit IRC08:55
*** rezroo has joined #airshipit08:55
*** roman_g has joined #airshipit09:46
-openstackstatus- NOTICE: The mail server for lists.openstack.org is currently not handling emails. The infra team will investigate and fix during US morning.10:24
*** roman_g has quit IRC11:00
*** rezroo has quit IRC11:03
*** ianychoi has quit IRC11:24
openstackgerritAlan Meadows proposed airship/airshipctl master: [#45] iso generation pulls network data from ephemeral host  https://review.opendev.org/70863612:20
openstackgerritDmitry Ukov proposed airship/airshipctl master: Refactor document Bundle interface  https://review.opendev.org/70813212:43
openstackgerritDmitry Ukov proposed airship/airshipctl master: Implement document rendering sub-command  https://review.opendev.org/69865812:43
openstackgerritDmitry Ukov proposed airship/airshipctl master: Fix go get for airshipctl module  https://review.opendev.org/71040012:49
*** roman_g has joined #airshipit12:57
openstackgerritMerged airship/shipyard master: (fix) Address image build issues, bionic  https://review.opendev.org/70905613:00
*** rezroo has joined #airshipit13:21
*** irclogbot_2 has quit IRC13:22
*** irclogbot_2 has joined #airshipit13:22
openstackgerritDmitry Ukov proposed airship/airshipctl master: Show isogen container logs if debug is enabled  https://review.opendev.org/70659414:15
*** rpocase has joined #airshipit14:18
rpocaseAny one have any ideas why a genesis node would get start indefinitely on "Unable to connect to the server: EOF14:19
rpocaseThis is in context of initially bootstrapping kubernetes.14:19
*** dwalt has joined #airshipit14:21
openstackgerritAhmad Mahmoudi proposed airship/deckhand master: Replaced pip with pip3  https://review.opendev.org/71129914:27
openstackgerritAhmad Mahmoudi proposed airship/pegleg master: Updated shipyard bionic dependencies  https://review.opendev.org/71184214:31
*** AlexNoskov has joined #airshipit14:33
openstackgerritDmitry Ukov proposed airship/airshipctl master: [#45] iso generation pulls network data from ephemeral host  https://review.opendev.org/70863614:56
*** SRao has joined #airshipit14:58
openstackgerritDmitry Ukov proposed airship/airshipctl master: [#45] iso generation pulls network data from ephemeral host  https://review.opendev.org/70863615:07
*** segorov has quit IRC15:14
openstackgerritdiwakar thyagaraj proposed airship/maas master: Enable Docker default AppArmor profile to maas  https://review.opendev.org/70513615:14
*** segorov has joined #airshipit15:15
openstackgerritSirajudeen proposed airship/airshipctl master: [#26] - Skip Image download if already downloaded  https://review.opendev.org/71138315:23
*** sreejithp has joined #airshipit15:24
howelljust a heads-up: mirroring with slack is currently not working15:40
howellit should be back in a bit15:40
*** airshipbot has joined #airshipit15:41
mattmceueno/15:41
airshipbot<mattmceuen> \o15:42
howell:)15:42
mattmceuenFixed.  Stil need to get the permanent solution for mirroring set up - haven't had time, but will wrap it up this week I promise :)15:42
airshipbot<kk6740> do we have a plan to deprecate irc completely ?15:43
airshipbot<mattmceuen> Stas Egorov  [11:18 AM] Please, review image tagging: https://review.opendev.org/71199915:43
airshipbot<kk6740> and move to slack?15:43
airshipbot<mattmceuen> Sirajudeen  [10:38 AM] Code review pls, updated PS as per review comments https://review.opendev.org/#/c/711383/15:43
mattmceuenkostiantyn: nope no plans to deprecate IRC15:43
mattmceuenhoping for a "best of both worlds" experience15:43
openstackgerritSirajudeen proposed airship/airshipctl master: [#26] - Skip Image download if already downloaded  https://review.opendev.org/71138315:44
alexanderhughestest15:45
openstackgerritRoman Gorshunov proposed airship/governance master: Update contents table  https://review.opendev.org/71248415:49
openstackgerritMerged airship/airshipctl master: [#86] remove unused code  https://review.opendev.org/71150615:50
openstackgerritMerged airship/airshipctl master: Switch from shell to make module  https://review.opendev.org/71135815:50
openstackgerritMerged airship/airshipctl master: Target test files only in git diff check  https://review.opendev.org/71214815:50
*** jamesgu has joined #airshipit16:17
airshipbot<mattmceuen> FYI -- got a smidge of clarity around the future of kustomize in the SIG-CLI meeting, mostly that "functions are probably the future for plugins" and "there's a lot going on in kustomize" :slightly_smiling_face:16:26
airshipbot<mattmceuen> The plan is for the next SIG-CLI meeting (in two weeks) to be dedicated to a kustomize design and future deep dive16:27
roman_grpocase: "Unable to connect to the server: EOF" - check out why kubelet wasn't able to start. E.g. in journalctl logs.16:27
airshipbot<mattmceuen> That'll let all the kustomize luminaries to join, and I'd encourage folks from our team who have their hands in kustomize to join as well16:27
openstackgerritAlexander Hughes proposed airship/airshipctl master: [#91] refactor secrets as top level command  https://review.opendev.org/71168416:34
alexanderhughesLF reviews on https://review.opendev.org/#/c/710845/ to address gate failures mentioned during the flight plan call16:37
openstackgerritRoman Gorshunov proposed airship/governance master: Add missing contribution guidelines document  https://review.opendev.org/71249916:37
openstackgerritRoman Gorshunov proposed airship/governance master: Add missing contribution guidelines document  https://review.opendev.org/71249916:41
openstackgerritSirajudeen proposed airship/airshipctl master: [#26] - Skip Image download if already downloaded  https://review.opendev.org/71138316:41
rpocaseroman_g: roman_g: systemctl seems to think kubelet is running. I see a handful of errors: node <genesis> not found, various failed to list errors against k8s API, and Failed to set some node status fields: failed to validate nodeIP: Node IP: "<genesis_calico>" not found in the host's network. The first and last seem like possibly leading causes, not but sure where to look from here16:47
*** airshipbot has quit IRC16:49
*** airshipbot has joined #airshipit16:49
openstackgerritFrancis Bacon Yi proposed airship/airshipctl master: [WIP] Increase config pkg test coverage  https://review.opendev.org/71250316:50
roman_grpocase: do you have DNS set up properly? Which site from treasuremap are you using?16:51
rpocaseroman_g: Started from airsloop. Are there instructions for DNS for airsloop? The instructions for the general site authoring for names to register didn't seem to match for airsloop16:55
openstackgerritJagan Mohan Kavva proposed airship/maas master: Enable Docker default AppArmor profile to maas  https://review.opendev.org/70513616:57
rpocasealso, for upstream_dns_servers, should I be including my internal dns servers?  I guess I assumed I would get internal dns set through dhcp and upstream_dns_servers would be for extra defaults16:58
openstackgerritRoman Gorshunov proposed airship/images master: Add python3-requests and python3-docker packages  https://review.opendev.org/71250516:59
openstackgerritMerged airship/airshipctl master: [#70] Resolve kubectl gate errors  https://review.opendev.org/71084517:00
openstackgerritAlexander Hughes proposed airship/airshipctl master: [#70] Remove unnecessary kubectl subcommand  https://review.opendev.org/71008517:05
openstackgerritAlexander Hughes proposed airship/airshipctl master: [#91] refactor secrets as top level command  https://review.opendev.org/71168417:05
roman_grpocase: no, DHCP wouldn't work here. List your internal DNS server IP addresses in common-addresses.yaml17:13
rpocaseroman_g: Thanks! Will try that and see how it goes17:14
roman_grpocase: I couldn't find any reference to https://github.com/airshipit/treasuremap/search?q=genesis_calico&unscoped_q=genesis_calico in yamls in repo. This is what is shown in one of your error messages. Could it be that you've accidently did copy-paste error somewhere?17:16
*** SRao has quit IRC17:17
openstackgerritAhmad Mahmoudi proposed airship/deckhand master: Replaced pip with pip3  https://review.opendev.org/71129917:22
openstackgerritMerged airship/airshipctl master: [#45] iso generation pulls network data from ephemeral host  https://review.opendev.org/70863617:24
rpocaseroman_g: https://github.com/airshipit/treasuremap/blob/a75861256a58e5ee32d9ea60bcb9d6f3281555c8/site/airsloop/networks/common-addresses.yaml#L55-L5717:27
roman_grpocase: ah, so instead of "<genesis_calico>" you actually have IP address there17:34
rpocaseYep17:34
*** evrardjp has quit IRC17:35
*** evrardjp has joined #airshipit17:35
roman_grpocase: then you would need to recheck your common-addresses, pki-catalog, networks and nodes .yaml files17:37
roman_gand verify against actual node configuration17:38
openstackgerritJagan Mohan Kavva proposed airship/maas master: Enable Docker default AppArmor profile to maas  https://review.opendev.org/70513617:42
*** segorov has quit IRC18:10
rpocaseroman_g: Just retried with updated DNS and that doesn't seem to have changed anything. If I manually assign <genesis_calico_ip> to the host NIC then that particular warning goes away. Am I supposed to manually supply each static IP address to all nodes? (e.g., in this case assign 5 addresses to my PXE/host NIC to be in line with airsloop)18:14
*** segorov has joined #airshipit18:24
openstackgerritdiwakar thyagaraj proposed airship/maas master: [WIP]Enable Logs Output in Zuul for Deployment Scripts  https://review.opendev.org/70976318:31
openstackgerritdiwakar thyagaraj proposed airship/maas master: [WIP]Enable Logs Output in Zuul for Deployment Scripts  https://review.opendev.org/70976318:32
openstackgerritAhmad Mahmoudi proposed airship/deckhand master: WIP: Replaced pip with pip3  https://review.opendev.org/71129918:35
openstackgerritSreejith Punnapuzha proposed airship/airshipctl master: [#106] Add docker install role  https://review.opendev.org/71154018:42
openstackgerritSreejith Punnapuzha proposed airship/airshipctl master: [#106] Add docker install role  https://review.opendev.org/71154018:43
airshipbot<rp2723> Would it make sense to have sig-yaml sig-ui etc Channels in this workspace ?18:52
mattmceuenrp2723: if we need 'em, but I'd suggest we start in one channel.  I don't think we have "too much discussion" yet, and that would simplify channel mapping between IRC and slack18:58
openstackgerritAhmad Mahmoudi proposed airship/pegleg master: (fix) six package not found  https://review.opendev.org/71253519:02
openstackgerritSirajudeen proposed airship/airshipctl master: [#21] - prompt for config init options  https://review.opendev.org/71018019:20
alexanderhughesyes please, don't leave us IRC lovers behind with some complex mapping of distinct channels in slack to single space in IRC19:24
airshipbot<dwalt> :P19:25
airshipbot<dwalt> join us alexanderhughes19:26
alexanderhughesonly if you have milk and cookies19:26
airshipbot<dwalt> :cookie: :glass_of_milk:19:26
alexanderhughessold!19:26
rpocaseHow does one go about getting a slack invite? :)19:26
airshipbot<dwalt> You can join our workspace! airshipproject.slack.com19:27
airshipbot<dwalt> If you don't have one of the pre-approved domains, I think we need to extend an invite19:27
alexanderhughesRE https://airshipit.readthedocs.io/projects/airship-docs/en/latest/security/vulnerabilities.html what tools are we using to scan for golang vulnerabilities?19:27
rpocaseYeah, would need an invite link19:28
alexanderhughesbandit was adopted in python projects of airship119:28
airshipbot<dwalt> rpocase: I'm on IRC as well, could you send me your email?19:28
airshipbot<dwalt> rpocase: Thanks. Sent an invitation :slightly_smiling_face:19:28
alexanderhughesI imagine there are some practices that are best avoided, golang equivalencies to python's subprocess module for example19:28
alexanderhughesand we'll want to scan for those and gate if possible19:29
howellalexanderhughes: we're using gosec from within golangci-lint19:29
howellalthough we haven't configured it at all, so I'm not exactly sure what it's checking right now19:30
alexanderhughesexcellent!  was the first search result I got when looking for tools19:30
*** rpocase has quit IRC19:30
alexanderhugheswithout configuration looks like it checks this default list of rules https://github.com/securego/gosec#available-rules19:31
openstackgerritSirajudeen proposed airship/airshipctl master: [#21] - prompt for config init options  https://review.opendev.org/71018019:32
openstackgerritSirajudeen proposed airship/airshipctl master: [#21] - prompt for config init options  https://review.opendev.org/71018019:34
alexanderhughesso the other thing to check into is docker image vulnerabilities since we're putting those out.  quay.io uses clair to scan (compatible) images once pushed, but we haven't pushed any images yet... has anyone tried running clair locally yet against airshipctl?19:36
airshipbot<dwalt> I need to verify why our images are not pushing. I meant to do that after my change merged19:38
alexanderhughesok, going to try to run clair locally rq against master19:38
openstackgerritSirajudeen proposed airship/airshipctl master: [#21] - prompt for config init options  https://review.opendev.org/71018019:41
airshipbot<dwalt> I'm actually not sure why it's looking for a 'repo' variable http://zuul.openstack.org/build/d553b5f7183e45c3a8651bc3b7a888c4/log/job-output.txt#56819:43
airshipbot<dwalt> The role doesn't even have a variable called 'repo'19:43
airshipbot<dwalt> https://github.com/airshipit/airshipctl/blob/master/roles/airshipctl-publish-images/tasks/main.yaml19:43
openstackgerritSirajudeen proposed airship/airshipctl master: [#21] - prompt for config init options  https://review.opendev.org/71018019:44
alexanderhughesis image_repo even defined?  it's used in registry_url, but in pegleg it's just hardcoded https://opendev.org/airship/pegleg/src/branch/master/tools/gate/playbooks/docker-image-build.yaml#L11319:46
airshipbot<dwalt> alexanderhughes: It's defined in `jobs.yaml`https://github.com/airshipit/airshipctl/blob/master/zuul.d/jobs.yaml#L9419:48
openstackgerritAhmad Mahmoudi proposed airship/deckhand master: WIP: Replaced pip with pip3  https://review.opendev.org/71129919:54
openstackgerritAhmad Mahmoudi proposed airship/pegleg master: (fix) six package not found  https://review.opendev.org/71253520:24
airshipbot<sirajudeen.yasin> @team, am getting -1 from zuul for this commit https://review.opendev.org/#/c/710180/ going through console in zuul, "FAIL: Test coverage is at 79.7%, which does not meet the required coverage (80%)" am trying to understand how is it related to my commit.. is it because i added a function but did not add a test case for it20:36
airshipbot<ih616h> @sirajudeen.yasin you need unit tests20:37
airshipbot<ih616h> your code has brought the overall coverage below the requirements20:38
airshipbot<ih616h> it might be worth it to change some of those calls from `Scan` to `Fscan`. That way you can use `os.Stdin` for the real thing, but something like `bytes.Buffer` for your tests20:40
openstackgerritAhmad Mahmoudi proposed airship/deckhand master: Replaced pip with pip3  https://review.opendev.org/71129920:44
openstackgerritVamsi Savaram proposed airship/airshipctl master: [#89] DNM: Gather container logs in check pipeline  https://review.opendev.org/71255520:56
openstackgerritJagan Mohan Kavva proposed airship/maas master: Enable Docker default AppArmor profile to maas  https://review.opendev.org/70513621:00
airshipbot<alexander.hughes> any way to mute openstackgerrit while on slack?21:01
openstackgerritFrancis Bacon Yi proposed airship/airshipctl master: [WIP] Increase config pkg test coverage  https://review.opendev.org/71250321:03
openstackgerritKostyantyn Kalynovskyi proposed airship/airshipctl master: Add repository to airship config template, ansible  https://review.opendev.org/71182221:07
openstackgerritKostyantyn Kalynovskyi proposed airship/airshipctl master: [WIP] Split document model, add entrypoints for repos  https://review.opendev.org/71157121:07
openstackgerritdiwakar thyagaraj proposed airship/maas master: Enable Docker default AppArmor profile to maas  https://review.opendev.org/70513621:12
openstackgerritDrew Walters proposed airship/airshipctl master: WIP: Provide Redfish feedback in RemoteDirect errors  https://review.opendev.org/71199221:29
openstackgerritIan Howell proposed airship/airshipctl master: [#73] WIP: Add airshipctl cluster status command  https://review.opendev.org/71081421:39
airshipbot<se6518> It's better to move it to a separate channel for notifications only. Slack can't block/mute users.21:39
*** jamesgu has quit IRC21:40
airshipbot<se6518> Please review local registry: https://review.opendev.org/71200321:46
airshipbot<dwalt> @se6518: review for review? https://review.opendev.org/71164421:47
openstackgerritJagan Mohan Kavva proposed airship/maas master: Enable Docker default AppArmor profile to maas  https://review.opendev.org/70513621:51
openstackgerritJagan Mohan Kavva proposed airship/maas master: Enable Docker default AppArmor profile to maas  https://review.opendev.org/70513621:52
openstackgerritFrancis Bacon Yi proposed airship/airshipctl master: [WIP] Increase config pkg test coverage  https://review.opendev.org/71250321:58
*** sreejithp has quit IRC22:01
openstackgerritdiwakar thyagaraj proposed airship/maas master: [WIP]Enable Logs Output in Zuul for Deployment Scripts  https://review.opendev.org/70976322:02
openstackgerritKostyantyn Kalynovskyi proposed airship/airshipctl master: [WIP] Split document model, add entrypoints for repos  https://review.opendev.org/71157122:04
*** dpawlik has quit IRC22:11
airshipbot<se6518> @dwalt would you, please, explain this from code review: "One concern about specifying a registry. Is this the direction we intend to go? I don't seen an issue for this and assumed we will pull all of our images." ?22:25
airshipbot<se6518> I'm changing registry only for local use22:25
airshipbot<se6518> for regular use it should be in airshipit space on quay.io22:26
*** portdirect has quit IRC22:31
*** mattmceuen has quit IRC22:31
*** jbryce has quit IRC22:31
*** seaneagan has quit IRC22:31
*** jemangs has quit IRC22:31
*** jbryce has joined #airshipit22:34
*** portdirect has joined #airshipit22:34
openstackgerritdiwakar thyagaraj proposed airship/maas master: [WIP]Enable Logs Output in Zuul for Deployment Scripts  https://review.opendev.org/70976322:36
*** jemangs has joined #airshipit22:36
*** francisy_ has quit IRC22:36
*** AlexNoskov has quit IRC22:36
*** phrobb has quit IRC22:36
*** vladyslav has quit IRC22:36
*** mnaser has quit IRC22:36
*** dwalt has quit IRC22:36
*** v1k0d3n has quit IRC22:36
*** alexanderhughes has quit IRC22:36
*** mnaser has joined #airshipit22:39
*** seaneagan has joined #airshipit22:39
*** phrobb has joined #airshipit22:40
*** v1k0d3n has joined #airshipit22:40
*** dwalt has joined #airshipit22:43
*** vladyslav has joined #airshipit22:44
*** AlexNoskov has joined #airshipit22:44
*** alexanderhughes has joined #airshipit22:44
openstackgerritJagan Mohan Kavva proposed airship/maas master: Enable Docker default AppArmor profile to maas  https://review.opendev.org/70513622:45
*** lamt has quit IRC22:46
*** evgenyl has quit IRC22:46
*** irclogbot_2 has quit IRC22:47
*** irclogbot_3 has joined #airshipit22:47
*** evgenyl has joined #airshipit22:47
*** openstackstatus has quit IRC22:48
*** gagehugo has quit IRC22:50
*** megheisler has quit IRC22:50
*** ian-pittwood has quit IRC22:50
*** alanmeadows has quit IRC22:50
*** alanmeadows has joined #airshipit22:53
*** gagehugo has joined #airshipit22:55
*** ildikov has quit IRC22:56
*** hemanth_n has quit IRC22:56
*** iamweswilson has quit IRC22:56
*** howell has quit IRC22:56
*** srwilkers has quit IRC22:56
*** jayahn has quit IRC22:56
*** aprice has quit IRC22:56
*** megheisler has joined #airshipit22:57
*** ian-pittwood has joined #airshipit22:57
*** ildikov has joined #airshipit22:59
*** aprice has joined #airshipit23:00
*** srwilkers has joined #airshipit23:01
*** iamweswilson has joined #airshipit23:02
*** howell has joined #airshipit23:02
openstackgerritMerged airship/airshipctl master: tools: Use local connection for ansible runner  https://review.opendev.org/71164423:02
*** hemanth_n has joined #airshipit23:04
*** zogger has quit IRC23:06
*** ianychoi has joined #airshipit23:06
*** zogger has joined #airshipit23:12
openstackgerritAlan Meadows proposed airship/airshipctl master: Properly locate tests within a config_test pkg for pkg/config and cmd/config  https://review.opendev.org/71258423:21
openstackgerritAlan Meadows proposed airship/airshipctl master: Properly locate tests within a config_test pkg for pkg/config and cmd/config  https://review.opendev.org/71258423:22
*** segorov has quit IRC23:34
*** KeithMnemonic has quit IRC23:51
*** KeithMnemonic has joined #airshipit23:51
*** rezroo has quit IRC23:54
*** rezroo has joined #airshipit23:54

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