)]}' { "log": [ { "commit": "a739f2290078be903c47054bf89faef361e42732", "tree": "ec80a3c552dfee56f38eb9387914747e07b93590", "parents": [ "838f42daaf2297dc2e52d99fbc28b6ead94ff4dd" ], "author": { "name": "Mark Peek", "email": "mark@peek.org", "time": "Sat Aug 17 19:06:15 2013 -0700" }, "committer": { "name": "Mark Peek", "email": "mark@peek.org", "time": "Sat Aug 17 19:06:15 2013 -0700" }, "message": "Fix quote in servers.go to include omitentry\n" }, { "commit": "838f42daaf2297dc2e52d99fbc28b6ead94ff4dd", "tree": "c983bcfdad61dba88e9473b298cc3c4f701d082a", "parents": [ "b82f3d66468c3b6bc54a0844d6215c900b208b85", "f339160052f04fbc9e0fe0ec17f7640a9d2fc106" ], "author": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Wed Aug 14 14:54:45 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Wed Aug 14 14:54:45 2013 -0700" }, "message": "Merge pull request #75 from rackspace/list-addresses\n\nList addresses" }, { "commit": "f339160052f04fbc9e0fe0ec17f7640a9d2fc106", "tree": "c983bcfdad61dba88e9473b298cc3c4f701d082a", "parents": [ "e21808f79ea9e6249d0294e6d535cee5c11f3676" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Wed Aug 14 14:53:32 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Wed Aug 14 14:53:32 2013 -0700" }, "message": "go fmt\n" }, { "commit": "e21808f79ea9e6249d0294e6d535cee5c11f3676", "tree": "ac581efd2c5a208cbd14148e5d7fdb296eb348cd", "parents": [ "b82f3d66468c3b6bc54a0844d6215c900b208b85" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Wed Aug 14 14:48:09 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Wed Aug 14 14:48:09 2013 -0700" }, "message": "Add ListAddresses method for cloud servers\n" }, { "commit": "b82f3d66468c3b6bc54a0844d6215c900b208b85", "tree": "2389512051b857347272f41e397e70107769a7c3", "parents": [ "cccca1a2714f41b24d526cbcef4ff840c8d1c9d5" ], "author": { "name": "Jesse Noller", "email": "jesse.noller@rackspace.com", "time": "Tue Aug 13 14:11:24 2013 -0500" }, "committer": { "name": "Jesse Noller", "email": "jesse.noller@rackspace.com", "time": "Tue Aug 13 14:11:24 2013 -0500" }, "message": "Added missing license, contributors file\n" }, { "commit": "cccca1a2714f41b24d526cbcef4ff840c8d1c9d5", "tree": "c4cf76d7ab9292996d8a5faeb6ec508ed6e1ada3", "parents": [ "e79d8a6383365c1e3a4babefb5878f10d2fbb2e7", "6c2efcc125aa328f10dd11de4ca987d539c58551" ], "author": { "name": "Jesse Noller", "email": "jnoller@gmail.com", "time": "Tue Aug 13 09:05:02 2013 -0700" }, "committer": { "name": "Jesse Noller", "email": "jnoller@gmail.com", "time": "Tue Aug 13 09:05:02 2013 -0700" }, "message": "Merge pull request #72 from philips/patch-1\n\nREADME: remove notes about private/public workspace scripts" }, { "commit": "6c2efcc125aa328f10dd11de4ca987d539c58551", "tree": "c4cf76d7ab9292996d8a5faeb6ec508ed6e1ada3", "parents": [ "e79d8a6383365c1e3a4babefb5878f10d2fbb2e7" ], "author": { "name": "Brandon Philips", "email": "brandon@ifup.org", "time": "Tue Aug 13 08:08:03 2013 -0700" }, "committer": { "name": "Brandon Philips", "email": "brandon@ifup.org", "time": "Tue Aug 13 08:08:03 2013 -0700" }, "message": "README: remove notes about private/public workspace scripts\n\nSince this project is now announced I assume these notes can go away." }, { "commit": "e79d8a6383365c1e3a4babefb5878f10d2fbb2e7", "tree": "688a42beb2c5cabbdaaad45d035306acd3dcee1c", "parents": [ "8a3a6e8d9c453b3f22eacaf7df1bc112772234c4", "414c15c7ca6f6f2e3fa8374a91b1dff0961dc5c7" ], "author": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Thu Aug 01 15:18:50 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Thu Aug 01 15:18:50 2013 -0700" }, "message": "Merge pull request #65 from rackspace/rebuild-server-for-real\n\nImplement Rebuild server functionality." }, { "commit": "414c15c7ca6f6f2e3fa8374a91b1dff0961dc5c7", "tree": "688a42beb2c5cabbdaaad45d035306acd3dcee1c", "parents": [ "8a3a6e8d9c453b3f22eacaf7df1bc112772234c4" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Aug 01 15:16:46 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Aug 01 15:16:46 2013 -0700" }, "message": "Implement Rebuild server functionality.\n\nI thought I\u0027d had this in already. Apparently not.\n" }, { "commit": "8a3a6e8d9c453b3f22eacaf7df1bc112772234c4", "tree": "debf5fb724e30937b21722acbc6509c8bc6f25ed", "parents": [ "8c1a57acfc1637a87f1346f4aad436bc27919f97", "cb9eca6c84837c896cb27cbaf0975a17acbcb62c" ], "author": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Wed Jul 31 15:51:04 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Wed Jul 31 15:51:04 2013 -0700" }, "message": "Merge pull request #64 from rackspace/update-docs\n\nUpdate docs" }, { "commit": "cb9eca6c84837c896cb27cbaf0975a17acbcb62c", "tree": "debf5fb724e30937b21722acbc6509c8bc6f25ed", "parents": [ "20f1aa47a35e57b6ed9782b31b90dadecca46cba" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Wed Jul 31 14:33:33 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Wed Jul 31 14:33:33 2013 -0700" }, "message": "Fix parameter omission in acceptance test\n" }, { "commit": "20f1aa47a35e57b6ed9782b31b90dadecca46cba", "tree": "2d721a379ab36e46307f0ec774b2b6e831bf2aad", "parents": [ "94761be7c65ab6256c477725eaf86f21713ce273" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Wed Jul 31 14:32:03 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Wed Jul 31 14:32:03 2013 -0700" }, "message": "go fmt reformatting\n" }, { "commit": "94761be7c65ab6256c477725eaf86f21713ce273", "tree": "a132dea4174bcf424e46c9e271c85f979a7318e3", "parents": [ "8c1a57acfc1637a87f1346f4aad436bc27919f97" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Wed Jul 31 14:31:17 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Wed Jul 31 14:31:17 2013 -0700" }, "message": "Update godoc documentation\n" }, { "commit": "8c1a57acfc1637a87f1346f4aad436bc27919f97", "tree": "85eee9b31e59b5ff99f5f8ecb414039594361a23", "parents": [ "e05d114303e6688841bc0d996f271050df3ddab6", "72ac2ddf193b5199b08f0ae7736fd6f6fed11e11" ], "author": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Wed Jul 31 13:47:19 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Wed Jul 31 13:47:19 2013 -0700" }, "message": "Merge pull request #63 from rackspace/update-server\n\nAdd update server capability." }, { "commit": "72ac2ddf193b5199b08f0ae7736fd6f6fed11e11", "tree": "85eee9b31e59b5ff99f5f8ecb414039594361a23", "parents": [ "e05d114303e6688841bc0d996f271050df3ddab6" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Wed Jul 31 13:45:05 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Wed Jul 31 13:45:28 2013 -0700" }, "message": "Add update server capability.\n\nThis requires an updated version of Perigee, for it lacked a Put()\nmethod. Make sure you \"go get -u github.com/racker/perigee\" prior to\ntesting, or you\u0027ll get method not defined errors.\n" }, { "commit": "e05d114303e6688841bc0d996f271050df3ddab6", "tree": "e75a1bbc81289cce15141f892cf4a9fe025c5f80", "parents": [ "530609ade69e82fa57589c1ffd8b052ff1f28f48", "15da6ab911a2885b599fe92a141cef1524bec91b" ], "author": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Tue Jul 30 14:05:06 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Tue Jul 30 14:05:06 2013 -0700" }, "message": "Merge pull request #62 from rackspace/rescue-servers\n\nExpose rescue/unrescue functionality." }, { "commit": "15da6ab911a2885b599fe92a141cef1524bec91b", "tree": "e75a1bbc81289cce15141f892cf4a9fe025c5f80", "parents": [ "530609ade69e82fa57589c1ffd8b052ff1f28f48" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jul 30 14:02:11 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jul 30 14:02:11 2013 -0700" }, "message": "Expose rescue/unrescue functionality.\n" }, { "commit": "530609ade69e82fa57589c1ffd8b052ff1f28f48", "tree": "0c0d821f43b2293d6d73992d4ec366fa8d0c5f56", "parents": [ "c8b11d197d04e843d5c52215f078bd16ac6d502a", "adbecf929e9da9d7d0ff90656d24290e19c74a19" ], "author": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Tue Jul 30 13:14:45 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Tue Jul 30 13:14:45 2013 -0700" }, "message": "Merge pull request #61 from rackspace/reboot-server\n\nExpose reboot server support." }, { "commit": "adbecf929e9da9d7d0ff90656d24290e19c74a19", "tree": "0c0d821f43b2293d6d73992d4ec366fa8d0c5f56", "parents": [ "c8b11d197d04e843d5c52215f078bd16ac6d502a" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jul 30 13:13:59 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jul 30 13:13:59 2013 -0700" }, "message": "Expose reboot server support.\n" }, { "commit": "c8b11d197d04e843d5c52215f078bd16ac6d502a", "tree": "975b7161bee28f8b5bed68691cd8fbcbe34be25b", "parents": [ "102b7df05e6356496b482ee77e7d7aef46f7dbc4", "bc6812399df8d424a7e952aa16ec016da45e12ff" ], "author": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Tue Jul 30 12:04:35 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Tue Jul 30 12:04:35 2013 -0700" }, "message": "Merge pull request #60 from rackspace/fix-list-flavors\n\nReturn details with list of flavors." }, { "commit": "bc6812399df8d424a7e952aa16ec016da45e12ff", "tree": "975b7161bee28f8b5bed68691cd8fbcbe34be25b", "parents": [ "102b7df05e6356496b482ee77e7d7aef46f7dbc4" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jul 30 12:01:13 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jul 30 12:01:13 2013 -0700" }, "message": "Return details with list of flavors.\n" }, { "commit": "102b7df05e6356496b482ee77e7d7aef46f7dbc4", "tree": "ff8f4b5d9eba7ab8bdf3bb57c3c1528044cf1145", "parents": [ "05aadc303cdaaf3d74ed3fd5b61a966e20ec2ffe", "9b23b343bc6e109952313cbdbc728dfa7bd66e61" ], "author": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Tue Jul 30 11:59:21 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Tue Jul 30 11:59:21 2013 -0700" }, "message": "Merge pull request #59 from rackspace/fix-list-images\n\nDeliver details with list images results" }, { "commit": "9b23b343bc6e109952313cbdbc728dfa7bd66e61", "tree": "ff8f4b5d9eba7ab8bdf3bb57c3c1528044cf1145", "parents": [ "05aadc303cdaaf3d74ed3fd5b61a966e20ec2ffe" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jul 30 11:58:16 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jul 30 11:58:16 2013 -0700" }, "message": "Deliver details with list images results\n" }, { "commit": "05aadc303cdaaf3d74ed3fd5b61a966e20ec2ffe", "tree": "afea286b4a784e86fb210756c2acf301cd680fa5", "parents": [ "3fdd9470ba633eafc6059ab8bc4642cd35f3f454", "887d780069d6c439c1ae8a7435366f893a40fcfc" ], "author": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Mon Jul 29 17:57:12 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Mon Jul 29 17:57:12 2013 -0700" }, "message": "Merge pull request #58 from rackspace/more-debt-reduction\n\nMore technical debt payoff." }, { "commit": "887d780069d6c439c1ae8a7435366f893a40fcfc", "tree": "afea286b4a784e86fb210756c2acf301cd680fa5", "parents": [ "3fdd9470ba633eafc6059ab8bc4642cd35f3f454" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Mon Jul 29 17:55:37 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Mon Jul 29 17:55:37 2013 -0700" }, "message": "More technical debt payoff.\n\nApplied refactoring across the entire acceptance test code base.\n" }, { "commit": "3fdd9470ba633eafc6059ab8bc4642cd35f3f454", "tree": "5eb049ebcbc6630b78e52568cb1e4114aec00b0d", "parents": [ "d1b17c46dd69d6c6e3427861afcd2fa500cbe707", "002b65141b5432b9602660d6c13069ce7ce827d0" ], "author": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Mon Jul 29 16:35:40 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Mon Jul 29 16:35:40 2013 -0700" }, "message": "Merge pull request #57 from rackspace/fix-get-server-details-test\n\nCreate server to get details of if no other available." }, { "commit": "002b65141b5432b9602660d6c13069ce7ce827d0", "tree": "5eb049ebcbc6630b78e52568cb1e4114aec00b0d", "parents": [ "d1b17c46dd69d6c6e3427861afcd2fa500cbe707" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Mon Jul 29 16:30:40 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Mon Jul 29 16:30:40 2013 -0700" }, "message": "Create server to get details of if no other available.\n\nIf you attempted to run 03-get-server-details.go on a cloud account\nwithout a pre-existing cloud server, it would fail. Clearly, this isn\u0027t\nintended, as it doesn\u0027t exercise the get-server-details functionality.\nI alter the code, as a last resort, create a new server.\n\nWe check first to see if a -i parameter has been set, and if so, we use\nit directly. Otherwise, we attempt to list servers available for the\nuser, and if at least one exists, we pick the first one we find. Only\nif those two conditions fail do we attempt to create a new server.\n" }, { "commit": "d1b17c46dd69d6c6e3427861afcd2fa500cbe707", "tree": "f651cf03d6f0c0ff62120fea4e5316316b0576a9", "parents": [ "ab1cc1645d94c62fd41ec78cdbec566bcc9067de", "41856d6f0f9d85ed8d9d1992f06b310ec9cf610e" ], "author": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Mon Jul 29 15:51:37 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Mon Jul 29 15:51:37 2013 -0700" }, "message": "Merge pull request #56 from rackspace/tech-debt-reduction\n\nSimplify a few acceptance tests for illustration." }, { "commit": "ab1cc1645d94c62fd41ec78cdbec566bcc9067de", "tree": "c6f00a0a2a85b335536214ba20e9839d0e5842b0", "parents": [ "d7ffabf11e8b5bd428fa41c04cf91bfb9fc14523", "f722dbf282581dac51c277577e5f25654e8faa77" ], "author": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Mon Jul 29 15:49:51 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Mon Jul 29 15:49:51 2013 -0700" }, "message": "Merge pull request #55 from rackspace/tech-debt-reduction\n\nRefactor common initialization sequences." }, { "commit": "41856d6f0f9d85ed8d9d1992f06b310ec9cf610e", "tree": "f651cf03d6f0c0ff62120fea4e5316316b0576a9", "parents": [ "f722dbf282581dac51c277577e5f25654e8faa77" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Mon Jul 29 15:48:22 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Mon Jul 29 15:48:22 2013 -0700" }, "message": "Simplify a few acceptance tests for illustration.\n\nExpect a big-bang PR following this one. These two changes illustrate\nhow to use the newly refactored code.\n" }, { "commit": "f722dbf282581dac51c277577e5f25654e8faa77", "tree": "c6f00a0a2a85b335536214ba20e9839d0e5842b0", "parents": [ "d7ffabf11e8b5bd428fa41c04cf91bfb9fc14523" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Mon Jul 29 15:44:30 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Mon Jul 29 15:44:30 2013 -0700" }, "message": "Refactor common initialization sequences.\n\nI\u0027ve been grossly violating the DRY principle for some time, in the name\nof coding velocity. After 9 or so acceptance tests, it\u0027s tiresome,\nerror-prone, and inconvenient to have to cut-and-paste the same\ninitialization code between modules. The time is now to fix this,\nbefore I write any more acceptance tests. And, now that I have an\ninterface I\u0027m happy with, writing future acceptance tests should go much\nfaster than even before.\n" }, { "commit": "d7ffabf11e8b5bd428fa41c04cf91bfb9fc14523", "tree": "78f757bcb7ee52679e5cd91612b29f435ca06c47", "parents": [ "1782218478568a0bd11f183b51c0ee00154af23d", "8512e9aabbc6fc3327dcdb5946b9718f70cf6673" ], "author": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Fri Jul 26 22:58:45 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Fri Jul 26 22:58:45 2013 -0700" }, "message": "Merge pull request #54 from rackspace/resize-server\n\nAdd resize server action." }, { "commit": "8512e9aabbc6fc3327dcdb5946b9718f70cf6673", "tree": "78f757bcb7ee52679e5cd91612b29f435ca06c47", "parents": [ "1782218478568a0bd11f183b51c0ee00154af23d" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Fri Jul 26 22:53:29 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Fri Jul 26 22:53:29 2013 -0700" }, "message": "Add resize server action.\n\nThis looks like it should do the trick. Intermittently, however, some\nservers created by the acceptance test will end up in an error state.\nIt is not consistently reproducible. When it does work, however, it\nworks as expected. I\u0027m not sure why the behavior is variable.\n" }, { "commit": "1782218478568a0bd11f183b51c0ee00154af23d", "tree": "4eccf45253137302c0873f6b061433d98d5c5f46", "parents": [ "a87758a43113f31ea962cceca82d6402a5305dbe", "1fc8eeec4f4f889c6a9f6223abb796a4f565fb87" ], "author": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Thu Jul 25 23:52:13 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Thu Jul 25 23:52:13 2013 -0700" }, "message": "Merge pull request #53 from rackspace/change-password\n\nChange password" }, { "commit": "1fc8eeec4f4f889c6a9f6223abb796a4f565fb87", "tree": "4eccf45253137302c0873f6b061433d98d5c5f46", "parents": [ "4b235afe12269eb4267249d4c8a9c454b41d4b80" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Jul 25 23:41:31 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Jul 25 23:41:31 2013 -0700" }, "message": "Move file into proper place.\n\nNot sure how this happened.\n" }, { "commit": "4b235afe12269eb4267249d4c8a9c454b41d4b80", "tree": "c702ac4c8aaf3071dddde658544f210a22114c1d", "parents": [ "80699603e7de076b6cd08e9f74affea590d4eaa3" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Jul 25 23:39:10 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Jul 25 23:39:10 2013 -0700" }, "message": "Rearrange delete-server so it always runs last.\n\nMany tests will create servers as part of their normal operation. We\nwant these to be cleaned up in one fell swoop. By running\ndelete-server.go last, we accomplish this goal.\n" }, { "commit": "80699603e7de076b6cd08e9f74affea590d4eaa3", "tree": "22a91908068798ac092273f4bff6a29afd9293f4", "parents": [ "8935ca3c3238f856d3ec8fd3e02791cfa528b2a6" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Jul 25 23:35:57 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Jul 25 23:35:57 2013 -0700" }, "message": "Make change-admin-password acceptence test autonomous\n\nUp to this commit, 09-change-admin-password.go required human\nintervention to run.\n" }, { "commit": "8935ca3c3238f856d3ec8fd3e02791cfa528b2a6", "tree": "d6bffe5ff30f60f4cd1584adfc7aa1e4d127e077", "parents": [ "5c305e115324107a05ebec572b95ed39eb30f596" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Jul 25 21:27:06 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Jul 25 21:27:06 2013 -0700" }, "message": "Refactor createServer functionality for re-use.\n\nPresently, only 04-create-server.go is capable of dynamically creating\nservers as-needed. However, many server action acceptance tests require\nservers of their own to perform tests on. Refactoring the logic for\ncreating servers into a reusable form permits other tests to take\nadvantage of this ability. This prevents human intervention from\nbecoming necessary to run acceptance tests.\n" }, { "commit": "5c305e115324107a05ebec572b95ed39eb30f596", "tree": "6fca77f2b365298cbb63145afa848170e164a862", "parents": [ "a87758a43113f31ea962cceca82d6402a5305dbe" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Jul 25 19:19:43 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Jul 25 19:19:43 2013 -0700" }, "message": "Add ability to change admin password on a server\n" }, { "commit": "a87758a43113f31ea962cceca82d6402a5305dbe", "tree": "bd0367d72264a0e27bd27c84241d3f67aa327019", "parents": [ "d6ffd4d2a2539e9f2235782003aeef81604f2434", "a0a558435aa4cb539da571c3d4b447e5f9427727" ], "author": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Wed Jul 24 13:17:15 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Wed Jul 24 13:17:15 2013 -0700" }, "message": "Merge pull request #52 from rackspace/ServerDetails\n\nFix ListServers to yield full details." }, { "commit": "a0a558435aa4cb539da571c3d4b447e5f9427727", "tree": "99efb5e037e7f1be0f0f0777710af24a97c1c825", "parents": [ "7327a6ed2d8fcd12473d60e9b0072eccbd58947a" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Wed Jul 24 13:14:17 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Wed Jul 24 13:14:17 2013 -0700" }, "message": "Fix ListServers to yield full details.\n\nAlso, provide ListServersLinksOnly() if you want to retain the older\nbehavior. LinksOnly variant isn\u0027t terribly useful though, as it\nprovides virtually _no_ useful information other than the mere existence\nof a server.\n\nFixes #45.\n" }, { "commit": "d6ffd4d2a2539e9f2235782003aeef81604f2434", "tree": "8badf618a57fa565818f89691cf0afddb170d569", "parents": [ "7327a6ed2d8fcd12473d60e9b0072eccbd58947a", "7bd1fba42f7d8c57a58374de0e9b6cf96ad12916" ], "author": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Tue Jul 16 17:35:42 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Tue Jul 16 17:35:42 2013 -0700" }, "message": "Merge pull request #47 from rackspace/reauth-enable-api\n\nEnable reauth capability for all existing API EPs." }, { "commit": "7bd1fba42f7d8c57a58374de0e9b6cf96ad12916", "tree": "8badf618a57fa565818f89691cf0afddb170d569", "parents": [ "7327a6ed2d8fcd12473d60e9b0072eccbd58947a" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jul 16 17:30:43 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jul 16 17:30:43 2013 -0700" }, "message": "Enable reauth capability for all existing API EPs.\n" }, { "commit": "7327a6ed2d8fcd12473d60e9b0072eccbd58947a", "tree": "dd34da18b9aa9c46e9297ffe3134a56fb86ff3c5", "parents": [ "54881f86e4392bb0e255ad97ebc6ad5735476998", "fb58669dd098ceb6fb65652fdbb2a1a639826a04" ], "author": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Tue Jul 16 17:11:13 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Tue Jul 16 17:11:13 2013 -0700" }, "message": "Merge pull request #46 from rackspace/reauth\n\nImplement Reauth capability" }, { "commit": "fb58669dd098ceb6fb65652fdbb2a1a639826a04", "tree": "dd34da18b9aa9c46e9297ffe3134a56fb86ff3c5", "parents": [ "9e64f6bc81fe02aff17838c08c721e0a6fe9c703" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jul 16 17:00:14 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jul 16 17:00:14 2013 -0700" }, "message": "Remove debugging infrastructure.\n\nBug was found, and things seems to work. In summary, a comparison for\nerr !\u003d nil was performed where I intended err \u003d\u003d nil instead.\n" }, { "commit": "9e64f6bc81fe02aff17838c08c721e0a6fe9c703", "tree": "6df26203a86bac45371407e18759b760f470f7aa", "parents": [ "0167aaa6c2ba91af68fe5c023b94bdd320c60d1f" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jul 16 14:26:50 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jul 16 14:26:50 2013 -0700" }, "message": "Convert ListImages to use reauth.\n\nThis commit does not work -- acceptance test 08-... is broken.\n" }, { "commit": "0167aaa6c2ba91af68fe5c023b94bdd320c60d1f", "tree": "8d900b76d60cb48c29430661dee97b2d03d37e9b", "parents": [ "659e14b849e32579d981e64f50d91a84393dfc0e" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jul 16 12:36:25 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jul 16 12:36:25 2013 -0700" }, "message": "Tie AccessProvider back to service Provider.\n\nTo issue a revokation request, I needed to know the Identity API\nendpoint. This endpoint exists in the Provider used to create the\nAccessProvider. This change threads a link back to the Provider.\n" }, { "commit": "659e14b849e32579d981e64f50d91a84393dfc0e", "tree": "622650ab135fa05f8a44788efa931b86c6f1df83", "parents": [ "2f50b144054ea924f1b548de0211f5cd61085900" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jul 16 12:04:54 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jul 16 12:04:54 2013 -0700" }, "message": "Add early acceptance test for reauthentication.\n" }, { "commit": "2f50b144054ea924f1b548de0211f5cd61085900", "tree": "045575dd66c284582fe7337dda6fa8e496c0fc30", "parents": [ "1206f85628a60373f28877cbfe6b73fe0950c7ed" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jul 16 11:38:03 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jul 16 11:38:03 2013 -0700" }, "message": "Simplify WithReauth() and its dependency workflow.\n\nThis change also threads the desired AccessProvider to the reauth\nhandler, so the reauth handler doesn\u0027t have to fish it out through some\nhard to maintain back-channel.\n" }, { "commit": "1206f85628a60373f28877cbfe6b73fe0950c7ed", "tree": "bfe4e9d1efbc87d47affd379cf302acbe96a441f", "parents": [ "2d0f6da68604dfcda9fa66e29061b41997d23163" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Mon Jul 15 17:56:51 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Mon Jul 15 17:56:51 2013 -0700" }, "message": "Add elementary support for reauth.\n\nPreemptively attempting to reauthenticate before a token expires is\nfraught with dangers. Everything from buggy API implementations to\nnatural lack of determinism when an API races against a server can cause\nopportunities to reauth to be missed. The recommended solution is to\njust detect when a 401 Unauthorized error happens, and attempt to handle\nit with a new attempt at reauthentication.\n\nThis commit does not implement the full logic to do this just yet.\nHowever, the groundwork seems to be in place, if my unit tests are any\nindication. Here\u0027s how I\u0027d _like_ to use it:\n\nLet\u0027s focus on ListServers. We\u0027d end up rewriting this function like\nso:\n\n// See the CloudServersProvider interface for details.\nfunc (gcp *genericServersProvider) ListServers() ([]Server, error) {\n var ss []Server\n\n url :\u003d gcp.endpoint + \"/servers\"\n errOuter :\u003d gcp.context.WithReauth(func() error { // NEW CODE\n errInner :\u003d perigee.Get(url, perigee.Options{\n CustomClient: gcp.context.httpClient,\n Results: \u0026struct{ Servers *[]Server }{\u0026ss},\n MoreHeaders: map[string]string{\n \"X-Auth-Token\": gcp.access.AuthToken(),\n },\n })\n return errInner\n }) // NEW CODE\n return ss, errOuter\n}\n\nNote how small the change to the existing code is: two lines of code.\n\nContext.WithReauth() works by invoking the supplied function,\noptimistically hoping that it\u0027d succeed (e.g., that error is nil). If\nso, we bee-line the results back to the ListServers implementation,\nwhere it returns its results in ss and a nil value for errOuter.\n\nIf it does fail with an error other than a 401 result, ss is undefined,\nand the error will propegate back out to errOuter, where again it\u0027ll be\nreturned to the caller.\n\nIf the error is a 401 error, however, we invoke the configured\nreauthentication handler for the given context. This code is\nresponsible for attempting the reauthentication process. If an error\noccurs in this handler, it too will propegate out to errOuter.\n\nIf everything succeeds up to this point, however, the function defined\nabove is called a second time. Assuming it succeeds, it will overwrite\nthe desired result (ss in our case) with a valid value, and return a nil\nerror. This will, as above, produce the desired outcome and all is\nwell.\n\nIf it fails a second time, however, you\u0027re on your own. Even if the 2nd\nfailure is another 401 result, that error will propegate out to\nerrOuter, where it\u0027ll be returned to the caller.\n\nKnown problems with the code as it stands:\n\n1) Procedure passed to WithReauth() is expected to return an\ninterface{}, being the desired, unmarshalled object from a web request\nagainst an API. But, I had forgotten that I could use a free variable\n(as I do in the example above) and avoid having to return it explicitly.\nI\u0027ll need to remove this result slot, saving code and complexity.\n\n2) I *may* need to pass an AccessProvider as part of the WithReauth()\ncall, so that the reauth handler has something to work with. Otherwise,\nI\u0027ll need to define an interface for all xyzProvider entities to recover\nthe AccessProvider it uses. I\u0027m not entirely sure if this breaks\nencapsulation or not.\n\nOtherwise, the logic is 99.44% finished, and unit tests all pass with\nthis commit. Acceptance tests aren\u0027t affected, as no production logic\nhas yet been changed to use this mechanism.\n" }, { "commit": "2d0f6da68604dfcda9fa66e29061b41997d23163", "tree": "e874663e553b91132b133cdfcd8d0c85a93fb113", "parents": [ "54881f86e4392bb0e255ad97ebc6ad5735476998" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Mon Jul 15 16:41:52 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Mon Jul 15 16:41:52 2013 -0700" }, "message": "Ensure authenticate never re-auths.\n\nOther API functions will support re-auth as a matter of course.\nIf an auth token expires, we need to re-authenticate to acquire a new\ntoken. If re-authentication were itself to attempt re-auth, we\nwould end up in an endless loop.\n\nIf after authenticating gophercloud receives a 401 Unauthorized\nresponse, then we must assume that the provided credentials are\nincorrect.\n" }, { "commit": "54881f86e4392bb0e255ad97ebc6ad5735476998", "tree": "60053fa06cb3381442138104a98ab7d1915265d0", "parents": [ "962512dac3f310d4fc9869f2adf2d403d435de82", "286e4de1c95161c9ec6e756efe7c6f7a3bab2615" ], "author": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Fri Jul 12 11:35:10 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Fri Jul 12 11:35:10 2013 -0700" }, "message": "Merge pull request #44 from rackspace/delete-server\n\nDelete server" }, { "commit": "286e4de1c95161c9ec6e756efe7c6f7a3bab2615", "tree": "60053fa06cb3381442138104a98ab7d1915265d0", "parents": [ "e3b2d7a79e372b9be08095a5774cf588c94f6a92" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Fri Jul 12 11:33:31 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Fri Jul 12 11:33:31 2013 -0700" }, "message": "Support delete server.\n" }, { "commit": "e3b2d7a79e372b9be08095a5774cf588c94f6a92", "tree": "0be78a0d5ed7344c0b81fe6225f76e33af4edf52", "parents": [ "962512dac3f310d4fc9869f2adf2d403d435de82" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Fri Jul 12 11:08:02 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Fri Jul 12 11:08:02 2013 -0700" }, "message": "Fix 04-create-server test to use a common prefix.\n\nWe need servers to dispose of for the 07-delete-server acceptance test.\nCreating a bunch of servers in that test only to turn around and delete\nthem again will be unnecessarily time-consuming. Instead, we\u0027ll use\nListServers() to discover a set of pre-existing servers to delete.\n04-create-server already creates a batch, and will always execute before\n07-delete-server (since 4 \u003c 7). Altering 04-create-server to put a tag\nin the name of the servers it creates will allow 07-delete-server to\nfind them and delete them again.\n\nThe only exception is if any intervening tests fail. You\u0027ll need to\nclean up manually in that case.\n" }, { "commit": "962512dac3f310d4fc9869f2adf2d403d435de82", "tree": "b1a37db344e24b9c2b2d8c77e5220877a33c2880", "parents": [ "bba5cfd7d14e24e54a73a2d41cf322275f0cf2a9", "4b0ba75a9f8c3c4621367741f6ea314cf42bec88" ], "author": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Fri Jul 12 10:57:54 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Fri Jul 12 10:57:54 2013 -0700" }, "message": "Merge pull request #41 from rackspace/create-server\n\nCreate server" }, { "commit": "4b0ba75a9f8c3c4621367741f6ea314cf42bec88", "tree": "b1a37db344e24b9c2b2d8c77e5220877a33c2880", "parents": [ "8a8d531a17aee0193c4c91e53a537d6fc8ecd6cb" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Jul 11 17:42:26 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Jul 11 17:42:26 2013 -0700" }, "message": "Finish acceptance test for CreateServer() function\n\nTwo problems remain -- server status is coming up blank, and\nmeasurements of disk and RAM are coming up zeros. Not sure why this is.\nIt used to work in Gorax. More comparative testing between Gorax and\nGophercloud is needed.\n\nHowever, what is in this commit is sufficient to get the acceptance\ntests running again, at least on Rackspace\u0027s infrastructure. If you\nattempt to run these tests on HP or other infrastructure, they will\nfail for you.\n" }, { "commit": "8a8d531a17aee0193c4c91e53a537d6fc8ecd6cb", "tree": "b798a7474a58849027fcffe9428779f4dd902865", "parents": [ "bf1fb00d12e60224ebd7119b97c603fd929a03ad", "bba5cfd7d14e24e54a73a2d41cf322275f0cf2a9" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Jul 11 17:16:22 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Jul 11 17:16:22 2013 -0700" }, "message": "Merge branch \u0027master\u0027 into create-server\n" }, { "commit": "bba5cfd7d14e24e54a73a2d41cf322275f0cf2a9", "tree": "1a044515a836954237633d8e9fc5bb376986d4c3", "parents": [ "7196a10b1f1f433829b9870810c99e84b59e5607", "bc3f10fab901e7f63bab900f18277f0791826285" ], "author": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Thu Jul 11 17:15:24 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Thu Jul 11 17:15:24 2013 -0700" }, "message": "Merge pull request #40 from rackspace/list-flavors\n\nAdd ListFlavors() and acceptance test." }, { "commit": "bc3f10fab901e7f63bab900f18277f0791826285", "tree": "1a044515a836954237633d8e9fc5bb376986d4c3", "parents": [ "7196a10b1f1f433829b9870810c99e84b59e5607" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Jul 11 17:13:24 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Jul 11 17:13:24 2013 -0700" }, "message": "Add ListFlavors() and acceptance test.\n\nThis is required to support server creation acceptance testing. Without\nthe ability to detect flavors automatically, it isn\u0027t possible to\nautomate acceptance testing, as then a human operator would be required\nto manually intervene while testing.\n" }, { "commit": "bf1fb00d12e60224ebd7119b97c603fd929a03ad", "tree": "28058866d3fe45bc9bdae1243ad09ebf453af55e", "parents": [ "e91ff6df88b445adcb5bb39f9838c652128b745b", "7196a10b1f1f433829b9870810c99e84b59e5607" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Jul 11 17:04:52 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Jul 11 17:04:52 2013 -0700" }, "message": "Merge branch \u0027master\u0027 into create-server\n\nConflicts:\n\tinterfaces.go\n" }, { "commit": "7196a10b1f1f433829b9870810c99e84b59e5607", "tree": "f1e936e83ebd3c369d2d22c61a574d552a4034b3", "parents": [ "704a750fa2d098d0537ced60a36a8cdc7e4612e2", "0a6e45abb78bc2fa05956d13f99608897230d364" ], "author": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Thu Jul 11 17:02:57 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Thu Jul 11 17:02:57 2013 -0700" }, "message": "Merge pull request #39 from rackspace/list-images-4\n\nAdd ListImages() support and acceptance test." }, { "commit": "0a6e45abb78bc2fa05956d13f99608897230d364", "tree": "f1e936e83ebd3c369d2d22c61a574d552a4034b3", "parents": [ "704a750fa2d098d0537ced60a36a8cdc7e4612e2" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Jul 11 17:00:41 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Jul 11 17:00:41 2013 -0700" }, "message": "Add ListImages() support and acceptance test.\n\nThis is needed to support a (currently broken) acceptance test for\ncreating a cloud server. Without the ability to auto-detect an image, a\nhuman operator is required to specify an image manually, which defeats\nthe value of an automated test.\n" }, { "commit": "e91ff6df88b445adcb5bb39f9838c652128b745b", "tree": "f5076f4412ea433c377122931020dd682d30e339", "parents": [ "704a750fa2d098d0537ced60a36a8cdc7e4612e2" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Jul 11 15:46:10 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Jul 11 15:46:10 2013 -0700" }, "message": "Add create-server acceptance test and support code\n\nThis check-in is not complete; it will break acceptance tests. The\nproblem is that I cannot run the test in full-quiet mode yet; I need to\nsupport listing of images and flavors before I can do that. That will\nallow the acceptance test to choose a server flavor and OS image\nappropriate for the acceptance test.\n" }, { "commit": "704a750fa2d098d0537ced60a36a8cdc7e4612e2", "tree": "d371823f13567774bfd4c5ecc5e59fbc48279fc3", "parents": [ "02f5e83ac462bf3ae8d768db9b895e57de457167" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Wed Jul 10 15:23:43 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Wed Jul 10 15:23:43 2013 -0700" }, "message": "Remove duplicate code from acceptance tests.\n\nJust cleaning things up here.\n" }, { "commit": "02f5e83ac462bf3ae8d768db9b895e57de457167", "tree": "0351d482e1c8dd1daee895868e7583696134733a", "parents": [ "489484776a96585dfd7eebf186b975b15f2022ce" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Wed Jul 10 13:52:27 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Wed Jul 10 13:52:27 2013 -0700" }, "message": "Add ServerById() function and acceptance test.\n" }, { "commit": "489484776a96585dfd7eebf186b975b15f2022ce", "tree": "b2e001d59e7960412dd1f5e35ff767fb7e807619", "parents": [ "fce34bf772da11f1ae4a8c15d2bf05c8ac669c17", "1dd740af61898a80dd5e5acada4a558b86de0c76" ], "author": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Mon Jul 08 17:30:55 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Mon Jul 08 17:30:55 2013 -0700" }, "message": "Merge pull request #37 from rackspace/list-servers--rename-compute-api\n\nRename ComputeApi to ServerApi" }, { "commit": "fce34bf772da11f1ae4a8c15d2bf05c8ac669c17", "tree": "f7d23b9e10f586ff7c1eaefd14dfae253789dd38", "parents": [ "743795c1d58bf699c66a760c9c6f881ced1a4e81", "bc0d54a28253182cde9f78be19e03ae91a8326be" ], "author": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Mon Jul 08 17:29:15 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Mon Jul 08 17:29:15 2013 -0700" }, "message": "Merge pull request #35 from rackspace/list-servers\n\nAdd support for Listing Servers." }, { "commit": "1dd740af61898a80dd5e5acada4a558b86de0c76", "tree": "b2e001d59e7960412dd1f5e35ff767fb7e807619", "parents": [ "bc0d54a28253182cde9f78be19e03ae91a8326be" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Mon Jul 08 15:48:40 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Mon Jul 08 15:48:40 2013 -0700" }, "message": "Rename ComputeApi to ServerApi\n\nComputeApi never sat well with me. OpenStack documents prefer to call\ntheir computing service \"Cloud Server\" API instead. So, I renamed all\npublic-facing types and procedure names to reflect this usage.\n\nFixes #33\n" }, { "commit": "bc0d54a28253182cde9f78be19e03ae91a8326be", "tree": "f7d23b9e10f586ff7c1eaefd14dfae253789dd38", "parents": [ "743795c1d58bf699c66a760c9c6f881ced1a4e81" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Mon Jul 08 14:45:21 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Mon Jul 08 14:45:21 2013 -0700" }, "message": "Add support for Listing Servers.\n\nUp to now, all you could do is authenticate with Gophercloud. Now, you\ncan list your servers too!\n\nThis is a pretty straight-ahead port of logic from Gorax to Gophercloud.\n\nAfter you gain access to the ComputeApi, you simply invoke\nListServers(). This returns a slice of Server structures, one each for\nthe servers your account has access to. See the 02-list-servers.go\nacceptance test.\n\nNote that no unit-level tests exist for this functionality. Writing a\nunit test for this code would only serve to test the compiler\u0027s ability\nto emit bug-free machine code. (Observe ListServers() has no control\nflow points, and all its dependencies already have their own unit\ntests.)\n\nBeing that this is a straight-ahead port from Gorax, some things remain\nto be done:\n\n1) Provide an API for accessing Rackspace-specific fields. (#34)\n2) Assuming (1) is successful, remove Rack-specific fields from Server\n and other structures of interest. (#34)\n3) Rename ComputeApi to something more descriptive -- like ServerApi().\n (#33)\n4) Implement token re-authentication logic. (#12)\n" }, { "commit": "743795c1d58bf699c66a760c9c6f881ced1a4e81", "tree": "c570f59c7b3c1c7c4deff246addc850428e2bef6", "parents": [ "e7418aa97f14b17e5730fba2d973155158fbf4ea", "2e2b8776e9bdfe8e1e4d12692e8ac7e254914f9b" ], "author": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Thu Jul 04 16:01:29 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Thu Jul 04 16:01:29 2013 -0700" }, "message": "Merge pull request #32 from rackspace/cloud-servers-1\n\nEpic refactoring to improve testability." }, { "commit": "2e2b8776e9bdfe8e1e4d12692e8ac7e254914f9b", "tree": "c570f59c7b3c1c7c4deff246addc850428e2bef6", "parents": [ "e7418aa97f14b17e5730fba2d973155158fbf4ea" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Jul 04 15:40:15 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Jul 04 15:40:15 2013 -0700" }, "message": "Epic refactoring to improve testability.\n\nIt all started when I wanted to write ListServers(), but doing that\nmeant acquiring the test context, so that I could intercept server-side\ncommunications. However, that couldn\u0027t be easily acquired with the old\nsoftware configuration. In fact, it was patently impossible to do,\nwithout breaking a lot of encapsulation and conflating concerns.\n\nAdditionally, I knew I didn\u0027t want to make ComputeApi() a method of an\nAccessProvider interface; considering how similar the OpenStack APIs\nare, I was banking on that design decision causing a lot of duplicate\ncode, even if said code was simple. Not only that, but it conflated\nconcerns; again, something I wanted to avoid.\n\nSo, I needed to do a couple of things.\n\n1) Realize that module-global functions are delegators to the global\ncontext. My original implementation of ComputeApi() wasn\u0027t, which meant\nthat I had zero access to any contexts created in unit testing.\n\n2) Realize that the Context interface is the true Gophercloud global\nAPI. This meant I had to make a ComputeApi() method on the Context\ninterface, and implement it. This proved very convenient -- it granted\nme access automatically to all test contexts.\n\nAs a checklist bullet point, whenever adding a new global-level function\nto gophercloud, do it in at least these steps: a) add the function as a\nmethod on Context. Seriously -- this is the only real way to make it\ntestable. b) Add a very dumb delegator function in global_context.go\nwhich dispatches to its eponymously-named method on globalContext.\n\n3) Making this simple change was sufficient to start to test an\nimplementation of ListServers(). However, invoking \"c :\u003d TestContext();\nc.foo(); c.bar();\" was becoming repetitive and error-prone. So, I\nrefactored that into a Java-style DSL. These things aren\u0027t terribly Go\nidiomatic, but for my needs here, they work nicely.\n\n4) I refactored the two different implementations of custom transports\ninto a single \"transport-double\" type. This type will supports both\ncanned responses and some basic request validation. It\u0027s expandable by\nsimply adding more configuration fields and DSL methods of its own.\n\n5) api.go is no more -- it previously served two separate purposes, each\nof which has its own source file now. interfaces.go holds the\ndefinition of all publicly visible APIs, while global_context.go\nprovides the default global Context, its initialization, and the\nmodule-global entry points that delegate to the global context.\n\nWith these changes having been made, *now* I\u0027m ready to actually start\ntesting ListServers() development! It only took 24 hours and 4\nrefreshes of the feature branch just to get this far. :-)\n\nThe nice thing is, though, that going forward, these changes should\ncontribute to making future endpoint binding implementations\nsignificantly easier than what I had to do before.\n" }, { "commit": "e7418aa97f14b17e5730fba2d973155158fbf4ea", "tree": "4dbdc42443d445867c78a4e50728bdadcec09a25", "parents": [ "daf65dccfea6f631599b59aafd8a1c5ac4b3e4d4", "fca35b7d2e4753c88f16e2ff88cddf29caf1f060" ], "author": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Wed Jul 03 13:21:29 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Wed Jul 03 13:21:29 2013 -0700" }, "message": "Merge pull request #31 from rackspace/service-catalog-search\n\nImplement Access.FirstEndpointUrlByCriteria\r\n\r\nThe FirstEndpointUrlByCriteria method is a key enabler for writing API\r\nprovider interface constructors. A simple, albeit incomplete, use-case\r\nfor Cloud Servers API demonstrates how it\u0027s used internally.\r\n\r\nIn order to list servers, we need access to a cloud server API. This is\r\nthe job of the ComputeApi() function.\r\n\r\nComputeApi(), in turn, tries hard not to contrain the user in choosing\r\nan endpoint, while still offering an interface optimized for the common\r\ncase of using an existing service provider\u0027s endpoints. Otherwise, the\r\nuser will end up having to use nested functions and bizarre predicate\r\nsequences like this:\r\n\r\n func(ce *CatalogEntry, ee *EntryEndpoint) bool {\r\n if ce !\u003d nil {\r\n return ce.Name \u003d\u003d \"cloudComputeOpenStack\"\r\n }\r\n if ee !\u003d nil {\r\n return ee.Region \u003d\u003d \"DFW\" \u0026\u0026 ee.VersionId \u003d\u003d \"2\"\r\n }\r\n return false\r\n }\r\n\r\nThe current interface just encapsulates this kind of logic into a simple\r\nstructure, taking 66% fewer lines, and zero chance for error:\r\n\r\n ApiCriteria{\r\n Name: \"cloudComputeOpenStack\",\r\n Region: \"DFW\",\r\n VersionId: \"2\",\r\n }\r\n\r\nFindFirstEndpointByConstraint() is invoked (via\r\nAccessProvider.FirstEndpointUrlByConstraint()) to actually look for a\r\nmatching endpoint in the provider\u0027s service catalog. This interprets\r\nthe ApiCriteria structure settings, except for UrlChoice. If it finds a\r\ncandidate endpoint, the user may select public or private endpoints via\r\nthe ApiCriteria.UrlChoice setting (which the\r\nFirstEndpointUrlByCriteria() function interprets). If nothing is found,\r\nan ErrEndpoint error will be returned to the caller. Of course, this\r\nbeing a brand new implementation, it just returns the default of\r\n\"nothing found\" for all queries anyway.\r\n\r\nIf not specified, a criteria\u0027s UrlChoice defaults to PublicURL." }, { "commit": "fca35b7d2e4753c88f16e2ff88cddf29caf1f060", "tree": "4dbdc42443d445867c78a4e50728bdadcec09a25", "parents": [ "daf65dccfea6f631599b59aafd8a1c5ac4b3e4d4" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jul 02 18:30:28 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jul 02 18:30:28 2013 -0700" }, "message": "Implement Access.FirstEndpointUrlByCriteria\n\nThe FirstEndpointUrlByCriteria method is a key enabler for writing API\nprovider interface constructors. A simple, albeit incomplete, use-case\nfor Cloud Servers API demonstrates how it\u0027s used internally.\n\nSee squashed commit history below for more detailed rationale behind the\nAPI design.\n\nSquashed commit of the following:\n\ncommit 625c31f754dcdcd2d348cf4cf5499a03ba6b2de1\nAuthor: Samuel A. Falvo II \u003csam.falvo@rackspace.com\u003e\nDate: Tue Jul 2 18:21:36 2013 -0700\n\n Fix service name typo\n\ncommit c6abcbe20bfe31a8c9399e78c186dca64d050140\nAuthor: Samuel A. Falvo II \u003csam.falvo@rackspace.com\u003e\nDate: Tue Jul 2 18:15:41 2013 -0700\n\n Added decision logic to FFEBC function.\n\ncommit bccf7178464c5071a81d63ef16fd20d7a241146f\nAuthor: Samuel A. Falvo II \u003csam.falvo@rackspace.com\u003e\nDate: Tue Jul 2 17:18:14 2013 -0700\n\n Added ListServers and its dependencies.\n\n In order to list servers, we need access to a cloud server API. This is\n the job of the ComputeApi() function.\n\n ComputeApi(), in turn, tries hard not to contrain the user in choosing\n an endpoint, while still offering an interface optimized for the common\n case of using an existing service provider\u0027s endpoints. Otherwise, the\n user will end up having to use nested functions and bizarre predicate\n sequences like this:\n\n func(ce *CatalogEntry, ee *EntryEndpoint) bool {\n if ce !\u003d nil {\n return ce.Name \u003d\u003d \"cloudComputeOpenStack\"\n }\n if ee !\u003d nil {\n return ee.Region \u003d\u003d \"DFW\" \u0026\u0026 ee.VersionId \u003d\u003d \"2\"\n }\n return false\n }\n\n The current interface just encapsulates this kind of logic into a simple\n structure, taking 66% fewer lines, and zero chance for error:\n\n ApiCriteria{\n Name: \"cloudComputeOpenStack\",\n Region: \"DFW\",\n VersionId: \"2\",\n }\n\n FindFirstEndpointByConstraint() is invoked (via\n AccessProvider.FirstEndpointUrlByConstraint()) to actually look for a\n matching endpoint in the provider\u0027s service catalog. This interprets\n the ApiCriteria structure settings, except for UrlChoice. If it finds a\n candidate endpoint, the user may select public or private endpoints via\n the ApiCriteria.UrlChoice setting (which the\n FirstEndpointUrlByCriteria() function interprets). If nothing is found,\n an ErrEndpoint error will be returned to the caller. Of course, this\n being a brand new implementation, it just returns the default of\n \"nothing found\" for all queries anyway.\n\n If not specified, a criteria\u0027s UrlChoice defaults to PublicURL.\n\ncommit 9549f0b30e0736962dad55f3f38f88124e076fb9\nAuthor: Samuel A. Falvo II \u003csam.falvo@rackspace.com\u003e\nDate: Tue Jul 2 17:10:14 2013 -0700\n\n Removed VIM temp swap file\n\ncommit 8e00ad5ac3466cbec3c539e8b21bea6d23ab37f7\nAuthor: Samuel A. Falvo II \u003csam.falvo@rackspace.com\u003e\nDate: Tue Jul 2 16:20:22 2013 -0700\n\n Add ApiCriteria to API\n\ncommit 6f3b41929a496c6a0734221bf12ef27035b71e39\nAuthor: Samuel A. Falvo II \u003csam.falvo@rackspace.com\u003e\nDate: Tue Jul 2 16:18:49 2013 -0700\n\n Add acceptance test for list servers\n" }, { "commit": "daf65dccfea6f631599b59aafd8a1c5ac4b3e4d4", "tree": "599f4e91aca206d6da8fe568873aeaaa3dd7aff5", "parents": [ "43fa244bacff5277b68739b3c50586110196532c", "73e3e874bec15a06a8a2b165ad602477ff7397fe" ], "author": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Fri Jun 28 11:43:26 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Fri Jun 28 11:43:26 2013 -0700" }, "message": "Merge pull request #10 from rackspace/getting-started-docs\n\nUpdate README.md -\u003e README.asciidoc, with typo fixes and Getting Started" }, { "commit": "73e3e874bec15a06a8a2b165ad602477ff7397fe", "tree": "599f4e91aca206d6da8fe568873aeaaa3dd7aff5", "parents": [ "1a28751ec60e4019b5d1129fe8d6ce88c939c795" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Fri Jun 28 11:40:02 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Fri Jun 28 11:40:07 2013 -0700" }, "message": "Align call-outs\n" }, { "commit": "1a28751ec60e4019b5d1129fe8d6ce88c939c795", "tree": "906979d566cdf56dc1a6d910b7e8cbdad1f4d9b6", "parents": [ "fc2cdefed8f8dba336222c17f2ad651269176d92" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Fri Jun 28 11:37:42 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Fri Jun 28 11:37:42 2013 -0700" }, "message": "Remove typo\n" }, { "commit": "fc2cdefed8f8dba336222c17f2ad651269176d92", "tree": "c3f487a0ea147797bf534b320d50c96f281e78db", "parents": [ "56e961f1257c11057ebc6c507d89922efdc39924" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Fri Jun 28 11:36:00 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Fri Jun 28 11:36:00 2013 -0700" }, "message": "Fix admonition again\n" }, { "commit": "56e961f1257c11057ebc6c507d89922efdc39924", "tree": "fde124308bc3f8eb3193023114f6b9078788a488", "parents": [ "9566bf8689231b467af3652679ab52e38b28edc7" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Fri Jun 28 11:35:00 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Fri Jun 28 11:35:00 2013 -0700" }, "message": "Add link to gophercloud-dev, fix admonition\n" }, { "commit": "9566bf8689231b467af3652679ab52e38b28edc7", "tree": "b27ed8dce366463d35431560b30f4fa78126cefe", "parents": [ "b4abb058ae399858dce42154dac40edd45ef75ac" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Fri Jun 28 11:28:23 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Fri Jun 28 11:28:23 2013 -0700" }, "message": "Replace table with callouts\n" }, { "commit": "b4abb058ae399858dce42154dac40edd45ef75ac", "tree": "c7f2e081ba56f28efeb5d6f564674284b53aff94", "parents": [ "4c11721051d516cf999666f30bfc7837ce4990d4" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Fri Jun 28 11:25:09 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Fri Jun 28 11:25:09 2013 -0700" }, "message": "Compensate for lack of gh support for admonitions\n" }, { "commit": "4c11721051d516cf999666f30bfc7837ce4990d4", "tree": "202bc82bf82ab077d058d93d26201ee81d28cdb0", "parents": [ "841204d289c81f16426cc7ba4727d85924c5fe98" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Fri Jun 28 11:23:42 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Fri Jun 28 11:23:42 2013 -0700" }, "message": "Fix accidental code block syntax\n" }, { "commit": "841204d289c81f16426cc7ba4727d85924c5fe98", "tree": "22e1ea3ba5b28338cc85209d570e37e5396aa4f2", "parents": [ "7b830c970d2a1011093f95a0a1f6df39fa2b9cd0" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Fri Jun 28 11:20:29 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Fri Jun 28 11:20:29 2013 -0700" }, "message": "Change filename to match new format\n" }, { "commit": "7b830c970d2a1011093f95a0a1f6df39fa2b9cd0", "tree": "80e2591679c0e129efa65d7ec7d272b2cb30f438", "parents": [ "07552815c87658057cfee56cfae9e50a9cfee889" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Fri Jun 28 11:20:14 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Fri Jun 28 11:20:14 2013 -0700" }, "message": "Translate from markdown to asciidoc\n" }, { "commit": "07552815c87658057cfee56cfae9e50a9cfee889", "tree": "4f2495bb9915030652a9cbd8b572ab44d5536926", "parents": [ "43fa244bacff5277b68739b3c50586110196532c" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Jun 27 15:02:13 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Jun 27 15:02:13 2013 -0700" }, "message": "Update README.md with typo fixes and Getting Started\n" }, { "commit": "43fa244bacff5277b68739b3c50586110196532c", "tree": "f416308c2716b9cd2839199a65b532f3a6b5c1ed", "parents": [ "a9b147cbd6be27c58777dbd447022fb4a7578a67", "4e89518cd94761fba4f9449d02530d1317f53681" ], "author": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Thu Jun 27 11:08:16 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Thu Jun 27 11:08:16 2013 -0700" }, "message": "Merge pull request #9 from rackspace/authentication\n\nAdd Authentication functionality.\r\n\r\nNow that I have the time to write this up, here\u0027s the description that should have gone into the git commit message.\r\n\r\ngophercloud needs to authenticate against a provider. However, gorax\u0027s API isn\u0027t ideal from a multi-provider perspective. Thus, instead of requiring the user to instantiate Identity objects, configuring them, and then authenticating in a 3-step process, I create a single public function, Authenticate(), which performs (essentially) these tasks.\r\n\r\nI cannot predict the future, and cannot guarantee Identity V3 compatibility in its current form. However, in an attempt to anticipate the future, the Authenticate function is designed to automatically guess which Identity API you intend on using based on which set of credentials you provide it. The underlying assumption is that a V3 token is compatible with a V2 token; once we have the token, it should be usable with other V2 and V3 APIs as appropriate.\r\n\r\nUnlike Ruby or Python, Go lacks support for keyword arguments. There are two ways to overcome this deficiency: (1) Make a function that accepts one or more interface{} types, and rely on type-checks to disambiguate meaning from supplied parameters; and, (2) use a structure and rely upon Go\u0027s automatic initialization of unspecified fields to well-known \"zero\" values. Here\u0027s a comparison of the two approaches from the point of view of the caller:\r\n\r\n// option 1 -- use list of interface{} types\r\nacc, err :\u003d gophercloud.Authenticate(\"rackspace-us\", gophercloud.Username(\"sfalvo\"), gophercloud.Password(\"my-pass-here\"), gophercloud.TenantId(\"blah\"))\r\n\r\n// option 2 -- use of a dedicated options structure\r\naccRackspace, err :\u003d gophercloud.Authenticate(\"rackspace-us\", gophercloud.AuthOptions{\r\n Username: \"sfalvo\",\r\n Password: \"my-pass-here\",\r\n TenantId: \"blah\",\r\n})\r\nAs can be seen, the latter requires much less physical typing (assuming one doesn\u0027t rename the gophercloud package to just \u0027g\u0027 in the import statement), and thus less chance for error. That\u0027s why I decided to use an options structure instead. It also impacts the design of the callee as well; with option (1), I\u0027d have to manually loop through all the parameters, using a type-case statement to decode the supplied parameters and fill in variables as they\u0027re discovered, while in (2) I just inspect the options structure directly. Less code means fewer bugs.\r\n\r\nSince the method of authentication remains the same across all providers, assuming universal use of V2 APIs, I associate an AuthEndpoint field with each Provider instance. That\u0027s the only per-provider piece of information defined at the moment.\r\n\r\nMost other SDKs hard-wire their providers; however, this is grossly inconvenient for unit-testing purposes. Therefore, I wrap what would otherwise be global state into a Context structure. TestContext exists to create a blank context, which can be used by unit tests at will. You\u0027ll notice that the init() function (in the api.go file) uses it to create the one, true, global context, and pre-populates it with the otherwise statically defined list of providers. Through this mechanism, users of the library needn\u0027t concern themselves with contexts and their proper initialization. Instead, they can just use the package-global functions, and they should \"just work.\"\r\n\r\nNote that the result of Authenticate() is a structure instance, allowing the client access to the service catalog, tenant ID information, and user information. As we flesh out additional APIs for the Go SDK, we will add methods to this Access structure, allowing more convenient access to various APIs. For example, one hypothetical approach to working with Cloud Compute services would involve using Access as a factory:\r\n\r\ncompute, err :\u003d accRackspace.CloudComputeApi()\r\nThis conforms to the first two levels of the desired \u003corganization.service.entity.method\u003e SDK organization, and provides the appropriate propegation of state that allows for token re-auth in a fully transparent manner, if necessary." }, { "commit": "4e89518cd94761fba4f9449d02530d1317f53681", "tree": "f416308c2716b9cd2839199a65b532f3a6b5c1ed", "parents": [ "d1ee7987fd4c036272d39d80bc3b89b6d22a518c" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Wed Jun 26 15:44:18 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Wed Jun 26 15:44:18 2013 -0700" }, "message": "Add global Authenticate() function.\n\nAlso finishes work started in last commit.\n" }, { "commit": "d1ee7987fd4c036272d39d80bc3b89b6d22a518c", "tree": "8d6fd2780e0b05e3000cceee0c5f80a92b8cbc52", "parents": [ "839428e0c2bc7006b922bfec7f1fdb04c4c749e6" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Wed Jun 26 14:32:45 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Wed Jun 26 14:32:45 2013 -0700" }, "message": "Parse out auth response and fill in access object\n" }, { "commit": "839428e0c2bc7006b922bfec7f1fdb04c4c749e6", "tree": "4cab775ad5a75c81c2f484ab0ef0816a66fb3176", "parents": [ "00a233d1e1a75d630ee886000642f41de4c969e2" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jun 25 18:02:24 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jun 25 18:02:24 2013 -0700" }, "message": "go fmt\n" }, { "commit": "00a233d1e1a75d630ee886000642f41de4c969e2", "tree": "94980c6a5e4b9e92972d58fa4db3429f3597b8b3", "parents": [ "978fed843429bf8a6e3c20c0f28e5f58eb2cfc09" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jun 25 18:00:53 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jun 25 18:00:53 2013 -0700" }, "message": "code cleanup\n" }, { "commit": "978fed843429bf8a6e3c20c0f28e5f58eb2cfc09", "tree": "010c2124a715ae2734dc02f42a455b96d5fb3e6a", "parents": [ "5d0d74ce130097c44e1c6290a607c74ec58a59ee" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jun 25 17:59:19 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jun 25 17:59:19 2013 -0700" }, "message": "Verify proper Tenant ID encoding behavior\n" }, { "commit": "5d0d74ce130097c44e1c6290a607c74ec58a59ee", "tree": "0bc7a6766c8a3cf2acb56d6cf7533816ce511902", "parents": [ "fd78c30473c2f64b2c0ef3263b0f1e1f963c36a8" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jun 25 17:23:18 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jun 25 17:23:18 2013 -0700" }, "message": "Make Authenticate issue an HTTP request\n" }, { "commit": "fd78c30473c2f64b2c0ef3263b0f1e1f963c36a8", "tree": "212a485ebff4a28e1c9e9e1f9a10b7a07088923c", "parents": [ "1d3fa667c76ec6dabb67a0b13ce07c3022768f4c" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jun 25 16:35:32 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jun 25 16:35:32 2013 -0700" }, "message": "Add provider registry support.\n" }, { "commit": "1d3fa667c76ec6dabb67a0b13ce07c3022768f4c", "tree": "ab8cc3e95e3080b9a8526668661d58eec190cc28", "parents": [ "d6092adfb69759791c3f3fdd99e5642066e3e642" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jun 25 15:29:32 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jun 25 15:29:32 2013 -0700" }, "message": "Add authentication acceptance test.\n\nBefore coding on the Authentication functionality of gophercloud, we\nneeded the acceptance test to know when we were finished.\n\nSince the authentication test is written in Go, I had to adjust the\nscripts/test-all.sh file to invoke Go programs directly.\n" }, { "commit": "d6092adfb69759791c3f3fdd99e5642066e3e642", "tree": "721a3677d2efe67040252c11c5fc0356307fafeb", "parents": [ "a9b147cbd6be27c58777dbd447022fb4a7578a67" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jun 25 14:07:25 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Tue Jun 25 14:07:25 2013 -0700" }, "message": "Add script to launch acceptance tests\n" }, { "commit": "a9b147cbd6be27c58777dbd447022fb4a7578a67", "tree": "f33df8f533b31fee167616d7e5558dd22aea83cd", "parents": [ "a1f6f222ec7bc16423aa1c6ed74c36c5f191c9f2", "563bf05417e6d210a4b4a4b89674b05d4bac5675" ], "author": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Tue Jun 25 13:46:49 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "kc5tja@arrl.net", "time": "Tue Jun 25 13:46:49 2013 -0700" }, "message": "Merge pull request #3 from rackspace/script-environment-setup\n\nScript to automate environment creation" }, { "commit": "563bf05417e6d210a4b4a4b89674b05d4bac5675", "tree": "f33df8f533b31fee167616d7e5558dd22aea83cd", "parents": [ "ff3f09dc53e2911fe4dac14d3d22564c4d60f0f8" ], "author": { "name": "stackedsax", "email": "alex@alexscammon.com", "time": "Tue Jun 25 13:21:00 2013 -0700" }, "committer": { "name": "stackedsax", "email": "alex@alexscammon.com", "time": "Tue Jun 25 13:21:00 2013 -0700" }, "message": "Making some changes to make the temporary installation steps even more explicit\n" }, { "commit": "ff3f09dc53e2911fe4dac14d3d22564c4d60f0f8", "tree": "096a969d725de50b180849ce1bb373f75c3a25e1", "parents": [ "f614c937c01213cb695b89b6ecd02c581a8432b4" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Jun 20 17:54:26 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Jun 20 17:54:26 2013 -0700" }, "message": "README updated\n" }, { "commit": "f614c937c01213cb695b89b6ecd02c581a8432b4", "tree": "1c43b828c8c8e15aac65a57628f3b25eb020ef9a", "parents": [ "a1f6f222ec7bc16423aa1c6ed74c36c5f191c9f2" ], "author": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Jun 20 15:18:43 2013 -0700" }, "committer": { "name": "Samuel A. Falvo II", "email": "sam.falvo@rackspace.com", "time": "Thu Jun 20 15:18:43 2013 -0700" }, "message": "Script to automate environment creation\n" }, { "commit": "a1f6f222ec7bc16423aa1c6ed74c36c5f191c9f2", "tree": "e521c99b28b4f33b139808b939e2fd9f99007387", "parents": [], "author": { "name": "Glen Campbell", "email": "glen@broadpool.com", "time": "Thu Jun 20 11:10:46 2013 -0700" }, "committer": { "name": "Glen Campbell", "email": "glen@broadpool.com", "time": "Thu Jun 20 11:10:46 2013 -0700" }, "message": "Initial commit\n" } ] }