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

[Feature] flytectl cross-domain commands #578

Open
1 of 13 tasks
wild-endeavor opened this issue Nov 2, 2020 · 5 comments
Open
1 of 13 tasks

[Feature] flytectl cross-domain commands #578

wild-endeavor opened this issue Nov 2, 2020 · 5 comments
Labels
enhancement New feature or request flytectl Issues related to flytectl -Flytes CLI good first issue Good for newcomers stale

Comments

@wild-endeavor
Copy link
Contributor

wild-endeavor commented Nov 2, 2020

Motivation: Why do you think this is important?
For certain commands in flyte-cli like update-workflow-meta need to have the domain specified. This is fine but usually, when users are archiving a workflow, they want to do it for all domains. We should have an option in flytectl to do this, a --all-domains switch or something.

Goal: What should the final outcome look like, ideally?
See above.

Describe alternatives you've considered
None.

Flyte component

  • Overall
  • Flyte Setup and Installation scripts
  • Flyte Documentation
  • Flyte communication (slack/email etc)
  • FlytePropeller
  • FlyteIDL (Flyte specification language)
  • Flytekit (Python SDK)
  • FlyteAdmin (Control Plane service)
  • FlytePlugins
  • DataCatalog
  • FlyteStdlib (common libraries)
  • FlyteConsole (UI)
  • Other

GitHub repo(s)
flytectl

@wild-endeavor wild-endeavor added enhancement New feature or request untriaged This issues has not yet been looked at by the Maintainers labels Nov 2, 2020
@kumare3
Copy link
Contributor

kumare3 commented Feb 28, 2021

@pmahindrakar-oss OR @evalsocket - can either of you help with this issue?

@kumare3 kumare3 added this to the 0.12.0 milestone Feb 28, 2021
@kumare3 kumare3 removed the untriaged This issues has not yet been looked at by the Maintainers label Feb 28, 2021
@EngHabu EngHabu modified the milestones: 0.12.0, 0.13.0 Apr 1, 2021
@yindia yindia self-assigned this Apr 2, 2021
@yindia
Copy link
Contributor

yindia commented Apr 13, 2021

@kumare3 & @wild-endeavor Do we need this feature for all commands? or we just need it for archiving.

@kumare3 kumare3 removed this from the 0.13.0 milestone May 5, 2021
@EngHabu EngHabu added the flytectl Issues related to flytectl -Flytes CLI label Aug 25, 2021
@yindia yindia removed their assignment Oct 1, 2021
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Dec 20, 2022
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Dec 20, 2022
flyteorg#578)

fix: correctly coerce defaultValue when rendering LaunchPlan details

Signed-off-by: Rahul Mehta <rahul@theoremlp.com>

Signed-off-by: Rahul Mehta <rahul@theoremlp.com>
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Jul 24, 2023
* Update `flyteplugins`

Signed-off-by: Bernhard Stadlbauer <11799671+bstadlbauer@users.noreply.github.com>

* Update `flytepropeller`

Signed-off-by: Bernhard Stadlbauer <11799671+bstadlbauer@users.noreply.github.com>

* Fix imports

Signed-off-by: Bernhard Stadlbauer <11799671+bstadlbauer@users.noreply.github.com>

* Bump flyteplugins to v1.1.7

Signed-off-by: Bernhard Stadlbauer <11799671+bstadlbauer@users.noreply.github.com>

* Return `err` instead of `nil`

Signed-off-by: Bernhard Stadlbauer <11799671+bstadlbauer@users.noreply.github.com>

---------

Signed-off-by: Bernhard Stadlbauer <11799671+bstadlbauer@users.noreply.github.com>
Co-authored-by: Dan Rammer <daniel@union.ai>
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Jul 24, 2023
* Alter ID sequence to bigint

Signed-off-by: Hongxin Liang <honnix@users.noreply.github.com>

* Unit test it

Signed-off-by: Hongxin Liang <honnix@users.noreply.github.com>

---------

Signed-off-by: Hongxin Liang <honnix@users.noreply.github.com>
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Aug 9, 2023
* Update `flyteplugins`

Signed-off-by: Bernhard Stadlbauer <11799671+bstadlbauer@users.noreply.github.com>

* Update `flytepropeller`

Signed-off-by: Bernhard Stadlbauer <11799671+bstadlbauer@users.noreply.github.com>

* Fix imports

Signed-off-by: Bernhard Stadlbauer <11799671+bstadlbauer@users.noreply.github.com>

* Bump flyteplugins to v1.1.7

Signed-off-by: Bernhard Stadlbauer <11799671+bstadlbauer@users.noreply.github.com>

* Return `err` instead of `nil`

Signed-off-by: Bernhard Stadlbauer <11799671+bstadlbauer@users.noreply.github.com>

---------

Signed-off-by: Bernhard Stadlbauer <11799671+bstadlbauer@users.noreply.github.com>
Co-authored-by: Dan Rammer <daniel@union.ai>
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Aug 21, 2023
* Alter ID sequence to bigint

Signed-off-by: Hongxin Liang <honnix@users.noreply.github.com>

* Unit test it

Signed-off-by: Hongxin Liang <honnix@users.noreply.github.com>

---------

Signed-off-by: Hongxin Liang <honnix@users.noreply.github.com>
@github-actions
Copy link

Hello 👋, This issue has been inactive for over 9 months. To help maintain a clean and focused backlog, we'll be marking this issue as stale and will close the issue if we detect no activity in the next 7 days. Thank you for your contribution and understanding! 🙏

@github-actions github-actions bot added the stale label Aug 26, 2023
@github-actions
Copy link

github-actions bot commented Sep 2, 2023

Hello 👋, This issue has been inactive for over 9 months and hasn't received any updates since it was marked as stale. We'll be closing this issue for now, but if you believe this issue is still relevant, please feel free to reopen it. Thank you for your contribution and understanding! 🙏

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Sep 2, 2023
@eapolinario eapolinario reopened this Nov 2, 2023
@github-actions github-actions bot removed the stale label Nov 3, 2023
Copy link

Hello 👋, this issue has been inactive for over 9 months. To help maintain a clean and focused backlog, we'll be marking this issue as stale and will engage on it to decide if it is still applicable.
Thank you for your contribution and understanding! 🙏

@github-actions github-actions bot added the stale label Jul 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request flytectl Issues related to flytectl -Flytes CLI good first issue Good for newcomers stale
Projects
None yet
Development

No branches or pull requests

6 participants