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

[FR] Change docker publish flow to use release tag instead of latest tag #666

Closed
4 tasks
blrchen opened this issue Sep 14, 2022 · 0 comments · Fixed by #668
Closed
4 tasks

[FR] Change docker publish flow to use release tag instead of latest tag #666

blrchen opened this issue Sep 14, 2022 · 0 comments · Fixed by #668
Assignees
Labels
feature New feature or request

Comments

@blrchen
Copy link
Collaborator

blrchen commented Sep 14, 2022

Willingness to contribute

No. I cannot contribute a bug fix at this time.

Feature Request Proposal

Docker image publishing process change proposal

  • For release branches, automatically publish docker image with corresponding release tag, e.g, feathr-registry:releases-v0.7.2
  • Ensure deployment picks up latest released docker image, seems the easilest way to do this is link latest tag to LKG release tag.

Motivation

What is the use case for this feature?

No impact to end user experience. Only impacts deployment experience, this ask ensures correct registry docker image is used for deployment.

Details

No response

What component(s) does this feature request affect?

  • Python Client: This is the client users use to interact with most of our API. Mostly written in Python.
  • Computation Engine: The computation engine that execute the actual feature join and generation work. Mostly in Scala and Spark.
  • Feature Registry API: The frontend API layer supports SQL, Purview(Atlas) as storage. The API layer is in Python(FAST API)
  • Feature Registry Web UI: The Web UI for feature registry. Written in React
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants