Skip to content

Conversation

@heiruwu
Copy link
Contributor

@heiruwu heiruwu commented Mar 11, 2024

Because

  • user may want to push to multiple registries, it is undesirable to define in instill.yaml

This commit

  • separate build and push script
  • remove registry from model config

@linear
Copy link

linear bot commented Mar 11, 2024

@heiruwu heiruwu merged commit 5bd79f9 into main Mar 11, 2024
@heiruwu heiruwu deleted the INS-3888 branch March 11, 2024 22:44
heiruwu added a commit that referenced this pull request Mar 13, 2024
Because

- user may want to push to multiple registries, it is undesirable to
define in `instill.yaml`

This commit

- separate `build` and `push` script
- remove `registry` from model config
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

No open projects
Status: 👋 Done

Development

Successfully merging this pull request may close these issues.

3 participants