Skip to content

feat: add live backup copy tracking for export sql command#2518

Merged
rinatkhaziev merged 1 commit intotrunkfrom
add-live-backup-tracks-for-export-sql-command
Aug 27, 2025
Merged

feat: add live backup copy tracking for export sql command#2518
rinatkhaziev merged 1 commit intotrunkfrom
add-live-backup-tracks-for-export-sql-command

Conversation

@luismulinari
Copy link
Contributor

@luismulinari luismulinari commented Aug 27, 2025

Description

This PR enhances telemetry tracking for the vip export sql command by adding the live_backup_copy parameter to track usage of the live backup copy feature.

Pull request checklist

New release checklist

Steps to Test

Outline the steps to test and verify the PR here.

Example:

  1. Check out PR.
  2. Run npm run build
  3. Run ./dist/bin/vip-cookies.js nom
  4. Verify cookies are delicious.

Move tracker initialization after parsing CLI options to include
live_backup_copy parameter in telemetry data
@github-actions
Copy link
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@luismulinari luismulinari requested a review from a team August 27, 2025 19:47
@sonarqubecloud
Copy link

@rinatkhaziev
Copy link
Contributor

Tested, works

  @automattic/vip:analytics:clients:tracks trackEvent() {
  'events[0][_en]': 'vip_cli_dev_export_sql_command_execute',
  'events[0][cli_version]': '3.20.0',
  'events[0][os_name]': 'darwin',
  'events[0][os_version]': '24.6.0',
  'events[0][node_version]': 'v20.19.1',
  'events[0][app]': 2160,
  'events[0][env]': 'testing',
  'events[0][generate_backup]': undefined,
  'events[0][live_backup_copy]': true,
  'events[0][is_vip]': true
} +783ms

@rinatkhaziev rinatkhaziev merged commit 0751cc5 into trunk Aug 27, 2025
19 checks passed
@rinatkhaziev rinatkhaziev deleted the add-live-backup-tracks-for-export-sql-command branch August 27, 2025 20:35
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.

2 participants