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] Validation of Flyte fields to create new projects #519

Closed
1 of 13 tasks
kosigz-lyft opened this issue Sep 23, 2020 · 3 comments
Closed
1 of 13 tasks

[Feature] Validation of Flyte fields to create new projects #519

kosigz-lyft opened this issue Sep 23, 2020 · 3 comments
Labels
enhancement New feature or request stale untriaged This issues has not yet been looked at by the Maintainers

Comments

@kosigz-lyft
Copy link
Contributor

Motivation: Why do you think this is important?
In a previous PR, I added the capability for teams using Flyte to inject custom labels onto their projects, e.g. "slack" => "#machinelearning" or "email" => "[email protected]". This is useful for adding metadata to projects. However, one problem we have observed is that it's difficult to keep these labels up to date (and populated) if there is not some logic that organizations can add to the Flyte Admin control plane to disallow creation of new projects should labels not be included.

Goal: What should the final outcome look like, ideally?
A generic piece that organizations can use to add validation to the labels on a Flyte project. e.g. being able to require that all projects in your organization have a Slack channel attached.

Describe alternatives you've considered
The alternative is to build an external system to Flyte that monitors for changes, i.e. new projects without a Slack, and alerts.

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

[Optional] Propose: Link/Inline
Draft: flyteorg/flyteadmin#119

Is this a blocker for you to adopt Flyte
This feature will be very useful.

@kosigz-lyft kosigz-lyft added enhancement New feature or request untriaged This issues has not yet been looked at by the Maintainers labels Sep 23, 2020
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Dec 20, 2022
* fix: cache icon fro map task
* fix: icon and text center

Signed-off-by: eugenejahn <[email protected]>
@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
@kumare3
Copy link
Contributor

kumare3 commented Dec 22, 2023

@kumare3 kumare3 closed this as completed Dec 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request stale untriaged This issues has not yet been looked at by the Maintainers
Projects
None yet
Development

No branches or pull requests

3 participants