Monday, 2018-12-10

hubbot1FAILING CHECK JOBS on master: tripleo-ci-centos-7-scenario008-multinode-oooq-container, tripleo-ci-fedora-28-standalone @ https://review.openstack.org/604298, master: tripleo-ci-centos-7-scenario009-multinode-oooq, tripleo-ci-centos-7-scenario008-multinode-oooq-container, tripleo-ci-fedora-28-standalone @ https://review.openstack.org/560445, stable/ocata: tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001 @  (1 more message)01:59
*** agopi-pto has joined #oooq02:28
*** agopi-pto is now known as agopi02:29
*** udesale has joined #oooq03:40
hubbot1FAILING CHECK JOBS on master: tripleo-ci-centos-7-scenario008-multinode-oooq-container, tripleo-ci-fedora-28-standalone @ https://review.openstack.org/604298, stable/pike: tripleo-ci-centos-7-scenario004-multinode-oooq-container, tripleo-ci-centos-7-scenario003-multinode-oooq-container @ https://review.openstack.org/602248, master: tripleo-ci-centos-7-scenario009-multinode-oooq, tripleo-ci-centos-7-scenario008 (1 more message)03:59
*** ykarel has joined #oooq04:40
*** chkumar|off is now known as chandan_kumar04:54
*** ykarel_ has joined #oooq04:59
*** ykarel has quit IRC05:02
*** ratailor has joined #oooq05:56
hubbot1FAILING CHECK JOBS on master: tripleo-ci-centos-7-scenario008-multinode-oooq-container, tripleo-ci-fedora-28-standalone @ https://review.openstack.org/604298, stable/pike: tripleo-ci-centos-7-scenario004-multinode-oooq-container, tripleo-ci-centos-7-scenario003-multinode-oooq-container @ https://review.openstack.org/602248, master: tripleo-ci-centos-7-scenario009-multinode-oooq, tripleo-ci-centos-7-scenario008 (1 more message)05:59
*** saneax has joined #oooq06:04
*** jbadiapa has joined #oooq06:08
*** ykarel_ has quit IRC06:30
*** jfrancoa has joined #oooq07:01
*** quiquell has joined #oooq07:13
quiquellGood morning07:13
*** apetrich has joined #oooq07:24
*** udesale has quit IRC07:24
*** kopecmartin|off is now known as kopecmartin07:31
quiquellkopecmartin: Good morning07:38
quiquellkopecmartin: added story here https://review.openstack.org/#/c/622343/07:38
kopecmartinquiquell, good morning, thanks07:40
*** ykarel_ has joined #oooq07:48
*** quiquell is now known as quiquell|brb07:51
*** gkadam has joined #oooq07:52
*** ccamacho has joined #oooq07:55
hubbot1FAILING CHECK JOBS on master: tripleo-ci-centos-7-scenario008-multinode-oooq-container, tripleo-ci-fedora-28-standalone @ https://review.openstack.org/604298, stable/pike: tripleo-ci-centos-7-scenario004-multinode-oooq-container, tripleo-ci-centos-7-scenario003-multinode-oooq-container @ https://review.openstack.org/602248, master: tripleo-ci-centos-7-scenario009-multinode-oooq, tripleo-ci-centos-7-scenario008 (1 more message)08:00
*** quiquell|brb is now known as quiquell08:08
*** amoralej|off is now known as amoralej08:11
*** ykarel_ is now known as ykarel08:16
*** ykarel is now known as ykarel|lunch08:17
quiquellchandan_kumar, kopecmartin: Can we workflow https://review.openstack.org/#/c/622343/  ?08:21
quiquellIs the last piece missing to have fedora28 tripleo jobn with tempest08:22
quiquellLater issues can be fixed with launchpad08:22
kopecmartinquiquell, sure, once check jobs pass, we'll workflow it08:24
quiquellkopecmartin: cool thanks08:25
quiquellmarios_|ruck: Good morning08:31
*** ykarel|lunch is now known as ykarel08:42
*** tosky has joined #oooq08:45
marioso/ quiquell08:46
quiquellmarios: What is the state of stdln scenarios ?08:47
marios_|ruckquiquell: scen 1 was green on friday :) ready to merge, 4 was on tempest. but the queue was massive, something must have happened on weekend08:48
marios_|ruckquiquell: 2/3 also ready afaik but don't know if merged checking08:49
*** udesale has joined #oooq08:49
quiquellmarios_|ruck: I need to fix this https://review.openstack.org/#/c/621517/ so we can switch to them from multinode08:51
*** d0ugal has quit IRC08:55
marios_|ruckquiquell: ack will check in sec08:56
*** d0ugal has joined #oooq08:56
*** holser_ has joined #oooq08:57
*** jtomasek has joined #oooq08:57
*** rascasoft has joined #oooq08:59
*** apetrich has quit IRC09:13
*** apetrich has joined #oooq09:14
*** apetrich has quit IRC09:19
quiquellmarios_|ruck: I see scenario002 passing alrgith09:30
quiquellmarios_|ruck: rfolco moving to standalone-minimal so it's good to merge09:31
quiquellsshnaidm: Can you workflow https://review.openstack.org/#/c/618709/  ?09:31
*** derekh has joined #oooq09:31
*** chem has joined #oooq09:32
marios_|ruckquiquell: also scen 1 green with "full" standalone tempest  (no overrides) https://review.openstack.org/#/c/619508/09:32
marios_|ruckquiquell: but do we want to depends-on your patch?09:32
*** apetrich has joined #oooq09:32
marios_|ruckquiquell: cus we might see some fails. well they're non voting i gues..09:32
quiquellwhat patch ?09:32
marios_|ruckchandan_kumar: thanks for help with scen 4 will update in a bit with your comment09:33
quiquellmarios_|ruck: Don't we have to have specific scenario001 tempest stuff ?09:33
marios_|ruckquiquell: (we had a nit on the env file in ceph so new depends-on in scens 1/4)09:33
marios_|ruckquiquell: but it merged at https://review.openstack.org/#/c/623217/09:34
quiquellmarios_|ruck: It's merged09:34
marios_|ruckquiquell: yeah just cos you were out /update/fyi you asked earlier :)09:34
quiquellmarios_|ruck: ack, and about featureset_override with tempest stuff for sceanrio001 why are we missing it ?09:34
quiquellor we don't need it ?09:34
marios_|ruckquiquell: so, for tempest, i see it runs scen 1 like https://url.corp.redhat.com/scen1-pass-tempest09:34
arxcruzmarios_|ruck: quiquell https://review.openstack.org/622271 quickly one, doc change :)09:35
marios_|ruckquiquell: sec, the env file is like09:35
quiquellmarios_|ruck: I think we are missing something like this https://github.com/openstack/tripleo-quickstart/blob/master/config/general_config/featureset016.yml#L136-L16109:36
quiquellmarios_|ruck: at standalone sceanrio00109:36
quiquellarxcruz: ^ is this correct ?09:36
marios_|ruckquiquell: ack it is running basic tests by default but i just saw panko at https://github.com/openstack/tripleo-heat-templates/blob/master/ci/environments/scenario001-standalone.yaml09:36
marios_|ruckquiquell: sounds right, at least is similar to what chandan_kumar pointed to for scen 409:36
arxcruzquiquell: what you mean if is correct or not ?09:37
marios_|ruckquiquell: so we run base tests plus whatever is in the env file09:37
marios_|ruckarxcruz: if that's what we should do for scenario1 standalone09:37
quiquellarxcruz: +209:37
arxcruzmarios_|ruck: yes it is correct, and you can remove newton09:37
arxcruzquiquell: gracias09:38
marios_|ruckthanks quiquell will update in a bit then f09:38
quiquellmarios_|ruck: will -1 it with the poiunt to multinode scenario001 is ok ?09:38
marios_|ruckfor bot scen1/4 (well 4 already has it like https://review.openstack.org/#/c/619520/16/zuul.d/standalone-jobs.yaml09:38
marios_|ruckquiquell: ^09:38
marios_|ruckquiquell: ack09:38
quiquellmarios_|ruck: we have to find what they activate for sceanrio004 multinode tempest09:38
marios_|ruckquiquell: i did well chandan_kumar did and pointed for me thats how i updated it09:39
marios_|ruckquiquell: i'm just missing to add the white_regex apparently09:39
ssbarnea|rover marios_|ruck hi!  i am rebasing the centod76 CR as it needed it09:40
arxcruzmarios_|ruck: quiquell what's the review?09:40
quiquellmarios_|ruck: Ahh I see scenario004 is correct09:40
quiquellssbarnea|rover: o/ mate09:41
marios_|ruckthanks arxcruz09:41
quiquellarxcruz: https://review.openstack.org/#/c/61950809:41
marios_|ruckarxcruz: here https://review.openstack.org/#/c/619508/ https://review.openstack.org/61952009:41
marios_|ruckarxcruz: (scen1, scen4)09:41
marios_|ruckssbarnea|rover: o/09:42
quiquellmarios_|ruck: scenario003 is missing the chandam comment too09:43
*** vinaykns has joined #oooq09:44
arxcruzmarios_|ruck: quiquell yeah, if this scenario will execute telemetry tests need to change the alarm granularity otherwise the test will fail, this happens because the test runs faster than default telemetry granularity can run09:44
marios_|ruckarxcruz: thanks please add a comment09:45
arxcruzmarios_|ruck: but scenario001 are testing telemetry ?09:46
quiquellarxcruz: do we have to add the test_white_regex: "" thingy ?09:46
quiquellarxcruz: to all of them ?09:46
quiquellchandan_kumar: ^ ¿09:47
quiquellarxcruz: how important is this ? http://logs.openstack.org/56/618056/21/check/tripleo-ci-fedora-28-standalone/b253aa8/job-output.txt.gz#_2018-12-06_23_21_53_01014609:51
arxcruzquiquell: very important09:51
quiquellarxcruz: FUCK09:51
quiquellups09:52
quiquellDidn't want to do capital09:52
arxcruzthe testrepository.subunit is how openstack-health collect the data from our jobs09:52
quiquell_fuck_09:52
quiquellarxcruz: and this ? http://logs.openstack.org/56/618056/21/check/tripleo-ci-fedora-28-standalone/b253aa8/job-output.txt.gz#_2018-12-06_23_21_57_20563009:52
arxcruzif there's no testrepository.subunit in the root of logs.o.o o-health doesn't generate the data09:52
arxcruzquiquell: let me check09:52
quiquellarxcruz: Cool thanks09:53
quiquellarxcruz: only html generation is working I think09:53
quiquellarxcruz: the junit is for jenkins, isn't it ?09:54
arxcruzquiquell: i don't care, but i think some guys on downstream need that09:54
arxcruzthis is to jenkins report the tests09:54
arxcruzwe don't use jenkins anymore09:54
quiquellarxcruz: maybe we can install subunit from pip there ?09:54
quiquellarxcruz: pip version has all the goodies09:54
arxcruzquiquell: sure, i don't mind09:54
marios_|ruckhttps://github.com/openstack/tripleo-heat-templates/blob/master/ci/environments/scenario001-standalone.yaml09:56
marios_|ruckarxcruz: yeah here ^09:56
quiquellarxcruz: Looks like they have fix the package, they just need to merge the pull request https://src.fedoraproject.org/rpms/subunit/pull-request/209:57
arxcruzmarios_|ruck: so yeah, need to do all that mobo jambo09:57
arxcruzomg, they weren't using gobject introspection09:58
arxcruzit's like, years ago09:58
quiquellarxcruz: what do you mean ?09:58
arxcruzquiquell: https://github.com/testing-cabal/subunit/pull/34/commits/5ba871c0851dbff28320ec5887960a90a7aab26009:58
marios_|ruckhaha thanks arxcruz09:58
arxcruzthey were using old python bindings09:58
quiquellarxcruz: that's very old shit ?09:59
arxcruzsince Gtk 3 python bindings were using gobject-introspection09:59
arxcruzand gtk 3 is here for a few years already09:59
quiquellarxcruz: So have being like that until someone open the carpet ?09:59
hubbot1FAILING CHECK JOBS on master: tripleo-ci-centos-7-scenario008-multinode-oooq-container, tripleo-ci-fedora-28-standalone @ https://review.openstack.org/604298, stable/pike: tripleo-ci-centos-7-scenario003-multinode-oooq-container, tripleo-ci-centos-7-scenario004-multinode-oooq-container @ https://review.openstack.org/602248, master: tripleo-ci-centos-7-scenario009-multinode-oooq, tripleo-ci-centos-7-scenario008 (1 more message)10:00
arxcruzquiquell: seems so10:02
quiquellarxcruz: question, do you know why do we override tempest_plugins here ?10:08
quiquellarxcruz: I think you also asked it10:08
quiquellarxcruz: I suppose we have to run standard tempest stuff from featureset05210:08
arxcruzquiquell: well, if we are using ond versions of openstack, the plugins name were different10:08
quiquellarxcruz: but this is the same for normal standalone10:09
quiquellarxcruz: nothing special at fedora28 maybe that we don't have this package at RDO or something ?10:09
quiquellmarios_|ruck: Can we workflow this https://review.openstack.org/#/c/619488/ ?10:14
arxcruzquiquell: where?10:15
quiquellarxcruz: https://review.openstack.org/#/c/618056/10:16
arxcruzquiquell: so10:18
arxcruzquiquell: what this fedora28 do ?10:18
*** udesale has quit IRC10:18
arxcruzinstall only one service, or it's a normal deployment, with compute node and controller service also installed ?10:18
quiquellarxcruz: we just have to do same as tripleo-ci-centos-7-standalone but for fedora2810:19
quiquellarxcruz: So I don't understand why do we need a featureset_override there10:19
arxcruzquiquell: so, if you have compute node and controller also installed, no need for it10:19
arxcruz:)10:19
arxcruzquiquell: was just a question, since i don't know too much about it, nevertherless i +1 :)10:20
quiquellarxcruz: ahh is the python3 stuff, but this is already fixed in the Depends-On10:20
quiquellarxcruz: thanks mate !10:20
arxcruzquiquell: i did nothing :P10:21
chandan_kumarquiquell: marios_|ruck test_white_regex is needed for all scenario jobs as there we are running specific tempest tests10:33
quiquellchandan_kumar: so we havce to add test_white_regex: "" ?10:33
chandan_kumarquiquell: test_white_regex: ""10:34
quiquellchandan_kumar: ack10:34
arxcruzchandan_kumar: quiquell so we can add it on the parent job10:35
arxcruzactually, what's the default ?10:35
chandan_kumarquiquell: https://github.com/openstack/tripleo-quickstart/blob/master/config/general_config/featureset052.yml#L3010:35
quiquellarxcruz: this will not work10:35
chandan_kumararxcruz: it is smoke currently10:35
quiquellarxcruz: as teh var is featureset_override it will override all of them10:35
quiquellarxcruz: base values will not work there10:35
chandan_kumararxcruz: we donot need to add anything in parent job10:35
arxcruzso, if we don't have a tempest_whitelist it will use the test_white_regex10:36
chandan_kumararxcruz: yes10:36
arxcruzso, on fedora28 case, it will run smoke tests10:36
arxcruzwhich i think it's okay10:36
quiquellchandan_kumar, arxcruz: this is good now ? https://review.openstack.org/61870910:37
quiquellchandan_kumar, arxcruz: standalone + scenario00210:37
chandan_kumarquiquell: perfect10:37
quiquellmarios_|ruck: going to update sceanrio003 with it10:39
quiquellchandan_kumar: and sceanrio003 ? https://review.openstack.org/#/c/61876110:41
quiquellmarios_|ruck: Can you put back the +10:41
quiquellmarios_|ruck: +2 at sceanrio003 ? https://review.openstack.org/#/c/618761 so we can workflow it10:41
chandan_kumarmarios_|ruck: can we update this one https://review.openstack.org/#/c/619520/ also then we are done with scenario?10:45
quiquell chandan_kumar sceanrio001 is ok ?10:45
chandan_kumarquiquell: review link?10:46
quiquellchandan_kumar: 200~https://review.openstack.org/#/c/619508/10:46
chandan_kumarquiquell: need some work10:47
quiquellchandan_kumar: Ahh yep this is the one missing the whole tempest stuff ok10:48
chandan_kumarquiquell: lot of things10:48
chandan_kumarquiquell: commenting on it10:48
chandan_kumarquiquell: donbe10:51
chandan_kumarquiquell: we need tempest_workers vars in zuulv3 so that we can use it as a overrides10:51
chandan_kumarlet me puch a patch10:51
quiquellchandan_kumar: cool thanks I think is the last stuff missing10:52
quiquellchandan_kumar: To test python3 with tempestconf without hackind around pip and virtualenv we need something like this https://github.com/ansible/ansible/issues/6345#issuecomment-3712946010:58
quiquellchandan_kumar: we cannot change ansible_python_interpreter from the zuul job10:58
chandan_kumarquiquell: oh, yes, letme update the patch11:01
chandan_kumarquiquell: marios_|ruck arxcruz https://review.openstack.org/#/c/624051/11:01
arxcruzchandan_kumar: why? it's set in validate-tempest to ncpu/2 already11:02
arxcruzunless there's a specific case11:02
quiquellchandan_kumar: humm wait, maybe is enough just running at fedora28 those jobs, since it's a python3 only thingy11:02
chandan_kumararxcruz: https://github.com/openstack/tripleo-quickstart/blob/master/config/general_config/featureset016.yml#L16511:02
arxcruzok11:02
chandan_kumararxcruz: it is needed for scenario001 otherwise telemetry tests will screw up11:03
*** udesale has joined #oooq11:11
*** sshnaidm has quit IRC11:15
*** sshnaidm has joined #oooq11:16
*** dtantsur|afk is now known as dtantsur11:25
chandan_kumarkopecmartin: left few comment on profile patch11:31
*** jtomasek has quit IRC11:34
rfolcochandan_kumar, this does not sound coherent to me: test_white_regex="" and test_white_list has 3 tests...11:34
rfolcohttps://review.openstack.org/#/c/618761/15/zuul.d/standalone-jobs.yaml11:34
chandan_kumarrfolco: it is under tempest_whitelist11:41
chandan_kumarrfolco: the main reason behind keeping test_white_regex to '' in order to override these tests https://github.com/openstack/tripleo-quickstart/blob/master/config/general_config/featureset052.yml#L3011:43
chandan_kumarso that we should not run the same test multiple times there in order to increase the timing11:43
rfolcochandan_kumar, I left my comment there. I believe if you specify tempest_whitelist, you should not need to set test_white_regex too11:44
rfolcodoes not make sense to me... if a white list is given, means I want to run only the ones I add there11:44
chandan_kumarrfolco: yes but right now we have both options to run tests,11:45
chandan_kumarrfolco:if we have single test we can use test_white_regex but if we have multiple tests we can go with tempest_whitelist11:47
sshnaidmdoes anyone know how to see all tasks assigned to you in taiga?11:47
sshnaidmrfolco, panda ^^11:47
rfolcosshnaidm, I don't know. I was playing with taiga-python api to extract some queries like this11:47
rfolcobut still don't have anything to share11:48
sshnaidmweird, so necessary thing and not shown somewhere in interface11:48
rfolcosshnaidm, agreed11:48
rfolcosshnaidm, looks like api is the only good way... https://github.com/taigaio/taiga-front/issues/137911:50
rfolcosshnaidm, I started playing with taiga-python, but my queries were not retrieving any data, auth is ok, it was taking forever to return anything, got stuck there.11:54
sshnaidmrfolco, from you link - need to go to dashboard (it's none squares icon) and you have there "Working on" section11:54
sshnaidmrfolco, it's kind of mess, but at least my tasks are there11:55
*** quiquell is now known as quiquell|brb11:55
rfolcosshnaidm, that one includes everything, right.11:56
hubbot1FAILING CHECK JOBS on master: tripleo-ci-centos-7-scenario008-multinode-oooq-container, tripleo-ci-fedora-28-standalone @ https://review.openstack.org/604298, master: tripleo-ci-centos-7-scenario009-multinode-oooq, tripleo-ci-centos-7-scenario008-multinode-oooq-container, tripleo-ci-fedora-28-standalone @ https://review.openstack.org/560445, stable/ocata: tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001 @  (1 more message)12:00
kopecmartinchandan_kumar, any idea, what's wrong with packstack? http://logs.openstack.org/43/622343/11/check/python-tempestconf-tempest-packstack-admin/710381f/job-output.txt.gz#_2018-12-10_08_38_29_00630012:05
chandan_kumarssbarnea|rover: arxcruz kopecmartin need some help here http://logs.openstack.org/65/622865/9/check/python-tempestconf-tempest-devstack-py3/aebd972/job-output.txt.gz#_2018-12-10_11_15_37_41162312:05
chandan_kumarpatch is this https://review.openstack.org/62286512:05
chandan_kumarkopecmartin: checking here12:06
arxcruzchandan_kumar: from the ansible doc:12:10
arxcruzAn optional path to a virtualenv directory to install into. It cannot be specified together with the 'executable' parameter (added in 2.1). If the virtualenv does not exist, it will be created before installing packages12:10
arxcruzso, you can't use virtualenv and executable parameter together12:10
*** quiquell|brb is now known as quiquell12:10
chandan_kumarkopecmartin: so better to go with versin there?12:11
chandan_kumararxcruz: ^^12:11
arxcruzchandan_kumar: i think if you use virtualenv_command instead of executable is enough, because virtualenv_command will point to python312:12
arxcruzchandan_kumar: other option is run ansible on python312:12
pandasshnaidm: in the taskboard, upper right corner, an icon with a funnel, the filter menu will open, choose ASSIGNED TO and your user12:12
pandasshnaidm: in the backlog, SHOW FILTER just above the user stories, then same thing12:14
*** jtomasek has joined #oooq12:17
sshnaidmpanda, thanks12:19
ssbarnea|roverchandan_kumar: what is the problem there? i find it self explanatory.12:19
ssbarnea|roverarxcruz: is right: if you use virtualenv it does make no sense to mention a python executable, as this is defined by the venv itself.12:20
chandan_kumarkopecmartin: https://review.openstack.org/#/c/624068/12:25
chandan_kumarssbarnea|rover: arxcruz done, let see how it goes12:31
arxcruzchandan_kumar: ack12:32
arxcruzdo you guys have a good mechanical keyboard to indicate ?12:32
quiquellarxcruz: I bough a corsair, quite nice12:33
chandan_kumararxcruz: what about DAS keyboard?12:33
quiquellarxcruz: It wakes up all my neighbours in the morning :-)12:33
arxcruzlol12:33
arxcruzchandan_kumar: nope, didn't like, i want a small one, no need the num keys12:34
arxcruzso, i want one wireless12:34
ssbarnea|roverarxcruz: avoid apple magic keyboard like plague, i am also looking for an alternative kb. blame apple for all my typos12:34
quiquellsshnaidm: o/12:35
arxcruzssbarnea|rover: i have a logitech k81012:35
sshnaidmquiquell, hey12:35
quiquellsshnaidm: How is repro doing, do we merge it ?12:35
arxcruzwhich is great, i love it, but it's 6 years old, and it's already saying it's dieing12:36
sshnaidmquiquell, let's review it before again and then merge it12:36
quiquellsshnaidm: acl12:36
quiquellack12:36
sshnaidmquiquell, I think rlandy should review too12:36
quiquellsshnaidm: ok, let's wait for her review12:37
*** ratailor has quit IRC12:40
ssbarnea|rovermarios_|ruck: centos76 failed on standalone, see http://logs.openstack.org/80/618780/10/check/tripleo-ci-centos-7-nextrelease-standalone/8019d60/logs/undercloud/home/zuul/install_packages.sh.log.txt.gz#_2018-12-10_10_16_3412:41
ssbarnea|roverdoes this error ring a bell for you?12:41
ssbarnea|roveror maybe better to ask on #rdo channel? these rpms have different versions on that repository12:42
marios_|ruckssbarnea|rover: looking12:43
marios_|ruckssbarnea|rover: not seen this one but is it transient (have you logstrashed it yet? me looks)12:45
marios_|ruck2018-12-10 10:16:34 | http://mirror.gra1.ovh.openstack.org:8080/rdo/centos7/ce/73/ce73fdfae2f5af0f5599708c84c44b6911595ce7_9b470851/openstack-tripleo-common-10.1.1-0.20181207042338.5168b91.el7.noarch.rpm: [Errno 14] HTTP Error 404 - Not Found12:45
marios_|ruck2018-12-10 10:16:34 | Trying other mirror.12:45
*** tosky has quit IRC12:47
marios_|ruckhttp://logstash.openstack.org/#/dashboard/file/logstash.json?query=message:%5C%22no%20more%20mirrors%20to%20try%5C%2212:48
marios_|ruckssbarnea|rover: ^12:48
marios_|ruckssbarnea|rover: make it 2 days12:48
quiquellchandan_kumar: This change a python-tempestconf feels unrelated to the failures https://review.openstack.org/#/c/622343/12:52
quiquellarxcruz: ^ ?12:52
chandan_kumarquiquell: i have fix up for packstack failure https://review.openstack.org/62406812:53
quiquellchandan_kumar: and devstack ?12:54
chandan_kumarquiquell: devstack one I am checking12:54
chandan_kumarquiquell: http://logs.openstack.org/43/622343/11/check/python-tempestconf-tempest-devstack-demo/84cb5f4/job-output.txt.gz#_2018-12-10_08_53_36_60872112:56
chandan_kumarquiquell: so member role is not there, and demo user does not have permission to create that12:56
chandan_kumarquiquell: i think we use to create that12:57
weshaymarios_|ruck, ssbarnea|rover thanks for holding down the fort, anything I can help w/?12:57
quiquellchandan_kumar: does not feel related to the python3 changes from the review12:57
chandan_kumarquiquell: not related to your patch12:58
*** rlandy has joined #oooq12:58
marios_|ruckweshay: o/ so what happened with the zuul queue was there reset12:59
marios_|ruckweshay: nothing major atm, some updates on the etherpad. ssbarnea|rover just looking into a new issue with mirrors like http://logstash.openstack.org/#/dashboard/file/logstash.json?query=message:%5C%22no%20more%20mirrors%20to%20try%5C%2212:59
weshaymarios_|ruck, unsure, but it was merging a lot of changes on friday13:00
weshaymarios_|ruck, k13:00
marios_|ruckoops scrum time13:00
rlandypanda: need to chat with you for a  few minutes after the meeting re; hash tags to use in f2813:12
pandarlandy: ok13:14
*** jtomasek_ has joined #oooq13:14
chandan_kumarquiquell: rfolco best solution is move this https://github.com/openstack/tripleo-quickstart/blob/master/config/general_config/featureset052.yml#L30 to tempest_whitelist13:15
*** jtomasek has quit IRC13:15
chandan_kumarjob is done13:15
quiquellchandan_kumar: you mean put all the tests from the regex to the tempest_whitelist ?13:16
chandan_kumarquiquell: yes13:18
chandan_kumarquiquell: let me push the patch13:18
quiquellchandan_kumar: cool13:18
marios_|ruckssbarnea|rover: do we have any ruck/rover calls today /me thought there was ci escalation today13:19
*** jtomasek_ is now known as jtomasek13:19
kopecmartinchandan_kumar, the change you just proposed to python-tempestconf (Create Member role for demo user) .. is it supposed to fix this issue? http://logs.openstack.org/67/621567/7/check/python-tempestconf-tempest-devstack-demo/8557471/job-output.txt.gz#_2018-12-10_11_01_45_86264513:21
chandan_kumarkopecmartin: yes13:21
chandan_kumarkopecmartin: or Am i missing something?13:21
kopecmartinchandan_kumar, great, thanks, please add some description to the commit message13:22
marios_|ruckssbarnea|rover: yah it was canceled13:23
*** amoralej is now known as amoralej|lunch13:23
ssbarnea|rovermarios_|ruck: i was cheching the same as i got a popup but there was no meeting.13:24
*** saneax has quit IRC13:27
chandan_kumarquiquell: marios_|ruck rfolco https://review.openstack.org/62408113:28
quiquellchandan_kumar: Cool so with this we don't need the regex stuff ?13:33
chandan_kumarquiquell: yes we donot need test_white_Regex stuff but I should wait to see the jobs13:33
chandan_kumarresults13:34
*** dtrainor has quit IRC13:34
ssbarnea|rovermarios: please read https://bugs.launchpad.net/tripleo/+bug/1807703 and tell me if I am wrong or not.13:48
openstackLaunchpad bug 1807703 in tripleo "[WARNING] Ansible is in a world writable directory" [Undecided,New]13:48
weshayquiquell, add a skeleton update of those 3 issues to the board before you go13:49
*** dtrainor has joined #oooq13:59
hubbot1FAILING CHECK JOBS on master: tripleo-ci-centos-7-scenario008-multinode-oooq-container, tripleo-ci-fedora-28-standalone @ https://review.openstack.org/604298, master: tripleo-ci-centos-7-scenario009-multinode-oooq, tripleo-ci-centos-7-scenario008-multinode-oooq-container, tripleo-ci-fedora-28-standalone @ https://review.openstack.org/560445, stable/ocata: tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001 @  (1 more message)14:00
ssbarnea|roverweshay: ^^ if you can have a look at https://review.rdoproject.org/etherpad/p/ruckrover-sprint23#26 it would be great. I am not sure what is causing ti.14:02
marios_|ruckssbarnea|rover: checking (in scrum )14:08
marios_|ruckssbarnea|rover: should i file the bug for http://logs.openstack.org/80/618780/10/check/tripleo-ci-centos-7-nextrelease-standalone/8019d60/logs/undercloud/home/zuul/install_packages.sh.log.txt.gz#_2018-12-10_10_16_3414:12
ssbarnea|rovermarios_|ruck: yep, this was my question as I didn't know what caused it. i guess it does not hurt to do so.14:12
marios_|ruckssbarnea|rover: ack will do in a bit then (after this call, have another but at least its noted on the etherpad for now thanks)14:13
*** vinaykns has quit IRC14:14
*** openstackstatus has joined #oooq14:17
*** ChanServ sets mode: +v openstackstatus14:17
*** amoralej|lunch is now known as amoralej14:19
weshaypanda, https://bluejeans.com/u/whayutin/14:22
weshaypanda, 5 seconds14:22
chandan_kumarquiquell: marios_|ruck rfolco I think it will fix https://review.openstack.org/624081 white regex issue14:23
chandan_kumarotherwise I will take a look tomororw14:23
weshaypanda, ?14:24
rfolcochandan_kumar, will take a look, first impression is that one is an workaround, not a fix :)14:24
*** gkadam has quit IRC14:24
chandan_kumarrfolco: i think i have added the info in commit message for both cases14:25
chandan_kumar:-)14:25
chandan_kumarsee ya guys14:25
rfolcothx chandan_kumar14:25
*** chandan_kumar is now known as chkumar|off14:25
rfolcohave a good one14:25
quiquellweshay, panda : will send you email with links to cards in the dashboard regarding f28 main issues14:26
*** quiquell is now known as quiquell|off14:30
marios_|ruckssbarnea|rover: there?14:56
marios_|ruckssbarnea|rover: i just noticed that no more mirrors issue is for the centos 7.6 job, so wondering do we still want the bug for upstream ci (that job is not visible yet?) added in https://review.openstack.org/#/c/618780/9/zuul.d/standalone-jobs.yaml14:57
ssbarnea|rovermarios_|ruck: no need to file bug, I did a recheck. jpena told me to recheck as there were sync issues...14:58
marios_|ruckssbarnea|rover: can't see it in current jobs? (so maybe leave it on the etherpad for now, added pointer to the review)14:58
marios_|ruckssbarnea|rover: i was trying to find more examples for the bug14:58
marios_|ruckssbarnea|rover: ack then14:58
ssbarnea|roverwe will file bug if it happens again, already in etherpad14:58
ssbarnea|rovermarios_|ruck: see https://review.rdoproject.org/r/#/c/17732/ -- i am curious about possible side effects...15:00
rlandypanda: pls ping me when you are done15:02
marios_|ruckack in call ssbarnea|rover will do in a bit15:14
*** holser_ has quit IRC15:38
*** holser__ has joined #oooq15:39
hubbot1FAILING CHECK JOBS on master: tripleo-ci-centos-7-scenario008-multinode-oooq-container, tripleo-ci-fedora-28-standalone @ https://review.openstack.org/604298, master: tripleo-ci-centos-7-scenario009-multinode-oooq, tripleo-ci-centos-7-scenario008-multinode-oooq-container, tripleo-ci-fedora-28-standalone @ https://review.openstack.org/560445, stable/ocata: tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001 @  (1 more message)16:00
weshayrfolco, do you want to chat about scen000 / updates16:01
weshayto clarify16:01
rfolcoweshay, your bj?16:01
weshaysure16:02
rfolcoweshay, connection frozen16:04
weshayrfolco, https://logs.rdoproject.org/openstack-periodic/git.openstack.org/openstack-infra/tripleo-ci/master/periodic-tripleo-centos-7-master-promote-consistent-to-tripleo-ci-testing/2411f88/16:10
weshayrfolco, openstack-periodic-python316:10
weshayrlandy, panda please join my bluejeans16:23
weshayto talk about f28 promotions16:23
pandaweshay: now16:24
panda?16:24
weshayya.. if you can16:24
*** holser__ has quit IRC16:26
*** holser_ has joined #oooq16:26
*** chkumar|off has quit IRC16:32
*** jfrancoa has quit IRC16:44
*** ccamacho has quit IRC16:52
*** ykarel has quit IRC16:52
*** ykarel has joined #oooq16:53
*** udesale has quit IRC16:58
*** ykarel is now known as ykarel|away17:03
*** rascasoft has quit IRC17:08
*** kopecmartin is now known as kopecmartin|off17:17
*** dsneddon has quit IRC17:18
*** dsneddon has joined #oooq17:19
*** dtantsur is now known as dtantsur|afk17:21
*** derekh has quit IRC17:29
pandassbarnea|rover: can you point me to the code that understand which distro we are using based on the node name ?17:49
rlandypanda: weshay: periodic-tripleo-ci-centos-7-standalone-master is not even in the current promotion criteria ... https://github.com/rdo-infra/ci-config/blob/master/ci-scripts/dlrnapi_promoter/config/master.ini#L3217:53
rlandydo we want the centos7 standalone job added as well?17:53
sshnaidmrlandy, can you please review https://review.rdoproject.org/r/#/c/17552/ and try the latest version if it works?17:54
rlandyyep - will do later this afternoon17:55
pandarlandy: I didn't see it pass once17:59
hubbot1FAILING CHECK JOBS on master: tripleo-ci-centos-7-scenario008-multinode-oooq-container, tripleo-ci-fedora-28-standalone @ https://review.openstack.org/604298, master: tripleo-ci-centos-7-scenario009-multinode-oooq, tripleo-ci-centos-7-scenario008-multinode-oooq-container, tripleo-ci-fedora-28-standalone @ https://review.openstack.org/560445, stable/ocata: tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001 @  (1 more message)18:00
weshayrlandy, yup.. of course :)18:00
weshaythanks for catching it18:00
pandaweshay: can you point me to the code that understands which distro we are testing based on the node name ?18:04
weshaynot the node name, the running instance18:04
weshayssbarnea|rover, ^ fyi.. /me looks18:04
weshayhttp://logs.openstack.org/60/624060/3/check/tripleo-ci-fedora-28-standalone/f8bfef5/job-output.txt.gz#_2018-12-10_12_56_46_72258818:10
weshay2018-12-10 12:56:47.784871 | primary | +++(/home/zuul/src/git.openstack.org/openstack/tripleo-ci/scripts/oooq_common_functions.sh:180): python_cmd(): distribution=Fedora18:10
pandaok you're using ansible_distribution18:12
weshaythis seems off to me18:13
weshaybut it's around here18:13
weshay local release_file=$LOCAL_WORKING_DIR/config/release/tripleo-ci/${DISTRIBUTION:-CentOS}-${DISTRIBUTION_MAJOR_VERSION:-7}18:13
weshaypanda, toci picks the release config file18:14
rfolcorlandy, the 4th job is f28 standalone ?18:26
rfolcorlandy, tripleo-ci-testing hash, container build, image build, and ?18:27
rfolcoI thik the only one that exists is f28 standalone18:28
rlandyyes18:28
rfolcorlandy, I created my own tasks, sorry if any duplicates18:29
rlandyweshay: panda is right  - the centos-7 standlaone job in periodic has never passed: https://review.rdoproject.org/zuul/builds?job_name=periodic-tripleo-ci-centos-7-standalone-master18:34
rlandy"job_id": "periodic-standalone-featureset052",18:34
rlandywhich would be the same for the fedora-28 job18:34
rlandyw would have to differentiate18:35
rlandypanda: weshay: rfolco: ^^18:35
rfolcoperodic-{fedora28|centos7}-standalone-featureset052 ?18:37
*** amoralej is now known as amoralej|off18:41
*** holser_ has quit IRC18:43
rlandyit picks up a job id which doe s not register nodes18:47
rfolcorlandy, I see this missing in the current job definition: environment_type: standalone18:52
*** ykarel|away has quit IRC19:08
rfolcoweshay, when you say build job for overcloud image, is this one ? https://github.com/openstack-infra/tripleo-ci/blob/master/zuul.d/build-image.yaml#L4719:34
*** ade_lee has joined #oooq19:39
hubbot1FAILING CHECK JOBS on master: tripleo-ci-centos-7-scenario008-multinode-oooq-container, tripleo-ci-fedora-28-standalone @ https://review.openstack.org/604298, master: tripleo-ci-centos-7-scenario009-multinode-oooq, tripleo-ci-centos-7-scenario008-multinode-oooq-container, tripleo-ci-fedora-28-standalone @ https://review.openstack.org/560445, stable/ocata: tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001 @  (1 more message)20:00
rlandyrfolco: it's in the base job20:20
rfolcooh ok rlandy20:21
rlandypanda: still around>20:51
rlandyweshay: around?20:58
weshayrlandy, aye20:58
rlandyweshay: what do we want to do about the job_id in promoter ini files?20:59
weshayrlandy, example id ?20:59
rlandythis woudl cause a clash with standalone centos7 and fedora2820:59
weshaysorry?21:00
rlandyhttps://github.com/rdo-infra/ci-config/blob/master/ci-scripts/dlrnapi_promoter/config/master.ini21:00
weshaythe standalone for f28 has fedora in the name21:00
weshayhow is this a problem?21:00
weshayoh21:00
weshaydang it21:00
rlandyyep21:00
rlandywe got a problem21:01
weshayrlandy, meh.. I think we just end up naming the standalone's to include the dist in rdo21:01
rlandyweshay: k - will talk to panda about it tomorrow21:01
rlandythe name has to match in what the promoter script is expecting21:02
weshayrlandy, seems like we're not going to be able to move the upstream naming21:02
rlandyagain - this is only temp21:02
* weshay wonders if it makes sense to just rename rdo jobs the same as upstream21:02
rlandyin future we can assume one arch21:02
weshayif you can't beat'em21:02
rlandylol21:02
weshayrlandy, ya.. we can assume arch21:02
weshayrlandy, and possible interp21:03
rlandywe have a temp mixed case21:03
rlandywith pure f28, we can guess at the arch21:03
* rlandy leaves notes in the card21:04
weshaystill we'll want the jobs to reflect it21:04
weshaynode-job is a fairly good name21:04
rlandyI think we should use the entire job name now21:09
rlandyleft note for panda21:09
weshay+121:10
*** jtomasek has quit IRC21:32
hubbot1FAILING CHECK JOBS on master: tripleo-ci-centos-7-scenario008-multinode-oooq-container, tripleo-ci-fedora-28-standalone @ https://review.openstack.org/604298, master: tripleo-ci-centos-7-scenario009-multinode-oooq, tripleo-ci-centos-7-scenario008-multinode-oooq-container, tripleo-ci-fedora-28-standalone @ https://review.openstack.org/560445, stable/ocata: tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001 @  (1 more message)22:00
rlandyrfolco: ping re: new tasks/reviews added22:07
rlandyI am not understanding what the goal is here22:07
rlandyiiuc, there should not be more reviews needed for the centos7 containers piece22:07
rlandyonly the f-28 independent pipeline22:08
weshayrfolco, ^ I agree w/ rlandy22:08
rlandyrfolco: I commented on the review22:09
pandarlandy: rfolco weshay  https://review.rdoproject.org/r/17744  and https://review.rdoproject.org/r/1774522:09
rlandybut I thought that review was for the independent pipeline22:09
rlandypanda: hello22:09
rlandypanda: did you see my comment on the card?22:10
rlandywe have a problem with using job_id22:10
rlandywe will clash22:11
rlandyI would like to change that to be the full jobs name22:11
rlandypanda: agree ^^?22:11
pandarfolco: why are you putting card on the mixed pipeline user story ?22:13
pandarlandy: which card ?22:14
rlandypanda: https://tree.taiga.io/project/tripleo-ci-board/task/482?kanban-status=1447278 - pls see comment in reviews section22:14
rlandynoticed the problem while editing the ini file22:15
pandaah the reviews ok22:15
rlandysorry - didn;t know where else to put it22:15
rlandysince I found it working on a review22:15
* rlandy will never get taiga right22:16
pandawe always used job id in the past because infra always told us the names where unreliable, and they could change at any time without warning22:17
pandabut it's been a while since we've run a promotion job in upstream infra22:18
rlandypanda: problem now is that the job id is duplicated22:18
weshaypanda, ha22:18
weshayya22:18
weshaycome on man22:18
rlandyyou know, you can't please all of the people all of the  time22:18
pandawe decide our own destiny, the problem for me is just consistency at this point22:18
* rlandy wants viable promotion criteria22:18
rlandywhich we don't have atm22:19
weshaypanda, ya.. I was saying to rlandy that if you can't beat'em join em22:19
weshayw/ regards to the names22:19
rlandypanda: what do suggest then?22:19
pandain the script that call dlrn api we pass the job id22:19
rlandywe have to include the os now22:19
weshaywhat about the ini file?22:19
rlandyfor mixed promotions22:19
weshaypanda, ya.. but the job_id == to the name22:20
weshayin those jobs22:20
weshaynobody cares what we call the jobs in rdo-cloud really22:20
weshaymy thought is just to fold back to the upstream naming method22:21
weshayas I suspect we'll never change those22:21
weshaywhich afaict.. is tripleo+node+$name22:21
pandawell, when we migrated to zuulv3 all the names changed22:21
pandawe probably won't do the same thing anytime soon again22:22
weshayfrom legacy to $foo?22:22
weshayya.. the cost of gaining consensus is high22:22
pandaand vice-versa22:22
rlandyI don;t see that adding yet another combination will help22:22
pandawe didn't need to change the promotion there at least22:22
* weshay is confused22:23
rlandychange the promotion where?22:23
* rlandy is also confused22:23
panda20 minutes to get this link ..... grrrrrrr https://review.rdoproject.org/r/gitweb?p=config.git;a=blob;f=playbooks/tripleo-ci-periodic-base/post.yaml;h=5df57f21a180d0d4ebcc9741747f40029572e9fb;hb=refs/heads/master22:23
pandaso line 13 or 1522:24
pandahow do you change the name ?22:24
pandayou change the name just for that job ?22:24
rlandywhat is that?22:24
pandaor you change it for all ?22:24
pandarlandy: that is the task + script that report the success to dlrn api22:24
pandarlandy: specifying the job id22:25
rlandyhttps://logs.rdoproject.org/openstack-periodic/git.openstack.org/openstack-infra/tripleo-ci/master/periodic-tripleo-ci-centos-7-standalone-master/9dd4b9a/job-output.txt.gz#_2018-12-02_16_43_27_27246422:25
rlandythat afaict is the job id22:26
rlandywhich would be the same for two standalone jobs22:26
rlandywhat would we add here:22:26
rlandyhttps://github.com/rdo-infra/ci-config/blob/master/ci-scripts/dlrnapi_promoter/config/master.ini22:26
rlandyfor the TWO standalone jobs22:26
rlandyto differentiate them?22:27
weshaypanda, can we overload nodes?22:27
weshay1ctrl_centos22:27
weshay1ctrl_fedora22:27
weshayor we just add a field22:28
weshayand update the rdo jobs22:28
pandaweshay: that could cause the plasma coils to fluctuate, and generate a disruption in the quantum fields.22:28
rlandywhich in english means???22:28
weshayexport TOCI_JOBTYPE="periodic-{{ environment_type }}-{{distro}}-{{ nodes }}-featureset{{ featureset }}"22:28
weshayand we could think about22:28
weshayexport TOCI_JOBTYPE="periodic-{{ environment_type }}-{{distro}}-{{arch}}-{{ nodes }}-featureset{{ featureset }}"22:29
weshaywhile we're at it22:29
weshaysuspect ppc is coming for some reason :))22:29
rlandyproblem stands in https://github.com/rdo-infra/ci-config/blob/master/ci-scripts/dlrnapi_promoter/config/master.ini22:29
pandaadding distro is ok, but the name then will be the same for all the jobs at that point, we need to change all the .ini files22:29
weshayya22:29
weshaythat's ok22:29
rlandyexactly22:29
rlandythat is what my comment said22:29
rlandydoes anyone know where attila is now??22:30
pandathe hun ?22:30
rlandythe darazs22:31
rlandythat was one big assumption22:31
pandarlandy: I didn't understand that you wanted to change ALL the names22:31
weshayrlandy, panda should we just discuss for 5min on blue22:32
weshay?22:32
weshayif we're not clear.. or tomorrow22:32
weshaysooo late for mr panda22:32
rlandyto me all the names in the ini files need to specify  os22:32
weshaywho keeps working sooo late22:32
weshayrlandy, ya..22:32
rlandywe can - up to panda22:32
rlandylate for him22:32
pandarlandy: weshay no need to discuss, I'm ok with it22:32
weshayreally if we're smart we'll think about arch as well no22:33
pandaadd distro and arch, then change ALL the promotion criteria22:33
rlandywell - I dont want to go make the wrong change on every file22:33
weshayrlandy, wait.. let's chat22:33
weshaybecause baremetal is coming too22:33
rlandysomeone shoot me22:33
weshaylolz22:34
weshayhttps://bluejeans.com/u/whayutin/22:34
weshaylolz22:34
weshaymuah ha ha22:34
pandawe need to recalibrate the warp capacitors too22:34
rlandypanda: ok with you - to talk now?22:34
pandarlandy: I'll whisper22:35
rfolcopanda, rlandy sorry I was at the supermarket, wife buys like there was no tomorrow22:48
rfolcopanda, so should I create a new story for the independent f28 pipeline ?22:49
pandarfolco: #447 is not the right US ?22:50
rlandyrfolco: we're chatting re: ini files22:51
weshayrlandy, https://logs.rdoproject.org/openstack-periodic/git.openstack.org/openstack-infra/tripleo-ci/master/periodic-tripleo-ci-centos-7-standalone-master/c992e32/logs/undercloud/home/zuul/standalone_deploy.log.txt.gz22:53
rlandyweshay: https://tree.taiga.io/project/tripleo-ci-board/task/501?kanban-status=144727823:06
rlandyjust fyi23:06
*** panda is now known as panda|off23:06
weshaythanks23:06
rlandyweshay: there are a number fo places in tripleo-common where docker.io is assumed ... https://github.com/openstack/tripleo-common/search?q=docker.io&unscoped_q=docker.io ... my guess is that we need to override this: https://github.com/openstack/tripleo-common/blob/master/container-images/container_image_prepare_defaults.yaml#L1623:17
rlandycorrect?23:18
rlandythere would be no overcloud containers for standalone23:18
weshayrlandy, ya.. /me looks23:22
weshaybut correct23:22
weshayrlandy, Emilien and Steve Baker are contacts23:22
rlandyweshay: ok - I'll add them as reviewers23:23
rlandyweshay: we will have to override that var in  promotion tests23:23
rlandywhich would mean a featureset_overrude23:23
rlandyoverride23:23
weshayI think it's just this though https://github.com/openstack/tripleo-common/blob/c5f954d52c6e7d07391b2b2928b974ad6d1e6aba/container-images/container_image_prepare_defaults.yaml23:23
rlandyya23:24
rlandybut I was going to add a var23:24
rlandyah - but I think I see what you think23:24
weshayline 1623:24
weshayya23:24
rlandyif promotion bits are set23:24
rlandythen choose another val23:24
rlandydon;t set it in job23:24
rlandyok23:24
rlandyI was going to override the default but that is easier if we can determine we are in a periodic job23:25
weshayrlandy, hrm.. isn't that in the toci jina23:27
rlandyweshay: ya I am looking for a var that would carry23:28
rlandythrough run-v323:28
weshayhttps://github.com/openstack-infra/tripleo-ci/blob/master/playbooks/tripleo-ci/templates/toci_gate_test.sh.j2#L12523:28
* weshay looking23:29
weshaynot sure if that is the one23:29
weshayhttps://github.com/openstack-infra/tripleo-ci/blob/master/playbooks/tripleo-ci/templates/toci_gate_test.sh.j2#L12623:29
rlandy zuul_log_path?23:29
rlandykind if hacky23:29
rlandy ZUUL_PIPELINE=openstack-periodic23:30
rlandy      pipeline: openstack-periodic23:31
rlandythe zuul inventory23:31
rlandyweshay: zuul.pipeline23:31
rlandyor zuul.job23:31
rlandy job: periodic-tripleo-ci-centos-7-standalone-master23:31
rlandybut I think pipeline is correct23:32
weshayrlandy, ha.. easy23:32
weshayif pipeline == periodic23:32
weshayset a flag to update23:32
weshayhttps://github.com/openstack/tripleo-quickstart-extras/blob/master/roles/standalone/templates/standalone-container-prep.sh.j2#L723:32
rlandycontains periodic23:32
rlandypipeline: openstack-periodic23:32
rlandynot straight equal23:33
weshayand set docker_registry_namespace23:33
rlandyafaict it's not docker_registry_namespace23:33
weshayrlandy, all you need to override is https://github.com/openstack/tripleo-quickstart-extras/blob/master/roles/standalone/templates/standalone-container-prep.sh.j2#L523:33
weshay?23:33
rlandyhttps://github.com/openstack/tripleo-common/blob/master/container-images/container_image_prepare_defaults.yaml#L1623:34
rlandyit's just namespace here23:34
weshayrlandy, I think this https://github.com/openstack/tripleo-common/blob/master/container-images/container_image_prepare_defaults.yaml#L1623:35
weshayis handled already23:35
weshayhttps://github.com/openstack/tripleo-quickstart-extras/tree/master/roles/standalone/templates23:35
weshayreview that ^23:35
rlandyok - so - changing a tqe value is easier23:36
weshayya23:36
weshayand https://github.com/openstack/tripleo-quickstart-extras/blob/master/roles/standalone/templates/standalone_config.yaml.j223:36
rlandywe would do that in release file23:36
weshayI think you can override the tripleo-common default w/ https://github.com/openstack/tripleo-quickstart-extras/blob/master/roles/standalone/templates/standalone_config.yaml.j223:36
weshayrlandy, we want to make the override only in ci if possible, because it's only needed for ci23:37
weshayso I probably said the wrong thing earlier23:37
rlandyhttps://github.com/openstack/tripleo-quickstart/blob/master/config/release/tripleo-ci/CentOS-7/consistent-master.yml23:37
weshaybut I think everything you need is in that dir23:37
rlandyI would make the change there23:37
weshayhrm23:38
weshayI don't think those are used23:38
rlandyhttps://github.com/openstack/tripleo-quickstart/blob/master/config/release/tripleo-ci/CentOS-7/promotion-testing-hash-master.yml23:38
weshayrlandy, ya..23:38
weshay+123:38
rlandydocker_registry_host: trunk.registry.rdoproject.org23:38
rlandydocker_registry_namespace: tripleomaster23:38
weshayyup yup23:38
rlandythose values are already there23:39
weshaycreate the same thing under Fedora-28/23:39
rlandyif that was working, centos7 would be passing23:39
weshayhttps://github.com/openstack/tripleo-quickstart/tree/master/config/release/tripleo-ci/Fedora-2823:39
weshaypiece of cake23:39
weshay:)23:39
rlandynot really23:39
weshay?23:39
rlandybecause that's how those files are now23:39
rlandyand it;s broken23:39
weshayno23:40
weshaythere is no Fedora-28 version23:40
rlandyif these files were being passed, it would work23:40
weshaythey are not23:40
rlandyyeah - but centos is also broken23:40
weshayha23:40
weshayya23:40
rlandyam I making sense, now?23:40
weshayso which release file is that job using23:40
weshayya23:40
rlandyI really don;t know - let's look23:40
weshay2018-12-04 17:00:44.947622 | primary | +(./toci_quickstart.sh:46): export 'QUICKSTART_DEFAULT_RELEASE_ARG=--extra-vars @/home/zuul/workspace/.quickstart/config/release/tripleo-ci/CentOS-7/promotion-testing-hash-master.yml'23:41
weshayhttps://logs.rdoproject.org/openstack-periodic/git.openstack.org/openstack-infra/tripleo-ci/master/periodic-tripleo-ci-centos-7-standalone-master/c992e32/job-output.txt.gz#_2018-12-04_17_00_44_94762223:41
rlandy QUICKSTART_RELEASE=promotion-testing-hash-master23:42
weshayya23:42
rlandyso why is it broken???23:42
rlandyonly for standalone23:42
weshayrlandy, docker_registry_host != the right var23:43
weshayrlandy, so the file is passed23:43
weshaybut w/ the wrong keys23:43
rlandydocker_registry_namespace23:43
weshayright23:43
rlandyhow are the other jobs passing?23:43
weshay    "docker_registry_host": "trunk.registry.rdoproject.org",23:44
weshayhttps://logs.rdoproject.org/openstack-periodic/git.openstack.org/openstack-infra/tripleo-ci/master/periodic-tripleo-ci-centos-7-standalone-master/c992e32/logs/undercloud/var/log/extra/dump_variables_vars.json.txt.gz23:44
rlandy    "docker_registry_host": "trunk.registry.rdoproject.org",23:44
rlandy    "docker_registry_namespace": "tripleomaster",23:44
weshaydocker.io\/{{ docker_registry_namespace }}/' \23:45
weshaythat line23:45
weshayhttps://github.com/openstack/tripleo-quickstart-extras/blob/master/roles/standalone/templates/standalone-container-prep.sh.j2#L723:45
weshayshould be23:45
weshay{{ docker_registry_host }}\/{{ docker_registry_namespace }}/' \23:45
rlandycorrect23:45
rlandynow we agree23:45
weshayya23:46
weshayand that if statement needs an update23:46
* rlandy adds reviews23:46
rlandythank you23:46
weshay+123:46
weshaynp :)23:46
weshaythank you23:46
* weshay steps away for a bit23:46

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