Sunday, 2020-03-15

openstackgerritMohammed Naser proposed openstack/project-config master: opendev: add openstack/devstack  https://review.opendev.org/71312301:04
openstackgerritMohammed Naser proposed openstack/project-config master: opendev: add openstack/devstack  https://review.opendev.org/71312301:06
openstackgerritMohammed Naser proposed openstack/project-config master: opendev: move vexxhost to seperate tenant  https://review.opendev.org/71312301:10
openstackgerritMohammed Naser proposed openstack/project-config master: opendev: move vexxhost to seperate tenant  https://review.opendev.org/71312301:11
*** DSpider has joined #opendev06:31
openstackgerritAndreas Jaeger proposed zuul/zuul-jobs master: Keep doc/source/roles.rst sorted  https://review.opendev.org/71312808:30
openstackgerritSorin Sbarnea proposed zuul/zuul-jobs master: Improve ensure-tox role  https://review.opendev.org/70864211:06
openstackgerritSorin Sbarnea proposed zuul/zuul-jobs master: tox: allow tox to be upgraded  https://review.opendev.org/69005711:26
openstackgerritSorin Sbarnea proposed zuul/zuul-jobs master: tox: allow tox to be upgraded  https://review.opendev.org/69005711:33
openstackgerritJeremy Stanley proposed opendev/system-config master: Set up LE certs for docs.airshipit.org static site  https://review.opendev.org/70660014:14
openstackgerritJeremy Stanley proposed opendev/system-config master: Add a new docs.airshipit.org vhost on static01  https://review.opendev.org/70660114:14
openstackgerritMohammed Naser proposed opendev/system-config master: nodepool-builder: mount SSH keys into the container  https://review.opendev.org/71313615:43
mnaserfungi: what special hostname? i wasn't following too closely the conversation over the past few days15:46
fungilooking15:46
clarkbI believe all the changes so far have been captured on host in the compose file including the hostname15:48
fungilooks like we were passing --hostname nb01opendev to podman run? per 20:11-20:13 utc friday15:49
clarkbwe arent using podman anymore15:49
fungibut then there was a problem with that name not resolving inside the container15:49
clarkbpodman didnt work due to mount propagation issues15:49
clarkbthere should be a hostname override in the compose file and docker compose is applying that15:50
fungiahh, okay, there's ~1k lines of scrollback dealing with this, so still trying to wrap my head around where it ended up15:50
clarkbfor the host resolution we edited /etc/hosts and since we usehostnetworking that resolved it15:50
clarkbfor this issue I think we need to make sure the ssh key is in place, then add the mount to the compose file then docker compose stop/shutdown/whatever the command is then docker conpose ul15:51
clarkb*up15:51
fungiaha, yep, /etc/hosts on nb01.opendev.org includes a line like15:51
fungi127.0.1.1 nb01.opendev.org nb01 nb01opendev.opendev.org nb01opendev15:51
clarkband there is a flag to make it background so you dont have to do the disconnection corvus did15:51
clarkbthen trigger an image rebuild15:52
clarkbI can help with that in a bit but not right this second. 15-30 minutes probably15:52
fungiwe don't have the convenience script in place to call the rpc client through docker yet either, looks like?15:54
fungii guess we can do that from another nodepool server?15:55
mnaserfungi: you can probably do that with the docker cli15:56
mnaserfungi: docker exec <container-naem> nodepool list15:56
mnaser_should_ work15:56
fungicannot access '/home/nodepool/.ssh': No such file or directory15:57
clarkbfungi: ya I would do it from another server for simplicity15:57
fungi~nodepool does resolve to /home/nodepool15:57
fungibut that doesn't seem to contain a .ssh directory15:57
fungithat does exist on our other builders15:58
clarkbfungi: note becayse we arebind mounting we can actually stash the ssh key wherever we want. But consistency with the other builders is probably good for now16:00
fungiwell, i'm trying to figure out what's supposed to be putting the key on the server at all and where it might be doing that currently16:00
clarkbI expect the new ansible just doesnt do it16:01
clarkbpuppet does it on the old hosts16:01
clarkbalso I've just had a lightbulb16:01
clarkbI think we can set this in the /etc/nodepool/nodepool.yaml config instead and that will avoid needing to restart anything16:02
fungii guess looks like you're already editing that as i see a .swp file16:06
clarkbya managed to make it to computer then wanted to do lightbulb idea16:07
clarkbfungi: check git diff on /opt/project-config16:07
clarkbthat should be used by the next run so now I think we can trigger a build16:08
clarkbfungi: do you want to do that or should I?16:08
clarkbI'm adding a task to the storyboard story now16:09
fungisorry, just finished adding the public key16:10
fungi~nodepool/.ssh/id_rsa.pub is there now and has correct permissions (as does the empty ~nodepool/.ssh/id_rsa and the ~nodepool/.ssh dir)16:11
clarkbcool so we can switch it to the consistent way if we want16:11
fungicopied from what nb01.openstack.org has16:11
clarkb(but the config update should be sufficient for now if we want to wait on restarting services)16:11
fungioh, got it, now i see what you meant by editing nodepool.yaml16:11
fungii thought you needed to edit it to add the mount16:12
fungibut that would have been the dockerfile16:12
fungior docker-compose file or whatever16:12
fungiso you added the literal public key content instead16:12
clarkbfungi: ya, beacuse nodepool rereads its config and will apply that on the next run16:13
fungihonestly i think adding it to the nodepool config makes more sense if no processes running on nodepool actually need it and it's just for embedding into the image16:13
clarkbbasically this allows us to bypass restarting things16:13
fungiif we can get by without populating ~nodepool/.ssh at all on the builders then all the better16:13
clarkbfungi: ya I could see us switchign to that form globally (its nice and verbose too rather than hidden in the system setup)16:13
clarkbfungi: https://storyboard.openstack.org/#!/story/2007407 task 39072 (see notes)16:14
* fungi makes thumbs-up16:15
clarkbfungi: should I trigger the build now?16:16
fungiyep please16:16
fungii guess the argument for keeping it in ~nodepool/.ssh is that the launchers use it? though it doesn't seem like they actually do if the nodes were getting handed over to zuul in this state16:16
clarkbthats a fun nodepool cli behavior16:16
fungioh?16:17
clarkbI can't trigger the build on the old servers because they have paused that image16:17
fungihah16:17
clarkbso we do need to sort out the docker exec here16:17
clarkb`sudo docker exec nodepoolbuildercompose_nodepool-builder_1 nodepool image-build fedora-30` seems to be it16:18
fungioh, cool16:18
clarkbfedora-30-0000000558.log if people want to follow along16:18
clarkbmnaser: ^ fyi16:18
clarkbfungi: when figuring that command out I was sure to run whoami as the command first just to be double sure that I had the right user16:19
clarkbfungi: though it would have simply failed if not16:19
fungiyup16:19
openstackgerritMerged opendev/system-config master: Set up LE certs for docs.airshipit.org static site  https://review.opendev.org/70660016:19
clarkbfungi: I could also have run `sudo docker exec -it nodepoolbuildercompose_nodepool-builder_1 bash` and dont it interactively16:20
clarkb*done it16:20
fungii see16:20
clarkbfwiw I'm not 100% sure the builders reload configs like launchers do but am like ~99% sure16:22
clarkbif we still see this failing then next step would be to stop start it via docker-compose16:22
clarkbI'm going to step away from the computer now. It is sunny and dry today and nto snowing anymore so probably going to try and enjoy the weather16:23
fungienjoy!16:23
clarkbas noted in -infra dnf seems to be on a lunch break. I'm not sure what is going on, but that likely needs more in depth debugging than I can do today17:00
clarkbdisks are nowhere near full17:00
AJaegerstill nothing for two hours on https://nb01.opendev.org/fedora-30-0000000558.log ;(17:42
AJaegerclarkb, checking https://nb01.opendev.org/fedora-30-0000000557.log - there's also 2hours and 20 mins break during install, so it might continue...17:43
AJaegerand same package - haveged. That looks like something is broken with it ;(17:45
clarkbthabks for checking. I think we just eait then17:48
*** jkt has quit IRC18:39
*** corvus has quit IRC18:39
*** jkt has joined #opendev18:40
*** corvus has joined #opendev18:40
AJaegerso, build continued...18:57
funginice19:03
AJaeger"Build completed successfully"19:23
AJaegerhow long will it take to upload?19:26
clarkbusually within 10 minutes some cloudshave the images but then depends on the others. I guess we can delete the old image to ensure the new one is the only one used19:27
AJaegerI'm following http://zuul.opendev.org/t/zuul/stream/e1ef0a6b23d445dcaf7be094d62ad21d?logfile=console.log - looks good so far19:38
AJaegerpassed \o/19:41
AJaegernow zuul-jobs-test-install-docker-centos-7 has a post_failure19:41
fungibut at least it's not f30? ;)19:41
AJaegeryep. f30 success :)19:42
AJaegerhttp://zuul.opendev.org/t/zuul/status/change/712547,519:42
AJaeger(second f30 test still running)19:42
fungizbr: ^ i guess you can recheck your failed builds now19:42
* AJaeger rechecked on of zbr's already19:44
AJaeger2nd f30 job passed as well, so seems we're goog19:45
*** mordred has quit IRC20:30
*** mordred has joined #opendev20:31
*** jaicaa has quit IRC21:11
*** jaicaa has joined #opendev21:13
*** AJaeger has quit IRC21:13
*** AJaeger has joined #opendev21:26
*** factor has joined #opendev21:31
openstackgerritIan Wienand proposed opendev/system-config master: Switch back to docker for gerrit and nodepool-builder  https://review.opendev.org/71310121:41
openstackgerritIan Wienand proposed opendev/system-config master: Switch back to docker for gerrit and nodepool-builder  https://review.opendev.org/71310122:44
*** DSpider has quit IRC23:00
openstackgerritIan Wienand proposed opendev/system-config master: nodepool-builder: put container configs in /etc  https://review.opendev.org/71314823:15
openstackgerritIan Wienand proposed opendev/system-config master: nodepool-builder: put container configs in /etc  https://review.opendev.org/71314823:22
openstackgerritIan Wienand proposed opendev/system-config master: Switch back to docker for gerrit and nodepool-builder  https://review.opendev.org/71310123:27
openstackgerritIan Wienand proposed openstack/diskimage-builder master: bindep: remove lsb-release  https://review.opendev.org/71315023:39

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