Skip to content

Conversation

@Eshanatnight
Copy link
Contributor

Fixes #721 .

Description

  1. Add delete_ingester_meta func in ObjectStorage Trait
    2. Update Permission Actions
    3. Update PostError
    4. Add delete Ingeter Endpoint -> api/v1/cluster/ingester_ip%3Aingester_port

This PR has:

  • been tested to ensure log ingestion and log query works.
  • added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
  • added documentation for new or modified features or behaviors.

@Eshanatnight Eshanatnight force-pushed the add-delete-ingester-endpoint branch 2 times, most recently from 40e74cb to b140428 Compare March 28, 2024 08:11
@Eshanatnight Eshanatnight force-pushed the add-delete-ingester-endpoint branch from b140428 to e2a0fa4 Compare March 28, 2024 10:50
1. Add delete_ingester_meta func in ObjectStorage Trait
2. Update Permission Actions
3. Update PostError
4. Add delete Ingeter Endpoint ->
`api/v1/cluster/ingester_ip%3Aingester_port`
@Eshanatnight Eshanatnight force-pushed the add-delete-ingester-endpoint branch 2 times, most recently from f8d2c78 to d36bf3f Compare March 29, 2024 11:00
@Eshanatnight Eshanatnight force-pushed the add-delete-ingester-endpoint branch from d36bf3f to 564c20c Compare March 29, 2024 11:06
@nikhilsinhaparseable nikhilsinhaparseable merged commit 1b5d2e0 into parseablehq:distributed Mar 29, 2024
@Eshanatnight Eshanatnight deleted the add-delete-ingester-endpoint branch March 29, 2024 12:55
Eshanatnight added a commit to Eshanatnight/parseable that referenced this pull request Apr 1, 2024
* fix: query fail when an Ingest Server is down

* add delete ingester endpoint

1. Add delete_ingester_meta func in ObjectStorage Trait
2. Update Permission Actions
3. Update PostError
4. Add delete Ingeter Endpoint ->
`api/v1/cluster/ingester_ip%3Aingester_port`

* fix: handle edge cases while parsing ingester indentifiers
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.

2 participants