Thursday, 2017-02-09

sdake_bmace_ maybe here: http://eavesdrop.openstack.org/irclogs/%23openstack-kolla/%23openstack-kolla.2017-02-08.log.html#t2017-02-08T13:47:4400:00
*** mannamne has joined #openstack-kolla00:01
sdake_groan daily gerrit backups00:01
sdake_sbezverk i am attempting to reivew yoru gate patch - howeer gerrit pokey00:02
sbezverksdake_: it is not 100% corect, basically that traceback indicates that nova cannot get keystone url for authentication00:02
bmace_sdake_: well, my 3.0.2 containers are coming up at least, and keystone is working.. even if it isn't 100% the swap between 3.x and 2.x is easy enough to tweak in the devenv startup00:02
sbezverksdake_: we did not have too much time to go in details00:02
sdake_sbezverk what port is it trying to get the keystone url for?00:02
sdake_sbezverk port 35357?00:03
sbezverkbmace_: the issue is visible only when you start VM00:03
bmace_sbezverk: i hit an issue and fixed it in my keystone startup where it was getting an internal only addr / port so if you look at my devenv change i moved it to find the external port of the keystone-admin service and that port also00:04
sbezverkit fails to launch and nova sceduler and compute have each a traceback00:04
sbezverkbmace_: could you please paste the link for your change?00:05
bmace_sbezverk: https://review.openstack.org/#/c/429295/11/tools/build_local_admin_keystonerc.sh00:05
sbezverksdake_: I am waiting for the gate results for refactor if it comes back relatively green, then please review00:05
bmace_without that the keystone test from a remote system most definitely does not work00:05
sdake_sbezverk will do at my folks house - going to dinner now00:06
sdake_bmace_ to fix that you need to set the external port config option on cloud.yaml00:06
sbezverkbmace_: I do not tink this change has any impact on nova-compute process00:06
sdake_sbezverk right he is talking about a different problem - keystone endpoint list i suspect00:07
sbezverkhere if I read it right, you build keystone_rc file00:07
sdake_sbezverk the same problem we had with keystone endpoitn list00:07
sbezverksdake_: ok then it is completely different issue00:07
bmace_sdake_: just like horizon, why would the cluster ip not be external by default?00:08
openstackgerritSerguei Bezverkhi proposed openstack/kolla-kubernetes master: Gate refactoring  https://review.openstack.org/42815000:08
sdake_bmace_ I had the same problem and same question00:11
sbezverkbmace_: I would agree for horizon it makes sense to have it external.. but from security reason it is better to make the end user to do it, if he/she needs00:11
sdake_bmace_ i am not sure i got a suitable answer00:11
sdake_the problem without an externally facing keystone endpoint is that it is impossible  to bootstrap openstack with things like images, networks, flavors, etc00:12
sdake_also keystone endpoint lsit doesn't work which means nova boot doesn't work00:12
sdake_nova boot depends on keystone endpoint list, not keystone catalog list00:12
sdake_(in newton)00:12
sdake_in mitaka that was not the case00:12
*** dave-mccowan has joined #openstack-kolla00:13
sdake_sbezverk could you expand on the security aspect when nova booting doesn't work properly without keystone endpoint list being functional?00:13
bmace_also, wouldn't you need these exposed if someone plans to use the cli at all?  i don't know how many people use horizon vs. cli, etc.. but external by default feels to be more likely what people want.. having a single change you can make to hide them all could be there for the paranoid folks.00:13
sdake_bmace_ it is configurable during helm install time00:13
sdake_bmace_ i think the default is wrong :)00:13
sdake_and i think sbezverk disagrees :)00:14
bmace_sdake_: sort of my points, yes, agreed00:14
bmace_i think the default case should == the most common case00:14
sbezverksdake_: in my cluster I do not have keystone admin exposed externally and nova do not have any problem with that, so I am not sure I undersatnd this problem00:14
sdake_nova booting may work fine with keystone, however, most people actually use the api00:14
sdake_sbezverk how are ou booting vms (via horizon?)00:14
sbezverkno, cli00:15
sdake_hmm well maybe it needs more digging - although we do need to be able to bootstrap the cluster as well as the admin user00:15
sdake_and at present,  there is no way to do that with the cli00:15
sdake_few operators bootstrap openstack via horizon00:16
sdake_they have scripts and more scripts and stuff :)00:16
sbezverksdake_: kfox1111 is an operator and I do not remeber him complaining about it00:17
*** krtaylor has joined #openstack-kolla00:20
*** sdake_ has quit IRC00:20
*** salv-orlando has quit IRC00:23
*** Pavo has quit IRC00:24
*** Pavo has joined #openstack-kolla00:25
*** lamt has quit IRC00:26
*** lamt has joined #openstack-kolla00:26
*** yingjun has joined #openstack-kolla00:28
*** lamt has quit IRC00:31
*** mannamne has quit IRC00:31
*** huzhengchuan has joined #openstack-kolla00:39
openstackgerritLi Yingjun proposed openstack/kolla-ansible master: Fix telegraf task  https://review.openstack.org/43064800:45
*** duonghq has joined #openstack-kolla00:45
*** duonghq__ has joined #openstack-kolla00:46
jascott1my god py3 takes 13mins?! https://benchmarksgame.alioth.debian.org/u64q/performance.php?test=nbody00:48
jascott1hipster node only takes ~28 secs00:49
kfox1111hey all.00:53
jascott1hey kfox1111 !00:54
jascott1how does it go?00:54
*** zhuzeyu has quit IRC00:54
kfox1111jascott1: yeah, that isn't the best python3 app... there's math libs that would make it a lot faster I think.00:55
jascott1ah00:55
kfox1111pretty good. how about you?00:55
kfox1111just got prometheus figured out to the point where if my ceph drops an osd, it will ping me on slack. :)00:55
jascott1good, fixing up PR based on technosophos's review. almost finished00:55
kfox1111nice. :)00:56
jascott1awesome!00:56
kfox1111really liking prometheus.00:56
sbezverkkfox1111: I think I found something wierd in nova config: check this out: http://paste.openstack.org/show/598154/00:56
jascott1yeah after this I hope to get to checkout computekit and all the neat stuff you guys have been working on00:56
kfox1111sbezverk: looking00:56
sbezverkI have never seen this config before00:56
jascott1i havent gotten to try it but hear good things00:56
sbezverkespecially since it point to wrong ip00:56
kfox1111sbezverk: yeah, thats wierd looking. where did that come from?00:57
sbezverkmust be from kolla00:57
sbezverkit in in config generated by genconfig00:57
kfox1111wonder if thats a recent thing.00:58
kfox1111hah00:59
kfox1111awesome00:59
kfox1111https://github.com/CanonicalLtd/prometheus-openstack-exporter00:59
sbezverkfu....g it was added feb 2nd01:01
sbezverkfacfabf3 (Jeffrey Zhang    2017-02-01 21:46:32 +0800 50)01:01
sbezverkfacfabf3 (Jeffrey Zhang    2017-02-01 21:46:32 +0800 51) - name: Creating the placement project, user, and role01:01
sbezverkfacfabf3 (Jeffrey Zhang    2017-02-01 21:46:32 +0800 52)   kolla_toolbox:01:01
kfox1111yeah, we need to add cross repo gating asap. :/01:01
sbezverkI think we started see it since you unpin ansible01:02
kfox1111or wait to the ptg, get configmaps figured out and get out of kolla-ansible.01:02
*** cuongnv has joined #openstack-kolla01:03
sbezverkyeaah, we need to get away of other poeple breaking us, we can do it better ourselves ;)01:03
*** huzhengchuan has quit IRC01:03
sbezverkcould you pin back ansible to old version we used before?01:03
sbezverkjust to prove the point01:03
kfox1111apply:01:05
kfox1111git fetch https://git.openstack.org/openstack/kolla-kubernetes refs/changes/65/427365/4 && git format-patch -1 --stdout FETCH_HEAD | patch -p1 -R01:05
openstackgerritLi Yingjun proposed openstack/kolla-ansible master: Fix telegraf task  https://review.openstack.org/43064801:06
sbezverkkfox1111: thanks I will add it to the gate setup01:06
kfox1111full review's here:01:06
kfox1111https://review.openstack.org/#/c/427365/401:06
kfox1111if thats easier.01:06
*** zhubingbing_ has joined #openstack-kolla01:06
kfox1111we totally need a container for prometheus-openstack-exporter01:07
sbezverkkfox1111: again stats?01:10
kfox1111seems to gather some very interesting metrics.01:12
*** zhangqiankun has joined #openstack-kolla01:14
openstackgerritSerguei Bezverkhi proposed openstack/kolla-kubernetes master: Gate refactoring  https://review.openstack.org/42815001:14
sbezverkkfox1111: nice, it will make cncf people happy ;)01:14
*** Pavo has quit IRC01:15
*** huzhengchuan has joined #openstack-kolla01:16
kfox1111I've been looking for a monitoring system dynamic enough to deal with cloud stuff.01:17
kfox1111prometheus is the first system I've seen that can keep up with it properly.01:17
kfox1111not really surprising considering its to kubernetes what borgmon is to borg, and designed based on google's borgmon.01:18
sbezverkkfox1111: interesting, when config gen problem gets resolved will try it out too01:18
kfox1111been using the upstream helm chart for testing. its pretty good.01:18
*** sdake has joined #openstack-kolla01:18
sdakesbezverk sorry had to drop off01:19
sdakesbezverk to relocate01:19
sdakere kfox not complaining, im not sure how he plans to bootstrap his cluster without external 35357 port access01:19
sdakekfox1111 - if your around would love to hear how you do an operation like init-runonce01:19
*** sdake has quit IRC01:20
*** sdake has joined #openstack-kolla01:21
openstackgerritSerguei Bezverkhi proposed openstack/kolla-kubernetes master: Gate refactoring  https://review.openstack.org/42815001:21
sbezverksdake: we found potential out gate breakage01:22
sbezverkin nova compute01:22
sbezverkhere is the config which got generated for us by kolla-ansible01:23
kfox1111sdake: whats the behavior of init-runonce?01:23
sdakekfox1111 moment:01:23
sdakehttps://github.com/openstack/kolla-ansible/blob/master/tools/init-runonce01:24
*** salv-orlando has joined #openstack-kolla01:24
*** mgiles has quit IRC01:25
kfox1111looks similar to the script that tests the system.01:25
kfox1111so, loads a bunch potentially useful defaults in...01:26
sdakeright - i call it clodu bootstrapping maybe it has a different name01:26
kfox1111yeah.01:26
openstackgerritSerguei Bezverkhi proposed openstack/kolla-kubernetes master: Gate refactoring  https://review.openstack.org/42815001:26
kfox1111I'd think a k8s job would be well suited to run that.01:27
sdakeand it would be configured via an external configmap?01:28
kfox1111so maybe a kolla-simple-cloud-init helm package or something like that, with a job that sets all that stuff up?01:28
sdakeso that sounds good short-to-medium term01:28
kfox1111yeah.01:28
sdakein the meantime, we need to unblock devs today01:28
sdakeand if people can't bootstrap their cloud - they hae problems01:29
*** tvignaud has joined #openstack-kolla01:29
sdakei'd propose changing the defaults temporarily01:29
kfox1111what defaults?01:29
sdakeor atleast documenting how they can be changed - i don't even recall what changes I made01:29
kfox1111thats registering stuff into the cloud.01:29
*** zhurong has joined #openstack-kolla01:29
sdakekfox1111 there is an external_port_interface thing in the keystone object01:29
*** salv-orlando has quit IRC01:29
kfox1111all the things can be overridden with helm vars.01:29
sbezverkyep, just provide your values.yaml file01:30
*** rwallner has joined #openstack-kolla01:30
sbezverkI think I showed you already with cloud,yaml01:31
sdakesbezverk indeed you did01:31
sbezverksdake ^^01:31
sdakesbezverk and like a moron i didn't take notes01:31
sbezverkbut you should still that file? right01:31
sbezverkI mean have that file01:31
sdakei was so excited keystone was working I was not taking notes :)01:31
sdakeI may have it01:31
sdakeI dont recall how it was generated01:31
sdakekfox1111 told me the other day01:32
sdakedon't recall that either01:32
sdakebrain = can't remember01:32
kfox1111tools/build_example_yaml.py01:32
sbezverkkfox1111: it looks like this config screwes things up auth_url = http://10.10.10.254:3535701:33
sbezverkI am still waiting to confirm, but something tells me it should be it01:34
*** caowei has joined #openstack-kolla01:34
*** zhurong has quit IRC01:35
*** rwallner has quit IRC01:35
kfox1111hmm...01:38
kfox1111is that suposed to be an external address?01:39
kfox1111does it have any overlap with the pod networks or service network?01:39
*** adrian_otto has quit IRC01:39
*** yingjun has quit IRC01:40
sbezverkkfox1111: it does not overlap, but I think it overrides our auth_url = http://keystone-admin:35357/v301:40
sbezverkkfox1111: they put default kolla's vip address01:41
sbezverkwithout checking for orchestration engine01:41
kfox1111whats admin_port_external set to?01:44
kfox1111yeah, I guess in the kolla generated config files it should be: keystone_admin_url: "http://keystone-admin:35357/v3"01:45
bmace_that is essentially what i build dynamically in that keystonerc change.01:46
*** yingjun has joined #openstack-kolla01:46
kfox1111so  in the gate, we append: tests/conf/ceph-all-in-one/kolla_config to /etc/kolla/globals.yaml before running genconfig.01:48
sbezverkbmace_: kolla gen config also adds this for us, but I guess they forgot about it when added new feature01:48
kfox1111was something similar being done here?01:48
kfox1111sbezverk: oh. is this part of the gate being broken your debugging?01:49
sbezverk yeah, Branch 3 issue01:51
kfox1111ah.01:51
kfox1111so, we unpined on feb 5...01:52
kfox1111its feb 8... fantastic.01:52
kfox1111that didn't take long to break things. :/01:52
bmace_lol01:53
kfox1111looks like there were 39 ps's merged in that time.01:54
kfox1111I'd recommend doing a bisect in the gate.01:54
kfox1111unless someone watching kolla-ansible knows changes that may effect this.01:55
sbezverkyeeeesssss that was it01:57
kfox1111find it?01:58
sbezverkafter pinning to older version branch 3 turned to green01:58
kfox1111k. so then we just have to narrow down the ps then...01:59
*** adrian_otto has joined #openstack-kolla01:59
kfox1111a bisect will take upwards of about 5 runs.01:59
kfox1111if unlucky.01:59
kfox1111if you do 3 at a time instead of 1, it could be done in parallel and go faster.02:00
yingjunHi, i got a issue when running 'kolla-build': `jinja2.exceptions.UndefinedError: 'users' is undefined`, could anyone help?02:01
yingjunfull log: http://paste.openstack.org/show/598160/02:02
*** adrian_otto has quit IRC02:03
*** Pavo has joined #openstack-kolla02:05
*** david-lyle has joined #openstack-kolla02:06
openstackgerritSerguei Bezverkhi proposed openstack/kolla-ansible master: Fixing authentication url for kubernetes  https://review.openstack.org/43125002:07
sbezverkkfox1111:  https://review.openstack.org/43125002:07
*** Pavo has quit IRC02:07
sbezverkI think it should do it02:08
openstackgerritSerguei Bezverkhi proposed openstack/kolla-kubernetes master: Gate refactoring  https://review.openstack.org/42815002:08
*** dave-mccowan has quit IRC02:09
sbezverkkfox1111: bisect would be hard to do at the gate :( I will take a chance with this fix, because it is an issue for sure, then we will see..02:10
*** wirehead_ has quit IRC02:12
kfox1111k.02:12
kfox1111should be able to just change the git commit being checked out to each commimt a git bisect recommends.02:12
kfox1111yeah. that looks promising.02:13
*** wirehead_ has joined #openstack-kolla02:13
*** yingjun has quit IRC02:14
openstackgerritWei Cao proposed openstack/kolla master: Add dragonflow container  https://review.openstack.org/41819302:14
*** Jeffrey4l has quit IRC02:17
*** Jeffrey4l has joined #openstack-kolla02:20
*** salv-orlando has joined #openstack-kolla02:26
*** unicell has quit IRC02:26
*** salv-orlando has quit IRC02:30
openstackgerritSerguei Bezverkhi proposed openstack/kolla-kubernetes master: Gate refactoring  https://review.openstack.org/42815002:32
*** japestinho has joined #openstack-kolla02:34
clilesis there a document or something that describes how kolla handles policy.json?02:42
clilesI found the commit, nm02:44
sdakekfox1111 those breakages wont happen once we have cross repo gating02:50
*** duonghq__ has quit IRC02:57
*** openstackstatus has quit IRC02:58
*** denaitre has quit IRC02:59
*** leseb has quit IRC02:59
*** denaitre has joined #openstack-kolla03:01
*** sp_ has joined #openstack-kolla03:01
*** openstackstatus has joined #openstack-kolla03:01
*** ChanServ sets mode: +v openstackstatus03:01
*** saneax-_-|AFK is now known as saneax03:04
*** leseb has joined #openstack-kolla03:05
Jeffrey4lzhubingbing_, please review https://review.openstack.org/43063803:05
*** yuanying has quit IRC03:05
zhubingbing_done <Jeffrey4l03:06
*** yuanying has joined #openstack-kolla03:07
*** yuanying has quit IRC03:11
*** MasterOfBugs has quit IRC03:16
Jeffrey4lthanks.03:17
*** sudipto has joined #openstack-kolla03:18
*** sudipto_ has joined #openstack-kolla03:18
*** rwallner has joined #openstack-kolla03:20
*** rwallner has quit IRC03:25
openstackgerritJeffrey Zhang proposed openstack/kolla master: [TEST][MASTER][KOLLA] Testing openstack ubuntu testing repo  https://review.openstack.org/43128903:28
*** goldyfruit has joined #openstack-kolla03:30
*** saneax is now known as saneax-_-|AFK03:32
openstackgerritJeffrey Zhang proposed openstack/kolla-ansible master: Optimize reconfigure action for horizon  https://review.openstack.org/42271903:32
openstackgerritMerged openstack/kolla-ansible master: Optimize reconfiguration for aodh  https://review.openstack.org/42367003:33
sbezverkJeffrey4l: could you workflow it please?03:45
*** sdake has quit IRC03:45
sbezverkhttps://review.openstack.org/43063803:45
*** sayantani01 has joined #openstack-kolla03:45
Jeffrey4lsbezverk, done ;)03:45
sbezverkJeffrey4l: thanks a lot!!03:47
Jeffrey4lnp03:47
openstackgerritMerged openstack/kolla-ansible master: Fixing authentication url for kubernetes  https://review.openstack.org/43125003:48
*** jrobinson is now known as jrobinson-afk03:58
openstackgerritSerguei Bezverkhi proposed openstack/kolla-kubernetes master: Gate refactoring  https://review.openstack.org/42815004:02
*** crushil has quit IRC04:04
*** goldyfruit has quit IRC04:06
*** crushil has joined #openstack-kolla04:17
*** jrobinson-afk is now known as jrobinson04:28
*** adrian_otto has joined #openstack-kolla04:29
*** rwallner has joined #openstack-kolla04:31
openstackgerritSerguei Bezverkhi proposed openstack/kolla-kubernetes master: Gate refactoring  https://review.openstack.org/42815004:31
*** sdake has joined #openstack-kolla04:34
*** adrian_otto has quit IRC04:36
*** rwallner has quit IRC04:36
*** mdnadeem has joined #openstack-kolla04:39
openstackgerritJeffrey Zhang proposed openstack/kolla master: [TEST][MASTER][KOLLA] Testing openstack ubuntu testing repo  https://review.openstack.org/43128904:40
sdakebmace_ around?04:43
openstackgerritSteven Dake proposed openstack/kolla-kubernetes master: Change endpoint-test -1 to 1 return code  https://review.openstack.org/43112604:43
openstackgerritSteven Dake proposed openstack/kolla-kubernetes master: Document the magic in endpoint_test.sh  https://review.openstack.org/43111404:43
*** penfold has quit IRC04:44
*** yingjun has joined #openstack-kolla04:50
*** saneax-_-|AFK is now known as saneax04:51
openstackgerritSerguei Bezverkhi proposed openstack/kolla-kubernetes master: Gate refactoring  https://review.openstack.org/42815005:06
sbezverksdake: gte refactor ps is ready for review.. so your chance to unblock other folks from pushing their gate ps ;) https://review.openstack.org/42815005:07
sdakesbezverk i reviewed it05:08
sdakesbezverk it had a few nitpicky things - otherwise looks pretty good05:08
sdakesbezverk i could find no errors except the magic numbers and whatnot05:08
sbezverkhm I did not get email let me check gerrit05:09
sdakeya patch set 7705:10
sdakethere is a top pin on asnible 2.1 for example05:10
sdakewe should be able to run without top pins05:10
bmace_sdake: heading out in a minute.. going to post my latest changes.. stuff is coming up very nicely on the 2.x branch05:10
bmace_i think near the end of the nova / horizon stuff it started puking because my env was resource starved05:11
sdakebmace_ good t ohear05:11
sdakeminus the puking part :)05:11
sbezverksdake: had to pin ansible because it breaks our gate05:11
openstackgerritBorne Mace proposed openstack/kolla-kubernetes master: WIP: Added support for containerized dev bringup  https://review.openstack.org/42929505:12
*** yingjun has quit IRC05:12
bmace_sdake: feel free to try it out.. after you start up the container just run setup_dev_env.. it does all the ceph setup, then at the end i have a line with the syntax to be passed to ceph_workflow_service, and that should all be working also05:13
bmace_sdake: back in a couple of hours05:13
sdakebmace_ it will need a rebase as soon as sbezverk 'z patch merges05:13
sdakesbezverk how does it break the gate?05:14
sdakesbezverk thats a problem :(05:14
sdakesbezverk we need  to get to the bottom of that - at a later point05:15
sbezverksdake There was a change introduced in ansible which I thought I found05:15
sbezverkbut it tuned out no05:15
*** saneax is now known as saneax-_-|AFK05:16
sbezverkanyway I am not goint to block other folks while chasing the issue05:16
sbezverkit can be done in parallel05:16
sdakeright05:16
sdakei wsn't suggesting blocking05:16
sdakejust finding the cuse at some point05:16
sdakecan you log a bug for it05:17
sdakeit maybe a kolla-ansible bug in which csae we need it fixed soon by kolla-ansible deliverable05:17
sdakesince release is coming up shortly05:17
sdake(march 10th)05:17
*** sudipto has quit IRC05:18
*** sudipto_ has quit IRC05:18
openstackgerritSerguei Bezverkhi proposed openstack/kolla-kubernetes master: Gate refactoring  https://review.openstack.org/42815005:19
sbezverksdake: when gate refactor gets merged Iwill do it, right now gates and compute kit is a priority05:21
*** crushil has quit IRC05:21
sbezverkI am off, bye05:21
*** tvignaud has quit IRC05:24
*** sayantani01 has quit IRC05:30
*** unicell has joined #openstack-kolla05:32
openstackgerritzhubingbing proposed openstack/kolla master: Add fluent-plugin-parser pluin  https://review.openstack.org/43132305:33
*** krtaylor has quit IRC05:34
zhubingbing_sdake05:35
zhubingbing_https://review.openstack.org/#/c/431323/105:36
sdakesup zhubingbing_05:36
sdakesbezverk roger05:36
zhubingbing_please review it05:36
zhubingbing_;)05:36
sdakesbezverk evening to ya :)05:36
*** yingjun has joined #openstack-kolla05:36
*** sdake has quit IRC05:39
*** sdake has joined #openstack-kolla05:39
*** sudipto has joined #openstack-kolla05:40
*** sudipto_ has joined #openstack-kolla05:40
openstackgerritzhubingbing proposed openstack/kolla-ansible master: Fix fluentd config error  https://review.openstack.org/43132605:42
openstackgerritzhubingbing proposed openstack/kolla-ansible master: Fix fluentd config error  https://review.openstack.org/43132605:52
*** japestinho has quit IRC06:01
*** saneax-_-|AFK is now known as saneax06:02
*** yuanying has joined #openstack-kolla06:07
*** salv-orlando has joined #openstack-kolla06:09
*** sacharya_ has quit IRC06:11
*** imcsk8_ has joined #openstack-kolla06:12
*** imcsk8 has quit IRC06:12
*** ssurana has joined #openstack-kolla06:13
*** rwallner has joined #openstack-kolla06:17
*** rwallner has quit IRC06:21
*** rwallner has joined #openstack-kolla06:28
*** NachoDuck_ has joined #openstack-kolla06:29
*** ssurana has quit IRC06:29
*** portdirect__ has joined #openstack-kolla06:30
*** mgkwill_ has joined #openstack-kolla06:30
openstackgerritJeffrey Zhang proposed openstack/kolla-ansible master: use_forwarded_for is moved to api group  https://review.openstack.org/43133706:30
*** ediardo_ has joined #openstack-kolla06:30
*** ediardo has quit IRC06:33
*** portdirect has quit IRC06:33
*** denaitre has quit IRC06:33
*** huzhengchuan has quit IRC06:33
*** mgkwill has quit IRC06:33
*** NachoDuck has quit IRC06:33
*** cuongnv has quit IRC06:33
*** denaitre has joined #openstack-kolla06:33
*** portdirect__ is now known as portdirect06:33
*** rwallner has quit IRC06:33
*** ediardo_ is now known as ediardo06:33
*** mgkwill_ is now known as mgkwill06:33
*** NachoDuck_ is now known as NachoDuck06:34
*** huzhengchuan has joined #openstack-kolla06:36
openstackgerritDuong Ha-Quang proposed openstack/kolla-ansible master: Change Barbican default secret store to pkcs11  https://review.openstack.org/43013706:38
openstackgerritZeyu Zhu proposed openstack/kolla-kubernetes master: Fix the typo in ceph-guide.rst  https://review.openstack.org/43134106:38
*** inc0 has joined #openstack-kolla06:41
*** cuongnv has joined #openstack-kolla06:43
*** unicell1 has joined #openstack-kolla06:43
*** unicell has quit IRC06:43
openstackgerritzhubingbing proposed openstack/kolla-ansible master: Fix fluentd config error  https://review.openstack.org/43132606:48
*** inc0 has quit IRC06:51
*** sdake has quit IRC06:56
*** salv-orl_ has joined #openstack-kolla06:59
*** salv-orlando has quit IRC07:02
*** imcsk8_ has quit IRC07:04
*** imcsk8 has joined #openstack-kolla07:04
openstackgerritKris proposed openstack/kolla-ansible master: Wrong type of interface in mongodb bootstrap_cluster.js template  https://review.openstack.org/43134807:06
*** satyar has joined #openstack-kolla07:09
*** sayantani01 has joined #openstack-kolla07:10
duonghqzhubingbing_, do you deploy kolla-ansible recently?07:10
zhubingbing_yes07:11
duonghqdo you get trouble with rabbitmq?07:11
zhubingbing_what‘s problems?07:11
zhubingbing_no07:11
*** sacharya has joined #openstack-kolla07:12
duonghqzhubingbing_, http://paste.openstack.org/show/598180/07:13
duonghqit keep restart07:13
zhubingbing_http://paste.openstack.org/show/598181/07:14
duonghqI should same as your07:14
*** sacharya has quit IRC07:17
*** tovin07_ has joined #openstack-kolla07:22
duonghqJeffrey4l, are you there?07:23
*** salv-orl_ has quit IRC07:28
*** pcaruana has joined #openstack-kolla07:36
openstackgerritWei Cao proposed openstack/kolla-ansible master: Add Octavia doc link in README.rst  https://review.openstack.org/43135507:37
*** prameswar has joined #openstack-kolla07:39
*** Serlex has joined #openstack-kolla07:41
openstackgerritMerged openstack/kolla-ansible master: Fix telegraf task  https://review.openstack.org/43064807:46
*** unicell has joined #openstack-kolla07:48
*** unicell1 has quit IRC07:49
*** unicell has quit IRC07:55
*** jrobinson has quit IRC07:57
*** matrohon has joined #openstack-kolla08:00
*** sdake has joined #openstack-kolla08:05
openstackgerritzhubingbing proposed openstack/kolla-ansible master: test  https://review.openstack.org/43136508:09
openstackgerritMerged openstack/kolla master: Add fluent-plugin-parser pluin  https://review.openstack.org/43132308:20
*** dciabrin has joined #openstack-kolla08:21
*** sdake has quit IRC08:24
openstackgerritzhubingbing proposed openstack/kolla-ansible master: Add nova-serialproxy releasenotes  https://review.openstack.org/43137608:26
*** rwallner has joined #openstack-kolla08:31
*** matrohon has quit IRC08:35
*** rwallner has quit IRC08:36
*** sayantani01 has quit IRC08:37
openstackgerritzhubingbing proposed openstack/kolla-ansible master: Add placement-api releasenotes  https://review.openstack.org/43138308:37
*** sayantani01 has joined #openstack-kolla08:37
*** egonzalez has joined #openstack-kolla08:45
openstackgerritzhubingbing proposed openstack/kolla-ansible master: Add searchlight releasenotes  https://review.openstack.org/43138708:47
zhubingbing_sup egonzalez08:49
egonzalezhey zhubingbing_08:50
openstackgerritzhubingbing proposed openstack/kolla-ansible master: Add radosgw-keystone-integration releasenotes  https://review.openstack.org/43139008:56
*** openstackgerrit has quit IRC09:03
*** openstackgerrit has joined #openstack-kolla09:04
openstackgerritzhubingbing proposed openstack/kolla-ansible master: Add cinder-volume-hnas releasenotes  https://review.openstack.org/43139409:04
*** skuda has joined #openstack-kolla09:04
openstackgerritzhubingbing proposed openstack/kolla master: Add zun dockerfile releasenotes  https://review.openstack.org/43139609:11
openstackgerritzhubingbing proposed openstack/kolla master: Add zun dockerfile releasenotes  https://review.openstack.org/43139609:12
*** Jeffrey4l has quit IRC09:17
*** Jeffrey4l has joined #openstack-kolla09:17
*** gfidente has joined #openstack-kolla09:25
*** gfidente has quit IRC09:25
*** gfidente has joined #openstack-kolla09:25
*** mgoddard has joined #openstack-kolla09:31
*** yingjun has quit IRC09:33
openstackgerritzhubingbing proposed openstack/kolla-ansible master: Add miss releasenotes  https://review.openstack.org/43137609:36
*** sdake has joined #openstack-kolla09:37
openstackgerritzhubingbing proposed openstack/kolla-ansible master: Add miss kolla-ansible releasenotes  https://review.openstack.org/43137609:37
*** japestinho has joined #openstack-kolla09:40
*** japestinho has quit IRC09:41
*** japestinho has joined #openstack-kolla09:41
openstackgerritzhubingbing proposed openstack/kolla-ansible master: Add miss kolla-ansible releasenotes  https://review.openstack.org/43137609:42
*** hrw has quit IRC09:42
*** hrw has joined #openstack-kolla09:44
*** mannamne has joined #openstack-kolla09:49
*** duonghq has quit IRC09:51
*** wagiel has joined #openstack-kolla09:56
*** sdake has quit IRC09:57
*** dosaboy has joined #openstack-kolla09:58
*** dosaboy has quit IRC09:58
*** dosaboy has joined #openstack-kolla10:00
*** tovin07_ has quit IRC10:01
*** david-lyle has quit IRC10:02
*** sp_ has quit IRC10:05
*** klindgren_ has joined #openstack-kolla10:06
*** cuongnv has quit IRC10:07
*** sayantan_ has joined #openstack-kolla10:08
*** klindgren has quit IRC10:08
*** sayantani01 has quit IRC10:08
*** zioproto has joined #openstack-kolla10:13
*** proto_ has joined #openstack-kolla10:13
*** zioproto has quit IRC10:14
*** proto_ has quit IRC10:14
*** zioproto has joined #openstack-kolla10:14
openstackgerritzhubingbing proposed openstack/kolla master: Add kolla miss releasenotes  https://review.openstack.org/43141810:14
openstackgerritzhubingbing proposed openstack/kolla master: Add miss kolla releasenotes  https://review.openstack.org/43141810:15
*** sp_ has joined #openstack-kolla10:17
zioprotohello all, anyone knows when stable/ocata will be branched ?10:18
zioprotoI have this last patch that is importnat to me :) https://review.openstack.org/#/c/427139/10:18
openstackgerritzhubingbing proposed openstack/kolla-ansible master: Add miss kolla-ansible releasenotes  https://review.openstack.org/43137610:19
*** zhubingbing_ has quit IRC10:21
*** sambetts|afk is now known as sambetts10:27
pbourkeany of you deploy ironic recently?10:29
*** caowei has quit IRC10:29
*** matrohon has joined #openstack-kolla10:31
*** pcaruana has quit IRC10:32
*** pcaruana has joined #openstack-kolla10:33
*** satyar has quit IRC10:34
*** sayantani01 has joined #openstack-kolla10:35
*** sayantan_ has quit IRC10:35
*** yuanying has quit IRC10:36
*** wagiel has quit IRC10:42
n0isyn0isehi kolla team, I do not  understand why "kolla-ansible reconfigure" is not working after my deployment, is there any adjustment to do compared to the documentation ?10:45
n0isyn0isei'm using the master branch10:45
egonzalezn0isyn0ise: what is not working? what is failing?10:47
n0isyn0iseit seems kolla-ansible it just not looking in /etc/kolla/config10:49
*** zioproto has quit IRC10:50
n0isyn0isei there something more to do than enabling the node_custom_config: "/etc/kolla/config" flag ?10:50
n0isyn0iseand adding the custom conf to that folder (as mentionned in documentation) ?10:51
*** dciabrin has quit IRC10:51
egonzalezn0isyn0ise: which service are using custom config and in what path?10:51
*** sdake has joined #openstack-kolla10:52
n0isyn0iseas i did not get any result, I try with the same example as in documentation : I try to reconfigure nova with the "scheduler_max_attemps = 100"10:53
*** dciabrin has joined #openstack-kolla10:54
*** skramaja has quit IRC10:54
n0isyn0iseso I create /etc/kolla/config/nova/nova-scheduler.conf with the following insidde : [DEFAULT]10:54
n0isyn0isescheduler_max_attempts = 10010:54
n0isyn0iseenven if i found that quite strange compared to the config i can see in nova.conf on my compute nodes...10:55
n0isyn0isei think it would be something like : [scheduler] \ max_attemps = 10010:55
n0isyn0isein a /etc/kolla/config/nova-compute folder10:56
n0isyn0isei tried in multiple ways but none of them worked, so I guess i should do something wrong, but don't know what...10:57
egonzalezpaths are /etc/kolla/config/nova/nova-*.conf or /etc/kolla/config/nova.conf (this to apply changes to all nova services) or /etc/kolla/config/nova/<hostname>/nova.conf10:58
egonzalezn0isyn0ise: yeah, need the section to merge the config10:58
n0isyn0isek thx, i will try anew11:01
n0isyn0isewith /etc/kolla/config/nova.conf it worked !11:08
n0isyn0isethx egonzalez ! :)11:08
*** sdake has quit IRC11:11
*** qiliang27 has quit IRC11:20
*** qiliang27 has joined #openstack-kolla11:22
*** zhuzeyu has joined #openstack-kolla11:23
*** sudipto has quit IRC11:27
*** sudipto_ has quit IRC11:27
*** dgonzalez has quit IRC11:30
*** prameswar has quit IRC11:44
*** zhuzeyu has quit IRC11:44
*** yingjun has joined #openstack-kolla11:48
japestinhohello everyone, is there any installaion note regarding to kolla-kubernetes?11:49
*** dgonzalez has joined #openstack-kolla11:51
*** kwazar has quit IRC11:52
*** zhubingbing has joined #openstack-kolla11:52
*** fragatina has quit IRC12:00
*** cargonza has joined #openstack-kolla12:01
*** skramaja has joined #openstack-kolla12:01
*** tvignaud has joined #openstack-kolla12:03
*** pbourke has quit IRC12:07
*** pbourke has joined #openstack-kolla12:07
*** prameswar has joined #openstack-kolla12:08
*** dave-mccowan has joined #openstack-kolla12:13
openstackgerritMerged openstack/kolla-ansible master: Fix fluentd config error  https://review.openstack.org/43132612:14
openstackgerritMerged openstack/kolla-ansible master: Fix copying over nfs_shares task failure  https://review.openstack.org/43063812:15
openstackgerritMerged openstack/kolla-ansible master: Custom init-runonce values  https://review.openstack.org/42468212:16
*** zhangqiankun has quit IRC12:20
*** satyar has joined #openstack-kolla12:30
*** magicboiz has joined #openstack-kolla12:35
*** salv-orlando has joined #openstack-kolla12:35
des_consoladogah how the hell do I troubleshoot this ironic_dnsmasq containers then D:12:43
*** kwazar has joined #openstack-kolla12:44
des_consoladoI'm tcpdumping on the interface it's supposed to be using. I can see the DHCP requests coming in, broadcast packets12:44
des_consoladothe dnsmasq process is running and it just doesn't do anything. I've straced it from the host and it does nothing. It is running though. The container is running fine.12:45
des_consoladoin the logs dir I only see logs from ironic-api, -conductor and -inspector12:45
des_consoladoso there aren't any dnsmasq logs12:45
*** tvignaud has quit IRC12:46
openstackgerritEduardo Gonzalez proposed openstack/kolla-ansible master: WIP: Add cli options to cleanup-images  https://review.openstack.org/43150312:50
openstackgerritEduardo Gonzalez proposed openstack/kolla-ansible master: WIP: Add cli options to cleanup-images  https://review.openstack.org/43150312:51
*** rhallisey has joined #openstack-kolla12:53
*** salv-orl_ has joined #openstack-kolla12:59
openstackgerritJeffrey Zhang proposed openstack/kolla-ansible master: Run simple_cell_setup after the new container is up during upgrade  https://review.openstack.org/43150813:00
*** eaguilar has quit IRC13:00
Jeffrey4legonzalez, could u test https://review.openstack.org/431508 ? it works in my env during upgrade newton to ocata.13:01
*** salv-orlando has quit IRC13:02
*** rwallner has joined #openstack-kolla13:02
*** rwallner has quit IRC13:03
*** Pavo has joined #openstack-kolla13:03
*** rwallner has joined #openstack-kolla13:03
hrwspeaking of reviews... https://review.openstack.org/#/c/430940 (non-x86 archs support) waits for skilled reviewers ;D13:05
egonzalezJeffrey4l: tried before and it works13:07
egonzalezI'm going to retry13:07
Jeffrey4legonzalez, thanks ;)13:07
*** zioproto has joined #openstack-kolla13:07
*** sudipto has joined #openstack-kolla13:11
*** sudipto_ has joined #openstack-kolla13:11
*** eaguilar has joined #openstack-kolla13:16
*** eaguilar has quit IRC13:17
japestinhoI got error when running kolla-ansible deploy, while starting heka container I got error "maximum retry count cannot be used with restart policy"13:19
japestinhowhat's wrong with that error?13:19
Jeffrey4ljapestinho, kolla do not support docker-engine 1.13 now. try to downgrade to 1.12.x13:20
*** zioproto has quit IRC13:20
japestinhoah thank you Jeffrey4l I'll try to downgrade docker version on my node13:21
Jeffrey4lnp13:22
*** penfold has joined #openstack-kolla13:22
*** huzhengchuan has quit IRC13:33
*** tvignaud has joined #openstack-kolla13:37
*** mdnadeem has quit IRC13:40
openstackgerritSerguei Bezverkhi proposed openstack/kolla-kubernetes master: Adding compute kit based on iscsi backend  https://review.openstack.org/43115613:43
openstackgerritSerguei Bezverkhi proposed openstack/kolla-kubernetes master: Adding compute kit based on iscsi backend  https://review.openstack.org/43115613:47
*** mgiles has joined #openstack-kolla13:49
*** rwallner has quit IRC13:50
*** schwicht has joined #openstack-kolla13:51
*** yingjun has quit IRC13:52
egonzalezzhubingbing: around?13:54
*** eaguilar has joined #openstack-kolla13:58
*** schwicht has quit IRC14:02
*** rwallner has joined #openstack-kolla14:03
*** schwicht has joined #openstack-kolla14:03
egonzalezSomeone have faced this error in nova_compute container. stable/newton with ceph enabled14:04
egonzalezOSError: [Errno 2] No such file or directory: '/etc/ceph/rbdmap'14:04
egonzalezJeffrey4l: ValidationError: Cell mappings are not created, but required for Ocata. Please run nova-manage cell_v2 simple_cell_setup before continuing.14:09
*** Pavo has quit IRC14:10
*** saneax is now known as saneax-_-|AFK14:12
*** Pavo has joined #openstack-kolla14:14
*** goldyfruit has joined #openstack-kolla14:15
*** sp__ has joined #openstack-kolla14:15
*** sayantani01 has quit IRC14:17
*** Pavo has quit IRC14:19
*** schwicht has quit IRC14:24
*** lamt has joined #openstack-kolla14:25
*** krtaylor has joined #openstack-kolla14:26
*** lamt has quit IRC14:26
Jeffrey4legonzalez, which task raise this?14:29
egonzalezbootstrap nova14:30
Jeffrey4lmy test is fine. hrm.14:30
Jeffrey4legonzalez, have u build latest nova-api container?14:30
egonzalezused latest registry14:31
egonzalezso, yep14:31
*** eaguilar has quit IRC14:31
Jeffrey4llatest registry? means? hub.docker.com?14:31
egonzaleztarballs.o.o14:31
Jeffrey4legonzalez, where the newton registry come from?14:32
Jeffrey4lnewton image.14:32
egonzaleznewton is from dockerhub14:32
Jeffrey4lok.14:32
Jeffrey4legonzalez, could u post the traceback from bootstrap_nova?14:33
egonzalezJeffrey4l: http://paste.openstack.org/show/598247/14:33
Jeffrey4land you told me this patch should work, right?14:34
egonzalezit worked iirc, but now i doubt of it14:34
*** yingjun has joined #openstack-kolla14:36
egonzalezhrm, I tested with centos binary , that code was not packaged yet, in source it is14:40
Jeffrey4lok.14:42
egonzalezshould be, was implemented in dec14:42
Jeffrey4lone issue is: the default database name is nova_api_cell0 in newton and nova_cell0 in ocata.14:43
* egonzalez is getting crazy14:43
Jeffrey4llet me try again.14:43
egonzalezredeploying again too14:43
Jeffrey4lif my patch do not works. another solution is: create the proper cell item by using "create_cell" and "map_cell0" and discover_hosts14:44
egonzalezcreate-cell command is only in ocata14:44
Jeffrey4l;(14:44
egonzalezJeffrey4l: have you ever see this error: OSError: [Errno 2] No such file or directory: '/etc/ceph/rbdmap'14:47
egonzalezstable/newton centos-binary14:47
Jeffrey4legonzalez, saw this long time ago.14:47
*** crushil has joined #openstack-kolla14:47
egonzaleznova_compute and cinder_volume is restarting14:47
Jeffrey4li never tried centos-binary.14:48
Jeffrey4lcentos+source works fine.14:48
egonzalezJeffrey4l: don't know if my boss will like non binary deployments ;)14:48
Jeffrey4lah.  source is more stable, imo. ;)14:49
egonzalezyeah, no doubt of it14:49
Jeffrey4l /etc/ceph/rbdmap is provided by ceph-common.14:50
*** sdake has joined #openstack-kolla14:51
Jeffrey4lno one will remove this file.14:51
Jeffrey4lit is wired.14:51
*** sp__ has quit IRC14:51
openstackgerritSerguei Bezverkhi proposed openstack/kolla-kubernetes master: Gate refactoring  https://review.openstack.org/42815014:52
*** ipsecguy has quit IRC14:52
openstackgerritSerguei Bezverkhi proposed openstack/kolla-kubernetes master: Adding compute kit based on iscsi backend  https://review.openstack.org/43115614:53
*** sdake_ has joined #openstack-kolla14:54
Jeffrey4legonzalez, found this http://git.openstack.org/cgit/openstack-dev/grenade/tree/projects/60_nova/from-newton/upgrade-nova#n914:54
*** sdake has quit IRC14:55
*** schwicht has joined #openstack-kolla14:55
egonzalezJeffrey4l: cool, seems a good option14:56
Jeffrey4lyes. i will add it into my patch14:58
Jeffrey4li didn't know openstack has https://github.com/openstack-dev/grenade such a project.14:58
egonzalezI tried something similar, instead of simple_cell_setup after map_cell0 made map_cell_and_hosts, but complains of missing db tables since after map_cell0 a db_sync must be made15:01
Jeffrey4lnova should add this into its own upgrade releasenote.15:02
openstackgerritRonan-Alexandre Cherrueau proposed openstack/kolla master: Multi-Regions Support  https://review.openstack.org/43158815:04
egonzalezJeffrey4l: yeah, they throw the bomb and forgot about people who uses it15:04
*** Pavo has joined #openstack-kolla15:06
*** Pavo has quit IRC15:07
*** fguillot has joined #openstack-kolla15:12
*** adrian_otto has joined #openstack-kolla15:18
egonzalezJeffrey4l: still having same issue centos-source 3.0.2 with ceph15:20
Jeffrey4legonzalez, try run a nova-compute container manually, and check whether the file exist.15:20
egonzalezJeffrey4l: yep, it is15:20
egonzalezowned by root15:21
Jeffrey4ldo the nova user can read it?15:21
*** sayantani01 has joined #openstack-kolla15:22
*** sayantani01 has quit IRC15:26
*** saneax-_-|AFK is now known as saneax15:27
*** jtriley has joined #openstack-kolla15:29
*** sacharya has joined #openstack-kolla15:33
*** eaguilar has joined #openstack-kolla15:35
*** adrian_otto has quit IRC15:37
*** vcn has joined #openstack-kolla15:37
egonzalezJeffrey4l: set_config remove dest path if exists, rbdmap is in that path and removed15:43
*** yingjun has quit IRC15:43
openstackgerritMerged openstack/kolla master: Move fluentd td.repo to base for consistency  https://review.openstack.org/43019315:43
Jeffrey4legonzalez, ok. that's a bug in set_config..15:43
egonzalezINFO:__main__:Removing existing destination: /etc/ceph/15:43
Jeffrey4lbut why source is OK?15:43
egonzalezJeffrey4l: nope, deployed source (stable/newton 3.0.2)15:44
Jeffrey4lok. i thought you are deploying master code.15:44
egonzalezwas that fixed?15:45
Jeffrey4li refactored set_config in master.15:45
Jeffrey4li guess it is OK in master.15:45
Jeffrey4lbut i have a newton env and master env. which both work fine.15:45
Jeffrey4lbut they are all centos+source.15:45
Jeffrey4lno idea whether it is matter.15:45
Jeffrey4legonzalez, may need fix this in newton.15:46
*** sayantani01 has joined #openstack-kolla15:46
egonzalezonly have this issue when enabled_ceph, I had no other issue but this15:47
Jeffrey4lmy env are both run with enable_ceph=yes. ..15:47
egonzalezusing kolla from pip in newton or git?15:48
Jeffrey4lgit/15:48
egonzalezguess i have to at least build my images instead of pull from dockerhub15:50
*** zhubingbing has quit IRC15:50
Jeffrey4legonzalez, not necessary. seem it is a bug.15:51
Jeffrey4legonzalez, it is caused by https://github.com/openstack/kolla/blob/ab364066e7ae4c87f1be53a6d7c13f9042032bb7/docker/base/set_configs.py#L7215:52
Jeffrey4land https://github.com/openstack/kolla/blob/ab364066e7ae4c87f1be53a6d7c13f9042032bb7/ansible/roles/nova/templates/nova-compute.json.j2#L1215:53
Jeffrey4lif dest is a diretory, shoudn't remove the folder.15:53
*** prameswar has quit IRC15:53
*** rwallner_ has joined #openstack-kolla15:55
*** rwallner has quit IRC15:55
*** adrian_otto has joined #openstack-kolla16:00
openstackgerritPaul Bourke (pbourke) proposed openstack/kolla master: Allow the use of previously built parent images  https://review.openstack.org/42632216:01
*** zhubingbing has joined #openstack-kolla16:03
*** adrian_otto1 has joined #openstack-kolla16:05
*** adrian_otto has quit IRC16:06
*** erlon has joined #openstack-kolla16:06
*** eaguilar has quit IRC16:10
*** saneax is now known as saneax-_-|AFK16:11
*** eaguilar has joined #openstack-kolla16:11
*** sacharya has quit IRC16:14
*** sacharya has joined #openstack-kolla16:17
*** vcn has quit IRC16:18
*** sp__ has joined #openstack-kolla16:21
*** fragatina has joined #openstack-kolla16:22
*** adrian_otto1 has quit IRC16:26
*** sacharya_ has joined #openstack-kolla16:27
*** sacharya has quit IRC16:27
*** adrian_otto has joined #openstack-kolla16:29
*** srwilkers has joined #openstack-kolla16:38
*** sayantani01 has quit IRC16:39
*** sayantani01 has joined #openstack-kolla16:39
*** david-lyle has joined #openstack-kolla16:40
*** sacharya has joined #openstack-kolla16:40
*** sacharya_ has quit IRC16:41
*** inc0 has joined #openstack-kolla16:43
inc0o/16:43
*** zhubingbing has quit IRC16:44
srwilkerso/16:51
sdake_morning peeps16:52
*** sp__ has quit IRC16:53
*** penfold has quit IRC16:54
*** penfold has joined #openstack-kolla16:54
openstackgerritMerged openstack/kolla-kubernetes master: Document the magic in endpoint_test.sh  https://review.openstack.org/43111416:56
openstackgerritMerged openstack/kolla-kubernetes master: Change endpoint-test -1 to 1 return code  https://review.openstack.org/43112616:57
sdake_yay ;-)16:58
*** matrohon has quit IRC16:59
*** david-lyle has quit IRC17:00
*** Pavo has joined #openstack-kolla17:01
*** skramaja has quit IRC17:02
*** sacharya has quit IRC17:07
*** sacharya has joined #openstack-kolla17:08
*** Pavo has quit IRC17:10
*** egonzalez has quit IRC17:12
*** Pavo has joined #openstack-kolla17:12
*** sacharya has quit IRC17:12
*** sacharya has joined #openstack-kolla17:14
sdake_is the kolla-kubernetes gate busted?17:15
sbezverksdake_: srwilkers: gate refactor is ready to merge. https://review.openstack.org/#/c/428150/7817:15
sdake_sbezverk reivewing17:15
sdake_sbezverk i ntoiced the endpoint patch merged with all kinds of gate failures17:15
sbezverksdake_: just branch 3 is busted17:16
sdake_sbezverk i just wanted to verify i didn't break the gate17:16
sbezverkthe rest should work just fine17:16
sdake_nice17:16
*** lamt has joined #openstack-kolla17:18
*** sacharya_ has joined #openstack-kolla17:18
*** sacharya has quit IRC17:19
openstackgerritSerguei Bezverkhi proposed openstack/kolla-kubernetes master: Adding compute kit based on iscsi backend  https://review.openstack.org/43115617:20
sdake_sbezverk gate refactor looks fantastically good17:21
*** sacharya has joined #openstack-kolla17:21
sdake_sbezverk i'm sure it will create some pain for the other rebasrs, but might as well get it done :)17:21
sdake_inc0 portdirect srwilkers if you guys want to have a look - that gate refactor is blocking other work https://review.openstack.org/#/c/428150/7817:22
*** sacharya_ has quit IRC17:23
klindgren_Morning - question for you guys.  With kolla and building containers from source.  What is the recommended way to also get kolla to install specific/patched dependent packages.17:24
klindgren_Like say we want it to include a patched version of oslo_messaging17:24
klindgren_akaik, creating a nova container using source, basically jsut does a pip install into a venv.  So is the recomended path to modify requirements.txt for nova to either install the specific version that you want, and run your own pip mirror where you publish that version to?  Or replace oslo_messaging entry withe an egg include to have pip fetch it from source?17:26
*** crushil has quit IRC17:26
klindgren_or is their some generic way in kolla to do that, which I am not aware of?17:27
*** eaguilar has quit IRC17:27
SamYapleklindgren_: there are jinja2 macros that you can take advantage of17:28
openstackgerritSayantani Goswami proposed openstack/kolla-ansible master: WIP Deployment Guide  https://review.openstack.org/42796517:28
SamYaplefor a more native approach, you could for nova, oslo_messaging, and openstack-requirements and point kolla to those sources17:28
SamYapleeach has thier upside17:29
*** japestinho has quit IRC17:29
*** pcaruana has quit IRC17:30
*** sacharya_ has joined #openstack-kolla17:30
*** eaguilar has joined #openstack-kolla17:30
*** sacharya has quit IRC17:30
*** v1k0d3n has quit IRC17:31
*** bmace_ has quit IRC17:32
*** bmace_ has joined #openstack-kolla17:32
klindgren_SamYample what jinja2 macro?  would this be the "customizable("packages")) function?17:35
SamYapleklindgren_: there are a few macros sprinkled throughout17:36
*** sudipto has quit IRC17:39
*** sudipto_ has quit IRC17:39
*** sacharya_ has quit IRC17:39
*** sacharya has joined #openstack-kolla17:40
*** crushil has joined #openstack-kolla17:41
openstackgerritRonan-Alexandre Cherrueau proposed openstack/kolla-ansible master: Multi-regions Support  https://review.openstack.org/43165817:42
*** Administrator__ has joined #openstack-kolla17:43
*** sacharya has quit IRC17:44
*** sacharya has joined #openstack-kolla17:46
*** zhugaoxiao has quit IRC17:46
*** rwallner_ has quit IRC17:46
*** david-lyle has joined #openstack-kolla17:53
*** mgoddard has quit IRC17:55
*** penfold has quit IRC17:58
*** srwilkers has quit IRC18:00
*** sacharya has quit IRC18:00
*** sacharya has joined #openstack-kolla18:00
*** sambetts is now known as sambetts|afk18:02
openstackgerritSerguei Bezverkhi proposed openstack/kolla-kubernetes master: Adding compute kit based on iscsi backend  https://review.openstack.org/43115618:05
kfox1111morning18:07
openstackgerritSayantani Goswami proposed openstack/kolla-ansible master: Deployment Guide  https://review.openstack.org/42796518:09
bmace_morning :)18:13
kfox1111sbezverk: reviewing18:14
*** zhugaoxiao has joined #openstack-kolla18:15
sbezverkkfox1111: good morning18:16
*** unicell has joined #openstack-kolla18:17
sbezverk kfox1111: when you finish, please check compute_kit ps specifically building keystone admin rc file bit, I would like to know what you think about this problem18:17
*** rwallner has joined #openstack-kolla18:17
*** Administrator__ has quit IRC18:18
*** ssurana has joined #openstack-kolla18:19
*** mannamne has quit IRC18:19
kfox1111k18:25
*** vhosakot has joined #openstack-kolla18:25
kfox1111arg...18:26
bmace_vararg ...  ;)18:27
kfox1111should have put in a block before I reviewed...18:27
kfox1111just submitted the review quick to try and block.18:27
sbezverkkfox1111: so what should I do then?18:28
*** penfold has joined #openstack-kolla18:28
sbezverkIt is probably gets merged as is18:29
kfox1111lets see what happens.18:29
kfox1111if it merges, we can followup patch a few things.18:29
kfox1111it was looking really good really.18:29
inc0vararg sounds like some sort of overgrown wolf18:30
sbezverkkfox1111: thanks. meanwhile could you check compute kit piece?18:30
kfox1111yeah. sec18:30
kfox1111have the link handy?18:31
sbezverkkfox1111:  https://review.openstack.org/43115618:31
kfox1111nm. found it.18:31
kfox1111thanks. :)18:31
sbezverkkfox1111: about your suggestion to have 2 --values: I am not sure how it is going to work wrt to merging18:32
kfox1111it merges itself.18:33
kfox1111two values chunks works ok last I tried it.18:33
sbezverkkfox1111: what if both chunks have common entries?18:33
kfox1111it merges them.18:33
kfox1111it uses the same merger that the packages values.yaml and the specified values.yaml files all use.18:33
*** rwallner has quit IRC18:38
*** rwallner has joined #openstack-kolla18:38
*** sdake_ has quit IRC18:38
kfox1111sbezverk: so, what is the concern with it? why isn't keystone-admin set with an external ip?18:41
kfox1111its alittle hard to follow, as part of it is in the other ps,18:45
*** david-lyle has quit IRC18:45
kfox1111but it looks like its sharing all the config with the iscsi stuff?18:45
*** mgoddard has joined #openstack-kolla18:47
sbezverkkfox1111: when I deploy compute kit I cannot get to keystone even  thought keystone admin has public external18:47
sbezverkbecause kestone admin rc gets always cluster IP18:48
kfox1111should get a cluster ip too, in addition to the external ip...18:49
kfox1111hmm...18:49
sbezverkkfox1111: when it prints OS vars18:49
sbezverkit uses keystone internal svc address18:50
kfox1111yeah. admin has both: http://logs.openstack.org/56/431156/7/experimental/gate-kolla-kubernetes-deploy-centos-binary-2-helm-compute-kit-nv/88076fa/logs/svc/kolla-keystone-admin.txt18:50
sbezverkwhich is not accessible from outside18:50
kfox1111hmm... lets back up a bit.18:50
kfox1111can you please explain what the issue is?18:50
bmace_that is why i changed my code that generates the keystone admin rc file to use the keystone-admin external ip / port :)18:50
openstackgerritSerguei Bezverkhi proposed openstack/kolla-kubernetes master: Adding compute kit based on iscsi backend  https://review.openstack.org/43115618:51
*** sayantani01 has quit IRC18:51
kfox1111ah. keystone public doesn't have an external address anymore.18:51
sbezverkafter deploying compute kit, openstack volume ist fails18:51
*** sayantani01 has joined #openstack-kolla18:51
kfox1111so, yeah, keystone-public shoudl have an external ip in the gate.18:52
sbezverkkfox1111: I added it into common workflow18:52
kfox1111how is this wokring with other gate jobs though....18:52
sbezverkkfox1111: I think other jobs are getting it from all_values.yaml18:53
kfox1111but then compute kit should too shouldn't it?18:54
*** jmccarthy has left #openstack-kolla18:54
sbezverkkfox1111: not sure why it does not get it. have meeting in 5 minutes for 1 hour after that I will apply changes from your review.. bbiab18:55
kfox1111k.18:55
*** krtaylor has quit IRC18:57
*** salv-orlando has joined #openstack-kolla19:00
*** fguillot has quit IRC19:00
kfox1111sbezverk: yeah, keystone-public-svc has global.keystone.all.port_external=true19:01
*** salv-orl_ has quit IRC19:02
*** mgoddard has quit IRC19:06
*** markus-k has quit IRC19:12
*** markus-k has joined #openstack-kolla19:14
*** david-lyle has joined #openstack-kolla19:15
*** MasterOfBugs has joined #openstack-kolla19:16
*** tvignaud has quit IRC19:16
*** tvignaud has joined #openstack-kolla19:17
harlowjaanyone seen the following issue https://gist.github.com/harlowja/8521b90a06429cc8f903103d593ca1c8 ?19:31
*** sdake has joined #openstack-kolla19:31
*** david-lyle has quit IRC19:31
harlowjapretty sure its related to setuptools borkage, but just wanted to see if i'm alone here19:31
*** skuda has quit IRC19:32
harlowjabasically all builds are messed up i think due to that19:32
*** fragatina has quit IRC19:33
*** fragatina has joined #openstack-kolla19:33
inc0works for me (standard operators answer #1)19:34
*** david-lyle has joined #openstack-kolla19:34
*** v1k0d3n has joined #openstack-kolla19:36
*** eaguilar has quit IRC19:38
harlowjahmmm19:39
harlowjawhat version of setuptools gets installed into yours19:39
*** david-lyle has quit IRC19:40
*** david-lyle has joined #openstack-kolla19:42
inc0harlowja, 34.0.219:43
*** tonanhngo has joined #openstack-kolla19:43
harlowjak, hmmm, i'll check what i get19:43
*** mgoddard has joined #openstack-kolla19:46
*** srwilkers has joined #openstack-kolla19:47
srwilkerso/19:47
openstackgerritMerged openstack/kolla master: Add miss kolla releasenotes  https://review.openstack.org/43141819:50
sdakehey peeps - have to get the car ashed19:52
sdakewashed19:52
sdakebbl19:52
*** inc0 has quit IRC19:54
*** ubuntu has joined #openstack-kolla19:55
*** ubuntu is now known as Guest7236019:55
*** Jeffrey4l has quit IRC19:57
*** Jeffrey4l has joined #openstack-kolla19:58
*** krtaylor has joined #openstack-kolla19:59
*** Guest72360 has quit IRC19:59
*** ubuntu1 has joined #openstack-kolla20:01
hrwif someone has some free time then https://review.openstack.org/#/c/430940 (non-x86 archs support) waits for skilled reviewers. Small patch, does not change much for usual targets, adds two important architectures into list of supported ones and makes it easier for those which may come20:02
harlowjahttps://github.com/pypa/setuptools/issues/937 inc :-/20:02
harlowjanot cool setuptools20:03
harlowjalol20:03
harlowjaweird that it happened though, especially since i think kolla installs pip20:04
*** ubuntu1 has quit IRC20:05
*** david-lyle has quit IRC20:06
*** inc0 has joined #openstack-kolla20:06
*** matrohon has joined #openstack-kolla20:06
*** fguillot has joined #openstack-kolla20:09
*** sayantani01 has quit IRC20:09
*** gfidente is now known as gfidente|afk20:10
*** salv-orlando has quit IRC20:10
*** sdake has quit IRC20:14
*** vhosakot_ has joined #openstack-kolla20:16
*** skuda has joined #openstack-kolla20:16
*** tonanhngo has quit IRC20:17
*** vhosakot has quit IRC20:19
*** tonanhngo has joined #openstack-kolla20:20
-openstackstatus- NOTICE: Restarting gerrit due to performance problems20:20
hrwharlowja: you want setuptools 33.1 - 34 has issues20:20
harlowjaya, i wonder how 34 worked for inc020:21
harlowjagonna try unfreezing the packaging requirement in upper-constraints20:21
harlowjawant to see if thats causing this20:21
*** tonanhngo_ has joined #openstack-kolla20:21
inc0well don't forget that setuptools will get installed before pip hence before upper-constrains20:21
*** Serlex has left #openstack-kolla20:22
hrwharlowja: update pip to 9.0.1 first and then upgrade setuptools?20:23
harlowjawell kolla already installs pip from the latest20:23
harlowjahttps://github.com/openstack/kolla/blob/master/docker/openstack-base/Dockerfile.j2#L236-L24120:24
*** tonanhngo has quit IRC20:24
hrwlet me dig a moment20:24
*** mannamne has joined #openstack-kolla20:25
hrwharlowja: pip install --upgrade six appdirs packaging;pip install --upgrade setuptools - maybe that way20:26
*** tonanhngo_ has quit IRC20:26
harlowjaya, maybe20:26
hrwI cursed a lot during creation of virtualenv for kolla20:26
*** crushil has quit IRC20:28
*** Pavo has quit IRC20:28
*** skuda has quit IRC20:31
*** vhosakot_ has quit IRC20:33
*** schwicht has quit IRC20:33
*** vhosakot has joined #openstack-kolla20:34
*** harlowja_ has joined #openstack-kolla20:35
*** crushil has joined #openstack-kolla20:35
*** tonanhngo has joined #openstack-kolla20:35
*** lamt has quit IRC20:36
*** adrian_otto has quit IRC20:36
*** harlowja has quit IRC20:37
*** satyar has quit IRC20:38
*** tvignaud has quit IRC20:39
*** lamt has joined #openstack-kolla20:39
*** jtriley has quit IRC20:42
sbezverkkfox1111: ping20:46
kfox1111hey20:46
*** mgoddard has quit IRC20:48
sbezverkkfox1111: I am a bit worried about the change you suggested. if i make common workflow values and ceph/iscsi values independent then I cannot combine then into a single script which mean ALL even microservices need to be changed to use 2 x --values20:48
sbezverkit becomes even bigger change :(20:48
*** jrobinson has joined #openstack-kolla20:48
sbezverkwhat if I add comments to common_workflow and ceph/iscsi explaining how it should be maintainded?20:49
kfox1111sbezverk: yeah. thats what I was thinking too. but it should be much less volnerable to breakages I think.20:50
kfox1111ok. how about this. we do that second step in a followup ps?20:50
kfox1111easier to review that way. but fixes the issue quickly20:50
sbezverkmy concent is with these very key changes I might end troubleshooting more time and sdake will start crtiticizing me again ;)20:50
sbezverkkfox1111: for sure20:51
kfox1111yeah. I get that.20:51
kfox1111ok. lets go with taht plan.20:51
sbezverkI will implement 3 others changes you suggested right after20:51
kfox1111k.20:52
sbezverkkfox1111: thanks a lot20:52
*** srwilkers has quit IRC20:52
kfox1111np.20:52
*** sdake has joined #openstack-kolla21:02
*** eaguilar has joined #openstack-kolla21:03
*** sdake_ has joined #openstack-kolla21:07
*** sdake has quit IRC21:07
sbezverkkfox1111: can you remove the block then please, so we could move forward21:07
kfox1111k21:08
harlowja_hrw so ya, updating the packaging constraint fixed it21:09
harlowja_https://gist.github.com/harlowja/c15e75c84ed1626a1a78f115edb7a92e basically21:09
harlowja_updating/removing the constraint21:09
openstackgerritMerged openstack/kolla-kubernetes master: Gate refactoring  https://review.openstack.org/42815021:12
*** sdake_ has quit IRC21:14
sbezverkkfox1111: starting working on part 2 of gate refactoring. :)21:15
kfox1111cool. thanks. :)21:15
*** gfidente|afk has quit IRC21:15
bmace_after i finish this pass run, and it is looking good, may wait until all gate refactoring is done before dealing w/ merge issues, etc.21:19
bmace_for devenv21:20
*** fragatina has quit IRC21:20
*** srwilkers has joined #openstack-kolla21:20
sbezverkbmace_: next step is just few minor changes, so please go ahead, nothing major will change, all files are in now..21:22
*** tvignaud has joined #openstack-kolla21:23
sbezverkbmace_: I will do just several beautifications ;)21:23
bmace_sbezverk: thanks, good to hear it!21:23
*** nihilifer has quit IRC21:30
*** nihilifer has joined #openstack-kolla21:31
*** sdake has joined #openstack-kolla21:31
*** Jeffrey4l_ has joined #openstack-kolla21:35
*** Jeffrey4l has quit IRC21:35
*** krtaylor has quit IRC21:41
*** matrohon has quit IRC21:44
*** adrian_otto has joined #openstack-kolla21:45
*** skuda has joined #openstack-kolla21:45
*** penfold has quit IRC21:51
*** penfold has joined #openstack-kolla21:52
*** david-lyle has joined #openstack-kolla21:53
bmace_well, success.. the pre-merge end to end dev env came up with at least all services running.. going to commit them as is and then work on the merge.21:56
*** penfold has quit IRC21:57
*** harlowja_ has quit IRC21:58
bmace_well, so much for that, won't let you do the review if there is a known merge conflict.. damn smart gerrit! ;)21:58
*** salv-orlando has joined #openstack-kolla21:59
*** salv-orlando has quit IRC22:00
*** salv-orlando has joined #openstack-kolla22:01
*** david-lyle has quit IRC22:03
*** david-lyle has joined #openstack-kolla22:04
*** harlowja has joined #openstack-kolla22:05
inc0back22:07
*** fragatina has joined #openstack-kolla22:09
*** mgoddard has joined #openstack-kolla22:10
*** schwicht has joined #openstack-kolla22:12
kfox1111bmace_: great news. :)22:12
kfox1111bmace_: darn. that sucks.22:12
inc0kfox1111: you're using weechat right?22:12
kfox1111irssi22:12
bmace_kfox1111: meh, had to be done either way.. at least this way folks can start to use it right away.. well, can't merge until after the kubernetes python lib is accepted too, so still may be a few days until it can go in22:12
inc0aw...ok...22:13
*** fguillot has quit IRC22:19
*** jrobinson is now known as jrobinson-afk22:19
*** david-lyle has quit IRC22:20
*** fragatina has quit IRC22:21
*** sdake has quit IRC22:21
*** david-lyle has joined #openstack-kolla22:28
*** crushil has quit IRC22:33
*** sdake has joined #openstack-kolla22:36
sdakebmace_ work on merge?22:38
bmace_sdake: working on merge.  ran end to end flawlessly pre-merge with an 8gb master node.22:41
sdakeyou mean pre-merge of the gate  rework?22:41
*** vhosakot has quit IRC22:41
bmace_sdake: yes, need to merge in the gate changes now.22:41
sdakewhat happens with a 4gb master and 4gb minions?22:41
*** vhosakot has joined #openstack-kolla22:42
sdakebmace_ i present to a meetup on the 15th - may like to use the dev env for that22:42
sdakei've invited the community to listen in22:42
bmace_sdake: well, i can tell you with 2gb controller pods start self terminating like crazy about mid way through neutron deploy22:42
bmace_sdake: i have enough mem on my box for 8gb, so i didn't do the interim 4.. might work22:43
sdakeyou mena 2gb kubernetes compute nodes?22:43
bmace_sdake: the env that did not work was 2gb across the board, control + compute.. because control blows up.  what did work was 8gb control, 2gb compute22:43
openstackgerritSerguei Bezverkhi proposed openstack/kolla-kubernetes master: WIP Gate refactor part deux  https://review.openstack.org/43176522:44
*** mannamne has quit IRC22:44
sdakeby control - do you mean the kubernetes compute node was labelled a controller?22:44
sdakekubernetes has a master, and compute nodes22:44
bmace_sdake: and feel free to use it.  not 100% sure it will be merged in depending on what happens with the kubernetes library timing but you can always run with just the patch22:44
*** fragatina has joined #openstack-kolla22:44
sbezverkkfox1111: can you check this ps I want to make sure it is what you meant in your comment https://review.openstack.org/43176522:44
sdakebmace_ cool - will need your code rebased to use it i think ;-)22:45
bmace_sdake: yes, obviously the one running all the control kolla pods..22:45
sdakebmace_ roger thanks22:45
*** jrobinson-afk is now known as jrobinson22:47
*** mgoddard has quit IRC22:47
*** krtaylor has joined #openstack-kolla22:49
*** dave-mccowan has quit IRC22:50
*** david-lyle has quit IRC22:50
*** david-lyle has joined #openstack-kolla22:50
sdakehttps://docs.google.com/document/d/1TW3P4c8auWwYy-w_5afIPDcGNLK3LZf0m14943eVfVg/edit?ts=58877ea722:53
sdakekolla cores, have a look ^22:53
sdakeand anyone else that is curious of course :)22:53
*** adrian_otto has quit IRC22:55
openstackgerritSerguei Bezverkhi proposed openstack/kolla-kubernetes master: WIP Gate refactor part deux  https://review.openstack.org/43176523:02
openstackgerritMerged openstack/kolla-ansible master: Add Octavia doc link in README.rst  https://review.openstack.org/43135523:05
*** david-lyle_ has joined #openstack-kolla23:06
*** david-lyle has quit IRC23:06
openstackgerritMerged openstack/kolla-ansible master: use_forwarded_for is moved to api group  https://review.openstack.org/43133723:06
*** david-lyle_ has quit IRC23:09
openstackgerritBorne Mace proposed openstack/kolla-kubernetes master: WIP: Added support for containerized dev bringup  https://review.openstack.org/42929523:10
*** vhosakot has quit IRC23:10
*** esmiurium has joined #openstack-kolla23:11
*** vhosakot has joined #openstack-kolla23:11
*** yuanying has joined #openstack-kolla23:12
kfox1111sbezverk: looking23:13
kfox1111sbezverk: yeah. perfect. :)23:15
kfox1111I wont +2 until the gate passes. thats bit me too many times. :/23:15
kfox1111but has my +2. :)23:15
*** rwallner has quit IRC23:16
openstackgerritSerguei Bezverkhi proposed openstack/kolla-kubernetes master: WIP Gate refactor part deux  https://review.openstack.org/43176523:17
bmace_kfox1111: sbezverk: isn't "job" a strange name for it?  why not $storage_type or some such?23:18
kfox1111bmace_: not sure what your alking about.23:19
*** lamt has quit IRC23:19
bmace_kfox1111: thought you guys were talking about the part 2 gate refactor.. was talking about the ceph_workflow_service change.23:19
kfox1111bmace_: we are. just not sure what part of the code you are talking about.23:20
kfox1111file/line maybe?23:20
kfox1111oh. I think I see what your saying.23:20
bmace_ceph_workflow_service 16 + 2823:20
bmace_https://review.openstack.org/#/c/431765/2/tests/bin/ceph_workflow_service.sh23:21
kfox1111why "ceph" getting a job value?23:21
kfox1111its in terms of the gate jobs.23:21
kfox1111I think.23:21
kfox1111which happens to line up to storage_type in this case.23:22
kfox1111hmm...23:23
kfox1111yeah. it lines up to the job type in the gate.23:23
kfox1111so, "ceph" "ceph-multi" "external-ovs" "helm-entrypoint" etc.23:23
bmace_i guess if is only used in the gate it makes more sense.. for someone using it anywhere else, devenv, etc.. it is pretty non-obvious without some doc23:24
kfox1111yeah.23:25
kfox1111there could be a job=devenv option.23:26
kfox1111or we could figure out something else to rename job too.23:26
kfox1111its just historical as thats what the code got written for. totally open to changing it.23:27
bmace_if it comes down to just reflecting storage type, that seems like a more straight forward name.  it is always ceph_multi in the devenv container.23:27
kfox1111its not always.23:27
kfox1111it can be "ceph" "ceph-multi" "external-ovs" "helm-entrypoint" etc.23:27
bmace_not in the devenv container, unless folks want to go and hand edit stuff23:28
*** fragatin_ has joined #openstack-kolla23:28
kfox1111yeah.23:28
kfox1111the other option is to split it out further.23:28
*** fragatin_ has quit IRC23:28
kfox1111the gate job type implies a storage_type.23:28
kfox1111we could break those two apart and in most of the scripts require both.23:28
sbezverkkfox1111: the only concern I have is element_name I think without pinning them, some service like cinder-volume will fail23:28
kfox1111so, then it woudl be 'devenv' 'ceph'23:28
kfox1111or something.23:28
kfox1111sbezverk: shouldn't, as there shoudl be a default. if it doesn't have a default, I think thats a bug?23:29
bmace_yeah, i see.. well, i'm ok then leaving as it for the time being :)23:29
kfox1111bmace_: yeah. I totally see where your coming from and agree it shoudl be fixed. just not sure when.23:29
bmace_not an actual functional issue anyway, just felt odd when i was looking through the changes.23:29
kfox1111yeah.23:30
*** fragatina has quit IRC23:31
sbezverkbmace_: I do not mind to change it for something else, it kind of made sense to me but I am totally opened23:31
sbezverkI can name it gate_job23:32
*** fragatina has joined #openstack-kolla23:32
sbezverkin this case it should be more clear23:32
bmace_sbezverk: after kfox1111 gave some more background as to where it was coming from, at this point it really doesn't purely represent storage type, so unless we rework things i think it is fine as is23:32
bmace_sbezverk: sure23:32
bmace_sbezverk: definitely a bit better23:32
sbezverkbmace_: cool will change it, I am positive more revisions are comming ;)23:33
kfox1111designate's not in a good place. thats sad. :/23:34
kfox1111it had a lot of potential.23:34
bmace_sbezverk: that v2 change looked pretty good at first glance though.  glad to see nothing major, as you said.  probably no or very minor merge issues from it23:34
openstackgerritMerged openstack/kolla master: Fix pushing images to local registry  https://review.openstack.org/42633823:35
sbezverkbmace_: my concern that this change could uncover some mini bugs we might have, where default value for element_name does not get assigned..23:36
sbezverkbut we will see23:36
*** mgiles has quit IRC23:38
*** skuda has quit IRC23:48
*** fragatina has quit IRC23:50
*** fragatina has joined #openstack-kolla23:50
*** ipsecguy has joined #openstack-kolla23:52
*** srwilkers has quit IRC23:52

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!