Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[O11y][MySQL] Fix optional chaining in the replica_status data stream pipeline #12028

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 0 commits into from
Dec 16, 2024
Merged

[O11y][MySQL] Fix optional chaining in the replica_status data stream pipeline #12028

merged 0 commits into from
Dec 16, 2024

Conversation

harnish-crest-data
Copy link
Contributor

@harnish-crest-data harnish-crest-data commented Dec 9, 2024 •
edited by shmsr
Loading

Proposed commit message

Fix optional chaining in the replica_status data stream pipeline

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

Related issues

Screenshots

image

@harnish-crest-data harnish-crest-data added Integration:mysql MySQL flaky-test Unstable or unreliable test cases. Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations] bugfix Pull request that fixes a bug issue labels Dec 9, 2024
@harnish-crest-data harnish-crest-data self-assigned this Dec 9, 2024
@harnish-crest-data harnish-crest-data changed the title [O11y][MySQL] Fix optional chaining in the replica_status data stream pipeline [Don't Merge][O11y][MySQL] Fix optional chaining in the replica_status data stream pipeline Dec 9, 2024
@harnish-crest-data
Copy link
Contributor Author

/test

@harnish-crest-data harnish-crest-data changed the title [Don't Merge][O11y][MySQL] Fix optional chaining in the replica_status data stream pipeline [O11y][MySQL] Fix optional chaining in the replica_status data stream pipeline Dec 9, 2024
@elastic-vault-github-plugin-prod
Copy link

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@harnish-crest-data harnish-crest-data marked this pull request as ready for review December 9, 2024 10:24
@harnish-crest-data harnish-crest-data requested a review from a team as a code owner December 9, 2024 10:24
@elastic-sonarqube Elastic SonarQube
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
97.4% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube

@elasticmachine
Copy link

💚 Build Succeeded

History

  • 💚 Build #19267 succeeded 78347335bef32b162523e9ceea95ab94ded57011
  • 💔 Build #19266 failed eab27f571f16e2a319e83a95074edf2907a4068f
  • 💔 Build #19265 failed 1133bdb2906fd086dcb7741656246ca388aa5167
  • 💚 Build #19204 succeeded e5d085666c0e0fb95be8a06910aff50f8e1fb2cf
  • 💔 Build #19203 failed 8f8dfc09774ffde602bb3c574ec5f537eaeed208
  • 💔 Build #19201 failed 8f8dfc09774ffde602bb3c574ec5f537eaeed208

cc @harnish-elastic

@muthu-mps
Copy link
Contributor

@harnish-elastic - Can you confirm that the log response format for the timestamp fields are in the expected format which we use in our pipeline tests?

@harnish-crest-data
Copy link
Contributor Author

@harnish-elastic - Can you confirm that the log response format for the timestamp fields are in the expected format which we use in our pipeline tests?

Yes, it is!

@harnish-crest-data harnish-crest-data merged commit c94aaa8 into elastic:main Dec 16, 2024
5 checks passed
@elastic-vault-github-plugin-prod
Copy link

Package mysql - 1.25.2 containing this change is available at https://epr.elastic.co/package/mysql/1.25.2/

harnish-crest-data added a commit to chavdaharnish/integrations that referenced this pull request Feb 4, 2025
@harnish-crest-data
[O11y][MySQL] Fix optional chaining in the replica_status data stream…
60cd3fe 
… pipeline (elastic#12028)

* fix optional chaining in the replica_status data stream pipeline
harnish-crest-data added a commit to chavdaharnish/integrations that referenced this pull request Feb 5, 2025
@harnish-crest-data
[O11y][MySQL] Fix optional chaining in the replica_status data stream…
f2f33c4 
… pipeline (elastic#12028)

* fix optional chaining in the replica_status data stream pipeline
@harnish-crest-data harnish-crest-data deleted the mysql-resolve-flaky-test branch February 7, 2025 06:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@muthu-mps muthu-mps muthu-mps approved these changes

@andrewkroh andrewkroh Awaiting requested review from andrewkroh

Assignees

@harnish-crest-data harnish-crest-data

Labels
bugfix Pull request that fixes a bug issue flaky-test Unstable or unreliable test cases. Integration:mysql MySQL Team:Obs-InfraObs Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations]
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[LogsDB] [Stack 8.17.0-SNAPSHOT] [mysql] Failing test daily: system test: default (variant: percona_8_0_36) in mysql.replica_status
4 participants
@harnish-crest-data @elasticmachine @muthu-mps @andrewkroh