Skip to content

Conversation

@dmurray-lacework
Copy link
Collaborator

@dmurray-lacework dmurray-lacework commented Nov 30, 2022

Signed-off-by: Darren Murray darren.murray@lacework.net

Summary

The type of limit_by_labels field has been updated in the terraform-provider.
This change updates the type of limit_by_labels variable allowed by the module.

How did you test this change?

Add limit_by_labels to the examples,

run compatibility tests
run tf plan locally
run e2e

Issue

Signed-off-by: Darren Murray <darren.murray@lacework.net>
@dmurray-lacework dmurray-lacework marked this pull request as ready for review November 30, 2022 14:51
Signed-off-by: Darren Murray <darren.murray@lacework.net>
@afiune
Copy link
Contributor

afiune commented Apr 21, 2023

This was essentially a breaking change since we changed the variable limit_by_labels from map(string)
to list(object())

This should have been released as v1.0.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants