Skip to content
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

Fixed the operation not allowed error for the table aws_directory_service_directory Closes #2155 #2170

Merged
merged 1 commit into from Apr 18, 2024

Conversation

ParthaI
Copy link
Contributor

@ParthaI ParthaI commented Apr 18, 2024

Integration test logs

Logs
N/A

Example query results

Results
Before fix:

> select name, type, event_topics, snapshot_limit from aws_directory_service_directory

Error: aws: operation error Directory Service: GetSnapshotLimits, https response error StatusCode: 400, RequestID: b75299bd-9b0c-4818-9609-e61aba43c002, ClientException: Snapshot limits can be fetched only for VPC or Microsoft AD directories. : RequestId: b75299bd-9b0c-4818-9609-e61aba43c002 (SQLSTATE HV000)

+----------+----------+--------------+------------------------------------------------------------------------------------------------+
| name     | type     | event_topics | snapshot_limit                                                                                 |
+----------+----------+--------------+------------------------------------------------------------------------------------------------+
| test.com | SimpleAD | []           | {"ManualSnapshotsCurrentCount":0,"ManualSnapshotsLimit":5,"ManualSnapshotsLimitReached":false} |
+----------+----------+--------------+------------------------------------------------------------------------------------------------+



After fix:

> select name, type, event_topics, snapshot_limit from aws_directory_service_directory
+----------+-------------+--------------+------------------------------------------------------------------------------------------------+
| name     | type        | event_topics | snapshot_limit                                                                                 |
+----------+-------------+--------------+------------------------------------------------------------------------------------------------+
| test.com | ADConnector | []           | <null>                                                                                         |
| test.com | SimpleAD    | []           | {"ManualSnapshotsCurrentCount":0,"ManualSnapshotsLimit":5,"ManualSnapshotsLimitReached":false} |
+----------+-------------+--------------+------------------------------------------------------------------------------------------------+

@bigdatasourav bigdatasourav merged commit 90518c1 into main Apr 18, 2024
1 check passed
@bigdatasourav bigdatasourav deleted the fix-ds-error-op-not-allow branch April 18, 2024 08:31
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.

None yet

2 participants