Skip to content
This repository has been archived by the owner on Jun 6, 2024. It is now read-only.

fix: exclude defaultSchemaLocations in offline k8s validation #643

Merged
merged 15 commits into from
May 25, 2022

Conversation

royhadad
Copy link
Contributor

@royhadad royhadad commented May 24, 2022

use cases:

offline mode:
uses only the schema locations provided by the user

  • no --schema-location provided: skips k8s validation with warning
  • local --schema-location provided: validates normally
  • remote --schema-location provided: will fail because of no internet and display the error message in the correct format

online mode:
gives precedence to schema locations provided by the user, uses Datree's default schema locations as a fallback

@royhadad royhadad requested a review from a team May 24, 2022 15:39
@royhadad royhadad changed the title fix: exclude defaultSchemaLocations in offline mode fix: exclude defaultSchemaLocations in offline k8s validation May 24, 2022
@royhadad royhadad marked this pull request as draft May 24, 2022 16:42
@royhadad royhadad requested a review from eyarz May 25, 2022 08:42
@royhadad royhadad marked this pull request as ready for review May 25, 2022 08:43
@royhadad royhadad force-pushed the add_offline_check_in_k8s_validator branch 6 times, most recently from b61ab2f to 65e3952 Compare May 25, 2022 09:18
@royhadad royhadad force-pushed the add_offline_check_in_k8s_validator branch from 65e3952 to 46b341d Compare May 25, 2022 09:20
@royhadad royhadad linked an issue May 25, 2022 that may be closed by this pull request
@royhadad royhadad merged commit 8203b93 into main May 25, 2022
@delete-merged-branch delete-merged-branch bot deleted the add_offline_check_in_k8s_validator branch May 25, 2022 10:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

support schema validation in offline mode
2 participants