Friday, 2018-05-18

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-pike-branch, gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-queens-branch, gate-tripleo-ci-centos-7-ovb-3ctlr_1comp- (2 more messages)00:13
*** rlandy has quit IRC00:35
*** agopi has joined #oooq01:02
*** agopi has quit IRC02:07
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-pike-branch, gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-queens-branch, gate-tripleo-ci-centos-7-ovb-3ctlr_1comp- (2 more messages)02:13
hubbotFAILING CHECK JOBS on stable/queens: tripleo-ci-centos-7-3nodes-multinode, gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-queens, gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset035-queens, tripleo-ci-centos-7-containers-multinode @ https://review.openstack.org/567224, stable/ocata: gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-ocata, tripleo-ci-centos-7-undercloud-upgrades @  (2 more messages)04:13
*** ykarel has joined #oooq04:41
*** ykarel_ has joined #oooq04:43
*** ykarel_ is now known as ykarel1105:13
*** marios has joined #oooq05:13
*** ykarel11 is now known as ykarel_05:13
*** pgadiya has joined #oooq05:14
*** pgadiya has quit IRC05:14
*** quiquell|off is now known as quiquell05:33
*** jfrancoa has joined #oooq05:33
*** jfrancoa has quit IRC05:35
*** jfrancoa has joined #oooq05:38
*** ratailor has joined #oooq05:44
quiquellsshnaidm|bbl: Let's try to merge this https://review.openstack.org/#/c/567320/05:51
hubbotFAILING CHECK JOBS on stable/queens: tripleo-ci-centos-7-3nodes-multinode, gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-queens, gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset035-queens, tripleo-ci-centos-7-containers-multinode @ https://review.openstack.org/567224, stable/ocata: gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-ocata, tripleo-ci-centos-7-undercloud-upgrades @  (2 more messages)06:13
*** ccamacho has quit IRC06:22
*** sshnaidm|bbl is now known as sshnaidm|off06:23
*** ccamacho has joined #oooq06:40
sshnaidm|offquiquell, a few comments there06:41
*** holser__ has joined #oooq06:45
*** matbu has joined #oooq06:46
sshnaidm|offquiquell, I see you did scripts for zuul queues06:48
sshnaidm|offquiquell, well, I did them too, that's why I wrote in etherpad "scripts are done" :)06:48
*** florianf has joined #oooq06:49
*** kopecmartin has joined #oooq06:50
quiquellsshnaidm|off: Yep, we can replace one for another, Just wanted to have something running06:52
quiquellAnd see how gauge works in grafana06:53
*** saneax-_-|AFK is now known as saneax07:01
*** tesseract has joined #oooq07:14
*** skramaja has joined #oooq07:15
*** jtomasek has joined #oooq07:19
*** skramaja has quit IRC07:24
*** skramaja_ has joined #oooq07:24
*** amoralej|off is now known as amoralej07:48
*** bogdando has joined #oooq07:53
*** kopecmartin has quit IRC07:54
*** kopecmartin has joined #oooq07:56
quiquellsshnaidm|off: Sagi, get_dlrn_hash is still not used in the main()07:57
quiquellMaybe we can try to merge this and do the fix in the wiring07:57
quiquellSo we start to reduce the commit's maze07:57
quiquellSo we can merge the integration for upgrades07:58
quiquellWhat do you think ?07:58
sshnaidm|offquiquell, I don't like merging breakable code tbh07:58
sshnaidm|offthe fixes are trivial07:58
quiquellsshnaidm|off: Ok, Ãlet's add a PS to it, they are going to kill me07:59
quiquellsshnaidm|off: You are right, let's catch t he problems in the origin07:59
*** tosky has joined #oooq08:02
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-pike-branch, gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-queens-branch, gate-tripleo-ci-centos-7-ovb-3ctlr_1comp- (2 more messages)08:13
*** gkadam has joined #oooq08:18
quiquellsshnaidm|off: Fixed https://review.openstack.org/#/c/56732008:21
sshnaidm|off+208:22
quiquellCool08:23
*** ykarel_ is now known as ykarel|lunch08:25
quiquellsshnaidm|off: Do you know why https://review.openstack.org/#/c/567649 is not running gate jobs ?08:27
sshnaidm|offquiquell, non merged dependencies?08:28
quiquellsshnaidm|off: We will have to rebase everything...08:29
*** arxcruz has quit IRC08:30
*** lucas-afk is now known as lucasagomes08:32
quiquellDamn they are going to kill me08:43
quiquellsshnaidm|off: Do you have the zuul's queue script around ?09:05
*** arxcruz has joined #oooq09:23
*** ykarel|lunch is now known as ykarel|09:27
*** anande has joined #oooq09:31
*** zoli is now known as zoli|lunch09:33
*** panda|off is now known as panda09:37
quiquellpanda: Good morning09:39
quiquellWe need a +2 here https://review.openstack.org/#/c/56732009:39
quiquellDid have to add a PS09:40
quiquellWe will have to rebase all the commits... I think09:40
*** skramaja_ is now known as skramaja10:13
hubbotFAILING CHECK JOBS on stable/queens: tripleo-ci-centos-7-3nodes-multinode, gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-queens, gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset035-queens, tripleo-ci-centos-7-containers-multinode @ https://review.openstack.org/567224, stable/ocata: gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-ocata, tripleo-ci-centos-7-undercloud-upgrades @  (2 more messages)10:13
quiquellThanks panda, let get this thing merge and continue with the rest of the maze10:19
*** sshnaidm|off has quit IRC10:41
*** saneax is now known as saneax-_-|AFK10:43
*** zoli|lunch is now known as zoli11:01
*** atoth has quit IRC11:12
*** sshnaidm|off has joined #oooq11:12
*** panda is now known as panda|lunch11:14
*** quiquell is now known as quiquell|lunch11:27
*** rfolco has joined #oooq11:46
*** jaosorior has quit IRC11:54
*** atoth has joined #oooq12:00
*** anande has quit IRC12:10
*** jaganathan has quit IRC12:12
hubbotFAILING CHECK JOBS on stable/queens: tripleo-ci-centos-7-3nodes-multinode, gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-queens, gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset035-queens, tripleo-ci-centos-7-containers-multinode @ https://review.openstack.org/567224, stable/ocata: gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-ocata, tripleo-ci-centos-7-undercloud-upgrades @  (2 more messages)12:13
*** EmilienM is now known as EvilienM12:21
*** trown|outtypewww is now known as trown12:24
*** rlandy has joined #oooq12:35
*** holser__ has quit IRC12:47
*** holser__ has joined #oooq12:48
*** quiquell|lunch is now known as quiquell12:55
*** amoralej is now known as amoralej|lunch12:55
quiquelltrown: https://review.openstack.org/#/c/567320/ at the gates :-)12:56
trownquiquell: ya thanks for getting sagi +212:57
rlandyweshay: 1-on-1?13:03
weshayrlandy, sorry .. 2 secs13:04
rlandyweshay: no rush13:04
*** ratailor has quit IRC13:08
weshayrlandy, k.. ready13:09
*** dtantsur is now known as dtantsur|brb13:11
weshayrlandy, <EmilienM> mwhahaha, weshay : containerized undercloud upgrade is back green http://logs.openstack.org/41/567641/5/check/tripleo-ci-centos-7-containerized-undercloud-upgrades/10becf2/logs/undercloud/home/zuul/undercloud_upgrade.log.txt.gz13:19
weshayhttps://review.openstack.org/#/c/567641/13:19
*** ratailor has joined #oooq13:21
*** ratailor has quit IRC13:25
*** myoung|ruck|off is now known as myoung|ruck13:37
chandankumarDone for the day, heading home early, see ya on monday13:41
*** bogdando has quit IRC13:42
*** holser__ has quit IRC13:43
weshaychandankumar, have a good weekend13:43
*** amoralej|lunch is now known as amoralej13:47
*** Goneri has joined #oooq13:51
quiquellget_dlrn_hash merged !13:59
quiquelltrown, rlandy, panda: tox is failing14:04
quiquellThe latest unit test I added is failing14:04
quiquellThis is going to prevent the next commit in the maze to be merged14:05
quiquellhttps://review.openstack.org/#/c/56764914:05
quiquellplain pytest call works by calling from tox fails14:07
quiquell:-/14:07
*** skramaja has quit IRC14:09
weshayquiquell, ready?14:09
quiquellYep14:10
weshayquiquell, https://bluejeans.com/411356779814:10
rlandyquiquell: looking14:11
quiquellalling with tox fails14:11
quiquellhave to be runnign over virtualenv14:11
hubbotFAILING CHECK JOBS on stable/queens: tripleo-ci-centos-7-3nodes-multinode, gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-queens, gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset035-queens, tripleo-ci-centos-7-containers-multinode @ https://review.openstack.org/567224, stable/ocata: gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-ocata, tripleo-ci-centos-7-undercloud-upgrades @  (2 more messages)14:13
*** dtantsur|brb is now known as dtantsur14:14
rlandy  TypeError: assert_called() takes exactly 1 argument (3 given)14:18
rlandychecking versions14:18
*** trown is now known as trown|brb14:25
rlandytrying your review14:28
*** trown|brb is now known as trown14:30
rlandyweshay: ping re: https://review.openstack.org/#/c/567641/- this has merge already14:30
rlandybut queens branch has not14:31
trownquiquell: rlandy I fixed it14:32
rlandytrown; cool - thanks14:32
trownoops... no i didnt, but I will :P14:33
rlandylol14:33
trowni fixed it wrong, but I know how to fix it right14:33
trownhaste makes waste :P14:34
*** quiquell is now known as quiquell|mtg14:34
trownrlandy: could you review https://review.openstack.org/567936 if we get that in the gate, we can get all the way to having all the lint/unittests jobs merged14:38
rlandylooking14:39
rlandytrown: +2'ed14:40
rlandywe are using that integration testing14:40
rlandyI can w+114:40
rlandyI just did :)14:41
trownrlandy: nice, then we just have a couple bigger patches left... but at least rebases will be cheaper14:41
trownI need to actually review the next one from quiquell|mtg14:41
rlandyyep - I have some reviews to do as well14:42
rascamyoung|ruck, rlandy, hey folks, if you can take a look at this https://code.engineering.redhat.com/gerrit/#/c/139243/ it's a straight forward's one14:50
ajoguys, any idea of what could be happening to me,14:53
ajoI'm deploying with tripleo/master14:53
ajoand when I create instances:14:53
ajo+--------------------------------------+----------+----------------------------------+--------+------------+-------------+----------+14:53
ajo| ID                                   | Name     | Tenant ID                        | Status | Task State | Power State | Networks |14:53
ajo+--------------------------------------+----------+----------------------------------+--------+------------+-------------+----------+14:53
ajo| e621a700-db3c-4aa7-9da0-32b35b49c9ab | cirros-1 | d0bcc605896848898aed2cdd1c4873e8 | BUILD  | scheduling | NOSTATE     |          |14:53
ajo| 47dbcc44-59fe-4bfc-a4e4-d0aecfab4e31 | cirros-2 | d0bcc605896848898aed2cdd1c4873e8 | BUILD  | scheduling | NOSTATE     |          |14:53
ajo| 694c2249-bb56-4ecc-921f-3147439332a1 | cirros-3 | d0bcc605896848898aed2cdd1c4873e8 | BUILD  | scheduling | NOSTATE     |          |14:53
ajo+--------------------------------------+----------+----------------------------------+--------+------------+-------------+----------+14:53
ajothey remain in scheduling forever14:53
myoung|ruckrasca: looking14:58
myoung|ruckrasca: is tested already?14:59
myoung|ruckrasca: only asking as we're at 12d master, 11d queens and I'm working in RDO2 today :)15:00
rlandymyoung|ruck: you're on this?15:00
*** quiquell|mtg is now known as quiquell15:01
rascamyoung|ruck, yes, you need to consider that apart from two IHA modifications (so not involving our pipelines deployments) this is the 1:1 copy of the original one15:01
myoung|ruckpanda|lunch, devel tools @ https://www.google.com/url?q=https%3A%2F%2Fbluejeans.com%2F705085945515:01
quiquelltrown: Did you fix it ?15:04
trownquiquell: ya15:04
trownmock.called doesnt take arguments15:05
trownneeded mock.called_with15:05
quiquelltrown: running just pytest in the directory works15:05
quiquellMaybe is the mock version ?15:05
trownquiquell: hmm not sure... that seems pretty broken15:06
trownquiquell: but ya running pytest from the directory would use packages15:06
quiquellOk, so let's always use tox15:06
quiquellSorry about it15:07
*** panda|lunch is now known as panda15:08
pandamyoung|ruck: weshay https://etherpad.openstack.org/p/devmode-replacement-feedback15:09
quiquelltrown: Do you have a bj minute ?15:13
quiquellThere is something I don't understand15:14
trownsure15:15
quiquellhttps://bluejeans.com/789106523215:16
trownquiquell: oh, was just getting logged in so I could do that :P... joining yours15:17
quiquellUps ok...15:17
*** gkadam_ has joined #oooq15:21
rlandyweshay: reckicked https://review.openstack.org/#/c/569229/ with the fix ti stable/queens15:22
rlandyto15:22
*** gkadam has quit IRC15:24
rascamyoung|ruck, rlandy, maybe you folks can also help me on this one: I'm overriding the artcl_collect_list var to get also HA test results (which are in the undercloud's homedir under validate_ha_logs), but this seems to be ignored during the collect_logs execution15:28
rlandyrasca: where are you doing the override?15:28
rlandyit's defined in a few places15:28
*** anande has joined #oooq15:28
rascarlandy, inside the config file I'm passing to the tripelo-quickstart command line15:29
rlandycan you post your logs?15:29
rlandywant to check what was passed to the actual collect logs15:30
rascarlandy, take this for example https://thirdparty.logs.rdoproject.org/jenkins-oooq-rhos-13-puddle-bmu-had00-lab-float_nic_with_vlans-8/15:31
quiquellOk leaving now, have a good weekend guys15:32
*** quiquell is now known as quiquell|off15:32
*** Goneri has quit IRC15:33
*** atoth has quit IRC15:36
*** ykarel| is now known as ykarel|away15:36
rlandyok - si it's running a function out of locala15:37
rlandylocal15:37
rascarlandy, what do you mean?15:39
rlandybash ci-scripts/collect-logs.sh minimal external &> $WORKSPACE/collect_logs.txt15:40
rascarlandy, so are you saying I have a problem with all my jobs15:41
rascain the way I invoke collect_logs15:41
rlandywell, I don't see http://git.app.eng.bos.redhat.com/git/tripleo-environments.git/tree/ci-scripts15:41
rlandyhas a collect-logs.sh15:42
rlandyso it's using https://github.com/openstack/tripleo-quickstart/blob/master/ci-scripts/collect-logs.sh15:43
rlandyso your settings would have to be passed there15:44
rlandybut  comparing to upstream jobs ...15:44
*** tesseract has quit IRC15:46
quiquell|off weshay zuul's api15:47
quiquell|offhttps://github.com/openstack-infra/zuul/blob/5f267259c72e5e486499576521e0cdc27127265f/tests/unit/test_web.py15:47
quiquell|offwe can even have info about refspecs15:47
quiquell|offhttp://zuul.openstack.org/api/status/change/564206,315:47
quiquell|offNow I am leaving15:47
*** anande has quit IRC15:47
rlandyrasca: I am just looking at a upstream log to compare - sec15:48
rascarlandy, no worries, of course I'm talking just about rdophase2 and ospphase015:48
rlandyyeah - but we are not passing settings to your collect logs call15:48
rlandyI want to see how we do it in upstream15:49
rascarlandy, ack thanks (need to go afk for a while)15:50
*** ykarel|away has quit IRC15:51
*** ykarel has quit IRC15:51
*** lucasagomes is now known as lucas-afk15:57
rlandyrasca: ok - so you have the option of https://github.com/openstack/tripleo-quickstart/blob/master/ci-scripts/collect-logs.sh#L5115:58
rlandyadding settings to a log-env15:58
rlandyas is done here: https://github.com/openstack/tripleo-quickstart-extras/tree/master/config/general_config15:58
rlandyyou can define a log env: https://github.com/openstack/tripleo-quickstart/blob/master/ci-scripts/collect-logs.sh#L916:00
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: gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-pike-branch, tripleo-ci-centos-7-3nodes-multinode, gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-master, gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-ocata- (2 more messages)16:13
*** marios has quit IRC16:14
*** ykarel|away has joined #oooq16:15
*** jfrancoa has quit IRC16:23
*** jaganathan has joined #oooq16:23
*** jaganathan has quit IRC16:24
*** jaganathan has joined #oooq16:24
*** hamzy has quit IRC16:31
rascaok rlandy so you're saying that in the second step of my deployments that does collect_logs I'm not passing the config file16:38
rlandyappears not16:38
rascarlandy, ok, but I'm relying on what we have in tripleo-environments16:39
rascanot adding anything16:39
rascathis means that we never pass that config16:39
rascaand for the majority of the jobs this is good anyway16:39
rascabecause of the defaults16:39
rascafor the rest (==my jobs) it is not16:39
*** dtantsur is now known as dtantsur|afk16:40
rlandyrasca: that's ok - you can add a local config16:40
rlandyuse an env var to define that env16:41
rlandyin the job16:41
rlandyand that config will be used16:41
rlandyrasca: to step back - what do you means second step of your deployments?16:42
rascarlandy, take this https://rhos-dev-jenkins.rhev-ci-vms.eng.rdu2.redhat.com/job/oooq-rhos-13-puddle-bmu-had00-lab-float_nic_with_vlans/configure as an example16:43
rascarlandy, we have build (quickstart.sh command line)16:44
rascarlandy, and post-build actions, with collect_logs16:44
*** gkadam_ has quit IRC16:44
rascarlandy, these are attached by some sort of automation myoung|ruck should know16:44
rascarlandy, because I never explicitly configured them16:45
*** dougbtv_ has joined #oooq16:45
rlandyrasca: correct16:45
rlandythat is a post build publisher16:45
rascarlandy, I need to change things there just for my jobs then16:45
*** trown is now known as trown|outtypewww16:45
rlandythe publisher will need to be edited to pick up an env var16:45
rlandyrasca; I'll get the code ... sec16:46
*** dougbtv has quit IRC16:48
rlandyhttp://git.app.eng.bos.redhat.com/git/tripleo-environments.git/tree/jenkins/jobs/tripleo-quickstart/defaults.yaml#n37316:49
rlandyif we added an env file16:50
rlandyand modified the publisher to pick it up, we could edit the settings you want16:50
* myoung|ruck was onboarding calories and catches up to rasca and rlandy16:53
rlandymyoung|ruck: do you agree with my assessment?16:55
*** dougbtv__ has joined #oooq16:56
rascarlandy, I'm officially lost :D16:56
*** dougbtv_ has quit IRC16:57
rlandyrasca: juts going to let myoung|ruck comment and then I can push a change for you16:59
*** ykarel|away has quit IRC16:59
myoung|ruckrasca: so for the jobs defined in tripleo-env, we're using this http://git.app.eng.bos.redhat.com/git/tripleo-environments.git/tree/config/general_config/external-logs.yml16:59
myoung|ruckso 'external' is passed to the collect logs script, which makes quickstart find external-logs.yml when https://github.com/openstack/tripleo-quickstart/blob/master/ci-scripts/collect-logs.sh#L51 occurs17:00
*** Goneri has joined #oooq17:01
myoung|ruckas the triple-env setup.cfg file  (http://git.app.eng.bos.redhat.com/git/tripleo-environments.git/tree/setup.cfg#n23) causes tripleo-env/config to be copied into workspace/config17:01
rlandymyoung|ruck: so  I was not sure if you want to edit external for all jobs17:01
myoung|ruckrasca: so while this isn't perfect, the simplest thing (today) is probably to make a new config file (yes the duplication sucks)17:01
myoung|ruckrasca: or we can edit external-logs in place17:02
rlandyso I suggested that rasca can create a diff config file17:02
rascarlandy, myoung|ruck, I'm more than lost now, I'm lost in space and time17:02
rlandyand we could change the publisher to recognize a env file17:02
rlandybut if you are happy to exit external, ok by me17:02
rlandywe should just switch case it17:02
myoung|ruckIMHO the core issue is that the list of things to collect is a static variable, vs. something that can be appended to.  I think we can fix this (long term) in quickstart easily enough...but like rlandy says if we want something now/today, we can make a new config file, then in the JJB for your jobs just change 'external' to 'yourmonikor' - i can help17:03
myoung|ruckrasca: i can hop on BJ to unstick show...it's a little confusing by TLDR rlandy's suggestion is the most direct17:03
* myoung|ruck captures collect logs appending issue as an RFE...he's prototyped a fix last year that never was birthed17:04
rascarlandy, myoung|ruck, if you have an example I can study by myself17:04
myoung|ruckrasca: I'll make an etherpad with the links and example of what's happening now17:04
myoung|ruckIRC is a bad medium IMHO for this :)17:04
rascamyoung|ruck, much appreciated thanks17:05
*** zoli is now known as zoli|gone17:05
rlandywe could edit external17:05
*** zoli|gone is now known as zoli17:05
rlandyjust switch case it17:05
myoung|ruckrlandy, rasca, https://review.rdoproject.org/etherpad/p/rdo2-collect-logs-customization17:07
rlandytrown|outtypewww: can we rebase this: https://review.openstack.org/#/c/569120/?17:09
rlandyI just did :)17:14
myoung|ruckrasca: make sense?17:20
*** anande has joined #oooq17:23
*** anande has quit IRC17:30
myoung|ruckrasca https://review.rdoproject.org/etherpad/p/rdo2-collect-logs-customization @ L67 I think is the fast path to having updated settings for your job.  This is (I think) what rlandy was suggesting.17:35
rlandymyoung|ruck: correct - that was my suggestion17:36
myoung|ruckrasca: I need to go afk for a Dr. appt, back in 90ish.  Can look/merge later17:36
rlandyto be able to pass foo17:36
rlandyin place of external17:37
* myoung|ruck nods at rlandy17:37
myoung|ruckrlandy: I'll capture the RFE on L25 later on today, I have the (minor) patch in my noggin and will propose next week when (if) things quiet down17:38
rlandycool17:38
* myoung|ruck vanishes for 90m and will return17:38
*** kopecmartin has quit IRC17:44
rascamyoung|ruck, rlandy, I've commented the etherpad and going off right now, we can continue on Monday17:45
weshaymyoung|ruck, all the rdo jobs are failing on yum repo issues, proably need an alert bug in #tripleo to make them aware of the issue.. as a tracker17:58
rlandypanda: hi - still around?18:05
rlandyping re: https://trello.com/c/bKu4ia6u/741-assuming-the-known-output-from-the-tool-hardcode-the-output-and-work-on-fs37-5018:05
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: gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-pike-branch, tripleo-ci-centos-7-3nodes-multinode, gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-master, gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-ocata- (2 more messages)18:13
weshayrfolco, http://rdoci-doc.usersys.redhat.com/tripleo-environments/oooq-downstream.html18:18
weshayrfolco, http://rhos-release.virt.bos.redhat.com/ci-images/internal-requirements-new.txt18:18
weshayrfolco, https://code.engineering.redhat.com/gerrit/gitweb?p=tripleo-environments.git;a=blob;f=setup.cfg;h=0c5dd3fd181ea963ee9f71d696de153bc4313aa2;hb=HEAD18:19
rlandymyoung|ruck: ok - finally working on the reproducer bug18:28
*** tosky has quit IRC18:39
rlandymyoung|ruck: https://bugs.launchpad.net/tripleo/+bug/1771563/comments/118:41
openstackLaunchpad bug 1771563 in tripleo "ovb reproducer fails with "AnsibleUndefinedVariable: 'dict object' has no attribute 'ansible_host'"" [High,Triaged]18:41
rlandyworked for me18:41
*** tosky has joined #oooq18:42
*** rnoriega has quit IRC18:46
*** rnoriega has joined #oooq18:47
*** dougbtv_ has joined #oooq19:11
*** dougbtv_ has quit IRC19:18
rlandyweshay: I still a failure on containerized-undercloud-upgrades - (the change is building though).19:18
rlandyhttps://review.openstack.org/#/c/567641/ merged19:19
weshayrlandy, nice19:20
rlandythe failure is the same though19:20
weshayhrm.. you should you got the patch in your build19:20
rlandygenerating inventory19:20
weshay?19:20
weshayprobably not19:21
rlandyI didn't - the patch merged19:21
weshayrlandy, I'm misunderstanding something19:21
* weshay looks at the job generally19:21
weshayrlandy, it's green ◆ tripleo-ci-centos-7-containerized-undercloud-upgrades19:22
rlandyhttp://logs.openstack.org/29/569229/2/check/tripleo-ci-centos-7-containerized-undercloud-upgrades/6644950/job-output.txt.gz#_2018-05-17_23_19_13_65861619:25
rlandywhich seems wrong because that error should have been fixed a while back19:26
weshaynot everything was rebased19:27
weshaythe job is working now19:27
weshayrlandy, http://cistatus.tripleo.org/19:27
weshaymyoung|ruck, ping19:27
rlandyok - it's kind of immaterial to the change but it would be good to see a full run19:28
weshayright.. it's been passing most of the day19:28
weshayhttp://logs.openstack.org/18/567318/6/check/tripleo-ci-centos-7-containerized-undercloud-upgrades/02d8195/19:29
rlandyeither way - there are too many patches outstanding19:31
weshayrlandy, ya.. that's quite the dep chain19:31
rlandyand we keep getting This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.19:31
rlandyand die19:31
rlandy bye-bye  - thanks for trying19:31
weshayrlandy, so.. want some help merging?19:31
rlandyweshay: checking out where we are with the merge chain19:32
rlandythat should sort out a lot19:32
rlandywe are here: https://review.openstack.org/#/c/567649/19:33
weshayrlandy, Sagi has a point19:33
rlandyPatch Set 32: -Verified19:33
rlandyStarting gate jobs.19:33
weshayhttps://review.openstack.org/#/c/567060/27/toci_quickstart.sh19:33
weshaycan we scrub the env for anything related to OS_PASSWORD19:34
rlandyweshay: he does and panda and I had a long discussion about it yesterday19:34
weshaygrep -v -i password19:34
rlandymy concern in using select variables19:34
rlandyis that me may miss exactly what is messing up the run19:34
rlandybut really - it's not worth killing the whole sprint over19:35
rlandysagi had some parallel work he wanted to incorporate19:35
rlandybut it changed the workflow or our cirrent sprint19:35
rlandyhonestly, at this point we could kill the variable capture19:36
rlandyit's not essential19:36
rlandyI asked panda to address it at monday's meeting and get consensus and fix it19:36
rlandypanda is of the option that if we are spewing something confidential, better we discover it earlier than later19:37
rlandyweshay: ^^ feel free to wight in19:37
rlandyweigh in19:37
weshayrlandy, agree.. if there is concern about it.. let's turn it off in upstream ci19:37
weshaymaybe enable it on the recreate script19:38
rlandyI can make it a dry run only thing19:38
weshaybut isn't dry run executed upstream?19:38
rlandythe playbook args19:38
rlandyand releases stuff is stored anyways19:38
rlandyno19:38
rlandydryrun is only executed if you add --dryrun to the job name19:39
rlandywhat is executed upstream is the collection of the vars19:39
rlandyand playbook commands output19:39
rlandyand releases output19:39
rlandyI think I'll just make it execute only in dryrun and get this merged19:40
weshayah19:41
weshayrlandy, adding that to the name19:41
weshayhrm19:42
weshayhow does that help?19:42
weshaywouldn't you need to change the zuul config for that?19:42
rlandyyes - it just skips the playbook execution19:42
rlandyso you see what would happen19:42
weshayand it would still be logged upstream19:42
rlandyyou get logs19:42
weshaybut if we're worried about the leak19:42
rlandywhat would be logged upstream??19:42
weshayif we create a job w/ the name $-dryrun19:43
weshayif we had a passwd leak.. it would leak19:43
rlandyunlikely we would ever have a job with -dryrun run upstream19:43
weshayjust in rdo?19:44
weshaywas the thought?19:44
weshayor just local?19:44
rlandyjust in your reproducer19:44
rlandyyeah19:44
weshayok19:44
rlandyso you see the releases19:44
myoung|ruckweshay: pong, ack, looking, will file/alrt19:44
rlandyand the playbook commands19:44
rlandyetc.19:44
rlandywithout waiting for the whole job to finish19:45
weshaymyoung|ruck, FYI.. was a dupe https://bugs.launchpad.net/tripleo/+bug/177169219:46
openstackLaunchpad bug 1771549 in tripleo "duplicate for #1771692 Containers multinode jobs fails on stable queens with overcloud deploy timeout" [Critical,Triaged] - Assigned to Sagi (Sergey) Shnaidman (sshnaidm)19:46
myoung|ruckweshay: ack19:47
weshayrlandy, so sagi's comment is invalid?19:48
weshayor just fixed now19:48
rlandyit's valid19:48
rlandywhether you agree or not is a matter of opinion19:48
rlandyor whether you think the benefit outweighs the risk19:49
rlandyI am sending an email to sagi, panda and you19:49
rlandyto discuss at monday's meeting and resolve19:49
rlandyI am not stuck on either approach19:49
*** holser__ has joined #oooq20:10
hubbotFAILING CHECK JOBS on stable/queens: tripleo-ci-centos-7-containers-multinode, tripleo-ci-centos-7-3nodes-multinode, gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset035-queens, gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-queens @ https://review.openstack.org/567224, stable/ocata: gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-ocata, tripleo-ci-centos-7-undercloud-upgrades @  (2 more messages)20:13
*** brault has quit IRC20:24
*** brault has joined #oooq20:38
*** brault has quit IRC20:43
rlandymyoung|ruck: hi - did you see my comment re: ovb reproducer?21:07
rlandywfm21:07
myoung|ruckrlandy: auye, havne't tried again today, maybe I was hitting rdocloud at a bad time lol21:09
rlandymyoung|ruck: it's often a bad time :(21:09
*** florianf has quit IRC21:18
weshayrlandy, I marked 1771563 incomplete21:25
rlandyweshay: seems rught21:26
rlandyright21:26
rlandyI tested with bot options - worked fine21:26
weshayrlandy, anything else ready for review?21:28
rlandyweshay: https://review.openstack.org/#/c/56656521:29
rlandyhttps://review.openstack.org/#/c/56828721:29
rlandyhttps://review.openstack.org/#/c/56764921:29
rlandyhttps://review.openstack.org/#/c/56828521:30
rlandyweshay: someone told me yesterday 'no good deed goes unpunished when you ask'21:30
weshayheh21:30
rlandyrebasing the last patch21:31
*** dtrainor has quit IRC21:31
weshaytwo of those were already merged21:31
weshayrlandy, emit_releases is still not logged21:33
weshay:(21:33
weshay2018-05-18 16:53:54.473071 | primary | +(/opt/stack/new/tripleo-ci/toci_gate_test.sh:221): python /opt/stack/new/tripleo-ci/scripts/emit_releases_file/emit_releases_file.py --stable-release master --featureset-file /opt/stack/new/tripleo-quickstart/config/general_config/featureset037.yml --output-file /home/zuul/workspace/logs/releases.sh --log-file /home/zuul/workspace/logs/emit_releases_file.log21:33
weshaythink there is another patch in play21:33
rlandywhich review?21:34
weshayhttps://review.openstack.org/#/c/566565/11/toci_gate_test-oooq.sh21:34
weshayhttp://logs.openstack.org/65/566565/11/check/tripleo-ci-centos-7-scenario000-multinode-oooq-container-updates/3bae60f/job-output.txt.gz21:34
rlandy000 would not log - that is correct - 37 should21:37
weshayrlandy, ???21:37
weshaywhere are my peoples21:38
rlandythis patch logs ...21:38
weshaywhere is that21:38
weshaytripleo-ci-centos-7-scenario000-multinode-oooq-container-updates is fs03721:39
weshaybtw21:39
rlandythis is for sure missing ... http://logs.openstack.org/65/566565/11/check/tripleo-ci-centos-7-containerized-undercloud-upgrades/fe3f0b6/logs/21:39
rlandywe are looking for $WORKSPACE/logs/releases.sh21:40
rlandysec - debugging patches21:41
rlandy567060 should log when it runs21:43
rlandybut release.sh should be there in the previous patch21:43
*** dtrainor has joined #oooq21:43
rlandyhttps://review.openstack.org/#/c/566565 does not depend on the right patch21:43
rlandyweshay: ^^21:44
rlandynote the Depends-on here https://review.openstack.org/#/c/56706021:44
rlandythe output shows up in https://review.openstack.org/#/c/567936/21:45
rlandypatch maze here21:45
weshayrlandy, where? http://logs.openstack.org/36/567936/14/check/tripleo-ci-centos-7-scenario000-multinode-oooq-container-updates/47aa5bf/logs/21:46
rlandy --output-file --log-file21:47
rlandyie: the ability to add output21:47
rlandy https://review.openstack.org/#/c/567060 is where you should see the integration working21:48
weshayrlandy, why don't I see it in the fs37 jobs?21:48
rlandyfrom which patch?21:49
weshay<rlandy> the output shows up in https://review.openstack.org/#/c/567936/21:49
weshayso /me looking for emit_release_file.py in http://logs.openstack.org/36/567936/14/check/tripleo-ci-centos-7-scenario000-multinode-oooq-container-updates/47aa5bf/job-output.txt.gz21:50
rlandyLet me rephrase - the ability to pass output and log arguments shows up in /567936/21:50
rlandybut it is not exposed to toci in that patch21:50
rlandylet me rekick 56706021:50
weshayk.. thanks21:51
rlandyok - here we go now21:52
rlandyweshay: this is kind of like the days of creation ... in theory light showed up on day one but the sun and moon don't make an appearance until day 421:53
weshayheh21:53
weshayrlandy, k.. reading the irc chat.. I thought these were rebased such that one should see this stuff working21:53
rlandyI can understand why you think that :)21:54
rlandy567060 is rebased on 566565 and depends on I4c8f389978717848e755f12856dd454d605c913721:55
rlandywhich should be the top patch on the script side21:55
rlandywatching https://review.rdoproject.org/jenkins/job/gate-tripleo-ci-centos-7-multinode-1ctlr-featureset037-updates-master/936/console21:56
rlandy21:58:53 +(/opt/stack/new/tripleo-ci/toci_gate_test.sh:226): python /opt/stack/new/tripleo-ci/scripts/emit_releases_file/emit_releases_file.py --stable-release master --featureset-file /opt/stack/new/tripleo-quickstart/config/general_config/featureset037.yml --output-file /home/jenkins/workspace/gate-tripleo-ci-centos-7-multinode-1ctlr-featureset037-updates-master/logs/releases.sh --log-file /home/jenkins/workspace/gate-tripleo-c22:00
rlandyi-centos-7-multinode-1ctlr-featureset037-updates-master/logs/emit_releases_file.log22:00
rlandythat should log22:01
*** jtomasek has quit IRC22:06
*** jtomasek has joined #oooq22:12
hubbotFAILING CHECK JOBS on stable/queens: tripleo-ci-centos-7-3nodes-multinode, gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-queens, gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset035-queens, tripleo-ci-centos-7-containers-multinode @ https://review.openstack.org/567224, stable/ocata: gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-ocata, tripleo-ci-centos-7-undercloud-upgrades @  (2 more messages)22:13
*** jtomasek has quit IRC22:17
*** holser__ has quit IRC22:36
rlandyomg - passed generating inventory file :)22:38
*** d0ugal_ has joined #oooq23:12
*** d0ugal has quit IRC23:13
*** rnoriega has quit IRC23:17
*** lhinds has quit IRC23:17
*** pliu has quit IRC23:18
*** faceman has quit IRC23:19
rlandyweshay: see logged files here: https://logs.rdoproject.org/60/567060/28/openstack-check/gate-tripleo-ci-centos-7-multinode-1ctlr-featureset037-updates-master/Z21fc3090e03f491cb172dc68b5b9b0ea/undercloud/home/jenkins/23:31
*** rlandy has quit IRC23:34
*** tosky has quit IRC23:42
*** jaganathan has quit IRC23:58

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