*** altlogbot_3 has quit IRC | 00:23 | |
*** irclogbot_0 has quit IRC | 00:23 | |
*** Goneri has quit IRC | 01:02 | |
*** irclogbot_2 has joined #oooq | 01:33 | |
*** irclogbot_2 has quit IRC | 01:39 | |
*** irclogbot_3 has joined #oooq | 01:39 | |
*** irclogbot_3 has quit IRC | 01:43 | |
*** apetrich has quit IRC | 02:10 | |
*** irclogbot_1 has joined #oooq | 02:53 | |
*** irclogbot_1 has quit IRC | 02:59 | |
*** skramaja has joined #oooq | 03:08 | |
*** ykarel|away has joined #oooq | 03:09 | |
*** irclogbot_2 has joined #oooq | 03:33 | |
*** irclogbot_2 has quit IRC | 03:39 | |
*** gkadam has joined #oooq | 03:59 | |
*** surpatil has joined #oooq | 04:00 | |
*** gkadam_ has joined #oooq | 04:01 | |
*** gkadam_ has quit IRC | 04:01 | |
*** irclogbot_3 has joined #oooq | 04:01 | |
*** surpatil has joined #oooq | 04:02 | |
*** gkadam has quit IRC | 04:03 | |
*** irclogbot_3 has quit IRC | 04:07 | |
*** ykarel|away has quit IRC | 04:14 | |
*** udesale has joined #oooq | 04:19 | |
*** irclogbot_1 has joined #oooq | 04:21 | |
*** irclogbot_1 has quit IRC | 04:33 | |
*** ykarel|away has joined #oooq | 04:35 | |
*** soniya29 has joined #oooq | 04:37 | |
*** surpatil has quit IRC | 04:39 | |
*** ykarel|away is now known as ykarel | 04:49 | |
*** raukadah is now known as chandankumar | 04:55 | |
*** irclogbot_2 has joined #oooq | 05:20 | |
*** marios has joined #oooq | 05:22 | |
*** irclogbot_2 has quit IRC | 05:25 | |
*** soniya29 has quit IRC | 05:49 | |
*** rfolco has quit IRC | 06:00 | |
*** jtomasek has joined #oooq | 06:06 | |
*** ratailor has joined #oooq | 06:07 | |
*** ratailor has quit IRC | 06:07 | |
*** ratailor has joined #oooq | 06:08 | |
*** ykarel has quit IRC | 06:12 | |
*** jfrancoa has joined #oooq | 06:14 | |
*** udesale has quit IRC | 06:15 | |
*** ykarel has joined #oooq | 06:15 | |
*** irclogbot_0 has joined #oooq | 06:16 | |
*** jfrancoa has quit IRC | 06:18 | |
*** irclogbot_0 has quit IRC | 06:21 | |
*** udesale has joined #oooq | 06:22 | |
*** holser has joined #oooq | 06:23 | |
*** irclogbot_3 has joined #oooq | 06:24 | |
*** ratailor has quit IRC | 06:24 | |
*** irclogbot_3 has quit IRC | 06:29 | |
*** jfrancoa has joined #oooq | 06:32 | |
*** bogdando has joined #oooq | 06:38 | |
*** sshnaidm|pto is now known as sshnaidm|afk | 06:45 | |
*** zbr has quit IRC | 06:58 | |
*** zbr has joined #oooq | 07:08 | |
*** ratailor has joined #oooq | 07:10 | |
zbr | morning. anyone around? | 07:11 |
---|---|---|
zbr | chandankumar: panda|rover : need help with https://review.opendev.org/#/c/679756/1 | 07:13 |
*** chem has joined #oooq | 07:22 | |
*** apetrich has joined #oooq | 07:23 | |
*** tosky has joined #oooq | 07:25 | |
*** surpatil has joined #oooq | 07:27 | |
surpatil | hello | 07:27 |
*** irclogbot_1 has joined #oooq | 07:30 | |
*** ratailor has quit IRC | 07:30 | |
*** irclogbot_1 has quit IRC | 07:35 | |
*** jpena|off is now known as jpena | 07:39 | |
*** kopecmartin|off is now known as kopecmartin | 07:40 | |
*** brault has joined #oooq | 07:41 | |
*** udesale has quit IRC | 07:42 | |
*** udesale has joined #oooq | 07:43 | |
*** ykarel has quit IRC | 07:43 | |
*** ykarel has joined #oooq | 07:45 | |
*** ykarel is now known as ykarel|lunch | 07:45 | |
*** irclogbot_1 has joined #oooq | 07:50 | |
*** irclogbot_1 has quit IRC | 07:53 | |
zbr | hi | 08:06 |
* marios pokerface | 08:06 | |
zbr | i guess that soon we could start counting build costs in MW, maybe we can get our own nuclear-power plant? see https://review.opendev.org/#/c/677256/ -- and multiply by number of rechecks. | 08:08 |
panda|rover | zbr: just the regular 28 jobs per recheck | 08:15 |
* panda|rover trollface | 08:15 | |
marios | counterproposal zbr ruck and rover should have a bicycle connected to the zuul power supply | 08:15 |
marios | we will all be super fit by end of year | 08:15 |
marios | (or dead) | 08:15 |
marios | still | 08:15 |
marios | worth a try no? | 08:15 |
panda|rover | +1 | 08:19 |
panda|rover | zbr: commented your patch | 08:20 |
*** irclogbot_0 has joined #oooq | 08:22 | |
*** sanjayu_ has joined #oooq | 08:23 | |
panda|rover | why is irclogbot_0 hailing from a host called "mail.leafe.com" ? | 08:23 |
panda|rover | irclogbot_0: are you real ? | 08:24 |
*** irclogbot_0 has quit IRC | 08:25 | |
marios | exception ?! ^ | 08:26 |
panda|rover | raise | 08:27 |
panda|rover | marios: did you see my comments to your comments ? | 08:28 |
panda|rover | marios: do you have any comments to the comments to your comments ? | 08:29 |
*** derekh has joined #oooq | 08:36 | |
*** ratailor has joined #oooq | 08:39 | |
*** ratailor has quit IRC | 08:39 | |
*** ratailor has joined #oooq | 08:40 | |
chandankumar | panda|rover: Morning | 08:42 |
chandankumar | panda|rover: I have added this patch https://review.opendev.org/#/c/679932/1 in tripleo-ansible for podman login and it leads to this error https://logs.rdoproject.org/93/21593/43/check/registry-login-rdo-integration-redhat-8/23c601d/job-output.txt.gz#_2019-09-04_08_36_21_489156 | 08:43 |
chandankumar | I am going to update the patch https://review.rdoproject.org/r/#/c/21593/ | 08:44 |
chandankumar | one more question How can I taste these jobs locally? | 08:44 |
panda|rover | they added a wonderful option in zuul, if you modify a parent configuration, a child add itself to the tests. | 08:44 |
panda|rover | chandankumar: the integration jobs ? | 08:45 |
chandankumar | panda|rover: yes | 08:45 |
panda|rover | chandankumar: they are integration job, and they are testing the integration between login roles and rdo nodes | 08:46 |
panda|rover | chandankumar: by their nature, you can't test them locally, because you need rdo nodes. The only option would be the reproducer | 08:47 |
panda|rover | chandankumar: fortunately they take only a few minutes to run | 08:48 |
panda|rover | chandankumar: you may want to make that explicit in a comment to the job configuration. They integrate the login role with the zuul flow for rdo job in rdo nodes. | 08:51 |
chandankumar | panda|rover: sure will add the same in tripleo-ansible | 08:52 |
*** ratailor has quit IRC | 08:55 | |
*** brault has quit IRC | 08:56 | |
*** ykarel|lunch is now known as ykarel | 08:59 | |
chandankumar | kopecmartin: Hello | 09:01 |
chandankumar | kopecmartin: left some comment on spec review | 09:01 |
kopecmartin | chandankumar: hi, i see, i'm looking at it .. you mentioned that the release should be bumped, but what about the version? shouldn't it be changed from 2.2.1 to 2.3.0? Also i noticed we didn't add a changelog nad didn't change a release in the patch from master https://review.rdoproject.org/r/#/c/21929/ and stein https://review.rdoproject.org/r/#/c/21930/ | 09:03 |
kopecmartin | is that wrong? | 09:03 |
chandankumar | kopecmartin: version will be bumped only when new tempestconf version will come | 09:04 |
chandankumar | does new version bump is merged in rdoinfo for all release? | 09:04 |
kopecmartin | chandankumar: except of queens yes, here are all the reviews related https://tree.taiga.io/project/tripleo-ci-board/task/1210 | 09:05 |
chandankumar | ok checking | 09:06 |
marios | panda|rover: zbr chandankumar sshnaidm|afk * please try review that when you next have time thanks https://review.opendev.org/#/c/663977 (turek patch for tagging we need it) | 09:07 |
chandankumar | kopecmartin: https://review.rdoproject.org/r/#/c/21926/ stein we need a release bump and changelog | 09:07 |
*** irclogbot_3 has joined #oooq | 09:08 | |
chandankumar | where 2.3.0 is landed there update the release and changelog | 09:08 |
chandankumar | and leave other review as it is | 09:08 |
panda|rover | marios: I reviewed that yesterday, but I got confused. | 09:10 |
marios | panda|rover: ack maybe revisit if you get chance and ask about the confusion | 09:11 |
marios | comment there | 09:11 |
marios | panda|rover: as you see in comments i had a discussion with rfolco about the tags wasn't clear for me | 09:11 |
*** irclogbot_3 has quit IRC | 09:11 | |
panda|rover | marios: triied to talk with rfolco yesterday, the doubts remain. | 09:12 |
panda|rover | marios: but I'll add the comments | 09:12 |
marios | panda|rover: ok we have a call today not sure you can make it but if you do you can also discuss there thanks otherwise i'll point to your comments | 09:12 |
*** dtrainor_ has joined #oooq | 09:16 | |
kopecmartin | chandankumar: ok, so if i get it correctly, the version is changed automatically when whatever somewhere happens .. good and regarding the spec change for rocky and queens - i just need to bump the release and add a changelog, correct? | 09:17 |
*** dsneddon_ has joined #oooq | 09:18 | |
chandankumar | kopecmartin: https://review.rdoproject.org/r/#/q/project:openstack/tempestconf-distgit | 09:18 |
chandankumar | kopecmartin: for rokcy and stein version bump is merged | 09:18 |
*** dsneddon has quit IRC | 09:18 | |
chandankumar | kopecmartin: for rocky update the release and changelog | 09:18 |
*** dtrainor has quit IRC | 09:18 | |
*** jjoyce has quit IRC | 09:19 | |
kopecmartin | ok, i get it | 09:19 |
*** jjoyce has joined #oooq | 09:19 | |
chandankumar | kopecmartin: for queens, first update the tempestconf patch | 09:19 |
*** surpatil has quit IRC | 09:19 | |
chandankumar | with openstacksdk | 09:20 |
chandankumar | then rebase the rdoinfo patch | 09:20 |
*** surpatil has joined #oooq | 09:20 | |
chandankumar | kopecmartin: got it? | 09:20 |
kopecmartin | chandankumar: yes | 09:20 |
*** dtantsur|afk is now known as dtantsur | 09:22 | |
chandankumar | zbr: Hello | 09:29 |
chandankumar | zbr: need some help here https://review.rdoproject.org/r/#/c/21593/44/playbooks/integration/registry-login.yaml@17 | 09:29 |
chandankumar | is there a way to include to two tasks file? | 09:30 |
zbr | i do not understand the question | 09:30 |
*** ykarel is now known as ykarel|afk | 09:30 | |
zbr | tbh, i never used "tasks_from" as I found it very weird option. | 09:32 |
chandankumar | zbr: How Can i pass two playbooks while using tasks_from when including a role? | 09:32 |
zbr | roles were designed to be included only once, when you include a role the 2nd time, ansible will (likely) skipping running the code again. | 09:33 |
zbr | but the final behavior depends on multiple factors. | 09:33 |
panda|rover | zbr: I think that option was disabled by default | 09:33 |
zbr | 1st: do not confuse tasks with playbooks. you pass tasks, not playbooks, | 09:34 |
zbr | i need to google able role inclusion in ansible and if there is a way to enforce running role again all tiime. | 09:34 |
zbr | mainly what you do is you add two include_role tasks one after another | 09:34 |
panda|rover | zbr: and the roles don't make much sense anymore. Roles were intended as "this server acts a a httpd server" | 09:34 |
panda|rover | chandankumar: why don't you just call install first and then login, without the loop ? | 09:36 |
chandankumar | panda|rover: that is also doable, let me do that | 09:37 |
zbr | yeah, avoid loop with includes at all costs! | 09:37 |
panda|rover | zbr: why ? | 09:38 |
*** sshnaidm|afk is now known as sshnaidm|ruck | 09:41 | |
zbr | running loop with groups of tasks was always problematic in ansible, regardless if they were implemented via includes or blocks. | 09:43 |
zbr | remember that ansible is not a programming language, even if the way we use it it makes use believe is very close to one. | 09:43 |
panda|rover | why ? | 09:43 |
zbr | search ansible issue tracker about bugs related to them, opened and closed. | 09:44 |
zbr | it will keep you busy till the end of the week. | 09:44 |
zbr | if you can avoid a loop, do it. | 09:44 |
zbr | loop on a single task is perfectly ok, but includes are *not* tasks. | 09:45 |
zbr | ... even if they are listed under "tasks" :D | 09:45 |
zbr | that is why I always used variable to tell role what to do, and avoided tasks_from construct. | 09:46 |
*** skramaja has quit IRC | 09:46 | |
zbr | to make a comparision with non coding, is like not calling a function by its entry point,... ;) | 09:46 |
zbr | if you do a jump at at specific address, expect a bumpy road ahead. | 09:47 |
panda|rover | zbr: I see it more as calling a specific function in a file, because you just need that function in that file and nothing else | 09:48 |
zbr | does this role have allow_duplicates: true in meta? | 09:49 |
panda|rover | zbr: I think it's enabled by default. | 09:49 |
zbr | *NOT* | 09:50 |
zbr | unless someone is adding it to the config, but ansible default is *not*. | 09:50 |
zbr | panda|rover: read https://docs.ansible.com/ansible/latest/user_guide/playbooks_reuse_roles.html -- especially the last note on allow_duplicates | 09:51 |
zbr | i did not test but I am ready to bet the without allow_duplicates, importing role twice with different params will not run the same code twice. | 09:54 |
zbr | but you are welcomed to try with multiple ansible versions, multiple ansible.cfg files, ... and so on. | 09:55 |
chandankumar | kopecmartin: commented on queens | 09:56 |
panda|rover | zbr: https://docs.ansible.com/ansible/latest/modules/include_role_module.html | 09:56 |
panda|rover | zbr: allow_duplicates: yes is the default for include_role | 09:56 |
kopecmartin | chandankumar: oh, ok, thanks | 09:57 |
chandankumar | chicken egg problem | 09:58 |
*** ykarel|afk is now known as ykarel | 09:58 | |
zbr | panda|rover: ok. how about import_role? or role: construct? or role: construct from zuul? i am curious as I got mixed messages digging online. | 09:59 |
marios | panda|rover: gonna start with linters working @ https://review.rdoproject.org/r/#/c/21895/ | 09:59 |
panda|rover | zbr: roles is the classic way, and there's no way to specify a allow_duplicated if not on meta.yaml | 09:59 |
panda|rover | marios: ok | 10:00 |
zbr | panda|rover: and that is what zuul is doing, right? | 10:00 |
panda|rover | zbr: weren't we talking about the test chandankumar is trying to run ? | 10:00 |
panda|rover | zbr: I'm not sure what zuul is using | 10:01 |
zbr | sure, diverged a little bit. | 10:01 |
panda|rover | zbr: but once you specify include_role, you're in control, not zuul. | 10:01 |
zbr | i wish that would be true.... not sure who is in control, maybe the buy behind the curtain | 10:02 |
chandankumar | kopecmartin: rebase this one https://review.rdoproject.org/r/#/c/21879/ | 10:07 |
zbr | chandankumar is going to hate me, added comments. luckly very easy to fix. | 10:10 |
chandankumar | zbr: hehe | 10:11 |
chandankumar | zbr: let precommit handle this stuff | 10:12 |
zbr | chandankumar: not all are checked, putting when just after name is not checked for example but is a very good practice | 10:15 |
zbr | especially with blocks | 10:16 |
panda|rover | zbr: how do I decrease your verbosity ? | 10:16 |
panda|rover | zbr -v 0 | 10:16 |
zbr | NotImplemented() | 10:17 |
panda|rover | zbr-ctrl verbosity-level ERROR | 10:17 |
zbr | panda|rover: can you access https://sbarnea.com/ ? | 10:23 |
panda|rover | zbr: is that your api endpoint ? | 10:24 |
panda|rover | zbr-ctrl --host sbarnea.com verbosity-level ERROR | 10:25 |
zbr | not yet, soon. | 10:25 |
*** irclogbot_3 has joined #oooq | 10:28 | |
*** udesale has quit IRC | 10:32 | |
*** udesale has joined #oooq | 10:32 | |
*** irclogbot_3 has quit IRC | 10:33 | |
chandankumar | panda|rover: https://review.opendev.org/#/c/673481/32/tasks/registry-login.yml | 10:35 |
chandankumar | panda|rover: here is container_registry_login is used but in podman tripleo_container_registry_logins is used | 10:35 |
chandankumar | :-( | 10:35 |
chandankumar | *triple-podman | 10:35 |
* chandankumar updates the patch | 10:36 | |
panda|rover | chandankumar: oh, ok | 10:36 |
panda|rover | chandankumar: remove also line 2 | 10:37 |
chandankumar | ok | 10:37 |
panda|rover | chandankumar: was used only to block the change to merge | 10:37 |
chandankumar | ok | 10:37 |
chandankumar | panda|rover: http://codesearch.openstack.org/?q=container_registry_logins&i=nope&files=&repos= - big mess | 10:41 |
*** irclogbot_0 has joined #oooq | 10:42 | |
chandankumar | panda|rover: on renaming the var it will break the tht | 10:42 |
panda|rover | chandankumar: change the vars we are passing ... | 10:44 |
chandankumar | panda|rover: yes making change in rdo-jobs aptch | 10:45 |
panda|rover | chandankumar: ok. I have one question | 10:45 |
chandankumar | yes | 10:45 |
*** irclogbot_0 has quit IRC | 10:45 | |
panda|rover | chandankumar: has having the integration job helped you taking over the task ? Set the expectation maybe on what should be done ? | 10:46 |
panda|rover | ok, that's two questions | 10:47 |
chandankumar | panda|rover: yes, it helped and it also helps in understanding the stuff getting added in the role | 10:50 |
panda|rover | chandankumar: that's partialy what TDD is, setting the expectations fo your task upfront | 10:52 |
sshnaidm|ruck | zbr, check it again please https://review.opendev.org/#/c/679756/ | 10:52 |
sshnaidm|ruck | zbr, and https://review.opendev.org/#/c/679564/ | 10:52 |
zbr | sshnaidm|ruck: yep, i know just fixed few seconds ago. | 10:52 |
sshnaidm|ruck | zbr, comment about tee doesn't make sense there | 10:53 |
zbr | `curl -s --fail --show-error` should do the trick. | 10:53 |
zbr | sshnaidm|ruck: right, but I cannot remove the comment. | 10:53 |
sshnaidm|ruck | zbr, you don't need show-error for showing error | 10:53 |
sshnaidm|ruck | zbr, just need to save it to file before it's eaten by pipe | 10:54 |
zbr | here we have disagree, i am against create another file for each script, there is no end-to to that. error should be visible on stderr, and that is what show-error does. | 10:54 |
zbr | we always have two stream, we should not forget about stderr -- and lucky for us curl can respect it | 10:55 |
zbr | with these 3 params curl should send data to grep only if the request is sucessful. | 10:56 |
zbr | did some tests: invalid server, no response, invalid url (404),... valid url. | 10:56 |
*** irclogbot_0 has joined #oooq | 10:58 | |
panda|rover | zbr: I tested unresolvable host and 404 | 10:58 |
panda|rover | one is in stderr, the other is in stout | 10:58 |
panda|rover | 404 is not considered an erro | 10:59 |
panda|rover | r | 10:59 |
zbr | since when? | 10:59 |
zbr | http spec is clear, 2xx are success. | 11:00 |
zbr | also that ansible code does not say a thing about considering a 404 a non error. | 11:00 |
panda|rover | zbr-ctrl set argumentative-level BASIC | 11:01 |
panda|rover | zbr: 404 is not an error for curl, culr completed the request, and the result is a 404 | 11:01 |
panda|rover | but it's not considered an error for curl | 11:01 |
*** irclogbot_0 has quit IRC | 11:02 | |
zbr | panda|rover: have you read the doc of --fail option on curl? | 11:02 |
panda|rover | zbr: ---fail is an hack, exactly to address this dicotomy between curl errors and http errors | 11:04 |
panda|rover | has the get_url module been rewritten ? | 11:05 |
zbr | panda|rover: in this case, you are welcomed to reimplement the dlrn parsing in pure ansible (or even writing a module for that) | 11:05 |
panda|rover | I remember it was worse than curl back in 2.4 maybe now it's more usable | 11:05 |
zbr | panda|rover: it is better but not amazing afaik. | 11:06 |
panda|rover | zbr: whatever you do, you need to reimplement something. using --fail fails silently for 404, and you have to interpret the exit code of curl | 11:06 |
panda|rover | if $? == 22 then "we have a 4040" | 11:07 |
zbr | panda|rover: please TEST before saying stuff | 11:07 |
panda|rover | iptables -s zbr -j DROP | 11:07 |
zbr | https://seashells.io/v/RRjjkScw | 11:07 |
panda|rover | /ignore --all zbr | 11:07 |
zbr | panda|rover: the entire trinity is needed: -s --fail --show-error | 11:08 |
zbr | with this you will get usable output on stderr | 11:08 |
zbr | there is only one use case which is not covered yet, 401 at least based on docs but others are. | 11:09 |
panda|rover | zbr: -w writes on stdout | 11:09 |
zbr | curl -s --fail --show-error https://httpbin.org/status/401 > /dev/null | 11:10 |
zbr | this combination works for all cases. | 11:10 |
zbr | try other values and let me know if you spot something that returns success on a failure | 11:10 |
zbr | ignore the -w -- i used it for debug. | 11:10 |
panda|rover | zbr: too many digressions. What was the original issue again ? | 11:11 |
panda|rover | -w can be used to write to stdout | 11:11 |
panda|rover | stderr* | 11:11 |
zbr | forgetting the digressions I am pleased I learned two useful things today: how to use curl correctly, and how allow-duplicates works. | 11:11 |
*** udesale has quit IRC | 11:12 | |
chandankumar | panda|rover: https://logs.rdoproject.org/93/21593/46/check/registry-login-rdo-integration-redhat-8/7dc3df2/job-output.txt.gz#_2019-09-04_11_03_07_025733 | 11:13 |
chandankumar | panda|rover: I reached here but assert is not working | 11:13 |
chandankumar | the task itself got failed | 11:13 |
chandankumar | Do I need to add a resume mode there? | 11:13 |
chandankumar | *rescue mode in podman itself | 11:14 |
panda|rover | chandankumar: you can use the rescue in the playbook in rdo-jobs | 11:14 |
panda|rover | chandankumar: in particular we are interested to see it if fails for invalid credentials. if it's failing for any other error, we fail the playbook | 11:15 |
zbr | i even include test commands on https://review.opendev.org/#/c/679756/ | 11:16 |
*** zbr is now known as zbr|swimming | 11:23 | |
*** jpena is now known as jpena|lunch | 11:28 | |
*** soniya29 has joined #oooq | 11:34 | |
*** irclogbot_0 has joined #oooq | 11:42 | |
*** irclogbot_0 has quit IRC | 11:47 | |
chandankumar | marios: https://review.rdoproject.org/r/#/c/21547/ please have a look thanks! | 11:52 |
chandankumar | :x | 11:57 |
panda|rover | chandankumar added I comment to the patch. When you have the integration playbook ready, copy it to the config patch | 12:00 |
panda|rover | chandankumar: I will not +2 unless the playbook is the same, at least for the invocation part, not for the tests | 12:00 |
panda|rover | chandankumar: that's the only way to guarantee we are not merging broken stuff | 12:01 |
chandankumar | panda|rover: sure | 12:01 |
*** ykarel is now known as ykarel|afk | 12:01 | |
*** tesseract has joined #oooq | 12:03 | |
sshnaidm|ruck | where do new packages arrive first - in rhel or centos? | 12:07 |
*** rfolco has joined #oooq | 12:08 | |
*** ykarel|afk has quit IRC | 12:09 | |
chandankumar | sshnaidm|ruck: It comes in Both, we have parallel delorean builders | 12:09 |
sshnaidm|ruck | chandankumar, also for non-openstack packages? | 12:10 |
sshnaidm|ruck | jpena|lunch, ^^ | 12:11 |
chandankumar | sshnaidm|ruck: nope, it depends both are managed independently | 12:11 |
chandankumar | In RHEL, it comes from RHUI, totally depends on RHEL schedule | 12:13 |
chandankumar | In centos land, the packages needs to be available in Fedora if it is python and then comes to centos otherwise different centos-sig takes care of the packages | 12:13 |
weshay | sshnaidm|ruck fyi http://pastebin.test.redhat.com/794293 | 12:20 |
weshay | maybe dlrn is not quite back onlinew | 12:21 |
weshay | online | 12:21 |
panda|rover | weshay: consequence of the outage, should be fixed now | 12:21 |
weshay | panda|rover that was from today.. | 12:21 |
sshnaidm|ruck | weshay, which timezone? I saw it working | 12:21 |
weshay | unless I'm missing something.. or another outtage | 12:22 |
sshnaidm|ruck | weshay, when exactly? | 12:22 |
weshay | 2019-09-04 04:02:16.257842 | 12:22 |
panda|rover | weshay: sshnaidm|ruck reported here https://bugs.launchpad.net/tripleo/+bug/1842448 and https://trello.com/c/Cz787NGG/1099-cixlp1842448tripleociproa-outage-on-trunkrdoprojectsorg-is-causing-all-the-ci-jobs-to-fail comments from jpena|lunch | 12:22 |
openstack | Launchpad bug 1842448 in tripleo "Outage on trunk.rdoprojects.org is causing all the CI jobs to fail" [Critical,In progress] - Assigned to Gabriele Cerami (gcerami) | 12:22 |
sshnaidm|ruck | weshay, tz? | 12:23 |
weshay | sshnaidm|ruck it's an rdo queens periodic.. so I'm guessing utc | 12:23 |
sshnaidm|ruck | weshay, let's not focus on old errors | 12:23 |
panda|rover | weshay: sshnaidm|ruck it was an error in dlrn api fixed by jpena this morning. | 12:25 |
sshnaidm|ruck | panda|rover, yeah, saw this in etherpad, periodic jobs run and report now, no any problem | 12:26 |
sshnaidm|ruck | weshay, why does nobody ask about oooci? :( | 12:28 |
panda|rover | we are green | 12:28 |
panda|rover | like hulk | 12:29 |
*** jpena|lunch is now known as jpena | 12:32 | |
*** soniya29 has quit IRC | 12:32 | |
*** ykarel has joined #oooq | 12:33 | |
weshay | sshnaidm|ruck I'm not trying to focus on old errors, just looking at the latest runs in periodic | 12:35 |
weshay | if it's old.. that's fine.. I'll recheck tomorrow | 12:35 |
sshnaidm|ruck | weshay, we have it covered in https://etherpad.openstack.org/p/ruckroversprint15 | 12:35 |
sshnaidm|ruck | panda|rover, can you please add lp to etherpad? ^ | 12:36 |
weshay | cool thanks sshnaidm|ruck | 12:36 |
sshnaidm|ruck | weshay, did you see my mail about reducing ovb jobs? | 12:38 |
chandankumar | sshnaidm|ruck: https://review.opendev.org/#/c/679932/ please have alook | 12:40 |
sshnaidm|ruck | chandankumar, hmm.. shouldn't it have no_log? | 12:41 |
*** zbr|swimming is now known as zbr | 12:42 | |
chandankumar | sshnaidm|ruck: yes correct | 12:42 |
chandankumar | sshnaidm|ruck: done | 12:45 |
zbr | i need to ask again: can't we switch to a permalink for ruckrover etherpad instead of creating another one each sprint? making CR to update urls,... how about having only one. | 12:45 |
sshnaidm|ruck | chandankumar, why do you register it? How should it help? | 12:47 |
*** udesale has joined #oooq | 12:48 | |
chandankumar | sshnaidm|ruck: in order to do the integration testing in RDO-jobs and test the podman functionality we are doing it https://review.rdoproject.org/r/#/c/21593/ | 12:48 |
chandankumar | the main goal is to reuse the same tripleo-podman role in rdo config repo for authenication | 12:49 |
chandankumar | having testing in place helps it | 12:49 |
*** surpatil has quit IRC | 12:52 | |
sshnaidm|ruck | chandankumar, I see, it looks like no_log is not necessary because it prints output like that: http://paste.openstack.org/show/770644/ and env vars are not populated | 12:56 |
sshnaidm|ruck | chandankumar, sorry for noise | 12:56 |
panda|rover | also, we don't rely on the single roles | 12:57 |
panda|rover | in config there's a big no_log:true at the start of the playbook | 12:57 |
panda|rover | so we don't accidentally leak passwords even if roles are | 12:57 |
marios | rfolco: panda|rover want to sync in couple mins as per calendar invite | 12:58 |
marios | panda|rover: don't worry if you're busy with rucking | 12:58 |
panda|rover | marios: in the worst case I'll ignore everything you say | 12:59 |
chandankumar | sshnaidm|ruck: no problem, removed | 12:59 |
marios | panda|rover: thanks ! as usual then :D | 12:59 |
* chandankumar is feeling lonely, marios please also add me to meetings | 13:00 | |
panda|rover | chandankumar: no you no. | 13:00 |
marios | panda|rover: rfolco can you hear me | 13:02 |
panda|rover | marios: we can't hear you | 13:02 |
marios | i think no? | 13:02 |
marios | panda|rover: k | 13:02 |
*** bogdando has quit IRC | 13:02 | |
*** bogdando has joined #oooq | 13:03 | |
panda|rover | the optional invitee is the only one in the meeting now :) | 13:03 |
weshay | sshnaidm|ruck qq.. is there any patern to the deployment failures for fs001 on vexx? | 13:05 |
weshay | timeserver? chrony | 13:06 |
weshay | rfolco can you please update the owners in https://tree.taiga.io/project/tripleo-ci-board/epic/1030 | 13:10 |
rfolco | weshay, yes, fixing. Sorry, updated tasks only. Will reflect on user stories. | 13:15 |
weshay | thanks | 13:15 |
chandankumar | sshnaidm|ruck: https://review.opendev.org/#/c/679932/ tested here https://review.rdoproject.org/r/#/c/21593/ and logs here https://logs.rdoproject.org/93/21593/49/check/registry-login-rdo-integration-redhat-8/b36fd27/job-output.txt.gz#_2019-09-04_13_07_01_059851 | 13:21 |
sshnaidm|ruck | chandankumar, ack | 13:24 |
*** Vorrtex has joined #oooq | 13:29 | |
*** chem` has joined #oooq | 13:33 | |
*** chem has quit IRC | 13:34 | |
*** irclogbot_0 has joined #oooq | 13:38 | |
*** udesale has quit IRC | 13:39 | |
*** udesale has joined #oooq | 13:39 | |
panda|rover | chandankumar: added comment to https://review.rdoproject.org/r/21593 | 13:40 |
*** irclogbot_0 has quit IRC | 13:42 | |
zbr | marios: do you have a minute? | 13:51 |
marios | o/ zbr | 13:51 |
zbr | marios: look at https://review.opendev.org/#/c/678816/1 -- and click the fist job results, you will see filename in task name. before that it was impossible to see the filename in the output due to default verbosity. | 13:52 |
zbr | mainly it does improve the user experience, otherwise you are forcing the log-reader to go to the source code to figure out what is the filename/ | 13:53 |
zbr | we have this problem in many places because we assumed that stdout/stderror are allways visible, guess what they are not always. | 13:53 |
marios | zbr: will see filename in task name. - where please point so i can follow | 13:54 |
marios | zbr: looking there for example https://openstack.fortnebula.com:13808/v1/AUTH_e8fd161dc34c421a979a9e6421f823e9/logs_16/678816/1/check/tripleo-build-containers-centos-7/eadba06/ | 13:54 |
zbr | https://zuul.opendev.org/t/openstack/build/eadba06616594c6a92cc27e859560764/console | 13:54 |
sshnaidm|ruck | weshay, what we'll do with https://trello.com/c/P7H0Ao6b/1090-cixlp1841096tripleociproa-fluentd-missing-deps-in-building-containers-for-rhel-8 | 13:56 |
sshnaidm|ruck | weshay, is it still relevant? | 13:56 |
weshay | sshnaidm|ruck that patch is in the gate I think | 13:56 |
weshay | to switch to rsyslog | 13:56 |
sshnaidm|ruck | weshay, ok, so this card is solved, we don't need fluentd and its deps | 13:57 |
marios | zbr: ack so you just want it on the task name that is the whole point | 13:57 |
weshay | sshnaidm|ruck correct | 13:57 |
marios | zbr: it doesn't change anything else | 13:57 |
weshay | sshnaidm|ruck ack to close | 13:57 |
zbr | marios: exactly, like the task after it which already does this. | 13:57 |
marios | zbr: k thanks for ping revoting and pasting some of this there | 13:57 |
*** irclogbot_1 has joined #oooq | 13:58 | |
weshay | sshnaidm|ruck so we'll need to see how the packagers want to handle that.. I'll raise it in the meeting I'm in | 14:01 |
weshay | sshnaidm|ruck will let you know | 14:01 |
*** irclogbot_1 has quit IRC | 14:02 | |
*** irclogbot_1 has joined #oooq | 14:04 | |
panda|rover | chandankumar: are there many jobs taht are still using validate_tempest out there ? | 14:07 |
chandankumar | panda|rover: all multinode jobs using validate-tempest | 14:07 |
*** irclogbot_1 has quit IRC | 14:08 | |
chandankumar | except fs010 | 14:09 |
*** irclogbot_0 has joined #oooq | 14:10 | |
chandankumar | sshnaidm|ruck: moved your code at correct place https://review.opendev.org/#/c/680004/ | 14:11 |
*** irclogbot_0 has quit IRC | 14:14 | |
sshnaidm|ruck | chandankumar, ok, I wonder if we can just detect default python interpreter, but it's for followup maybe | 14:17 |
weshay | sshnaidm|ruck marios https://github.com/openstack/tripleo-quickstart/blob/master/config/release/tripleo-ci/CentOS-7/master.yml#L145 vs.. https://github.com/openstack/tripleo-quickstart/blob/master/config/release/tripleo-ci/RedHat-8/master.yml#L112 | 14:17 |
chandankumar | or better switch to ansible_python_interpreter | 14:17 |
sshnaidm|ruck | chandankumar, yeah, maybe.. | 14:17 |
weshay | marios zbr can you guys please make sure the rhel8 master.yml release file is up to date w/ centos-7 master.. we're missing ceph-ansible install at least | 14:18 |
sshnaidm|ruck | weshay, ok, so it even doesn't try | 14:18 |
weshay | aye | 14:18 |
sshnaidm|ruck | weshay, let's give it a chance | 14:18 |
*** radez has quit IRC | 14:18 | |
*** radez has joined #oooq | 14:19 | |
weshay | sshnaidm|ruck you got me covered for the cix call? I have a conflict | 14:21 |
sshnaidm|ruck | weshay, yep | 14:21 |
chandankumar | weshay: https://review.opendev.org/#/c/680004/ | 14:22 |
chandankumar | switched to ansible python interprter | 14:22 |
chandankumar | it will work | 14:22 |
chandankumar | with venv | 14:22 |
chandankumar | sshnaidm|ruck: needs +w on this https://review.opendev.org/#/c/679932/ | 14:23 |
chandankumar | https://tree.taiga.io/project/tripleo-ci-board/task/1182 patches are ready to get merged in order | 14:24 |
marios | weshay: ack sshnaidm|ruck are you doing it? | 14:24 |
marios | or zbr or should i add ? | 14:24 |
marios | 17:18 < weshay> marios zbr can you guys please make sure the rhel8 master.yml release file is up to date w/ centos-7 master.. we're missing ceph-ansible install at least | 14:24 |
weshay | not i | 14:24 |
marios | ;) | 14:24 |
marios | sshnaidm|ruck: is doing it /testing with https://review.rdoproject.org/r/21945 | 14:25 |
marios | https://review.opendev.org/#/c/680007/ | 14:26 |
chandankumar | Does ceph-ansible is available in RHUI? | 14:29 |
*** tesseract has quit IRC | 14:30 | |
weshay | chandankumar sshnaidm|ruck I heard it should be thee, but I haven't been able to verify yet | 14:31 |
weshay | probably is there | 14:31 |
sshnaidm|ruck | chandankumar, we'll see in https://review.rdoproject.org/r/#/c/21945/ | 14:32 |
zbr | weshay: marios : please have a look at https://review.opendev.org/#/c/679825/ f28 removal on tq | 14:32 |
marios | zbr: k in few mins tho | 14:33 |
*** irclogbot_3 has joined #oooq | 14:36 | |
chandankumar | sshnaidm|ruck: reusing the existing patch https://review.opendev.org/#/c/617522/ | 14:37 |
chandankumar | jpena: Hello | 14:39 |
*** tesseract has joined #oooq | 14:39 | |
chandankumar | jpena: In RHEL-8 ceph-ansible comes from RHUI as in centos it comes from different repo? | 14:39 |
*** irclogbot_3 has quit IRC | 14:40 | |
jpena | chandankumar: I think so, in centos it should come from the ceph repo | 14:40 |
chandankumar | jpena: /me checks in the RHUI repo | 14:41 |
sshnaidm|ruck | chandankumar, ansible_python_interpreter' is undefined | 14:43 |
*** tesseract has quit IRC | 14:44 | |
sshnaidm|ruck | chandankumar, seems like need to gather facts | 14:44 |
chandankumar | yes might be | 14:45 |
*** ykarel is now known as ykarel|afk | 14:46 | |
chandankumar | weshay: jpena ceph-ansible noarch 4.0.0-0.1.rc9.el8cp rhui-custom-deps 189 k | 14:47 |
chandankumar | it is available in RHUI | 14:47 |
*** irclogbot_0 has joined #oooq | 14:50 | |
*** irclogbot_0 has quit IRC | 14:54 | |
*** ykarel|afk is now known as ykarel | 15:00 | |
chandankumar | time to log out now, see ya tomorrow | 15:02 |
*** chandankumar is now known as raukadah | 15:02 | |
panda|rover | raukadah: I don't believe you | 15:02 |
panda|rover | raukadah: I'll start kckbanning you so you don't work on the evening. | 15:03 |
*** Goneri has joined #oooq | 15:11 | |
weshay | +1 | 15:11 |
*** bogdando has quit IRC | 15:12 | |
*** tesseract has joined #oooq | 15:14 | |
*** irclogbot_0 has joined #oooq | 15:18 | |
*** irclogbot_0 has quit IRC | 15:22 | |
holser | Good morning folks | 15:22 |
holser | Can we retrigger only failed jobs such as tripleo-ci-centos-7-containers-multinode | 15:22 |
holser | ? | 15:23 |
weshay | holser for a gerrit change | 15:24 |
weshay | ? | 15:24 |
holser | yeah | 15:24 |
weshay | that would be nice wouldn't it | 15:24 |
holser | indeed | 15:24 |
holser | like "recheck <name>" | 15:25 |
weshay | afraid not | 15:25 |
holser | Why should we waste CI resources? | 15:25 |
weshay | not our policy, that opendev policy | 15:25 |
weshay | that is | 15:25 |
*** zbr is now known as zbr|out | 15:25 | |
marios | holser: i mean, for development you can run just the job you want by updating the zuul layout but for recheck i don't know never seen that | 15:25 |
holser | weshay, thank you very much for explanation | 15:25 |
marios | holser: and weshay is confirming | 15:25 |
* marios almost out | 15:26 | |
weshay | holser if you are only interested in iterating on a particular job until it works.. there are ways to do that | 15:26 |
weshay | holser but to get zuul to +1 all the jobs need to run at the same time | 15:26 |
holser | I see | 15:26 |
weshay | ya | 15:26 |
holser | otherwise it looses the state | 15:26 |
holser | gotcha | 15:27 |
holser | all or nothing | 15:27 |
*** jfrancoa has quit IRC | 15:32 | |
*** irclogbot_1 has joined #oooq | 15:32 | |
*** irclogbot_1 has quit IRC | 15:36 | |
*** irclogbot_3 has joined #oooq | 15:36 | |
*** altlogbot_1 has joined #oooq | 15:38 | |
*** marios is now known as marios|out | 15:44 | |
*** marios|out has quit IRC | 15:46 | |
*** dtantsur is now known as dtantsur|afk | 15:55 | |
*** ykarel is now known as ykarel|away | 16:14 | |
*** kopecmartin is now known as kopecmartin|off | 16:18 | |
*** zbr|out is now known as zbr | 16:19 | |
*** jaosorior has joined #oooq | 16:23 | |
zbr | sshnaidm|ruck: can we wf https://review.opendev.org/#/c/679756/ ? (curl) | 16:23 |
zbr | weshay: wf https://review.opendev.org/#/c/679825/ -- i cannot. | 16:25 |
sshnaidm|ruck | zbr, done | 16:32 |
*** sshnaidm|ruck is now known as sshnaidm|bbl | 16:32 | |
zbr | thanks | 16:35 |
rfolco | raukadah, sshnaidm|bbl weshay panda|rover zbr: staging promoter provisioning in zuul and molecule tests are ready, and green. Please review https://review.rdoproject.org/r/#/c/21873/ and https://review.rdoproject.org/r/#/c/21909/ and | 16:38 |
rfolco | actually only 2 patches... I combined provisioning and job definition/triggers in https://review.rdoproject.org/r/#/c/21873/ | 16:38 |
rfolco | https://review.rdoproject.org/r/#/c/21909/ is the molecule test | 16:39 |
rfolco | review begging time! | 16:39 |
zbr | rfolco: gave a +2, waiting for another one to give +W | 16:40 |
rfolco | zbr, you rock! | 16:41 |
panda|rover | rfolco: did you test a standalone at least once ? | 16:44 |
rfolco | panda|rover, good point. Will do that in a rdo cloud vm. | 16:45 |
rfolco | panda|rover, I did a few pses ago | 16:45 |
panda|rover | rfolco: we need to be sure we don't break the current server | 16:46 |
panda|rover | because the role will be applied immediately | 16:46 |
panda|rover | rfolco: also... I'm not sure what happened here | 16:46 |
panda|rover | rfolco: 21873 and 21909 conflict with each other | 16:47 |
panda|rover | rfolco: you merge one and you'll have to fix conflicts on the other | 16:47 |
rfolco | panda|rover, let me check | 16:48 |
panda|rover | rfolco: last note .... it will be a wonderful day when we deliver code and test on the same patch. It's still something that we really see separate. | 16:48 |
*** holser has quit IRC | 16:48 | |
-openstackstatus- NOTICE: Zuul job logs stored in OVH may fail. We have updated the base job to remove OVH from our storage location. If you have POST_FAILURES a recheck should fix them at this point. | 16:50 | |
*** jpena is now known as jpena|off | 16:53 | |
rfolco | panda|rover, conflict fixed, needed to rebase again the top one | 16:55 |
panda|rover | rfolco: why did you divide like this ? separating code and tests ? | 17:00 |
*** derekh has quit IRC | 17:00 | |
rfolco | panda|rover, yes, to make the zuul job provisioning part work with the job definition/triggers | 17:00 |
rfolco | so we have all zuul in the top patch | 17:00 |
rfolco | then molecule ones in the 2nd one | 17:00 |
rfolco | I thought it would be more clean as well | 17:03 |
rfolco | provisioning (zuul job green) --> molecule tests --> promoter setup | 17:03 |
*** udesale has quit IRC | 17:07 | |
*** rfolco is now known as rfolco|bank | 17:18 | |
ykarel|away | weshay, fyi cr repo is populated, may be u already known http://mirror.centos.org/centos/7/cr/x86_64/Packages/ | 17:34 |
weshay | ykarel|away thanks /me looks | 17:34 |
raukadah | ykarel|away: it has python 3 | 17:47 |
raukadah | http://mirror.centos.org/centos/7/cr/x86_64/Packages/python3-3.6.8-10.el7.x86_64.rpm | 17:47 |
ykarel|away | raukadah, yup | 17:48 |
*** tesseract has quit IRC | 18:01 | |
*** ykarel|away has quit IRC | 18:05 | |
*** holser has joined #oooq | 18:25 | |
zbr | weshay: just abandon https://review.opendev.org/#/c/612653/1 | 18:39 |
weshay | zbr want me to -2 instead? | 18:40 |
weshay | 10 months | 18:40 |
weshay | nvrmind | 18:40 |
zbr | better bandon, so it cleans the queue | 18:40 |
zbr | abandon makes it disapear, -2 does not. | 18:40 |
zbr | -2 when there is still hope to get it ;) | 18:41 |
weshay | ya.. it's gone.. didn't see how old it was | 18:41 |
zbr | i started to do some clean-up on reviews, that's why i may ping few people that can press the big A button. | 18:41 |
*** irclogbot_3 has quit IRC | 18:54 | |
*** irclogbot_1 has joined #oooq | 18:54 | |
*** irclogbot_1 has quit IRC | 19:02 | |
*** irclogbot_1 has joined #oooq | 19:02 | |
*** dtrainor_ has quit IRC | 19:05 | |
*** dtrainor has joined #oooq | 19:06 | |
*** holser has quit IRC | 19:22 | |
zbr | weshay: put a +W as it will likely pass in next hour, lots it on rebase. https://review.opendev.org/#/c/665994/ | 19:46 |
weshay | k | 19:46 |
*** holser has joined #oooq | 19:48 | |
weshay | zbr any other ones? | 19:48 |
zbr | weshay: https://review.opendev.org/#/c/617106/ -- either workflow it or abandon it, it does not matter. | 19:49 |
zbr | fyi, is likely bot generated. | 19:49 |
zbr | there is nothing special about "2.0", likely that we already need much newer than this anyway. | 19:49 |
weshay | wow.. going through some old ones | 19:49 |
zbr | weshay: i will show you why: i have query that reports ones that i already reviewed. | 19:50 |
zbr | mainly where effort was already invested. | 19:50 |
weshay | oh that's cool | 19:50 |
zbr | weshay: mainly is https://pypi.org/project/gri/ :D | 19:51 |
zbr | gt . | 19:52 |
weshay | nice | 19:55 |
weshay | zbr++ | 19:55 |
*** rfolco|bank is now known as rfolco | 19:55 | |
zbr | just made a new released that adds listing for incoming reviews (-i param). incoming = where you are reviewer | 20:00 |
zbr | the entire listing is sorted by a score which computed based on "how closer is to get merged". | 20:01 |
zbr | probably I should have called it hotness, but to my surprise with very little tuning, it produced the desired results. | 20:02 |
*** jtomasek has quit IRC | 20:08 | |
zbr | weshay: to abandon: https://review.opendev.org/#/c/669356/ | 20:23 |
zbr | weshay: workflow https://review.opendev.org/#/c/611779/ | 20:37 |
weshay | oh lord. | 20:38 |
* weshay looks | 20:38 | |
zbr | yeah, that is the same thing i said when I seen it. | 20:38 |
zbr | a proof that we do not do a lot of reviews on our own repos. | 20:38 |
weshay | zbr lolz | 20:38 |
weshay | can you do the same for tqe? | 20:39 |
zbr | i don't want to bother, the --- stuff is not really a blocker. | 20:40 |
*** holser has quit IRC | 20:41 | |
sshnaidm|bbl | weshay, https://review.rdoproject.org/r/#/c/21945/ | 21:01 |
*** sshnaidm|bbl is now known as sshnaidm|afk | 21:01 | |
zbr | https://review.opendev.org/#/c/679618/ - any of you. | 21:05 |
*** Vorrtex has quit IRC | 21:14 | |
*** aakarsh has quit IRC | 21:14 | |
*** holser has joined #oooq | 21:22 | |
*** dsneddon_ is now known as dsneddon | 21:52 | |
*** holser has quit IRC | 22:06 | |
*** tosky has quit IRC | 23:02 | |
*** holser has joined #oooq | 23:49 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!