Tuesday, 2019-09-10

*** saneax has joined #oooq00:23
*** saneax has quit IRC01:03
*** rfolco has quit IRC01:41
*** Vorrtex has quit IRC01:55
*** apetrich has quit IRC02:10
*** ykarel|away has joined #oooq02:25
*** hamzy_ has quit IRC03:06
*** rlandy|bbl has quit IRC03:12
*** saneax has joined #oooq03:28
*** saneax has quit IRC03:33
*** ykarel|away has quit IRC03:53
*** gkadam has joined #oooq03:59
*** udesale has joined #oooq04:07
*** ykarel|away has joined #oooq04:08
*** ykarel|away is now known as ykarel04:09
*** surpatil has joined #oooq04:29
*** jtomasek has joined #oooq04:38
*** skramaja has joined #oooq04:41
*** pierreprinetti has joined #oooq04:58
*** pierreprinetti has quit IRC04:59
*** ratailor has joined #oooq05:04
*** marios has joined #oooq05:17
*** jtomasek has quit IRC05:29
*** raukadah is now known as chandankumar05:34
*** surpatil has quit IRC05:43
*** soniya29 has joined #oooq05:56
*** surpatil has joined #oooq05:57
*** saneax has joined #oooq06:02
*** jfrancoa has joined #oooq06:13
*** dtantsur|afk is now known as dtantsur06:15
*** jfrancoa has quit IRC06:20
*** kopecmartin|off is now known as kopecmartin06:26
*** jfrancoa has joined #oooq06:33
*** bogdando has joined #oooq06:52
*** akahat has joined #oooq06:57
*** surpatil is now known as surpatil|brb06:59
*** tesseract has joined #oooq07:05
*** apetrich has joined #oooq07:24
*** surpatil|brb has quit IRC07:29
*** jpena|off is now known as jpena07:33
*** gkadam has quit IRC07:34
*** surpatil|brb has joined #oooq07:37
*** apetrich has quit IRC07:39
*** ykarel is now known as ykarel|lunch07:39
*** apetrich has joined #oooq07:40
*** apetrich has quit IRC07:45
zbrmorning! can we merge https://review.opendev.org/#/c/679409/ ?07:48
*** apetrich has joined #oooq07:52
*** holser has joined #oooq07:53
zbrarxcruz: ping me when you are ready for a molecular experiment07:54
*** jtomasek has joined #oooq08:06
arxcruzzbr: ack08:23
*** panda|rover has quit IRC08:23
arxcruzsshnaidm|afk: i'm getting this:08:23
arxcruz2019-09-09 16:07:21 | 2019-09-09 16:07:04Z [overcloud.Controller.1.Controller]: CREATE_FAILED  ResourceInError: resources.Controller: Went to status ERROR due to "Message: Exceeded maximum number of retries. Exhausted all hosts available for retrying build failures for instance 8c2be614-9f6a-4c75-b5fe-67faa3902b01., Code: 500"08:23
arxcruzany idea ?08:23
arxcruzovb job08:24
*** panda has joined #oooq08:24
*** derekh has joined #oooq08:33
*** soniya29 has quit IRC08:34
*** sshnaidm|afk is now known as sshnaidm|ruck08:37
sshnaidm|ruckarxcruz, where?08:38
arxcruzsshnaidm|ruck: http://logs.rdoproject.org/00/673400/23/openstack-check/tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001/909edb4/logs/undercloud/home/zuul/overcloud_deploy.log.txt.gz08:42
arxcruzit's on a check job, but it's consistent :/08:42
arxcruzand the change doesn't affect it08:42
*** surpatil|brb has quit IRC08:43
*** surpatil has joined #oooq08:44
chandankumarmarios: https://review.opendev.org/#/c/681179/08:44
chandankumarmarios: left nitpick08:44
sshnaidm|ruckarxcruz, controller is not created from first time, but it's created later, look at Stack CREATE completed successfully08:44
arxcruzsshnaidm|ruck: you're right08:45
arxcruzsshnaidm|ruck: but it's failing the deployment and i don't know why08:46
arxcruzsshnaidm|ruck: nevermind, it's timeout08:46
arxcruzi'll try to run the reproducer08:46
marioschandankumar: ack thanks08:50
chandankumarsshnaidm|ruck: Hello08:54
sshnaidm|ruckchandankumar, hi08:55
chandankumarsshnaidm|ruck: https://review.opendev.org/#/c/655423/59/roles/validate-tobiko/tasks/tobiko-results.yml@26 does it need to be handled on collect-logs side or use sudo permission there?08:55
sshnaidm|ruckarxcruz, yeah, many ovb jobs fail with timeout now, known issue..08:55
sshnaidm|ruckchandankumar, this directory is created in collect-logs with become, user can't write to /var/log08:57
sshnaidm|ruckchandankumar, not sure why this in "extras", why not in "working_dir"?08:58
sshnaidm|ruckchandankumar, like we have tempest.html there08:58
chandankumarsshnaidm|ruck: ok, we need to dump the logs in working_dir and add a task in collect logs to copy there?08:58
sshnaidm|ruckchandankumar, it's not logs, I see one html file08:59
sshnaidm|ruck{{ tobiko_log_file }}.html08:59
sshnaidm|ruckchandankumar, it's same format as tempest.html08:59
sshnaidm|ruckchandankumar, maybe we can even add stackviz there.. in future09:00
chandankumarsshnaidm|ruck: https://github.com/openstack/ansible-role-collect-logs/blob/67821d2c4992d129a0b59cbfefab08358f547577/tasks/publish.yml#L8909:00
*** ykarel|lunch is now known as ykarel|09:01
*** ykarel| is now known as ykarel09:01
chandankumarsshnaidm|ruck: thanks, commented on the review09:02
sshnaidm|ruckchandankumar, I think collect-logs will automatically copy all html files from working_dir09:02
chandankumarsshnaidm|ruck: nope https://github.com/openstack/ansible-role-collect-logs/blob/67821d2c4992d129a0b59cbfefab08358f547577/tasks/publish.yml#L8609:04
sshnaidm|ruckchandankumar, well, or not: https://github.com/openstack/tripleo-ci/blob/master/toci-quickstart/config/collect-logs.yml#L22-L2809:04
chandankumarwe are explicitly coping09:04
sshnaidm|ruckchandankumar, undercloud/home/stack/tempest/tempest - it's not for CI \09:05
chandankumar*copyinh09:05
chandankumarsshnaidm|ruck: oh ok09:05
sshnaidm|ruckchandankumar, this publish.yaml looks awful.. need to rewrite it completely09:08
chandankumarsshnaidm|ruck: ok, will create a user story for that09:09
chandankumarkopecmartin: ^^09:09
sshnaidm|ruckchandankumar, "/home/stack" doesn't exist in CI, so it always fails09:10
sshnaidm|ruckand "|| true" there to hide it... awful09:10
*** jaosorior has joined #oooq09:11
*** rfolco has joined #oooq09:14
sshnaidm|ruckmarios, hi, answered on https://review.opendev.org/#/c/681048/09:17
mariossshnaidm|ruck: thx revoted09:19
*** soniya29 has joined #oooq09:22
sshnaidm|ruckbogdando, hi, can  you please explain why it's needed? https://review.rdoproject.org/r/#/c/22154/09:29
bogdandosshnaidm|ruck: zuul places comments on the patches if configs are wrong09:30
sshnaidm|ruckbogdando, right (sometimes)09:30
bogdando:)09:30
sshnaidm|ruckbogdando, but what is this change for?09:31
bogdandodebugging experience improving09:31
bogdandosay you want to amend the patch submitted for testing09:31
bogdandoand using os_autohold_node=false09:31
bogdandoto retry the job and see if it doesn't start for a reason09:32
bogdandoso no need to grep scheduler logs and search if it says something about broken zuul config09:32
bogdandoit will post it on the patch09:32
sshnaidm|ruckbogdando, yeah, but I don't understand how this patch is related to that09:34
bogdandow/o it, no comments provided09:34
sshnaidm|ruckbogdando, you create zuul.yaml a little before it's created by script09:34
bogdandoby zuul in the patch09:34
bogdandoI commit it09:34
bogdandoit needs to be commited09:34
bogdandothen we submit the zuul.yaml under test09:34
bogdandoand now zuul can post comments onto it09:35
sshnaidm|ruckbogdando, so you mean zuul.yaml should present in the repo yet before any change?09:35
bogdandoyes09:35
bogdandowith empty check jobs09:36
bogdandolike p[09:36
bogdando[]09:36
sshnaidm|ruckbogdando, well, this is very not obvious :)09:36
bogdandojust that Paul recommended, and it worked for me :)09:36
bogdandonot the job tho, just comments on failed configs :(09:36
marioshttps://www.theregister.co.uk/2019/09/09/the_end_of_python_2_2020/ zbr :D09:37
sshnaidm|ruckbogdando, great, I think it worth a few more words in commit, because it's something from internal knowledge of zuul devs09:37
bogdandowill do09:38
sshnaidm|ruckbogdando, thanks for explanation, good to know such things09:38
sshnaidm|ruckbogdando, and for your reproducer improvements :)09:38
bogdandosshnaidm|ruck: currently I'm trying to fix the same issue rlandy had in the past http://eavesdrop.openstack.org/irclogs/%23zuul/%23zuul.2018-12-06.log.html#t2018-12-06T15:32:1109:43
bogdandohttps://review.opendev.org/68118209:43
bogdandothat patch seems did not that I expected, may be I need better dance with that delegate_facts... like in the set_facts itself?..09:43
bogdandoreading https://medium.com/opsops/perfecting-fact-gathering-in-ansible-d367b45cbda6 and it looks confusing09:44
sshnaidm|ruckbogdando, hmm.. but there are no facts in this task, it's just command task09:47
sshnaidm|ruckbogdando, btw, don't you need to set also gate jobs=[] in https://review.rdoproject.org/r/#/c/22154/ ?09:54
sshnaidm|ruckor it's enough to have check09:55
sshnaidm|ruckchandankumar, wrt https://review.opendev.org/#/c/655423/59/roles/validate-tobiko/tasks/tobiko-results.yml - html is not in the list10:03
sshnaidm|ruckchandankumar, so it's not taken by collect logs automatically, we can add tobiko files to collect-logs config though10:04
chandankumarsshnaidm|ruck: ok10:04
*** udesale has quit IRC10:09
*** surpatil has quit IRC10:09
*** apetrich has quit IRC10:10
kopecmartinwhat is the new link for grafana dashboard cockpit thing? the one i have bookmarked is no longer valid it seems10:10
*** surpatil has joined #oooq10:10
*** udesale has joined #oooq10:10
sshnaidm|ruckkopecmartin, use http://cockpit-ci.tripleo.org it always points to current dashboard10:17
sshnaidm|ruckkopecmartin, or dashboard-ci.tripleo.org and then choose the board10:18
kopecmartinsshnaidm|ruck: cool, thanks10:18
sshnaidm|ruckfolks, please review sova patch https://review.opendev.org/#/c/678630/10:20
chandankumararxcruz: I think you know this person akahat :-)10:38
chandankumarsshnaidm|ruck: does someone looking at ansible time out issue during overcloud deploy?10:56
sshnaidm|ruckchandankumar, in ovb jobs?10:57
chandankumarsshnaidm|ruck: yes in ovb, fs03910:57
sshnaidm|ruckchandankumar, I suspect it's because of 10 days w/o master promotion, so maybe ok when we promote10:57
chandankumarsshnaidm|ruck: ok10:57
sshnaidm|ruckchandankumar, and I hope we do it today10:58
*** jpena is now known as jpena|lunch10:59
*** udesale has quit IRC11:14
bogdandosshnaidm|ruck: reproducer only checks, does not merge11:16
bogdandoat least for the first execution11:16
bogdandoso we don't need gate configs prolly11:16
sshnaidm|ruckbogdando, ack.. fixing CI for reproducer now, to let these patches in11:21
arxcruzchandankumar: ?11:28
*** ykarel is now known as ykarel|afk11:50
zbrsshnaidm|ruck: are we now good with https://review.opendev.org/#/c/679409/ ?11:52
sshnaidm|ruckzbr, let me finish with master promotion please11:53
zbrsshnaidm|ruck: so I guess we are very close to it11:53
sshnaidm|ruckzbr, and it should be tested better than previous one, when we hit problems with rhel811:53
sshnaidm|ruckzbr, do you have a patch that removes all "hostvars" configs?11:55
zbrnope11:56
sshnaidm|ruckzbr, good to have it, it was the reason why we move to 2.8 actually11:57
zbri would say that consistency is a better reason / not having to wonder why some code behaves differently...11:58
sshnaidm|ruckzbr, so are you going to prepare this patch?12:03
zbrsshnaidm|ruck: plan too, after i close bj12:04
weshaymorning12:07
*** jpena|lunch is now known as jpena12:10
*** apetrich has joined #oooq12:12
sshnaidm|ruckweshay, hi, I don't know if to make it promotion-blocker bug: https://bugs.launchpad.net/tripleo/+bug/184342212:20
openstackLaunchpad bug 1843422 in tripleo "OVB featureset039 all-TLS job fails in master" [Critical,Triaged]12:20
sshnaidm|ruckweshay, it hardly works in master, I removed it temporarly from master criteria12:21
weshaysshnaidm|ruck agree master is foobar'd .   Let's get in touch w/ harry and see how they want to proceed12:22
weshaysshnaidm|ruck can we double check that featureset is close to fs00112:22
*** rlandy has joined #oooq12:23
sshnaidm|ruckweshay, yeah, will check12:23
rfolcomarios, panda rlandy: I just realized that our sync mtg conflicts with bug triage one12:29
rlandyrfolco: we need to sync12:29
mariosrfolco: ok.. so lets take it up on the community call then...12:29
mariosrlandy: ?12:29
mariosand take it from there12:29
rlandyok - I just don;t want to ditch the meeting altogether12:30
mariosrlandy: ack but not sure what else to do ... meeting is meant to start now12:30
rlandymarios: ack - that's fine12:30
rfolcogo without me, will catch up with you guys later12:30
mariosrfolco: panda rlandy lets sync @ community meeting then and continue from there12:31
rlandymarios: thanks for the vm setup instructions12:31
* marios really needs to talk about the manifests, too12:31
mariosrlandy: ack thanks panda for walking me through it12:31
rlandyat least I have a working env now12:31
rlandypy 36 test worry me though12:31
rlandyfedora12:31
rfolcochandankumar, weshay: are we going to do bug triage now?12:32
rfolcoI'm happy to do it offline if you can't12:32
*** apetrich has quit IRC12:33
*** surpatil has quit IRC12:33
weshaychandankumar rfolco ya12:34
* weshay joins12:34
pandamarios: can I see those instructions ?12:34
*** soniya29 has quit IRC12:34
mariospanda: its bash history rlandy do you have the link i gave yesterday?12:34
mariospanda: not instructions12:35
pandamarios: ok then maybe I have it already12:35
rlandymarios: http://pastebin.test.redhat.com/79462612:36
weshaychandankumar join our bug scrub12:36
weshaychandankumar also you see I still have a bug there in rc.local12:36
weshaybut the box is up :)12:36
chandankumarweshay: but earlier the labs dir was empty12:37
weshaychandankumar ya.. the image was not updated12:37
weshaychandankumar goal.. is to have the labs and rpms preinstalled w/ that customize script12:37
chandankumarweshay: ok12:38
chandankumarweshay: we also need to pre install wget there12:38
*** derekh has quit IRC12:43
*** derekh has joined #oooq12:43
weshaychandankumar ah ya.. I have that12:44
sshnaidm|ruckweshay, +w please https://review.opendev.org/#/c/681048/12:45
*** aakarsh|3 has quit IRC12:46
sshnaidm|ruckarxcruz, chandankumar can you please take a look at https://bugs.launchpad.net/tripleo/+bug/1843259 ?12:54
openstackLaunchpad bug 1843259 in tripleo "Periodic rocky fs020 job fails tempest tests tempest.scenario.test_security_groups_basic_ops.TestSecurityGroupsBasicOps.test_cross_tenant_traffic and tempest.scenario.test_security_groups_basic_ops.TestSecurityGroupsBasicOps.test_multiple_security_groups " [Critical,Triaged] - Assigned to Gabriele Cerami (gcerami)12:54
arxcruzsshnaidm|ruck: i saw it yesterday, yeah, it seems to be a real one, need to talk with someone from keystone i guess ?12:55
sshnaidm|ruckarxcruz, keystone.. which dfg is it? <-- weshay ?12:55
arxcruzsshnaidm|ruck: not sure if keystone, the issue is, that the two vm's are able to ping each other, and should not, that's the issue with the test12:56
arxcruzso, there's no security, or security is not working properly12:56
sshnaidm|ruckarxcruz, mmm.. it's networking then12:56
*** Goneri has joined #oooq12:57
*** udesale has joined #oooq12:57
*** hamzy_ has joined #oooq12:59
weshaysshnaidm|ruck keystone is security13:00
weshayharry13:00
sshnaidm|ruckrfolco, are you doing bug triage?13:03
rfolcosshnaidm|ruck, yes13:03
pandawhere do we do community call ?13:03
sshnaidm|ruckTripleo CI community meeting - https://bluejeans.com/5878458097 (NEW*)13:07
*** ykarel|afk is now known as ykarel13:10
weshayzbr check out doug hellman's email on rh-openstack-dev13:20
weshayzbr let me know what you think about that13:20
*** aakarsh|3 has joined #oooq13:21
sshnaidm|ruckrfolco, https://bugs.launchpad.net/tripleo/+bug/1841941 is not promotion blocker13:22
openstackLaunchpad bug 1841941 in tripleo "Cannot run scenario4 on rhel8 due to missing pcs required by manila" [Critical,Triaged]13:22
*** skramaja has quit IRC13:23
weshaysshnaidm|ruck it's on the cix board..  so either way..  we're good on that bug13:24
zbrweshay: i agree with him. as you probably observed I got enough of waiting for it and I find outselves put in a very bad situation13:24
zbras in make in work without having a base13:24
*** aakarsh|2 has joined #oooq13:24
*** ratailor has quit IRC13:25
zbrall that fedora28 work...13:25
zbrweshay: if I would be on a decision making position I would say: yes, we will start working on that when the requirement is available (at least a pre-release)13:26
*** aakarsh|3 has quit IRC13:27
sshnaidm|ruckzbr, weshay centos7 supports some python3, epel contains even python-3.6 for it13:31
zbrsshnaidm|ruck: weshay re py3 and centos, please read https://bugs.centos.org/view.php?id=1638913:32
zbrwithout selinux, py3 on c7 is *useless* AFAIK.13:33
zbrand they do not want to fix it (maybe some extra peer pressure could make them change their mind?)13:33
rfolcosshnaidm|ruck, can remove if is not right.13:35
sshnaidm|ruckrfolco, I don't think we need it, script may duplicate it in CIX board13:35
*** Vorrtex has joined #oooq13:47
*** gouthamr has left #oooq13:50
*** jtomasek has quit IRC14:00
*** jtomasek has joined #oooq14:01
*** aakarsh|3 has joined #oooq14:02
*** aakarsh|2 has quit IRC14:05
weshaysshnaidm|ruck it checks the board before adding14:14
weshaysshnaidm|ruck re: the script14:14
weshayperhaps I need to add the new column14:14
weshaywill check that14:14
sshnaidm|ruckweshay, ok14:14
*** surpatil has joined #oooq14:14
zbrpanda: rlandy : i added some fixes on https://review.rdoproject.org/r/#/c/21895/ -- but tests are still failing, please have a look.14:23
rlandyzbr: ack - on the same issue14:23
rlandythe missing http+docker://localhost/v1.35...tion/tripleomaster/hello-world:cc26d47faedf3643237cfb494d001d1add50267c/json14:24
rlandyat least it's consistent14:24
zbri will look on the molecule result now, clearly it should produce a valid results.html, it does not and I will fix that.14:25
*** dtantsur is now known as dtantsur|afk14:25
pandamaybe ti's better that we create the staging containers directly14:27
pandazbr: rlandy that happens beacuase the staging environment needs the staging containers to handle. Maybe the step to pull them is missing, and at this point it may be better if we create them locally and then mangle them, one less weak point14:30
rlandyzbr: why did you remove all those deps?14:32
rlandypanda: should we pull them?14:32
rlandyisn't that the real test?14:33
zbrrlandy: unittests and molecule tests have different requirements. we do not want to combine them.14:34
pandarlandy: the promoter logic will pull them, but to put them there in the first place, we can create them locally.14:34
rlandypanda: create them in https://review.rdoproject.org/r/#/c/21895/27/ci-scripts/infra-setup/roles/promoter/tasks/setup_staging.yaml?14:35
rlandyor in https://review.rdoproject.org/r/#/c/21895/27/ci-scripts/dlrnapi_promoter/tests/staging-setup/test_staging_environment.py?14:36
pandarlandy: in stagig_setup.yaml or in the staging_setup.py, whatever is easier. We just need a file, a Dockerfile and the docker build command. see https://github.com/openstack/ansible-role-container-registry/blob/master/molecule/login/playbook.yml#L15314:37
*** Vorrtex has quit IRC14:37
*** ykarel is now known as ykarel|afk14:42
rlandyzbr: where do we put the molecule requirements then?14:42
rlandywe still need the molecule tests to pass14:42
chandankumarweshay: I will take a look at the stuff tomorrow14:50
*** akahat has quit IRC14:52
rlandypanda: http://pastebin.test.redhat.com/79597314:56
rlandy^^ like this?14:57
rlandyI can't read the full path to see where it expects the file14:57
*** udesale has quit IRC15:00
*** udesale has joined #oooq15:00
zbrrlandy: panda: i see that you are doing something fishy: trying to run docker in a unittest job, which do not have docker installed on them.15:05
zbron rdo tox-py* are running as containers, so no chance of installing docker on them.15:05
zbron the other hand rdo-tox-molecule job does have a docker setup on it.15:05
arxcruzsshnaidm|ruck: i'm having problems with the reproducer15:05
arxcruzit's on ansible-role-tripleo-ci-reproducer : Wait for job to start15:05
arxcruzand then timeout15:06
arxcruzi'm using rdo-cloud15:06
rlandyzbr: we need to separate molecule and the py tests15:06
sshnaidm|ruckarxcruz, is it first run?15:06
rlandyI am running the molecule tests on my test vm15:06
rlandydo you want access to that?15:06
arxcruzsshnaidm|ruck: i tried a few times, now i remove all docker stuff, reboot the machine, and trying again15:06
rlandyalso installed docker-ce there15:07
rlandyso it's different15:07
sshnaidm|ruckarxcruz, just wait 15-20 min and come back checking if tenant is ready: http://localhost:9000/t/tripleo-ci-reproducer/status15:07
sshnaidm|ruckarxcruz, it takes some more time that planned..15:07
*** surpatil has quit IRC15:08
sshnaidm|ruckarxcruz, it takes 1.2 glass of beer time15:08
pandazbr: rlandy the stating environment python script is tested with tox -epy*, it's python15:08
zbrrlandy: i already did the split.15:08
zbrpanda: maybe you can explain http://logs.rdoproject.org/95/21895/29/check/tox-py27/47c461b/tox/reports.html15:09
zbrthis code is not molecule, but it tries to use docker.15:09
zbrafaik i am done with fixing the way jobs are triggered, all report real code errors now.15:10
pandazbr: it's the staging environment preparation, we are preparing containers to be availabe to the promotion process15:10
zbrpanda: are you trying to run docker commands? you cannot do this in a unittest job.15:10
pandarlandy: line 23 of your paste, for now it should be ok, but you need to remove the :v1.35 at the end15:10
pandazbr: not even in a functional test ? :( do I have to mock docker ?15:11
zbrpanda: mock for unit, or move this test to run in molecule job, and be sure is not run as unit.15:11
pandazbr: yes, probably the test should be self contained on a single repo15:11
zbrpanda: here is a hint: add @pytest.mark.webtest before the test to assure it will be run by molecule job instead of tox-py* ones.15:12
zbrsorry: @pytest.mark.molecule15:12
pandazbr: there's nothing molecule related there15:13
weshaysshnaidm|ruck you have a sec to review what fs001 rhel8 is running against15:13
zbris the fact that you need docker, and you do not have docker.15:13
weshayit's not running often enough in check afaict15:13
pandazbr: do we really need to run this on molecule ? also, docker on docker again15:13
weshaysshnaidm|ruck https://review.rdoproject.org/zuul/builds?job_name=tripleo-ci-rhel-8-ovb-3ctlr_1comp-featureset00115:13
weshayshould at least be running against ci related changes15:13
zbryou need to run it on a machine that has docker, unittests do not have.15:13
arxcruzsshnaidm|ruck: http://paste.openstack.org/show/774839/15:14
*** saneax has quit IRC15:14
arxcruzsshnaidm|ruck: so, i need to rerun again in a few minutes ?15:14
pandazbr: I need a more comprehensive explanation15:14
zbrpanda: if you prefer you can even add @pytest.mark.docker -- but be sure you change tox.ini command to add: -m "not docker" for py* envs.15:14
zbrlet me grab a coffee, 3min15:15
pandazbr: why can't the unit tests run docker. Also, I'm testing the classes there, not the single methods. It's more functional than unit15:15
rlandysec - call15:15
weshaysshnaidm|ruck sorry to bug you.. have a sec?15:15
sshnaidm|ruckweshay, yep15:16
weshaysshnaidm|ruck ok.. blue15:16
sshnaidm|ruckarxcruz, just. wait.15:17
rlandypanda: thanks - I'll add that change as soon as zbr tells us he is done15:18
pandarlandy: zbr needs to explain what we need to change in the testing and why ...15:19
rlandypanda: zbr: should we meet? would that be quicker?15:19
zbrhttps://bluejeans.com/265541792815:22
pandarlandy: meeting is quicker, just waiting for zbr to caffeinate himself.15:22
zbrto-read: http://doc.pytest.org/en/latest/example/markers.html15:23
zbrrlandy: ^ please join15:23
*** holser has quit IRC15:34
*** holser has joined #oooq15:39
*** apetrich has joined #oooq15:41
*** ykarel|afk is now known as ykarel|away15:42
*** surpatil has joined #oooq15:43
*** ykarel|away has quit IRC15:47
* marios almost out15:50
*** holser has quit IRC16:00
*** marios is now known as marios|out16:03
weshaymarios|out always makes me so nervous when he says that16:04
*** ykarel|away has joined #oooq16:06
*** tesseract has quit IRC16:06
*** jfrancoa has quit IRC16:07
*** aakarsh|2 has joined #oooq16:10
sshnaidm|ruckrlandy, don't we have injection of dlrn_hash to jobs with tripleo-repos? just in case..16:11
rlandysshnaidm|ruck: not sure I understand the question ...16:12
rlandyare you asking me if we do have injection?16:12
rlandyyou can specify the hash as a var16:12
sshnaidm|ruckrlandy, like dlrn_hash_tag with featureset_overrides, but for jobs with tripleo-repos like building jobs16:12
*** aakarsh|3 has quit IRC16:13
sshnaidm|ruckrlandy, because dlrn_hash_tag will work only with jobs that uses repo-setup role16:13
rlandyyeah16:13
rlandywe can use that override code elsewhere16:13
rlandythe var should always default16:13
sshnaidm|ruckrlandy, it does not work with jobs using tripleo-repos role16:14
rlandysshnaidm|ruck: k - if you want to create a bug, I'll look at it16:14
chandankumarpanda: how is the jobs working on centos 7.7 in fs021?16:15
*** kopecmartin is now known as kopecmartin|off16:16
rlandyzbr: http://logs.rdoproject.org/95/21895/33/check/tox-py27/412fc1f/tox/reports.html16:17
*** marios|out has quit IRC16:18
pandachandankumar: did not check in fs021, but centos7.7 is working in all the environments.16:18
chandankumarpanda: https://logs.rdoproject.org/openstack-periodic-master/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-7-ovb-1ctlr_2comp-featureset021-master/84b1a81/job-output.txt.gz16:18
chandankumarfrom this one stuff looks good to me16:18
pandachandankumar: and here we have the rest  https://review.opendev.org/61878016:19
rlandyzbr: http://pastebin.test.redhat.com/79601616:20
weshaypanda++16:21
chandankumarpanda: thanks :-)16:23
pandachandankumar: this is testing both centos7.7. during deployment and even before quickstart is started. there's an additional task there that updates the node to centos7.7 before anything else.16:28
chandankumarpanda: oh, great :-)16:29
* chandankumar is talking cr too much, now people are talking on infra channel also16:29
*** jpena is now known as jpena|off16:39
*** derekh has quit IRC16:44
*** ykarel|away has quit IRC16:54
rlandy    TASK [Staged promotion run] ****************************************************16:56
rlandyERROR! Could not find specified file in role: tasks/promotion_run16:56
rlandyignore_errors did not fix that16:56
rlandypanda: hmmm ^^ can you put a block rescue within an always16:58
pandarlandy: mmmhh, nested blocks17:01
pandarlandy: that task should not belong to the always branch17:01
pandarlandy: should not be part of the cleanup17:02
rlandypanda: ok - so I am looking for where that call is17:02
rlandy in staging_environment.py teardown17:02
rlandy python {{ ci_config_remote_src_dir }}/ci-scripts/dlrnapi_promoter/tests/staging-setup/staging_environment.py teardown --env-id 1234517:03
*** brault has joined #oooq17:03
rlandyall I see that is executed17:03
*** ykarel|away has joined #oooq17:03
pandarlandy: ok, so your goal is to not fail the playbook when https://review.rdoproject.org/r/#/c/21895/33/ci-scripts/infra-setup/roles/promoter/molecule/default/playbook.yml@104 fails17:05
rlandyah17:06
*** gouthamr has joined #oooq17:06
pandarlandy: that's where the ignore_errors (or block:..rescue:) is needed17:06
rlandyI am not sure that will work - but let's try17:07
pandarlandy: the staging preparation passes successfully because i'm not testing that the containers are uploaded successfully17:07
rlandypanda: k - trying the rescue but here17:07
rlandybit17:07
pandaso technically for the playbook, that part passes17:07
*** brault has quit IRC17:08
pandarlandy: I don't see why it shouldn't work but if it still doesn't comment that part and move on17:09
rlandyk - testing out - few minutes17:09
*** udesale has quit IRC17:12
*** brault has joined #oooq17:22
*** brault has quit IRC17:27
*** jtomasek has quit IRC17:37
*** surpatil has quit IRC17:47
rlandy stderr: 'usermod: group ''993'' does not exist'18:15
rlandypanda: zbr: hmm ... we're not better off18:15
rlandy^^18:15
rlandyand still missing image18:15
pandarlandy: usermod -g should work with groud ids18:22
pandarlandy: instead of -G18:23
rlandypanda:looking at marios notes he had -a -G18:24
rlandyoh nvm  that was a name18:24
rlandycorrecting18:25
pandarlandy: on second thought18:29
pandarlandy: if a group does not have a name it probably doesn't exist18:29
pandarlandy: and you can't add user to it18:29
pandarlandy: so maybe it's better to add this group to the system18:29
rlandy sudo usermod -a -G <name> {{ promoter_user }}"18:30
rlandyE           requests.exceptions.HTTPError: 404 Client Error: Not Found for url: http+docker://localhost/v1.35/distribution/tripleomaster/hello-world:cc26d47faedf3643237cfb494d001d1add50267c/json18:32
rlandywell we are back with that18:32
rlandygid seemed to work18:32
rlandyneed to be there18:34
*** saneax has joined #oooq18:35
pandarlandy: mh,pytest didn't skip the test there then18:36
rlandymaybe we didn;t save that condition18:36
pandaadding @pytest.mark.docker should have been enough18:37
rlandy      stderr: 'usermod: group ''993'' does not exist'18:37
rlandyso gid did not work18:37
pandarlandy: yep, you need to be sure the group exists first18:37
rlandycreating18:38
*** ykarel|away has quit IRC18:39
rlandypanda: is the group name docker?18:46
rlandyis that set somewhere?18:46
rlandyie: via variable?18:46
zbrpanda: i am afraid we may need to add -m "not docker" to pytest.18:47
zbrif i remember well the inclusion is implicit for markers.18:47
rlandyzbr - will try that18:50
rlandyediting the tests anyways18:50
pandazbr: rlandy yes tested with tox, we need to add -m "not molecule and not docker"18:53
zbryeah, until I find a better way lets do that. i am also out, too late.18:53
pandarlandy: docker group name, yes18:53
pandazbr: there's no better way18:53
pandathere is only zuul18:54
*** panda is now known as panda|rover|off18:54
panda|rover|offzbr: we should register markers ..19:00
zbrpanda|rover|off: is there life after zuul?19:00
zbrtime to netfix for me19:00
*** holser has joined #oooq19:03
*** aakarsh|3 has joined #oooq19:05
*** aakarsh|2 has quit IRC19:07
*** saneax has quit IRC19:08
rlandyrdo-infra/ci-config19:10
rlandy21895,3619:10
rlandy3 min19:10
rlandy6 min19:10
rlandy    tox-linterssuccess19:10
rlandy    tripleo-ci-promotion-stagingsuccess (non-voting)19:10
rlandy    rdo-tox-molecule19:10
rlandy    (non-voting)19:10
rlandy    tox-jjbsuccess19:11
rlandy    tox-py27success19:11
rlandy    tox-py3619:11
rlandygetting there19:11
rlandypanda|rover|off: zbr: ^^19:11
*** panda|rover|off has quit IRC19:37
*** panda has joined #oooq19:38
sshnaidm|ruckplease vote https://review.opendev.org/#/c/681302/19:46
*** aakarsh|2 has joined #oooq19:46
*** aakarsh|3 has quit IRC19:49
rfolcoweshay, https://tree.taiga.io/project/tripleo-ci-board/task/1245 has the design w/ the drawings, please review.19:53
weshaythanks19:57
weshay!19:57
weshayrfolco these are nice man20:00
weshayrfolco what tool are you using to make these?20:00
rfolcoweshay, draw.io, links are in the task for the source, all saved in our multi-arch folder in google drive20:04
weshaynice man20:12
*** aakarsh|2 has quit IRC20:19
*** Goneri has quit IRC21:00
rlandyweshay: hey there21:06
rlandyweshay: remember this error: http://logs.rdoproject.org/95/21895/39/check/rdo-tox-molecule/f603bf8/tox/reports.html21:06
rlandy[31m[1mE   ImportError: No module named mock[0m21:07
*** holser has quit IRC21:15
*** holser has joined #oooq21:24
*** sshnaidm|ruck is now known as sshnaidm|afk21:33
weshayrlandy ya21:36
* weshay looks21:36
rlandyweshay: nvm - fixed that error21:36
weshayheh k21:36
rlandygot one error left to getting the staging stuff to mergable21:37
weshaynice.. even the python3.6?21:38
rlandyweshay: https://review.rdoproject.org/r/#/c/21895/21:40
rlandycheck it out21:40
rlandygreeeeeeennnnnnnnn21:40
rlandyweshay: pls also check the design on https://tree.taiga.io/project/tripleo-ci-board/us/1211?milestone=23859521:42
rlandyfor anything missing21:43
rlandyrfolco: what's the situation with running the service as a script?21:43
rlandy^^ we are ready to start developing python tests21:43
rlandyzbr: panda: if you're both happy with the changes on https://review.rdoproject.org/r/#/c/21895/, we can change the message and merge21:46
rlandywhatever - changing commit message so we are ready21:49
*** panda has quit IRC22:26
*** panda has joined #oooq22:28
*** brault has joined #oooq22:43
*** holser has quit IRC22:46
*** holser has joined #oooq22:47
*** brault has quit IRC22:59
*** holser has quit IRC23:04
*** holser has joined #oooq23:05
*** holser has quit IRC23:09
rfolcorlandy, still working on it. I'll try to get something for tomorrow23:55
*** aakarsh|2 has joined #oooq23:58

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