Skip to content

[updatecli] Update default stack version to 8.18.0#2544

Closed
github-actions[bot] wants to merge 0 commit intomainfrom
updatecli_main_bump-elastic-stack-default-version
Closed

[updatecli] Update default stack version to 8.18.0#2544
github-actions[bot] wants to merge 0 commit intomainfrom
updatecli_main_bump-elastic-stack-default-version

Conversation

@github-actions
Copy link
Contributor

Bump elastic-stack default version

[updatecli] Update default stack version to 8.18.0

1 file(s) updated with "$1 \"8.18.0\"": * internal/install/stack_version.go

GitHub Action workflow link
Updatecli logo

Created automatically by Updatecli

Options:

Most of Updatecli configuration is done via its manifest(s).

  • If you close this pull request, Updatecli will automatically reopen it, the next time it runs.
  • If you close this pull request and delete the base branch, Updatecli will automatically recreate it, erasing all previous commits made.

Feel free to report any issues at github.com/updatecli/updatecli.
If you find this tool useful, do not hesitate to star our GitHub repository as a sign of appreciation, and/or to tell us directly on our chat!

@github-actions github-actions bot requested a review from a team as a code owner April 16, 2025 01:29
@mrodm
Copy link
Contributor

mrodm commented Apr 16, 2025

There is some change related to failure_store in 8.18.0 that affects this false positive test @jsoriano

Currently, elastic-package uses this query to decide whether or not failure store is available: GET https://localhost:9200/_search?failure_store=only

func (client *Client) IsFailureStoreAvailable(ctx context.Context) (bool, error) {
// FIXME: Using the low-level transport till the API SDK supports the failure store.
request, err := http.NewRequest(http.MethodGet, "/_search?failure_store=only", nil)

Trying this in 8.17.4 results in this response (so failure store is available):

 $ curl -u elastic:changeme -s -k "https://localhost:9200/_search?failure_store=only"
{"took":3,"timed_out":false,"_shards":{"total":12,"successful":12,"skipped":0,"failed":0},"hits":{"total":{"value":0,"relation":"eq"},"max_score":null,"hits":[]}}

However, in 8.18.0 that argument failure_store is not available anymore for _search, and it returns "Bad Request":

 $ curl -u elastic:changeme -s -k "https://localhost:9200/_search?failure_store=only"
{"error":{"root_cause":[{"type":"illegal_argument_exception","reason":"request [/_search] contains unrecognized parameter: [failure_store]"}],"type":"illegal_argument_exception","reason":"request [/_search] contains unrecognized parameter: [failure_store]"},"status":400}

@mrodm
Copy link
Contributor

mrodm commented Apr 16, 2025

If it is forced to check the failure store in the system tests by applying this patch:

--- cmd/testrunner.go
+++ cmd/testrunner.go
@@ -543,10 +543,11 @@ func testRunnerSystemCommandAction(cmd *cobra.Command, args []string) error {
        if err != nil {
                return err
        }
-       checkFailureStore, err := esClient.IsFailureStoreAvailable(ctx)
-       if err != nil {
-               return fmt.Errorf("can't check if failure store is available: %w", err)
-       }
+       // checkFailureStore, err := esClient.IsFailureStoreAvailable(ctx)
+       // if err != nil {
+       //      return fmt.Errorf("can't check if failure store is available: %w", err)
+       // }
+       checkFailureStore := true
 
        if runTearDown || runTestsOnly {
                if variantFlag != "" {

The query to check the failure store also fails:

--- Test results for package: failure_store - START ---
╭───────────────┬─────────────┬───────────┬───────────┬───────────────────────────────────────────────────────────────────────────────┬───────────────╮
│ PACKAGE       │ DATA STREAM │ TEST TYPE │ TEST NAME │ RESULT                                                                        │  TIME ELAPSED │
├───────────────┼─────────────┼───────────┼───────────┼───────────────────────────────────────────────────────────────────────────────┼───────────────┤
│ failure_store │ test        │ system    │ fail      │ ERROR: failed to check failure store: search request returned status code 400 │ 28.790439283s │
╰───────────────┴─────────────┴───────────┴───────────┴───────────────────────────────────────────────────────────────────────────────┴───────────────╯
--- Test results for package: failure_store - END   ---
Done
Error: failed to process results: one or more test cases failed

@github-actions github-actions bot closed this Apr 17, 2025
@github-actions github-actions bot force-pushed the updatecli_main_bump-elastic-stack-default-version branch from 20d6233 to b959f2f Compare April 17, 2025 01:28
@elasticmachine
Copy link
Collaborator

elasticmachine commented Apr 17, 2025

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants