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

remove space_id parameter from private locations #733

Merged
merged 5 commits into from
Sep 9, 2024

Conversation

biscout42
Copy link
Contributor

Fixes #726

 private location is agnostic to kibana spaces. The field space_id was previously incorrectly used, because its value was a `namespace` of agent policy (data stream).

This PR removes the field. As there is no way to update a private location via API (only replace), there is no benefit to introduce `namespace` field for private locations.
@biscout42 biscout42 enabled auto-merge (squash) August 28, 2024 14:53
Copy link
Member

@tobio tobio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IIUC, the space id parameter in the API is more or less ignored, and instead the namespace/space_id is inherited from the agent policy? Assuming that's correct, LGTM

Can we get a changelog entry for this one calling out the breaking change.

@biscout42
Copy link
Contributor Author

Thanks, @tobio ,

IIUC, the space id parameter in the API is more or less ignored, and instead the namespace/space_id is inherited from the agent policy?

Yes, that is correct.

@biscout42 biscout42 changed the title remove space_id paramter from private locations remove space_id parameter from private locations Sep 9, 2024
@biscout42 biscout42 merged commit 7b9f454 into elastic:main Sep 9, 2024
20 checks passed
tobio added a commit to jdesnoes/terraform-provider-elasticstack that referenced this pull request Sep 11, 2024
…-data-source

* origin/main: (23 commits)
  Migrate index resource to plugin framework (elastic#698)
  Remove duplicate code (elastic#761)
  fix(deps): update module github.com/hashicorp/terraform-plugin-framework-jsontypes to v0.2.0 (elastic#760)
  fix(deps): update module github.com/golangci/golangci-lint to v1.61.0 (elastic#759)
  Add support for the`frequency` field in the Create Rule API (elastic#753)
  Prevent a provider panic when the referenced snapshot repo does not exist (elastic#758)
  remove space_id paramter from private locations (elastic#733)
  chore(deps): update dependency go to v1.23.1 (elastic#755)
  chore(deps): update docker.elastic.co/elasticsearch/elasticsearch docker tag to v8.15.1 (elastic#756)
  chore(deps): update golang:latest docker digest to 4a3c2bc (elastic#754)
  chore(deps): update docker.elastic.co/kibana/kibana docker tag to v8.15.1 (elastic#757)
  Fixup error handling during saved object imports (elastic#738)
  fix(deps): update module github.com/goreleaser/goreleaser to v2 (elastic#748)
  chore(deps): update golangci/golangci-lint-action action to v6 (elastic#746)
  chore(deps): update codecov/codecov-action action to v4 (elastic#745)
  fix(deps): update module github.com/go-resty/resty/v2 to v2.14.0 (elastic#741)
  chore(deps): update actions/setup-go action to v5 (elastic#744)
  chore(deps): update actions/checkout action to v4 (elastic#743)
  fix(deps): update module github.com/deepmap/oapi-codegen/v2 to v2.3.0 (elastic#740)
  chore(deps): update dependency go to v1.23.0 (elastic#739)
  ...
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.

[Bug] Inconsistent state for elasticstack_kibana_synthetics_private_location space_id
2 participants