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

release-21.1: log,importccl,backupccl: add event logs for import and restore jobs #62830

Merged
merged 1 commit into from
Apr 1, 2021

Conversation

adityamaru
Copy link
Contributor

Backport 1/1 commits from #62658.

/cc @cockroachdb/release


This change adds a new CommonJobEventDetails and two implementations
of the interface in Import and Restore, to structure the events
emitted during different phases of the job execution.

Both Import/Restore will emit logs at the following stages:

  • When the job has completed preparing and is beginning execution.
  • When the job has succeeded.
  • When the job has started reverting.
  • When the job has finished reverting and is in a terminal failed state.

Fixes: #57575

Release note (sql change): IMPORT and RESTORE now emit structured events
to the OPS channel. Refer to the generated documentation for details.

@adityamaru adityamaru requested review from dt, knz and a team March 30, 2021 21:34
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@knz
Copy link
Contributor

knz commented Apr 1, 2021

Lgtm

@adityamaru adityamaru force-pushed the backport21.1-62658 branch from 964a159 to 71225ee Compare April 1, 2021 17:00
This change adds a new `CommonJobEventDetails` and two implementations
of the interface in `Import` and `Restore`, to structure the events
emitted during different phases of the job execution.

Both Import/Restore will emit logs at the following stages:
- When the job has completed preparing and is beginning execution.
- When the job has succeeded.
- When the job has started reverting.
- When the job has finished reverting and is in a terminal failed state.

Release note (sql change): IMPORT and RESTORE now emit structured events
to the OPS channel. Refer to the generated documentation for details.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants