Monday, 2021-07-12

*** ysandeep- is now known as ysandeep05:59
*** bhagyashris__ is now known as bhagyashris|ruck06:51
*** amoralej|off is now known as amoralej07:01
ysandeepsshnaidm, hey o/ could you please add these patches in your review list: https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/800210 and https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/800215 and 07:27
ysandeepI am wondering if we can start using tripleo-operator for train as well for node import: https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/800215/2/roles/overcloud-prep-images/tasks/overcloud-prep-images.yml#49 , Earlier I noticed you posted a patch which flipped this for ussuri+07:30
*** ysandeep is now known as ysandeep|lunch08:16
*** ykarel is now known as ykarel|lunch08:17
sshnaidmysandeep|lunch, looks fine, but I don't see the log file of introspection08:28
sshnaidmoh, found your testproject patch..09:03
*** cgoncalves_ is now known as cgoncalves09:13
*** ykarel|lunch is now known as ykarel09:47
*** ysandeep|lunch is now known as ysandeep09:58
ysandeepsshnaidm, thank you!09:58
*** ysandeep is now known as ysandeep|afk10:57
*** dviroel|out is now known as dviroel11:24
*** bhagyashris_ is now known as bhagyashris|ruck11:32
bhagyashris|ruckarxcruz, hi11:47
bhagyashris|rucki have one doubt , here we skip one tempest test https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/799779/3/roles/validate-tempest/vars/tempest_skip.yml#1913 for victoria , ussuri and train branch but looking at the result on victoria integration line the tempest test is still running https://logserver.rdoproject.org/openstack-periodic-integration-stable2/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-8-scenar11:49
bhagyashris|ruckio004-standalone-victoria/a4f66f9/logs/undercloud/var/log/tempest/stestr_results.html.gz11:49
bhagyashris|ruckand it'e getting skip for train and ussuri 11:49
*** ysandeep|afk is now known as ysandeep11:52
zbrwho can give me some basic info around get-hash ansible module? 11:52
zbris that already mearged or a pending pr?11:53
zbri am ready to write the POC of using ansible-test to validate the module.11:53
ysandeepzbr: marios can give you best information for get-hash stuff, he is on PTO today, will be back tomorrow.11:57
zbrysandeep: i managed to find the change from dviroel adding one module, enough to get me started.11:57
dviroelzbr: hi, yeah, you probably already found that https://opendev.org/openstack/tripleo-repos/src/branch/master/tripleo-get-hash11:58
zbri will create a collection named tripleo.tripleo_repos to host them, so FQMN would likely be something like tripleo.tripleo_repos.yum_config11:59
dviroelzbr: ok11:59
zbrpublishing collection would not be needed and I will keep current location of the modules, but for testing purposes i need to make it be collection compatible.11:59
dviroelgreat, thanks zbr12:03
arxcruzbhagyashris|ruck: checking 12:05
arxcruzbhagyashris|ruck: found the error... :/ 12:06
zbrdviroel: in fact i think i will want to move the modules to standard layout documented at https://docs.ansible.com/ansible/2.9/dev_guide/developing_collections.html, current one does not make any sense.12:09
bhagyashris|ruckarxcruz, what is the error12:12
dviroelzbr: i'm not sure why marios started with the current layout, that i'm also following in yum-config, but you can bring this up to discussiio12:13
dviroelzbr: up to discussion when he gets back from pto12:13
dviroelzbr: +1 to follow the standards, of course :)12:14
zbrmy guess is lack of knowledge around collection layout, but i will have it ready by tomorrow. 12:14
zbrwe are still not used to package ansible code as collectiosn as team12:14
dviroelzbr: ok, i can make the same thing for yum-config afterwards, you don't need to worry about it (the ansible module is still under review too)12:18
zbris missing only +W, and tbh, they need the same changes. I am making mine changes on top of yours, just to make it easier.12:19
zbrmeanwhile, i would recommend everyone to read https://docs.ansible.com/ansible/devel/dev_guide/developing_collections_structure.html in *full*, that document contains lots of things we will all need to know quite well.12:21
akahatReview Request: https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3434112:21
akahatReview Request: https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3267912:22
dviroelzbr: ack, tks12:23
*** ysandeep is now known as ysandeep|afk12:25
*** amoralej is now known as amoralej|lunch12:33
arxcruzbhagyashris|ruck: jsut a sec 12:46
weshay|ruckbhagyashris|ruck, https://hub.docker.com/billing12:51
weshay|ruckw/ the userid in the infra doc12:52
weshay|ruckwe're paying to not be rate limited12:52
weshay|ruckbhagyashris|ruck, was it wallaby?12:52
bhagyashris|ruckweshay|ruck, no no i was talking about infra issue12:53
arxcruzbhagyashris|ruck: there are several releases12:53
bhagyashris|ruckhttps://bugs.launchpad.net/tripleo/+bug/193583412:53
bhagyashris|ruckweshay|ruck, ^12:53
* weshay|ruck looks12:53
arxcruzbhagyashris|ruck: https://opendev.org/openstack/openstack-tempest-skiplist/src/branch/master/roles/validate-tempest/vars/tempest_skip.yml#L1917-L192512:54
bhagyashris|ruckarxcruz, you mean it doesn't pick several release 12:54
arxcruzas you can see there are several releases options, should have only one releases, with the list12:54
arxcruzbhagyashris|ruck: the validator was passing, i'm fixing the validator and updating the yml file 12:54
bhagyashris|ruckarxcruz,  but here also there are several releases https://opendev.org/openstack/openstack-tempest-skiplist/src/branch/master/roles/validate-tempest/vars/tempest_skip.yml#L1900-L191312:55
bhagyashris|ruckarxcruz, got it i see12:56
dviroelsshnaidm: hi, wdyt about W+1 this one https://review.opendev.org/c/openstack/tripleo-repos/+/796529 and I provide a follow up patch for that later? zbr is already working on a refactoring on top of this change too12:56
arxcruzbhagyashris|ruck: yeah, this is wrong, i'm trying to fix it 13:01
bhagyashris|ruckysandeep|afk, akahat zbr soniya29 scrum time13:02
akahatbhagyashris|ruck, already in :)13:02
sshnaidmdviroel, ack, +w13:03
*** ysandeep|afk is now known as ysandeep13:05
bhagyashris|ruckhttps://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3267913:07
bhagyashris|ruck https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3431313:07
bhagyashris|ruckweshay|ruck, ^13:07
bhagyashris|ruck https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3434113:17
bhagyashris|ruckzbr, https://review.opendev.org/c/openstack/tripleo-ci-health-queries/+/79895813:22
*** amoralej|lunch is now known as amoralej13:31
bhagyashris|ruckhttps://bugs.launchpad.net/tripleo/+bug/193583413:43
bhagyashris|ruckysandeep, rlandy pojadhav 13:48
bhagyashris|ruckplz join13:48
bhagyashris|ruckhttps://meet.google.com/xnf-tvdh-pmk?authuser=013:48
ysandeepbhagyashris|ruck, dviroel ^^13:48
bhagyashris|ruckakahat, ^13:48
ysandeeprlandy, https://meet.google.com/xnf-tvdh-pmk?authuser=013:49
bhagyashris|ruckhttps://hackmd.io/iYAL3ueUTzWCHE5qFA6jVQ?both#Proposed-design-and-next-steps13:50
rlandyjoining13:50
arxcruzbhagyashris|ruck: weshay|ruck rlandy https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/80047414:03
arxcruzplease review, i'm still investigating how to validate the releases... 14:03
akahatweshay|ruck, rlandy bhagyashris|ruck chkumar|rover please review: https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3434115:02
akahatonce it is updated i can fix the downstream promoter .15:03
akahatonce it is merged *15:03
*** ykarel is now known as ykarel|away15:03
rlandyack15:03
bhagyashris|ruckakahat, ack15:03
* bhagyashris|ruck dinner brb15:04
rlandyarxcruz: voted - thanks15:04
*** dviroel is now known as dviroel|lunch15:05
rlandyarxcruz: ok to merge?15:05
*** ysandeep is now known as ysandeep|away15:35
bhagyashris|ruckweshay|ruck, leaving for the day.  there are some infra related issue that we discussed not major one.15:57
rlandyweshay|ruck: bhagyashris|ruck: ysandeep|away: moving https://projects.engineering.redhat.com/secure/RapidBoard.jspa?rapidView=4285&projectKey=TRIPLEOCI&view=detail&selectedIssue=TRIPLEOCI-522 to done16:02
rlandysince we have the design noe16:02
rlandynow16:02
*** dviroel|lunch is now known as dviroel16:10
rlandyweshay|ruck: bhagyashris|ruck: ysandeep|away: dviroel: Added tasks to the board for the centos stream compose line16:19
*** ysandeep|away is now known as ysandeep16:20
ysandeeprlandy, thanks! 16:20
rlandythey are not job specific - ie: so the code can be tested outside of jobs16:20
dviroelgreat, thanks16:20
rlandylet me know if I left anything out that should be in this sprint16:20
rlandydviroel: I'll try set get the images server set up later today16:20
rlandywill put details in card 16:21
dviroelrlandy: ok16:21
*** amoralej is now known as amoralej|off16:26
*** ysandeep is now known as ysandeep|away16:26
weshay|ruckakahat, looking at https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/34341 which jobs can find the staging promotion logs in these days?16:26
akahatweshay|ruck, staging-job-1, staging-job-216:28
weshay|ruckakahat, where are those jobs.. don't see them running against this review16:30
akahatweshay|ruck, https://logserver.rdoproject.org/41/34341/12/check/tox-py36-ci-config/56839ff/tox/reports.html16:33
akahatsearch staging-job-116:33
weshay|ruckakahat, thanks!16:36
weshay|ruckrlandy, you have a sec for a zuul yaml consult?16:41
rlandysure16:41
weshay|ruckthanks meet.google.com/jad-zcip-heh16:41
weshay|ruckrlandy, https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/80043716:44
rlandygoing to check on electricity status - back in about hour17:10
rlandyweshay|ruck: ^^ fyi17:10
weshay|ruckk17:10
akahatweshay|ruck, rlandy, chkumar|rover http://38.102.83.109/promoter_logs/centos8_wallaby.log17:57
weshay|ruckakahat, nice18:00
weshay|ruckthat may be too much to fit re: the width.. but still a HUGE Improvement18:01
akahatweshay|ruck, yes. that is one pain :|18:01
weshay|ruckwe could just dupe the call..18:01
weshay|ruckfirst one w/ passed18:01
weshay|rucksecond w/ failing18:01
weshay|ruckwould be longer logs, but would fit18:02
weshay|ruckafaict18:02
akahatyeah.. that could be done.. 18:03
akahator we can just put only aggreagate hash and jobs there remove the rest.18:03
weshay|ruckakahat, that still may not give you enough room18:04
weshay|ruckmy vote would be to call first w/ passed18:04
weshay|ruckcall second w/ failed18:04
weshay|ruckmake it a function that togles the column name.. and passed/failed.. and the appropriate list of jobs18:05
weshay|ruckakahat, if we nuked18:06
weshay|ruckcommit hash, distro hash, extended hash && timestamp18:07
weshay|ruckyou may be able to use use one box18:07
akahatweshay|ruck, that's what i was suggesting.. we can put only aggregate hash and remove the rest.18:07
weshay|ruckagg hash, promoted, successful, missing18:07
weshay|ruckmay do it18:07
weshay|ruckakahat, but also nuke timestamp18:08
weshay|ruckakahat, and component18:08
akahatweshay|ruck, okay. on it.18:08
weshay|ruck++18:08
weshay|ruckakahat, if centos-7 s/aggregate_hash/commit_hash18:09
akahatyeah.. that one need to keep in mind.18:10
akahatweshay|ruck, how about this: http://paste.openstack.org/show/807402/18:36
akahatlooks good?18:36
weshay|ruckakahat++18:37
weshay|ruckakahat, since ur here: https://review.opendev.org/q/topic:%22scenario010-branchful%22+(status:open%20OR%20status:merged)18:39
weshay|ruckfyi18:39
weshay|ruckjust trying w/ train to start18:39
* dviroel ☕19:17
rlandyweshay|ruck: hey ... I am on the image server and am permission denied to create a dir.19:21
rlandyare you able to?19:21
rlandyiirc, we had to get infra to do that?19:21
weshay|rucktry here in a sec19:21
rlandyalso the dir struct is a bit diff19:22
weshay|ruckrlandy, http://images.rdoproject.org/centos8/19:23
rlandyweshay|ruck: so ... I think it should be centos_compose/centos819:24
weshay|ruckhttp://images.rdoproject.org/centos8/compose/19:26
weshay|ruckright?19:26
weshay|ruckwe don't need another distro dir19:26
rlandydviroel: ^^ fyi ... we're permission denied on the images server at the top level dir  - will ask infra tomorrow for that dir create19:30
dviroelrlandy: ok ;)19:31
weshay|ruckrlandy, https://review.opendev.org/q/topic:%22upgrade-ci-tqe%22+(status:open%20OR%20status:merged)19:32
weshay|ruckdviroel, so for your molecule job.. just use the local filesytem19:33
* rlandy looks19:33
weshay|ruckdviroel, don't connect out to images.rdoproject.org19:33
weshay|ruckrlandy, btw.. the only place we have the uploader key.. is the promoter19:34
weshay|ruckdon't think it's a zuul secret19:34
rlandyweshay|ruck: I can connect from my laptop19:34
weshay|ruckmaybe it is.. but not sure19:34
dviroelweshay|ruck: ok, thanks19:34
rlandysee pvt - I guess that is how I connected19:35
weshay|ruckdviroel, also.. another note.. if by chance we can get tripleo-tox-molecule running against tripleo-repos... we would have caught the error.. that hit Alex.19:35
weshay|ruckdviroel, /me toying w/ that now19:36
weshay|ruckbut I need to ramp up on molecule19:36
dviroelweshay|ruck: great, that would be important for tripleo-repos19:37
* rlandy checks zuul secrets19:37
rlandymust be there19:37
rlandyhttps://github.com/rdo-infra/review.rdoproject.org-config/blob/master/zuul.d/secrets.yaml#L5219:38
rlandyweshay|ruck: ^^19:38
dviroel++19:38
rlandyhttps://github.com/rdo-infra/review.rdoproject.org-config/blob/master/zuul.d/secrets.yaml#L48019:38
weshay|ruckok.. cool19:38
rlandy^^ that one19:38
rlandyas well19:38
rlandywe'll have to check which one is used where19:38
rlandyweshay|ruck: can I edit those reviews?19:48
rlandyhttps://review.opendev.org/c/openstack/tripleo-ci/+/800518/6/zuul.d/upgrades-jobs-templates.yaml#14919:49
weshay|rucksure19:49
rlandyk19:50
rlandyto get rid of zuul -119:51
* dviroel brb19:53
weshay|ruckrlandy, I'm not hitting any zuul issues there anymore19:57
weshay|ruckrlandy, it's running https://zuul.openstack.org/status#80051819:57
rlandyhttps://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/80052019:58
rlandy The project template "tripleo-upgrades-wallaby-pipeline-for-master"19:58
rlandy  was not found.19:58
rlandyweshay|ruck: ^^19:58
weshay|ruckah ya.. 20:03
weshay|ruckwhy is that?20:04
weshay|ruckthe name is right20:04
weshay|ruckshouldn't have to merge.. but I don't know...20:05
rlandyweshay|ruck: yeah - initially I thought your spacing was wrong20:05
rlandybut it's not :)20:05
weshay|ruckmaybe templates have to merge20:05
rlandymaybe?20:06
weshay|ruck¯\_(ツ)_/¯20:06
rlandybut I'd be surprised20:06
rlandyI don;t see what's wrong though ... yet20:06
weshay|ruckrlandy, https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/800510 https://logserver.rdoproject.org/21/34321/4/check/periodic-tripleo-ci-centos-8-ovb-3ctlr_1comp-featureset001-master/9d91e3e/logs/undercloud/var/log/tempest/stestr_results.html.gz20:21
rlandyweshay|ruck: haven20:28
rlandyhaven't tested that on internal yet20:29
rlandybut let's merge20:29
rlandyand I'l put it back if it's still an issus there20:29
weshay|ruckya20:31
rlandyI +2ed20:31
weshay|ruckthanks20:31
weshay|ruckrlandy, def.. infra related20:31
weshay|rucknot code20:31
rlandyweshay|ruck: yeah - I am just a bit behind qualifying internal20:32
rlandygoing to work there in a few after I update the modify image patch20:32
rlandyweshay|ruck: and I still can't find anything wrong with your template code :)20:33
rlandyso maybe it is the merge20:33
rlandyor I stand corrected20:33
weshay|ruckaye.. I think marios should way in on that anyhoo.. so no rush20:33
weshay|ruckI emailed him to see what he thinks20:34
rlandyweshay|ruck: yeah - also need to talk to marios about the other review in upgrade20:35
rlandythis one ...20:36
rlandyhttps://review.opendev.org/c/openstack/tripleo-ci/+/796771/9/zuul.d/periodic.yaml20:36
weshay|ruckhttps://review.opendev.org/c/openstack/tripleo-ci/+/79786521:03
*** dviroel is now known as dviroel|out21:41
weshay|ruckrlandy, am I crazy.. 21:44
weshay|ruckrlandy, so undercloud-upgrade is not voting in dlrn21:44
weshay|ruckhttps://logserver.rdoproject.org/95/34495/1/check/periodic-tripleo-ci-centos-8-undercloud-upgrade-master/10a251d/job-output.txt21:44
weshay|ruckperhaps due to https://review.rdoproject.org/r/c/rdo-jobs/+/3447321:44
rlandyperiodic-tripleo-ci-centos-8-undercloud-upgrade-master21:45
rlandyperiodic voting?21:45
rlandyconfused21:45
rlandycheck job or periodic21:46
rlandyreporting to dlrn?21:46
weshay|ruckrlandy, it's not running the post playbook that votes21:47
weshay|ruckhttps://review.rdoproject.org/r/c/rdo-jobs/+/3445921:47
rlandyok - not reporting to dlrn21:47
* weshay|ruck testing21:47
weshay|ruckhttps://review.rdoproject.org/r/c/testproject/+/3449521:48
rlandyyep - no reporting21:48
* rlandy check tripleo-ci-base-singlenode-rdo-centos-821:49
weshay|ruckI think I'm safe to promote master.. /me taking out of critieria manually21:50
weshay|ruckbased on https://logserver.rdoproject.org/95/34495/1/check/periodic-tripleo-ci-centos-8-undercloud-upgrade-master/10a251d/logs/undercloud/home/zuul/undercloud_upgrade.log.txt.gz21:50
rlandysec - tracking down the dlrn reporting parent job21:50
rlandyhttps://github.com/rdo-infra/review.rdoproject.org-config/blob/master/zuul.d/tripleo-rdo-base.yaml#L18021:52
rlandydoesn't have dlrn reporting playbooks21:52
rlandyso we need ...21:52
rlandyhttps://github.com/rdo-infra/review.rdoproject.org-config/blob/master/zuul.d/tripleo-rdo-base.yaml#L41221:53
rlandyweshay|ruck: parent off tripleo-ci-base-singlenode-periodic-centos-821:54
rlandy- job:21:55
rlandy    name: periodic-tripleo-ci-centos-8-undercloud-upgrade-master21:55
rlandy    parent: tripleo-ci-base-singlenode-rdo-centos-821:55
weshay|ruckk..21:55
weshay|ruckthanks21:55
rlandyswitch to tripleo-ci-base-singlenode-periodic-centos-8 and it should report21:55

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!