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

8.18 test plan #15523

Open
6 of 7 tasks
endorama opened this issue Feb 3, 2025 · 7 comments
Open
6 of 7 tasks

8.18 test plan #15523

endorama opened this issue Feb 3, 2025 · 7 comments
Assignees

Comments

@endorama
Copy link
Member

endorama commented Feb 3, 2025

Manual Test Plan

List of changes: v8.18.0...v8.17.1 (v8.18.0 tag is yet to be created)

When picking up a test case, please add your name to this overview beforehand and tick the checkbox when finished.
Testing can be started when the first build candidate (BC) is available in the CFT region.

Smoke Testing ESS setup

Thanks to #8303 further smoke tests are run automatically on ESS now.
Consider extending the smoke tests to include more test cases which we'd like to cover

go-docappender library

Upgraded to 2.4.0, which includes:

apm-data library

No relevant changes.

Test cases from the GitHub board

apm-server 8.18.0 test-plan

Add yourself as assignee on the PR before you start testing.

Tasks

Preview Give feedback
  1. backport-8.18 backport-8.x backport-9.0 test-plan v8.18.0
    1pkg
  2. backport-8.x test-plan v8.18.0
  3. test-plan test-plan-ok v8.18.0
    endorama
  4. backport-8.16 backport-8.17 backport-8.x test-plan v8.18.0
    1pkg
  5. backport-skip test-plan test-plan-ok v8.18.0
    inge4pres
  6. backport-skip test-plan test-plan-ok v8.18.0
    inge4pres

Regressions

@inge4pres
Copy link
Contributor

inge4pres commented Feb 5, 2025

#14882 and #14939 are tested and working fine

@endorama
Copy link
Member Author

#15260 includes an automatic test case, skipping manual test

@endorama
Copy link
Member Author

#15177 is a minor refactor without any functional changes, I removed it from 8.18 test plan

@endorama
Copy link
Member Author

endorama commented Feb 11, 2025

#15212 includes a non functional change: apm-data has been updated to allow specifying a custom hash function but in 8.x we want to retain using crypto/md5. I removed it from 8.18 test plan.

@inge4pres I suspect this introduced some different changes in 9.x that we need to review.

@endorama
Copy link
Member Author

go-docappender elastic/go-docappender#227 has automatic test cases, skipping manual testing

@endorama
Copy link
Member Author

#15360 has been confirmed by @kruskall that will target 8.19 and I confirmed backports have been merged after feature freeze so should not be tested for 8.18.

@1pkg
Copy link
Member

1pkg commented Feb 11, 2025

#15492 and #15159 were validated against 8.18 using a local APM Server binary with the following TBS config:

  sampling.tail:
    enabled: true
    interval: 1m
    policies:
      - sample_rate: .01
    storage_limit: 1B
    discard_on_write_failure: false/true

The indexing worked as expected and the following logs were observed:

Negative case{"log.level":"warn","@timestamp":"2025-02-11T15:24:49.511-0800","log.logger":"sampling","log.origin":{"function":"github.com/elastic/apm-server/x-pack/apm-server/sampling.(*Processor).ProcessBatch","file.name":"sampling/processor.go","file.line":152},"message":"processing trace failed, discarding by default","service.name":"apm-server","error":{"message":"configured storage limit reached (current: 510, limit: 1)"},"ecs.version":"1.6.0"}

Positive case {"log.level":"warn","@timestamp":"2025-02-11T15:25:44.620-0800","log.logger":"sampling","log.origin":{"function":"github.com/elastic/apm-server/x-pack/apm-server/sampling.(*Processor).ProcessBatch","file.name":"sampling/processor.go","file.line":155},"message":"processing trace failed, indexing by default","service.name":"apm-server","error":{"message":"configured storage limit reached (current: 530, limit: 1)"},"ecs.version":"1.6.0"}

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

No branches or pull requests

3 participants