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

Add commands to fetch active and archived launchplans #331

Merged
merged 3 commits into from
Jun 20, 2022

Conversation

SmritiSatyanV
Copy link
Contributor

@SmritiSatyanV SmritiSatyanV commented Jun 20, 2022

@codecov
Copy link

codecov bot commented Jun 20, 2022

Codecov Report

Merging #331 (341f575) into master (0cbf036) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #331   +/-   ##
=======================================
  Coverage   69.49%   69.49%           
=======================================
  Files         139      139           
  Lines        4826     4826           
=======================================
  Hits         3354     3354           
  Misses       1230     1230           
  Partials      242      242           
Flag Coverage Δ
unittests 69.07% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
cmd/get/launch_plan.go 87.14% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0cbf036...341f575. Read the comment docs.

@@ -0,0 +1,104 @@
.. _flytectl_compile:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you check if this file is added in some index .

@@ -55,6 +55,16 @@ Retrieve all the launch plans with filters:

flytectl get launchplan -p flytesnacks -d development --filter.fieldSelector="name=core.basic.lp.go_greet"

Retrieve all active launch plans with filters:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit : you can omit the with filters in both the lines.

We probably want to add all properties which are filterable in the docs not just for launchplan, but for executions, workflows, task , named entity etc and this should be available easily on docs.flyte.org

Can we create a separate ticket to capture this effort

Signed-off-by: SmritiSatyanV <[email protected]>
Signed-off-by: SmritiSatyanV <[email protected]>
@SmritiSatyanV SmritiSatyanV merged commit 7c83248 into master Jun 20, 2022
@SmritiSatyanV SmritiSatyanV deleted the list-active-lp-Smriti branch June 20, 2022 10:20
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