Skip to content
This repository has been archived by the owner on Oct 9, 2023. It is now read-only.

CRD validation on flytepropeller project structure #367

Closed
wants to merge 10 commits into from

Conversation

pingsutw
Copy link
Member

TL;DR

Adding CRD Validation for Flyteworkflw
I applied the generated CRD and ran:

core.control_flow.dynamics.wf 
core.control_flow.run_conditions.multiplier
core.flyte_basics.lp.my_wf
core.control_flow.run_conditions.basic_boolean_wf
core.control_flow.map_task.my_map_workflow
core.control_flow.chain_tasks.chain_tasks_wf
core.type_system.enums.enum_wf
core.scheduled_workflows.lp_schedules.date_formatter_wf

successfully

Type

  • Bug Fix
  • Feature
  • Plugin

Are all requirements met?

  • Code completed
  • Smoke tested
  • Unit tests added
  • Code documentation added
  • Any pending items have an associated Issue

Complete description

auto-generate CRD YAML with openapi schema

Tracking Issue

flyteorg/flyte#355

Follow-up issue

NA

Signed-off-by: Kevin Su <[email protected]>
Signed-off-by: Kevin Su <[email protected]>
Signed-off-by: Kevin Su <[email protected]>
Signed-off-by: Kevin Su <[email protected]>
Signed-off-by: Kevin Su <[email protected]>
Signed-off-by: Kevin Su <[email protected]>
Signed-off-by: Kevin Su <[email protected]>
Signed-off-by: Kevin Su <[email protected]>
Signed-off-by: Kevin Su <[email protected]>
Signed-off-by: Kevin Su <[email protected]>
@codecov
Copy link

codecov bot commented Nov 19, 2021

Codecov Report

Merging #367 (10f75b8) into master (5f37d78) will decrease coverage by 6.27%.
The diff coverage is 0.00%.

@EngHabu
Copy link
Contributor

EngHabu commented Dec 10, 2021

Closing since we reached a different decision here: flyteorg/flyte#1928

@EngHabu EngHabu closed this Dec 10, 2021
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.

2 participants