| *** wchrisj has quit IRC | 00:08 | |
| *** mfer has joined #openstack-sdks | 00:16 | |
| *** mfer has quit IRC | 00:16 | |
| *** wchrisj has joined #openstack-sdks | 00:52 | |
| *** samchoi has quit IRC | 01:07 | |
| *** wchrisj has quit IRC | 01:25 | |
| *** wchrisj has joined #openstack-sdks | 02:00 | |
| *** wchrisj has quit IRC | 02:04 | |
| *** rgbkrk has joined #openstack-sdks | 03:07 | |
| *** VeggieMeat has joined #openstack-sdks | 03:39 | |
| *** rgbkrk has quit IRC | 04:30 | |
| *** wchrisj has joined #openstack-sdks | 05:00 | |
| *** wchrisj has quit IRC | 05:37 | |
| *** jamieh has joined #openstack-sdks | 10:05 | |
| *** jamieh is now known as Guest72786 | 10:05 | |
| *** krames has joined #openstack-sdks | 12:42 | |
| *** krames has quit IRC | 12:50 | |
| *** krames has joined #openstack-sdks | 12:51 | |
| *** mfer has joined #openstack-sdks | 13:22 | |
| *** bknudson has joined #openstack-sdks | 13:32 | |
| *** Guest72786 is now known as jamie_h | 13:44 | |
| *** wchrisj has joined #openstack-sdks | 13:54 | |
| *** mhagedorn has joined #openstack-sdks | 14:12 | |
| shaunak | @mfer @samchoi @jamie_h php sdk meeting starting in 5 min at https://plus.google.com/hangouts/_/calendar/ZGtucGdkY2tlb3BzZGNycWZyMXNxMnRtMXNAZ3JvdXAuY2FsZW5kYXIuZ29vZ2xlLmNvbQ.hm0g5j4n6jmk95jf8pc1g9sn7o | 15:25 |
|---|---|---|
| *** sharwell__ has quit IRC | 15:26 | |
| *** jamie_h has quit IRC | 16:11 | |
| *** jamieh has joined #openstack-sdks | 16:13 | |
| *** jamieh is now known as jamie_h | 16:14 | |
| jamie_h | shaunak mfer samchoi - sorry, my mac had a meltdown | 16:14 |
| *** jdag___ is now known as jdag | 17:02 | |
| *** samchoi has joined #openstack-sdks | 17:06 | |
| *** jamielennox is now known as jamielennox|away | 17:07 | |
| wchrisj | krames: yo | 17:48 |
| krames | hey Chris | 17:49 |
| wchrisj | hey,got a sec? | 17:49 |
| krames | sure | 17:49 |
| wchrisj | question about the underlying model implementation in fog-core; I might have stumbled onto a bug | 17:49 |
| krames | okay | 17:50 |
| wchrisj | in user.rb, we use this to update a user: | 17:50 |
| wchrisj | service.update_user(self.id, attributes) | 17:50 |
| wchrisj | however, with the attr_accessor in play: | 17:51 |
| wchrisj | attr_accessor :name, :tenant_id, :password, :email, :enabled | 17:51 |
| wchrisj | if you call "attributes", it always returns just the id paam | 17:51 |
| wchrisj | nothing else | 17:51 |
| krames | yeah | 17:51 |
| wchrisj | if I comment out the attr_accessor, it returns ALL the attributes | 17:51 |
| wchrisj | what's going on? seems backwards | 17:52 |
| wchrisj | or am I missing something? | 17:52 |
| krames | I bet the attr_accessor line is overriding the methods generated by the attribute line | 17:52 |
| krames | you should only use attr_accessor for transient data | 17:52 |
| krames | everything that might be coming from the service should use attribute | 17:53 |
| wchrisj | interesting... | 17:53 |
| wchrisj | anything we can do to prevent this from hanging up someone else? | 17:53 |
| wchrisj | I just wasted 1+ hours debugging this | 17:54 |
| wchrisj | very innocuous | 17:54 |
| krames | aside from trying to overload attr_accessor | 17:54 |
| wchrisj | subtle | 17:54 |
| krames | to check to see if those methods have been declared already | 17:54 |
| wchrisj | ahh - in the underlying model itself? | 17:54 |
| krames | not crazy about that though | 17:55 |
| krames | because it could cause problems for code outside of fog | 17:55 |
| krames | do you think better docs would have helped? | 17:55 |
| wchrisj | maybe a generator for a model? | 17:56 |
| wchrisj | with gotchas like this documented INSIDE the output | 17:56 |
| wchrisj | ? | 17:57 |
| krames | I was thinking about that too | 18:04 |
| krames | I think generators were one of the topics for the summit | 18:05 |
| *** krames_ has joined #openstack-sdks | 18:20 | |
| *** krames has quit IRC | 18:23 | |
| wchrisj | krames: I just finished all of my spec updates - would you mind merging that PR? | 18:34 |
| wchrisj | elight: OK by you? | 18:35 |
| elight | wchrisj: The PR we just reviewed? Lemme double check real fast. | 18:35 |
| wchrisj | coolio | 18:35 |
| wchrisj | elight: you DID mention moving this repo to fog account, right? | 18:37 |
| elight | wchrisj: Indeed, to the fog org | 18:37 |
| wchrisj | yep | 18:37 |
| krames_ | https://github.com/fog/fog/issues/2769 | 18:37 |
| wchrisj | exactly | 18:38 |
| wchrisj | now is prob a good time to do that before we open this new PR | 18:38 |
| elight | wchrisj: Good to go | 18:41 |
| *** jamie_h has quit IRC | 18:49 | |
| wchrisj | thanks elight: | 18:56 |
| *** jamieh has joined #openstack-sdks | 18:56 | |
| *** jamieh is now known as Guest719 | 18:57 | |
| *** Guest719 has quit IRC | 18:58 | |
| *** mhagedorn has left #openstack-sdks | 20:00 | |
| *** rgbkrk has joined #openstack-sdks | 20:05 | |
| *** krames_ has quit IRC | 20:09 | |
| mfer | shaunak are you using this handle instead now? | 20:22 |
| terrylhowe | aka ycombinator | 20:33 |
| *** kc5tja has left #openstack-sdks | 20:38 | |
| *** jamielennox|away is now known as jamielennox | 20:51 | |
| *** mfer has quit IRC | 21:30 | |
| *** sharwell__ has joined #openstack-sdks | 21:38 | |
| *** dhellmann is now known as dhellmann_ | 21:48 | |
| *** edleafe has left #openstack-sdks | 21:49 | |
| *** edleafe has joined #openstack-sdks | 21:49 | |
| *** bknudson has quit IRC | 22:24 | |
| wchrisj | elight: krames: https://wiki.openstack.org/wiki/VersionDiscovery | 22:33 |
| *** bknudson has joined #openstack-sdks | 23:37 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!