Skip to content

fix beats test failure#8182

Merged
kvalliyurnatt merged 1 commit intoelastic:mainfrom
kvalliyurnatt:fix_beats_test
Nov 6, 2024
Merged

fix beats test failure#8182
kvalliyurnatt merged 1 commit intoelastic:mainfrom
kvalliyurnatt:fix_beats_test

Conversation

@kvalliyurnatt
Copy link
Contributor

@kvalliyurnatt kvalliyurnatt commented Nov 5, 2024

With the latest 8.17.0-SNAPSHOT build of beats, seems like "-system.hostfs=/hostfs" does not work, we need it to be "--system.hostfs=/hostfs"

Beats PR: elastic/beats#41277

@kvalliyurnatt kvalliyurnatt added >bug Something isn't working >test Related to unit/integration/e2e tests exclude-from-release-notes Exclude this PR from appearing in the release notes v2.16.0 labels Nov 5, 2024
@thbkrkr
Copy link
Contributor

thbkrkr commented Nov 5, 2024

buildkite test this -f p=gke,s=8.17.0-SNAPSHOT -m t=TestMetricbeatDefaultConfig,t=TestBeatKibanaRef

@naemono
Copy link
Contributor

naemono commented Nov 5, 2024

Wait, don't we need to version gate these changes depending on the version of beats we're testing? Won't this fail on older beats versions @kvalliyurnatt ?

@kvalliyurnatt
Copy link
Contributor Author

@naemono I tested this on 8.16.0 locally and -- seems to be compatible

docker run --rm -ti docker.elastic.co/beats/metricbeat:8.16.0-SNAPSHOT -e --system.hostfs=/hostfs

@naemono
Copy link
Contributor

naemono commented Nov 5, 2024

@naemono I tested this on 8.16.0 locally and -- seems to be compatible

docker run --rm -ti docker.elastic.co/beats/metricbeat:8.16.0-SNAPSHOT -e --system.hostfs=/hostfs

We support the latest 7.x version, and the latest 6.x version in our e2e tests, and I suspect this will break those.

@naemono
Copy link
Contributor

naemono commented Nov 5, 2024

Well, a quick test of 6.8.23 and 7.17.25 locally seems to work, so maybe I'm wrong here.

@kvalliyurnatt
Copy link
Contributor Author

Well, a quick test of 6.8.23 and 7.17.25 locally seems to work, so maybe I'm wrong here.

I did the same, it seems to work. seems like -- was always supported

Copy link
Contributor

@naemono naemono left a comment

Choose a reason for hiding this comment

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

In the case of this working for previous versions, 👍

@kvalliyurnatt
Copy link
Contributor Author

buildkite test this -f p=gke -m s=8.16.0,s=7.17.25,s=6.8.23 -m t=TestMetricbeatDefaultConfig,t=TestBeatKibanaRef

@naemono
Copy link
Contributor

naemono commented Nov 5, 2024

buildkite test this -f p=gke,s=8.16.0,s=7.17.25,s=6.8.23 -m t=TestMetricbeatDefaultConfig,t=TestBeatKibanaRef

@thbkrkr
Copy link
Contributor

thbkrkr commented Nov 6, 2024

buildkite test this -f p=gke,E2E_TAGS=beat -m s=8.16.0,s=7.17.25,s=6.8.23


buildkite test this -f p=gke -m s=8.16.0,s=7.17.25,s=6.8.23 -m t=TestMetricbeatDefaultConfig,t=TestBeatKibanaRef

== buildkite test this -f p=gke -m t=TestMetricbeatDefaultConfig,t=TestBeatKibanaRef

@kvalliyurnatt You should not use -m,--mixed multiple times in the same invocation, only the last one is used.

buildkite test this -f p=gke,s=8.16.0,s=7.17.25,s=6.8.23 -m t=TestMetricbeatDefaultConfig,t=TestBeatKibanaRef

== buildkite test this -f p=gke,s=6.8.23 -m t=TestMetricbeatDefaultConfig,t=TestBeatKibanaRef

@naemono You should not declare different values for the same variable in -f,--fixed, only the last one is used.


Reminder:

buildkite test this -f var1=x,var2=x,.. -m var=x1,var=x2,..
  • fixed variables need to be different: var1=..,var2=..
  • mixed variables can be the same with different values: var=x1,var=x2

@kvalliyurnatt
Copy link
Contributor Author

buildkite test this -f p=gke,E2E_TAGS=beat -m s=8.16.0,s=7.17.25,s=6.8.23

buildkite test this -f p=gke -m s=8.16.0,s=7.17.25,s=6.8.23 -m t=TestMetricbeatDefaultConfig,t=TestBeatKibanaRef

== buildkite test this -f p=gke -m t=TestMetricbeatDefaultConfig,t=TestBeatKibanaRef

@kvalliyurnatt You should not use -m,--mixed multiple times in the same invocation, only the last one is used.

buildkite test this -f p=gke,s=8.16.0,s=7.17.25,s=6.8.23 -m t=TestMetricbeatDefaultConfig,t=TestBeatKibanaRef

== buildkite test this -f p=gke,s=6.8.23 -m t=TestMetricbeatDefaultConfig,t=TestBeatKibanaRef

@naemono You should not declare different values for the same variable in -f,--fixed, only the last one is used.

Reminder:

buildkite test this -f var1=x,var2=x,.. -m var=x1,var=x2,..
  • fixed variables need to be different: var1=..,var2=..
  • mixed variables can be the same with different values: var=x1,var=x2

got it, thanks @thbkrkr

So something like. this would have worked ?

test this -f p=gke -m s=8.16.0,s=7.17.25,s=6.8.23,t=TestMetricbeatDefaultConfig,t=TestBeatKibanaRef

@thbkrkr
Copy link
Contributor

thbkrkr commented Nov 6, 2024

test this -f p=gke -m s=8.16.0,s=7.17.25,s=6.8.23,t=TestMetricbeatDefaultConfig,t=TestBeatKibanaRef

Not exactly, it would run 5 testsuites in //:

  • test this -f p=gke,s=8.16.0 # run all tests on gke in 8.16.0
  • test this -f p=gke,s=7.17.25 # run all tests on gke in 7.17.25
  • test this -f p=gke,s=6.8.23 # run all tests on gke in 6.8.23
  • test this -f p=gke,t=TestMetricbeatDefaultConfig # run TestMetricbeatDefaultConfig on gke in current stack version
  • test this -f p=gke,t=TestBeatKibanaRef # run TestBeatKibanaRef on gke in current stack version

@thbkrkr
Copy link
Contributor

thbkrkr commented Nov 6, 2024

buildkite test this -f p=gke,E2E_TAGS=beat -m s=6.8.23,s=7.17.25,s=8.16.0-SNAPSHOT

We live in the future, 8.16.0 is not released yet :)

@kvalliyurnatt kvalliyurnatt enabled auto-merge (squash) November 6, 2024 15:54
@kvalliyurnatt kvalliyurnatt merged commit 05b374b into elastic:main Nov 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>bug Something isn't working exclude-from-release-notes Exclude this PR from appearing in the release notes >test Related to unit/integration/e2e tests v2.16.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants