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

Cleanup of Unused Properties and Enable Test Parallelisation #32

Merged
merged 14 commits into from
Feb 5, 2024

Conversation

shaswot77
Copy link
Contributor

@shaswot77 shaswot77 commented Jan 31, 2024

Description

  • Removed unused properties
  • Tidied up tests so its doesn't create more resources than it needs to
  • Using uuid during test resource creation to avoid conflicts as resources cannot be created with same name
  • Enabled test parallelisation which was previously disabled due to resource name conflicts
  • Removed test-setup as it is mostly duplicates of existing examples
  • Extend dashboard example to now include monitoring and scoped object

Checklist:

  • Added a label: bug-fix, feature, or enhancement
  • PR description written

Test

I have deployed using the current public version of the provider and then with the changes introduced on this PR. No issues were detected apart from breaking change introduced in #31

@shaswot77 shaswot77 added the enhancement New feature or request label Jan 31, 2024
@shaswot77 shaswot77 changed the title Cleanup of unused properties Cleanup of Unused Properties and Enable Test Parallelisation Feb 1, 2024
@shaswot77 shaswot77 merged commit 699553d into main Feb 5, 2024
6 checks passed
@shaswot77 shaswot77 deleted the work/ss/tidyup branch February 5, 2024 09:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Development

Successfully merging this pull request may close these issues.

2 participants