SamYaple | sweet, right before the deadline | 00:00 |
---|---|---|
SamYaple | UTC 00:00 just happened | 00:00 |
SamYaple | L3 has begun! | 00:00 |
*** achanda has quit IRC | 00:01 | |
*** achanda has joined #kolla | 00:01 | |
bmace | but the deadline reflects what is merged, not what is up for review, right? | 00:03 |
SamYaple | bmace: i was just kidding | 00:04 |
SamYaple | the deadline is in our control to a degree | 00:04 |
SamYaple | lol who is on #openstack ? | 00:05 |
SamYaple | did you see what someone just posted? | 00:05 |
bmace | missed it, re-post? | 00:06 |
SamYaple | 00:03 < annilai> /msg NickServ VERIFY REGISTER annilai uutwdoohtogl | 00:06 |
SamYaple | lol | 00:06 |
bmace | i guess now if you ever want to pretend to be them you are all set unless someone mentions it. | 00:07 |
SamYaple | i told them to change there password and quick | 00:07 |
SamYaple | all i can see is hunter2 though | 00:07 |
SamYaple | for those not getting the joke http://www.bash.org/?244321 | 00:09 |
bmace | hrm, it says variables set in one roles are available to other roles.. so if we just toss the per role stuff in roles/service/vars/main.yml the docs say other roles should be able to see it, and i don't see anything about override breaking | 00:13 |
SamYaple | bmace only if the role runs | 00:13 |
SamYaple | and only after it runs | 00:13 |
SamYaple | haproxy needs all those ports and it runs first | 00:13 |
SamYaple | with --tags the other roles may not run at all | 00:14 |
bmace | hrm, i guess that part is omitted from the documentation | 00:14 |
bmace | having a xservice.yml up with defaults / globals and doing a -e@ for each is kind of ugly but would technically work. | 00:15 |
SamYaple | what we have now isn't ugly and does work though | 00:15 |
SamYaple | why fight it | 00:15 |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Add Ansible support for Heat https://review.openstack.org/205528 | 00:16 |
SamYaple | i mean no matter what we will have to specify at least the keystone port in the defaults.yml or globals.yml | 00:16 |
SamYaple | we reference it when building our auth strings | 00:17 |
bmace | i think we should, at a minimum, break out the vars in ansible/roles/<service>/defaults/main.yml so they can be easily modified programatically without needing to specifically ignore all the jinja2 stuff in there. easy enough to just include some file that just has the vars. | 00:17 |
SamYaple | what do you mean? | 00:17 |
SamYaple | you can modify any of those variables | 00:17 |
SamYaple | just put them in globals.yml | 00:17 |
SamYaple | that will override them | 00:17 |
SamYaple | this is the design of ansible | 00:18 |
SamYaple | its literally how all ansible things work | 00:18 |
bmace | so, keystone_public_port is similar to nova_api_port right, where multiple services need it, which is why it is up in etc/kolla/defaults.yml | 00:22 |
bmace | but keystone_public_port is also defined in keystone/defaults/main.yml whereas the nova_api port is not in the nova/defaults/main.yml | 00:22 |
SamYaple | bmace: correct. all ports are needed by multiple services | 00:22 |
SamYaple | bmace: that should be removed then | 00:22 |
*** shadower has quit IRC | 00:23 | |
*** shadower has joined #kolla | 00:23 | |
SamYaple | yea we dont want those in two places | 00:23 |
*** achanda has quit IRC | 00:25 | |
bmace | ok. i was just looking around and seeing some inconsistencies, and we either want them in one place consistently, or in both places consistently. and i can really live with whichever. if there are values that must exist at a global scope and don't exist anywhere else, maybe have them in a different file though from the file we want people to use to override values that exist elsewhere? | 00:25 |
*** achanda has joined #kolla | 00:25 | |
SamYaple | i dont understand. what is broken with what we are doing? | 00:25 |
SamYaple | this is vanilla ansible stuff | 00:26 |
bmace | it is like you just agreed, that the keystone value should be yanked from the role var file because it is a value needed at a global scope. but we always need it to be somewhere. | 00:27 |
bmace | having the same file be where people override variables, and where these required global variables define is, i think, problematic. some of the values will be safe to yank, like if they don't want to override something anymore.. some will break everything, because they are required across multiple roles | 00:28 |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Removes unneeded variables https://review.openstack.org/208104 | 00:28 |
SamYaple | for the record bmace, people shouldn't ever change the defaults.yml | 00:29 |
SamYaple | i was considering making it read-only | 00:29 |
SamYaple | the only file people should every touch is the globals.yml file | 00:29 |
SamYaple | and maybe the passwords.yml file, but that is going to be autogen'd anyway | 00:29 |
*** sdake has joined #kolla | 00:30 | |
sdake | ayr | 00:31 |
sdake | evenin folks | 00:31 |
bmace | ah, i suppose it is. time flies when you are having fun :) | 00:32 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Fix missing quotes in tools/genenv https://review.openstack.org/207888 | 00:33 |
SamYaple | sdake: just fyi when you do that you screw with pbourke stats | 00:33 |
SamYaple | youre the committer now | 00:33 |
sdake | oh didnt know that | 00:34 |
sdake | well i dont think he will mind since he said he is out until tuesday | 00:34 |
SamYaple | yea thats fair, just fyi | 00:35 |
SamYaple | i wish the rebase button kept the original commitor | 00:35 |
SamYaple | probably that is a bug that could be filed, but i dont know where | 00:35 |
SamYaple | sdake: it took me 6 patches to change my affiliation | 00:36 |
sdake | nice ;) | 00:36 |
SamYaple | the 5th one jenkins finally +1 | 00:36 |
SamYaple | but my commit message had a typo :/ | 00:36 |
SamYaple | i shouldnt be an openstack dev | 00:36 |
sdake | is the change live yet or still awaiting =2? | 00:36 |
SamYaple | waiting | 00:37 |
SamYaple | its only been 2 minutes since jenkins returned on the final patchset | 00:37 |
SamYaple | the ordering was the big issue | 00:37 |
SamYaple | orderd by launchpad id | 00:37 |
SamYaple | and 8 comes after - alphabetically apparently | 00:38 |
SamYaple | i mean i always thought the alphabet had no numbers or hypens, but eh | 00:38 |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Removes unneeded variables https://review.openstack.org/208104 | 00:40 |
bmace | its all about the ascii, 45 vs. 56 :) | 00:41 |
SamYaple | bmace: in this case its all about the Closes-Bug and not the Closed-Bug! | 00:41 |
bmace | ah, so picky!! we need some fuzzy matching | 00:42 |
SamYaple | warm and fuzzy | 00:42 |
bmace | best kind of fuzzy | 00:42 |
SamYaple | i like our community. its warm and fuzzy | 00:42 |
SamYaple | like mold | 00:42 |
SamYaple | good to see I am the one with the reviews holding up the tag again :/ | 00:43 |
SamYaple | sdake: you see those stats | 00:44 |
SamYaple | 70 bugs and 16 bluebrints | 00:44 |
SamYaple | bam | 00:44 |
sdake | you mean for l2? | 00:44 |
SamYaple | yea | 00:44 |
sdake | you mean in total or just you | 00:44 |
SamYaple | total | 00:44 |
sdake | ya pretty good output for 1 month | 00:45 |
sdake | some of the blueprints are misleading because they contain 20+commits thatwere hard | 00:45 |
SamYaple | over 2 bugs a day and 1 blueprint every two days | 00:45 |
*** Hui has quit IRC | 00:46 | |
sdake | i'm gld i'm getting help maintaing launchpad now | 00:48 |
sdake | doing it all by myself was taxing ;) | 00:48 |
SamYaple | were there for you | 00:48 |
SamYaple | i like that we are using launchpad more | 00:48 |
SamYaple | raking in that sweet sweet karma | 00:48 |
sdake | i wonder how much karma it rquires to reach nirvana | 00:49 |
* sdake can't handle any more lifetimes ;) | 00:49 | |
SamYaple | a shotguns worth | 00:49 |
sdake | looks like we will slip a day | 00:50 |
sdake | hope a core logs in tomorrow to hit the review queue | 00:50 |
sdake | harmw indicated he would when he woke up | 00:51 |
SamYaple | sounds like we should promote a core right now | 00:51 |
SamYaple | is there a battlefield promotion for Core reviews? | 00:51 |
SamYaple | is that a thing? | 00:51 |
sdake | no :) | 00:51 |
SamYaple | see bmace i tried | 00:51 |
sdake | there is a battlefield promotion to ptl tho ;) | 00:51 |
bmace | lol | 00:52 |
bmace | temporary promotion? lol | 00:52 |
SamYaple | wow ive had 7 patches merge today, 5 in the queue left | 00:53 |
SamYaple | most productive day ever amiright | 00:53 |
bmace | and answered a bunch of my goofy ansible questions, you are a machine! | 00:54 |
SamYaple | its easy you make things up | 00:54 |
sdake | samyou have almost caugt up with me in reviews ;) | 00:54 |
sdake | too bad coolsvap isnt into reviewing much | 00:56 |
* sdake makes reminder to harass coolsvp to do more reviews less code ;) | 00:56 | |
* SamYaple says "lol +2+1" | 00:56 | |
sdake | well time for idiot tube | 00:59 |
SamYaple | same here | 00:59 |
sdake | speaking of tubes, my new amp tubes arrived tody and worked yay :) | 01:00 |
SamYaple | got some rifftrax to watch | 01:00 |
SamYaple | awesome! | 01:00 |
*** jasonsb_ has quit IRC | 01:18 | |
*** echoingu_ has quit IRC | 01:28 | |
*** echoingumesh has joined #kolla | 01:29 | |
vinkman | sdake: another one? I thought you got one not very long ago... | 01:30 |
vinkman | or is this just the tubes not the amp? | 01:31 |
bmace | i think just new tubes. | 01:31 |
bmace | i think maybe even a series of one.. which is essentially what the internet is | 01:31 |
bmace | one = ones | 01:31 |
vinkman | Ahh..kk… I have tried but I am too death to hear the difference between a tube and a transistor amp… | 01:32 |
*** echoingumesh has quit IRC | 01:33 | |
bmace | ditto. been listening to mp3s too long. it actually sounds ok to me. | 01:33 |
vinkman | plus all my music is digital anyway… | 01:33 |
vinkman | LOL… So true! | 01:34 |
bmace | sdake is hard core. records! | 01:34 |
vinkman | Hmm. probably still 45's, none of the 33 junk | 01:36 |
*** superflyy has joined #kolla | 01:49 | |
*** erkules has quit IRC | 01:52 | |
*** erkules has joined #kolla | 01:53 | |
*** erkules_ has joined #kolla | 01:56 | |
openstackgerrit | Swapnil Kulkarni (coolsvap) proposed stackforge/kolla: Add build from source for Swift https://review.openstack.org/208110 | 01:57 |
*** erkules has quit IRC | 01:59 | |
sdake | samyaple if you could review https://review.openstack.org/#/c/208110/ it would be much appreciated ;) | 02:04 |
sdake | i hve almost all 33's ony a few 45s | 02:04 |
sdake | just because that is all I can get :( | 02:04 |
sdake | vinkman these are just NOS tubes to replace the stock tubes | 02:10 |
*** jasonsb has joined #kolla | 02:15 | |
*** jasonsb has quit IRC | 02:19 | |
*** coolsvap|away is now known as coolsvap | 02:20 | |
openstackgerrit | Merged stackforge/kolla: Fix missing quotes in tools/genenv https://review.openstack.org/207888 | 02:24 |
*** achanda has quit IRC | 02:34 | |
*** achanda has joined #kolla | 02:34 | |
openstackgerrit | Swapnil Kulkarni (coolsvap) proposed stackforge/kolla: Add some missing containers for fedora https://review.openstack.org/208117 | 02:43 |
sdake | nice work coolsvap ;) | 02:44 |
coolsvap | sdake, thanks i am just browsing through the code to add missing pieces | 02:45 |
sdake | does fedora work as a container os atm? | 02:45 |
sdake | notbuild but actually work | 02:45 |
coolsvap | i will need to build all the containers to do that, i will give you an update on that | 02:46 |
sdake | i think we need a whole lot of symlinks for build from source | 02:46 |
sdake | i htink the base image needs attention likely | 02:47 |
coolsvap | yes | 02:47 |
coolsvap | we also need to think about updating repos | 02:47 |
*** coolsvap is now known as coolsvap|away | 03:18 | |
*** jasonsb has joined #kolla | 03:18 | |
*** dimsum__ has quit IRC | 03:22 | |
*** dimsum__ has joined #kolla | 03:23 | |
*** superflyy has quit IRC | 03:25 | |
sdake | coolsvap do you ind if I take https://bugs.launchpad.net/kolla/+bug/1478451 | 03:39 |
openstack | Launchpad bug 1478451 in kolla "horizon config-external.sh has incorrect source & target" [High,New] - Assigned to Swapnil Kulkarni (coolsvap) | 03:39 |
SamYaple | sdake: review | 03:48 |
SamYaple | now i sleep | 03:48 |
sdake | review what | 03:51 |
sdake | samyaple ? | 03:52 |
sdake | coolsvap|afk do you mind if I reassign https://review.openstack.org/#/c/206789/ | 03:56 |
sdake | coolsvap|away ^^ | 03:56 |
*** dimsum__ has quit IRC | 03:58 | |
openstackgerrit | Swapnil Kulkarni (coolsvap) proposed stackforge/kolla: Add build from source for Swift https://review.openstack.org/208110 | 04:00 |
sdake | coosvap|away pingola | 04:00 |
*** coolsvap|away is now known as coolsvap | 04:00 | |
coolsvap | sdake, sure np | 04:02 |
coolsvap | sdake, sorry lost connectivity | 04:02 |
sdake | allgood | 04:02 |
sdake | thanks | 04:02 |
*** mfalatic has quit IRC | 04:11 | |
*** diogogmt has quit IRC | 04:19 | |
*** unicell has joined #kolla | 04:46 | |
*** sdake has quit IRC | 04:47 | |
*** sdake has joined #kolla | 04:55 | |
*** sdake has quit IRC | 05:03 | |
*** achanda has quit IRC | 05:08 | |
*** achanda has joined #kolla | 05:09 | |
*** coolsvap is now known as coolsvap|away | 05:33 | |
*** gfidente has joined #kolla | 07:11 | |
*** gfidente has quit IRC | 07:19 | |
*** achanda has quit IRC | 08:08 | |
*** vinkman has quit IRC | 08:49 | |
*** erkules_ is now known as erkules | 10:11 | |
*** erkules has quit IRC | 10:11 | |
*** erkules has joined #kolla | 10:11 | |
*** dimsum__ has joined #kolla | 10:51 | |
*** diogogmt has joined #kolla | 12:15 | |
*** diogogmt has quit IRC | 12:26 | |
openstackgerrit | Merged stackforge/kolla: Add build from source for Swift https://review.openstack.org/208110 | 15:06 |
SamYaple | morning folks | 15:14 |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Updated all tags for Liberty2 https://review.openstack.org/208157 | 15:19 |
*** pbourke has quit IRC | 15:31 | |
*** pbourke has joined #kolla | 15:32 | |
*** dimsum__ has quit IRC | 16:13 | |
*** superflyy has joined #kolla | 16:44 | |
*** sdake has joined #kolla | 16:49 | |
*** vbel has quit IRC | 16:49 | |
*** vbel has joined #kolla | 16:49 | |
sdake | morning | 16:51 |
SamYaple | morning! | 16:51 |
SamYaple | https://review.openstack.org/208157 | 16:51 |
SamYaple | i think thats the last merge beefore l2 tag | 16:51 |
*** dimsum__ has joined #kolla | 17:00 | |
*** dimsum__ has quit IRC | 17:08 | |
*** dimsum__ has joined #kolla | 17:09 | |
SamYaple | sdake: would that tag be better as "liberty-l2"? | 17:16 |
SamYaple | since i imagine we will want tags "liberty-1" liberty-2" for the actual releases | 17:16 |
*** dimsum__ has quit IRC | 17:27 | |
*** sdake has quit IRC | 17:52 | |
*** superflyy has quit IRC | 18:34 | |
*** jmccarthy has quit IRC | 18:36 | |
*** jmccarthy has joined #kolla | 18:37 | |
*** achanda has joined #kolla | 18:39 | |
*** vinkman has joined #kolla | 19:10 | |
harmw | have no fear, for I am here! | 19:15 |
harmw | or whatever :> | 19:15 |
*** mordred has quit IRC | 19:20 | |
openstackgerrit | Merged stackforge/kolla: Removes unneeded variables https://review.openstack.org/208104 | 19:25 |
openstackgerrit | Merged stackforge/kolla: Add some missing containers for fedora https://review.openstack.org/208117 | 19:28 |
openstackgerrit | Merged stackforge/kolla: Adds an example openrc file https://review.openstack.org/208097 | 19:29 |
openstackgerrit | Merged stackforge/kolla: Use sleep during database bootstrap https://review.openstack.org/208066 | 19:32 |
openstackgerrit | Merged stackforge/kolla: Fixes race condition in ansible playbooks https://review.openstack.org/208067 | 19:32 |
*** daneyon has joined #kolla | 19:37 | |
*** daneyon has quit IRC | 19:41 | |
*** britthou_ has joined #kolla | 19:53 | |
*** britthou_ has quit IRC | 19:55 | |
*** britthou_ has joined #kolla | 19:56 | |
*** britthouser has quit IRC | 19:56 | |
*** britthou_ has quit IRC | 20:04 | |
*** mordred has joined #kolla | 20:12 | |
*** sdake has joined #kolla | 20:39 | |
sdake | morning | 20:39 |
sdake | samyaple I dont know - i'll look how how other projects are tagging | 20:46 |
*** britthouser has joined #kolla | 21:05 | |
*** britthou_ has joined #kolla | 21:07 | |
*** britthouser has quit IRC | 21:10 | |
*** dimsum__ has joined #kolla | 21:17 | |
*** sdake has quit IRC | 21:27 | |
*** achanda has quit IRC | 21:38 | |
*** ccrouch1 has joined #kolla | 21:59 | |
*** achanda has joined #kolla | 22:44 | |
*** vinkman has quit IRC | 22:44 | |
*** vinkman has joined #kolla | 22:45 | |
*** achanda has quit IRC | 22:48 | |
*** sdake has joined #kolla | 22:54 | |
*** achanda has joined #kolla | 23:02 | |
*** sdake has quit IRC | 23:49 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!