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

added version check before starting reconciliation #64

Merged
merged 1 commit into from
Feb 26, 2024

Conversation

amirmalka
Copy link
Contributor

@amirmalka amirmalka commented Feb 22, 2024

Overview

An improvement at the synchronizer server - we check for the client version before starting a reconciliation task since Batch is not supported for clients running older synchronizer version

Copy link

Summary:

  • License scan: failure
  • Credentials scan: failure
  • Vulnerabilities scan: failure
  • Unit test: success
  • Go linting: success

@amirmalka amirmalka force-pushed the check-version-for-reconciliation branch from 9637ef0 to 298ffb2 Compare February 26, 2024 09:40
@amirmalka amirmalka changed the title WIP added version check before starting reconciliation added version check before starting reconciliation Feb 26, 2024
Copy link

Summary:

  • License scan: failure
  • Credentials scan: failure
  • Vulnerabilities scan: failure
  • Unit test: success
  • Go linting: success

Signed-off-by: Amir Malka <amirm@armosec.io>
@amirmalka amirmalka force-pushed the check-version-for-reconciliation branch from 298ffb2 to 4a49acb Compare February 26, 2024 14:54
Copy link

Summary:

  • License scan: failure
  • Credentials scan: failure
  • Vulnerabilities scan: failure
  • Unit test: success
  • Go linting: success

1 similar comment
Copy link

Summary:

  • License scan: failure
  • Credentials scan: failure
  • Vulnerabilities scan: failure
  • Unit test: success
  • Go linting: success

@amirmalka amirmalka merged commit e357e3e into main Feb 26, 2024
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants