Thursday, 2019-10-03

*** rlandy has quit IRC00:13
*** ykarel|away has joined #oooq01:58
*** apetrich has quit IRC02:10
*** dsneddon has quit IRC03:50
*** dsneddon has joined #oooq04:01
*** dsneddon has quit IRC04:08
*** udesale has joined #oooq04:13
*** ykarel|away has quit IRC04:22
*** surpatil has joined #oooq04:29
*** soniya29 has joined #oooq04:37
*** dsneddon has joined #oooq04:39
*** ykarel|away has joined #oooq04:41
*** saneax has joined #oooq04:46
*** ykarel|away is now known as ykarel04:56
*** marios has joined #oooq05:15
*** ratailor has joined #oooq05:26
*** akahat has joined #oooq05:39
*** skramaja has joined #oooq05:43
*** ykarel is now known as ykarel|meeting05:44
*** ykarel_ has joined #oooq05:48
*** ratailor_ has joined #oooq05:48
*** ykarel|meeting has quit IRC05:50
*** ratailor has quit IRC05:52
*** dsneddon has quit IRC06:05
*** surpatil has quit IRC06:10
*** surpatil has joined #oooq06:10
*** surpatil has quit IRC06:12
*** surpatil has joined #oooq06:13
*** jfrancoa has joined #oooq06:15
*** dsneddon has joined #oooq06:20
*** jaosorior has joined #oooq06:21
*** holser has joined #oooq06:24
*** dsneddon has quit IRC06:25
*** holser has quit IRC06:28
*** surpatil has quit IRC06:30
*** surpatil has joined #oooq06:34
*** saneax has quit IRC06:34
*** dsneddon has joined #oooq06:35
*** dsneddon has quit IRC06:40
*** holser has joined #oooq06:46
*** bogdando has joined #oooq06:49
chandankumarmarios: you are too fast :-)06:52
marioschandankumar: don't worry about updating it man lets just wait for zuul06:53
*** dsneddon has joined #oooq06:54
*** udesale has quit IRC06:58
*** udesale has joined #oooq06:59
*** akahat has quit IRC07:00
*** udesale has quit IRC07:02
*** udesale has joined #oooq07:02
*** dsneddon has quit IRC07:05
*** dsneddon has joined #oooq07:06
*** tesseract has joined #oooq07:12
*** tosky has joined #oooq07:16
*** ykarel_ is now known as ykarel|meeting07:24
*** ratailor__ has joined #oooq07:35
*** ykarel_ has joined #oooq07:36
*** ykarel|meeting has quit IRC07:36
*** ykarel_ is now known as ykarel07:37
*** ratailor_ has quit IRC07:37
*** amoralej|off is now known as amoralej07:39
*** dsneddon_ has joined #oooq07:39
*** surpatil is now known as surpatil|lunch07:41
*** dsneddon has quit IRC07:42
*** udesale has quit IRC07:48
*** udesale has joined #oooq07:48
bogdandoo/ please merge https://review.opendev.org/#/q/topic:ci-repro+(status:open+OR+status:merged)07:49
bogdandothis is needed for zuul-based reproducer and libvirt multinode jobs07:49
*** jpena|off is now known as jpena07:49
*** ykarel is now known as ykarel|lunch07:49
*** kopecmartin|off is now known as kopecmartin07:53
*** akahat has joined #oooq07:54
*** jaosorior has quit IRC08:19
*** panda|off is now known as panda08:20
*** recheck_ has quit IRC08:22
*** recheck has joined #oooq08:22
*** recheck has quit IRC08:25
*** recheck has joined #oooq08:25
*** recheck has quit IRC08:27
*** recheck has joined #oooq08:27
*** ccamacho has joined #oooq08:31
*** jbadiapa has joined #oooq08:34
zbr|ruckpanda: ping!08:40
*** ykarel|lunch is now known as ykarel08:40
*** derekh has joined #oooq08:42
pandazbr|ruck: ICMP TTL exceeded.08:42
pandaI just want to die.08:42
zbr|ruckpanda: your wish will be granted, but not necessarily at a time you want.08:45
zbr|ruckpanda: https://review.rdoproject.org/r/#/c/22822/08:45
zbr|rucki am wondering why emilien is added automatically to each review, i wonder if this is caused by my git-review or if is server side08:46
zbr|ruckapparently I do not have the feature enabled, so it must be server side.08:49
pandazbr|ruck: I'm going to block any change on that repo until we agree on a plan to move forward. I'm proposing to either branch the repo, or clone the promoter role directory so we have an older promoter code and a new promoter code. The old promoter part shall not accept any change at all, as we have no way to test the old code, any change put there will just break it. TOo fragile, just fork it, leave it alone08:50
pandaand then kill it.08:50
pandamarios: ^08:51
zbr|ruckpanda: you are better looking at sprint tasks. there is too much pressure now and not enough time.08:52
mariospanda: ack i think we will be discussing this in depth @ scrum ... 'new' directory sounds clean if we can get away without branching it08:52
pandazbr|ruck: not enough time for ?08:53
zbr|ruckmarios: yeah, I like marios proposal, we can have a "promoter2" role, for a while.08:53
pandamarios: zbr|ruck taht also means that we need to update all the code that we have taht is not merged yet08:53
marioszbr|ruck: its panda proposal :D08:53
zbr|ruckpanda propose new branch, but new role seems easier to deal with.08:54
pandazbr|ruck: I proposed new dir too.08:54
zbr|ruckas long you do not copy the entire ci-scripts it should be fine.08:55
pandaok since we are all in agreement, I'll make the changes08:56
zbr|ruckpanda: can you please tell me why my pipefail change broke molecule-delegated. at https://review.rdoproject.org/r/#/c/22763/08:56
pandanot sure where ... I have a stack of 6 changes now all floating08:57
pandaah yes, I have it.08:57
zbr|ruckpanda: make small ones and i will be glad to help you merge them.08:57
pandazbr|ruck: I already have small ones, only 200 lines each08:57
pandawell one is 50008:57
pandathis is the smallest they can get08:57
chandankumarmarios: any thing more needed on this https://review.opendev.org/#/c/684260/ as rocky job is passing08:58
*** jbadiapa has quit IRC08:59
mariosack revoted chandankumar09:05
mariospanda: could i add a pip install dlrn here https://review.rdoproject.org/r/#/c/22772/6/playbooks/molecule-delegated-promote-images.yml09:05
chandankumarmarios: thanks!09:05
mariospanda: i tried added there https://review.rdoproject.org/r/#/c/22772/6/playbooks/molecule-delegated-promote-images-pre.yml but it didn't like it09:05
pandamarios: remove it from here https://review.rdoproject.org/r/#/c/22772/6/playbooks/molecule-delegated-promote-images-pre.yml@28 and you should be fine. dlrn is not an rpm package AFAIK just python package09:11
mariospanda: k (trying with pip but will remove it if you think not necessary... needed it in my local virtual env thats why i added it there)09:11
zbr|rucki fixed the venv on promoter, next run should pass, it failed to install docker because some files were owned by root inside promoter_venv09:12
pandado not fixe anything on the promoter09:15
pandamarios: pip is fine, just try not to install as an rpm package, because it doesn't exist09:16
pandazbr|ruck: nobody should touch the promoter at all at this point, it worked until now, we revert any change that we have, and leave it alone. If you want to fix something fix it on the new code09:17
mariospanda: ack also added docker there (via pip)... the scenario is going to fail didn't deal with the promote-images.sh images yet  justt trying to get the ci to the same point as my local09:17
zbr|ruckmarios: can you help me debug something? i want to find out why that image was not found on the image server. http://promoter.rdoproject.org/centos7_stein.log09:32
zbr|ruckthat line: The promoted hash was not found: https://images.rdoproject.org/centos7/stein/rdo_trunk09:33
pandazbr|ruck: how old is this hash ?09:34
marioszbr|ruck: weird ... i was going to say maybe the upload job failed for that hash but then the other jobs wouldn't run if it was the case09:34
mariospanda: but the log is from today09:35
pandahow old is this hash ?09:35
pandathe log may be from today, but the hash is what matters09:35
pandathere is a script that delets old images in the images server09:35
mariospanda: yeah i mean why would it promote an old hash but running today09:35
pandaif they don't have any links attached to them09:35
pandaso it's entirely possible the images were uploaded 5 days ago, and since no promotion happened, they were erased09:36
pandaevery small action on the promoter has a devastating cascading effect ...09:37
*** udesale has quit IRC09:53
*** udesale has joined #oooq09:54
*** surpatil has joined #oooq09:56
*** surpatil|lunch has quit IRC09:56
*** jaosorior has joined #oooq09:59
*** jaosorior has quit IRC10:06
zbr|rucktomorrow i am going in PTO, never being so happy about that.10:09
*** dsneddon_ has quit IRC10:09
zbr|ruckpanda: please check https://review.rdoproject.org/r/#/c/22822/10:09
pandazbr|ruck: what would this accomplish ?10:13
zbr|ruckpanda: is stated clearly in the message. also we do have a rule about no non-voting jobs for long time.10:14
zbr|rucki had a case where one of my changes broke it and I almost ignored it, because I did (wrongly) assume that the nv jobs broken.10:14
zbr|ruckif is working and not flaky it must be voting.10:15
pandazbr|ruck: let me summarize what it's happening there right now. We have the new code that has tests. We also have a production promoter server using old code that CANNOT be tested in any way other than manually pulling changes and see if we can block the promotions for 5 days. You want to make voting a job taht tests the new code only, that would also potentiall block changes to the old code for no reasons.10:16
mariospanda: wdyt https://review.rdoproject.org/r/#/c/22655/8/ci-scripts/dlrnapi_promoter/tests/staging-setup/staging_environment.py@15710:17
mariosbrb10:17
zbr|ruckpanda: your sentence makes no sense to me.10:18
pandazbr|ruck: take a look ata https://review.rdoproject.org/r/22805, patches 1 and 210:20
pandathis was needed to fix the promoter, as it was using ansible in side the virtualenv10:20
pandaand not working at all10:20
pandazbr|ruck: tdo-tox-molecule was failing because now the ansibe-playbook was not inside the virtualenv10:21
pandazbr|ruck: so a job that is testing the new promoter code, was blocking a change that was needed for the old promoter code, that one that uses ansible from package10:21
pandathe solution here was to remove part of the new code, to make promoter work again, but the test was testing that the new code works10:22
pandacomplete mismatch10:22
zbr|ruckbj in 5? i need to get a coffee, maybe we can clarify few more things before i get insane10:23
*** jaosorior has joined #oooq10:24
pandaI'm already insane10:25
*** dsneddon_ has joined #oooq10:26
zbr|ruckhttps://bluejeans.com/265541792810:30
pandamarios: what you're asking is partially here https://review.rdoproject.org/r/#/c/22664/10:31
*** dsneddon_ has quit IRC10:33
mariospanda: k will check thx but i think we need it anyway i.e. it should match the real one. might be a bit messy to 'fix' it now i wonder how many places reference the path as /centos/ already10:34
*** saneax has joined #oooq10:45
*** jaosorior has quit IRC10:51
*** jpena is now known as jpena|lunch11:00
*** dsneddon_ has joined #oooq11:03
*** udesale has quit IRC11:17
amoralejzbr|ruck, we've updated ovs version in queens https://review.rdoproject.org/r/#/c/22769/ you should start getting it updated in jobs in some minutes11:29
amoralejlet us know if you observe anything abnormal11:29
zbr|ruckamoralej: sure, if I get pass the other broken stuff.11:33
amoralejor if this update breaks anything that was working now :)11:34
amoralejhopefully not, it passed ci11:35
*** tosky_ has joined #oooq11:44
*** tosky has quit IRC11:46
*** soniya29 has quit IRC11:47
*** dsneddon_ has quit IRC11:47
*** ratailor__ has quit IRC11:50
*** saneax has quit IRC11:54
*** amoralej is now known as amoralej|lunch11:55
*** akahat has quit IRC11:58
EmilienMzbr|ruck: no worries, I'll remove myself11:59
weshayzbr|ruck, can you please kick a testproject that builds the stein images (upload featureset02) for 2019-10-03 04:07:30,949 30687 ERROR    promoter ERROR: The promoted hash was not found: https://images.rdoproject.org/centos7/stein/rdo_trunk/a361c5edd5734320f781e10358a602f4ba57fc04_647b08e411:59
weshayamoralej|lunch, anything abnormal w/ queens.. lolz12:00
weshaynow I have to revalidate :(12:01
*** tosky_ is now known as tosky12:01
weshayzbr|ruck, rhel-8 promoted :)12:04
weshaystein missing overcloud images12:04
rfolcopanda, hi o/12:05
rfolcopanda, what happened w/ delegated job failing on last check ?12:05
*** dsneddon_ has joined #oooq12:06
*** dsneddon_ has quit IRC12:12
*** jpena|lunch is now known as jpena12:12
*** soniya29 has joined #oooq12:15
zbr|ruckweshay: like https://review.rdoproject.org/r/#/c/22797/ but for stein?12:16
weshayzbr|ruck, ya.. but this time ur going to have to specify the hash please12:17
weshay2019-10-03 04:07:30,949 30687 ERROR    promoter ERROR: The promoted hash was not found: https://images.rdoproject.org/centos7/stein/rdo_trunk/a361c5edd5734320f781e10358a602f4ba57fc04_647b08e412:17
rfolcopanda, marios I see some attempts to fix delegated container-push test... not sure why this started failing though...12:18
rfolcohttps://review.rdoproject.org/r/#/c/22656/5..8/ci-scripts/infra-setup/roles/promoter/molecule/container-push/playbook.yml12:18
zbr|ruckweshay: takes time, as I need to dig how to do it12:20
weshaynp12:20
mariosrfolco: not from me (I just rebased for conflict today )12:20
rfolcomarios, ok, I'm running it locally to find out what broke it12:25
weshayzbr|ruck, easiest way might be a dnm depends-on patch that adjusts a promotion job release file12:27
weshaypanda, rfolco so.. rhel8 master promoted afaict.. but I don't see a big message saying so in http://promoter.rdoproject.org/redhat8_master.log12:28
weshay2019-10-03 05:46:06,773 9109 INFO     promoter SUCCESS promoting tripleo-ci-testing as current-tripleo ({'timestamp': 1570018326, 'distro_hash': '9e5988125e88f803ba20743be7aa99079dd275f2', 'promote_name': 'tripleo-ci-testing', 'user': 'review_rdoproject_org', 'repo_url': 'https://trunk.rdoproject.org/rhel8-master/96/8c/968c9526ff2eeafaa13207a791f61173db604fff_9e598812', 'full_hash': '968c9526ff2eeafaa13207a791f61173db604fff_9e598812',12:28
weshay'repo_hash': '968c9526ff2eeafaa13207a791f61173db604fff_9e598812', 'commit_hash': '968c9526ff2eeafaa13207a791f61173db604fff'})12:28
weshayah.. I guess that12:28
weshaypanda, zbr|ruck feature request :)12:28
panda?12:29
weshaya line of \0/\0/\0/\0/\0/\0/\0/\0/\0/\0/\0/\0/\0/\0/\0/\0/\0/\0/\0/\0/\0/\0/\0/\0/\0/\0/\0/\0/\0/\0/\0/12:29
weshaypanda, ya.. sorry.. was just hard to find12:29
pandaweshay: after we have the test we can turn the promoter inside out, starting from the logging.12:30
weshaya line of \0/ \0/ \0/ \0/ \0/ \0/ \0/ \0/ \0/ \0/ \0/ \0/ \0/ \0/\0/ \0/ \0/ \0/ \0/ \0/12:30
weshayk12:30
pandarfolco: which one ?12:30
rfolcoweshay, lets add some ascii art to the logs w/ a nice pic like a trophy or a big smile12:30
pandarfolco: wonderful, but only AFTER we have test merged12:30
pandarfolco: and the server is using the tested code12:31
pandawe have a bit to discuss during the scrum12:31
chandankumarweshay: time for 1:112:31
rfolcopanda, delegated container-push is failing since patchset 6 in your top patch12:31
chandankumarweshay: https://meet.google.com/jfx-pwrz-meb12:31
pandarfolco: waht is failing right now ?12:31
rfolcohttps://review.rdoproject.org/r/#/c/22656/5..8/ci-scripts/infra-setup/roles/promoter/molecule/container-push/playbook.yml12:31
rfolcops5 --> molecule-delegated SUCCESS in 4m 55s (non-voting)12:32
rfolcops6 --> molecule-delegated FAILURE in 4m 33s (non-voting)12:32
pandaWTH12:33
pandawhat's "playbook" ?12:33
rfolconothing changed from 5 to 612:33
rfolcoin the playbook/test itself12:33
rfolcopanda, check if any image left behind task is failing... looking for what broke it12:34
*** rlandy has joined #oooq12:34
pandawe need to regroup12:36
pandarfolco: the new conditional  with |length == 0 is not working as expected12:37
rfolcopanda@radio: stick together team. fire in the hole!12:37
rfolcoroger that12:37
*** dsneddon_ has joined #oooq12:38
rfolcopanda, it was in ps5...12:38
*** amoralej|lunch is now known as amoralej12:44
pandarfolco: mmmmhh12:48
pandarfolco: I know what it is12:49
pandarfolco: it's the validation container12:49
pandarfolco: the one that validates the login12:49
rfolcopanda, whhhaaaaat? looks a harmless test12:49
pandarfolco: a harmless test taht adds a new container to the registry wihtout deleting it12:50
rfolcoaaaaaaah12:50
pandarfolco: so when then check "did we remove everything ?" comes, it finds the nop container still and it says: you forgot to delete something12:50
rfolcopanda, perfect. I thought we were deleting that test in the other patch12:51
pandarfolco: we have to reaorganize the patches now or we'll not be able to merge anything12:52
rfolcopanda, ok good lets chat in scrum, will grab coffee and some bad words to insult people breaking my patch12:53
beaglesbtw am aware that octavia scenario 10 job is giving grief on master right now http://zuul.openstack.org/builds?job_name=tripleo-ci-centos-7-scenario010-standalone13:00
beagleslooking into it13:00
*** skramaja has quit IRC13:01
weshayrlandy, rfolco https://review.rdoproject.org/r/#/c/22800/ https://review.rdoproject.org/r/#/c/22801/13:01
chandankumarrlandy: needs +w on this https://review.opendev.org/#/c/684260/13:07
rlandylooking13:08
rlandyweshay: https://github.com/rdo-infra/review.rdoproject.org-config/blob/a370e1e788f47088f8768901add4091671abb4af/zuul.d/upstream.yaml#L2113:15
weshayrlandy, thanks13:16
*** aakarsh has quit IRC13:17
zbr|ruckweshay: ping when you can.13:25
weshayzbr|ruck, hola13:25
mariosrfolco: (cos we discussed before fyi that is blocked https://tree.taiga.io/project/tripleo-ci-board/task/1315?kanban-status=1447274 until we merge https://review.rdoproject.org/r/#/c/22656/ which wires up the manifest push... until then we have no _manifest in rdo registry13:25
mariosrfolco: err in docker.io/quay i mean13:25
zbr|ruckweshay: https://meet.google.com/mtg-nhiw-tdd13:26
weshayzbr|ruck, I'm in scrum.. can chat live at the top of the hour13:26
zbr|ruckok, because I am kinda clueless regarding building that specific image13:27
*** udesale has joined #oooq13:33
weshayzbr|ruck, for stein.. fs02 upload images does it13:35
weshayzbr|ruck, can you please jump off the promotion server.. I'm limiting access to just myself at this time13:36
weshaythanks13:36
zbr|rucki am out13:36
weshayzbr|ruck, thanks.. periodic-tripleo-ci-centos-7-ovb-1ctlr_1comp-featureset002-stein-upload13:37
weshayis the job.. that builds the images, tests them.. and uploads13:37
* weshay checks the image server.. this should not have promoted in the first place w/ that missing13:37
zbr|ruckyeah but you asked me to promote a specific hash, that is what did not know.13:37
weshayhttps://github.com/rdo-infra/ci-config/blob/master/ci-scripts/dlrnapi_promoter/config/CentOS-7/stein.ini#L2013:38
weshayah.. rlandy we can specify a hash in testproject review right? ^13:39
rlandyack13:39
rlandyweshay: see how we do that in reproducer13:40
weshayzbr|ruck,13:41
weshayvars:13:41
weshayfeatureset_override:13:41
weshay              dlrn_hash_tag: a317d33dbcda3681a967ce1ed983020ff3512311_58c6e4ba13:41
weshayindented correctly of course13:41
zbr|ruckthanks!13:42
zbr|ruckweshay: https://review.rdoproject.org/r/#/c/22868/1/.zuul.yaml ?13:45
*** aakarsh has joined #oooq13:50
*** surpatil has quit IRC13:54
*** soniya29 has quit IRC13:54
weshay2019-10-03 09:23:44.917571 | primary | +(./toci_quickstart.sh:46): main(): export 'QUICKSTART_DEFAULT_RELEASE_ARG=--extra-vars @/home/zuul/workspace/.quickstart/config/release/tripleo-ci/RedHat-8/master.yml'13:55
weshaydlrn_hash_tag: current-tripleo13:55
*** rnoriega- has joined #oooq14:06
*** Vorrtex has joined #oooq14:07
*** rnoriega_ has quit IRC14:09
rlandyweshay: can you post the job link please14:11
weshayhttp://logs.rdoproject.org/53/682853/13/openstack-check/tripleo-ci-rhel-8-scenario004-standalone-rdo/3dd6834/job-output.txt14:12
rlandy2019-10-03 09:33:56 |       tag: current-tripleo14:15
*** Goneri has joined #oooq14:17
weshayrlandy,  proxy issue http://mirror.regionone.rdo-cloud-tripleo.rdoproject.org:8080/rdo/rhel8-master/current-tripleo/delorean.repo14:19
rlandyinteresting - commit is correct14:21
weshayzbr|ruck, ok.. proxy issues in rdo :)14:29
weshayzbr|ruck, https://bugs.launchpad.net/tripleo/+bug/184650414:30
openstackLaunchpad bug 1846504 in tripleo "rdo mirror/proxy not working correctly" [Critical,Triaged] - Assigned to wes hayutin (weshayutin)14:30
weshayzbr|ruck, looks ok now..14:30
weshayzbr|ruck, I can chat now14:30
rfolcopanda, pls let me know when you get validation nop fixed / removed or if you want me to do that14:30
bogdandohi. please merge https://review.opendev.org/#/q/topic:ci-repro+(status:open+OR+status:merged)14:31
bogdandothis is needed for zuul-based reproducer and libvirt multinode jobs14:31
bogdandorlandy, weshay: ^^14:31
weshaybogdando, cool /me looks14:31
bogdandothanks!14:32
*** ykarel is now known as ykarel|afk14:33
*** rfolco is now known as rfolco|bbl14:35
rlandylooking14:35
weshayrlandy,  I think we can merge both of those?14:54
rlandyweshay: ack14:54
*** jpena is now known as jpena|off14:59
pandarfolco|bbl: delegated job fixed15:00
bogdandorlandy, weshay: thanks!15:03
weshaypanda, this is an anchor definition right?  &scenario00415:08
weshaywhen one uses &15:08
weshayzbr|ruck, ^15:09
zbr|ruckyeag15:10
rlandyweshay: just fyi - to fox failing master/stein bm ...  https://code.engineering.redhat.com/gerrit/182659 Add overcloud_container_cli to match OVB15:11
zbr|ruckweshay: this explains it ok-ish: https://confluence.atlassian.com/bitbucket/yaml-anchors-960154027.html15:11
zbr|rucksimple case: to define &foo. and to use/expand &foo15:11
zbr|ruckbut if you need to override the expansion, you will see <<: *foo construct15:12
zbr|ruckit does not work with lists, only with dictionaries15:12
*** ykarel|afk is now known as ykarel|away15:13
pandaweshay: yes, the *scenario004 will be the reference to that definition15:13
pandaweshay: rlandy rfolco|bbl marios all patches are rebased and are passing voting tests. I'll start to merge stuff with your blessing now.15:17
mariospanda: ack15:18
rlandypanda: sec - I need to change staging/playbook15:19
rlandywill update15:19
*** jbadiapa has joined #oooq15:19
pandarlandy: yesterday I forgot to mention that you can test the test_promoter_test.py locally wiht the command "tox -epy27 -epy35  ci-scripts/dlrnapi_promoter/tests/staging-setup/test_promoter_test.py" it will emulate what the job upstream is doing, but locally and only for the test that concerns you15:20
rlandypanda: https://review.rdoproject.org/r/#/c/22804/ fails a lot of tests - not sure we want to merge that yet15:21
pandarlandy: I'll start merging my 4 patches in the stack, no rush15:21
rlandypanda: sorry - got caught up in bm  stuff15:21
mariospanda: i think we need to tweak the images setup a bit more, not just version15:23
*** Vorrtex has quit IRC15:23
mariospanda: for example add a current-tripleo there too15:23
mariospanda: but we can do it on top15:23
mariospanda: i'll explain tomorrow, but basically the promote-images.sh is expecting an existing 'current-tripleo' that is can rename to previous-current15:24
mariospanda: you around tomorrow?15:24
pandamarios: yes15:24
pandamarios: I think I fixed that in the latest refactor15:25
mariospanda: ok lets do a sync in our morning then if you have time ... the skeleton is there now https://review.rdoproject.org/r/#/c/22772/11/ci-scripts/infra-setup/roles/promoter/molecule/promote-images/playbook.yml have a look if you get time15:25
rlandypanda: no worries - I'll merge the test stuff when it passes - won't bock anyone else's work15:25
pandamarios: the stage_info.yaml file now contains all the commits with some special pointer15:25
pandamarios: for example you can point directly at the currently promoted commit,, the previous promoted commit, and the one that is supposed to be promoted15:26
pandarlandy: let me know if I can help15:26
*** ykarel|away has quit IRC15:26
mariospanda: see comment there and logs https://review.rdoproject.org/r/#/c/22772/11/ci-scripts/infra-setup/roles/promoter/molecule/promote-images/playbook.yml (for more on what i mean, otherwise we talk tomorrow anyway thanks)15:27
rlandypanda: hmmm ... looks like linters are failing on unrelated changes15:33
rlandyhttp://logs.rdoproject.org/04/22804/2/check/tox-linters/dffcb2d/job-output.txt15:34
*** udesale has quit IRC15:35
*** udesale has joined #oooq15:35
pandarlandy: look for timestamp 2019-10-03 15:23:50.78028415:36
*** tesseract has quit IRC15:36
rlandyk - fixing15:36
*** dsneddon_ has quit IRC15:41
*** bogdando has quit IRC15:42
weshayrlandy,  fyi https://review.opendev.org/68643015:42
rlandywill look in a sec15:43
rlandy019-10-03 15:23:50.781394 | rdo-centos-7 | ci-scripts/dlrnapi_promoter/tests/staging-setup/test_promoter_test.py:72:60: E226 missing whitespace around arithmetic operator15:46
rlandy2019-10-03 15:23:50.781446 | rdo-centos-7 |         mock_urllib.side_effect = url_lib.HTTPError(*[None]*5)15:46
rlandypanda: ^^ fixed all error except above. what is your intention here?15:47
weshaymarios, panda need reviews on this.. and an update to promotion criteria https://review.rdoproject.org/r/#/c/22867/15:48
mariosrfolco|bbl: rlandy panda setup call tomorrow during 'scrum' time? (should i)15:49
rlandymarios: ack - thanks15:49
*** ykarel|away has joined #oooq15:49
*** jbadiapa has quit IRC15:50
pandamarios: yes15:51
panda... if you don't see me tomorrow is probably because storm lorenzo wiped out my home.15:52
mariosweshay: ack15:54
weshaythanks15:54
weshayrfolco|bbl, ugh15:55
weshayrfolco|bbl, ping me when ur back15:55
pandarfolco|bbl: ping also me when ur back15:57
panda:)15:57
*** rfolco|bbl is now known as rfolco16:00
rfolcoweshay, here o/16:00
rfolcoback from lunch16:00
*** marios has quit IRC16:01
rfolcook gimme 10 min brb16:04
zbr|ruckpanda: ouch,  on saturday I have a flight, and guess where I am supposed to fly from... https://www.google.com/maps/@49.663661,-11.8838656,5z/data=!3m1!4b1!4m3!15m2!1m1!1s%2Fg%2F11fnqbt29816:05
weshayzbr|ruck, I think you could probaby fix this one https://bugs.launchpad.net/tripleo/+bug/1846523 it's either kolla or a bad rpm spec I think16:06
openstackLaunchpad bug 1846523 in tripleo "tripleo rhel8 swift-proxy-server container fails to build due to python2 package in spec" [Critical,Triaged]16:06
zbr|ruckweshay: sure, looking now.16:07
pandazbr|ruck: oh happy days16:07
weshayzbr|ruck, btw.. I wanted to chat w/ you re: the docker login test.. so as you have time16:07
zbr|ruckweshay: now.16:08
weshayzbr|ruck, ok16:09
weshayzbr|ruck, https://bluejeans.com/u/whayutin16:09
*** jbadiapa has joined #oooq16:10
*** dsneddon_ has joined #oooq16:14
*** jbadiapa has quit IRC16:27
weshayzbr|ruck, https://review.opendev.org/#/c/685177/16:27
weshayhttps://review.opendev.org/#/c/685184/2/roles/create-reproducer-script/templates/reproducer-quickstart.sh.j216:27
*** udesale has quit IRC16:28
*** ccamacho has quit IRC16:31
chandankumarsee ya tomorrow16:34
chandankumarweshay: please have a look at emit relase review https://review.opendev.org/684260 when free!16:34
*** chandankumar is now known as raukadah16:34
zbr|ruckweshay: very weird python2-ceilometermiddleware is nowhere to be found. but the other one is.16:36
raukadah:-(16:36
weshayraukadah, don't be sad... just need more reviews :)16:36
weshaysame thing I said on the mtg16:36
raukadahweshay: I will bring in tuesday tripleo meeting if not get merged then16:37
weshay++16:37
ykarel|awayzbr|ruck, python2- packages are removed for rhel8 and fedora16:39
ykarel|awayu seeing issues somewhere?16:39
weshayykarel|away, ya.. container builds for rhel816:39
ykarel|awayweshay, nothing should rely on python2 packages now16:39
ykarel|awayswift is full python316:40
weshayaye16:40
zbr|ruckykarel|away: we agree here, but we need to figureout where this comes from: http://logs.rdoproject.org/openstack-periodic-master/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-rhel-8-master-containers-build-push/7a736e5/logs/containers-consolidated-builds.log.txt.gz16:40
* ykarel|away looks16:40
raukadahykarel|away: time to revisit rhel8 workaround patch in kolla16:40
raukadahmay be we can get rid of that16:40
zbr|ruckouch... i bet that patch is again the cause16:41
*** jpena|off is now known as jpena16:41
ykarel|awayif workaroud is related to py2 packages then yes need to get rid of it16:41
raukadaharxcruz|ruck: weshay we got some patches for f21 skip list autmoation https://review.rdoproject.org/r/22862 and https://review.rdoproject.org/r/22870 please review when free16:42
zbr|ruckfound!16:43
ykarel|awayues https://review.opendev.org/#/c/671183/9/docker/swift/swift-proxy-server/Dockerfile.j216:43
zbr|ruckdone. updated review.16:46
ykarel|awayack16:50
*** dsneddon_ has quit IRC16:53
*** dsneddon_ has joined #oooq16:54
*** derekh has quit IRC16:55
*** dsneddon_ has quit IRC16:59
weshay rlandy can you please workflow this.. I may be wrong about how to do it.. but using it to test. https://review.opendev.org/#/c/682853/17:00
rlandylooking17:00
*** dsneddon_ has joined #oooq17:00
weshayrlandy, oh wait17:01
weshaysorry sorry17:01
rlandyweshay: openstack/tripleo-heat-templates17:01
rlandyI can +1 vote17:01
weshay            "standalone_container_cli": "podman",17:01
rlandyscenario004 runs and passes17:01
rlandyso that is ok17:02
rlandypodman is correct for where we don;t use pacemaker17:02
weshaypodman is correct for rhel 8 period, w/ or w/o pacemaker17:03
ykarel|awayzbr|ruck, re. stein-upload job https://review.rdoproject.org/r/#/c/22868/1/.zuul.yaml? from where u got that hash17:03
ykarel|awayi think a361c5edd5734320f781e10358a602f4ba57fc04_647b08e4 is needed to have promotion17:04
weshayrlandy, hrm.. it's still using docker17:04
weshay2019-10-03 10:54:43 | fatal: [standalone]: FAILED! => {"changed": false, "checksum": "ef190563a3dbc423ae8c85c94f1dec85274be531", "msg": "Destination directory /etc/docker does not exist"}17:04
weshaybut container_cli is set to podman17:04
weshaythat's where I'm getting mixed up17:04
* rlandy looks17:06
rlandyweshay: which test did you look at?17:07
weshayhttp://logs.rdoproject.org/53/682853/13/openstack-check/tripleo-ci-rhel-8-scenario004-standalone-rdo/773f529/17:07
rlandythat should be podman17:07
rlandyI think we went down this road17:08
rlandygetting reviews17:08
*** dsneddon_ has quit IRC17:09
weshayrlandy,  ah look at that17:09
weshayhttp://logs.rdoproject.org/53/682853/13/openstack-check/tripleo-ci-rhel-8-scenario004-standalone-rdo/773f529/logs/undercloud/home/zuul/standalone_parameters.yaml.txt.gz17:09
weshay  ContainerCli: podman17:09
rlandyContainerCli: podman17:09
rlandyyep17:09
weshayseems like a boog17:10
*** dsneddon_ has joined #oooq17:10
weshayrlandy, so the passed default_params are not overriding https://opendev.org/openstack/tripleo-heat-templates/src/branch/master/ci/environments/scenario004-standalone.yaml#L10617:10
rlandy think it's in the heat templates17:10
zbr|ruckykarel|away: from wes17:11
ykarel|awayzbr|ruck, good to cross check than17:11
ykarel|awayhttps://trunk-primary.rdoproject.org/api-centos-stein/api/civotes_detail.html?commit_hash=a361c5edd5734320f781e10358a602f4ba57fc04&distro_hash=647b08e47a72f7533142c09074f227628f08f9fa17:12
rlandyhttps://review.opendev.org/#/c/679308/17:14
weshayrlandy, perhaps the bug is here.. for not switching on container_cli https://opendev.org/openstack/ansible-role-container-registry/src/branch/master/tasks/main.yml17:14
rlandybut that was an ovb failure17:14
*** tosky has quit IRC17:14
*** dsneddon_ has quit IRC17:14
rlandyhttps://opendev.org/openstack/tripleo-heat-templates/src/branch/master/ci/environments/scenario004-standalone.yaml#L106 - so correct17:15
rlandyI remember scenarios004 is an iisue17:16
rlandylook in zuul17:16
rlandyhttps://github.com/rdo-infra/rdo-jobs/blob/master/zuul.d/standalone-jobs.yaml#L44917:18
rlandyfeatureset_override: *scenario004_podman17:18
rlandy^^ does that work??17:18
rlandyweshay: since when do we allow  featureset_override: *scenario004_podman?17:19
weshayrlandy, ya.. the issue is not w/ ci afaict17:19
weshayci has podman not docker17:19
weshayif I were to try to fix this...17:19
weshayI would set container_registry_deploy_docker to false in http://logs.rdoproject.org/53/682853/13/openstack-check/tripleo-ci-rhel-8-scenario004-standalone-rdo/773f529/logs/undercloud/home/zuul/standalone_parameters.yaml.txt.gz17:20
rlandyyou can try - there is speculative testing17:20
rlandybut strange now17:21
weshayrlandy, do we have a osp-15 or 16 job?17:21
weshaystandalone?17:21
rlandyhttps://review.rdoproject.org/zuul/builds?job_name=tripleo-ci-rhel-8-scenario004-standalone-rdo17:22
rlandyweshay: ack - we have 1517:22
rlandygetting17:22
weshayrlandy,  I don't think this is invoked in https://opendev.org/openstack/ansible-role-container-registry in podman17:22
rlandyweshay: https://sf.hosted.upshift.rdu2.redhat.com/zuul/t/tripleo-ci-internal/builds?job_name=tripleo-ci-rhel-8-standalone-rhos-1517:23
weshaythanks for the assist17:23
rlandyhttps://sf.hosted.upshift.rdu2.redhat.com/logs/periodic/code.engineering.redhat.com/openstack/tripleo-ci-internal-jobs/master/tripleo-ci-rhel-8-standalone-rhos-15/274fdf9/logs/undercloud/home/zuul/standalone_parameters.yaml.txt.gz17:24
rlandythat says podman17:24
pandaweshay: which ansible version we want to use in the promoter ? 2.4 from default repo  only ?17:24
*** dsneddon_ has joined #oooq17:24
rlandypanda: dig me out of my misery here ... how do I line break this to make python + linters happy? commit['full_hash'] == stage_info['promotions']['promotion_candidate']['full_hash']:17:30
pandarlandy: commit['full_hash'] = \17:31
pandarlandy: and then indent 4 the next line17:31
weshaypanda, what we have is working.. I 2.4 did not work when we tried I think17:31
weshayI think we have 2.8.x17:32
pandarlandy: or promotion_candidate = stage_info['promotions']['promotion_candidate']  ... then ... commit['full_hash'] = promotion_candidate['full_hash']17:32
weshayansible-2.8.4-1.el7.noarch17:32
rlandypanda: I tried option one  - linters not happy17:32
weshaypanda, if we want to freeze it.. we should jinja2 the /etc/yum/config17:32
pandaoh wait17:32
rlandytrying option two17:32
pandait's a conditional, then use the second17:32
rlandyack17:32
pandaweshay: ansible 2.8 comes from epel17:33
pandaweshay: and they both work ... we just need to agree on the level of compatibility in the tasks .. for example if this needs to run in ansible 2.4 I need to use with_items instead of "loop"17:34
rfolcopanda, https://review.rdoproject.org/r/#/c/22737/ is ready to merge IMO17:35
rfolcopanda, tripleo-common integration test17:35
weshaypanda, think it makes sense to use 2.8.417:36
rfolconow will adapt the same workflow for the zuul end-to-end job17:36
pandarfolco: that's what YOU think17:36
weshayand freeze the rpm config on the box w/ yum config17:36
*** amoralej is now known as amoralej|off17:36
weshayrfolco, we need to chat when you have  a sec17:36
pandaweshay: to use 2.8 from package we need to install epel too then17:36
pandatoo tired to do it now, next set of patches ...17:37
pandarfolco: please rebase over 22664 ... it's about to merge17:37
pandarfolco: and it's changeing a few things17:37
weshaypanda, epel is fine outside of tripleo work.. but locking is a good idea17:37
rfolcopanda, you make my life so ... so... exciting17:38
pandaweshay: ok, good to know we have the green light for ansible 28 but ONLY from package. focusing on moving the integration test forward, hope we have time to add that too17:38
pandarfolco: I know right ?17:38
rfolcoweshay, can I ping you later ? maybe in 1 hour17:39
weshayrfolco, ya.. that's fine..17:39
rfolcoweshay, thx17:39
rlandytox-linters success - yay!17:41
*** ykarel|away has quit IRC17:43
pandanevermind, we absolutely need ansible2.817:48
pandainstalling EPEL17:48
*** rfolco has quit IRC17:52
*** holser has quit IRC17:56
weshayrlandy, apparently this fixes it https://review.rdoproject.org/r/#/c/22876/3/zuul.d/standalone-jobs.yaml18:01
rlandyha - ok18:01
weshayhttps://github.com/openstack/tripleo-heat-templates/blob/master/environments/podman.yaml18:02
rlandywe should see it tum18:03
rlandyrun18:03
weshayrlandy, doesn't matter it's boosted18:04
weshayand nonvoting18:04
rlandycharming18:04
* rlandy loves non-voting jobs18:04
rlandyalmost as much as linters18:04
*** jpena is now known as jpena|off18:05
*** ykarel|away has joined #oooq18:11
pandamaerged 2266418:14
weshayrlandy,  it's being worked on..18:14
pandawe are in business again18:14
pandafrom here we can merge all the other stuff18:14
*** recheck has quit IRC18:34
*** recheck has joined #oooq18:34
*** recheck has quit IRC18:37
*** recheck has joined #oooq18:37
weshayneed to increase the ovb timeout18:37
weshay2019-09-25 19:14:18 | Ansible timed out at 3768 seconds.18:37
rlandyweshay: I just did that for ovb18:38
rlandydeploy timeout18:38
rlandyhad the same error18:38
weshayis that set in tripleo-ci/env/ovb?18:39
weshayrlandy, oh.. you mean .. it immediately overran the new timeout18:39
rlandyweshay: there is job timeout and deploy timeout18:39
rlandyI increased deploy_timeout18:39
weshayya.. that is what I'm looking at18:39
weshay2019-09-25 19:14:18 | Ansible timed out at 3768 seconds.18:40
weshayin the overcloud-deploy18:40
rlandyI override it in env_settings18:40
rlandyit was 9018:40
rlandyI have it at 15018:40
rlandywiggle room18:40
rlandyperformance regression????18:40
weshayhrm.. 3768 is 63min18:40
rlandyweshay: check the time deploy actually ran18:41
weshaytimeout is still set to 9018:41
weshay    --libvirt-type qemu  --timeout 9018:41
rlandyfrom the top of the log ti the bottom18:41
weshayhttp://logs.rdoproject.org/openstack-periodic-latest-released/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-stein/d83fc7e/logs/undercloud/home/zuul/overcloud-deploy.sh18:41
weshayk18:41
rlandyso then is the whole test timing out?18:41
weshay90 min18:44
weshayit hasn't change18:44
weshaywhere is your change18:44
zbr|ruckweshay: can we get rid of kolla cherry-picks? is the 2nd time in less than two weeks when we had to rebase them18:46
weshayzbr|ruck, we loose rhel 8 if we get rid of it dude18:46
zbr|ruckgettting rid  == merging them somehpw18:46
rlandyweshay: my change?18:47
zbr|rucksomehow I have the impression that we are all fighting to promote some cherry picks, i am not sure we can "scale" well this way. it was supposed to be a short term hack, more of a POC18:48
weshayzbr|ruck, the cherry picks are there until infra and centos get 818:48
weshayand probably kolla18:49
rlandypanda: think this is staging env error ... http://logs.rdoproject.org/04/22804/12/check/tox-py36/ee33b00/tox/reports.html18:49
rlandyweshay: I just kicked stein and master bm to compare18:55
*** rfolco has joined #oooq18:57
rfolcopanda, I'll check what broke my test18:58
*** holser has joined #oooq19:00
rlandyhttps://review.rdoproject.org/r/#/c/22804/19:12
rlandypanda: ^^ ok - tests are passing19:12
rlandyenough to merge19:12
*** rlandy is now known as rlandy|brb19:13
*** zbr|ruck is now known as zbr19:14
*** ykarel|away has quit IRC19:17
weshayrlandy|brb,  zbr rfolco  please wf https://review.rdoproject.org/r/#/c/22876/19:18
rfolcolooking19:18
weshayrfolco, job just passed in zuul19:19
rfolcoweshay, link handy?19:19
weshayrfolco, https://review.rdoproject.org/zuul/stream/df419e82a08647cc974c46aaf544ee1b?logfile=console.log19:21
*** holser has quit IRC19:22
weshayrfolco, do you have wf?19:24
rfolcoweshay, no :(19:24
weshayzbr,  do you have wf? https://review.rdoproject.org/r/#/c/22876/19:25
*** holser has joined #oooq19:31
rfolcoweshay, wanna chat?19:54
weshaypanda++19:56
weshayzbr, you have the queens patch up?19:58
weshayzbr, see it, I'm blind.. duh20:12
pandarfolco: I needed to modify molecule-delegated-pre a bit to not use ansible from virtualenv20:22
pandarfolco: maybe that's what it's breaking your patch20:22
rfolcopanda, no, full_hash has been changed20:23
rfolcoI fixed and rebased20:23
rfolcowaiting for results, it works local20:23
rfolcopanda, hope you don't make me rebase on anything else20:23
pandarfolco: ah yea, now it comes from stage_info['promotions']['promotion_cadidate']['full_hash']20:23
pandarfolco: I'm done with the patches20:24
rfolcopanda, as soon as I get +1 from zuul, please do a review and 2+ w+ if possible20:25
rfolcoimprovements can be done later, lets get something in for ths sprint20:25
rfolcopanda, green light https://review.rdoproject.org/r/#/c/22737/20:30
rfolcopanda, http://logs.rdoproject.org/37/22737/23/check/molecule-tripleo-common/4fb614a/job-output.txt20:31
*** rlandy|brb is now known as rlandy20:38
rlandyCould not resolve host: mirror.regionone.rdo-cloud.rdoproject.org20:39
rlandyhmmm20:39
rfolcorlandy, panda w+ if no objections please https://review.rdoproject.org/r/#/c/22737/20:40
pandarfolco: are you using any information from the staging environment script ?20:41
rlandyoh what now?20:41
rfolcopanda, I rely on artifacts like registries... containers etc20:41
rlandyhttps://review.rdoproject.org/r/#/c/22804/20:42
rfolcopanda, and I use /tmp/stage-info file as well20:42
rlandyhoe does that pass in check  and fail in gate???20:42
rlandyhow20:42
rfolcothats weird20:43
rlandyrecheck I guess20:43
rlandypanda: wrt your comment, I will uncomment the log check20:44
rlandybut wanted to get in what was there atm20:44
*** Goneri has quit IRC20:46
*** tosky has joined #oooq20:50
rlandyI may need this ... https://review.opendev.org/#/c/686430/20:52
weshayrlandy, ugh.. so i think I screwed up the order20:55
weshay  -e /usr/share/openstack-tripleo-heat-templates/environments/low-memory-usage.yaml -e /usr/share/openstack-tripleo-heat-templates/environments/podman.yaml -e /usr/share/openstack-tripleo-heat-templates/ci/environments/scenario004-standalone.yaml \20:55
weshaypodman.yml probably needs to be last?20:55
weshayrequest for preemptive wf https://review.rdoproject.org/r/#/c/22889/1/zuul.d/standalone-jobs.yaml20:57
*** aakarsh has quit IRC20:59
pandarlandy: commented on your patch21:06
pandarfolco: merged your patch21:06
rfolcopanda, \o/21:06
pandarfolco: probably option 3 is the easiest as you don't need dlrn db at all during tests, as we are mocking dlrnapi response21:07
pandarfolco: merged with comments ... :)21:08
pandaI meant rlandy: probably option 3 is the easiest as you don't need dlrn db at all during tests, as we are mocking dlrnapi response21:08
pandabut if they're really parallel, then other things may step on each other's toes, not just the db21:11
weshaypretty please https://review.rdoproject.org/r/#/c/22889/21:21
rlandyok21:21
*** jfrancoa has quit IRC21:21
pandarlandy: I'm seeing if I can get the tests to run serially intead of parallal21:22
rlandypanda: looking into option 321:22
pandaso we don't spawn 4 staging env setup21:22
rlandyweird that they pass check and not gate21:23
rlandyshould be the same21:23
pandarlandy: since parallelization is arbitrary, it'entirely possible that in check, the 4 tests were assigned to the same executor, and they ran in fact in serial21:23
rlandyah ok21:23
pandarlandy: testing serialization in https://review.rdoproject.org/r/2289021:24
rlandyweshay: ok  to wf https://review.rdoproject.org/r/#/c/22889/ - not sure how preemptive21:29
weshayrlandy, ya.. the job that is running isn't even testing scen00421:29
weshayso it just doesn't matter21:29
weshayrlandy, more interested in21:29
weshayyour thoughts on the order21:29
weshayI think the last one wins21:29
weshayrlandy, but it's safe to wf21:29
weshayfor sure21:29
rlandyweshay: always with yaml files21:29
rlandyk - w+ - we can revert21:30
weshayoh. that's how all yaml files work?21:30
weshaylast wins21:30
weshayguess that makes sense21:30
pandacrap, 150 nova reviews on rdo, nothing will ever ever merge now.21:32
rlandypanda: those aren't real21:33
rlandythey just show up for some reason every now and again21:34
rlandynova doesn21:34
rlandy't trigger on rdo21:34
pandaI know, but the wait is real ...21:35
pandarlandy: the serialization works with my tests. Le'ts see if I can undertand how to activate it on uinttest class.21:35
rlandythere is something about 5pm ish that makes rdo go nuts21:35
rlandypanda: great21:35
rlandyat least that explains the sporadic failures21:36
rlandyhttps://sf.hosted.upshift.rdu2.redhat.com/zuul/t/tripleo-ci-internal/stream/07a1dd98adb648ed95571649e302355c?logfile=console.log21:36
rlandyyay- master is getting somewhere21:36
rlandyweshay: ^^ we can compare that time to ovb when it starts deploying21:37
pandarlandy: seems to be working, mind if I upload new PS ?21:40
pandarlandy: otherwise it's really a 2 lines change21:42
pandarfolco: you should limit the tripleo-common job to the relevant files21:46
pandarfolco: mmhh, and also molecule-delegated21:46
rfolcopanda, hmm ok, just like staging job ?21:46
pandarfolco: possibly, yes, but you should include also the molecule files themselves21:48
rfolcopanda, ok will do it later tonight21:49
pandarfolco: you're working on the 2-components integration test now ?21:49
rfolcopanda, yes, and on the improvements for the molecule one21:49
rfolcopanda, am I blocking you somehow ?21:50
pandarfolco: not at all21:51
pandarfolco: keep calm and carry on21:51
rfolcopanda, cool, let it go21:52
rlandypanda: go4it22:02
rlandyno point in multiple patch sets22:03
rlandy2019-10-03 21:44:12.960456 | rdo-centos-7 | ci-scripts/dlrnapi_promoter/tests/staging-setup/test_staging_environment.py:98:1: E302 expected 2 blank lines, found 122:04
rlandy2019-10-03 21:44:12.960527 | rdo-centos-7 | @pytest.mark.serial22:04
rlandypanda: ^^ just fyi - your patch has a linter failure22:04
rlandylucky you :(22:04
pandarlandy: already fixed22:05
pandarlandy: and I've now push new PS in your change22:06
pandapushed*22:06
rlandypanda: thanks22:08
rlandynova jobs are gone22:09
pandathey went nova.22:09
rlandyit's some sort of a zuul hitch22:09
rlandythey show up everywhere22:09
rlandyand then gather themselves back to the nova homeland22:10
pandarlandy: py27 and py36 passes, when jobs complete, I'm going to merge22:10
rlandypanda: ack22:10
rlandyI will put the log parser and clean up changes on top of whatever merges22:11
rlandymakes life a bit easier22:11
weshaypanda, rlandy  I have something that will make our lives a little easier.. haven't tested it.. and I'm probably doing it wrong.. but you'll get the drift22:11
weshaypatch in a few22:11
rlandyamazing22:11
rlandylove living on the edge22:11
rlandyhaven't tested it.. and I'm probably doing it wrong..22:12
pandarlandy: when marios' work merges tomorrow (morning I hope) you'll have your logs and stuff to gnaw.22:13
rlandyok22:13
rlandyreally I can test it off the real logs22:13
rlandythen we should see the test kick on playbooks/staging22:14
*** ksambor has quit IRC22:16
weshayrlandy, panda something like https://review.rdoproject.org/r/2289222:17
pandaweshay: as I said, once we have the tests in place, we can turn the logging inside out. we can even put an audio library to hear a fanfare22:19
weshaylolz :)22:19
weshaytotally fine.. just wanted to plat a seed and watch it grow :)22:19
pandaweshay: or a give with you dancing22:19
pandagif*22:19
weshaypanda, aye.. you see the point though of providing the link to the dlrn_api html report22:20
weshayas I wait for queens to promote22:20
rlandy logger.info('\0/ \0/ \0/ woot!!!')22:22
rlandycute22:22
rlandydecent idea22:23
rlandyhttps://review.rdoproject.org/r/#/c/22804/ aya22:23
rlandyyay22:23
pandaweshay: we already have the information of what passed and what failed. We can put it in the log together with that link for further investigation22:26
weshaypanda, it's much easier to see the entire history of the runs on a hash22:26
weshaypanda, we don't provide that22:26
weshaypanda, for example22:26
weshayhttps://trunk-primary.rdoproject.org/api-centos-master-uc/api/civotes_detail.html?commit_hash=b53ec95f1f048d235cf9ad0d9cc5d672cb9fa24a&distro_hash=3ec6d70a4dff9654f09b0ae697ce9714fa9aae7222:26
weshaypanda, please add that feature at the appropriate time :)22:27
pandaweshay: we could in the same logs, we have all the infor needed, and we could even use a nice table format for that. Anyway, ywah food for thoughts.22:28
weshaypanda, dude22:28
weshayif you could render it as a link.. that would cool.. and no22:28
weshaydon't render it as a table..22:28
weshayit still won't be the entire history22:29
weshaypanda, do what I'm asking please22:29
rlandypanda++22:30
rlandylooking better on gates22:30
* rlandy has to go ... back later22:31
*** rlandy is now known as rlandy|bbl22:31
pandaweshay: when I'm finished with the promoter you won't even recognize it. It will even solve problems automatically and make the coffee while you relax.22:32
weshaypanda, that's also not what I want to hear22:32
weshayI want an html link.. that's it22:32
pandaweshay: you're still listening to me ? I'm dropping on the keyboard ... the long night of the merges has ended ... and we finally merged everything. Tomorrow will be a big day, we'll probably have the promotion job running end to end finally with all the components.22:34
weshaythat is very cool!!22:34
weshay\0/22:34
weshaywell done :)22:34
pandacompletionist out. Peace.22:35
weshay:)22:35
*** panda is now known as panda|off22:35
recheck[rdo-blog] RDO is ready to ride the wave of CentOS Stream → https://blogs.rdoproject.org/2019/10/rdo-centos-stream/22:50
weshaysay what???22:50
*** jtomasek has quit IRC23:07
*** holser has quit IRC23:08
*** tosky has quit IRC23:23
*** jtomasek has joined #oooq23:41
*** jtomasek_ has joined #oooq23:52
*** jtomasek_ has quit IRC23:54
*** jtomasek has quit IRC23:54
*** jtomasek has joined #oooq23:54

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