| akahat|rover | o/ | 04:25 |
|---|---|---|
| *** pojadhav is now known as pojadhav|ruck | 04:36 | |
| ysandeep | good morning o/ | 05:44 |
| ysandeep | akahat|rover, some jobs failed in your testrun because of missing containers: https://review.rdoproject.org/zuul/status#46525 | 05:56 |
| ysandeep | Would be good to also include container build in testproject itself. | 05:56 |
| ysandeep | akahat|rover, do you want to meet for few to sort out the next steps? | 05:59 |
| ysandeep | akahat|rover, regarding your testrun which is running component jobs, it will fail because https://review.opendev.org/c/openstack/tripleo-quickstart/+/869099 check release files don't have systemd in exclude | 06:01 |
| ysandeep | akahat|rover, I am updating your patch to just exclude systemd for a test | 06:04 |
| marios | o/ | 06:05 |
| ysandeep | marios, good morning o/ | 06:05 |
| marios | morning | 06:07 |
| ysandeep | pojadhav|ruck, hey o/ could you please swap chandankumar next ruck/rover swift with me(starting tomorrow). He covered me last week. | 06:15 |
| pojadhav|ruck | ysandeep, yep | 06:15 |
| ysandeep | pojadhav|ruck, thanks! | 06:15 |
| pojadhav|ruck | ysandeep, done | 06:17 |
| pojadhav|ruck | bhagyashris, soniya29 : swapped your names too as requested | 06:18 |
| ysandeep | pojadhav++ | 06:18 |
| soniya29 | pojadhav|ruck, thank you | 06:21 |
| chandankumar | ysandeep: thank you :-) | 06:22 |
| chandankumar | Good morning #oooq | 06:22 |
| ysandeep | chandankumar, good morning o/ and thanks to you! | 06:24 |
| ysandeep | akahat|rover, updated testprojects, lets see after few hours. | 06:25 |
| akahat|rover | ysandeep, o/ thanks.. i've added different packages in the releases. | 06:30 |
| akahat|rover | so we can narrow down the package list | 06:31 |
| akahat|rover | ysandeep, also build-containers job is failing because of https://bugs.launchpad.net/tripleo/+bug/2001626 | 06:31 |
| ysandeep | akahat|rover, ^^ Fixed now, i have included container build in testproject itself | 06:33 |
| ysandeep | at this point we have tried different packages, lets try systemd alone in exclude - testproject running right now | 06:34 |
| akahat|rover | ysandeep, zed is not yet merged: https://review.opendev.org/c/openstack/tripleo-common/+/869207 | 06:34 |
| ysandeep | ahh okay | 06:35 |
| marios | bogdan need to remove his -1 there | 06:37 |
| marios | for the zed patch | 06:37 |
| marios | but he is not on irc yet | 06:37 |
| akahat|rover | yes.. we need to wait for hi.m | 06:37 |
| marios | (it isn't going to gate i think because of bogdan -1) | 06:37 |
| akahat|rover | is review.opendev.org is down? not able to post the comment. | 06:47 |
| marios | ah i see Tengu did go through with making it non voting on master patch (comment is there akahat|rover ) | 06:48 |
| akahat|rover | marios, yes.. he fixed master.. this patch wil fix zed: https://review.opendev.org/c/openstack/tripleo-common/+/869207/2#message-18bab14c2c57f4b4260a29f7877275b0191f9208 | 06:49 |
| marios | akahat|rover: ysandeep: chandankumar: please vote there when you have sec we need that back in | 06:49 |
| marios | https://review.opendev.org/c/openstack/tripleo-common/+/869242/2 | 06:50 |
| marios | ok cool so master is unblocked at least | 06:50 |
| marios | (gate i mean) | 06:50 |
| akahat|rover | we don't have nay hack to remove w -1 from bogdan? I think admins can do it. | 06:51 |
| marios | akahat|rover: looks like only owner can do it | 06:56 |
| marios | should be around soon still 0756 in central eu | 06:56 |
| akahat|rover | ok. | 06:57 |
| akahat|rover | marios, chandankumar ysandeep please take a look when you free: https://review.opendev.org/c/openstack/tripleo-common/+/869208 | 07:02 |
| marios | ack akahat|rover but for same repo better to rebase instead of depends-on | 07:04 |
| akahat|rover | marios, tired with rebase.. was not working.. so updated with depends-on. | 07:05 |
| marios | akahat|rover: what didnt work | 07:05 |
| chandankumar | yes depends on does not make sense here, rebase would be better | 07:05 |
| akahat|rover | rebase from gerrit ui didn't worked. | 07:06 |
| akahat|rover | let me try again | 07:06 |
| marios | akahat|rover: its ok doesn't matter now just for future ref | 07:09 |
| marios | akahat|rover: doesnt matter because other patch merged already | 07:10 |
| akahat|rover | yeah.. it got merged.. rebased and removed depends-on. | 07:10 |
| marios | ack revoted | 07:11 |
| marios | akahat|rover: pojadhav|ruck: lets do quick sync at usual time please ? ( invite sent) should be brief today | 07:28 |
| akahat|rover | marios, yeah.. sure. accepted. | 07:29 |
| * akahat|rover lunch | 07:30 | |
| *** soniya29 is now known as soniya29|lunch | 07:30 | |
| pojadhav|ruck | marios, ack | 07:31 |
| pojadhav|ruck | marios, ysandeep : i think we should have RR handover today before planning meeting, otherwise it will too late for us. what say ? | 07:36 |
| ysandeep | pojadhav|ruck, works for me before planning mtg | 07:37 |
| pojadhav|ruck | ysandeep, thanks, will send invite for the same | 07:38 |
| marios | pojadhav|ruck: sure | 07:40 |
| marios | thx pojadhav|ruck | 07:41 |
| marios | pojadhav|ruck: add a google meet to the event | 07:42 |
| pojadhav|ruck | already added | 07:42 |
| pojadhav|ruck | i can see google meet in the invite | 07:42 |
| marios | pojadhav|ruck: yeah my bad thanks | 07:43 |
| ysandeep | akahat|rover, all the 3 jobs passed node provisioning with systemd exclude: https://review.rdoproject.org/zuul/status#45217 | 07:47 |
| marios | ysandeep: great lets try rerun it few times today please? | 07:48 |
| ysandeep | yeah, that's the plan, few more jobs are running here already: https://review.rdoproject.org/zuul/status#46525 | 07:49 |
| * pojadhav|ruck afk for half hour | 07:52 | |
| Tengu | marios: heya! | 07:59 |
| marios | \o | 07:59 |
| Tengu | master is merged, so we should be unlocked. zed and re-enabling master upgrade are in gate. | 07:59 |
| marios | yeah thanks | 07:59 |
| Tengu | we're on track now. | 07:59 |
| marios | i saw and we workflowed the partial revert | 07:59 |
| marios | thanks Tengu | 07:59 |
| Tengu | np | 07:59 |
| Tengu | happy to help :) | 07:59 |
| marios | but we need bogdan to remove his -1 | 07:59 |
| marios | from zed | 08:00 |
| Tengu | ah, right. | 08:00 |
| marios | should be in soon hopefully | 08:00 |
| Tengu | that's why -w is annoying. | 08:00 |
| Tengu | grumpf.... that edpm thing is driving me mad. | 08:01 |
| Tengu | chandankumar: heya! seems it's still not deploying on the right node..... how can I get the subnode-1 IP in order to create the appropriate entry in the inventory? | 08:02 |
| chandankumar | Tengu: /etc/nodepool/sub_nodes_private contains the ip of second node | 08:04 |
| chandankumar | you can find this file on undercloud | 08:04 |
| chandankumar | just do a cat | 08:04 |
| akahat|rover | ysandeep, \o/ that's good news. :) | 08:04 |
| chandankumar | something like this https://review.rdoproject.org/r/c/testproject/+/46393/20/playbook/run.yaml#149 | 08:04 |
| Tengu | hmm ok. | 08:05 |
| Tengu | calling slurp will be more "ansible", but ok. | 08:05 |
| Tengu | lemme do some tweaking. | 08:05 |
| *** soniya29|lunch is now known as soniya29 | 08:14 | |
| *** ysandeep is now known as ysandeep|lunch | 08:16 | |
| marios | akahat|rover: o/ | 08:31 |
| * pojadhav|ruck lunch | 08:46 | |
| *** ysandeep|lunch is now known as ysandeep | 09:34 | |
| ysandeep | akahat|rover, two periodic line job also passed node provisionin, I am rerunning some jobs | 09:37 |
| akahat|rover | ysandeep, this also shows good results: https://review.rdoproject.org/r/c/testproject/+/45217 | 09:38 |
| akahat|rover | i'll re-run all the fs01 jobs.. it will give us good results | 09:38 |
| ysandeep | akahat|rover, I have updated https://review.rdoproject.org/r/c/testproject/+/45217 , lets see | 09:39 |
| akahat|rover | oh.. you also update it :| | 09:39 |
| ysandeep | akahat|rover, if all passes this time, lets remove the DNM from the main patch | 09:39 |
| akahat|rover | ysandeep, +1 | 09:40 |
| ysandeep | akahat|rover, https://review.rdoproject.org/r/c/testproject/+/45217/8/.zuul.yaml soo many ovb together might cause resource crunch | 09:40 |
| ysandeep | akahat|rover, could you please comment few for now | 09:41 |
| akahat|rover | ysandeep, ah.. 2 2 nodes.. | 09:41 |
| akahat|rover | yes.. let's go with master first. | 09:41 |
| arxcruz | hey guys, i need to go to the doctor, brb in one or two hours | 09:41 |
| ysandeep | arxcruz, o/ sure | 09:45 |
| Tengu | chandankumar: wow..... it's fun: apparently I have *exactly* the same behavior on my local lab. | 10:07 |
| Tengu | so there's something fishy with either the playbook, or the inventory. Need to investigate a bit. | 10:08 |
| Tengu | at least, it's faster to debug locally :). | 10:08 |
| Tengu | especially since I can interrupt pretty fast. | 10:08 |
| Tengu | fun... fun, fun, fun. | 10:14 |
| Tengu | [WARNING]: Could not match supplied host pattern, ignoring: [] <- I think this is the issue. | 10:15 |
| Tengu | and this one comes from... gimme a sec... | 10:15 |
| Tengu | hosts: "{{ deploy_source_host | default([]) }}:{{ deploy_target_host | default('overcloud') }}" | 10:15 |
| Tengu | and I think here we get a tiny issue. | 10:15 |
| Tengu | o____O | 10:25 |
| Tengu | chandankumar: it's..... eewww.. guess I'll need to grab some more coffee. Like, some casks actually. | 10:26 |
| chandankumar | Tengu: glad you found the issue. :-) | 10:26 |
| chandankumar | sorry little busy with docs writting. | 10:26 |
| Tengu | chandankumar: well, I still don't understand what's going on | 10:27 |
| Tengu | but there's something really, really fishy in the way ansible takes and understand the inventory.... | 10:27 |
| Tengu | chandankumar: https://paste.openstack.org/show/bODLyx65Z1lneE0UPmL4/ so, the ansible_host is correct; the ansible_fqdn and ansible_hostname are NOT. | 10:29 |
| Tengu | i.e. it's taking the IP of the compute, and does something wrong in order to map it to the current host (which is, well, named "standalone", for that matter. | 10:30 |
| Tengu | oh. wrong. ok. still "undercloud". only the prompt is correct. | 10:30 |
| Tengu | anyway. it's running against the undercloud/standalone instead of the remote, while it DOES have the correct IP. | 10:30 |
| akahat|rover | ysandeep, i've updated your patch.. if we have enough passing jobs we can push it to gate. https://review.opendev.org/c/openstack/tripleo-quickstart/+/868605 | 10:41 |
| akahat|rover | marios, ^^ | 10:42 |
| ysandeep | akahat|rover, could you please update the comments also | 10:42 |
| ysandeep | from Pin kernel to pin systemd | 10:42 |
| marios | thx | 10:44 |
| akahat|rover | done | 10:51 |
| ysandeep | akahat|rover, thanks, need some more small changes: https://review.opendev.org/c/openstack/tripleo-quickstart/+/868605 | 10:53 |
| ysandeep | https://review.opendev.org/c/openstack/tripleo-quickstart/+/868605/15#message-e5dc90dcb7f47ff8aa030d90db83fa0b4f8a7ce6 | 10:53 |
| akahat|rover | ysandeep, ok | 10:57 |
| ysandeep | akahat|rover, ++ thank you, looks good now | 11:03 |
| ysandeep | The only issue I see is, we are currently excluding affected 252* version but as soon as newer systemd releases (and if our bug is not solved by then) we will again hit this issue. | 11:06 |
| * ysandeep not sure of cadence of systemd release | 11:07 | |
| Tengu | chandankumar: ok. apparently something in the edpm playbooks make it works only for local connection.... still need to dig a bit more, but that's one of the issue. | 11:08 |
| ysandeep | akahat|rover, 4/4 jobs passed node provision again: https://review.rdoproject.org/zuul/status#45217 , looks like we finally have the culprit | 11:08 |
| Tengu | i.e. it remaps things to the current localhost. "yay". | 11:08 |
| Tengu | I'll check with James as well, maybe it's intended. Would be bad imho, but heh. | 11:08 |
| *** rlandy|out is now known as rlandy | 11:09 | |
| Tengu | wondering if the way the inventory is built/displayed isn't the main issue....... | 11:10 |
| chandankumar | Tengu: when I was testing, I was running the playbook from second node to deploy compute on the second node itself | 11:16 |
| chandankumar | if I remove this task https://opendev.org/openstack/tripleo-quickstart-extras/src/branch/master/roles/external-data-plane-management/tasks/compute_services_on_target_host.yml#L29 | 11:17 |
| chandankumar | it does not make any difference | 11:17 |
| chandankumar | it will run using localhost | 11:17 |
| chandankumar | the way you are trying to execute from the same node, I have not tried that | 11:17 |
| chandankumar | s/same/undercloud node | 11:18 |
| chandankumar | let's check with james what he says about that | 11:18 |
| arxcruz | back | 11:22 |
| rlandy | marios: akahat|rover: pojadhav|ruck: should we sync for a few? | 11:35 |
| pojadhav|ruck | rlandy, yep | 11:36 |
| marios | rlandy: ok | 11:38 |
| ysandeep | do you mind me listening in the background as I am the next ruck/rover, We will probably need a shorter handover sync if I join now. | 11:38 |
| marios | yes good idea ysandeep | 11:38 |
| marios | imo we can even skip that | 11:39 |
| rlandy | marios: ysandeep: okp you are on next week | 11:39 |
| marios | yes | 11:40 |
| rlandy | marios: ysandeep: so up to you if you want to sync now or after scrum | 11:40 |
| rlandy | I think now is better | 11:40 |
| ysandeep | marios, +1 to just meet now and skip later | 11:40 |
| marios | akahat|rover: pojadhav|ruck: around? | 11:40 |
| pojadhav|ruck | yes | 11:40 |
| rlandy | after scrum is late for ysandeep and pojadhav|ruck | 11:40 |
| marios | rlandy: we have it scheduled *before* scrum but still now is good if akahat|rover is around | 11:40 |
| marios | k lets try in 20 mins top of the hour rlandy ok for you then or have calls | 11:41 |
| marios | give akahat|rover a chance if he is afk/break | 11:41 |
| rlandy | marios: ok - can you move the meeting there then? | 11:42 |
| marios | sure pojadhav|ruck has to do it it is hers can you please pojadhav|ruck ? | 11:42 |
| marios | pojadhav|ruck: move the rr hanforr in 20 mins | 11:42 |
| pojadhav|ruck | marios, ack | 11:42 |
| rlandy | thanks | 11:42 |
| marios | akahat|rover: ysandeep: ^^^ 20 mins | 11:42 |
| rlandy | marios; and sorry - misread the BEFORE part | 11:43 |
| marios | ack | 11:43 |
| rlandy | soniya29: hey - you around? | 11:43 |
| ysandeep | marios, ack works for me | 11:43 |
| *** dviroel|ourt is now known as dviroel | 11:45 | |
| pojadhav|ruck | folks, I have created new agenda for 73 sprint planning, kindly add agenda if any.. thank you !! https://hackmd.io/0ruKGb6eTwGvdvwWE4nk8g#2023-01-05-Unified-Sprint-73-Planning | 11:46 |
| akahat|rover | marios, pojadhav|ruck rlandy i'm here. | 11:47 |
| pojadhav|ruck | *add new topics/tasks that needs to be done this sprint. | 11:47 |
| soniya29 | rlandy, yes | 11:48 |
| pojadhav|ruck | akahat|rover, i have set RR handover in around 15 mins. | 11:48 |
| * soniya29 stepping out for an hour | 11:51 | |
| marios | akahat|rover: call in 9 mins | 11:51 |
| pojadhav|ruck | marios, rlandy : RR handover | 12:04 |
| Tengu | chandankumar: host_vars/lab2-oc0-compute-1:ansible_connection: local | 12:05 |
| Tengu | of course....... | 12:05 |
| Tengu | #facepalm | 12:05 |
| Tengu | chandankumar: so this explain the issue in CI as well :). | 12:07 |
| Tengu | easy enough to correct. | 12:07 |
| akahat|rover | marios, rlandy https://review.opendev.org/c/openstack/tripleo-quickstart/+/868605 | 12:11 |
| Tengu | imho this host_vars/localhost shouldn't list that ansible_connection parameter, it should be in the inventory 02-computes | 12:15 |
| Tengu | dang. that one was a hard nut to crash. | 12:15 |
| chandankumar | Tengu: awesome :-) | 12:24 |
| chandankumar | you can send a review to fix tripleo-ansible inventory | 12:25 |
| chandankumar | https://opendev.org/openstack/tripleo-ansible/src/branch/master/tripleo_ansible/inventory | 12:25 |
| chandankumar | we ship this file in the tripleo-ansiblee container | 12:25 |
| Tengu | sure - lemme do some more testing, but apparently it should really, really work better now. | 12:28 |
| Tengu | dang. | 12:28 |
| *** ysandeep is now known as ysandeep|afk | 12:32 | |
| * pojadhav|ruck afk for ~1 hr | 12:34 | |
| Tengu | chandankumar: https://review.opendev.org/c/openstack/tripleo-ansible/+/869357 | 12:34 |
| Tengu | also took the opportunity to make the inventory files format more consistent. | 12:34 |
| Tengu | pfff.... how many days did I lose because of that? ;_; | 12:35 |
| * akahat|rover back in hour.. | 12:41 | |
| chandankumar | Tengu: great. | 13:01 |
| chandankumar | more than 2 weeks | 13:01 |
| chandankumar | but it was a great improvement to the roles and inventory :-) | 13:01 |
| Tengu | heh... yeah. still. | 13:02 |
| soniya29 | rlandy, hello | 13:49 |
| * soniya29 i am back | 13:49 | |
| rlandy | soniya29: hi - let's meet for 5 | 13:49 |
| soniya29 | okay | 13:50 |
| rlandy | soniya29: https://meet.google.com/jmg-ytnf-hax?pli=1&authuser=0 | 13:50 |
| *** dasm|off is now known as dasm | 13:50 | |
| dasm | o/ | 13:51 |
| rlandy | dasm: gm | 14:00 |
| dasm | rlandy: \o | 14:02 |
| Tengu | Folks, we may have an issue in tripleo-ansible repository: https://zuul.opendev.org/t/openstack/builds?job_name=openstack-tox-pep8&project=openstack%2Ftripleo-ansible&skip=0 not sure yet about the cause, probably something in the tox.ini, I think Rabi corrected something not that long ago somewhere. So just FYI. | 14:07 |
| Tengu | last successful: https://review.opendev.org/c/openstack/tripleo-ansible/+/869020/ (in Gate) - then all red. | 14:08 |
| Tengu | maybe related (that's the patch I thought about, from Rabi) | 14:08 |
| dasm | akahat|rover: pojadhav|ruck ^ are you aware of that? | 14:09 |
| akahat|rover | dasm, yes.. its known | 14:10 |
| dasm | Tengu: ^ | 14:10 |
| dasm | akahat|rover: ack | 14:10 |
| Tengu | 'k - is there already some patch around? | 14:11 |
| Tengu | oh, wait - patch is still active. | 14:12 |
| Tengu | guess we'll have to wait a bit. | 14:13 |
| akahat|rover | q: playbooks from rdo picking master here: PRE-RUN END RESULT_NORMAL: [trusted : review.rdoproject.org/config/playbooks/tripleo-rdo-base/container-login.yaml@master] | 14:20 |
| akahat|rover | and it removed from master here: https://opendev.org/openstack/tripleo-ansible/commit/69bac5fdc5306c022cc17d0ecc6a9db8c5adab79 | 14:20 |
| akahat|rover | regarding this: https://logserver.rdoproject.org/openstack-periodic-integration-zed-centos9/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-9-scenario010-kvm-standalone-zed/8a282a5/job-output.txt | 14:21 |
| akahat|rover | lp: https://bugs.launchpad.net/tripleo/+bug/2001916 | 14:21 |
| akahat|rover | job broken since long time.. https://review.rdoproject.org/zuul/builds?job_name=periodic-tripleo-ci-centos-9-scenario010-kvm-standalone-wallaby&skip=0 | 14:22 |
| pojadhav|ruck | folks, sprint planning meeting in 4 mins : arxcruz, marios, ysandeep, bhagyashris, svyas, soniya29, pojadhav, akahat|rover , chandankumar, frenzy_friday, dviroel, rlandy, dasm, Tengu, jgilaber | 14:26 |
| pojadhav|ruck | https://hackmd.io/0ruKGb6eTwGvdvwWE4nk8g#2023-01-05-Unified-Sprint-73-Planning | 14:26 |
| pojadhav|ruck | https://meet.google.com/avd-ijsb-ejt?authuser=0 | 14:26 |
| *** ysandeep|afk is now known as ysandeep | 14:26 | |
| Tengu | wow.... my external compute is deployed. | 14:29 |
| pojadhav|ruck | dasm, chandankumar, dviroel : planning meeting | 14:30 |
| * pojadhav|ruck dinner | 15:44 | |
| chandankumar | see ya people tomorrow! | 15:45 |
| rlandy | akahat|rover: marios: ysandeep: ok to merge this after gate clears failing tests? https://review.opendev.org/c/openstack/tripleo-quickstart/+/868605 | 15:48 |
| rlandy | I voted | 15:48 |
| ysandeep | rlandy, yes | 15:48 |
| ysandeep | 10/10 job passed with systemd in exclude | 15:49 |
| rlandy | great | 15:50 |
| rlandy | bhagyashris; hi - can you come to review meeting on friday and we can merge up your patches? | 15:51 |
| ysandeep | rlandy, for tomorrow's podified friday call.. doug/me would like us to spend sometime dividing our work in jira tasks. | 15:51 |
| ysandeep | dviroel, ^^ | 15:51 |
| rlandy | ysandeep++ | 15:52 |
| rlandy | sounds good | 15:52 |
| * ysandeep out, see everyone tomorrow | 15:59 | |
| *** ysandeep is now known as ysandeep|out | 16:00 | |
| *** dviroel is now known as dviroel|lunch | 16:01 | |
| *** soniya29 is now known as soniya29|out | 16:08 | |
| *** pojadhav|ruck is now known as pojadhav|out | 16:13 | |
| bhagyashris | rlandy, sure | 16:32 |
| Tengu | chandankumar: fyi, EDPM is now deploying fine on tripleo-lab, had to tweak some stuff. I have a better understanding about the CI job itself, and I'm confident we should get something fine shortly now. | 16:32 |
| * marios off in few have a good one oooq o/ | 16:32 | |
| rlandy | rechecking https://review.opendev.org/c/openstack/tripleo-ansible/+/869020 | 16:33 |
| rlandy | marios: have a good night | 16:33 |
| Tengu | rlandy: is the Zed patch merged? the one about the missing files? | 16:33 |
| marios | thanks rlandy | 16:33 |
| Tengu | it is | 16:33 |
| Tengu | \o/ | 16:33 |
| rlandy | <opendevreview> Merged openstack/tripleo-common stable/zed: Fix missing 50-edk2-ovmf-*.json files handling https://review.opendev.org/c/openstack/tripleo-common/+/869207 | 16:33 |
| marios | yes https://review.opendev.org/c/openstack/tripleo-common/+/869207 | 16:34 |
| marios | finally | 16:34 |
| rlandy | Tengu: and the upgrades voting | 16:34 |
| Tengu | yep - and I saw the last patch of that serie, re-enabling the upgrades jobs | 16:34 |
| Tengu | yeah | 16:34 |
| Tengu | that was a nice run. | 16:34 |
| rlandy | rechecking https://review.opendev.org/c/openstack/tripleo-ansible/+/869020 and | 16:34 |
| rlandy | the growvols wo | 16:34 |
| *** marios is now known as marios|out | 16:35 | |
| rlandy | https://review.opendev.org/c/openstack/tripleo-quickstart/+/868605 | 16:35 |
| rlandy | lunch - brb | 16:35 |
| Tengu | ok folks - getting offline. See you tomorrow | 16:46 |
| dasm | Tengu: o/ | 17:02 |
| *** dviroel|lunch is now known as dviroel | 17:10 | |
| dasm | dviroel: o/ can you +2 this one? it lost reviews due to rebasing: https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/45669 | 17:35 |
| reviewbot | Do you want me to add your patch to the Review list? Please type something like add to review list <your_patch> so that I can understand. Thanks. | 17:35 |
| dasm | dviroel: rlandy would be able to +W outstanding patches | 17:35 |
| * dviroel check | 17:36 | |
| akahat|rover | rlandy, https://review.opendev.org/c/openstack/tripleo-quickstart/+/868605 is good to go, voted. | 17:55 |
| rlandy | akahat|rover: thanks | 17:56 |
| * akahat|rover will be out in few mins.. see you tomorrow! | 17:58 | |
| rlandy | dviroel: ok - one left of dasm''s patches to look at pls: https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/45699 | 18:05 |
| rlandy | if you agree, I'll w that | 18:05 |
| dviroel | rlandy: done, lgtm | 18:36 |
| dasm | thanks | 18:38 |
| rlandy | dasm: ok - that should be all of your patches | 18:51 |
| rlandy | dasm: any idea what's going on here with tox test failures: https://review.opendev.org/c/openstack/tripleo-repos/+/866316? | 18:53 |
| rlandy | dropping review time ... | 19:02 |
| rlandy | just need one more core vote on: | 19:02 |
| rlandy | https://review.opendev.org/c/openstack/tripleo-quickstart/+/868605/ Exclude latest systemd*-252-* packages | 19:02 |
| dasm | just joined for review. no one's here. dropping and going over the list on my own | 19:05 |
| *** dviroel is now known as dviroel|afk | 21:19 | |
| *** rlandy is now known as rlandy|out | 22:13 | |
| dasm | o/ | 22:33 |
| *** dasm is now known as dasm|off | 22:33 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!