Tuesday, 2018-05-22

hubbotFAILING CHECK JOBS on stable/ocata: gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-ocata, tripleo-ci-centos-7-undercloud-upgrades @ https://review.openstack.org/564291, master: tripleo-ci-centos-7-3nodes-multinode, gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-ocata-branch @ https://review.openstack.org/560445, stable/queens: tripleo-ci-centos-7-containers-multinode, tripleo-ci-centos-7-3nodes- (1 more message)00:14
*** sanjay__u has quit IRC00:26
*** jtomasek has joined #oooq00:33
*** saneax is now known as saneax-_-|AFK00:55
*** jtomasek has quit IRC02:12
hubbotFAILING CHECK JOBS on stable/queens: tripleo-ci-centos-7-containers-multinode, tripleo-ci-centos-7-3nodes-multinode @ https://review.openstack.org/567224, stable/ocata: gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-ocata, tripleo-ci-centos-7-undercloud-upgrades @ https://review.openstack.org/564291, master: tripleo-ci-centos-7-3nodes-multinode, gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001 (1 more message)02:14
*** ykarel|away has joined #oooq03:42
*** udesale has joined #oooq03:50
*** links has joined #oooq04:07
*** skramaja has joined #oooq04:09
hubbotFAILING CHECK JOBS on stable/ocata: gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-ocata, tripleo-ci-centos-7-undercloud-upgrades @ https://review.openstack.org/564291, master: tripleo-ci-centos-7-3nodes-multinode, gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-ocata-branch @ https://review.openstack.org/560445, stable/queens: tripleo-ci-centos-7-3nodes-multinode, tripleo-ci-centos-7-containers- (1 more message)04:14
*** anande has joined #oooq04:20
*** ykarel|away has quit IRC04:29
*** jaganathan has quit IRC04:36
*** jaganathan has joined #oooq04:37
*** ykarel|away has joined #oooq04:45
*** saneax-_-|AFK is now known as saneax04:53
*** pgadiya has joined #oooq05:04
*** pgadiya has quit IRC05:04
*** ccamacho has quit IRC05:07
*** ratailor has joined #oooq05:17
*** quiquell|off is now known as quiquell05:38
*** gvrangan has joined #oooq05:42
*** ykarel|away is now known as ykarel05:44
*** marios has joined #oooq05:47
*** ccamacho has joined #oooq05:51
*** jaosorior has quit IRC05:57
*** jfrancoa has joined #oooq06:03
*** links has quit IRC06:08
hubbotFAILING CHECK JOBS on stable/ocata: gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-ocata, tripleo-ci-centos-7-undercloud-upgrades @ https://review.openstack.org/564291, master: tripleo-ci-centos-7-3nodes-multinode, gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-ocata-branch @ https://review.openstack.org/560445, stable/queens: tripleo-ci-centos-7-3nodes-multinode, tripleo-ci-centos-7-containers- (1 more message)06:14
*** links has joined #oooq06:26
*** tosky has joined #oooq06:50
*** kopecmartin has joined #oooq06:53
*** bogdando has joined #oooq06:57
*** ccamacho has quit IRC07:00
*** ccamacho has joined #oooq07:00
*** tesseract has joined #oooq07:22
*** zoli is now known as zoli|wfh07:23
*** zoli|wfh is now known as zoli07:24
*** ykarel is now known as ykarel|lunch07:38
*** anande has quit IRC07:53
*** links has quit IRC07:58
*** holser__ has joined #oooq08:07
*** dtantsur|afk is now known as dtantsur08:09
*** links has joined #oooq08:12
hubbotFAILING CHECK JOBS on stable/ocata: gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-ocata, tripleo-ci-centos-7-undercloud-upgrades @ https://review.openstack.org/564291, master: tripleo-ci-centos-7-3nodes-multinode, gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-ocata-branch @ https://review.openstack.org/560445, stable/queens: tripleo-ci-centos-7-3nodes-multinode, tripleo-ci-centos-7-containers- (1 more message)08:14
*** ykarel|lunch is now known as ykarel08:15
*** anande has joined #oooq08:17
*** jaosorior has joined #oooq08:18
*** anande has quit IRC08:40
ykarelis there some issue with promoter? 20b99f6998c088650b0c0cb066cc6aac3e5f9312_b333f915 is not promoted for master current-tripleo-rdo08:43
ykarelsshnaidm|rover, ^^08:43
sshnaidm|roverykarel, will check08:43
*** panda|bbl is now known as panda09:18
pandaquiquell: yesterday afternoon I could not do anything, any progress with the debugging ?09:19
quiquellpanda: Not yet09:20
quiquellGive me a minute09:20
pandaquiquell: anything I can do to help, tell me09:21
quiquellI will09:21
*** anande has joined #oooq09:22
quiquellsshnaidm|rover: At the end I am just convertin the ruck rover tool into a alerting thing and point it to sova to look deeper09:25
quiquellIt's a good pairing of tools09:25
*** ratailor has quit IRC09:26
*** ratailor has joined #oooq09:26
quiquellpanda: My reproducer has finish but not applied the zuul changes09:32
quiquellCan you help me ?09:33
pandaquiquell: yes09:33
quiquellCalling you to bj09:33
quiquellI am at your bj09:35
quiquellpanda: Tripleo-ci is sensible to zuul_changes10:03
quiquellLooking at nodepool-setup role10:03
quiquellpatch_repos.yaml10:03
pandaquiquell: looks like pas 12 works, we have logs http://logs.openstack.org/65/566565/12/check/tripleo-ci-centos-7-scenario000-multinode-oooq-container-updates/aab4d89/logs/emit_releases_file.txt.gz10:07
pandaugh, mail system is down10:10
quiquellweird, ps 12 is not changing the toci script10:10
quiquellBut if it's working we are ok10:10
quiquellGoing to check for trown's comments10:10
pandaquiquell: no but it's rebased on changes to the script10:10
quiquellLet me chek the changes10:11
quiquellA lot of stuff between 11 and 1210:14
hubbotFAILING CHECK JOBS on stable/ocata: gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-ocata, tripleo-ci-centos-7-undercloud-upgrades @ https://review.openstack.org/564291, master: tripleo-ci-centos-7-3nodes-multinode, gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-ocata-branch @ https://review.openstack.org/560445, stable/queens: tripleo-ci-centos-7-3nodes-multinode, tripleo-ci-centos-7-containers- (1 more message)10:14
quiquelljaja10:18
quiquellFoudn a big problem10:18
pandaouch10:21
pandaquiquell: details ?10:22
quiquellGive you a review in a minute10:22
chandankumarkopecmartin: arxcruz any more card waiting on me for QA?10:31
arxcruzchandankumar: i don't think so10:32
quiquellpanda: https://review.openstack.org/#/c/56995410:32
quiquellpanda: Can I add a Depends-On at your patch ?10:34
quiquellOr is better to rebase everything ?10:34
quiquellTo test it10:34
pandaquiquell: a depends on should be enough10:35
quiquellok10:35
quiquellGoing to change your commit to add it10:35
*** anande has quit IRC10:35
quiquellpanda: Also will try to address the adding the DLRN hash10:39
quiquellI suppose we have to override what's inside the release file yaml10:39
pandaquiquell: it's not enough10:39
quiquellhow so ?10:39
pandaquiquell: some of the other variables are calculated inside the release file. We discussed about this, I think we realized we need to add a variable with a default option for the hashes in the release file10:40
quiquellinside release file ?10:41
pandaquiquell: yes10:41
quiquellHumm I see know10:42
*** zoli is now known as zoli|lunch10:42
quiquellIt depens how ansible to the expansion10:42
quiquellif we have -e @foo.yaml -e bar=this10:42
quiquellIt going to use bar from foo.yaml or from command line ?10:43
pandaquiquell: IIUC when you -e @foo.yaml, everything inside the yaml file is rendered first, so if -e bar overrides a value in @foo.yml, it may be already too late to recalculate all the things in foo.yaml that use the old not overridden bar value10:45
pandaquiquell: and if we -e bar first and then -e foo last, then the bar value in foo will override the old bar value, and nothing will happen anyway10:46
pandabut I'm not 100% sure of this order of yaml rendering10:47
quiquellTesting it for learning10:47
quiquellpanda: You talk about stuff like undercloud_image_url10:47
quiquell?10:47
quiquellpanda: Has something interesting to show you10:49
quiquelllooks like10:49
quiquellansible-playbook -e @queens-undercloud-newton-overcloud.yml  -e dlrn_hash_tag=foo foo.yaml10:50
quiquellok: [localhost] => {10:50
quiquell    "msg": "value: https://images.rdoproject.org/queens/delorean/foo/undercloud.qcow2"10:50
quiquell}10:50
quiquellLooks like we can override the variables inside the release file10:50
quiquellSo adding them after should be enough10:50
pandaquiquell: mmmhh, so the ordering I understood was not correct10:50
quiquellJust test it10:51
quiquellGoing to try a patchset at your commit10:51
pandaquiquell: need to find a document with the exact ordering at this point. The test can go on and on for days :)10:51
quiquellI think it make sense10:51
quiquellIf you think about it10:51
quiquellIt's done in two steps10:51
quiquellansible first set variables to values10:52
quiquellthen it expand the templates10:52
quiquellgoing like the latest stuff i did we get right the first step10:52
pandaquiquell: it's not that simple I'm afraid, and if you ever coded for junja templates you know why10:52
quiquellpanda: Is not simple enough for the relases file ?10:53
pandaquiquell: sometimes you need more than one pass at the same template to exhaust all the possible variables10:53
quiquellOk, going to add a new review to test it10:53
quiquellWe can squash it if we want later on10:53
pandaquiquell: yes, mI mean, it's not simple to understand exacly what ansible does when rendering templates10:53
quiquellDamn... I am beign so naif10:53
pandabecause it will need more tha one passes to render everything correctly10:54
pandaquiquell: ??10:54
quiquellpanda: Let's throw a review and see what happends10:55
quiquellWe will never know if it's broken though...10:56
pandaquiquell: I too like to live dangerously10:56
quiquellhehe10:57
quiquellpanda: going to remove the variable from the release file and see if it works in the review11:00
quiquelljust in the review11:00
chandankumarpanda: arxcruz can we remove fs021 rdo ci job from python-tempestconf and tempest?11:00
chandankumarsince it is no longer monitored i think11:00
*** jaosorior has quit IRC11:01
*** jaosorior has joined #oooq11:02
pandachandankumar: I think this was one of the tempest squad action items, remove fs21 from pike and master jobs and create another fs for the pike an master job to test containerized deployments11:03
chandankumarkopecmartin: arxcruz ^^11:04
quiquellpanda: Going to test on this11:09
quiquellhttps://review.openstack.org/56995811:09
chandankumarpanda: regarding above review, since cinder v1 api is deprecated long time ago, it would not be a problem11:11
chandankumarsince jun11:11
chandankumar*juno11:11
pandachandankumar: ok, removing the -1, thanks11:13
pandachandankumar: approved11:19
chandankumarpanda: thanks :-)11:19
*** panda is now known as panda|lunch11:19
*** moguimar has quit IRC11:25
*** moguimar has joined #oooq11:25
weshaysshnaidm|rover, hey want to try and double check me on the bm master config?11:33
sshnaidm|roverweshay, ok11:33
weshayhttps://code.engineering.redhat.com/gerrit/#/c/139395/11:34
weshaysshnaidm|rover, that was my first pass11:35
weshaysshnaidm|rover, if you are not that familiar w/ these boxes we can chat about it11:35
weshaysshnaidm|rover, is there something more for https://review.openstack.org/#/c/569854/11:36
chandankumarpanda|lunch: for fs021 for containerized deployment didnot get that part11:36
*** ratailor has quit IRC11:36
sshnaidm|roverweshay, 569854  569930 569932 are ready to go11:39
sshnaidm|roverweshay, I don't know these boxes, but looks ok for me..11:40
weshaysshnaidm|rover, ya.. essentially we need restructure the configs so they can just take a featureset config11:40
*** dtantsur is now known as dtantsur|afk11:59
*** zoli|lunch is now known as zoli12:01
*** moguimar has quit IRC12:01
*** zoli is now known as zoli|wfh12:01
*** zoli|wfh is now known as zoli12:01
*** moguimar has joined #oooq12:01
*** moguimar has quit IRC12:02
*** amoralej is now known as amoralej|lunch12:04
*** moguimar has joined #oooq12:09
*** moguimar_ has joined #oooq12:11
*** moguimar_ has quit IRC12:12
*** moguimar_ has joined #oooq12:12
*** moguimar has quit IRC12:14
hubbotFAILING CHECK JOBS on stable/ocata: tripleo-ci-centos-7-undercloud-upgrades @ https://review.openstack.org/564291, master: tripleo-ci-centos-7-3nodes-multinode @ https://review.openstack.org/56044512:14
*** panda|lunch is now known as panda12:16
weshay\0/12:30
weshaycheck out that hubbot status :))12:30
weshaysshnaidm|rover, myoung|ruck ^12:30
chandankumarweshay: arxcruz sshnaidm|rover https://review.rdoproject.org/r/13851 please have a look at this one12:32
*** trown|outtypewww is now known as trown12:34
quiquellTrown have been checking the dlrh_hash thing12:43
quiquellGood morning btw12:43
quiquellhttps://review.openstack.org/#/c/569958/12:44
trownquiquell: yep just checking all those reviews, thanks for being active on that12:44
quiquellDoing a naif testing feels to me that ansible is usable but panda has concerns12:44
quiquelltrown: Np let's push this a little12:44
quiquellI have check that doing ansible-playbook -e @foo.yaml -e bar=blah12:45
trownquiquell: put comments on 569958 ... I think we have to workaround some tech debt theree12:45
trownthere12:45
quiquellfoo.yaml template get expanded with bar=blah12:45
trownya I think the way you did that is fine, just need to use a different (kind of obscure) var to ensure we get the dlrn hash we intend in all cases12:46
quiquelltrown: holy sh... get_build_command :-)12:47
trownthere is logic around mixed upgrades that will override any other settings12:47
quiquellI was not even close hehe12:47
panda"holy shell"12:47
quiquellExactly that panda12:48
trownso eventually we want to rip out that logic, since we have moved it to python... but until then we can use the other var to workaround our other workarounds12:48
panda"holy shift"12:48
trownyou are welcome panda :P12:48
trownETOOMANYWORKAROUNDS12:48
quiquellWorkarounds enchilada12:49
pandayo dawg12:49
* trown may have to actually make a yo dawg workaround meme12:49
quiquelltrown, panda: The hash is part of this sprint ?12:49
trownquiquell: ya12:49
quiquellok12:49
trownupdates are part of the sprint, and the hash is needed for updates12:50
pandayep, we need to have at least a workable output12:50
quiquellOk, let's test the get_build_command12:51
myoung|ruckback12:52
myoung|ruck%gatestatus12:52
hubbotFAILING CHECK JOBS on stable/ocata: tripleo-ci-centos-7-undercloud-upgrades @ https://review.openstack.org/564291, master: tripleo-ci-centos-7-3nodes-multinode @ https://review.openstack.org/56044512:52
* myoung|ruck rubs his eyes in disbelief12:52
myoung|ruckpanda, weshay, we on for syn in 7m?12:53
myoung|rucksync*12:53
quiquellpanda, trown: btw, still not getting why reproducer is not checking the correct zuul-change of tripleo-ci12:53
quiquellThe role and nodepool-setup is trying it12:53
quiquellNeed to debug that12:53
*** moguimar_ is now known as moguimar12:53
quiquellThat's another story12:53
pandamyoung|ruck: aye, sorry about yesterday12:53
weshaymyoung|ruck, yup12:53
quiquelltrown: I think we have to pash both get_build_command and dlrn_hash12:55
quiquellthere are templatized stuff in the release file12:55
trownget_build_command will override it all for containers12:55
trownbut you might be right for repos12:56
quiquellThere is stuff like this12:56
quiquellundercloud_image_url: https://images.rdoproject.org/queens/delorean/{{ dlrn_hash|default(dlrn_hash_tag) }}/undercloud.qcow212:56
quiquellipa_image_url: https://images.rdoproject.org/queens/delorean/{{ dlrn_hash|default(dlrn_hash_tag) }}/ironic-python-agent.tar12:56
quiquell12:56
quiquellin the release file12:56
trownquiquell: ya so we need both12:57
quiquellOk, adding them12:57
weshaydam yo12:58
weshay 12:37:52 up 593 days, 21:07,  1 user,  load average: 0.00, 0.00, 0.0012:58
pandawow, not kernel security updates for the past two years12:59
weshaypanda, meh13:00
weshayinternal box13:00
quiquellweshay: Looks like a skyrim steam stats13:04
*** amoralej|lunch is now known as amoralej13:11
*** skramaja has quit IRC13:19
quiquelltrown: Added both https://review.openstack.org/#/c/56995813:26
*** holser__ has quit IRC13:26
*** holser__ has joined #oooq13:27
*** gvrangan has quit IRC13:44
quiquellGuys do you know why this openstack image list --long -f value -c ID  --property latest=centos-7-latest13:48
quiquellGive me two images ?13:48
quiquell| 28b76dd3-4017-4b46-8dc9-98ef1cb4034f | CentOS-7-x86_64-GenericCloud-1801-01      | active |13:48
quiquell| fdfd5d39-a76d-45df-abec-17f768ba3054 | CentOS-7-x86_64-GenericCloud-1804_02      | active |13:48
quiquellOk a problem with rdo cloud13:49
trownya they shouldnt both have latest property i think13:50
quiquellWe cannot unset that ?13:52
*** quiquell_ has joined #oooq13:54
*** quiquell_ has quit IRC13:54
trownchecking who owns the images13:54
quiquelltrown: Do you have both with latest at your cloud ?13:55
trownquiquell: ya they are owned by14:00
trown8dbc460fe5fd4fcab0096c2c0aad3ece14:00
quiquelltrown: dmanchado is fixing it14:01
trownwhich is not openstack-nodepool... not sure which user it is, but infra would have to modify the images14:01
trownk14:01
quiquellBut reproducer is fuck up rigth now14:01
quiquellAlso RDO jobs will be I think14:01
*** moguimar has quit IRC14:05
*** quiquell is now known as quiquell|off14:06
*** zoli is now known as zoli|afk14:07
*** moguimar has joined #oooq14:09
rascamyoung|ruck, hey man -> https://code.engineering.redhat.com/gerrit/#/c/139505/ <- does this makes sense?14:10
*** moguimar_ has joined #oooq14:10
*** moguimar has quit IRC14:13
hubbotFAILING CHECK JOBS on stable/ocata: tripleo-ci-centos-7-undercloud-upgrades @ https://review.openstack.org/564291, master: tripleo-ci-centos-7-3nodes-multinode @ https://review.openstack.org/56044514:15
*** matbu has joined #oooq14:16
*** moguimar_ is now known as moguimar14:18
*** ccamacho has quit IRC14:19
rascabandini, I think we might want to skip today's triage call, there are btw 2 bugs not triaged in the list if you want to take a look: https://bugzilla.redhat.com/show_bug.cgi?id=1564654 and https://bugzilla.redhat.com/show_bug.cgi?id=157614814:24
openstackbugzilla.redhat.com bug 1564654 in puppet-pacemaker "OSP13: Overcloud deployment fails when using capital letters in customized stack name ( --stack TEST-STACK34 )." [High,New] - Assigned to rhos-maint14:24
openstackbugzilla.redhat.com bug 1576148 in pacemaker "Upper case letters in HostnameFormat parameters cause trouble" [Urgent,Post] - Assigned to kgaillot14:25
rascasorry folks, wrong channel :)14:25
*** jtomasek has joined #oooq14:29
*** ccamacho has joined #oooq14:31
*** ccamacho has quit IRC14:31
*** ccamacho has joined #oooq14:32
*** saneax is now known as saneax-_-|AFK14:36
chandankumartosky++14:46
hubbotchandankumar: tosky's karma is now 114:46
*** matbu has quit IRC14:49
*** matbu has joined #oooq14:58
*** zoli|afk is now known as zoli15:01
*** zoli is now known as zoli|wfh15:02
*** zoli|wfh is now known as zoli15:02
*** ykarel has quit IRC15:03
*** moguimar has quit IRC15:07
*** moguimar has joined #oooq15:09
weshaysshnaidm++ re: https://review.openstack.org/#/c/569932/15:13
*** udesale has quit IRC15:14
*** udesale has joined #oooq15:15
*** rlandy has joined #oooq15:19
rlandypanda: hi - what did I miss at Monday's meeting?15:21
*** yolanda has quit IRC15:21
weshayrlandy,!!15:23
weshayrlandy, need you expertise when you have a minute15:23
rlandyweshay: hi - sure - I'm here15:23
rlandywhat's up?15:23
pandarlandy: the script still needs some adjustment to work properly, we've had no output in some patchset , but looking good now. We have no decision on the env part of the dry run, but moving on to close the review today.15:24
rlandypanda: we can just remove that env dump15:25
rlandyor make it dump out env vars only with dry run15:25
*** yolanda has joined #oooq15:25
rlandyI don't want to hold the sprint work because of it15:25
rlandylet me know what you think and I can update that patch accordingly15:26
*** Goneri has joined #oooq15:26
weshayrlandy, ok.. when you have 10min15:27
weshayI need you on blue15:27
pandarlandy: I think your patch is on scope, stated the same thing in the last comment. waiting for more advantage/disadvantage analysis before calling a decision15:27
myoung|ruckrasca: merged https://code.engineering.redhat.com/gerrit/#/c/139505 Include validate ha logs in ha environments15:27
rlandyweshay: k - can join now15:28
weshayk15:29
rascamyoung|ruck, oh thanks! So I did right, good to know!15:30
myoung|ruckrasca: aye...i have an action item to create an RFE tracking LP to add the "append to collected log list" patch as well...but for now what you did in that review should work today w/ current collect-logs implementation (both in tripleo-env as well as current upstream roles)15:31
rascamyoung|ruck, but don't you think that even if we find a way to append instead of a full re declaration we will in any case need to have a specific yaml like in my case?15:32
*** jtomasek has quit IRC15:32
myoung|ruckrasca: yes, but it would avoid the duplication of the base array (and the drift it incurrs), and ideally you could just park the config anywhere...vs. in a discreet yaml15:33
* myoung|ruck shrugs15:33
weshayrlandy, https://github.com/openstack/tripleo-quickstart/blob/master/config/general_config/featureset001.yml#L138-L14315:34
*** Goneri has quit IRC15:35
*** moguimar has quit IRC15:36
rlandyweshay: https://github.com/openstack-infra/tripleo-ci/blob/master/toci-quickstart/config/testenv/ovb.yml#L3015:36
*** moguimar has joined #oooq15:36
*** links has quit IRC15:37
sshnaidm|roverpanda, didn't we agree yesterday to come to one decision?15:40
* myoung|ruck will bbiaf15:40
pandasshnaidm|rover: yes15:41
sshnaidm|roverpanda, I mean together15:41
weshayrlandy, http://paste.openstack.org/show/721634/15:42
*** moguimar has quit IRC15:43
pandasshnaidm|rover: we've not decided how, I asked to add more advantages/disadvantages on the current solution with a comment on the patch.15:44
*** moguimar has joined #oooq15:46
sshnaidm|roverpanda, and when did you want to discuss that?15:49
*** yolanda has quit IRC15:49
*** moguimar has quit IRC15:50
*** moguimar has joined #oooq15:52
pandasshnaidm|rover: so at this point in the sprint, I think we have two choices: Merge this as it is, or remove the env dump part. We've discussed it without getting to a solid conclusion. What's left to do for me is to list all the advantages, all the disadvantages and see what decision makes more sense.15:52
*** matbu has quit IRC15:54
sshnaidm|roverpanda, mmm... I thought to take it offline for discussion mean to gather all interested parties and come to something that all agree on it...15:55
sshnaidm|roverpanda, but seems like I was wrong15:55
*** bogdando has quit IRC15:58
pandasshnaidm|rover: we could do it, maybe not at this point on the sprint, and not before having a list of advantages and disadvantages. We had discussions, even monday. I think the only discussion we can do at this point is if the advantages make more sense than the disadvantages, so we can have a decision on some bullet points.16:00
pandaweshay: I have up to 1h till EOD now. Hard stop at 1h16:01
weshaypanda, k.. I need 5 min w/ rlandy16:02
weshayalmost done16:02
weshayrlandy, myoung|ruck https://code.engineering.redhat.com/gerrit/139519 fyi16:03
weshaythat's WIP btw16:06
sshnaidm|roverpanda, it doesn't explain for me why you don't want to keep agreements, but nevermind16:09
sshnaidm|roverrlandy, if you remove dump of env variables, I can handle it in my patch, anyway I have already list of variables to manage. I'll update that list with vars you introduced in last patches and also will add them to collect-logs function that anyway needs them.16:10
hubbotFAILING CHECK JOBS on stable/ocata: tripleo-ci-centos-7-undercloud-upgrades @ https://review.openstack.org/564291, master: tripleo-ci-centos-7-3nodes-multinode @ https://review.openstack.org/56044516:15
weshaypanda, ready16:15
rlandysshnaidm|rover: was in meeting - looking at the last comments on that patch16:16
*** tesseract has quit IRC16:17
*** marios has quit IRC16:19
*** ccamacho has quit IRC16:23
*** Goneri has joined #oooq16:31
rlandyweshay: left a comment on https://code.engineering.redhat.com/gerrit/#/c/139519 - we should only keep network related stuff in single-nic16:33
*** zoli is now known as zoli|gone16:33
*** zoli|gone is now known as zoli16:33
rlandypanda; sshnaidm|rover: I missed the discussion on monday  - and at this point, I'd rather not hold up the entire sprint over dumping the variables16:37
rlandypanda; sshnaidm|rover: my suggestion is that we dump the vars only in a dry run16:38
rlandythat way we have the debug capability16:38
rlandyreally - I think it's more important to get this patch merged16:39
EmilienMfor CI team: https://review.openstack.org/#/c/569902/116:43
sshnaidm|roverrlandy, fine with me16:44
rlandysshnaidm|rover: thanks - we can rethink it with your patch - I'd k=just like to get the other work in that patch in. waiting on panda's ok16:46
*** trown is now known as trown|lunch16:47
sshnaidm|roverEmilienM, we don't need this16:47
*** jfrancoa has quit IRC16:47
*** matbu has joined #oooq16:47
*** jtomasek has joined #oooq16:48
*** kopecmartin has quit IRC16:48
EmilienMsshnaidm|rover: oops ok16:49
*** Goneri has quit IRC16:50
pandarlandy: ack16:55
*** panda is now known as panda|off16:59
rlandyok - here we go16:59
weshayEmilienM, wtf.. you are supposed to be at SUMMIT16:59
weshayhrm.. failed on prep-cont https://rhos-dev-jenkins.rhev-ci-vms.eng.rdu2.redhat.com/job/tripleo-quickstart-master-rdo_trunk-baremetal-hp_dl360_envE-single_nic_vlans/123/console17:01
*** dtantsur|afk is now known as dtantsur17:07
*** jtomasek has quit IRC17:08
rlandysshnaidm|rover: for your review again pls :) https://review.openstack.org/#/c/567060/17:15
rlandylooking at envE job17:15
weshayrlandy, https://thirdparty.logs.rdoproject.org/jenkins-tripleo-quickstart-master-rdo_trunk-baremetal-hp_dl360_envE-single_nic_vlans-123/undercloud/home/stack/overcloud_prep_containers.log.txt.gz17:15
weshayso the containers are being pushed to undercloud_network_gateway: 10.12.150.19017:16
weshaydoes that seem right?17:16
rlandyI don't think so17:17
rlandywhy there?17:17
rlandylocal_docker_registry_host17:18
weshay2018-05-22 16:48:05 | ImageUploaderException: Could not push image 10.12.150.190:8787/tripleomaster/centos-binary-rsyslog-base17:18
rlandyits' defaulted17:18
rlandywe need to override17:18
*** matbu has quit IRC17:19
rlandyhttps://github.com/openstack/tripleo-quickstart-extras/blob/master/roles/extras-common/defaults/main.yml#L717:19
rlandyundercloud_network_cidr|nthhost(1)17:19
weshayya {undercloud_network_gateway|default(undercloud_network_cidr|nthhost(1))17:21
weshayrlandy, what ip should it hit?17:21
weshay10.12.150.252 >17:22
weshay?17:22
*** holser__ has quit IRC17:22
rlandyit would need to go to the virthost17:22
rlandyundercloud_undercloud_public_vip: 10.12.150.130?17:23
rlandyundercloud_local_ip: 10.12.150.129/2617:24
rlandy129 or 13017:24
rlandylet's check the env17:24
rlandywe need to see what is on the eno1 on the vm17:24
weshayya.. I think 10.12.150.13017:25
rlandyor better yet, define (undercloud_network_cidr17:25
weshayI can't hit the undercloud w/ the ip in the ss17:25
rlandythat is fine17:29
rlandyHost undercloud-root17:29
rlandyit asks me for password17:30
weshayya.. I can get to the host17:38
weshayrlandy, odd17:38
rlandythat file should work17:39
rlandythat is for sure the latest workspace?17:40
rlandyweshay: we cab get by this bu defining a undercloud_network_cidr17:41
weshayhttps://rhos-dev-jenkins.rhev-ci-vms.eng.rdu2.redhat.com/job/tripleo-quickstart-master-rdo_trunk-baremetal-hp_dl360_envE-single_nic_vlans/ws/17:41
myoung|ruckweshay: kicking pike want me to kick pike for the new hash (promote would flip yellow to green for pike rdo2) - but leaving envE disabled17:41
rlandyno such var defined right now17:42
weshaymyoung|ruck, focus on osp1317:42
rlandythat would solve the problem of defaulting to the gateway17:42
weshayoh17:42
weshayright.. we could just update the config.. and rekick17:42
rlandycorrect - although17:43
weshayw/ my guess is 13017:43
rlandyundercloud_network_gateway17:43
rlandyif that is defined17:43
rlandyit will use it instead of the default17:43
weshayright17:43
rlandyso we will have to override17:43
weshayrlandy, let's add that to our latest change17:43
rlandyexplictly17:43
weshayexplicitly17:43
weshayyas17:44
rlandyweshay: sure17:44
weshayyes17:44
weshayrlandy, merge it and retry17:44
rlandyweshay:I should? or you will?17:44
* rlandy has trouble with the tenses in IRC17:44
weshayrlandy, you better drive from here https://code.engineering.redhat.com/gerrit/#/c/139519/17:44
weshaygo ahead and make the changes you think are needed17:45
rlandyweshay: ack - ok17:45
weshayI'm going to have to take my kid to the doctor before our trip :(17:47
weshayI'm loosing time :)17:48
weshaytrown|lunch, fyi.. we're all confirmed re: interviews. Thanks again17:48
*** Goneri has joined #oooq17:49
*** matbu has joined #oooq17:49
*** ccamacho has joined #oooq17:53
*** links has joined #oooq18:00
rlandyhmmm ... Already running 1 builds across all nodes18:01
*** trown|lunch is now known as trown18:06
trownrlandy: do you think you could look at https://review.openstack.org/569115 ... last of the python script patches18:09
trownstill a couple more tripleo-ci patches for wiring in the script though18:10
rlandylooking18:10
*** udesale has quit IRC18:12
hubbotFAILING CHECK JOBS on stable/ocata: tripleo-ci-centos-7-undercloud-upgrades @ https://review.openstack.org/564291, master: tripleo-ci-centos-7-3nodes-multinode @ https://review.openstack.org/56044518:15
rlandytrown; I also updated https://review.openstack.org/#/c/567060/ to get rid of the 'dump env vars' controversy18:22
ccamachoguys quick question, I pushed https://review.openstack.org/#/c/569991/ to test in oooqextras the undercloud backup/restore, I have a depends on to oooqto set a variable to true and execute the playbook18:23
ccamachoam I missing something there?18:23
ccamachoto actually call the playbook?18:24
rlandymyoung|ruck: is env E still disabled?18:26
rlandya run kicked on it18:26
*** amoralej is now known as amoralej|off18:26
*** sanjay__u has joined #oooq18:28
rlandyccamacho: depending where you want that playbook to be included (which workflow - multinode, ovb) , you will need to add it to https://github.com/openstack/tripleo-quickstart-extras/blob/master/playbooks/multinode.yml or https://github.com/openstack/tripleo-quickstart-extras/blob/master/playbooks/baremetal-quickstart-extras.yml (which gets kicked with https://github.com/openstack/tripleo-quickstart-extras/blob/master/playbooks/ba18:30
rlandyremetal-full-deploy.yml)18:30
myoung|ruckrlandy: master is enabled, the rest disabled.  kill the queens run18:31
myoung|ruckit kicked before i disabled...killing it now18:32
myoung|ruckahh it's already dead :)18:33
myoung|ruckrlandy: cool https://code.engineering.redhat.com/gerrit/#/c/139519 merged, master running18:34
* myoung|ruck is working on rhos-13 gates18:34
rlandymyoung|ruck: thanks18:36
rlandydidn't want to kill the wrong thing18:36
ccamachorlandy I want to run it in this job tripleo-ci-centos-7-undercloud-oooq18:39
*** dtantsur is now known as dtantsur|afk18:40
rlandyplaybooks/multinode-undercloud.yml18:44
myoung|ruckweshay: fyi rhos-13 gates are looking promising (don't want to jinx it) - at least we appear to be past hostname issue.18:53
myoung|ruck2018-05-22 14:21:14 | ++ hostnamectl --static18:53
myoung|ruck2018-05-22 14:21:14 | + hostnamectl set-hostname --transient undercloud-75595.novalocal18:53
myoung|ruck2018-05-22 14:21:14 | ++ hostnamectl --static18:53
myoung|ruck2018-05-22 14:21:14 | + hostnamectl set-hostname --static undercloud-75595.novalocal18:53
myoung|ruckweshay: ^^ note we don't have the non-fqdn hostnames (like before)18:54
* myoung|ruck requests positive vibes --> https://rhos-dev-jenkins.rhev-ci-vms.eng.rdu2.redhat.com/job/tq-gate-rhos-13-ci-rhos-ovb-minimal-pacemaker-public-bond/318:54
*** matbu has quit IRC18:56
rlandymyoung|ruck: rhos-13 - so exciting!!18:57
myoung|ruckrlandy: aye we have a new puddle as of yesterday too18:57
myoung|ruckwe were getting hung up (before) on having the jobs fail on preinstall validation checks for a FQDN hostname18:58
*** links has quit IRC19:01
*** matbu has joined #oooq19:03
*** sshnaidm|rover is now known as sshnaidm|off19:04
*** Goneri has quit IRC19:08
*** matbu has quit IRC19:09
*** dtantsur|afk has quit IRC19:33
rlandywhat's happening with this patch? https://review.openstack.org/#/c/566565/19:37
rlandytrown: ^^ any idea. looks like there is still work to do here.19:37
* rlandy missed monday's discussion19:37
rlandyI need to rebase my patch on this work19:38
trownrlandy: I think that patch is good to go, was waiting to see the patches above work19:39
trownrlandy: the updates job on https://review.openstack.org/569958 is of particular interest19:39
trownrlandy: if that works, I think we are good to go19:39
rlandyk - there were a bunch of comments on adding in the dlrn hash19:40
trownrlandy: ya it is is done in the patch I linked ^, updated my comment to a +2 with explanation that it doesnt change current behavior wrt updates and can be merged19:42
hubbotFAILING CHECK JOBS on stable/ocata: tripleo-ci-centos-7-undercloud-upgrades @ https://review.openstack.org/564291, master: tripleo-ci-centos-7-3nodes-multinode @ https://review.openstack.org/56044520:15
*** jtomasek has joined #oooq20:18
*** d0ugal_ has joined #oooq20:19
*** d0ugal has quit IRC20:21
myoung|ruckweshay, rlandy, fyi we're well into OC deploy on TQ rhos-13 gates.  I need to dash to fetch mini-me.  will be back later to check on things.20:23
rlandymyoung|ruck++20:24
hubbotrlandy: myoung|ruck's karma is now 120:24
rlandymyoung|ruck: debugging the envE job20:24
rlandypls keep disabled for now20:24
myoung|ruckrlandy: ack, not messing with it20:24
* myoung|ruck is really gone now :)20:24
*** jtomasek has quit IRC20:47
weshayrlandy, ok.. same issue but at least we can get to the undercloud now :)20:48
weshayundercloud_public_host = 10.12.150.13020:49
rlandyweshay: I am testing locally with 10.12.150.12920:49
rlandylooks better20:49
weshaycool :)20:49
rlandyif that finishes will merge and rekick20:50
weshayword up20:50
rlandyguess at which ip it wants :)20:50
rlandyCompleted upload for image docker.io/tripleomaster/centos-binary-swift-object:e8480e465826874c7851e6bc5672c9cbdee1cdbb_c19a0cf920:51
rlandyCompleted upload for image docker.io/tripleomaster/centos-binary-swift-container:e8480e465826874c7851e6bc5672c9cbdee1cdbb_c19a0cf920:51
*** jtomasek has joined #oooq20:52
weshayheh20:58
weshayrlandy++20:58
hubbotweshay: rlandy's karma is now 320:58
rlandyweshay: yeah - looks ok - pushing change21:01
*** trown is now known as trown|outtypewww21:06
rlandylucky job 125?21:14
*** jtomasek has quit IRC21:14
weshaylady luck21:17
*** jtomasek has joined #oooq21:34
weshayrlandy, thanks for the help!!21:40
weshayI'm around but renicking ask I'll be in and out21:41
*** weshay is now known as weshay_pto21:41
rlandyweshay_pto: sure - enjoy your pto21:42
*** tosky has quit IRC21:49
*** jtomasek_ has joined #oooq22:05
*** d0ugal_ has quit IRC22:06
*** jtomasek has quit IRC22:06
*** d0ugal_ has joined #oooq22:08
*** matbu has joined #oooq22:14
hubbotFAILING CHECK JOBS on stable/ocata: tripleo-ci-centos-7-undercloud-upgrades @ https://review.openstack.org/564291, master: tripleo-ci-centos-7-3nodes-multinode @ https://review.openstack.org/56044522:15
*** jtomasek_ has quit IRC22:20
myoung|ruckweshay_pto: enjoy time away!22:36
*** dtrainor has joined #oooq22:44
*** dtrainor has quit IRC23:07
rlandywell, we git as far as a deploy failure23:14
rlandyprogress23:14
rlandychecking logs23:15
*** matbu has quit IRC23:43
rlandydeploy arguments are complicated23:45
rlandywe never pass single nic23:51

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