Skip to content

[datadog_service_account] Implement exact match filtering #2447

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

retsguj
Copy link
Contributor

@retsguj retsguj commented Jun 18, 2024

https://datadoghq.atlassian.net/browse/ACCESS-2606

I modeled this off of the existing exact match filtering in data_source_datadog_user

Not sure whether I should match by name or email first, or whether it really even matters.

It didn't make sense to me to exact match by role, even if the filter string will filter for roles...

@retsguj retsguj changed the title [datadog_service_account] Implement exact match [datadog_service_account] Implement exact match filtering Jun 18, 2024
@retsguj retsguj marked this pull request as ready for review June 18, 2024 17:42
@retsguj retsguj requested review from a team as code owners June 18, 2024 17:42
@nkzou nkzou merged commit 15ec09e into julia.gu/service-account-roles Jun 20, 2024
8 checks passed
@nkzou nkzou deleted the julia.gu/service-account-exact-match branch June 20, 2024 15:54
nkzou pushed a commit that referenced this pull request Jul 5, 2024
…counts (#2434)

* init

* fix test

* make docs

* hmm

* more

* make docs again

* try fix

* try unknown?

* oops

* [datadog_service_account] Implement exact match filtering (#2447)

* add exact match

* make docs
bharling pushed a commit to bharling/terraform-provider-datadog that referenced this pull request Jul 8, 2024
…counts (DataDog#2434)

* init

* fix test

* make docs

* hmm

* more

* make docs again

* try fix

* try unknown?

* oops

* [datadog_service_account] Implement exact match filtering (DataDog#2447)

* add exact match

* make docs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants