| *** ykarel|away is now known as ykarel | 04:44 | |
| *** marios is now known as marios|ruck | 05:49 | |
| marios|ruck | needs reviews please https://review.opendev.org/c/openstack/tripleo-ansible/+/801615 | 06:45 |
|---|---|---|
| soniya29|rover | pojadhav has power failure issues hence she will resume the work once power is back | 06:47 |
| *** amoralej|off is now known as amoralej | 07:07 | |
| *** ykarel is now known as ykarel|lunch | 08:34 | |
| soniya|rover | kopecmartin, chandankumar, do you have anything to discuss for today's tempest meeting? | 09:06 |
| *** zbr is now known as Guest2544 | 09:09 | |
| marios|ruck | need vote here please https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/801736 test at https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802618 (scen4 tested) | 09:37 |
| marios|ruck | thank you sshnaidm|afk chandankumar | 09:40 |
| pojadhav | marios|ruck, 0/ | 09:44 |
| *** sshnaidm|afk is now known as sshnaidm | 09:45 | |
| pojadhav | for the c7 updates ... do we are care about any release other than train and queens? I got comments on patch : https://review.rdoproject.org/r/c/rdo-jobs/+/34632 to avoid changes of job names for pike and rocky as its EOL. | 09:46 |
| *** pojadhav is now known as pojadhav|brb | 09:53 | |
| marios|ruck | pojadhav|brb: queens is also eol now :) | 09:59 |
| kopecmartin | soniya|rover: nothing specific | 10:00 |
| marios|ruck | pojadhav|brb: https://opendev.org/openstack/releases/commit/37d51ffe3b28b8680572a9cbc91971bd93d82b2d | 10:00 |
| marios|ruck | pojadhav|brb: so just train | 10:00 |
| *** ykarel|lunch is now known as ykarel | 10:17 | |
| soniya|rover | kopecmartin, chandankumar, we are cancelling today's tempest meeting since weshay|ruck, ysandeep|PTO, and arxcruz are on PTO and nothing much to discuss. | 10:52 |
| chandankumar | soniya|rover: ack | 10:52 |
| *** pojadhav|brb is now known as pojadhav | 11:03 | |
| pojadhav | marios|ruck, so i will revert the change made for pike, rocky and queens and only keep train job name changes. | 11:04 |
| pojadhav | for c-7 updates | 11:05 |
| marios|ruck | pojadhav: which change? | 11:10 |
| marios|ruck | pojadhav: maybe don't need revert just fixup with new patch? | 11:10 |
| pojadhav | marios|ruck, https://review.rdoproject.org/r/q/topic:%2522refactor_job_names%2522+(status:open+OR+status:open) | 11:11 |
| pojadhav | c-7 job names to match with upstream (FS37, FS30 and FS10) | 11:11 |
| pojadhav | marios|ruck, yeah.. will upload new patchset. | 11:12 |
| pojadhav | marios|ruck, also i got zuul error : https://review.rdoproject.org/r/c/rdo-jobs/+/34632/1#message-283f0496b1a9dd0868d2c3951d04b847b606851a | 11:12 |
| pojadhav | what should be the job name after refactoring.. actually I didnt get it. | 11:13 |
| marios|ruck | pojadhav: you're using the same name exactly | 11:14 |
| marios|ruck | pojadhav: that is the problem | 11:14 |
| pojadhav | marios|ruck, okay | 11:16 |
| marios|ruck | me afk few mins brb | 11:26 |
| *** rlandy is now known as rlandy|ruck | 11:52 | |
| rlandy|ruck | marios|ruck: soniya|rover: hey - all set for the program call? | 11:53 |
| marios|ruck | rlandy|ruck: o/ | 11:53 |
| rlandy|ruck | I'm joining but I'll let you talk | 11:53 |
| marios|ruck | rlandy|ruck: sure, ok will do | 11:53 |
| marios|ruck | rlandy|ruck: thanks for following through on train | 11:53 |
| marios|ruck | \o/ | 11:53 |
| marios|ruck | rlandy|ruck: i mental high fived you this morning ;) | 11:53 |
| rlandy|ruck | marios|ruck: wrt downstream - I marked 16.2 yellow because we got a promotion just befire outage | 11:53 |
| marios|ruck | rlandy|ruck: why don't you give status? | 11:54 |
| rlandy|ruck | 17 is red as we didn't get a promotion on friday and now it's out of water | 11:54 |
| marios|ruck | rlandy|ruck: upstream not much to mention so you can talk about dstream more | 11:54 |
| marios|ruck | rlandy|ruck: if you prefer not to i will fine no prob | 11:54 |
| rlandy|ruck | marios|ruck: ^^ that's really all there is to say | 11:54 |
| rlandy|ruck | you go ahead | 11:54 |
| marios|ruck | rlandy|ruck: ack | 11:54 |
| rlandy|ruck | I'll say something if need be | 11:54 |
| marios|ruck | rlandy|ruck: k | 11:54 |
| soniya|rover | rlandy|ruck, hello | 11:58 |
| rlandy|ruck | soniya|rover: hey - all ok in ruck/rover land? | 11:59 |
| chandankumar | rlandy|ruck: bhagyashris https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/34688 and https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/34273 When free. thanks ! | 11:59 |
| soniya|rover | rlandy|ruck, yeah | 11:59 |
| chandankumar | marios|ruck: against this https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/790926 https://338e746b16b3007d4bcf-acf78f12e623b559f4ddc7a30082b17f.ssl.cf5.rackcdn.com/790926/13/check/tripleo-ci-centos-8-scenario007-multinode-oooq-container/a47b348/logs/undercloud/var/log/tempest/tempest_run.log tempest test failure unrelated from this patch. | 12:01 |
| chandankumar | seen this? | 12:01 |
| soniya|rover | marios|ruck, I have fixed the grafana issue, https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/34714 | 12:01 |
| chandankumar | {0} tempest.scenario.test_network_basic_ops.TestNetworkBasicOps.test_network_basic_ops [18.447043s] ... FAILED - Details: {'code': 400, 'created': '2021-07-28T10:14:17Z', 'message': "Host 'centos-8-stream-ovh-bhs1-0025672773.localdomain' is not mapped to any cell"} | 12:01 |
| zbr | sshnaidm: do you remember which was the problem that made molecule-podman fail to gather facts? | 12:02 |
| sshnaidm | zbr, yes, container wasn't running | 12:03 |
| zbr | and why it was not running? | 12:03 |
| sshnaidm | zbr, because of various reasons | 12:04 |
| zbr | take a look at https://paste.openstack.org/show/807759/ | 12:04 |
| zbr | base on previous task I assume container started, so create suceeded. | 12:04 |
| zbr | but i guess, that it could be the case were a container quits in between | 12:05 |
| sshnaidm | zbr, check if it's running | 12:05 |
| zbr | is not, no container running. | 12:05 |
| sshnaidm | so that's it, "podman logs" | 12:06 |
| sshnaidm | podman logs --latest | 12:07 |
| zbr | nada, but i see that the image was downloaded. | 12:07 |
| zbr | interesting, running that image manually seems to work as expected. maybe i found another bug in molecule. | 12:08 |
| pojadhav | marios|ruck, when I search the occurances of "tripleo-ci-centos-7-multinode-1ctlr-featureset010" in rdo-jobs, but this job is not used anywhere, so I think i should remove that job definition. whats your opinion? | 12:11 |
| pojadhav | marios|ruck, https://review.rdoproject.org/codesearch/?q=tripleo-ci-centos-7-multinode-1ctlr-featureset010&i=nope&files=&repos= | 12:12 |
| marios|ruck | pojadhav: in call will check later | 12:14 |
| pojadhav | marios|ruck, yeah sure.. | 12:14 |
| zbr | sshnaidm: very weird thing, i identified the exact command used by podman driver and run it myself, it starts a container as I get its long id but when I do "podman ps", nothing, even using logs and container numebr show nothing, is like that execution never happened. | 12:24 |
| zbr | podman run -d --name centos8 --hostname=centos8 quay.io/pycontribs/centos:stream8 | 12:26 |
| zbr | when I try to run again it fails complaining that name is already in used by the container that is no longer running. | 12:26 |
| zbr | https://paste.openstack.org/show/807760/ | 12:27 |
| sshnaidm | podman ps -a ? | 12:29 |
| sshnaidm | probably one with same name failed, but didn't remove | 12:30 |
| zbr | well, i did clean them manually but they still fail to really start. even in foreground. | 12:30 |
| zbr | i am now trying another image | 12:31 |
| *** pojadhav is now known as pojadhav|afk | 12:31 | |
| zbr | conclusion so far is that if another container with the same name previously run, it will not run again. not sure why the error is not seen when using molecule. | 12:38 |
| zbr | when i run podman from cli, i can see the error like error creating container storage: the container name "centos8" is already in use by | 12:38 |
| sshnaidm | to remove all existing containers: podman rm -fa | 12:38 |
| sshnaidm | probably in molecule it's not processed right | 12:39 |
| sshnaidm | if it checks if container exists - it does | 12:39 |
| sshnaidm | but not running | 12:39 |
| zbr | i suspect that this line may not do what it is supposed to do https://github.com/ansible-community/molecule-podman/blob/master/lib/molecule_podman/playbooks/create.yml#L99 | 12:40 |
| zbr | now i know, local version of the driver is 0.3.0, which does not have this cleanup code. | 12:42 |
| zbr | https://github.com/ansible-community/molecule-podman/blob/0.3.0/lib/molecule_podman/playbooks/create.yml | 12:42 |
| zbr | i do not see any cleaning here. | 12:42 |
| zbr | but master seems to have it, time to check if it works with that. | 12:42 |
| zbr | sshnaidm: basically what happens is that podman reports container as starting, reporting its it but that one instantly disappears. when I try to use log on it, it does list nothing. | 12:53 |
| zbr | the stopped container is there but i have no explanation why it stopped. | 12:53 |
| zbr | and i see no reason why this container should run for less than second: https://gist.github.com/ssbarnea/80941778d05dbd9060f6a446a8ca951c#file-gistfile1-txt-L82 -- i also included inspect report on it. | 12:57 |
| sshnaidm | zbr, maybe in journal something you can see | 13:00 |
| sshnaidm | zbr, look at exit code in inspection "podman inspect centos8" | 13:01 |
| rlandy|ruck | soniya|rover: still having the tempest meeting? | 13:01 |
| soniya|rover | rlandy|ruck, nope | 13:01 |
| soniya|rover | we have cancelled it | 13:01 |
| rlandy|ruck | ah I see email | 13:01 |
| *** amoralej is now known as amoralej|lunch | 13:16 | |
| rlandy|ruck | marios|ruck: hey - gate stuck? | 13:22 |
| rlandy|ruck | tripleo-ci-centos-8-scenario000-multinode-oooq-container-updates | 13:22 |
| rlandy|ruck | one job queued | 13:23 |
| rlandy|ruck | holding all the jobs that are done | 13:23 |
| marios|ruck | rlandy|ruck: looking | 13:23 |
| rlandy|ruck | ping openstack-infra? | 13:23 |
| marios|ruck | rlandy|ruck: yeah... but they will likely just tesll us to abandon it... | 13:24 |
| marios|ruck | rlandy|ruck: urgh... didn't notice it. it has been in the gate since last night | 13:24 |
| marios|ruck | rlandy|ruck: if i abandon it will kill all the things below it but... | 13:24 |
| marios|ruck | we may just have to do that | 13:24 |
| marios|ruck | rlandy|ruck: man that is horrible there are a lot of green runs below it :( | 13:25 |
| rlandy|ruck | marios|ruck: I don't think we have choice unless infra can help us out | 13:25 |
| rlandy|ruck | go4it | 13:26 |
| rlandy|ruck | oh I see you chatting with infra | 13:26 |
| marios|ruck | rlandy|ruck: well lets see if folks say something give it couple mins | 13:27 |
| marios|ruck | rlandy|ruck: otherwise we can reset :/ | 13:27 |
| rlandy|ruck | ack | 13:27 |
| rlandy|ruck | no other way out | 13:27 |
| rlandy|ruck | zbr: did you get the infrared issue sorted? | 13:33 |
| marios|ruck | thanks rlandy|ruck++ completely missed that ;) | 13:33 |
| *** amoralej|lunch is now known as amoralej | 13:38 | |
| rlandy|ruck | ugh ... here we go ... https://artifacts.ci.centos.org/rdo/jenkins-tripleo-quickstart-promote-victoria-current-tripleo-delorean-minimal-100/undercloud/home/stack/undercloud_install.log | 13:43 |
| rlandy|ruck | The 'validation_id' argument should be a List | 13:44 |
| rlandy|ruck | didn't we fix that?? | 13:44 |
| rlandy|ruck | marios|ruck: ^^ that error look familiar? | 13:45 |
| rlandy|ruck | chandankumar: ^^ maybe from your shift | 13:45 |
| rlandy|ruck | on ci.centos jobs | 13:45 |
| rlandy|ruck | failing victoria undercloud install there | 13:46 |
| chandankumar | rlandy|ruck: it is from validation | 13:46 |
| chandankumar | we fixed it | 13:46 |
| rlandy|ruck | yeah - still failing ci.centos - weird | 13:46 |
| marios|ruck | rlandy|ruck: havent come across that one | 13:46 |
| rlandy|ruck | marios|ruck: yeah - I remember it from last week | 13:46 |
| rlandy|ruck | but it's in the ci.centos line | 13:47 |
| rlandy|ruck | so we are missing something there | 13:47 |
| chandankumar | marios|ruck: https://review.opendev.org/q/efdefd92de54d2dd0803f0367bf89bb577927037 | 13:47 |
| rlandy|ruck | ussuri also failing | 13:47 |
| chandankumar | rlandy|ruck: ^^ | 13:47 |
| chandankumar | https://bugs.launchpad.net/tripleo/+bug/1936465 | 13:47 |
| rlandy|ruck | chandankumar: thank ou | 13:47 |
| rlandy|ruck | you | 13:47 |
| rlandy|ruck | https://ci.centos.org/job/tripleo-quickstart-promote-ussuri-current-tripleo-delorean-minimal/187/console ussuri is progressing | 13:50 |
| rlandy|ruck | maybe victoria needs an update | 13:50 |
| rlandy|ruck | checking | 13:50 |
| marios|ruck | thanks chandankumar | 13:52 |
| rlandy|ruck | python3-tripleoclient.noarch 14.2.2-0.20210702232255.79637b0.el8 @delorean-component-tripleo | 13:57 |
| rlandy|ruck | ^^ old | 13:57 |
| rlandy|ruck | python-tripleoclient-14.2.2-0.20210702232255.79637b0.el8.src.rpm2021-07-02 23:24 330K | 14:02 |
| rlandy|ruck | python-tripleoclient-14.2.3-0.20210722165810.9b3bd5f.el8.src.rpm2021-07-22 16:59 331K | 14:02 |
| rlandy|ruck | ci.centos is probably not taking current on tripleo | 14:02 |
| rlandy|ruck | so we need to check component line there | 14:02 |
| chandankumar | sshnaidm: rlandy|ruck marios|ruck https://review.opendev.org/q/topic:%22remove_uneeded_tq_requirements%22+(status:open%20OR%20status:merged) testreport is here: https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/790926/13#message-ac7c5f22f60b1ffac1969c0982c904ad522ed367 | 14:03 |
| chandankumar | if more needed, please add comments before merging | 14:03 |
| chandankumar | we can do the testing | 14:03 |
| rlandy|ruck | thanks - will look in a bit | 14:03 |
| marios|ruck | chandankumar: ack adding to list | 14:05 |
| sshnaidm | chandankumar, commented on https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/790926 | 14:07 |
| chandankumar | sshnaidm: done https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/790926/13/playbooks/ovb-setup.yml#17 | 14:08 |
| chandankumar | sshnaidm: may be I need to put a tripleo-common changes and also trigger fs02 change there? | 14:10 |
| sshnaidm | chandankumar, yeah, a good idea | 14:10 |
| chandankumar | sshnaidm: let me propose one | 14:11 |
| sshnaidm | I'm fine with this patch, but more tests the better | 14:11 |
| chandankumar | marios|ruck: rlandy|ruck putting one more changes | 14:11 |
| rlandy|ruck | https://logserver.rdoproject.org/openstack-component-tripleo/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-8-scenario004-standalone-tripleo-victoria/25e331a/logs/undercloud/var/log/tempest/stestr_results.html.gz | 14:11 |
| chandankumar | to test more | 14:11 |
| rlandy|ruck | still failing | 14:11 |
| rlandy|ruck | https://opendev.org/openstack/openstack-tempest-skiplist/commit/3bcf24eb9d0e11a0a054944d1d0832468820872b | 14:11 |
| rlandy|ruck | we need that back | 14:11 |
| marios|ruck | rlandy|ruck: looking | 14:12 |
| rlandy|ruck | marios|ruck: victoria tripleo pipeline | 14:13 |
| marios|ruck | rlandy|ruck: hm that is from yesterday though https://logserver.rdoproject.org/openstack-component-tripleo/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-8-scenario004-standalone-tripleo-victoria/25e331a/logs/undercloud/var/log/tempest/ | 14:13 |
| marios|ruck | rlandy|ruck: checking when did fix merge | 14:13 |
| rlandy|ruck | been failing for a while | 14:13 |
| rlandy|ruck | the component job was never in the skip | 14:15 |
| rlandy|ruck | marios|ruck: ^^ that is blocking ci.centis | 14:15 |
| rlandy|ruck | centos | 14:15 |
| marios|ruck | rlandy|ruck: fixes merged 26/27 so i think we need to see what happens with today run ? | 14:15 |
| rlandy|ruck | marios|ruck: if we promote victoria, it may help | 14:16 |
| rlandy|ruck | check will never see it | 14:16 |
| rlandy|ruck | as it takes consistent | 14:16 |
| marios|ruck | rlandy|ruck: well v has been promoting pretty consistently last few days | 14:17 |
| marios|ruck | every day for a week now :) | 14:17 |
| marios|ruck | rlandy|ruck: let's move the cix back from done though and we can discuss in few mins on the call | 14:18 |
| marios|ruck | rlandy|ruck: https://trello.com/c/85yCXxqV/2017-cixlp1934879tripleociproa-victoriaussuritrain-c8tempestlibexceptionstimeoutexception-request-timed-out-failing-in-scenario004-st#comment-61016763c9dcb82538ab3bbc | 14:19 |
| marios|ruck | rlandy|ruck: oh that is a component job? | 14:19 |
| marios|ruck | rlandy|ruck: i missed that bit | 14:19 |
| marios|ruck | rlandy|ruck: that's why it was failing | 14:19 |
| marios|ruck | rlandy|ruck: i mean thats why it wasn't skipping the test and failing ... | 14:21 |
| rlandy|ruck | marios|ruck: so we have two options ... | 14:24 |
| rlandy|ruck | put the skip back in the DNM review | 14:24 |
| rlandy|ruck | rerun the component test | 14:24 |
| rlandy|ruck | promote component | 14:24 |
| rlandy|ruck | and see if that helps | 14:24 |
| rlandy|ruck | or put the skip back premanently | 14:24 |
| rlandy|ruck | marios|ruck: ^^ voting for the first to see if we get by that way | 14:25 |
| marios|ruck | rlandy|ruck: yeah we could just put the skip back for the component | 14:25 |
| * rlandy|ruck puts in review | 14:25 | |
| marios|ruck | rlandy|ruck: will have a look after the cix | 14:25 |
| marios|ruck | rlandy|ruck: ah thank you ronelle ! | 14:25 |
| *** ykarel is now known as ykarel|away | 14:45 | |
| rlandy|ruck | frenzy_friday: hey - you all set with tox/mocking? | 15:18 |
| frenzy_friday | rlandy|ruck, yeah for now. I'll worry about secrets after pushing the patch with mock | 15:19 |
| rlandy|ruck | frenzy_friday: sure - ping when ready to work on that | 15:19 |
| rlandy|ruck | ok - so back to ussuri failures | 15:19 |
| *** chem is now known as Guest2598 | 15:36 | |
| *** amoralej is now known as amoralej|off | 15:57 | |
| marios|ruck | rlandy|ruck: just chasing some ptl stuff so it doesn't slip but will go in a bit | 16:21 |
| marios|ruck | rlandy|ruck: thank you very much for all your help as usual | 16:21 |
| rlandy|ruck | marios|ruck: sure - will keep watch :) | 16:21 |
| *** marios|ruck is now known as marios|out | 16:38 | |
| pojadhav | rlandy|ruck, I have updated most of the patches about c7 updates.. we will look first for FS037 and FS10.. please have a look on those patches when you have bandwidth : https://review.rdoproject.org/r/q/topic:%2522refactor_job_names%2522+(status:open+OR+status:open) | 16:53 |
| pojadhav | rlandy|ruck, discussed with marios about releases that needs to care for C-7. he suggested me to care about train only. we will avoid to make change to pike, rocky and queens. | 16:55 |
| pojadhav | so made changes to the patches accordingly. | 16:56 |
| *** pojadhav is now known as pojadhav|out | 16:57 | |
| rlandy|ruck | pojadhav|out: k - I agree | 17:16 |
| rlandy|ruck | will look later | 17:16 |
| dviroel | rlandy|ruck: hi o/ | 19:16 |
| dviroel | rlandy|ruck: when you have a moment | 19:16 |
| dviroel | i was working on https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/34612 | 19:17 |
| rlandy|ruck | sure - in 5 min | 19:17 |
| dviroel | after some PS, seems to work as I was expecting | 19:17 |
| dviroel | some code changes were needed to work | 19:17 |
| dviroel | in PS12, i copied the artifacts to the output log dir, just to check, and this was the result: https://logserver.rdoproject.org/12/34612/8/check/tripleo-ci-compose-promote-latest-compose-to-tripleo-ci-testing/32d72d7/ | 19:18 |
| dviroel | s/PS12/PS8 | 19:19 |
| dviroel | looking at ci-config repo, i saw that the right thing is to add a molecule job, to have a job testing this code | 19:20 |
| dviroel | which will replace the current job that I added in layout.yaml for now | 19:20 |
| dviroel | my guess is that it will work with secrets too, according to some tests that I did in my local zuul, using a 'ssh-add' to add a private-key to the ssh-agent | 19:21 |
| dviroel | which looks like the process that other jobs in 'config' are doing | 19:22 |
| * dviroel needs to study how molecule jobs works | 19:22 | |
| * rlandy|ruck reads back | 19:23 | |
| rlandy|ruck | https://logserver.rdoproject.org/12/34612/8/check/tripleo-ci-compose-promote-latest-compose-to-tripleo-ci-testing/32d72d7/centos8/ | 19:24 |
| rlandy|ruck | dviroel: ^^ result looks good! | 19:24 |
| dviroel | :) | 19:24 |
| dviroel | we can add more artifacts as needed when we start to consume | 19:24 |
| rlandy|ruck | dviroel: so you're ready for the config/secrets job? | 19:25 |
| rlandy|ruck | molecule job is definitely preferred | 19:26 |
| rlandy|ruck | not required to start but good to have | 19:26 |
| rlandy|ruck | dviroel: do you think this is ready to merge? | 19:27 |
| rlandy|ruck | not that I would do that now | 19:27 |
| rlandy|ruck | just checking for approx readiness | 19:27 |
| dviroel | the only thing that is missing for config/secrets is to do the ssh key configuration for the job, like this | 19:27 |
| dviroel | https://github.com/rdo-infra/review.rdoproject.org-config/blob/master/playbooks/config/update_local.yaml#L3-L20 | 19:27 |
| dviroel | rlandy|ruck: i would think on adding a couple of unit tests and the molecule job to get it merged | 19:28 |
| rlandy|ruck | dviroel: suee | 19:28 |
| rlandy|ruck | sure | 19:28 |
| rlandy|ruck | go there first | 19:28 |
| rlandy|ruck | then we can merge | 19:28 |
| rlandy|ruck | and do the secrets bit | 19:28 |
| rlandy|ruck | but looking good - thank you | 19:28 |
| dviroel | this part of adding secret keys, is something that shoud be at 'config' right? | 19:29 |
| rlandy|ruck | looking at the ssh key piece ... | 19:29 |
| rlandy|ruck | https://github.com/rdo-infra/review.rdoproject.org-config/blob/master/playbooks/config/update_local.yaml#L3-L20 | 19:30 |
| rlandy|ruck | we'd probably want to switch case that | 19:30 |
| rlandy|ruck | if not secret is defined | 19:30 |
| rlandy|ruck | if it is, go with the secret | 19:30 |
| dviroel | at 'ci-config'? | 19:30 |
| rlandy|ruck | yeah | 19:31 |
| rlandy|ruck | and we will create another job in config | 19:31 |
| rlandy|ruck | parented off the one in cic-onfig | 19:31 |
| dviroel | an then in 'config' we just use the as parent job and add the secret? | 19:31 |
| rlandy|ruck | ci-config | 19:31 |
| dviroel | ack | 19:31 |
| rlandy|ruck | with the secret added | 19:31 |
| rlandy|ruck | I think that will work | 19:31 |
| rlandy|ruck | but I think we will need the job merged to try that in config | 19:32 |
| rlandy|ruck | because of the no speculative testing condition | 19:32 |
| rlandy|ruck | we'll have to code, merge, try | 19:32 |
| dviroel | ok, as long as I have everything in ci-config, i can use molecule jobs to help on validate | 19:33 |
| rlandy|ruck | yeah - that;s the safest approach | 19:33 |
| dviroel | rlandy|ruck: great, thanks | 19:33 |
| rlandy|ruck | what we were doing before, with a script | 19:33 |
| rlandy|ruck | is what wes wanted to get away from | 19:34 |
| dviroel | +1 | 19:34 |
| rlandy|ruck | to do eaxctly what you are suggested | 19:34 |
| rlandy|ruck | self contained, testable | 19:34 |
| rlandy|ruck | and min related to a real server upload | 19:34 |
| rlandy|ruck | we can touch base on this again at tomorrow's scrum | 19:35 |
| dviroel | sure, good idea, i'll need a feedback from the team for the ansible/roles/module approach too | 19:36 |
| dviroel | will update the task with some links | 19:37 |
| rlandy|ruck | ++ | 19:45 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!