Friday, 2020-03-13

openstackgerritJagan Mohan Kavva proposed airship/maas master: Enable Docker default AppArmor profile to maas  https://review.opendev.org/70513601:16
openstackgerritAhmad Mahmoudi proposed airship/pegleg master: WIP: Updating airshp clients  https://review.opendev.org/71282502:23
openstackgerritKostyantyn Kalynovskyi proposed airship/airshipctl master: Add single source of Seletors  https://review.opendev.org/71280202:57
openstackgerritRuslan Aliev proposed airship/airshipctl master: [WIP] Fix network and cloud data for ephemeral host  https://review.opendev.org/71282903:25
openstackgerritRuslan Aliev proposed airship/airshipctl master: [WIP] Fix network and cloud data for ephemeral host  https://review.opendev.org/71282903:28
openstackgerritSirajudeen proposed airship/airshipctl master: [#21] - prompt for config init options  https://review.opendev.org/71018003:32
openstackgerritAhmad Mahmoudi proposed airship/pegleg master: WIP: Updating pegleg airship clients  https://review.opendev.org/71282503:32
*** evrardjp has quit IRC05:35
*** evrardjp has joined #airshipit05:36
openstackgerritAhmad Mahmoudi proposed airship/pegleg master: WIP: Updating pegleg airship clients  https://review.opendev.org/71282505:56
openstackgerritAhmad Mahmoudi proposed airship/pegleg master: WIP: Updating pegleg airship clients  https://review.opendev.org/71282506:33
*** dpawlik has quit IRC06:37
*** dpawlik has joined #airshipit06:55
*** roman_g has quit IRC07:01
*** dpawlik has quit IRC07:10
*** dpawlik has joined #airshipit07:16
openstackgerritDmitry Ukov proposed airship/airshipctl master: Fix go get for airshipctl module  https://review.opendev.org/71040007:25
openstackgerritDmitry Ukov proposed airship/airshipctl master: Show isogen container logs if debug is enabled  https://review.opendev.org/70659408:53
openstackgerritVamsi Savaram proposed airship/airshipctl master: [#89] Gather container logs in check pipeline  https://review.opendev.org/71255509:07
*** rezroo has quit IRC09:28
openstackgerritRuslan Aliev proposed airship/airshipctl master: Fix network and cloud data for ephemeral host  https://review.opendev.org/71282909:30
openstackgerritDmitry Ukov proposed airship/airshipctl master: Implement document rendering sub-command  https://review.opendev.org/69865812:04
openstackgerritNikolay Fedorov proposed airship/airshipctl master: Add cert-manager  https://review.opendev.org/71293212:20
openstackgerritNikolay Fedorov proposed airship/airshipctl master: [WIP] Add cert-manager  https://review.opendev.org/71293212:25
openstackgerritNikolay Fedorov proposed airship/airshipctl master: Update kubeadm  https://review.opendev.org/71293512:30
openstackgerritNikolay Fedorov proposed airship/airshipctl master: Update baremetal-provider  https://review.opendev.org/71293612:33
openstackgerritAlexander Hughes proposed airship/airshipctl master: [#91] refactor secrets as top level command  https://review.opendev.org/71168413:00
openstackgerritdiwakar thyagaraj proposed airship/maas master: [WIP]Enable Logs Output in Zuul for Deployment Scripts  https://review.opendev.org/70976313:17
openstackgerritNikolay Fedorov proposed airship/airshipctl master: [WIP] Add cert-manager  https://review.opendev.org/71293213:20
openstackgerritNikolay Fedorov proposed airship/airshipctl master: Update kubeadm  https://review.opendev.org/71293513:20
openstackgerritNikolay Fedorov proposed airship/airshipctl master: Update baremetal-provider  https://review.opendev.org/71293613:20
openstackgerritNikolay Fedorov proposed airship/airshipctl master: Update cluster-api  https://review.opendev.org/71294413:20
openstackgerritJagan Mohan Kavva proposed airship/maas master: Enable Docker default AppArmor profile to maas  https://review.opendev.org/70513613:20
openstackgerritNikolay Fedorov proposed airship/airshipctl master: Upgrade Cluster API from v1alpha2 to v1alpha3  https://review.opendev.org/71294413:38
airshipbot<ih616h> This issue implies that we want to use the `argo` CLI as a subclient to `airshipctl`: https://github.com/airshipit/airshipctl/issues/9613:40
airshipbot<ih616h> I thought we decided to abandon the idea of subclients?13:40
*** rezroo has joined #airshipit13:55
openstackgerritdiwakar thyagaraj proposed airship/maas master: [WIP]Enable Logs Output in Zuul for Deployment Scripts  https://review.opendev.org/70976314:01
airshipbot<alexander.hughes> that was my understanding as well, and the basis for removing kubectl subcommand14:07
openstackgerritJagan Mohan Kavva proposed airship/maas master: Enable Docker default AppArmor profile to maas  https://review.opendev.org/70513614:17
openstackgerritAlexander Hughes proposed airship/airshipctl master: Use 16GB flavor for gate job  https://review.opendev.org/71216014:23
openstackgerritAlexander Hughes proposed airship/airshipctl master: Add Bare Metal Operator resources  https://review.opendev.org/70653314:23
openstackgerritMerged airship/airshipctl master: Fix go get for airshipctl module  https://review.opendev.org/71040014:32
airshipbot<sirajudeen.yasin> I just rebased this commit https://review.opendev.org/#/c/710180/ and zuul gave -1 due to coverage tests, so i assume it is because of few other recent commits as well.. can we create a issue to add more test cases wherever possible, I will try to work on it14:33
airshipbot<ih616h> That sounds like a good idea to me. There is currently this PS, which is also working toward an increase in coverage, so if you write any tests, just make sure you aren't duplicating any work: https://review.opendev.org/#/c/712503/14:35
airshipbot<sirajudeen.yasin> thanks Ian, the above commit should help a lot.. I will add if i can any more test cases14:37
openstackgerritIan Howell proposed airship/airshipctl master: WIP: Add the dynamic kubernetes client  https://review.opendev.org/71296314:40
airshipbot<alexander.hughes> so 712503 is two sided14:49
airshipbot<alexander.hughes> it increases test coverage to 84%, but also increases minimum threshold to 82%14:49
airshipbot<alexander.hughes> for the sake of being nimble we may want to keep threshold at 80% as we get new features in.  I'm an advocate for testing where we can, but some of the stuff being implemented is framework for future commands and until the future work is done it can't really be tested14:50
airshipbot<sirajudeen.yasin> i just ran coverage with that PS 712503 and got 82.2% while 82 is threshold.. might be due to other recent merges14:50
airshipbot<sirajudeen.yasin> i agree with alex14:52
airshipbot<dwalt> I've observed low quality tests getting added to codebases where the threshold artificially increases. It can become a nightmare14:52
airshipbot<sb464f> Hey Team GM. I am deploying one site internally and running into this issue. ```TASK [/tmp/airshipctl/roles/airshipctl-build-ephemeral-iso : build ephemeral node iso] **********************************************************************************************************************  fatal: [primary]: FAILED! => {"changed": true, "cmd": ["airshipctl", "bootstrap", "isogen"], "delta": "0:00:00.029178", "end14:54
airshipbot"msg": "non-zero return code", "rc": 1, "start": "2020-03-12 17:48:42.127352", "stderr": "", "stderr_lines": [], "stdout": "Current context manifest must have primary repository set", "stdout_lines": ["Current context manifest must have primary repository set"]}    ``` I passed the variables that role is using, am I missing any other variabkes..?14:54
airshipbot<ih616h> Can't help, but it might be worth mentioning that I'm also experiencing the above issue14:55
airshipbot<alexander.hughes> `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 return code", "rc": 1, "start": "2020-03-12 18:25:44.341066", "stderr": "", "stderr_lines": [], "stdout14:56
airshipbotbuilder 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 \"~G_~V_Secret\" found no documents"]}`14:56
airshipbot<alexander.hughes> I got that one last night, there's ongoing work to address14:56
airshipbot<dwalt> Could you share your airship config?14:56
airshipbot<alexander.hughes> Kostiantyn mentioned it14:56
airshipbot<dwalt> @sb464f14:56
airshipbot<sb464f> 1 sec14:56
airshipbot<sb464f> that has IP's of our site let me edit and send.14:57
airshipbot<sb464f> ```sudo cat /home/sai/.airship/config apiVersion: airshipit.org/v1alpha1 clusters:   dummycluster:     cluster-type:       ephemeral:         bootstrap-info: dummy_bootstrap_config         cluster-kubeconf: dummycluster_ephemeral contexts:   dummy_cluster:     context-kubeconf: dummy_cluster     manifest: dummy_manifest current-context: dummy_cluster kind: Config manifests:   dummy_manifest:     primary-repo14:58
airshipbot  primary:         checkout:           branch: "master"           force: false           remote-ref: ""           tag: ""         url: http://HOST_IP:8095/nc-airship-2.0/     ## this is temporary hack, as soon as we use `document pull` command in gate process     ## this will subpath will be airshipctl/manifests/site/test-bootstrap, as airshipctl     ## will be primary repository     sub-path: "manifests/site/test-bo14:58
airshipbot/home/sai/nc-airship-2.0/ modules-config:   bootstrapInfo:     dummy_bootstrap_config:       container:         volume: /data/srv/iso:/config         image: kkalynovskyi/image-builder:latest         containerRuntime: docker       remoteDirect:         remoteType: redfish         isoUrl: http://HOST_IP:8095/debian-custom.iso       builder:         userDataFileName: user-data         networkConfigFileName: network-conf14:58
airshipbotoutput-metadata.yaml users:```14:58
airshipbot<dwalt> That looks like the correct usage to me. I will have to take a look at the config module.14:59
*** segorov has quit IRC15:05
openstackgerritKostyantyn Kalynovskyi proposed airship/airshipctl master: Add single source of Seletors  https://review.opendev.org/71280215:12
airshipbot<kk6740> this issue is being addressed right now with this commit, @dwalt @alexander.hughes15:13
airshipbot<kk6740> https://review.opendev.org/71280215:13
airshipbot<kk6740> and there is one more that needs to be done, since we have new ways to get network data15:14
airshipbot<kk6740> but no network data15:14
airshipbot<dwalt> @kk6740 that's great! That won't fix @sb464f's problem though, right?15:15
openstackgerritAhmad Mahmoudi proposed airship/pegleg master: Bionic pegleg airship clients  https://review.opendev.org/71282515:16
airshipbot<kk6740> @dwalt it will15:16
airshipbot<kk6740> this line https://review.opendev.org/#/c/712802/5/roles/airshipctl-test-configs/templates/airshipconfig.j215:16
airshipbot<sb464f> i think it does https://review.opendev.org/#/c/712802/5/roles/airshipctl-test-configs/templates/airshipconfig.j215:16
airshipbot<kk6740> because of gates not working, these issues keep popping up,15:16
airshipbot<dwalt> oh, I flew past that one. Awesome. :slightly_smiling_face:15:17
airshipbot<kk6740> if those servers would stop giving node_failures, we could enable to job to be voting15:17
airshipbot<kk6740> and we could see15:17
airshipbot<ih616h> why don't we put that single line change into its own PS? It doesn't seem related to the rest of the change and would be uch easier to merge15:17
openstackgerritFrancis Bacon Yi proposed airship/airshipctl master: Increase config pkg test coverage  https://review.opendev.org/71250315:23
airshipbot<kk6740> anyone knows if there was any proposals how are we going to handle network userdata in new form of json-string ?15:25
airshipbot<alexander.hughes> there was json/yaml discussion during yaml sig.  was network userdata part of that?15:26
airshipbot<kk6740> i think we had only general acknowledgment of the issue15:26
airshipbot<alexander.hughes> ok, I hadn't heard any proposals.  I think it would have been during that call if we had one15:27
openstackgerritdiwakar thyagaraj proposed airship/maas master: [WIP]Enable Logs Output in Zuul for Deployment Scripts  https://review.opendev.org/70976315:29
airshipbot<kk6740> please review https://review.opendev.org/#/c/712802/15:39
airshipbot<alexander.hughes> minor refactor needs review https://review.opendev.org/#/c/711684/ to satisfy https://github.com/airshipit/airshipctl/issues/9115:41
airshipbot<ih616h> @alexander.hughes I thought we were holding off on that PS?15:42
airshipbot<alexander.hughes> Rodolfo assigned it as critical for beta since our last talk15:42
airshipbot<ih616h> right. good enough for me15:44
airshipbot<alexander.hughes> sorry I misspoke, it's labeled as 2.0.  but if it was still in design shouldn't be assigned imo15:44
*** mattmceuen has joined #airshipit15:44
airshipbot<ih616h> I'm also not sure if priority/critical makes sense on a 2.0 item...15:45
airshipbot<alexander.hughes> the way I rationalize it is this way: there are milestones we are working towards, within each milestone certain issues are more critical than others.  so when we get to beta->2.0 push this would be one of the first items that needs to be worked compared to a 2.0 medium or low15:46
airshipbot<alexander.hughes> but if it's done in advance saves us the effort later.  didn't intentionally pick this out as a future item just saw an opportunity for improvement and proposed/opened issue and it happened to be assigned to 2.015:47
airshipbot<kk6740> Does anyone have any information how make the push to quay finally work https://github.com/airshipit/images ?15:53
airshipbot<kk6740> or whoever has the key, maybe can push manually for now15:53
airshipbot<mattmceuen> qwqqqq15:56
airshipbot<kk6740> iddqd?15:56
airshipbot<ih616h> @mattmceuen having trouble with vim?15:57
airshipbot<mattmceuen> sigh15:57
airshipbot<mattmceuen> Kostiantyn - what's the automated image push blocked on?15:58
airshipbot<kk6740> only if i knew, i really miss the part with the pushes of images15:59
airshipbot<mattmceuen> I will take a look today15:59
airshipbot<kk6740> maybe someone who took care of pushing airshipctl image can fix this one as well, since bascially it is the same issue15:59
airshipbot<mattmceuen> If I can's sort it out I will push manually15:59
airshipbot<mattmceuen> *can't15:59
airshipbot<kk6740> thanks16:01
airshipbot<dwalt> airshipctl also has issues. What is the specific problem on airship/images?16:04
openstackgerritMerged airship/airshipctl master: [#91] refactor secrets as top level command  https://review.opendev.org/71168416:05
airshipbot<ih616h> I'm seeing test failures on master... :slightly_frowning_face:16:12
openstackgerritIan Howell proposed airship/airshipctl master: Fix a bug in pkg/k8s/client.TestNewClient  https://review.opendev.org/71301116:19
openstackgerritKostyantyn Kalynovskyi proposed airship/airshipctl master: Resolve gather-libvirt-logs role failures  https://review.opendev.org/71301216:20
airshipbot<ih616h> Seems to only happen locally. 713011 should fix it16:22
airshipbot<ih616h> happens because the unit test was using a real config file16:22
airshipbot<an310y> Hi guys, could you please merge https://review.opendev.org/#/c/709638/ https://review.opendev.org/#/c/680495/? They are already have +2 and Verified+116:23
airshipbot<an310y> And review/merge https://review.opendev.org/#/c/710480/ and https://review.opendev.org/#/c/711566/. These all CR’s related to virtual OSH deployment.16:25
openstackgerritMerged airship/pegleg master: Bionic pegleg airship clients  https://review.opendev.org/71282516:26
openstackgerritKostyantyn Kalynovskyi proposed airship/airshipctl master: Resolve gather-libvirt-logs role failures  https://review.opendev.org/71301216:28
*** openstackgerrit has quit IRC16:31
*** mfuller has joined #airshipit16:34
airshipbot<dwalt> @an310y those changes LGTM at a glance. I'd feel more comfortable approving if they had additional, positive reviews from others working on the seaworthy-virt efforts.16:35
*** openstackgerrit has joined #airshipit16:35
openstackgerritSreejith Punnapuzha proposed airship/airshipctl master: [#106] Add docker install role  https://review.opendev.org/71154016:35
openstackgerritdiwakar thyagaraj proposed airship/maas master: Enable Docker default AppArmor profile to maas  https://review.opendev.org/70513616:45
openstackgerritdiwakar thyagaraj proposed airship/maas master: [WIP]Enable Logs Output in Zuul for Deployment Scripts  https://review.opendev.org/70976316:46
airshipbot<an310y> @sreejith.punnapuzha @sirajudeen.yasin could you please review above CR’s ?16:57
airshipbot<sreejith.punnapuzha> Sure16:58
airshipbot<an310y> and two from previous msg as well. Thanks16:59
openstackgerritIan Howell proposed airship/airshipctl master: Fix a bug in pkg/k8s/client.TestNewClient  https://review.opendev.org/71301117:05
airshipbot<an310y> @dwalt I’ve asked @sirajudeen.yasin and @sreejith.punnapuzha, they have reviewed it a couple of minutes ago. Please let me know if I need to involve someone else.17:10
airshipbot<dwalt> That's great. Thanks @an310y:)17:11
airshipbot<mattmceuen> Quick poll, folks.  We have possession of the airshipit.slack.com workspace, if we want to use it.  If we wanted to use it, I'd send an invitation to everyone in airshipproject.slack.com, communicate it out, and close airshipproject down a couple days later or some such.17:14
airshipbot<mattmceuen> Slightly better branding, however I don't have a huge preference -- what do y'all think?17:15
airshipbot<dwalt> That feels like a more permanent home to me. As if we belong there17:21
airshipbot<mattmceuen> @alexander.hughes we haven't sent out the newsletter with instructions yet, right? :slightly_smiling_face:17:21
airshipbot<alexander.hughes> I have not. Waiting on final confirmation on slack link sounded like Monday17:22
airshipbot<alexander.hughes> I like consistent branding. Airshipit has my vote17:23
airshipbot<ih616h> +1 airshipit17:23
airshipbot<mattmceuen> cool.  ok, will try to get mirroring sorted out there by eod; we have a permanent home identified for the bot as well17:23
airshipbot<dwalt> Maybe more people will have pictures in the permanent workspace ;)17:24
airshipbot<sreejith.punnapuzha> +1 airshipit17:25
*** dpawlik has quit IRC17:30
airshipbot<kk6740> this is my real picture17:33
*** evrardjp has quit IRC17:35
*** evrardjp has joined #airshipit17:36
openstackgerritFrancis Bacon Yi proposed airship/airshipctl master: Increase config pkg test coverage  https://review.opendev.org/71250317:52
airshipbot<mattmceuen> invitations sent18:04
openstackgerritFrancis Bacon Yi proposed airship/airshipctl master: Increase config pkg test coverage  https://review.opendev.org/71250318:29
openstackgerritKostyantyn Kalynovskyi proposed airship/airshipctl master: Fix network and cloud data for ephemeral host  https://review.opendev.org/71282918:35
airshipbot<alexander.hughes> only see a general channel in the other workspace.  is that what we're using?18:43
*** airshipbot1 has joined #airshipit18:46
airshipbot<ih616h> there's a airshipit18:48
*** jbryce is now known as airship-irc-bot19:03
*** airship-irc-bot is now known as jbryce19:07
airshipbot<alexander.hughes> ah seems the new workspace you need to add the airshipit channel manually.  this one came included for me19:10
airshipbot<mattmceuen> Ah, that's probably some setting I can tweak after I'm an admin19:11
airshipbot<alexander.hughes> yeah small QOL change.  just throwing it out as a disclaimer for everyone making swap now19:11
airshipbot<kk6740> can we please merge this small item https://review.opendev.org/#/c/713012/219:11
jbrycemattmceuen: you should be an owner now in the new workspace19:11
*** airshipbot1 has quit IRC19:12
airshipbot<mattmceuen> o/ jbryce - thanks!f19:12
*** airship-irc-bot1 has joined #airshipit19:13
*** airship-irc-bot1 has quit IRC19:13
*** airship-irc-bot has joined #airshipit19:13
airship-irc-bot<jbryce> test from airshipit workspace19:15
jbrycetest from irc for new airshipit workspace19:15
airship-irc-bot<kk6740> please review this, very small, and needed fix for logs on zuul :slightly_smiling_face: https://review.opendev.org/#/c/713012/219:15
*** airship-irc-bot has quit IRC19:16
*** airship-irc-bot has joined #airshipit19:17
jbrycethe bot between the #airship channels is up and running and persists across restarts, etc19:18
mattmceuenthat's fantastic19:18
mattmceuenthanks for your help jbryce - I'll plan to ax the old workspace early next week after folks have a chance to move over19:19
jbrycesounds good19:20
airship-irc-bot<alexander.hughes> nit: can we get airship logo on airshipit workspace please19:28
openstackgerritdiwakar thyagaraj proposed airship/maas master: [WIP]Enable Logs Output in Zuul for Deployment Scripts  https://review.opendev.org/70976319:37
airship-irc-bot<kk6740> please take a look at these commits: https://review.opendev.org/#/c/712802/5 https://review.opendev.org/#/c/712829/4  They depend on each other and togather fix gate19:37
airship-irc-bot<kk6740> very important in a process of moving further in gating process19:38
openstackgerritIan Howell proposed airship/airshipctl master: WIP: Add the dynamic kubernetes client  https://review.opendev.org/71296319:42
openstackgerritIan Howell proposed airship/airshipctl master: Fix a failure when running unit tests locally  https://review.opendev.org/71301119:45
openstackgerritMerged airship/airshipctl master: Resolve gather-libvirt-logs role failures  https://review.opendev.org/71301220:09
openstackgerritDrew Walters proposed airship/airshipctl master: Provide Redfish feedback in RemoteDirect errors  https://review.opendev.org/71199220:45
openstackgerritMerged airship/treasuremap master: Fixing the typo in authoring_and_deployment  https://review.opendev.org/71178021:00
openstackgerritMerged airship/images master: Example network-config network_data.json format  https://review.opendev.org/71266721:01
openstackgerritdiwakar thyagaraj proposed airship/porthole master: Enable container resource limits  https://review.opendev.org/70926721:02
openstackgerritSreejith Punnapuzha proposed airship/airshipctl master: [#106] Add docker install role  https://review.opendev.org/71154021:22
openstackgerritSreejith Punnapuzha proposed airship/airshipctl master: [#106] Add docker install role  https://review.opendev.org/71154021:23
openstackgerritKostyantyn Kalynovskyi proposed airship/airshipctl master: Add single source of Seletors  https://review.opendev.org/71280221:31
openstackgerritKostyantyn Kalynovskyi proposed airship/airshipctl master: Fix network and cloud data for ephemeral host  https://review.opendev.org/71282921:32
openstackgerritKostyantyn Kalynovskyi proposed airship/airshipctl master: Fix network and cloud data for ephemeral host  https://review.opendev.org/71282921:32
mattmceuenThe reason the airhip/images images are failing to be published:21:42
mattmceuenFailed to import the required Python library (Docker SDK for Python: docker (Python >= 2.7) or docker-py (Python 2.6)) on ubuntu-bionic-rax-iad-0015106816's Python /usr/bin/python3. Please read module documentation and install in the appropriate location, for example via `pip install docker` or `pip install docker-py` (Python 2.6). The error was: No module named 'requests'21:43
mattmceuenDoes that ring a bell for anyone who has fixed it from other contexts?  I think Dmitry may have cloned those image handling jobs from somewhere, but not sure where21:44
mattmceuenhttps://www.irccloud.com/pastebin/FVoNcP5P/21:44
airshipbot<dwalt> I think that's because Ansible's `docker_login` module requires the `docker-py` package21:44
airshipbot<dwalt> I'll grab the relevant airshipctl implementation21:45
airship-irc-bot<andrew.walters> Here is what we use in airshipctl: https://github.com/airshipit/airshipctl/blob/master/roles/airshipctl-publish-images/tasks/main.yaml#L1321:46
airship-irc-bot<andrew.walters> Do you have a job link to confirm that's when it's happening?21:47
mattmceuenah, thanks - I'll take a look at that21:48
mattmceuenHere's the output I'm looking at: https://zuul.opendev.org/t/openstack/build/3349d65040bf46f1ac4a3da226794f3921:48
airship-irc-bot<dwalt> Cool. I think that will fix it21:49
mattmceuensorry dwalt, I'm missing something -- I don't see airshipctl installing `docker-py` package either, how does it get there?21:50
airship-irc-bot<dwalt> The error in the build link says `The error was: No module named 'requests'`21:50
airship-irc-bot<dwalt> `python3-requests` is what needs to be installed, I think21:51
airship-irc-bot<kk6740> please review when u have time: https://review.opendev.org/#/c/712802/5 https://review.opendev.org/#/c/712829/421:51
mattmceuenaha - yep I think you're right dwalt, thanks21:52
airship-irc-bot<dwalt> On closer inspection, those packages are for 2.6 :P `for example via `pip install docker` or `pip install docker-py` (Python 2.6).`21:53
airship-irc-bot<dwalt> no problem!21:53
openstackgerritSreejith Punnapuzha proposed airship/airshipctl master: [#106] Add docker install role  https://review.opendev.org/71154021:55
openstackgerritMatt McEuen proposed airship/images master: Fix publish job  https://review.opendev.org/71306621:56
airship-irc-bot<dwalt> Is the proper command to fix goimports `goimports -local "opendev.org/airship/airshipctl" -w .`?21:58
*** mfuller has quit IRC23:27

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