00:01:32 <ekcs> #startmeeting congressteammeeting 00:01:32 <openstack> Meeting started Thu Jan 19 00:01:32 2017 UTC and is due to finish in 60 minutes. The chair is ekcs. Information about MeetBot at http://wiki.debian.org/MeetBot. 00:01:33 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 00:01:36 <openstack> The meeting name has been set to 'congressteammeeting' 00:01:44 <ekcs> hi thinrichs ramineni_ ! 00:02:05 <aimeeu> greeting all 00:02:23 <ekcs> welcome back everyone! 00:03:00 <ekcs> only items I have in mind are talking about ocata-3. 00:03:07 <ekcs> Ocata-3 outstanding patches 00:03:08 <ekcs> Ocata-3 outstanding/new bugs 00:03:13 <ekcs> anything else on our minds? 00:03:35 <aimeeu> etherpad for PTG planning? 00:04:31 <masahito> hi all. 00:04:58 <ekcs> hi masahito 00:05:09 <ekcs> aimeeu: ok. 00:05:28 <bryan_att> hi all, bryan_att lurking 00:05:44 <ekcs> hi bryan_att ! 00:05:57 <bryan_att> hi! 00:06:08 <ekcs> ok let’s dive in then. 00:06:22 <ekcs> #topic etherpad for PTG planning 00:06:47 <ekcs> So aimeeu I’m guessing what you mean is putting an etherpad in place so people can start putting down ideas and thoughts in PTG planning? 00:06:55 <aimeeu> yes exactly 00:08:35 <aimeeu> and then adding the link here: #link https://wiki.openstack.org/wiki/PTG/Pike/Etherpads 00:08:51 <ekcs> ok great I can do that after the meeting 00:09:03 <ekcs> #action ekcs add PTG planning etherpad 00:09:12 <aimeeu> ekcs: thanks loads! 00:09:16 <ekcs> thanks for the minder! 00:09:23 <ekcs> moving on then. 00:09:30 <ekcs> #topic Ocata-3 outstanding patches 00:10:04 <ekcs> just for background, Ocata-3 is due next week. Testing is underway. 00:10:37 <ekcs> we did a good job of getting patches in for review last week, but we still have a few outstanding patches unmerged. 00:10:54 <ekcs> #link https://review.openstack.org/416777 00:10:54 <ekcs> #link https://review.openstack.org/417606 00:10:56 <ekcs> #link https://review.openstack.org/420653 00:10:56 <ekcs> #link https://review.openstack.org/410705 00:11:02 <ekcs> (anything else?) 00:11:38 <ekcs> I figure we can talk about whether we can/want to merge them quickly. 00:12:52 <thinrichs> 606 and 653 look about ready to me. Just a bug fix in 606, and then it and the dependent 653 can go in. 00:12:56 <thinrichs> Should be a quick fix. 00:14:14 <thinrichs> 705 looks to be waiting on @masahito to answer the question: is this patch good enough for now? 00:14:44 <ekcs> thinrichs: makes sense. 00:15:14 <masahito> thinrichs: I'll check it today. 00:15:55 <thinrichs> 777 I haven't looked at in depth. Only @masahito has. And it's somewhat thorny. If @masahito thinks its ready, fine with me. 00:17:47 <thinrichs> ramineni_: does your client patch need to go in? Or is it too late for that? 00:18:08 <thinrichs> https://review.openstack.org/#/c/412386/ 00:19:03 <masahito> thinrichs: And the comment for 606 is that the json module can load rule string from json data if it's surrounded by double-quota. 00:19:41 <thinrichs> masahito: It's that the policy language ONLY supports double-quotes. 00:19:52 <thinrichs> So you can write p("abc") but not p('abc') 00:20:19 <ramineni_> thinrichs: is client release done?? It's a simple patch , but not very imp ...As we have alternate way to do it too .. so either way should be fine 00:20:46 <ekcs> ramineni_: client release not yet done. plan to do next week concurrently with ocata-3. 00:20:52 <masahito> thinrichs: when I use congress client, it works with single client. 00:21:23 <thinrichs> masahito: You can insert p('abc') ? 00:21:41 <ramineni_> ekcs: ok 00:21:51 <masahito> thinrichs: yes 00:22:13 <thinrichs> That's surprising. Maybe we added Python-like string handling. 00:22:16 <masahito> the client changed single quote to double quote in it. 00:23:11 <ekcs> interesting it seems to convert to double quotes. 00:23:14 <ekcs> $ openstack congress policy rule create action "p('abc')" 00:23:14 <masahito> then it calls API with p("abc"). 00:23:15 <ekcs> +---------+--------------------------------------+ 00:23:16 <ekcs> | Field | Value | 00:23:18 <ekcs> +---------+--------------------------------------+ 00:23:19 <ekcs> | comment | | 00:23:21 <ekcs> | id | d1a4c333-a71d-4f49-8fa8-c42abadea41c | 00:23:22 <ekcs> | name | None | 00:23:23 <ekcs> | rule | p("abc") | 00:23:24 <ekcs> +---------+--------------------------------------+ 00:23:50 <thinrichs> Looking at the grammar, any Python string should work. Forgot about that. 00:24:11 <thinrichs> When we are printing out the rule we always use double-quotes. 00:24:48 <masahito> I think there're two option. 00:25:15 <thinrichs> masahito: then the comment in the 606 should be: have you tested the code with p("abc") AND p('abc')? 00:26:00 <masahito> thinrichs: p("abc") in json file doesn't work 00:26:56 <masahito> because json module thinks "rule": "p("abc")" is "rule": "p(" and abc")" 00:27:33 <thinrichs> What if someone escaped the quotes, such as "rule": "p(\"abc\")"? That would be valid JSON (I think). 00:27:53 <masahito> thinrichs: this works. 00:28:11 <thinrichs> Okay, then it's fine as is I'd say. 00:28:11 <masahito> my another idea is using yaml file instead of json 00:28:55 <thinrichs> We could always change it over to yaml later, but leaving it as JSON lets us get the functionality into ocata3 00:29:08 <thinrichs> I'd say we can merge those 2 right away. 00:29:48 <ekcs> +1 00:29:57 <masahito> +1 00:30:53 <ekcs> ok then! 00:32:14 <ekcs> ramineni_: what do you think about https://review.openstack.org/#/c/412386/ ? 00:32:38 <ekcs> it looks fine to me. a little strange that we’re going to have failing osc test from now on. 00:32:54 <ekcs> it’s required to support horizon getting subscribers? 00:33:46 <thinrichs> Sorry—one sec. Just saw Aimee's comments on 653. 00:34:16 <aimeeu> thinrichs: I answered my own question 00:34:37 <ramineni_> ekcs: no ..It's not required for subscribers patch 00:34:41 <thinrichs> aimeeu: ok. So that's ready to go in? 00:34:51 <aimeeu> yes absolutely 00:35:11 <ramineni_> ekcs ..but osc plugin failure is not related to the patch right 00:36:30 <ekcs> ramineni_: I was just guessing that osc failure has to do with what Tim said about adding command to osc dir. Maybe not. 00:37:07 <ramineni_> ekcs: no , but will check again 00:37:49 <ekcs> ok. 00:38:53 <ekcs> so on https://review.openstack.org/#/c/416777/ , if someone has time to take a look I’d love to get that merged in order to maximize the testing on it. 00:39:10 <ekcs> but since it’s not a new feature, it technically could wait till after O3 release. 00:41:00 <ekcs> ramineni_: ok sorry I’m a little slow on this. 00:42:00 <ekcs> nvm let’s move on I’ll catch up on it (the client patch) after. 00:42:15 <ekcs> any other thoughts about the patches before we move on? 00:43:57 <ekcs> #topic Ocata-3 outstanding/new bugs 00:45:00 <ekcs> ok does anyone have new/outstanding bugs we should look at ? 00:45:49 <ekcs> either to be fixed by ocata-3 or shortly after. probably good to triage the bugs before ocata-3 to understand if they are fixable once restrictions kick in post O3. 00:47:24 <thinrichs> Nothing from me 00:48:21 <ekcs> #link here’s the current bugs list with “new” on top. haha kind of annoying that tasks are mixed in with actual bugs. 00:48:26 <ekcs> #link https://bugs.launchpad.net/congress/+bugs?orderby=status&start=0 00:50:20 <ekcs> I’ve encountered a couple new bugs, but nothing that seems to be deep, hard-to-fix things. mostly on the periphery like DSD interfaces. 00:50:46 <ekcs> so i’m adding them as I see them. 00:51:18 <ekcs> ramineni_: I noticed the same bug you registered back in september. #link https://bugs.launchpad.net/congress/+bug/1626821 00:51:18 <openstack> Launchpad bug 1626821 in congress "swift driver fails with authorization issue while polling" [Undecided,Invalid] 00:51:59 <ekcs> any idea why it shows up only on infra dsvm test but not locally? 00:52:12 <ekcs> Trying to figure out if we need to dig into it or it doesn’t matter. 00:53:00 <ramineni_> ekcs: ya ..I remember debugging it before ..But it works all fine in local environment .. 00:53:15 <ramineni_> And on gate too it's temporary 00:53:35 <ekcs> anyway we have 8 minutes left. let’s go to open discussion. where we can still continue discussing bugs. 00:53:41 <ekcs> #topic open discussion 00:53:45 <ramineni_> Did u see the error in local environment too? 00:53:53 <ekcs> but feel free to throw out anything else. 00:54:29 <ekcs> ramineni_: no I didn’t see it locally either. I haven’t tested locally on xenial tho. thinking I should probably do that. cuz it could be a trusty vs xenial thing. 00:54:42 <ekcs> ramineni_: do you test on trusty or xenial? 00:54:53 <ramineni_> Trusty 00:55:17 <ramineni_> I think gate also on trusty at that time 00:55:51 <ekcs> ramineni_: hmm ok. well I guess we should test on xenial anyway so we’ll see what happens. 00:56:04 <ramineni_> Ok 00:59:13 <ekcs> ok well thanks all! have a good week and let’s have a great Ocata-3 release! 00:59:28 <thinrichs> Thanks all! 00:59:52 <masahito> Thanks all 00:59:57 <ekcs> see you all later! 01:00:14 <ekcs> #endmeeting