Monday, 2019-07-29

*** d0ugal has quit IRC00:05
*** d0ugal has joined #oooq00:21
*** dsneddon has joined #oooq00:50
*** dsneddon has quit IRC00:55
*** dsneddon has joined #oooq00:57
*** aakarsh has joined #oooq01:00
*** dsneddon has quit IRC01:42
*** dsneddon has joined #oooq02:07
*** rfolco|rover has joined #oooq02:08
*** rfolco|rover has quit IRC02:16
*** vinaykns has joined #oooq02:20
*** dsneddon has quit IRC02:33
*** vinaykns has quit IRC02:44
*** bhagyashris has joined #oooq02:49
*** dsneddon has joined #oooq02:59
*** dsneddon has quit IRC03:21
*** ykarel|away has quit IRC03:33
*** ykarel|away has joined #oooq03:33
*** ykarel|away has quit IRC03:37
*** ykarel|away has joined #oooq03:38
*** skramaja has joined #oooq03:51
*** udesale has joined #oooq03:59
*** ykarel|away has quit IRC04:17
*** ksambor has joined #oooq04:20
*** sabedevops has joined #oooq04:31
*** sabedevops has quit IRC04:37
*** ykarel|away has joined #oooq04:37
*** ykarel|away is now known as ykarel04:38
*** apetrich has joined #oooq05:09
*** jaosorior has joined #oooq05:26
*** marios|ruck has joined #oooq05:26
*** marios has joined #oooq05:26
*** raukadah is now known as chandankumar05:46
*** ratailor has joined #oooq05:47
*** ratailor_ has joined #oooq05:50
*** ratailor has quit IRC05:52
*** kopecmartin|off is now known as kopecmartin06:15
*** apetrich has quit IRC06:28
*** jfrancoa has joined #oooq06:32
*** saneax has joined #oooq06:37
*** tesseract has joined #oooq07:08
*** tosky has joined #oooq07:16
*** bhagyashris has quit IRC07:19
*** bogdando has joined #oooq07:25
*** jpena|off is now known as jpena07:30
*** ykarel is now known as ykarel|lunch07:33
*** brault has joined #oooq07:38
*** arxcruz|off is now known as arxcruz07:38
*** apetrich has joined #oooq07:45
*** bhagyashris has joined #oooq07:57
*** jaosorior has quit IRC07:59
chandankumarzbr: sshnaidm please have a look at this review https://review.opendev.org/#/c/673021/ and08:03
*** soniya29 has joined #oooq08:12
*** soniya29 has quit IRC08:20
*** jfrancoa has quit IRC08:36
*** yolanda has joined #oooq08:42
*** gkadam has joined #oooq08:44
*** Goneri has joined #oooq08:45
*** ykarel|lunch is now known as ykarel08:46
*** jaosorior has joined #oooq08:46
*** panda has quit IRC08:47
*** panda has joined #oooq08:48
*** soniya29 has joined #oooq08:50
*** derekh has joined #oooq08:53
zbrsshnaidm: can we merge https://review.rdoproject.org/r/#/c/21617 ? it would ease review of the bigger one.08:55
*** soniya29 has quit IRC08:58
*** dtantsur|afk is now known as dtantsur08:58
*** tesseract has quit IRC08:58
*** dsneddon has joined #oooq09:01
*** dsneddon has quit IRC09:05
*** ccamacho has joined #oooq09:13
sshnaidmzbr, I don't understand why you duplicate it09:17
zbrnot duplicating, is splitting because they were addressing different aspects, making much easier to review the code.09:18
sshnaidmzbr, let's concentrate on 21463 and finish it finally09:19
sshnaidmzbr, I'm testing it with live jobs, so should be fine09:19
sshnaidmzbr, but please revert this part: https://review.rdoproject.org/r/#/c/21463/24/roles/ovb-manage/tasks/find_undercloud_uuid.yml09:20
zbr the problem you mentioned, was because use of set -x, which i removed09:20
sshnaidmzbr, the problem that this part doesn't solve any problem and not belong to this task09:20
*** tesseract has joined #oooq09:20
sshnaidmzbr, we do not have problems with uuid09:21
marios|rucksshnaidm: did you update grafana lately?09:21
sshnaidmmarios|ruck, nope09:21
marios|rucksshnaidm: never mind refresh 'fixed it '09:21
marios|rucksshnaidm: ack sorry09:21
sshnaidmmarios|ruck, you can change things there, but until you press "save" refresh removes them09:22
marios|rucksshnaidm: when you have 5 mins can you check  https://review.rdoproject.org/r/#/c/21611/1/ci-scripts/infra-setup/roles/rrcockpit/README.md - in particular I'm not sure I got it right with the 'in order to see the data you have to initialise' etc09:22
zbrthere are clear issues with the uuid part, but I will do only minimal in this change (updating it now)09:23
sshnaidmzbr, which issues with uuid part do you have? Jobs work great09:24
sshnaidmmarios|ruck, which lines09:24
*** altlogbot_1 has quit IRC09:24
*** altlogbot_0 has joined #oooq09:24
marios|rucksshnaidm: like see https://review.rdoproject.org/r/#/c/21611/1/ci-scripts/infra-setup/roles/rrcockpit/README.md@6309:24
sshnaidmmarios|ruck, hmm.. I don't remember I did this tbh. If you just run docker compose there are "grafana sidecar" container that do everything for import09:28
marios|rucksshnaidm: ack ok i can try again and see if it replicates09:31
marios|rucksshnaidm: thanks for checking09:31
zbrsshnaidm: updated https://review.rdoproject.org/r/#/c/21463/ -- uuid part and also small bit related to use of ansible_os_family -- so we avoid confusion between distro and family with rhel.09:31
chandankumarsshnaidm: panda when free, please have a look at this https://review.rdoproject.org/r/#/c/21600/ container push09:33
chandankumarthanks!09:33
zbrsomeone pointed (maybe you?) that the overlayed loading of os specific vars can have unexpected results. this approach should avoid the confusion: https://review.rdoproject.org/r/#/c/21463/25/roles/ovb-manage/tasks/main.yml09:33
zbrmainly is about "redhat-8.yml" is refereing to the redhat os or redhat family. if we prefix families with "family-" we avoid the issue.09:34
zbrat this moment that is not very important but once we have centos-8, it will be.09:35
sshnaidmzbr, I think you can set openstack clients for stein version, I'll test it09:39
sshnaidmzbr, https://releases.openstack.org/stein/index.html#service-client-projects09:40
zbrsshnaidm: what we put there is debatable, but I would prefer to avoid that debate and only have some values. once we have code running we can decide. ok?09:47
sshnaidmzbr, what's debatable now?09:47
zbrsshnaidm: what value to put? minimal? current? future? what if a new patch comes out?  example: python-ironicclient lists initial 2.6.0, current 2.7.2. reality: tomorrow they could release a security patch 2.7.3, and we would have do maintenance in our ovb-manage code.09:49
*** Goneri has quit IRC09:49
*** apetrich has quit IRC09:50
zbrpersonally I woudl go for <2.8.0 in that case. seems bit more future proof.09:50
zbrto be correct: >=2.6.0,<2.8.009:50
zbrsounds ok?09:50
zbrideally we should be able to use a constraints file for that.09:51
sshnaidmzbr, the whole point is to use a specific version, not range, so that changes there don't break us09:51
sshnaidmzbr, it09:52
sshnaidmzbr, it's in trusted repo, so anyway every change should be reviewed09:52
sshnaidmzbr, just set Most Recent Version like we had before09:53
zbrwell, the way i see it  is that we would use deprecated code, and create more maintenance work for us (the most boring kind of).09:53
zbrsshnaidm: ok, will pick latest release for stein.09:53
zbrsshnaidm: done. added pinning to stein current ones, include comment.09:58
sshnaidmchandankumar, commented: https://review.rdoproject.org/r/#/c/21600/09:58
zbrsshnaidm: also the rdo-linters got reviewed by rdo https://review.rdoproject.org/r/#/c/21591/ -- it did worth having several pairs of eyes on it.10:01
*** bhagyashris has quit IRC10:02
chandankumar:-(10:03
*** Goneri has joined #oooq10:04
arxcruzchandankumar, https://tree.taiga.io/project/tripleo-ci-board/task/1147?kanban-status=1447274 can you tell me exactly what you want me to do ?10:11
* marios|ruck brb10:14
chandankumarsshnaidm: commented back10:20
chandankumararxcruz: may be zbr and panda & marios|ruck can help on that10:21
arxcruzpanda, zbr marios|ruck ^10:21
chandankumararxcruz: there is an example given in the test requirements10:21
* chandankumar needs to some time to go through the doc10:22
arxcruzjobs are defined such that:10:22
arxcruztripleo-buildcontainers/pre.yaml github.com/openstack/tripleo-ci/blob/master/playbooks/tripleo-buildcontainers/pre.yaml can be swapped out with something like paste.openstack.org/show/75310710:22
arxcruzchandankumar, this is not example, the only difference between the files is the name of the task basically10:22
arxcruzand the name of one role10:22
*** jaosorior has quit IRC10:23
sshnaidmchandankumar, set -e is implicit in ansible, it will fail task if there is fail in shell10:26
chandankumarsshnaidm: ok, got it thanks10:28
chandankumarsshnaidm: can I do the docker python changes in a new review?10:28
sshnaidmchandankumar, yeah, sure10:28
chandankumarsshnaidm: thanks!10:28
sshnaidmzbr, I meant to change it here: https://review.rdoproject.org/r/#/c/21463/26/roles/ovb-manage/defaults/main.yml10:29
sshnaidmzbr, let's have one set of pip packages10:29
pandaarxcruz: as far as I know taht was for rhel7, and it's not needed anymore10:29
marios|ruckarxcruz: looking10:31
marios|ruckarxcruz: i think panda was digging at that10:31
marios|ruckarxcruz: i have not10:31
chandankumarcreated new task for container build anyone wants to take https://tree.taiga.io/project/tripleo-ci-board/task/120310:48
zbrsshnaidm: not sure I understand, you want me to switch to use of stein packages for all distros?10:55
sshnaidmzbr, yep10:55
zbrahh, ok.10:55
*** saneax has quit IRC10:57
chandankumarsshnaidm: arxcruz currently at mutiple places we are using hostvars https://codesearch.rdoproject.org/?q=host-vars%3A&i=nope&files=.yaml&repos=10:59
chandankumaris it possible to define at one place instead of defining at multiple places for rhel8 jobs?10:59
chandankumarI mean setting ansible_python_interpreter in host_vars11:00
*** gkadam has quit IRC11:02
marios|rucksshnaidm: i think its ssl and/or selinux ?11:03
marios|rucksshnaidm: its just what we have upstream i.e. jobs for each of overcloud-full, hardened and ipa11:04
arxcruzchandankumar, where's the taiga task for this ?11:07
arxcruzi can work on that, i think we need to create a base job for those and set the var there11:08
chandankumararxcruz: putting in a card11:08
arxcruzchandankumar, add myself if you may11:11
chandankumararxcruz: https://tree.taiga.io/project/tripleo-ci-board/task/120511:13
chandankumarsshnaidm: zbr panda marios|ruck feel free to take a look at this task https://tree.taiga.io/project/tripleo-ci-board/task/1205 and have your comments on this, thanks!11:14
chandankumararxcruz: done11:14
chandankumararxcruz: let me know if more info needed11:15
*** gkadam has joined #oooq11:17
*** udesale has quit IRC11:26
*** jaosorior has joined #oooq11:34
*** rfolco|rover has joined #oooq11:37
*** rfolco|rover is now known as rfolco|ruck11:37
*** rfolco|ruck is now known as rfolco|rover11:39
*** saneax has joined #oooq11:39
rfolco|rovermarios, o/11:39
rfolco|rovermarios, will a get a coffee :)11:40
marios|ruckack rfolco|rover /me finishes lunch11:41
marios|ruckomnomnom11:41
*** gkadam has quit IRC11:42
weshaygreetings11:43
*** derekh has quit IRC11:43
zbro/11:44
*** gkadam has joined #oooq11:44
*** jpena is now known as jpena|lunch11:46
weshaychandankumar did you speak to vivek?11:57
weshaychandankumar https://drive.google.com/drive/u/1/folders/1oUNCnuy2iTSLy813I8JAnJ5J3fZkiap011:57
weshaysshnaidm marios|ruck you guys available ?11:58
sshnaidmweshay, yes11:58
marios|ruckweshay: me and rfolco|rover about to sync up before the escalation call11:58
marios|ruckweshay: need us to do something ?11:58
marios|ruckrfolco|rover: you joining?12:00
rfolco|roverbj issue looks like12:00
rfolco|roverdammn12:00
marios|ruckrfolco|rover: don't see you on the call12:00
marios|ruckrfolco|rover: let me rejoin too then12:00
chandankumarweshay: nope12:02
chandankumarweshay: I did not get any invite12:02
*** saneax has quit IRC12:03
rfolco|rovermarios|ruck, crap bj is not working today12:03
marios|ruckrfolco|rover: ack /me here if you want to retry/12:03
chandankumarweshay: please have a look at this task https://tree.taiga.io/project/tripleo-ci-board/task/1147?kanban-status=1447275 /me not sure what needs to be done or is it still relevant with RHEL-8 work?12:03
rfolco|rovermarios|ruck, trying...12:04
marios|ruckrfolco|rover: do you want to try your bluejeans/not sure what difference it would make12:05
marios|ruckrfolco|rover: ah i see you12:05
marios|ruckpanda: you have any update on the mac address thingy https://trello.com/c/23j209Gg/894-cixlp1818060tripleociproa-nodes-periodic-tripleo-ci-centos-7-ovb-3ctlr1comp-featureset001-master-fail-to-get-usable-ips-though-o12:17
marios|ruckpanda: http://logs.rdoproject.org/openstack-periodic-master/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-master/5b0a8a2/logs/undercloud/home/zuul/overcloud_deploy.log.txt.gz12:18
marios|ruckpanda: 2019-07-28 21:06:24 | [2019/07/28 09:03:17 PM] [ERROR] stdout: ERROR     : [/etc/sysconfig/network-scripts/ifup-eth] Error, some other host (EE:38:39:53:A7:9E) already uses address 172.17.0.67.12:18
marios|ruckpanda: just fyi am discussing now with folco for the handover ruck|rover we have the escalation call in 10 mins might come up12:18
pandamarios|ruck: it's impossible to catch red handed, we tried, no success, there is additional work the sf-ops would need to analize it more, and I think the general idea is "when we go to vexxhosts this will not happen"12:19
*** marios|ruck is now known as marios|ruck|call12:26
*** apetrich has joined #oooq12:27
marios|ruck|callpanda: ack just wondering if there was something... at least its not the most urgent issue/not happening a huge number of times/at least i haven't seen it a lot... luckily stioll relatively rare12:27
*** skramaja has quit IRC12:30
*** jpena|lunch is now known as jpena12:33
*** ykarel is now known as ykarel|afk12:34
weshayneed volunteers for an interview panel on Friday12:34
weshaychandankumar 11am UTC, you avail?12:35
marios|ruck|callweshay: sure12:35
chandankumarweshay: yes12:35
marios|ruck|callpanda: join me ^12:35
chandankumarweshay: sorry not available12:36
chandankumargoing to devconf.in12:36
weshaychandankumar hrm.. think I can ask Yatin?12:36
chandankumarweshay: he is also not available12:37
chandankumarBoth are going to devconf12:37
*** ratailor_ has quit IRC12:38
weshaychandankumar you mudda frankers12:41
weshay:)12:41
pandamarios|ruck|call: sorry was grabbing a sandwich, do you still need me ?12:41
*** amoralej is now known as amoralej|lunch12:45
*** marios|ruck|call is now known as marios|ruck12:47
*** rfolco|rover is now known as rfolco|ruck12:47
marios|ruckpanda: nop was just asking for any updates there is all thanks panda12:47
rfolco|ruckmarios|ruck, switching nicks12:47
mariosmarios|ruck: he means you... get out of here12:47
marios|ruckbut but12:48
mariosmarios|ruck: GO NOW !12:48
marios|ruck:(12:48
*** marios|ruck has quit IRC12:48
rfolco|ruckmarios|ruck, weshay what is my action/to-do on cix cards ? any issue to reproduce ?12:48
*** marios|rover has joined #oooq12:49
*** ykarel|afk has quit IRC12:49
mariosrfolco|ruck: look at https://trello.com/c/23j209Gg/894-cixlp1818060tripleociproa-nodes-periodic-tripleo-ci-centos-7-ovb-3ctlr1comp-featureset001-master-fail-to-get-usable-ips-though-o esp apevec comment12:49
weshayrfolco|ruck no... not that I saw12:49
mariosrfolco|ruck: is that issue he references the same as the upstream bug?12:50
mariosrfolco|ruck: is what they'll ask you wednesday ^12:50
rfolco|ruckmarios, ok, will see... thx12:52
marios|rovers12:59
marios|roverc12:59
marios|roverr12:59
marios|roveru12:59
marios|roverm12:59
marios|roverpanda: rfolco|ruck chandankumar panda arxcruz ^13:00
marios|roverzbr: ^13:00
chandankumarsshnaidm: and the push failed13:04
sshnaidmchandankumar, where?13:10
chandankumarsshnaidm: while pushing to rhel8 rdo registry13:10
chandankumarlogs are getting pushed13:10
sshnaidmchandankumar, logs?13:10
chandankumarwe can see what happened13:10
zbrsshnaidm: if is ok lets start to merge the ovb change chain.13:14
sshnaidmzbr, I +2 on https://review.rdoproject.org/r/#/c/21463/13:14
zbrsshnaidm: but that one is build on top of two minor ones.13:15
sshnaidmzbr, which ones??13:16
zbrhttps://review.rdoproject.org/r/#/c/21617/2/roles/ovb-manage/tasks/install_openstack_deps.yml can go in right away. no effective behavior change, only order13:16
zbrlooks at the gerrit dependency chain, they are all on the same chain. so you already tested them.13:17
sshnaidmzbr, no, it doesn't look like that.. I don't find these changes when cloning the main change13:18
sshnaidmzbr, I tested only https://review.rdoproject.org/r/#/c/21463/ why did you split it??13:19
sshnaidmzbr, we agreed to split molecule part only13:19
zbr1st: if you download it you cleargy get them. look at parents.13:20
zbrgerrit UI does not lie, I see all 4 of them listed.13:20
sshnaidmzbr, just try before arguing13:21
zbrthe only case where it may be missed is if you cherry picked it instead of cloning.13:21
sshnaidmzbr, it's very hard to work when we agree on something and you do it completely differently13:21
*** ykarel|afk has joined #oooq13:22
zbrsshnaidm: i cannot reparent ronelle change from UI13:25
zbri am asking again, what is wrong about https://review.rdoproject.org/r/#/c/21617/2/roles/ovb-manage/tasks/install_openstack_deps.yml ?13:26
sshnaidmzbr, I asked you one simple thing to do - to remove molecule parts only so that I can test it with reproducer, what's the problem to do that? Is it so difficult?13:28
chandankumarsshnaidm: https://logs.rdoproject.org/openstack-periodic-master/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-rhel-8-master-containers-build-push/3951e39/13:28
*** rascasoft has joined #oooq13:28
chandankumarsshnaidm: no files got collected related to that https://review.rdoproject.org/r/#/c/21600/22/playbooks/tripleo-ci-periodic-base/containers-push-podman-login-push.yaml13:30
chandankumargood new it does not break rhel-7 container build job13:30
zbri think we could do better without being condescending, i was asked to split stuff that were not specific to rhel8 from it, like module. I ended up splitting it in 4 changes, all in a single chain, which ends with molecule being the last one.  This made it easy to review, test and merge.13:31
zbri wonder why we cannot look at changes and evaluate them based on their content? moving a when clause at the top of block to avoid wrong indentation is just good practice.13:32
chandankumarsshnaidm: may be workspace is not got discovered13:32
sshnaidmchandankumar, let's see after a mtg13:33
chandankumarsshnaidm: sure13:33
*** aakarsh has quit IRC13:35
*** Vorrtex has joined #oooq13:38
*** amoralej|lunch is now known as amoralej13:50
*** aakarsh has joined #oooq13:58
marios|roverweshay: ping volunteer14:02
chandankumarsshnaidm: https://review.rdoproject.org/r/#/c/21625/14:03
chandankumarsshnaidm: workspace var comes from somewhere else14:03
chandankumarsshnaidm: so I need to put the correct path so that logs gets generated14:04
chandankumarthen we have logs for the same14:04
sshnaidmchandankumar, you can set workspace: {{ ansible_user_dir }}/workspace in vars above, and use it like "{{ workspace }}" later14:06
*** jfrancoa has joined #oooq14:07
sshnaidmchandankumar, like https://github.com/rdo-infra/review.rdoproject.org-config/blob/master/playbooks/tripleo-ci-periodic-base/pre.yaml#L414:07
*** ykarel|afk is now known as ykarel14:08
chandankumarsshnaidm: https://review.rdoproject.org/r/#/c/21625/14:10
chandankumarsshnaidm: once above merges, I will run it in check job for testing , is it ok?14:12
* chandankumar will be back in an hour14:14
*** panda is now known as panda|brb14:19
*** panda|brb is now known as panda14:25
*** jfrancoa has quit IRC14:31
*** jfrancoa has joined #oooq14:42
*** sabedevops has joined #oooq14:45
*** sbroderick has joined #oooq15:01
*** sabedevops has quit IRC15:01
chandankumarsshnaidm: https://review.rdoproject.org/r/#/c/21625/ can we get this review merged?15:05
sshnaidmchandankumar, +w15:05
chandankumarsshnaidm: thanks!15:06
*** sbroderick has quit IRC15:20
*** udesale has joined #oooq15:22
* marios shutdown sequence15:25
*** sabedevops has joined #oooq15:31
*** dsneddon has joined #oooq15:31
*** jfrancoa has quit IRC15:39
*** jfrancoa has joined #oooq15:39
*** udesale has quit IRC15:41
*** udesale has joined #oooq15:41
*** ykarel has quit IRC15:42
*** ykarel has joined #oooq15:42
*** derekh has joined #oooq15:54
*** jfrancoa has quit IRC15:56
*** jfrancoa has joined #oooq15:57
*** marios has quit IRC15:59
*** marios|rover has quit IRC15:59
*** bogdando has quit IRC16:04
*** vinaykns has joined #oooq16:06
*** sabedevops has quit IRC16:10
*** sabedevops has joined #oooq16:12
*** Goneri has quit IRC16:24
*** tesseract has quit IRC16:28
*** brault has quit IRC16:29
*** sabedevops has quit IRC16:37
*** jpena is now known as jpena|off16:42
*** dtantsur is now known as dtantsur|afk16:44
*** sabedevops has joined #oooq16:50
*** udesale has quit IRC16:57
*** derekh has quit IRC17:00
*** ykarel is now known as ykarel|away17:10
*** sabedevops has quit IRC17:22
*** sabedevops has joined #oooq17:36
*** sabedevops has quit IRC17:42
*** ykarel|away has quit IRC17:47
*** amoralej is now known as amoralej|off17:53
*** sabedevops has joined #oooq17:57
*** kopecmartin is now known as kopecmartin|off18:26
*** sabedevops has quit IRC18:33
*** sabedevops has joined #oooq18:34
*** sabedevops has quit IRC19:31
*** rascasoft has quit IRC19:39
*** rascasoft has joined #oooq19:44
*** brault has joined #oooq19:56
*** brault has quit IRC20:00
*** jfrancoa has quit IRC20:42
*** aakarsh has quit IRC22:14
*** tosky has quit IRC22:32
*** aakarsh has joined #oooq22:53
*** saneax has joined #oooq22:57
*** dsneddon has quit IRC23:03
*** Vorrtex has quit IRC23:07
*** dsneddon has joined #oooq23:31
*** dsneddon has quit IRC23:45

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