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

Fix to ESApi connector_sync_job_update_stats() to ensure undefined metadata passed as {} when undefined #3170

Merged
merged 5 commits into from
Jan 31, 2025

Conversation

mattnowzari
Copy link
Contributor

Related to https://github.com/elastic/search-team/issues/9194

Checklists

Pre-Review Checklist

  • this PR does NOT contain credentials of any kind, such as API keys or username/passwords (double check config.yml.example)
  • this PR has a meaningful title
  • this PR links to all relevant github issues that it fixes or partially addresses
  • this PR has a thorough description
  • Covered the changes with automated tests
  • Testes the changes locally
  • Added a label for each target release version (example: v7.13.2, v7.14.0, v8.0.0)

Related Pull Requests

#3110

…ield gets passed to client as {} and not None
@mattnowzari mattnowzari marked this pull request as ready for review January 31, 2025 15:25
@mattnowzari mattnowzari requested a review from a team as a code owner January 31, 2025 15:25
Copy link
Member

@jedrazb jedrazb left a comment

Choose a reason for hiding this comment

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

LGTM, ship if tested locally and sync completed and finished

@mattnowzari mattnowzari merged commit b038752 into main Jan 31, 2025
3 checks passed
@mattnowzari mattnowzari deleted the es_api_metadata_fix branch January 31, 2025 15:58
github-actions bot pushed a commit that referenced this pull request Jan 31, 2025
…tadata passed as {} when undefined (#3170)

Co-authored-by: Elastic Machine <[email protected]>
github-actions bot pushed a commit that referenced this pull request Jan 31, 2025
…tadata passed as {} when undefined (#3170)

Co-authored-by: Elastic Machine <[email protected]>
github-actions bot pushed a commit that referenced this pull request Jan 31, 2025
…tadata passed as {} when undefined (#3170)

Co-authored-by: Elastic Machine <[email protected]>
Copy link

💚 Backport PR(s) successfully created

Status Branch Result
9.0 #3171
8.18 #3172
8.x #3173

The backport PRs will be merged automatically after passing CI.

mattnowzari added a commit that referenced this pull request Jan 31, 2025
…ned metadata passed as {} when undefined (#3170) (#3171)

Co-authored-by: Matt Nowzari <[email protected]>
Co-authored-by: Elastic Machine <[email protected]>
mattnowzari added a commit that referenced this pull request Jan 31, 2025
…ined metadata passed as {} when undefined (#3170) (#3172)

Co-authored-by: Matt Nowzari <[email protected]>
Co-authored-by: Elastic Machine <[email protected]>
mattnowzari added a commit that referenced this pull request Jan 31, 2025
…ned metadata passed as {} when undefined (#3170) (#3173)

Co-authored-by: Matt Nowzari <[email protected]>
Co-authored-by: Elastic Machine <[email protected]>
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.

3 participants