Tuesday, 2015-10-06

adrian_ottoI'm thinking for a momenta bout how common cross-bay operations will be… how often will users want it00:00
adrian_ottomy first reaction is that they probably won't use it much at all00:00
adrian_ottoit will probably be much more common to act on a single bay00:01
adrian_ottoa user probably would like the convenience of using a bay <name | uuid> rather than only uuid though00:03
adrian_ottohave you considered that?00:03
suro-patzadrian_otto: in that case, it makes sense to have an option to read the bay-uuid/name from shell/bash environment, just like OS_USERNAME00:03
suro-patzso that for every operation it is not passed00:04
vilobhmmadrian_otto : since tenant boundary is restricted to a Bay; so can agree with you that it would be common to act on a single bay and hence magnum rc-list <bay-uuid (*One bay at a time*)>00:04
suro-patzand if exported, by default the operation would on that given bay00:04
adrian_ottointeresting idea suro-patz00:04
suro-patzanalogy of --os-user-name OS_USERNAME00:04
adrian_ottobut each user will probably have more than one00:05
vilobhmmadrian_otto : giving user a convinience of using bay <name|uuid> should be easy to implement. I can give it a try…gives more flexibility to user IMHO00:05
suro-patzit is with the assumption that most likely an user is operating on a given bay at a time00:05
suro-patzhe/she can pass it along as a cmd-line-arg, which will override the env00:06
adrian_ottowe could get fancy later, and allow for wildcard matches on the name00:06
adrian_ottofor example magnum rc-list 'foo%'00:06
adrian_ottoor something like that00:06
suro-patzmakes sense, as user may name his bays with common prefix00:07
adrian_ottowhen we implementa labels/tags we'll need an index that allows for that sort of usage00:07
adrian_ottomagnum rc-list —label=foo00:08
adrian_ottoget back an aggregate list of rs'c that match that tag00:08
adrian_ottok8s has functionality like that00:08
vilobhmmadrian_otto : each bay need to be tagged with the label then IMHO00:09
adrian_ottovilobhmm: if there is not a clean way to implement a parallel fetch of all bays, then I'm willing to accept the current implementation.00:09
vilobhmmadrian_otto : sure thanks! would enhance it with the option to specify both uuid/name00:10
vilobhmmif that helps for better user experience00:10
adrian_ottoI don't think we have conductors that can do async calls, do we?00:10
suro-patzadrian_otto: no00:10
adrian_ottook, so we would be limited by the number of conductors running, bummer00:11
suro-patzadrian_otto: but fetching the entire bay list every-time, looks costly00:11
adrian_ottoyes, I agree it's a legitimate concern00:11
adrian_ottook, I'm going to retract my suggestion for now00:13
adrian_ottobut I will suggest the <name | uuid> argument support as a consolation ;-)00:13
vilobhmmadrian_otto : sure..thanks !  yes for sure00:14
*** gangil has quit IRC00:14
suro-patzThe user/tenant filter applies by default - so having the choice to specify the bay_uuid/name would be really handy to prune the list-data, if there are many bays owned by a given user00:15
*** sthillma has quit IRC00:16
*** zz_dimtruck is now known as dimtruck00:16
*** sthillma has joined #openstack-containers00:17
adrian_ottosuro, if we filter by bay name, that should work nicely in the general case00:18
suro-patzadrian_otto: agree00:18
adrian_ottovilobhmm: I put follow-up comments in the review and the bug in accordance with our chat above.00:20
adrian_ottothanks for the discussion. This was productive.00:20
vilobhmmadrian_otto : thanks :) that was useful.00:21
*** achanda has joined #openstack-containers00:21
adrian_ottovilobhmm and humble00 are our new core reviewers as of 20 minutes ago00:22
adrian_ottoI will add them to the group, and announce it at tomorrow's team meeting00:22
suro-patzcongrats vilobhmm and humble00!00:22
vilobhmmadrian_otto : thanks :) happy to help and contribute more !00:22
vilobhmmsuro-patz : thanks ! :)00:23
*** gangil has joined #openstack-containers00:24
*** gangil has quit IRC00:24
*** gangil has joined #openstack-containers00:24
adrian_otto+2 powers have been granted.00:26
adrian_ottotime for me to end for today. I'll catch you again at 1600 UTC.00:27
*** suro-patz has quit IRC00:29
*** julim has joined #openstack-containers00:30
*** kebray has quit IRC00:32
*** dane_leblanc has joined #openstack-containers00:34
*** adrian_otto has quit IRC00:35
*** Tango has quit IRC00:39
*** julim has quit IRC00:41
*** ganeshna has joined #openstack-containers00:45
*** dims_ has joined #openstack-containers00:50
*** ganeshna has quit IRC00:51
*** ganeshna has joined #openstack-containers00:51
*** devkulkarni has quit IRC00:54
*** dimtruck is now known as zz_dimtruck00:54
*** zz_dimtruck is now known as dimtruck00:55
*** dims_ has quit IRC00:55
*** ganeshna has quit IRC00:57
*** dims_ has joined #openstack-containers00:58
*** eggshell has quit IRC00:59
*** agireud has quit IRC01:01
*** eghobo has joined #openstack-containers01:03
*** vikumar has joined #openstack-containers01:09
openstackgerritVilobh Meshram proposed openstack/python-magnumclient: Pass bay_uuid to k8s objects list method  https://review.openstack.org/23121901:11
*** sthillma has quit IRC01:15
*** erkules_ has joined #openstack-containers01:20
*** dimtruck is now known as zz_dimtruck01:21
*** ganeshna has joined #openstack-containers01:22
*** erkules has quit IRC01:23
*** eghobo has quit IRC01:25
*** SourabhP has quit IRC01:26
*** dims_ has quit IRC01:30
*** ganeshna has quit IRC01:31
*** devkulkarni has joined #openstack-containers01:36
*** mfalatic has quit IRC01:39
*** gangil has quit IRC01:40
*** mfalatic has joined #openstack-containers01:40
*** vilobhmm has quit IRC02:09
*** rlrossit has joined #openstack-containers02:13
*** tbh has joined #openstack-containers02:13
*** rlrossit has quit IRC02:13
*** unicell has quit IRC02:18
*** kebray has joined #openstack-containers02:18
*** gangil has joined #openstack-containers02:25
*** gangil has quit IRC02:25
*** gangil has joined #openstack-containers02:25
*** devkulkarni has quit IRC02:28
*** devkulkarni has joined #openstack-containers02:36
*** suro-patz has joined #openstack-containers02:44
*** dave-mccowan has quit IRC02:44
*** tbh has quit IRC02:49
*** sdake has joined #openstack-containers02:49
*** achanda has quit IRC02:50
*** muralia has quit IRC02:55
*** agireud has joined #openstack-containers02:57
*** eggshell has joined #openstack-containers02:57
*** muralia has joined #openstack-containers02:58
*** rebase33 has quit IRC03:00
*** kebray has quit IRC03:02
*** sdake has quit IRC03:09
*** sdake has joined #openstack-containers03:10
*** sdake has quit IRC03:13
*** kebray has joined #openstack-containers03:18
*** kebray has quit IRC03:18
*** kebray has joined #openstack-containers03:18
*** vikumar has quit IRC03:19
*** deepthi has joined #openstack-containers03:20
*** sdake has joined #openstack-containers03:33
*** vimal has joined #openstack-containers03:33
*** Tango has joined #openstack-containers03:43
*** devkulkarni has quit IRC03:44
*** VikasC has joined #openstack-containers03:55
*** eghobo has joined #openstack-containers04:01
*** eghobo has quit IRC04:02
*** puranamr has joined #openstack-containers04:07
*** puranamr has quit IRC04:12
*** eghobo has joined #openstack-containers04:16
*** SourabhP has joined #openstack-containers04:25
*** Marga__ has quit IRC04:26
*** banix has quit IRC04:31
*** daneyon has joined #openstack-containers04:35
*** daneyon_ has joined #openstack-containers04:35
*** daneyon has quit IRC04:39
*** achanda has joined #openstack-containers04:42
*** unicell has joined #openstack-containers04:42
*** SourabhP_ has joined #openstack-containers04:42
*** chandankumar has joined #openstack-containers04:43
*** SourabhP has quit IRC04:44
*** SourabhP_ is now known as SourabhP04:44
*** dims_ has joined #openstack-containers04:46
*** dane_leblanc has quit IRC04:55
*** subscope has joined #openstack-containers05:01
*** filler has quit IRC05:02
*** redrobot has quit IRC05:03
*** vimal has quit IRC05:04
*** SourabhP has quit IRC05:04
*** tristanC has quit IRC05:04
*** tristanC has joined #openstack-containers05:05
*** adrian_otto has joined #openstack-containers05:06
*** redrobot has joined #openstack-containers05:08
*** redrobot is now known as Guest2867805:09
*** f1ller has joined #openstack-containers05:09
*** sdake_ has joined #openstack-containers05:09
*** vimal has joined #openstack-containers05:10
*** suro-patz has quit IRC05:10
*** sdake has quit IRC05:13
*** eghobo_ has joined #openstack-containers05:18
*** eghobo has quit IRC05:22
*** tbh has joined #openstack-containers05:22
*** daneyon_ has quit IRC05:23
*** daneyon has joined #openstack-containers05:25
*** daneyon_ has joined #openstack-containers05:28
*** daneyon has quit IRC05:30
*** diga has joined #openstack-containers05:31
*** Marga_ has joined #openstack-containers05:32
*** adrian_otto has quit IRC05:34
*** ganeshna has joined #openstack-containers05:35
*** Marga_ has quit IRC05:36
*** Marga_ has joined #openstack-containers05:36
*** sdake_ has quit IRC05:38
*** unicell has quit IRC05:52
*** unicell has joined #openstack-containers05:53
*** unicell1 has joined #openstack-containers05:59
*** unicell has quit IRC06:00
*** eghobo has joined #openstack-containers06:13
*** gangil has quit IRC06:14
*** eghobo_ has quit IRC06:16
*** Marga_ has quit IRC06:29
*** Marga_ has joined #openstack-containers06:30
*** Tango has quit IRC06:40
*** j___ has joined #openstack-containers06:42
*** jwang_ has quit IRC06:54
*** jwang_ has joined #openstack-containers06:54
openstackgerritMerged openstack/magnum: Enabled ceilometer using plugin model  https://review.openstack.org/23103407:00
*** BertrandN has quit IRC07:03
*** j___ has quit IRC07:05
*** j___ has joined #openstack-containers07:06
*** BertrandN has joined #openstack-containers07:10
*** BertrandN has quit IRC07:16
*** achanda has quit IRC07:18
*** kebray has quit IRC07:19
*** achanda has joined #openstack-containers07:22
*** erkules_ is now known as erkules07:25
*** erkules has quit IRC07:25
*** erkules has joined #openstack-containers07:25
*** mathspanda has joined #openstack-containers07:38
*** yuanying has left #openstack-containers07:39
*** yuanying has joined #openstack-containers07:40
*** mathspanda has quit IRC07:41
*** Drago2 has joined #openstack-containers07:44
*** madhuri has joined #openstack-containers07:46
*** Drago2 has quit IRC07:46
*** eghobo has quit IRC07:51
*** BertrandN has joined #openstack-containers07:52
*** xek_ is now known as xek07:53
*** achanda has quit IRC07:57
*** achanda has joined #openstack-containers08:05
*** achanda has quit IRC08:16
*** ganeshna has quit IRC08:24
*** ganeshna has joined #openstack-containers08:36
*** alex_klimov has joined #openstack-containers08:39
*** achanda has joined #openstack-containers08:39
*** achanda has quit IRC08:50
*** deepthi has quit IRC09:00
*** deepthi has joined #openstack-containers09:12
*** ganeshna has quit IRC09:13
*** deepthi has quit IRC09:18
*** ganeshna has joined #openstack-containers09:23
*** j___ has quit IRC09:30
*** tbh has quit IRC09:31
*** deepthi has joined #openstack-containers09:34
*** j___ has joined #openstack-containers09:37
*** ganeshna has quit IRC09:41
*** j___ has quit IRC09:47
*** tbh has joined #openstack-containers09:47
*** exploreshaifali has joined #openstack-containers09:52
*** ganeshna has joined #openstack-containers09:57
*** vimal has quit IRC10:14
*** ganeshna has quit IRC10:14
*** ganeshna has joined #openstack-containers10:23
*** dave-mccowan has joined #openstack-containers10:24
*** j___ has joined #openstack-containers10:47
*** ganeshna has quit IRC10:49
*** tbh has quit IRC10:51
*** sdake has joined #openstack-containers10:52
*** daneyon_ has quit IRC11:03
*** ganeshna has joined #openstack-containers11:07
*** tbh has joined #openstack-containers11:09
*** madhuri has quit IRC11:09
*** ganeshna_ has joined #openstack-containers11:10
*** ganeshna has quit IRC11:11
*** ganeshna_ has quit IRC11:24
*** chandankumar has quit IRC11:31
*** deepthi has quit IRC11:48
*** devkulkarni has joined #openstack-containers12:07
*** sdake has quit IRC12:09
*** sdake has joined #openstack-containers12:10
*** exploreshaifali has quit IRC12:17
*** Marga_ has quit IRC12:18
*** Marga_ has joined #openstack-containers12:19
*** tbh has quit IRC12:20
*** VikasC has quit IRC12:30
*** annegentle has joined #openstack-containers12:45
*** julim has joined #openstack-containers12:50
*** Marga_ has quit IRC12:50
*** Marga_ has joined #openstack-containers12:51
*** jruano has joined #openstack-containers12:53
*** rlrossit has joined #openstack-containers12:57
*** sdake has quit IRC13:09
*** sdake has joined #openstack-containers13:10
*** sdake_ has joined #openstack-containers13:12
*** sdake has quit IRC13:14
*** j___ has quit IRC13:17
*** dave-mccowan has quit IRC13:19
*** Marga_ has quit IRC13:23
*** Marga_ has joined #openstack-containers13:23
*** j___ has joined #openstack-containers13:25
openstackgerritMerged openstack/magnum: Fix swarm bay failure reporting  https://review.openstack.org/23063913:26
*** dave-mccowan has joined #openstack-containers13:34
*** banix has joined #openstack-containers13:44
*** chandankumar has joined #openstack-containers13:45
*** Tango has joined #openstack-containers13:57
*** manjeets has joined #openstack-containers14:00
*** muralia has quit IRC14:00
*** muralia has joined #openstack-containers14:01
*** muralia1 has joined #openstack-containers14:03
Tangohongbin:  https://apps.na.collabserv.com/meetings/join?id=2897-229314:04
manjeets._.14:04
*** jay-lau-513 has joined #openstack-containers14:05
*** hongbin has joined #openstack-containers14:05
*** dane_leblanc has joined #openstack-containers14:05
*** manjeets has quit IRC14:05
*** muralia has quit IRC14:05
Tangohongbin: https://apps.na.collabserv.com/meetings/join?id=2897-229314:05
*** devkulkarni has quit IRC14:09
*** j___ has quit IRC14:09
*** manjeets has joined #openstack-containers14:13
*** Guest28678 is now known as redrobot14:13
*** chandankumar has quit IRC14:18
*** zz_dimtruck is now known as dimtruck14:19
*** fawadkhaliq has joined #openstack-containers14:28
*** achanda has joined #openstack-containers14:29
*** manjeets has quit IRC14:32
*** devkulkarni has joined #openstack-containers14:36
*** achanda has quit IRC14:37
*** adrian_otto has joined #openstack-containers14:47
*** diga has quit IRC14:47
*** adrian_otto has quit IRC14:57
*** achanda has joined #openstack-containers14:57
*** hongbin has quit IRC14:59
*** adrian_otto has joined #openstack-containers15:00
*** achanda has quit IRC15:01
*** rebase33 has joined #openstack-containers15:05
*** adrian_otto has quit IRC15:05
*** tbh has joined #openstack-containers15:05
*** david-ly_ is now known as david-lyle15:06
*** hongbin has joined #openstack-containers15:06
*** adrian_otto has joined #openstack-containers15:06
*** adrian_otto has quit IRC15:07
*** chandankumar has joined #openstack-containers15:09
*** chandankumar has quit IRC15:09
*** chandankumar has joined #openstack-containers15:09
*** ganeshna has joined #openstack-containers15:11
*** manjeets has joined #openstack-containers15:12
*** daneyon has joined #openstack-containers15:16
*** daneyon has quit IRC15:17
*** sdake_ is now known as sdake15:23
*** dims_ has quit IRC15:27
*** daneyon has joined #openstack-containers15:27
*** Daviey_ has quit IRC15:30
*** Daviey has joined #openstack-containers15:30
*** kebray has joined #openstack-containers15:32
*** dims_ has joined #openstack-containers15:32
*** jay-lau-513 has quit IRC15:37
*** manjeets1 has joined #openstack-containers15:43
*** daneyon_ has joined #openstack-containers15:43
*** VikasC has joined #openstack-containers15:44
*** daneyon has quit IRC15:46
*** adrian_otto has joined #openstack-containers15:47
*** manjeets_ has joined #openstack-containers15:48
adrian_ottoOur team meeting will begin in #openstack-meeting-alt at 1600 UTC … just over 10 minutes from now. I look forward to seeing you all there!15:49
*** manjeets1 has quit IRC15:50
*** manjeets has quit IRC15:51
*** coreyob has joined #openstack-containers15:53
*** bradjones___ has joined #openstack-containers15:58
*** unicell has joined #openstack-containers15:58
*** unicell1 has quit IRC15:59
openstackgerritOpenStack Proposal Bot proposed openstack/magnum: Updated from global requirements  https://review.openstack.org/23056616:00
*** BertrandN has quit IRC16:04
*** Marga_ has quit IRC16:06
*** eghobo has joined #openstack-containers16:14
*** suro-patz has joined #openstack-containers16:14
*** ganeshna has quit IRC16:15
*** eghobo_ has joined #openstack-containers16:19
*** gangil has joined #openstack-containers16:20
*** gangil has quit IRC16:20
*** gangil has joined #openstack-containers16:20
*** tbh has quit IRC16:20
*** tbh has joined #openstack-containers16:22
*** eghobo has quit IRC16:22
*** alex_klimov has quit IRC16:24
*** eghobo has joined #openstack-containers16:24
*** eghobo_ has quit IRC16:27
*** banix has quit IRC16:30
*** sthillma has joined #openstack-containers16:31
*** dimtruck is now known as zz_dimtruck16:34
*** unicell has quit IRC16:35
*** dims_ has quit IRC16:37
*** dims_ has joined #openstack-containers16:41
*** dims_ has quit IRC16:41
*** chandankumar has quit IRC16:41
*** dims_ has joined #openstack-containers16:42
*** dims__ has joined #openstack-containers16:44
*** dims_ has quit IRC16:45
*** vilobhmm has joined #openstack-containers16:54
*** jwcroppe_ has joined #openstack-containers16:55
*** jwcroppe has quit IRC16:56
*** bradjones___ has left #openstack-containers16:57
*** eghobo has quit IRC16:59
*** hongbin has quit IRC17:01
*** eghobo has joined #openstack-containers17:02
*** sdake has quit IRC17:03
*** VikasC has quit IRC17:04
*** sdake has joined #openstack-containers17:05
*** adrian_otto has quit IRC17:09
*** unicell has joined #openstack-containers17:12
*** Marga_ has joined #openstack-containers17:12
*** adrian_otto has joined #openstack-containers17:12
*** banix has joined #openstack-containers17:12
*** Tango has quit IRC17:12
*** manjeets_ has quit IRC17:18
*** david-ly_ has joined #openstack-containers17:21
*** david-lyle has quit IRC17:21
*** sthillma has quit IRC17:21
*** david-ly_ is now known as david-lyle17:22
*** sthillma has joined #openstack-containers17:23
*** achanda has joined #openstack-containers17:24
*** harlowja has quit IRC17:25
*** sthillma_ has joined #openstack-containers17:28
*** kebray has quit IRC17:28
*** dims_ has joined #openstack-containers17:29
*** sthillma has quit IRC17:31
*** sthillma_ is now known as sthillma17:31
*** dims__ has quit IRC17:32
*** tbh has quit IRC17:33
*** harlowja has joined #openstack-containers17:35
*** rlrossit has quit IRC17:38
*** fawadkhaliq has quit IRC17:42
*** fawadkhaliq has joined #openstack-containers17:43
*** tbh has joined #openstack-containers17:45
*** zz_dimtruck is now known as dimtruck17:46
*** exploreshaifali has joined #openstack-containers17:50
*** rlrossit has joined #openstack-containers17:50
*** kebray has joined #openstack-containers17:51
*** dims_ has quit IRC17:53
*** dave-mccowan has quit IRC17:54
*** dims_ has joined #openstack-containers18:01
*** kebray has quit IRC18:05
*** dims_ has quit IRC18:05
*** dims_ has joined #openstack-containers18:06
*** dave-mccowan has joined #openstack-containers18:07
*** Tango has joined #openstack-containers18:10
*** harshs has joined #openstack-containers18:24
*** tbh has quit IRC18:24
mfalaticdoes magnum service-list work?18:33
mfalaticbrand new install and I get "ERROR: Internal Server Error (HTTP 500)"18:33
suro-patzmfalatic: There is a policy conflict happening18:33
mfalaticok18:33
mfalaticnot just me then18:33
suro-patzit is only for admin18:33
mfalatichmm18:33
mfalaticOur quickstart uses admin so that'll happen for anyone following that. is this as-designed or a bug?18:34
suro-patzI see that as a bug I  am working on the same18:34
suro-patzmfalatic: Do you want to open a bug for the same?18:35
*** sthillma has quit IRC18:35
mfalatic(if it's unreported as such, I can create a bug for it, else what's your bug number?18:35
suro-patzmfalatic: please feel free to assign it to me, I also noticed this today morning18:35
mfalaticsuro-patz - not sure if a second bug on the same issue is necessary, if the goal is to make this work.18:35
mfalaticoic ok will open a bug now18:35
*** muralia1 has quit IRC18:36
*** sthillma has joined #openstack-containers18:38
mfalaticsuro-patz - The bug is created, but i don't have privileges to assign it to you directly. https://bugs.launchpad.net/magnum/+bug/150340218:42
openstackLaunchpad bug 1503402 in Magnum "magnum service-list fails for user/project admin" [Undecided,New]18:42
suro-patzmfalatic: Claimed the same. Thanks for reporting/confirming18:43
mfalaticGlad to help keep Magnum sane. :-)18:43
*** muralia has joined #openstack-containers18:50
*** wznoinsk has quit IRC18:51
*** muralia1 has joined #openstack-containers18:52
*** manjeets has joined #openstack-containers18:53
*** muralia has quit IRC18:55
*** kebray has joined #openstack-containers18:55
*** fawadkhaliq has quit IRC18:57
*** fawadkhaliq has joined #openstack-containers18:58
*** rebase33 has quit IRC19:03
*** adrian_otto has quit IRC19:04
rlrossitI have something going wrong when trying to bring up both a swarm bay or a k8s bay... the heat stack always fails because it times out waiting for k8s or swarm to finish configuring. Is this something going wrong with my env, or are other people seeing this?19:06
*** kebray has quit IRC19:08
*** rpothier has joined #openstack-containers19:08
eghoborlrossit: i was tested swarm today from master works19:10
eghoboyou just need to use atomic 3 image19:10
rlrossiteghobo: is atomic 5 not working?19:10
eghobofor swarm no, for kub yes but it has broken docker19:11
rlrossithmm I guess I can try atomic 3 and see what happens. I'll report back with my findings19:12
rlrossitthanks19:12
vilobhmmhongbin : ping19:21
*** sthillma_ has joined #openstack-containers19:21
*** sthillma has quit IRC19:22
*** eghobo has quit IRC19:22
*** irenab_ has joined #openstack-containers19:25
*** hongbin has joined #openstack-containers19:25
*** sthillma_ has quit IRC19:26
*** irenab has quit IRC19:27
*** irenab_ is now known as irenab19:27
vilobhmmhongbin : hi19:32
hongbinvilobhmm: sup19:32
vilobhmmI was working on https://review.openstack.org/#/c/231219/ For bays that allow duplicate resource names, we will need to catch ambiguity and raise an exception indicating that a uuid just be specified because multiple resources exist with the same name.19:32
vilobhmmso i realized specifically here https://github.com/openstack/python-magnumclient/blob/master/magnumclient/openstack/common/apiclient/utils.py#L7419:33
vilobhmmour managers don’t have find method since they inherit from https://github.com/openstack/python-magnumclient/blob/master/magnumclient/v1/pods.py#L2819:33
*** adrian_otto has joined #openstack-containers19:34
vilobhmmright now they inherit from they should also inherit from19:34
vilobhmmhttps://github.com/openstack/python-magnumclient/blob/master/magnumclient/common/base.py#L4119:34
vilobhmmthey should also inherit from https://github.com/openstack/python-magnumclient/blob/master/magnumclient/openstack/common/apiclient/base.py#L22819:35
vilobhmmotherwise if some object tries to access https://github.com/openstack/python-magnumclient/blob/master/magnumclient/openstack/common/apiclient/utils.py#L74 it will fail !19:35
hongbinI think I am lost, but my suggestion is to make the uuid works first19:36
hongbinThen, worry about the name in the next patch, possibly after liberty19:36
vilobhmmI have the patch up for uuid/name combination19:37
hongbink19:37
vilobhmmadrian_otto suggested to throw exception if user specifies name (and lets say we have resources with same name in the bay)19:37
hongbinGenerally speaking, you need to forward the bay_ident to condcutor19:37
hongbinand let conductor figure out if it is a name or uuid, and invokes the right k8s remote call19:38
vilobhmmhongbin : I agree and will modify the object from bay pacthes19:38
*** gangil has quit IRC19:38
* adrian_otto nods19:38
hongbinagree with throwing exception with multiplce resources on the same name19:39
vilobhmmadrian_otto : throwing an exception when resource with same name exist will need quite some work because we are not inheriting from right managers at present19:39
vilobhmmand this needs to change for all the managers since they don’t support find at this moment19:40
*** exploreshaifali has quit IRC19:40
vilobhmmso for this patch can we restrict the change to just passing bay_identifier rather than bay_uuid. I will work on the exception patch seperately19:40
vilobhmmadrian_otto, hongbin : ^^19:40
hongbinvilobhmm: will the k8s api throw exceptions if multiple resources have the same name?19:41
vilobhmmI plan to do so at contoller level itself by querying the bay with a specific identifier19:43
vilobhmmhongbin : ^^19:43
vilobhmmfor bay_identifier19:43
vilobhmmk8s resources won’t have duplicate names or uuid19:44
vilobhmmthe name or uuid is unique19:44
vilobhmmhttp://kubernetes.io/third_party/swagger-ui/#!/api%2Fv1/createNamespacedPod19:45
hongbinI am thinking if k8s throw exceptions on duplicated name, we just need to wrap the native exception in Magnum19:46
hongbinThen, life is much easier19:46
vilobhmmswagger documnetation says “name (string, optional): Name must be unique within a namespace. “19:46
vilobhmmuid (string, optional): UID is the unique in time and space value for this object.19:46
hongbinvilobhmm: yes, sounds like k8s will throw exceptions for duplicated name/uuid19:47
adrian_ottoso in that case, the duplicated resource name is not a problem, because docker also requires unique names.19:48
adrian_ottowhat about Mesos?19:48
hongbinwe don't have mesos conductor yet19:48
hongbinso there is no mesos resources managed by magnum19:48
adrian_ottook, so we can cross that bridge later19:48
vilobhmmadrian_otto : yes…but worthwhile to open a bug an keep track of it for mesos19:49
adrian_ottoso vilobhmm, it sounds like we can table the duplicate resource concern for now.19:49
vilobhmmadrian_otto : sounds good…19:49
adrian_ottowell, I'd only want an open bug for that if we do enough research to know if mesos (combined with marathon in the way we configure it) allows resources with duplicate names.19:49
adrian_ottoif it does not, then we can consider this a non-issue until a COE shows up that allows name duplication19:50
vilobhmmadrian_otto : sure..makes sense19:50
vilobhmmWas a useful discussion…thanks hongbin, adrian_otto !19:50
adrian_ottoand we could implement the name-to-uuid logic in a way that simply returns the first match19:50
*** rebase33 has joined #openstack-containers19:51
adrian_ottoso in the event that there were multiples you could simply repeat the operation19:51
adrian_ottountil they were all gone19:51
adrian_otto(at least for delete)19:51
vilobhmmadrian_otto : we have that code here https://github.com/openstack/python-magnumclient/blob/master/magnumclient/openstack/common/apiclient/utils.py#L35 ; its a place holder as of now19:52
vilobhmmin future can be used to see if input is name (and if duplictae name exist prompts user to specifiy uuid since it will be unique)19:53
*** fawadkhaliq has quit IRC19:53
*** fawadkhaliq has joined #openstack-containers19:54
hongbinI have scanned thought the marathon API, it looks marathon doesn't support name at all (they use id only).19:54
hongbinhttps://mesosphere.github.io/marathon/docs/rest-api.html#get-v2-apps-appid19:54
vilobhmmhongbin : cool..thanks for checking it19:55
adrian_ottoawesome. I love problems that don't need to be solved!19:57
adrian_ottoignore them all!19:57
vilobhmmadrian_otto, hongbin : so takeaway from this discussion is https://review.openstack.org/#/c/231219/ change bay_uuid to bay_identifier since the conductors we have will accept unique name/uuid…we should be good with handling duplicates and need not focus on handling exception for duplictaes from magnum client side19:57
adrian_ottoyes19:57
vilobhmmadrian_otto, hongbin : thanks19:58
*** gangil has joined #openstack-containers20:00
*** gangil has quit IRC20:00
*** gangil has joined #openstack-containers20:00
openstackgerritVilobh Meshram proposed openstack/python-magnumclient: Pass bay_ident to k8s objects methods  https://review.openstack.org/23121920:01
*** rebase33 has quit IRC20:01
*** rebase33 has joined #openstack-containers20:03
openstackgerritVilobh Meshram proposed openstack/python-magnumclient: Pass bay_ident to k8s objects methods  https://review.openstack.org/23121920:05
*** kebray has joined #openstack-containers20:08
*** sdake has quit IRC20:27
*** sdake has joined #openstack-containers20:28
*** fawadkhaliq has quit IRC20:29
*** eghobo has joined #openstack-containers20:43
*** gangil has quit IRC20:43
*** manjeets has quit IRC20:43
*** amit213 has quit IRC20:43
*** juggler has quit IRC20:43
*** preilly has quit IRC20:43
*** gus has quit IRC20:43
*** ewindisch has quit IRC20:43
*** rsFF has quit IRC20:43
*** gangil has joined #openstack-containers20:46
*** manjeets has joined #openstack-containers20:46
*** amit213 has joined #openstack-containers20:46
*** juggler has joined #openstack-containers20:46
*** preilly has joined #openstack-containers20:46
*** gus has joined #openstack-containers20:46
*** ewindisch has joined #openstack-containers20:46
*** rsFF has joined #openstack-containers20:46
*** fawadkhaliq has joined #openstack-containers20:48
*** manjeets has quit IRC20:48
adrian_ottovilobhmm: https://review.openstack.org/231219 is approved for merge. Once it merges, please order a recheck on https://review.openstack.org/223367 so we can approve that for merge as well.20:49
openstackgerritSurojit Pathak proposed openstack/magnum: Need to populate is_admin in context from role  https://review.openstack.org/23170620:50
openstackgerritSurojit Pathak proposed openstack/magnum: Functional tests for magnum service  https://review.openstack.org/23115720:53
*** sdake has quit IRC20:53
*** rlrossit has quit IRC20:54
suro-patzhongbin: adrian_otto: If available, please review https://review.openstack.org/#/c/231706/20:57
hongbinsuro-patz: ack20:58
*** sdake has joined #openstack-containers20:58
suro-patzhongbin: thanks!20:58
*** rlrossit has joined #openstack-containers21:00
*** suro-patz has quit IRC21:01
*** suro-patz has joined #openstack-containers21:04
openstackgerritMerged openstack/python-magnumclient: Pass bay_ident to k8s objects methods  https://review.openstack.org/23121921:07
*** harshs has quit IRC21:07
*** fawadkhaliq has quit IRC21:07
*** fawadkhaliq has joined #openstack-containers21:08
*** harshs has joined #openstack-containers21:08
*** rlrossit has left #openstack-containers21:15
*** rpothier has quit IRC21:15
*** eggshell has quit IRC21:16
*** eggshell has joined #openstack-containers21:16
*** fawadk has joined #openstack-containers21:18
*** alex_klimov has joined #openstack-containers21:18
openstackgerritSurojit Pathak proposed openstack/magnum: Functional tests for magnum service  https://review.openstack.org/23115721:20
*** manjeets has joined #openstack-containers21:21
*** fawadkhaliq has quit IRC21:22
*** manjeets has left #openstack-containers21:36
*** annegentle has quit IRC21:43
*** banix has quit IRC21:44
*** harlowja has quit IRC21:46
*** eghobo_ has joined #openstack-containers21:48
*** annegentle has joined #openstack-containers21:50
*** eghobo has quit IRC21:51
*** julim has quit IRC21:53
*** harshs has quit IRC21:53
*** harshs has joined #openstack-containers21:56
*** harshs has quit IRC21:56
*** harlowja has joined #openstack-containers21:57
*** dave-mccowan has quit IRC21:57
*** harshs has joined #openstack-containers21:57
*** harshs has quit IRC21:57
*** fawadk has quit IRC22:01
*** suro-patz has quit IRC22:04
*** jruano has quit IRC22:04
*** suro-patz has joined #openstack-containers22:05
*** dimtruck is now known as zz_dimtruck22:07
*** annegentle has quit IRC22:08
*** ewindisch has quit IRC22:09
*** erw has quit IRC22:09
*** gangil has quit IRC22:09
*** adrian_otto has quit IRC22:13
*** alex_klimov has quit IRC22:20
*** sthillma has joined #openstack-containers22:21
*** gangil has joined #openstack-containers22:23
*** gangil has quit IRC22:23
*** gangil has joined #openstack-containers22:23
*** manjeets has joined #openstack-containers22:33
*** Marga_ has quit IRC22:33
*** annegentle has joined #openstack-containers22:35
*** Marga_ has joined #openstack-containers22:37
*** tbh has joined #openstack-containers22:39
*** VikasC has joined #openstack-containers22:44
*** ewindisch has joined #openstack-containers22:44
*** erw has joined #openstack-containers22:45
*** sthillma_ has joined #openstack-containers22:45
*** sthillma has quit IRC22:47
*** sthillma_ is now known as sthillma22:47
*** gangil has quit IRC22:52
*** gangil has joined #openstack-containers22:53
*** gangil has quit IRC22:53
*** gangil has joined #openstack-containers22:53
*** devkulkarni has quit IRC22:53
*** jwcroppe_ is now known as jwcroppe22:54
*** annegentle has quit IRC22:55
*** muralia has joined #openstack-containers22:55
*** annegentle has joined #openstack-containers22:57
*** muralia1 has quit IRC22:57
*** subscope has quit IRC22:59
eghobo_hongbin: question?23:04
hongbineghobo_: sure, but will leave at a few minutes.23:04
eghobo_no rush, about https://review.openstack.org/#/c/230878/23:05
eghobo_i would like to do what Tom suggested, but not sure where to start ;)23:05
hongbineghobo_: I am not familiar with project-config, but can explain what I know about htat23:07
hongbinMagnum defined jobs for the gate https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/magnum.yaml23:08
eghobo_it will be great23:08
hongbinthinking how to explain it logically23:11
*** manjeets has quit IRC23:13
hongbineghobo_: The file above specified the builder for the gate check: gate-functional-dsvm-magnum23:13
hongbineghobo_: For the checks that are not there, they are in macros: https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/macros.yaml23:14
hongbineghobo_: For example, here is the docs builder: https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/macros.yaml#L11923:14
hongbineghobo_: I guess you can modify the docs builder to do something you want23:15
hongbineghobo_: like run additional scripts, etc.23:15
hongbineghobo_: And finally, here is the place you can add a script for the builder to run https://github.com/openstack-infra/project-config/tree/56bae8258a923f783617ea07bbf60b3c5d93ef93/jenkins/scripts23:16
hongbineghobo_: I guess sdake is an expert of project-config, sdake correct me if I am wrong (I have to run now)23:17
sdakei know a bit but definately ont an expert23:17
hongbin:)23:18
*** hongbin has quit IRC23:18
eghobo_hongbin: thx it's enough to start23:19
*** harshs has joined #openstack-containers23:21
*** sthillma_ has joined #openstack-containers23:24
*** sthillma has quit IRC23:26
*** sthillma_ is now known as sthillma23:26
*** agireud has quit IRC23:32
*** dane_leblanc has quit IRC23:32
*** eggshell has quit IRC23:32
*** achanda_ has joined #openstack-containers23:36
*** achanda has quit IRC23:40
*** achanda_ has quit IRC23:41
*** sdake_ has joined #openstack-containers23:41
*** jruano has joined #openstack-containers23:42
*** kebray has quit IRC23:43
*** sdake has quit IRC23:43
*** devkulkarni has joined #openstack-containers23:45
*** annegentle has quit IRC23:46
*** irenab has quit IRC23:48
*** dave-mccowan has joined #openstack-containers23:51
*** sdake_ has quit IRC23:52

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