17d1bb7dde
Revert "Merge pull request #422 from svanharmelen/f-shaving-off-some-cpu-cycles"
...
This reverts commit 428cce2478517c5b2ea8
2014-10-20 23:14:14 -07:00
428cce2478
Merge pull request #422 from svanharmelen/f-shaving-off-some-cpu-cycles
...
helper/schema: Shaving off some unneeded calls, saving CPU cycles
2014-10-20 23:13:18 -07:00
5390357e45
helper/schema: sets properly take into account the diff
2014-10-20 15:32:30 -07:00
e003334429
Shaving off some unneeded calls, saving CPU cycles
...
I know it’s very unlikely that a user will notice the difference, but
why range through the list, generate the set and calculate the
hashcode, only to find out that indexMap == nil (e.g. don’t do anything
with the generated hashcode).
As indexMap is only needed when len(parts) > 0, why not only create and
fill it (in one go) when len(parts) > 0?
2014-10-16 11:54:36 +02:00
59349cca11
helper/schema: sets must be treated atomically within ResourceData
...
This fixes a seemingly minor issue (GH-255) around plans showing changes
when in fact there are none. But in reality this turned out to uncover a
really terrible bug.
The effect of what was happening was that multiple items in a set were
being merged. Now, they were being merged in the right order, so if you
didn't have rich types (lists in a set) then you never saw the effect
since the later value would overwrite the earlier. But with lists (such
as in security groups), you would end up with the lists merging. So, if
you had one ingress rule with CIDR blocks and one with SGs, then after
the merge both ingress rules would have BOTH CIDR and SGs, resulting in
an incorrect plan (GH-255).
This fixes the issue by introducing a `getSourceExact` bitflag to the
ResourceData source. When this is set, ALL data must come from this
level, instead of merging lower levels. In the case of sets and diffs,
this is exactly what you want: "Get me the set 'foo' from the config and
the config ONLY (not the state or diff or w/e)".
Andddddd its fixed.
GH-255
2014-10-11 10:40:54 -07:00
3a107d2e50
helper/schema: set the field to empty if it is a list and computed
2014-10-10 15:58:38 -07:00
e5d64318bf
helper/schema: don't say that a set is computed if it exists
2014-10-10 15:46:24 -07:00
e0994c717d
helper/schema: fix issues around computed sets
2014-10-10 09:13:04 -07:00
dcaf653d6f
helper/schema: properly detect that a list is computed
2014-10-09 19:09:06 -07:00
00bdef3093
providers/aws: test tag removal
2014-10-08 18:21:21 -07:00
6eafac8a34
providers/aws: aws_vpc supports tags
2014-10-08 17:54:00 -07:00
f43528957e
helper/schema: support top-level TypeMap
2014-10-08 17:35:14 -07:00
0db9d98fff
helper/schema: can set Partial(true) multiple times
2014-10-08 16:35:14 -07:00
9b2b3a963f
ResourceDiff => InstanceDiff
2014-09-17 16:33:24 -07:00
81d9d70296
helper/schema: conforms to new API, tests pass
2014-09-16 17:07:13 -07:00
faa4e75960
helper/schema: Get on computed diff should be empty
2014-08-27 15:45:52 -07:00
0c9b65f3f4
helper/schema: documentation
2014-08-26 21:52:09 -07:00
87a488092c
helper/schema: support partial states
2014-08-26 20:19:44 -07:00
56cf1e6faa
Fix go vet complaints
2014-08-24 21:50:35 -07:00
9ed601d541
helper/schema: store state with processed properly
2014-08-22 12:09:06 -07:00
50026a6d5c
helper/schema: When having a StateFunc, make sure NewExtra contains
...
original
2014-08-22 08:57:44 -07:00
d009ea800a
helper/schema: add support for StateFunc
2014-08-22 08:45:54 -07:00
7be2f1b091
helper/schema: add GetOk
2014-08-21 23:03:04 -07:00
37cf52fa27
helper/schema: if no ID is set then return nil
2014-08-21 22:19:33 -07:00
3d3789920d
helper/schema: can set conninfo
2014-08-21 22:15:47 -07:00
9ab5577beb
helper/schema: set diff tests
2014-08-20 21:09:07 -07:00
ca18e971d1
helper/schema: can set sets
2014-08-20 21:09:06 -07:00
56dde5c0c1
helper/schema: can read and get the state of sets
2014-08-20 21:09:06 -07:00
5e975e47cf
helper/schema: diffing should use ResourceData for guidance
2014-08-20 18:11:40 -07:00
5fc41cc272
helper/schema: properly put "id" into attributes
2014-08-20 10:38:51 -07:00
ee0acc4a5d
helper/schema: support bools
2014-08-19 16:46:36 -07:00
ba819d1f37
helper/schema: don't include zero count in state
2014-08-18 19:12:49 -07:00
1b85f3a565
helper/schema: SetDependencies
2014-08-18 15:41:12 -07:00
dfede5791c
helper/schema: use reflection to set maps
2014-08-18 15:17:18 -07:00
3800dffdeb
helper/schema: computed values should not be Get-able
2014-08-18 15:10:53 -07:00
4ae1673580
helper/schema: support for NewRemoved
2014-08-18 15:07:09 -07:00
4c9271160e
helper/schema: can handle maps
2014-08-18 14:00:03 -07:00
66d7003e6f
helper/schema: HasChange
2014-08-18 10:00:41 -07:00
f1f4836c99
helper/schema: GetChange
2014-08-18 09:58:44 -07:00
513596cc37
providers/heroku: Heroku app is mostly converted
2014-08-18 09:32:40 -07:00
c418681cc3
helper/schema: create/update/delete should work for Resource
2014-08-17 20:20:11 -07:00
894187ec18
helper/schema: Resource.Refresh
2014-08-17 19:45:26 -07:00
c3f1f49640
helper/schema: final state for lists/objects works
2014-08-17 14:12:54 -07:00
06d30a559a
helper/schema: Set object
2014-08-17 11:38:16 -07:00
2d74a3cadd
helper/schema: basic set
2014-08-16 15:02:51 -07:00
bf6ad07505
helper/schema: ResourceData.Get can get the full object
2014-08-16 13:55:10 -07:00
b54acf4a0b
helper/schema: better docs
2014-08-15 17:46:05 -07:00
1d9528e595
helper/schema: ResourceData.Get works in most cases
2014-08-15 17:39:08 -07:00
31067ee8f6
helper/schema: ResourceData, starting tests
2014-08-15 16:32:43 -07:00
efa2335886
helper/schema: start the Diff function
2014-08-14 19:55:47 -07:00
Mitchell Hashimoto