| *** sdake_ has joined #kolla | 00:00 | |
| *** jasonsb has joined #kolla | 00:00 | |
| *** sdake has quit IRC | 00:03 | |
| sdake_ | jpeeler https://review.openstack.org/#/c/193369/ | 00:03 |
|---|---|---|
| *** sdake_ is now known as sdake | 00:06 | |
| *** dims has joined #kolla | 00:39 | |
| *** sdake_ has joined #kolla | 00:41 | |
| sdake_ | there is some anestaphine in the back of the medlock! | 00:41 |
| *** sdake has quit IRC | 00:44 | |
| *** vinkman has quit IRC | 00:45 | |
| *** dims has quit IRC | 00:47 | |
| *** dims has joined #kolla | 00:47 | |
| *** openstack has joined #kolla | 00:55 | |
| bmace | that is a whole lot of failed.. interesting yum failure message too, yeah, not happy | 01:00 |
| sdake_ | the fact that setup_gate.sh failed indicates its a fedora problem as wel las a centos problem | 01:01 |
| sdake_ | and I know for a fact they are in separate dcs | 01:01 |
| sdake_ | so must be an internet wide thing or infra thing | 01:01 |
| sdake_ | because the gate was failing for epel as well | 01:01 |
| sdake_ | in other cases | 01:01 |
| *** zhiwei has joined #kolla | 01:03 | |
| *** erkules_ has joined #kolla | 01:09 | |
| *** erkules has quit IRC | 01:11 | |
| *** ccrouch has quit IRC | 01:14 | |
| jpeeler | oh wow | 01:22 |
| jpeeler | yeah that's some serious failure | 01:22 |
| jpeeler | sdake_: so i'm not sure if we really need to remove the voting gate or not... | 01:24 |
| sdake_ | we are reerting it for now | 01:25 |
| sdake_ | and I asked mandre to add a retry | 01:25 |
| sdake_ | and then we will revert hte revert | 01:26 |
| sdake_ | atm, the gate is totally blocked | 01:26 |
| jpeeler | yeah | 01:26 |
| jpeeler | i don't think i've ever seen a reverted revert | 01:26 |
| jpeeler | i wonder if any other projects have so much trouble with the gate, though this doesn't seem to be our fault at all | 01:28 |
| sdake_ | status.fedoraproject.org says all is good | 01:29 |
| sdake_ | but I hae seen several irc netsplits today | 01:29 |
| sdake_ | so possibly the internet is busted | 01:29 |
| jpeeler | i'm pretty sure something in the gate is wonky | 01:29 |
| jpeeler | or that | 01:29 |
| sdake_ | but I want retries in the build script | 01:29 |
| jpeeler | guess we need to run a network precheck in our scripts to help us debug | 01:29 |
| jpeeler | i think i might tack that on that bug... | 01:30 |
| jpeeler | hrm, is that easier to do in python? maybe i'll just do it | 01:30 |
| sdake_ | is what easier to do i npython | 01:31 |
| sdake_ | its not tthat the network has completely failed, its that it is intermitently failed | 01:31 |
| jpeeler | the last one sure does look like complete failure | 01:31 |
| openstackgerrit | Martin André proposed stackforge/kolla: Attempt at fixing yum failing on epel repo https://review.openstack.org/193389 | 01:31 |
| mandre | let's see if that makes things better… | 01:32 |
| sdake_ | mandre its not just epel | 01:32 |
| sdake_ | its everything | 01:32 |
| sdake_ | i have seen failures in epel, setup_gate.sh at the fedora docker install stage, and in the mirrorlist retrieval for centos | 01:33 |
| sdake_ | anyway off for a break ttyal :) | 01:33 |
| jpeeler | i kind of want to see that run, and i don't think it will unless a + vote is given. so here goes | 01:36 |
| jpeeler | mandre: here's your run - https://jenkins07.openstack.org/job/check-kolla-functional-build-f21/24/console | 01:37 |
| mandre | thanks jpeeler | 01:38 |
| mandre | looking good so far | 01:39 |
| jpeeler | if it does work, i would feel a lot better if we go ahead and had retries too. think that'd be difficult to do? | 01:40 |
| mandre | thinking more about the proposed patch, I don't think it solves anything | 01:41 |
| mandre | I've never seen the base image failing, and the next line does a full packages upgrade anyway | 01:41 |
| jpeeler | it's not a fix i would have ever thought of | 01:41 |
| mandre | according to the error message, there is an option to skip unavailable repos, that may increase our success rate | 01:43 |
| mandre | yum-config-manager --save --setopt=<repoid>.skip_if_unavailable=true | 01:43 |
| jpeeler | i don't see how we can skip though | 01:43 |
| bmace | if they are ones we need we will just get different kinds of failures, and i can't imagine we have a ton of repos configured that we don't use. | 01:44 |
| jpeeler | correct | 01:44 |
| jpeeler | the one repo i have experienced this personally with is the google chrome repo. i just do another update and then it works | 01:44 |
| jpeeler | never really could figure out what the deal was | 01:45 |
| jpeeler | i don't know how easy it would be to check the build output for yum failure | 01:48 |
| jpeeler | but mandre, maybe modifying build_image to try again upon failure would suffice? | 01:48 |
| mandre | jpeeler, possibly, however that won't fix the root cause | 01:50 |
| mandre | you may get debug information exporting the URLGRABBER_DEBUG=1 variable before doing the yum install | 01:51 |
| jpeeler | i know... | 01:51 |
| mandre | sorry, I can't be of any more help right now, I have to run | 01:53 |
| jpeeler | thanks for looking at it! | 01:53 |
| mandre | it would be interesting to know exactly what it is that yum doesn't like | 01:54 |
| jpeeler | this might a silly question, but can Dockerfiles read env vars? | 01:55 |
| jpeeler | otherwise, modifying every Dockerfile doesn't sound fun | 01:55 |
| mandre | you can export ENV in the base image | 01:56 |
| jpeeler | ah right | 01:56 |
| mandre | the build is doing fine as far as I can tell, it's probably just a happy coincidence | 01:57 |
| mandre | oops, kollaglue/centos-rdo-neutron-base failed | 01:58 |
| mandre | so yeah, that didn't change anything | 01:58 |
| *** britthouser has joined #kolla | 01:58 | |
| mandre | bbl | 01:59 |
| *** britthou_ has quit IRC | 02:02 | |
| jpeeler | it's not fixed :/ | 02:07 |
| mandre | jpeeler, if you have some time to look at the issue I suggest you try setting the URLGRABBER_DEBUG variable to 1 and analyzing the logs | 02:12 |
| jpeeler | yeah i'll do that | 02:12 |
| *** daneyon_ has quit IRC | 02:13 | |
| mandre | setting "yum-config-manager --save --setopt=epel.skip_if_unavailable=true" may help unstuck the gate as well | 02:13 |
| jpeeler | can't really skip epel though | 02:15 |
| mandre | adding a retry for all yum operation is not a trivial change, it may require changing all dockerfiles (unless you manage to alias yum to a function that does the retry) | 02:15 |
| mandre | are all openstack-* packges coming from epel? | 02:16 |
| jpeeler | that's why i suggested modifying build_image in build-all-docker-images instead. but that can come later | 02:17 |
| mandre | in the latest build, it died trying to install openstack-neutron-ml2 | 02:17 |
| mandre | oh, i see | 02:17 |
| jpeeler | i don't know what the exact breakdown is for tje repos | 02:17 |
| mandre | yeah, adding the retry directly in the build script is also an option | 02:18 |
| jpeeler | not elegant, but could be effective | 02:18 |
| jpeeler | mandre: looking for the ultimate debug here, because i'm giving it one go and calling it a night. anything else other than URLGRABBER_DEBUG? | 02:19 |
| jpeeler | i was thinking of setting yum debug level to 10 | 02:20 |
| mandre | no idea, http://unix.stackexchange.com/questions/193480/cannot-use-epel-repo-get-nss-error-5990-pr-io-timeout-error only mentions URLGRABBER_DEBUG | 02:20 |
| *** tobe has joined #kolla | 02:24 | |
| openstackgerrit | Jeff Peeler proposed stackforge/kolla: Attempt to add maximum debugging to yum https://review.openstack.org/193397 | 02:29 |
| jpeeler | ugh, Dockerfile noob | 02:31 |
| openstackgerrit | Jeff Peeler proposed stackforge/kolla: Attempt to add maximum debugging to yum https://review.openstack.org/193397 | 02:32 |
| *** sdake_ has quit IRC | 02:34 | |
| *** prad has quit IRC | 02:35 | |
| openstackgerrit | Jeff Peeler proposed stackforge/kolla: Attempt to add maximum debugging to yum https://review.openstack.org/193397 | 02:39 |
| openstackgerrit | Jeff Peeler proposed stackforge/kolla: Attempt to add maximum debugging to yum https://review.openstack.org/193397 | 02:50 |
| *** sami__ has joined #kolla | 03:01 | |
| *** dims has quit IRC | 03:06 | |
| *** sami_ has quit IRC | 03:06 | |
| openstackgerrit | Jeff Peeler proposed stackforge/kolla: Attempt to add maximum debugging to yum https://review.openstack.org/193397 | 03:19 |
| *** sdake has joined #kolla | 03:33 | |
| sdake | evening | 03:33 |
| * jpeeler is going to bed | 03:35 | |
| jpeeler | sdake: see if this debug output means anything to you https://review.openstack.org/#/c/193397/ | 03:35 |
| sdake | jpeeler nothing jumps out at me | 03:37 |
| sdake | night jepeeler | 03:37 |
| *** zhiwei has quit IRC | 03:40 | |
| sdake | The Most Popular Problem occured when your Centos Operation System CLOCK not be correct | 04:02 |
| sdake | so https keys ( certificates ) goes down due to invalid time and date , you must set your clock correct then install EPEL repo. | 04:02 |
| sdake | but when this problem occured you can edit the /etc/yum.repos.d/epel.repo and comment the mirrorlist url (that have https) and uncomment baseurl. | 04:02 |
| sdake | by this method your downloads passthrough HTTP protocol and time miss mach cannot make yum install as crazy. | 04:02 |
| sdake | mandre ^^ | 04:02 |
| *** gfidente has quit IRC | 04:06 | |
| *** sdake_ has joined #kolla | 04:07 | |
| openstackgerrit | Steven Dake proposed stackforge/kolla: Test change https://review.openstack.org/193404 | 04:08 |
| *** sdake has quit IRC | 04:10 | |
| sdake_ | 2015-06-19 04:12:18.558 | Fri Jun 19 04:12:18 UTC 2015 | 04:15 |
| sdake_ | date looks good | 04:15 |
| sdake_ | that shoots down that theory | 04:15 |
| bmace | :( | 04:16 |
| openstackgerrit | Martin André proposed stackforge/kolla: Attempt at fixing yum failing on epel repo https://review.openstack.org/193389 | 04:16 |
| sdake_ | that leaves 443 firewalled as an option | 04:17 |
| sdake_ | mandre you only have "mirrorlis" not "mirrorlist" | 04:19 |
| mandre | bah, doesn't matter, it's commented out :) | 04:19 |
| sdake_ | ok | 04:20 |
| sdake_ | it could be time related, we just went through a dst change in the last couple of weeks | 04:20 |
| mandre | the weird thing is, it's not reliably failing | 04:21 |
| sdake_ | but some locals do differently adjust dst from the us | 04:21 |
| sdake_ | the gate runs on hp clouds and rax clouds | 04:21 |
| sdake_ | using baeurl seems like the wrong workaround to me :) | 04:22 |
| sdake_ | http://www.timeanddate.com/time/dst/2015.html | 04:23 |
| mandre | where is jpeeler finding the url of the jenkins builds? | 04:26 |
| sdake_ | status.openstack.org/zuul | 04:27 |
| sdake_ | use browser find to search for changeset | 04:27 |
| sdake_ | click big box (not the changset link) | 04:27 |
| sdake_ | it will hwo you the gates | 04:27 |
| mandre | cool, took me a wile to realize i can click the blue box | 04:29 |
| sdake_ | https://jenkins06.openstack.org/job/check-kolla-functional-build-f21/28/console | 04:29 |
| sdake_ | hwo/show | 04:29 |
| sdake_ | mandre i dont like that hcange it doesn't solve the root cause tho | 04:30 |
| sdake_ | because wedon't know the root cause | 04:30 |
| sdake_ | and loads everything up on one url rather then the mirror list | 04:31 |
| mandre | agreed, we should not disable https but fix it | 04:31 |
| mandre | i'm simply curious if that's a valid workaround | 04:31 |
| mandre | we could do a s/https/http if mirrorlist responds to http | 04:33 |
| sdake_ | https://review.openstack.org/#/c/193369/ | 04:33 |
| sdake_ | mandre read and add your comment in there plz | 04:34 |
| sdake_ | i read something today on #openstack-infra that there was a change in networking | 04:37 |
| sdake_ | or something like that | 04:38 |
| sdake_ | id be satisified with a mirrorlist ofhttp rather then https | 04:40 |
| sdake_ | we dont need https in this case | 04:40 |
| mandre | i may have screwed the patch, it's still trying to find a mirror | 04:41 |
| sdake_ | sedblack magic to me ;) | 04:41 |
| sdake_ | the intenet was busted today mandre | 04:42 |
| sdake_ | if that helps :) | 04:42 |
| openstackgerrit | Martin André proposed stackforge/kolla: Attempt at fixing yum failing on epel repo https://review.openstack.org/193389 | 04:43 |
| sdake_ | https://downdetector.com/archive/ | 04:43 |
| sdake_ | look at the 18th | 04:43 |
| mandre | the internet is seriously broken | 04:44 |
| sdake_ | probably some country rerouting traffic through their switches to monitor my irc chats | 04:46 |
| mandre | the cleaning lady pulled the cable again | 04:51 |
| juggler | lol | 04:52 |
| sdake_ | tcp/ip designed to withstand nuclear attack | 04:53 |
| sdake_ | doubt a cleaning lady pulling a plug would dod it :) | 04:53 |
| sdake_ | although the intenet is more fraglie then itused to be | 04:53 |
| juggler | heh yep | 04:54 |
| sdake_ | mandre where is yourzuul job - idon't see iti nthe list | 04:56 |
| sdake_ | nm i see it | 04:56 |
| mandre | https://jenkins07.openstack.org/job/check-kolla-functional-build-f21/28/console | 04:56 |
| sdake_ | looks like http is a valid mirrorlist server endpoint | 04:57 |
| sdake_ | it could just be that centos epel mirrorlist servering via https is broken some of the time | 04:58 |
| sdake_ | and since we access it 40 times or more per gate | 04:59 |
| sdake_ | our chances are high that one of them will fail | 04:59 |
| mandre | even the big tohoku earthquake was not able to break internet in japan, http://conferences.sigcomm.org/co-next/2011/workshops/SpecialWorkshop/papers/1569500743.pdf | 05:00 |
| sdake_ | mandre http://git.openstack.org/cgit/openstack-infra/devstack-gate/tree/functions.sh#n118 | 05:04 |
| sdake_ | note we are not running that code in our gate | 05:05 |
| sdake_ | because we don't do ad evstack-gate setup operation | 05:05 |
| sdake_ | perhaps we need something like it | 05:05 |
| *** Haomeng|2 has joined #kolla | 05:10 | |
| *** Haomeng has quit IRC | 05:13 | |
| mandre | my latest patch to use http mirror didn't work, am I modifying the correct file at least? | 05:15 |
| sdake_ | it looks like its still running mandre? | 05:16 |
| mandre | looking directly at jenkins console | 05:17 |
| openstackgerrit | Martin André proposed stackforge/kolla: Attempt at fixing yum failing on epel repo https://review.openstack.org/193389 | 05:17 |
| sdake_ | i was looking at it | 05:18 |
| sdake_ | 2015-06-19 05:17:46.129 | Finished: ABORTED | 05:19 |
| sdake_ | it didnt' finish becuase you submitteed a new patch mandre | 05:19 |
| sdake_ | did you see it fial a build it should have not fialed? | 05:19 |
| mandre | yeah, now it's gone because I submitted a new patch, it had plenty of "Cannot retrieve metalink for repository" errors | 05:19 |
| mandre | for the keystone image, etc. | 05:20 |
| sdake_ | mandre those aren't fatal, it just will try a new mirror in that case | 05:20 |
| sdake_ | when its fatal it prints out an assay warning about how it gave up | 05:20 |
| sdake_ | it is possible the mirrors are updating atm | 05:21 |
| sdake_ | if only there were this tool to search the internet to find out what centos is doing with their mirrors ;) | 05:21 |
| sdake_ | god my wife yelling at me, I have had it up to here with it | 05:22 |
| sdake_ | sorry for tkaing it out on you mandre | 05:22 |
| sdake_ | http://mirror-status.centos.org/ | 05:23 |
| sdake_ | changes in the last 1-7 hours | 05:23 |
| sdake_ | perhps that is the cuase | 05:23 |
| mandre | hmmm jenkins didn't like the last patch | 05:24 |
| sdake_ | the fact that the mirrors are syncing | 05:24 |
| sdake_ | could be cause for the repomod fialures | 05:25 |
| sdake_ | repomod not found is different from the 500 mirrorlist error | 05:25 |
| mandre | so you say this error is not fatal? http://logs.openstack.org/89/193389/4/check/check-kolla-functional-build-f21/f967e29/console.html#_2015-06-19_05_20_39_553 | 05:25 |
| mandre | i think that's what I has in the last build | 05:26 |
| sdake_ | another possibility is coprs | 05:27 |
| sdake_ | does coprs install https or http repos | 05:27 |
| sdake_ | see the wall of text, that is different from the repomod data not being correct or found | 05:28 |
| *** inc0 has joined #kolla | 05:28 | |
| sdake_ | 2015-06-19 05:20:39.550 | INFO:test_images:One of the configured repositories failed (Unknown), | 05:28 |
| sdake_ | I dont even know what that means | 05:28 |
| inc0 | good morning | 05:28 |
| sdake_ | inc0 yo | 05:28 |
| inc0 | wassup? | 05:29 |
| mandre | in /etc/yum.repos.d/sdake-pam.noaudit-epel-7.repo, we have baseurl=https | 05:29 |
| sdake_ | the world is imploding | 05:29 |
| inc0 | last time I heard it was exploding, so I guess that's an improvement? | 05:29 |
| sdake_ | maybe coprs is busted | 05:29 |
| sdake_ | no implosion = singularity | 05:29 |
| sdake_ | did I ever mention i hate computers ;) | 05:30 |
| inc0 | we kolla gained self-conciousness and now our containers are more intelligent than we are? | 05:30 |
| sdake_ | inc0 i mean the black hole singularity not the technology singualirty :) | 05:31 |
| sdake_ | if the containes were more self-ware they would cooperate or face deletion | 05:31 |
| sdake_ | ;) | 05:31 |
| inc0 | and fact that they don't cooperate prooves that they aren't that intelligent after all? | 05:32 |
| sdake_ | precisely | 05:32 |
| sdake_ | ok well I've had enough fun for today | 05:33 |
| inc0 | I'll post a blueprint "create a container head on the spike to promote cooperation" | 05:33 |
| sdake_ | if mandre cang et the gate unhammed | 05:33 |
| sdake_ | unjammed | 05:33 |
| sdake_ | yay for mandre :) | 05:33 |
| sdake_ | night all | 05:33 |
| *** sdake_ has quit IRC | 05:33 | |
| inc0 | mandre, did you fix single interface problem? | 05:33 |
| mandre | sdake_ I see you're not losing hope :) | 05:34 |
| mandre | inc0: nah, the gate is currently failing and we're trying to figure out what's going on | 05:35 |
| mandre | yum is unable to find a mirror, and that fails the build | 05:36 |
| mandre | it's happening more frequently now | 05:36 |
| inc0 | any chances we could create our own mirrors? | 05:36 |
| inc0 | somewhere inside infra? | 05:37 |
| mandre | we first thought that was an issue with https, where it's unable to validate the certificate, but it doesn't appear to be the case | 05:37 |
| mandre | we would have to mirror all distros, not sure this is a good idea | 05:38 |
| inc0 | where can I see traceback? | 05:38 |
| mandre | available mirrors are supposedly reliable | 05:38 |
| mandre | here is one: http://logs.openstack.org/89/193389/4/check/check-kolla-functional-build-f21/f967e29/console.html#_2015-06-19_05_20_39_550 | 05:39 |
| inc0 | hmm..it have installed package before, so there are repos available | 05:40 |
| inc0 | question is which one | 05:41 |
| *** nihilifer has joined #kolla | 05:44 | |
| inc0 | mandre, http://logs.openstack.org/89/193389/4/check/check-kolla-functional-build-f21/f967e29/console.html#_2015-06-19_05_20_02_533 this line changes protocol | 05:47 |
| inc0 | maybe one of mirrors doesn't have http at all? | 05:47 |
| mandre | the error message is not the most obvious | 05:50 |
| inc0 | far from it | 05:51 |
| inc0 | form obvious I mean | 05:51 |
| openstackgerrit | Martin André proposed stackforge/kolla: Attempt at fixing yum failing on epel repo https://review.openstack.org/193389 | 05:52 |
| mandre | this should add some debugging | 05:52 |
| inc0 | so now we wait | 05:53 |
| mandre | https://jenkins06.openstack.org/job/check-kolla-functional-build-f21/29/console | 05:53 |
| nihilifer | o/ | 05:54 |
| inc0 | cześć Michał | 05:54 |
| nihilifer | oh, you're already talking about failing gate? | 05:54 |
| nihilifer | siemanko :) | 05:54 |
| inc0 | yup, for a good morning start! | 05:54 |
| inc0 | that's a traceback out there! run for your lives! | 05:55 |
| mandre | inc0, if you don't mind I'll leave it to you, at least you have somewhere to start looking :) | 05:58 |
| inc0 | well, I can't promise anything | 05:58 |
| mandre | i really need to work on what they're paying me for now, I'll be back later | 06:00 |
| inc0 | sure, I'm in comfortable situation because *this* is my work;) | 06:00 |
| mandre | and don't forget sdake will be mad at you if you don't fix the gate :) | 06:00 |
| inc0 | I'll let you know what I'll find | 06:01 |
| mandre | putting some pressure | 06:01 |
| *** absubram has joined #kolla | 06:01 | |
| *** absubram has quit IRC | 06:01 | |
| mandre | we've pushed a patch to make the gate non voting https://review.openstack.org/#/c/193369/ | 06:02 |
| openstackgerrit | Michal Jastrzebski (inc0) proposed stackforge/kolla: Attempt at fixing yum failing on epel repo https://review.openstack.org/193389 | 06:06 |
| bmace | is that actually in now mandre? | 06:18 |
| mandre | bmace: the gate is still voting | 06:20 |
| *** Slower has joined #kolla | 06:24 | |
| bmace | well, sleep time, maybe the internet will be happier when i wake up ;) | 06:26 |
| *** gfidente has joined #kolla | 06:27 | |
| mandre | 'night | 06:27 |
| *** juggler has quit IRC | 06:43 | |
| *** dasm|afk is now known as dasm | 07:11 | |
| dasm | gmorning guys | 07:14 |
| openstackgerrit | Michal Jastrzebski (inc0) proposed stackforge/kolla: Keepalived container https://review.openstack.org/187981 | 07:16 |
| inc0 | damn, I've lost 2*+2:( | 07:17 |
| Haomeng|2 | one question, who can help me - encountered "ERRO[0052] Couldn't run auplink before unmount: exec: "auplink": executable file not found in $PATH" error for command "kolla start", any comments? | 07:22 |
| Haomeng|2 | details - http://paste.openstack.org/show/304149/ | 07:22 |
| inc0 | seems like you're missing a package | 07:25 |
| inc0 | try to install auplink (I'm not sure if that's a case, but well..) | 07:25 |
| SamYaple | hello | 07:27 |
| Haomeng|2 | inc0: ok, thank you | 07:28 |
| inc0 | good morning SamYaple | 07:28 |
| openstackgerrit | Michal Jastrzebski (inc0) proposed stackforge/kolla: Attempt at fixing yum failing on epel repo https://review.openstack.org/193389 | 07:32 |
| *** akwasnie_ has joined #kolla | 07:36 | |
| *** akwasnie_ is now known as akwasnie | 07:37 | |
| Haomeng|2 | inc0: I run "apt-get install aufs-tools" to install auplink command, and no previous auplink not found issue now, thank you | 07:37 |
| Haomeng|2 | inc0: but new issue encountered - http://paste.openstack.org/show/304150/, any ideas? | 07:37 |
| *** inc0 has quit IRC | 07:49 | |
| nihilifer | Haomeng|2: which version of docker-compose do you have? | 07:49 |
| nihilifer | maybe just try to install the newest docker-compose in virtualenv | 07:58 |
| Haomeng|2 | nihilifer: ok, let me try again, thank you | 08:01 |
| Haomeng|2 | nihilifer: my version is - docker-compose version: 1.4.0dev, not sure if this latest one | 08:06 |
| *** inc0 has joined #kolla | 08:20 | |
| *** inc0_ has joined #kolla | 08:22 | |
| *** inc0 has quit IRC | 08:25 | |
| *** inc0_ is now known as inc0 | 08:25 | |
| inc0 | SamYaple, why did you -2 on this? https://review.openstack.org/#/c/187225/12/docker/centos/binary/galera/start.sh | 08:29 |
| SamYaple | i explained in the comments inc0 | 08:30 |
| SamYaple | i only -2 it because sdake had +2 it and its a major issue | 08:30 |
| inc0 | uhh...-2 basically means patchset goes to trash | 08:30 |
| inc0 | anyway, how would you disable an ENV? | 08:32 |
| inc0 | after initial use? | 08:32 |
| SamYaple | inc0: i means do no merge, the patchset still exists | 08:32 |
| SamYaple | inc0: unset the variable, change it to false something like that | 08:33 |
| inc0 | well, technically yeah, but it's usually used as "this is bad idea, we don't want this stuff in at all" | 08:33 |
| inc0 | but you can't do this from inside container | 08:33 |
| inc0 | and we don't have anywhere to unset it | 08:33 |
| SamYaple | you can unset a variable from inside the container, but that wont stop recreates | 08:33 |
| inc0 | this has to be unset on host | 08:33 |
| SamYaple | this is something that needs to be figured out | 08:34 |
| SamYaple | otherwise were going to be trashing the db | 08:34 |
| inc0 | well, this is issue with pretty much everything | 08:34 |
| inc0 | I've submitted several bugs for that | 08:34 |
| inc0 | https://bugs.launchpad.net/kolla/+bug/1466474 | 08:34 |
| openstack | Launchpad bug 1466474 in kolla "Nova inits database each time container is run" [Undecided,New] | 08:34 |
| inc0 | in most of patches we don't even check anything | 08:35 |
| SamYaple | thats a different issue | 08:35 |
| inc0 | similar problem, different severity | 08:35 |
| SamYaple | setting --wsrep-new-cluster can trash the whole database | 08:35 |
| inc0 | but we need to figure this out as policy for every container | 08:35 |
| SamYaple | no | 08:35 |
| SamYaple | this is a different issue | 08:36 |
| SamYaple | db_sync is safe to rerun, this is not | 08:36 |
| inc0 | not always | 08:36 |
| inc0 | anyway, I too think we need to figure this out | 08:36 |
| inc0 | there will be other problems like that | 08:36 |
| SamYaple | can you point to one? | 08:37 |
| SamYaple | if you mean future problems, those arent in the codebase | 08:37 |
| SamYaple | im not merging this into the code base with a major issue like that | 08:37 |
| inc0 | well, imho sync_db isn't safe to run | 08:37 |
| inc0 | nto always anyway | 08:37 |
| SamYaple | why | 08:37 |
| inc0 | it not AS severe as this | 08:37 |
| inc0 | but alters can lock tables | 08:37 |
| inc0 | locket tables can mean downtime | 08:38 |
| inc0 | accidental downtime is bad | 08:38 |
| SamYaple | fair enough | 08:38 |
| inc0 | if you have huge databases | 08:38 |
| SamYaple | i dont disagree with you on that in anycase | 08:38 |
| inc0 | which in case of for example ceilometer you can | 08:38 |
| SamYaple | but that is not a corruption issue, thats a downtime issue | 08:38 |
| SamYaple | this is a lose all your data issue | 08:38 |
| inc0 | I've ran ceilometer alter once...it hold out whole system for 24h+ | 08:38 |
| inc0 | I know, I'm saying we're facing this problem | 08:39 |
| inc0 | in few cases, not just this one | 08:39 |
| inc0 | we need to figure it out | 08:39 |
| SamYaple | agree, but th db_sync is NOT a data corruption problem | 08:39 |
| SamYaple | galera container is | 08:39 |
| inc0 | still, we need to figure it out | 08:39 |
| inc0 | imho in several places | 08:39 |
| inc0 | so lets just move to figuring it out;) | 08:39 |
| inc0 | env can be set up | 08:40 |
| inc0 | hmm...I have an idea | 08:40 |
| SamYaple | cool, but its going to hold up the container until we do since this is the only issue that can lose data | 08:40 |
| inc0 | no, I don't. I thought about adding someting to .buildconf, but we need decision on deployment moment | 08:40 |
| inc0 | ENTRYPOINT? | 08:40 |
| SamYaple | cmd/entrypoint doesnt solve this | 08:41 |
| inc0 | how can we push parameter like "yes, I am sure I want to init this time" | 08:42 |
| SamYaple | with docker-compose? I do not know. it needs outside orchastration | 08:42 |
| SamYaple | basically init container | 08:43 |
| SamYaple | destroy init | 08:43 |
| SamYaple | (init container doesnt have to be a new image) | 08:43 |
| inc0 | and outside orchiestration can use this env we've set | 08:43 |
| inc0 | docker compose will require manual stuff anyway, so again, env we've set works | 08:44 |
| SamYaple | ok. so ill go through the scenario. | 08:44 |
| inc0 | if someone doesn't turn it off...well...all we can do is to add BOLD UPPERCASE RED warning to docs... | 08:45 |
| SamYaple | start contaienr 1 with --wsrep-new-cluster | 08:45 |
| SamYaple | start contaienr 2 adn 3 without it | 08:45 |
| SamYaple | restart container 1 | 08:45 |
| SamYaple | it wont join the cluster | 08:45 |
| inc0 | why? | 08:45 |
| SamYaple | --wsrep-new-cluster | 08:46 |
| inc0 | (if you'd remove env before_ | 08:46 |
| SamYaple | how does that remove it from docker-compose? | 08:46 |
| inc0 | changing openstack.env file won't update it on restart? | 08:46 |
| inc0 | it won't reread this? | 08:47 |
| SamYaple | no no | 08:47 |
| SamYaple | we have restart-always set to on | 08:47 |
| SamYaple | if the contaienr restarts it will run --wsrep-new-cluster again | 08:47 |
| inc0 | it can do this automatically | 08:47 |
| inc0 | yeah, I get it now | 08:47 |
| inc0 | hmmm... | 08:47 |
| inc0 | maybe additional check there? | 08:48 |
| SamYaple | basically the container has to check to see if the cluster exists which is a whole lotta stuff that should be outside orchastration in my opinion. but we are where we are | 08:49 |
| SamYaple | so that logic will need to be in the container | 08:49 |
| inc0 | how about lock file added on first init? | 08:50 |
| inc0 | we'll mount some external folder | 08:51 |
| inc0 | and add cluster_created file | 08:51 |
| inc0 | and if you explicitly want to recreate cluster, you need to set env up AND remove cluster_created file from host | 08:51 |
| SamYaple | how will you deal with a situation where all database nodes have stopped? | 08:51 |
| inc0 | we'll still have env only on master node | 08:51 |
| inc0 | and this node will also have lockfile | 08:52 |
| mstachow | morning | 08:52 |
| * mstachow ziew | 08:52 | |
| SamYaple | lets put it this way, im ok with however yo uwant to solve it for config-internal | 08:52 |
| SamYaple | morning mstachow | 08:52 |
| inc0 | let's solve this for bindconf later? we don't have anything besides config-internal | 08:53 |
| inc0 | right now | 08:53 |
| inc0 | we can file an critical bug at once | 08:53 |
| inc0 | anyway, I have meeting now, think about that plz, I'll ping you when I'm back | 08:54 |
| SamYaple | its solved for config-external | 08:54 |
| SamYaple | you just need to solve it fr config-internal before merge | 08:54 |
| *** inc0 has quit IRC | 08:58 | |
| *** inc0 has joined #kolla | 08:59 | |
| *** athomas has joined #kolla | 09:09 | |
| *** juggler has joined #kolla | 09:14 | |
| pbourke | harmw: you there? | 09:26 |
| *** jmccarthy has quit IRC | 09:28 | |
| *** jmccarthy has joined #kolla | 09:28 | |
| *** tobe has quit IRC | 09:28 | |
| openstackgerrit | Paul Bourke proposed stackforge/kolla: Fix 'unsupported locale setting' errors in init-runonce https://review.openstack.org/188432 | 09:55 |
| openstackgerrit | Paul Bourke proposed stackforge/kolla: Minor doc fixups in dev-quickstart.md https://review.openstack.org/190543 | 09:56 |
| openstackgerrit | Sam Yaple proposed stackforge/kolla: WIP Change fat neutron container into thin contianers https://review.openstack.org/193480 | 10:02 |
| *** inc0 has quit IRC | 10:03 | |
| *** inc0 has joined #kolla | 10:04 | |
| SamYaple | Hey guys, could use a review of https://review.openstack.org/#/c/193480/1/docker/centos/binary/neutron/neutron-base/ip_wrapper.py | 10:04 |
| *** dims has joined #kolla | 10:09 | |
| *** inc0 has quit IRC | 10:09 | |
| harmw | pbourke: yo | 10:10 |
| harmw | I'm reading your reply, you are right ofcourse on most of it :) | 10:11 |
| harmw | but | 10:11 |
| harmw | the multiple RUN statements, comments and mkdir's you've found in other Dockerfiles are on the 'to be fixed'-list :) | 10:12 |
| harmw | in fact, there is a review from fangfenghua that addresses those things | 10:13 |
| harmw | aligntment with docker best practices | 10:13 |
| harmw | re: hardcoded FROM, I think I'm wrong here since you target the -base container | 10:13 |
| harmw | pbourke: I hope that makes clear why I choose to -1 :) | 10:14 |
| *** Haomeng has joined #kolla | 10:38 | |
| *** vbel has quit IRC | 10:40 | |
| *** vbel has joined #kolla | 10:41 | |
| *** vbel has quit IRC | 10:41 | |
| *** Haomeng|2 has quit IRC | 10:41 | |
| pbourke | thanks harmw | 10:43 |
| *** inc0 has joined #kolla | 10:47 | |
| harmw | SamYaple: I'll gladly look at that tonight | 10:47 |
| harmw | pbourke: no problem, if I do weird reviews ever again that don't make sense - just let me know :p | 10:48 |
| *** inc0_ has joined #kolla | 10:48 | |
| pbourke | it was just funny as for a moment I thought we were looking at two different code bases ;) | 10:49 |
| harmw | hehe | 10:49 |
| *** inc0 has quit IRC | 10:49 | |
| *** inc0__ has joined #kolla | 10:50 | |
| *** jmccarthy has quit IRC | 10:51 | |
| pbourke | what happened with the gates last night? | 10:51 |
| inc0__ | pbourke, basically yum is raising errors | 10:52 |
| nihilifer | yum is failing with epel mirrors, afaik | 10:53 |
| inc0__ | https://review.openstack.org/#/c/193389/ look here | 10:53 |
| *** inc0_ has quit IRC | 10:54 | |
| *** jmccarthy has joined #kolla | 10:55 | |
| *** dims has quit IRC | 10:57 | |
| openstackgerrit | Paul Bourke proposed stackforge/kolla: Symlink all non Dockerfile resources in images https://review.openstack.org/190117 | 10:58 |
| openstackgerrit | Paul Bourke proposed stackforge/kolla: Add base image for oraclelinux https://review.openstack.org/191013 | 10:58 |
| *** dims has joined #kolla | 11:06 | |
| *** rhallisey_ has joined #kolla | 11:22 | |
| rhallisey_ | what's up with the gate failing everywhere... | 11:23 |
| SamYaple | thats what I want to know rhallisey_ | 11:26 |
| SamYaple | i dont know anything about the gate | 11:26 |
| rhallisey_ | looks like a few images are failing to build | 11:27 |
| rhallisey_ | ERROR: failed to build kollaglue/centos-rdo-ceilometer-notification:f69aa85 | 11:29 |
| rhallisey_ | RUN yum install -y openstack-ceilometer-notification && yum clean all | 11:29 |
| rhallisey_ | this fails | 11:29 |
| *** rhallisey_ is now known as rhallisey | 11:30 | |
| SamYaple | ugh ceilometer! | 11:31 |
| pbourke | SamYaple: here's a bug for it https://bugs.launchpad.net/kolla/+bug/1466677 | 11:39 |
| openstack | Launchpad bug 1466677 in kolla "build script errors out in yum" [Critical,In progress] - Assigned to Martin André (mandre) | 11:39 |
| pbourke | sdake has also submitted patch to make the gate non voting for now | 11:40 |
| SamYaple | if this were source builds.... ;) | 11:40 |
| SamYaple | did you know the steam summer sale is going on? | 11:40 |
| SamYaple | i did not | 11:40 |
| pbourke | yup | 11:40 |
| rhallisey | what's in the steam summer sale | 11:40 |
| * rhallisey looks | 11:41 | |
| pbourke | let us know if anything good | 11:41 |
| pbourke | :) | 11:41 |
| rhallisey | evolve is 1/2 off | 11:41 |
| SamYaple | i dont know but if you like the original metroid someone just sent me this list http://www.reddit.com/r/gaming/comments/3a47iy/if_only_nintendo_had_announced_a_metroid_like/cs9ajap | 11:41 |
| SamYaple | i know 3 of the games on there and they are awesome, so i bought the rest | 11:41 |
| rhallisey | arma 3 is 20$ | 11:42 |
| rhallisey | that's sick | 11:42 |
| SamYaple | i havent played games in so long | 11:43 |
| rhallisey | I don't have a good enough computer to play much on pc :( | 11:44 |
| rhallisey | but I wish | 11:44 |
| SamYaple | ive got a semi decent desktop for the games. its 2-3 years old at this point | 11:44 |
| SamYaple | havent played much in a year | 11:44 |
| *** jmccarthy has quit IRC | 11:47 | |
| *** jmccarthy has joined #kolla | 11:47 | |
| * jmccarthy hasn't done much proper gaming in ages :| | 11:50 | |
| pbourke | have we tried something like this I wonder https://serverfault.com/questions/637549/epel-repo-for-centos-6-causing-error | 11:50 |
| pbourke | seems we did | 11:52 |
| pbourke | :( | 11:52 |
| pbourke | I think they've pushed a new centos image upstream | 11:55 |
| openstackgerrit | Paul Bourke proposed stackforge/kolla: DO NOT MERGE: Attempt at fixing yum failing on epel repo https://review.openstack.org/193389 | 11:57 |
| pbourke | do you have to +1 to trigger a check? | 11:59 |
| *** erkules_ is now known as erkules | 11:59 | |
| *** erkules has joined #kolla | 11:59 | |
| *** dasm is now known as dasm|brb | 12:04 | |
| openstackgerrit | Paul Bourke proposed stackforge/kolla: DO NOT MERGE: Attempt at fixing yum failing on epel repo https://review.openstack.org/193389 | 12:06 |
| *** dasm|brb is now known as dasm | 12:08 | |
| *** athomas has quit IRC | 12:14 | |
| *** athomas has joined #kolla | 12:25 | |
| openstackgerrit | Michal Rostecki proposed stackforge/kolla: DO NOT MERGE: Attempt at fixing yum failing on epel repo https://review.openstack.org/193389 | 12:37 |
| rhallisey | does that work ^ | 12:39 |
| rhallisey | ? | 12:39 |
| nihilifer | we'll see. just tried to get rdo repo package without ssl, because pbrouke's patch was failing on yum -y install https://rdoproject.org/blabblah.rpm | 12:41 |
| *** dims has quit IRC | 12:46 | |
| nihilifer | aww, failed again on systemd-container | 12:46 |
| *** dims has joined #kolla | 12:46 | |
| *** dwalsh has joined #kolla | 12:48 | |
| openstackgerrit | Michal Rostecki proposed stackforge/kolla: DO NOT MERGE: Attempt at fixing yum failing on epel repo https://review.openstack.org/193389 | 12:50 |
| *** sdake has joined #kolla | 13:05 | |
| sdake | morning | 13:05 |
| sdake | gate fixed? | 13:05 |
| *** sdake_ has joined #kolla | 13:07 | |
| pbourke | not yet | 13:07 |
| pbourke | my theory is something has changed in the centos image as there was a new build pushed yesterday | 13:07 |
| sdake_ | diddn't know that! | 13:08 |
| pbourke | the git repo for their(centos) dockerfiles is a little strange though, i cant seem to get a proper history for it | 13:08 |
| sdake_ | well we ccan check the yum repo file contents | 13:09 |
| sdake_ | they possibly got damaged | 13:09 |
| sdake_ | or did you do that already | 13:09 |
| nihilifer | inc0__ tried with yum.repos.d at the morning | 13:10 |
| *** sdake has quit IRC | 13:10 | |
| pbourke | anyone know what the story is with systemd/fakesystemd? | 13:11 |
| pbourke | there's something about it here https://registry.hub.docker.com/_/centos/ | 13:11 |
| nihilifer | pbourke: fakesystemd, afaik, is the package installed in centos image by default to prevent installing real systemd packages | 13:11 |
| pbourke | nihilifer: yeah just reading it now. but we're trying to install systemd | 13:12 |
| pbourke | ? | 13:12 |
| nihilifer | just tried to do | 13:12 |
| nihilifer | rpm -e --no-deps fakesystemd | 13:12 |
| nihilifer | locally | 13:12 |
| nihilifer | but got another error, with epel... | 13:12 |
| pbourke | by the looks of it if you want to install systemd you need to all the steps outlined on the above page | 13:13 |
| pbourke | which we're not doing | 13:13 |
| pbourke | do we need systemd once we have thin neutron containers? | 13:15 |
| sdake_ | we dont need real systemd afaik | 13:15 |
| pbourke | ill try a build with that line removed, one sec | 13:15 |
| sdake_ | the issue is libvirt installs systemd as a part of its dep chain | 13:15 |
| sdake_ | so i had to uninstall it and reinstall it ofr some wonky reason | 13:16 |
| sdake_ | because systemd-container | 13:16 |
| pbourke | :/ | 13:16 |
| sdake_ | whatever that is | 13:16 |
| openstackgerrit | Paul Bourke proposed stackforge/kolla: DO NOT MERGE: Attempt at fixing yum failing on epel repo https://review.openstack.org/193389 | 13:17 |
| *** diga has quit IRC | 13:17 | |
| sdake_ | the issue has more to do with the yum repos | 13:19 |
| sdake_ | and less to do with the actual pacakgess installed I think | 13:19 |
| sdake_ | brb connecting to dev boxes | 13:19 |
| *** sdake has joined #kolla | 13:20 | |
| sdake | so the centos image was udpated 19 hours ago | 13:21 |
| sdake | that would be the prime candidate for the breakage | 13:21 |
| pbourke | yeah | 13:21 |
| pbourke | the question is what they did | 13:22 |
| pbourke | https://github.com/CentOS/sig-cloud-instance-images/blob/0a6a7fa816e834b29222fce2df0b858ab1b97a87/docker/Dockerfile | 13:22 |
| nihilifer | I just did now locally | 13:23 |
| nihilifer | rpm -e --no-deps fakesystemd | 13:23 |
| nihilifer | + sed stuff that inc0__ submitted at the morning | 13:23 |
| nihilifer | seems to be working | 13:23 |
| *** sdake_ has quit IRC | 13:24 | |
| sdake | does it work all the time :) | 13:24 |
| nihilifer | but disabling https and using hardcoded baserepo instead of mirrors isn't the way to go :P | 13:24 |
| pbourke | so pinning to the older version of centos 7.1 rather than latest has the same issue: https://jenkins04.openstack.org/job/check-kolla-functional-build-f21/37/console | 13:24 |
| sdake | disabling https is ok | 13:24 |
| sdake | using hardcode dbaserepo is wrong | 13:24 |
| sdake | do we know precisely waht the problem is | 13:25 |
| sdake | it is that mirrorlist fails? | 13:25 |
| pbourke | the problem is "Cannot retrieve metalink for repository: epel/x86_64", but I dont think anyone has pinned down what that means yet | 13:27 |
| pbourke | clock was suggested but that seems correct | 13:27 |
| sdake | are the yum repo files correct in the image | 13:28 |
| openstackgerrit | Michal Rostecki proposed stackforge/kolla: DO NOT MERGE: Attempt at fixing yum failing on epel repo https://review.openstack.org/193389 | 13:29 |
| pbourke | sdake: I think its one of the ones we're adding as part of the build | 13:30 |
| nihilifer | sdake: the yum repo file from epel is not in raw image - it's provided by epel-repo package | 13:30 |
| pbourke | The command [/bin/sh -c yum -y install pam+noaudit && yum clean all] returned a non-zero code: 1 -- we're adding a repo to do with pam? | 13:31 |
| sdake | yes i mean correct in the base image | 13:31 |
| sdake | yeswe are | 13:31 |
| *** akwasnie has quit IRC | 13:31 | |
| sdake | https://copr.fedoraproject.org/coprs/sdake/pam.noaudit/ | 13:32 |
| pbourke | found it | 13:33 |
| pbourke | try this locally: | 13:33 |
| pbourke | docker run -it centos /bin/bash | 13:33 |
| pbourke | curl https://copr.fedoraproject.org/coprs/sdake/pam.noaudit/repo/epel-7/sdake-pam.noaudit-epel-7.repo -o /etc/yum.repos.d/sdake-pam.noaudit-epel-7.repo | 13:34 |
| pbourke | yum -y install pam+noaudit | 13:34 |
| pbourke | goes splat | 13:34 |
| sdake | the dockerfile removes the existing pam on the system | 13:34 |
| sdake | so that is not the same test | 13:34 |
| *** jtriley has joined #kolla | 13:35 | |
| sdake | centos-rdo-base is not failing to buidl | 13:36 |
| sdake | pbourke Step 12 : RUN rpm -e --nodeps pam | 13:37 |
| pbourke | ah I see now yeah | 13:37 |
| pbourke | disregard | 13:37 |
| sdake | however step 13 just faied on centos-rdo-base for me | 13:37 |
| sdake | Cannot retrieve metalink for repository: epel/x86_64. Please verify its path and try again | 13:39 |
| sdake | need to know what is contained in tha tfile | 13:39 |
| nihilifer | sdake: try to add | 13:40 |
| nihilifer | RUN sed -i 's/^mirrorlist/#mirrorlist/' /etc/yum.repos.d/epel.repo | 13:41 |
| nihilifer | RUN sed -i 's/^#baseurl/baseurl/' /etc/yum.repos.d/epel.repo | 13:41 |
| sdake | i want to know contents of epel.repo unchanged :) | 13:41 |
| nihilifer | ah, ok | 13:41 |
| sdake | it is possible rpm -e pam is triggering problems with epel | 13:41 |
| larsks | sdake: highly unlikely. Sometimes repositories fail to answer, but work when you retry. | 13:42 |
| pbourke | sdake: contents fyi https://gist.github.com/brk3/ec38433a06f1efa3b92f | 13:42 |
| pbourke | nihilifer: you've tried s/https/http/ in that I take it? | 13:43 |
| nihilifer | yes, and after changint https to http in patchset 10, the gate at least didn't have problem with installing rdo-repo package | 13:44 |
| nihilifer | changing* | 13:44 |
| *** prad has joined #kolla | 13:44 | |
| pbourke | nihilifer: you only changed rdo in ps 10 though. not the epel.repo | 13:45 |
| sdake | 06:44:23] <sdake>is there some wierd outage with epel7 mirrorlist server | 13:45 |
| sdake | [06:45:21] <puiterwijk>sdake: we are having network issues. | 13:45 |
| sdake | [06:45:27] <puiterwijk>this should not impact mirrorlist, but for some reason it does | 13:45 |
| sdake | straight from #fedora-admin | 13:45 |
| nihilifer | pbourke: yes, I changes epel.repo in the newest patchset | 13:46 |
| nihilifer | changed* | 13:46 |
| pbourke | I guess we have our answer then | 13:46 |
| *** nihilifer has quit IRC | 13:48 | |
| openstackgerrit | Paul Bourke proposed stackforge/kolla: DO NOT MERGE: Attempt at fixing yum failing on epel repo https://review.openstack.org/193389 | 13:51 |
| sdake | well the gate is nonvoting now or should be in a short period | 13:52 |
| *** dims is now known as dimsum__ | 13:57 | |
| openstackgerrit | Paul Bourke proposed stackforge/kolla: DO NOT MERGE: Attempt at fixing yum failing on epel repo https://review.openstack.org/193389 | 13:57 |
| inc0__ | sooo...guys | 14:00 |
| sdake | 06:52:49] <nirik>it should be a good deal better now... | 14:00 |
| sdake | [06:53:12] <nirik>we isolated a network issue, but still have a db load issue. | 14:00 |
| sdake | [06:53:23] <nirik>but the network thing is likely what was hitting mirrorlists. | 14:00 |
| sdake | [06:54:08] kilted1 (~scotts@42-10-255-199.rev.celito.net) joined the channel | 14:00 |
| sdake | [06:56:50] <sdake>nirik hey hows it going:) | 14:00 |
| sdake | [06:56:54] Gibba (gwhite@nat/ibm/x-kvgdwhflkukgzmju) left IRC (Ping timeout: 245 seconds) | 14:00 |
| sdake | [06:57:00] <nirik>just lovely. | 14:00 |
| sdake | [06:57:02] <sdake>nirik I wanted to explain our use case a bit | 14:00 |
| inc0__ | whats up with gates? | 14:00 |
| sdake | [06:57:23] <nirik>well, I am trying to fix this stuff, so I can't really pay much attention. | 14:00 |
| sdake | [06:57:34] <sdake>nirik we build about 40 containers on every commit to our repo pre-commit | 14:00 |
| sdake | [06:57:35] <nirik>but can you retry whatever used mirrorlists? it should be fixed | 14:00 |
| sdake | [06:57:51] <sdake>this validates the commit works | 14:00 |
| sdake | [06:58:08] <sdake>this ends up doing about 100+ mirrorlist access operations | 14:00 |
| sdake | [06:58:16] plarsen (~plarsen@redhat/jboss/pdpc.professional.plarsen) joined the channel | 14:00 |
| sdake | [06:58:43] <sdake>nirik ack - just read when you get a chance :) | 14:00 |
| inc0__ | soo..they broke their mirrors right? | 14:00 |
| *** inc0__ is now known as inc0 | 14:03 | |
| sdake | inc0__ it is unclear who broke what | 14:04 |
| sdake | but the fact is something was broken and they have fixed half the problem | 14:04 |
| inc0 | so, do we need to change anything locally? | 14:05 |
| sdake | we need our build script to retry builds | 14:06 |
| sdake | it needs to be intelligent - only retrying if yum was the cause of the failure | 14:08 |
| sdake | and only optionally retrying based upon cmd line paramaeter | 14:08 |
| sdake | https://bugs.launchpad.net/kolla/+bug/1466677 | 14:09 |
| openstack | Launchpad bug 1466677 in kolla "build script errors out in yum" [Critical,In progress] - Assigned to Martin André (mandre) | 14:09 |
| sdake | the gate was changed to non-voting as well (I reverted jpeeler's change) | 14:10 |
| sdake | we will make voting again once the above bug/feature is implemented | 14:10 |
| sdake | so the gate should be unblocked shortly wrequiring rechecks | 14:10 |
| openstackgerrit | Michal Stachowski proposed stackforge/kolla: Galera container https://review.openstack.org/187225 | 14:12 |
| sdake | just use recheck operation - once the gate patch has been merged in gerrit | 14:12 |
| sdake | to recheck, in the review log leave the comment "recheck" | 14:12 |
| inc0 | also, can I ask you sdake and rhallisey to review keepalived, merge failed due to conflict and had to rabase | 14:13 |
| sdake | inc0 got a link | 14:13 |
| inc0 | I've never lost 2*+2+W this way;( | 14:13 |
| sdake | ya a rebase loses the acks | 14:14 |
| sdake | annoying property of gerrit | 14:14 |
| inc0 | https://review.openstack.org/#/c/187981/ | 14:14 |
| mstachow | is SamYaple here? | 14:14 |
| SamYaple | mstachow: yea | 14:15 |
| mstachow | Could You review my new patchset, please? | 14:15 |
| SamYaple | which one? | 14:15 |
| mstachow | It's lucky 13: https://review.openstack.org/#/c/187225/ | 14:16 |
| SamYaple | ok | 14:16 |
| inc0 | we'll need to figure out good way for init containers tho | 14:22 |
| mstachow | wow, tahnks Sam :O | 14:23 |
| *** nihilifer has joined #kolla | 14:25 | |
| *** nihilifer has quit IRC | 14:26 | |
| *** nihilifer has joined #kolla | 14:27 | |
| *** nihilifer has quit IRC | 14:31 | |
| *** nihilifer has joined #kolla | 14:32 | |
| sdake | as you have no doubt guessed, I am morpheus ! | 14:36 |
| inc0 | sdake, you should sleep more to achieve this title | 14:40 |
| inc0 | or rather, sleep at all | 14:40 |
| sdake | i got a good 9 hours yesterday | 14:40 |
| sdake | it was nice ;) | 14:40 |
| nihilifer | I see that build of image in patchset 13 worked http://logs.openstack.org/89/193389/13/check/check-kolla-functional-build-f21/f508375/console.html | 14:44 |
| nihilifer | at least before aborting ;) | 14:45 |
| openstackgerrit | Merged stackforge/kolla: Galera container https://review.openstack.org/187225 | 14:47 |
| sdake | btw guys | 14:47 |
| sdake | our gate is unjammed | 14:47 |
| sdake | the voting functional gate has been made nonvoting | 14:48 |
| sdake | so if folks want to pull the trigger ona recheck or rebase, we can get em merged | 14:48 |
| openstackgerrit | Merged stackforge/kolla: Cinder container https://review.openstack.org/170965 | 14:49 |
| *** daneyon has joined #kolla | 14:52 | |
| *** absubram has joined #kolla | 14:52 | |
| *** absubram has quit IRC | 14:54 | |
| *** absubram has joined #kolla | 14:55 | |
| inc0 | since we have mergefest, another +2 anyone plz?;) https://review.openstack.org/#/c/187981/ | 14:55 |
| inc0 | allright, have a nice weekend guys | 14:59 |
| inc0 | bye | 14:59 |
| *** daneyon_ has joined #kolla | 14:59 | |
| mstachow | huh It was really lucky 13 :O | 14:59 |
| *** inc0 has quit IRC | 15:01 | |
| *** daneyon has quit IRC | 15:02 | |
| sdake | the designate sink service here https://review.openstack.org/#/c/189393/ | 15:04 |
| sdake | needsa a erbase plz | 15:04 |
| jpeeler | sdake: this isn't funny, but it sort of is: http://status.fedoraproject.org/ | 15:04 |
| *** jasonsb has quit IRC | 15:04 | |
| sdake | jpeeler ya they updated that page after i bitched that their status page was green :) | 15:05 |
| jpeeler | OH | 15:05 |
| sdake | and complained we spent like 6 people over 8 hours jerking around figuring out its not our problem :) | 15:05 |
| jpeeler | there's this too, https://lists.fedoraproject.org/pipermail/devel/2015-June/211584.html | 15:05 |
| jpeeler | yeah... | 15:05 |
| sdake | jpeeler the nonvoting patch went in | 15:06 |
| sdake | we can revert it after mandre sorts out a retry operation | 15:06 |
| jpeeler | yeah i saw | 15:06 |
| sdake | I think we need retry anyway | 15:06 |
| jpeeler | agreed | 15:06 |
| sdake | that would have got our gate rolling een with the disruptions | 15:06 |
| sdake | i tlaked tohte fedora-admin guys, they said our use case (800 mirrorlist hits in short period) is a perfectly reasonable use case | 15:06 |
| sdake | and explained how we gate on their binaries | 15:07 |
| sdake | so they are not going to block us | 15:07 |
| sdake | and said hwe are not the cause of the problem | 15:07 |
| *** mstachow has quit IRC | 15:07 | |
| openstackgerrit | Michal Rostecki proposed stackforge/kolla: Add designate-sink service https://review.openstack.org/189393 | 15:09 |
| nihilifer | sdake: rebased :) | 15:09 |
| sdake | jpeeler this is a rebase: https://review.openstack.org/#/c/189393/ can you ack plz | 15:09 |
| jpeeler | done | 15:11 |
| *** daneyon_ has quit IRC | 15:12 | |
| pbourke | hate to hassle but might as well get re-ticked on these too https://review.openstack.org/#/q/owner:%22Paul+Bourke%22+status:open,n,z | 15:12 |
| sdake | pbourke use recheck on the ones that are marked -1 in the V field | 15:17 |
| pbourke | ok | 15:17 |
| jpeeler | yeah some of them were done before non-voting occurred | 15:17 |
| pbourke | i had to rebase a few of them | 15:17 |
| jpeeler | it seems a little unproductive to recheck when we know things are broken, but i guess maybe it'll get fixed faster than we think | 15:18 |
| pbourke | the never ending rebase cycle | 15:21 |
| *** nihilifer has quit IRC | 15:22 | |
| jpeeler | now that the cinder review merged, is bashate good to go? | 15:22 |
| bmace | just set recheck on mine. | 15:23 |
| bmace | which should be the last of bashate issues :) | 15:23 |
| *** dasm has quit IRC | 15:25 | |
| *** absubram has quit IRC | 15:26 | |
| sdake | hmm | 15:32 |
| sdake | recheck | 15:32 |
| sdake | I made the gate non-voting | 15:32 |
| sdake | so a recheck will fix it | 15:32 |
| sdake | will fix the gerrit -1 vote | 15:32 |
| sdake | and you are? | 15:34 |
| sdake | A smith. Agent smith. | 15:34 |
| openstackgerrit | Paul Bourke proposed stackforge/kolla: Symlink all non Dockerfile resources in images https://review.openstack.org/190117 | 15:37 |
| openstackgerrit | Merged stackforge/kolla: Fixed indentation issues to meet bashate compliance https://review.openstack.org/193320 | 15:40 |
| bmace | huzzah! | 15:43 |
| *** daneyon has joined #kolla | 15:46 | |
| *** jtriley has quit IRC | 15:47 | |
| *** jtriley has joined #kolla | 15:54 | |
| openstackgerrit | Merged stackforge/kolla: Add designate-sink service https://review.openstack.org/189393 | 15:54 |
| sdake | bashate gate still not passing | 15:56 |
| bmace | i have one that isn't in yet.. | 15:58 |
| bmace | in progress | 15:58 |
| openstackgerrit | Steven Dake proposed stackforge/kolla: Add base image for oraclelinux https://review.openstack.org/191013 | 16:01 |
| openstackgerrit | Paul Bourke proposed stackforge/kolla: Symlink all non Dockerfile resources in images https://review.openstack.org/190117 | 16:02 |
| sdake | pbourke when yu work on patch sets, take care to use an interactive rebae and resubmit all patches | 16:02 |
| sdake | thi will keep the "OUTDATED" from showing up in gerrit | 16:02 |
| pbourke | sdake: ok, thanks | 16:03 |
| sdake | gerrit doesn't know what to do with an outdated dependency | 16:03 |
| sdake | so it requires a rebase | 16:03 |
| pbourke | https://review.openstack.org/#/c/190117/ is the one that is causing a lot of headache | 16:03 |
| sdake | git review will ask you if you really want to do that | 16:03 |
| sdake | the answer is yes :) | 16:03 |
| pbourke | it needs to be rebased often and outdates everything that depends on it. so if it passes and people want to give it one more once over Id appreciate it | 16:04 |
| *** cbaesema has quit IRC | 16:04 | |
| sdake | pbourke ya but if you use git rebase -i to rebase your patch set in one repo | 16:04 |
| sdake | and use the edit option | 16:04 |
| sdake | you can edit just that patch | 16:05 |
| sdake | and then git review iwill keep all the updates in place | 16:05 |
| openstackgerrit | Merged stackforge/kolla: Fixed indenting issue to meet bashate compliance. https://review.openstack.org/193319 | 16:05 |
| bmace | alright, now bashate should be happy | 16:05 |
| bmace | hrm, new errrors? galera | 16:06 |
| openstackgerrit | Steven Dake proposed stackforge/kolla: Test if the bashate gate is passing - DO NOT MERGE https://review.openstack.org/193615 | 16:06 |
| openstackgerrit | Paul Bourke proposed stackforge/kolla: Symlink all non Dockerfile resources in images https://review.openstack.org/190117 | 16:10 |
| openstackgerrit | Paul Bourke proposed stackforge/kolla: Fix 'unsupported locale setting' errors in init-runonce https://review.openstack.org/188432 | 16:10 |
| openstackgerrit | Paul Bourke proposed stackforge/kolla: Minor doc fixups in dev-quickstart.md https://review.openstack.org/190543 | 16:11 |
| openstackgerrit | Paul Bourke proposed stackforge/kolla: Add base image for oraclelinux https://review.openstack.org/191013 | 16:11 |
| openstackgerrit | Paul Bourke proposed stackforge/kolla: Create keystone user in start.sh if it doesn't exist https://review.openstack.org/191071 | 16:11 |
| pbourke | http://vignette2.wikia.nocookie.net/simpsonstappedout/images/c/cc/Scorpio_flamethrower.png | 16:12 |
| *** pbourke has quit IRC | 16:13 | |
| *** pbourke has joined #kolla | 16:13 | |
| sdake | samyaple or jpeeler or rhallisey can one of you review the rebase https://review.openstack.org/#/c/190117/ | 16:14 |
| openstackgerrit | BorneMace proposed stackforge/kolla: Fixed non-bashate compliant method signature https://review.openstack.org/193621 | 16:15 |
| *** diga has joined #kolla | 16:17 | |
| openstackgerrit | BorneMace proposed stackforge/kolla: Fixed non-bashate compliant file, missing newline at end https://review.openstack.org/193622 | 16:19 |
| openstackgerrit | Paul Bourke proposed stackforge/kolla: Fix broken cinder/galera symlinks for fedora & rhel https://review.openstack.org/193623 | 16:20 |
| *** jtriley has quit IRC | 16:21 | |
| sdake | pbourke those should be 4 commits | 16:22 |
| *** jtriley has joined #kolla | 16:23 | |
| pbourke | sdake: kk | 16:23 |
| pbourke | was thinking maybe they should have been | 16:23 |
| sdake | ya different fixes = different commits | 16:24 |
| sdake | that way reviewers dont look at i tand go "damn thats big, let me go watch tv instead" | 16:24 |
| sdake | they may do that with a large patch stream | 16:24 |
| sdake | but atleast some of the patches get reviewed :) | 16:24 |
| sdake | make sense? | 16:24 |
| sdake | objective as a patch submitter is to reduce reviewer fatigue wherever possible | 16:25 |
| pbourke | sure thing thanks :) | 16:25 |
| sdake | brb | 16:26 |
| *** sdake_ has joined #kolla | 16:27 | |
| openstackgerrit | Paul Bourke proposed stackforge/kolla: Fix broken cinder symlinks for fedora https://review.openstack.org/193625 | 16:29 |
| sdake_ | jpeeler can you review https://review.openstack.org/#/c/190117/ so we can stop the rebase madness plz :) | 16:30 |
| openstackgerrit | Paul Bourke proposed stackforge/kolla: Fix broken cinder symlinks for fedora https://review.openstack.org/193625 | 16:30 |
| jpeeler | done | 16:31 |
| *** sdake has quit IRC | 16:31 | |
| sdake_ | thansk :) | 16:31 |
| sdake_ | bmace you may have to rebase your bashate changes, not sure :) | 16:31 |
| sdake_ | pbourke this blueprint doesn't link blueprint refactor-base-image-layout | 16:32 |
| sdake_ | is there a blueprint for that? | 16:32 |
| sdake_ | and if so, have a link | 16:32 |
| pbourke | sdake_: there is, let me check | 16:33 |
| bmace | sdake_: i had re-pulled right before i made those changes.. so i hope not :/ | 16:33 |
| bmace | not that it would be super hard if i do | 16:33 |
| sdake_ | well pbourke's change is going to change things a bit | 16:33 |
| pbourke | strange it seems to have disappeared | 16:34 |
| bmace | well, stop it pbourke ;) | 16:34 |
| sdake_ | pbourke can you make a new blueprint | 16:35 |
| sdake_ | it doesn't have to be super detailed | 16:35 |
| sdake_ | I have no idea where it went either | 16:35 |
| sdake_ | i remember reading it in detail | 16:35 |
| pbourke | https://blueprints.launchpad.net/kolla/+spec/refactor-base-image-layout | 16:35 |
| pbourke | its there | 16:35 |
| pbourke | but for some reason its not searchable in launchpad | 16:35 |
| openstackgerrit | Paul Bourke proposed stackforge/kolla: Fix broken cinder symlinks for rhel https://review.openstack.org/193623 | 16:37 |
| sdake_ | well i'm stumped | 16:37 |
| sdake_ | maybe its the length of the blueprint name | 16:37 |
| sdake_ | its listed here https://blueprints.launchpad.net/kolla/liberty | 16:38 |
| sdake_ | it is also listed in our liberty-1 blueprints | 16:40 |
| sdake_ | so that wfm | 16:40 |
| sdake_ | I don't know why it doesn't link which is irritating | 16:40 |
| sdake_ | but oh well :) | 16:40 |
| *** athomas has quit IRC | 16:41 | |
| pbourke | yeah Im hesitant to create a new one as we lose all the info and context of the first | 16:42 |
| pbourke | I can do so though if you like | 16:42 |
| diga | sdake_: Hi | 16:42 |
| sdake_ | sup diga | 16:42 |
| diga | Can we split this work - https://review.openstack.org/#/c/189157/ | 16:42 |
| sdake_ | pbourke no action needed on the blueprint linking problem | 16:43 |
| sdake_ | diga yes Id' like to split it up | 16:43 |
| diga | yes | 16:43 |
| diga | Better we will split this into two BP's | 16:43 |
| sdake_ | diga we are working on a prototype keystone ansible implementaiton to set the standard for how this work should be done | 16:43 |
| sdake_ | which two would you recommend | 16:44 |
| sdake_ | there are 5 actions in the specification | 16:44 |
| diga | sorry 5 work iteam | 16:44 |
| sdake_ | can you file a blueprint to rename kolla to kolla-compose | 16:44 |
| sdake_ | and file a blueprint to implement kolla-ansible | 16:44 |
| diga | yes | 16:44 |
| sdake_ | those are two that pop out immediatley | 16:45 |
| diga | yes | 16:45 |
| sdake_ | i'll approve them for liberty2 | 16:45 |
| sdake_ | make suer to link to the spec the work item it represents | 16:45 |
| diga | yes, sound good to me | 16:45 |
| diga | let me file those BP right way | 16:45 |
| pbourke | SamYaple I was looking at your ip_wrapper change but got distracted with the gate problems. Will have have another look Monday | 16:46 |
| sdake_ | after you finish with that we can decompoe the rest of them | 16:46 |
| pbourke | have a nice weekend all :) | 16:46 |
| sdake_ | enjoy pbourke | 16:46 |
| *** jasonsb has joined #kolla | 16:49 | |
| *** sdake has joined #kolla | 16:52 | |
| *** sdake_ has quit IRC | 16:55 | |
| diga | sdake: take a look at this - https://blueprints.launchpad.net/kolla/+spec/implement-kolla-ansible-utility-for-multinode | 16:56 |
| diga | sdake: you can make change if you want | 16:57 |
| sdake | diga i shortened name a bit | 16:57 |
| sdake | https://blueprints.launchpad.net/kolla/+spec/kolla-ansible-script | 16:58 |
| *** bradjones has quit IRC | 16:58 | |
| diga | yes | 16:58 |
| sdake | we need a rename-kolla-script blueprint as well | 16:58 |
| diga | :) | 16:58 |
| diga | okay | 16:58 |
| sdake | it is more then renaming i t is changign the docs and a bunch of stuff | 16:58 |
| *** bradjones has joined #kolla | 16:58 | |
| *** bradjones has joined #kolla | 16:58 | |
| sdake | the heat etmplate | 16:58 |
| sdake | etc | 16:58 |
| openstackgerrit | Merged stackforge/kolla: Symlink all non Dockerfile resources in images https://review.openstack.org/190117 | 16:59 |
| openstackgerrit | Merged stackforge/kolla: Create keystone user in start.sh if it doesn't exist https://review.openstack.org/191071 | 16:59 |
| sdake | hot | 16:59 |
| sdake | jpeeler can you take a quick look at this - its a 1 liner https://review.openstack.org/#/c/193622/ | 17:00 |
| sdake | and https://review.openstack.org/#/c/193621/ | 17:00 |
| sdake | alos a one liner | 17:00 |
| *** mfalatic has joined #kolla | 17:03 | |
| *** sdake_ has joined #kolla | 17:05 | |
| diga | okay | 17:06 |
| *** sdake has quit IRC | 17:09 | |
| *** bradjones has quit IRC | 17:11 | |
| *** bradjones has joined #kolla | 17:12 | |
| *** bradjones has joined #kolla | 17:12 | |
| openstackgerrit | Ryan Hallisey proposed stackforge/kolla: Add cinder into the docker compose setup https://review.openstack.org/173507 | 17:15 |
| *** nihilifer has joined #kolla | 17:16 | |
| sdake_ | jpeeler yet another rebase https://review.openstack.org/#/c/173507/ | 17:16 |
| openstackgerrit | Ryan Hallisey proposed stackforge/kolla: Access volume groups created by Cinder from the host https://review.openstack.org/187313 | 17:17 |
| sdake_ | jpeeler yet another rebase https://review.openstack.org/#/c/187313/ | 17:18 |
| sdake_ | rhallisey can you ack this bashate bug fix https://review.openstack.org/#/c/193622/ | 17:18 |
| sdake_ | rhallisey and htis other 1 liner https://review.openstack.org/#/c/193621/ | 17:19 |
| rhallisey | nice once we get all this stuff merged I think we're lookin good | 17:19 |
| sdake_ | one would hope so | 17:21 |
| sdake_ | bmace if your up for an adventure you can try to make a nonvoting bashate gate | 17:21 |
| *** inc0 has joined #kolla | 17:22 | |
| sdake_ | yo inc0 | 17:22 |
| inc0 | damn, you've broken me guys, it's been a while since I've logged to irc after work;) | 17:23 |
| inc0 | not to mention gerrit | 17:23 |
| bmace | bleeeeh, i have to rebase my bashate changes. | 17:24 |
| inc0 | sdake_, do we have concrede dates for kollapalooza? | 17:28 |
| sdake_ | told ya so bmace | 17:28 |
| sdake_ | inc0 waiting on facilities | 17:28 |
| sdake_ | I proposed 28th and 29th of June | 17:28 |
| sdake_ | but dont book yet, waiting on ack | 17:28 |
| *** rhallisey has quit IRC | 17:29 | |
| *** jasonsb has quit IRC | 17:31 | |
| inc0 | btw do you have problems with building our base images on ubuntu? | 17:32 |
| inc0 | selinux doesn't install for me:/ | 17:32 |
| *** rhallisey_ has joined #kolla | 17:32 | |
| inc0 | systemd** | 17:32 |
| *** dwalsh has quit IRC | 17:37 | |
| *** vinkman has joined #kolla | 17:37 | |
| openstackgerrit | BorneMace proposed stackforge/kolla: Fixed non-bashate compliant file, missing newline at end https://review.openstack.org/193622 | 17:44 |
| sdake_ | rather not june july | 17:46 |
| sdake_ | inc0 if you do the work knock yourself out :) | 17:46 |
| sdake_ | inc0 I proposed July 28th, July 29th to facilities, have to see if I can lock that up - but I'll let you know via ml as soon as I know | 17:47 |
| *** loth has joined #kolla | 17:48 | |
| inc0 | cool, as long as its after 17th of July, I should be able to come | 17:48 |
| inc0 | never been in USA, so that'll be a first;) | 17:48 |
| sdake_ | inc0 i'm going to be fighting hard for 28th/29th | 17:48 |
| bmace | ugh.. more bashate issues checked in, now mariadb.. fix inc. | 17:49 |
| *** rhallisey_ is now known as rhallisey | 17:49 | |
| openstackgerrit | BorneMace proposed stackforge/kolla: Fixed non-bashate compliant method signature https://review.openstack.org/193621 | 17:50 |
| sdake_ | rhallisey or jpeeler can you ack that one https://review.openstack.org/#/c/193621/ | 17:51 |
| jpeeler | done! | 17:52 |
| sdake_ | ta | 17:52 |
| openstackgerrit | BorneMace proposed stackforge/kolla: Fixed non-bashate compliance indentation in mariadb script https://review.openstack.org/193659 | 17:53 |
| bmace | unless someone sneaks another one in, that should be it for bashate :) | 17:54 |
| jpeeler | ok these types of commits can be in one together | 17:54 |
| sdake_ | jpeeler one more for review :) | 17:54 |
| bmace | sdake_ specifically asked me to keep them separate | 17:54 |
| sdake_ | jpeeler I prefer separat echanges per item | 17:54 |
| sdake_ | i mean i coudl have jammed my 18 bahate commits into one | 17:54 |
| jpeeler | sdake_: they are related though | 17:54 |
| openstackgerrit | Merged stackforge/kolla: Fixed non-bashate compliant file, missing newline at end https://review.openstack.org/193622 | 17:54 |
| sdake_ | but would anyone have reviweed it :) | 17:54 |
| sdake_ | unlikley | 17:54 |
| openstackgerrit | Merged stackforge/kolla: Add cinder into the docker compose setup https://review.openstack.org/173507 | 17:55 |
| jpeeler | well yeah, i mean if you have 20+ files | 17:55 |
| sdake_ | he had 4 files | 17:55 |
| sdake_ | i guess it doesn't matter | 17:55 |
| jpeeler | more clicking with more reviews! if the file count isn't that high and the changes are related, i say keep it in one commit | 17:56 |
| bmace | i don't really care one way or the other. there could be a sort of "max files / review" guideline for this sort of thing, though files isn't terribly reflective also, if there are a ton of changes per file, for example.. max LOC | 17:56 |
| sdake_ | i think we want to encourage smaller commits that are self contained to prevent reviewer fatigue | 17:56 |
| sdake_ | but ymmv :) | 17:56 |
| sdake_ | just trying to teach best practices jpeeler :) | 17:56 |
| sdake_ | but I guess judgement matters | 17:56 |
| jpeeler | i mean, there's viewer fatigue in multiple reviews each containing one space change as well | 17:57 |
| *** jasonsb has joined #kolla | 17:58 | |
| jpeeler | but perhaps not as much as 200 file review, so maybe it isn't worth hammering. but i just don't stay quiet heh | 17:58 |
| sdake_ | bmace if you are in for an adventure, you might take a stab at making the bashate gate voting | 17:58 |
| sdake_ | jpeeler its aok to make noies - I do it all the time ;) | 17:58 |
| sdake_ | hoise | 17:58 |
| sdake_ | nosie | 17:58 |
| sdake_ | after I learn how to type | 17:58 |
| sdake_ | NOISE | 17:58 |
| jpeeler | bmace: would you like to know where to look, or would that take away the fun | 17:59 |
| bmace | hrm, i have some guessts, based on all the gate chaos going on lately i saw some changes made in that area, especially to make the functional one non-voting | 18:00 |
| sdake_ | one note, the infra team doen't require bugs or blueprints attached to their changes | 18:00 |
| sdake_ | so don't create more work for them by filing one :) | 18:00 |
| jpeeler | ok, it's a two line change (3 including removal of newline) | 18:00 |
| *** prad_ has joined #kolla | 18:01 | |
| bmace | seems like that stuff is in a separate code repo, which makes sense :) | 18:01 |
| sdake_ | openstack-infra/project-config | 18:02 |
| sdake_ | shower time | 18:04 |
| sdake_ | bbl | 18:04 |
| harmw | jpeeler: I can agree with the 200files patchset | 18:09 |
| harmw | +1 for keeping functional bits and pieces together | 18:10 |
| nihilifer | inc0: ubuntu has some bug with aufs | 18:12 |
| nihilifer | and because of that some yum packages (i.e. systemd) cannot be installed | 18:12 |
| nihilifer | give me a moment, I will search a link with solution :P | 18:13 |
| nihilifer | which worked for me | 18:13 |
| inc0 | thank you nihilifer | 18:16 |
| sdake_ | nhi* I think inc0 wants to use ubuntu as a container os | 18:17 |
| sdake_ | which is unimplemented | 18:17 |
| sdake_ | lunch time bbl | 18:17 |
| inc0 | actually sdake_ no, as host:) | 18:17 |
| sdake_ | that used to work - i tested it | 18:18 |
| sdake_ | this is why pam+noauth is in the container image | 18:18 |
| nihilifer | inc0: http://logs.openstack.org/89/193389/13/check/check-kolla-functional-build-f21/f508375/console.html | 18:19 |
| sdake_ | ok gotta jet | 18:19 |
| nihilifer | fuck, not this link | 18:19 |
| bmace | i think the changes are ready for the voting bashate but i should probably wait until the last of my fixes actually go in. they are in-flight right now | 18:19 |
| nihilifer | inc0: https://github.com/docker/docker/issues/6980#issuecomment-62723470 | 18:19 |
| inc0 | o, thank you nihilifer | 18:20 |
| *** smeisner has quit IRC | 18:23 | |
| *** smeisner has joined #kolla | 18:35 | |
| *** jasonsb_ has joined #kolla | 18:37 | |
| *** jasonsb has quit IRC | 18:37 | |
| *** nihilifer has quit IRC | 18:51 | |
| harmw | god I just love those big reviews | 18:59 |
| harmw | thanks SamYaple :p | 18:59 |
| openstackgerrit | Merged stackforge/kolla: Fixed non-bashate compliance indentation in mariadb script https://review.openstack.org/193659 | 18:59 |
| openstackgerrit | Merged stackforge/kolla: Fixed non-bashate compliant method signature https://review.openstack.org/193621 | 19:00 |
| *** sdake_ has quit IRC | 19:19 | |
| *** bradjones has quit IRC | 19:20 | |
| *** daneyon has quit IRC | 19:22 | |
| *** bradjones has joined #kolla | 19:22 | |
| *** bradjones has joined #kolla | 19:22 | |
| jpeeler | docker-compose 1.3.0 was released yesterday, need to update all those git installs to use pip | 19:36 |
| *** gfidente has quit IRC | 19:42 | |
| *** dwalsh has joined #kolla | 19:45 | |
| *** inc0 has quit IRC | 20:09 | |
| *** sdake has joined #kolla | 20:16 | |
| *** sdake_ has joined #kolla | 20:17 | |
| sdake_ | yo | 20:17 |
| bmace | yoyo | 20:19 |
| bmace | just need a workflow on the bashate review to get voting in action | 20:20 |
| *** sdake has quit IRC | 20:21 | |
| jpeeler | does anybody have a working kolla deploy handy? i'm curious if "docker ps" shows the mariadb-data container or not | 20:22 |
| jpeeler | the note in the dockerfile makes it seem like it may exit | 20:22 |
| sdake_ | jpeeler only ps -a will show that | 20:22 |
| jpeeler | so the data container starts and then exits? | 20:22 |
| sdake_ | roger | 20:23 |
| jpeeler | good to know | 20:24 |
| *** prad_ has quit IRC | 20:26 | |
| *** daneyon has joined #kolla | 20:43 | |
| *** daneyon has quit IRC | 20:48 | |
| *** rhallisey has quit IRC | 20:50 | |
| *** britthouser has quit IRC | 20:50 | |
| sdake_ | jpeeler the reason the data container starts is simply to create teh mount blobs in the filesystem | 21:10 |
| sdake_ | that is why all the data contains execute bin/true (which on conclusion exit) | 21:10 |
| jpeeler | the mariadb one does /bin/bash | 21:11 |
| jpeeler | so if it was an invalid command there, there mounts would fail? | 21:12 |
| sdake_ | mariadb should be changed to /bin/true | 21:12 |
| sdake_ | i am not sure on that point jpeeler | 21:13 |
| jpeeler | that's the only data container right? | 21:13 |
| sdake_ | tehre are sevearl | 21:14 |
| *** daneyon has joined #kolla | 21:14 | |
| jpeeler | ah i found the nova-compute one | 21:14 |
| *** jasonsb_ has quit IRC | 21:18 | |
| sdake_ | jpeeler this needs a rebase https://review.openstack.org/#/c/189974/ | 21:19 |
| *** bradjones has quit IRC | 21:20 | |
| *** bradjones has joined #kolla | 21:22 | |
| *** bradjones has joined #kolla | 21:22 | |
| openstackgerrit | Jeff Peeler proposed stackforge/kolla: Fix Heat container env vars and dependencies https://review.openstack.org/189974 | 21:34 |
| *** jtriley has quit IRC | 21:41 | |
| *** dwalsh has quit IRC | 21:46 | |
| *** daneyon has quit IRC | 21:57 | |
| *** jasonsb has joined #kolla | 22:11 | |
| *** dimsum__ has quit IRC | 22:19 | |
| *** dimsum__ has joined #kolla | 22:20 | |
| *** sdake_ has quit IRC | 22:25 | |
| *** dimsum__ has quit IRC | 22:32 | |
| *** mfalatic has quit IRC | 23:03 | |
| *** jasonsb has quit IRC | 23:06 | |
| *** juggler has quit IRC | 23:06 | |
| *** juggler has joined #kolla | 23:06 | |
| vinkman | So, why is it not allowed to run the build script as root? I know in general it is bad form to run anything as root but docker is a pain running non root... | 23:13 |
| *** juggler has quit IRC | 23:16 | |
| *** juggler has joined #kolla | 23:17 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
