| *** kitch_ has joined #openstack-chef | 00:03 | |
| *** kitch_ has quit IRC | 00:03 | |
| *** otter768 has joined #openstack-chef | 00:42 | |
| *** otter768 has quit IRC | 00:47 | |
| *** zhiwei has quit IRC | 01:02 | |
| *** zhiwei has joined #openstack-chef | 01:18 | |
| *** ncerny has joined #openstack-chef | 02:13 | |
| *** otter768 has joined #openstack-chef | 02:43 | |
| *** ncerny has quit IRC | 02:45 | |
| openstackgerrit | Ma Wen Cheng proposed stackforge/cookbook-openstack-bare-metal: Update the deprecated group/name to Kilo release https://review.openstack.org/182561 | 02:45 |
|---|---|---|
| *** otter768 has quit IRC | 02:47 | |
| *** mitz has quit IRC | 02:52 | |
| *** wenchma has joined #openstack-chef | 03:43 | |
| *** libsysguy has joined #openstack-chef | 03:53 | |
| *** stevemar has joined #openstack-chef | 04:01 | |
| *** libsysguy has quit IRC | 04:23 | |
| *** pradipta has joined #openstack-chef | 04:26 | |
| *** otter768 has joined #openstack-chef | 04:44 | |
| *** stevemar has quit IRC | 04:46 | |
| *** otter768 has quit IRC | 04:49 | |
| *** yym has joined #openstack-chef | 04:57 | |
| *** libsysguy has joined #openstack-chef | 05:03 | |
| *** libsysguy has quit IRC | 05:57 | |
| *** ChrissKO has quit IRC | 06:09 | |
| *** wojdev has joined #openstack-chef | 06:11 | |
| openstackgerrit | Kasey Alusi proposed stackforge/cookbook-openstack-identity: Check whether to recreate service if it exists https://review.openstack.org/177609 | 06:32 |
| *** pradipta has quit IRC | 06:32 | |
| openstackgerrit | Kasey Alusi proposed stackforge/cookbook-openstack-identity: Check whether to recreate service if it exists https://review.openstack.org/177609 | 06:35 |
| *** otter768 has joined #openstack-chef | 06:45 | |
| *** otter768 has quit IRC | 06:50 | |
| *** wojdev has quit IRC | 07:01 | |
| *** wojdev has joined #openstack-chef | 07:03 | |
| *** yym has quit IRC | 07:14 | |
| *** Guest19895 has quit IRC | 07:21 | |
| *** wojdev has quit IRC | 07:25 | |
| *** chlong has quit IRC | 07:32 | |
| openstackgerrit | Yi Ming Yin proposed stackforge/cookbook-openstack-block-storage: Test demo for gerrit https://review.openstack.org/183428 | 08:01 |
| *** yym has joined #openstack-chef | 08:04 | |
| *** wojdev has joined #openstack-chef | 08:04 | |
| *** wojdev has quit IRC | 08:08 | |
| *** yym has quit IRC | 08:37 | |
| *** yym has joined #openstack-chef | 08:38 | |
| *** otter768 has joined #openstack-chef | 08:46 | |
| *** otter768 has quit IRC | 08:50 | |
| *** higgins has quit IRC | 08:58 | |
| *** higgins has joined #openstack-chef | 09:10 | |
| *** yym has quit IRC | 09:12 | |
| *** wojdev has joined #openstack-chef | 10:02 | |
| *** wenchma has quit IRC | 10:03 | |
| *** wojdev has quit IRC | 10:04 | |
| *** wojdev has joined #openstack-chef | 10:05 | |
| *** wojdev has quit IRC | 10:05 | |
| *** wojdev has joined #openstack-chef | 10:07 | |
| *** wojdev has quit IRC | 10:11 | |
| *** wojdev has joined #openstack-chef | 10:12 | |
| *** zhiwei has quit IRC | 10:18 | |
| *** wojdev has quit IRC | 10:19 | |
| *** wojdev has joined #openstack-chef | 10:21 | |
| *** wojdev has quit IRC | 10:29 | |
| *** wojdev has joined #openstack-chef | 10:30 | |
| *** wojdev has quit IRC | 10:32 | |
| *** wojdev has joined #openstack-chef | 10:33 | |
| *** otter768 has joined #openstack-chef | 10:47 | |
| *** rtheis has joined #openstack-chef | 10:49 | |
| *** otter768 has quit IRC | 10:51 | |
| *** wojdev has quit IRC | 11:49 | |
| *** ogny has joined #openstack-chef | 11:53 | |
| *** ncerny has joined #openstack-chef | 12:46 | |
| *** otter768 has joined #openstack-chef | 12:48 | |
| *** otter768 has quit IRC | 12:52 | |
| *** zhiwei has joined #openstack-chef | 13:03 | |
| *** lastops has joined #openstack-chef | 13:25 | |
| *** libsysguy has joined #openstack-chef | 13:48 | |
| openstackgerrit | Mark Vanderwiel proposed stackforge/openstack-chef-repo: Only clone repo and patch with depth of 1 https://review.openstack.org/183530 | 14:01 |
| *** libsysguy has quit IRC | 14:05 | |
| *** stevemar has joined #openstack-chef | 14:11 | |
| *** yym has joined #openstack-chef | 14:20 | |
| *** wojdev has joined #openstack-chef | 14:31 | |
| *** yym has quit IRC | 14:41 | |
| *** otter768 has joined #openstack-chef | 14:49 | |
| *** otter768 has quit IRC | 14:53 | |
| *** ogny has quit IRC | 15:00 | |
| *** ncerny has quit IRC | 15:18 | |
| *** wojdev has quit IRC | 15:47 | |
| *** wojdev has joined #openstack-chef | 15:53 | |
| *** zhiwei has left #openstack-chef | 15:54 | |
| *** shakamunyi has quit IRC | 16:00 | |
| *** barra204 has quit IRC | 16:00 | |
| *** wojdev has quit IRC | 16:16 | |
| *** kaseyalusi has joined #openstack-chef | 16:38 | |
| kaseyalusi | markvan: poasted a comment regarding https://review.openstack.org/#/c/177609/8/providers/register.rb | 16:42 |
| markvan | kaseyalusi: k | 16:42 |
| markvan | kaseyalusi: humm, I'm reading it differently. the endpoint code is passing in the 'service_id', service_uuid and then doing that as part of the checks on the search_uuid(output, uuid_field, key => value, | 16:44 |
| markvan | so, I thjink the same applies to services | 16:44 |
| *** otter768 has joined #openstack-chef | 16:49 | |
| kaseyalusi | But in search_uuid(output, uuid_field, for service_need_updated, the uuid_field param is the service id | 16:51 |
| kaseyalusi | whereas for endpoint, that uuid_field is the endpoint uuid | 16:51 |
| kaseyalusi | service list does not have a field called service_id, it is just id (== uuid_field) | 16:53 |
| *** otter768 has quit IRC | 16:55 | |
| *** Guest77307 has joined #openstack-chef | 17:17 | |
| *** Guest77307 is now known as redrobot | 17:20 | |
| markvan | kaseyalusi: yeah, I was thinking that just the "id" field would be used to allow the other columns to be checked. | 17:25 |
| *** redrobot has left #openstack-chef | 17:26 | |
| *** redrobot has joined #openstack-chef | 17:26 | |
| markvan | just using name and type seems like it might not be enough, as I think you cna have dup names, but no IDs | 17:26 |
| *** redrobot has quit IRC | 17:29 | |
| kaseyalusi | I think i am getting confused now... If the id field is not the same, the search_uuid will not return false anyways, it checks that field by default, right? | 17:31 |
| kaseyalusi | will return* | 17:31 |
| markvan | checking | 17:31 |
| markvan | no I don't think search_uuid will check for anything unless it's passed in explicitly , that's why I think we need to pass that uuid in so where on the right service row in the table | 17:33 |
| kaseyalusi | It seems like if thats the case, endpoint_need_updated? should be passing in its 'id' field as well | 17:34 |
| *** redrobot has joined #openstack-chef | 17:36 | |
| *** redrobot is now known as Guest63193 | 17:36 | |
| kaseyalusi | Also, the new_resource object doesnt have an id field, right? since that is automatically generated? | 17:38 |
| *** Guest63193 is now known as redrobot | 17:39 | |
| markvan | yeah, for the endpoint it's checking the other fields and not the id, cause like you stated, it's only there if it exists. | 17:41 |
| markvan | So, I thnk what you have might be close, but since there is only the name and type fields for a service, and type was used to search for the UUID, that only leaves the name, right? | 17:42 |
| markvan | so, probably your line 121, does not really need to check the type again, it will always be a match. | 17:43 |
| kaseyalusi | Actually, looking at that, if there are many compute services, this would delete one if the name was different, which it would be for multiple hypervisors. Seems like the first check should be on name? | 17:45 |
| kaseyalusi | I just mixed up nova service list there... | 17:52 |
| kaseyalusi | Is there a case where an operator would want to have two services of the same type with different names? If so, this would not allow them to do that. | 17:57 |
| openstackgerrit | Kasey Alusi proposed stackforge/cookbook-openstack-identity: Check whether to recreate service if it exists https://review.openstack.org/177609 | 18:00 |
| markvan | kaseyalusi: yeah, not sure is there's ever a need for same service type with different names, seems redundent | 18:04 |
| kaseyalusi | Ok, that makes sense. Then in that case, I think that recent patch set should fix that double checking. Thank you for the help, by the way! | 18:04 |
| markvan | kaseyalusi: np, yup that latest ps looks good. I'm going to run it thru a repo aio_test , and then try to force the case where the service name is diff to see this in action. | 18:10 |
| *** openstackgerrit has quit IRC | 18:22 | |
| *** openstackgerrit has joined #openstack-chef | 18:22 | |
| *** shakamunyi has joined #openstack-chef | 18:37 | |
| *** barra204 has joined #openstack-chef | 18:37 | |
| *** ChrissKO_ has joined #openstack-chef | 18:47 | |
| *** stevemar has quit IRC | 18:50 | |
| *** otter768 has joined #openstack-chef | 18:50 | |
| *** otter768 has quit IRC | 18:55 | |
| *** stevemar has joined #openstack-chef | 18:59 | |
| cmluciano | anyone ever see the knife openstack plugin hang on “waiting for sshd”? | 19:06 |
| *** barra204 has quit IRC | 19:48 | |
| *** shakamunyi has quit IRC | 19:48 | |
| *** thumpba has joined #openstack-chef | 19:54 | |
| *** DaveIshmael has joined #openstack-chef | 20:03 | |
| *** lastops has quit IRC | 20:21 | |
| *** otter768 has joined #openstack-chef | 20:51 | |
| *** otter768 has quit IRC | 20:56 | |
| *** aspiers has joined #openstack-chef | 21:12 | |
| *** stevemar has quit IRC | 21:17 | |
| *** os-chef-bot has quit IRC | 21:26 | |
| *** os-chef-bot has joined #openstack-chef | 21:26 | |
| *** rtheis has quit IRC | 21:36 | |
| *** kaseyalusi has quit IRC | 22:01 | |
| *** kaseyalusi has joined #openstack-chef | 22:24 | |
| *** otter768 has joined #openstack-chef | 22:52 | |
| *** otter768 has quit IRC | 22:56 | |
| *** ChrissKO_ has quit IRC | 22:59 | |
| *** ChrissKO_ has joined #openstack-chef | 23:00 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!