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

test: PC-13893 Make acceptance test forward compatible #312

Merged
merged 3 commits into from
Sep 23, 2024

Conversation

ditrytus
Copy link
Contributor

@ditrytus ditrytus commented Sep 23, 2024

Motivation

value field of objective for Composite SLO is being deprecated. Before the update in can be made, a N9 platform first needs to be updated and new version of SDK needs to be released. To be able to release changes in the platform, e2e tests of the latest released SDK and Terraform Provider need to pass against it.

Summary

E2E test assertions will pass with and without changes in N9 platform.

Related changes

nobl9/nobl9-go#551
nobl9/nobl9-go#549
https://github.com/nobl9/n9/pull/15406
#312
#295

Testing

Run make test/e2e on both environments containing and not containing https://github.com/nobl9/n9/pull/15406.

@ditrytus ditrytus merged commit 5766354 into main Sep 23, 2024
5 checks passed
@ditrytus ditrytus deleted the PC-13893-relax-acc-test branch September 23, 2024 11:59
ditrytus added a commit to nobl9/nobl9-go that referenced this pull request Sep 23, 2024
## Motivation

`value` field of `objective` for Composite SLO is being
[deprecated](#549). Before the
update in SDK can be made, a N9 platform first needs to be
[updated](nobl9/n9#15406). To be able to release
changes in the platform, e2e tests of the latest released SDK need to
pass against it.

## Summary

E2E test assertions will pass with and without
[changes](nobl9/n9#15406) in N9 platform.

## Related changes

#551
#549
nobl9/n9#15406
nobl9/terraform-provider-nobl9#312
nobl9/terraform-provider-nobl9#295

## Testing

Run `make test/e2e` on both environments containing and not containing
nobl9/n9#15406.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
go Pull requests that update Go code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants