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.

Sign up for GitHub

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

Jump to bottom

[Feature] Migrate to github container registry #627

Closed
5 of 13 tasks
EngHabu opened this issue Nov 30, 2020 · 0 comments
Closed
5 of 13 tasks

[Feature] Migrate to github container registry #627

EngHabu opened this issue Nov 30, 2020 · 0 comments
Labels
enhancement New feature or request untriaged This issues has not yet been looked at by the Maintainers
Milestone
0.10.0

Comments

@EngHabu
Copy link
Contributor

EngHabu commented Nov 30, 2020

Motivation: Why do you think this is important?
Github container registry allows true anonymous pulls, we can finally stop using dockerhub registry and consolidate CI in github.

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
@EngHabu EngHabu added enhancement New feature or request untriaged This issues has not yet been looked at by the Maintainers labels Nov 30, 2020
This was referenced Nov 30, 2020
Merged
Merged
Merged
Merged
Merged
Merged
Merged
@EngHabu EngHabu closed this as completed Dec 3, 2020
@EngHabu EngHabu added this to the 0.10.0 milestone Dec 3, 2020
eapolinario added a commit to eapolinario/flyte that referenced this issue Dec 20, 2022
@kumare3 @eapolinario
Feat: Intratask checkpoint Example (flyteorg#627)
7a1aa0f 
* [WIP] Checkpoint example work in progress

Signed-off-by: Ketan Umare <[email protected]>

* fixing use_secrets

Signed-off-by: Ketan Umare <[email protected]>

* Intratask checkpoint update

Signed-off-by: Ketan Umare <[email protected]>

* update

Signed-off-by: Ketan Umare <[email protected]>

* update

Signed-off-by: Ketan Umare <[email protected]>

* updated

Signed-off-by: Ketan Umare <[email protected]>

* print

Signed-off-by: Ketan Umare <[email protected]>

* Dockerfile

Signed-off-by: Ketan Umare <[email protected]>

* retryable exception

Signed-off-by: Ketan Umare <[email protected]>

* checkpoint

Signed-off-by: Ketan Umare <[email protected]>

* encode and decode

Signed-off-by: Ketan Umare <[email protected]>

* update

Signed-off-by: Ketan Umare <[email protected]>

* updated

Signed-off-by: Ketan Umare <[email protected]>

* better names

Signed-off-by: Ketan Umare <[email protected]>

* updated

Signed-off-by: Ketan Umare <[email protected]>

* fix logic

Signed-off-by: Ketan Umare <[email protected]>

* Checkpoint

Signed-off-by: Ketan Umare <[email protected]>

* Dockerfile update and code moved

Signed-off-by: Ketan Umare <[email protected]>

* version update

Signed-off-by: Ketan Umare <[email protected]>

* Fix import

Signed-off-by: Ketan Umare <[email protected]>

* updated to address comments

Signed-off-by: Ketan Umare <[email protected]>

* add version comment

Signed-off-by: Ketan Umare <[email protected]>

* Use flytekit==0.30.0

Signed-off-by: Eduardo Apolinario <[email protected]>

* Remove extra cruft

Signed-off-by: Eduardo Apolinario <[email protected]>

* s/a/n_iterations/

Signed-off-by: Eduardo Apolinario <[email protected]>

* Incorporate review feedback

Signed-off-by: Eduardo Apolinario <[email protected]>

Co-authored-by: Eduardo Apolinario <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request untriaged This issues has not yet been looked at by the Maintainers
Projects
None yet
Milestone
0.10.0
Development

No branches or pull requests
1 participant
@EngHabu