volcano icon indicating copy to clipboard operation
volcano copied to clipboard

We should recall `api` package back to `volcano` repo

Open zen-xu opened this issue 3 years ago β€’ 13 comments

Why is this needed:

We have separated the api package from the main repo. #1401

But it will bring some difficulties to development

e.g. If I want to add some experiment fields to api, I must submit a pr to api repo first. But an experiment idea shouldn't be merged into the main thread before it is mature.

What would you like to be added:

Maybe we can still keep api package in volcano repo and sync this sub package to its own repo (like what Kubernetes did)

zen-xu avatar May 10 '21 06:05 zen-xu

/cc @Thor-wl

zen-xu avatar May 10 '21 06:05 zen-xu

+1 for this.

shinytang6 avatar May 10 '21 07:05 shinytang6

But an experiment idea shouldn't be merged into the main thread before it is mature.

That should be ok for alpha, beta and GA feature.

k82cn avatar May 11 '21 02:05 k82cn

Hello πŸ‘‹ Looks like there was no activity on this issue for last 90 days. Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! πŸ€— If there will be no activity for 60 days, this issue will be closed (we can always reopen an issue if we need!).

stale[bot] avatar Dec 15 '21 20:12 stale[bot]

+1

hwdef avatar Dec 16 '21 02:12 hwdef

@hwdef Can you help for that?

Thor-wl avatar Dec 16 '21 02:12 Thor-wl

@hwdef Can you help for that?

Yes, I can work on it. But a robot may be needed to synchronize the two repo, I don’t know how to deal with it

hwdef avatar Dec 16 '21 02:12 hwdef

Maybe we can still keep api package in volcano repo and sync this sub package to its own repo (like what Kubernetes did)

That's better to me :)

k82cn avatar Dec 16 '21 02:12 k82cn

Hello πŸ‘‹ Looks like there was no activity on this issue for last 90 days. Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! πŸ€— If there will be no activity for 60 days, this issue will be closed (we can always reopen an issue if we need!).

stale[bot] avatar Mar 25 '22 01:03 stale[bot]

Closing for now as there was no activity for last 60 days after marked as stale, let us know if you need this to be reopened! πŸ€—

stale[bot] avatar May 31 '22 03:05 stale[bot]

I think this is still needed

hwdef avatar Jun 02 '22 07:06 hwdef

@Thor-wl

hwdef avatar Jun 02 '22 07:06 hwdef

Hello πŸ‘‹ Looks like there was no activity on this issue for last 90 days. Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! πŸ€— If there will be no activity for 60 days, this issue will be closed (we can always reopen an issue if we need!).

stale[bot] avatar Sep 08 '22 22:09 stale[bot]

still need

hwdef avatar Oct 17 '22 14:10 hwdef

Hello πŸ‘‹ Looks like there was no activity on this issue for last 90 days. Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! πŸ€— If there will be no activity for 60 days, this issue will be closed (we can always reopen an issue if we need!).

stale[bot] avatar Jan 21 '23 10:01 stale[bot]

still need

hwdef avatar Jan 21 '23 12:01 hwdef

Hello πŸ‘‹ Looks like there was no activity on this issue for last 90 days. Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! πŸ€— If there will be no activity for 60 days, this issue will be closed (we can always reopen an issue if we need!).

stale[bot] avatar Apr 25 '23 21:04 stale[bot]

still need

wangyang0616 avatar Apr 27 '23 01:04 wangyang0616

Hello πŸ‘‹ Looks like there was no activity on this issue for last 90 days. Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! πŸ€— If there will be no activity for 60 days, this issue will be closed (we can always reopen an issue if we need!).

stale[bot] avatar Aug 07 '23 05:08 stale[bot]

Can we include this in the next release cycle? We can have staging/ folder under the v/v and sync content to v/api

kevin-wangzefeng avatar Aug 10 '23 14:08 kevin-wangzefeng

Can we include this in the next release cycle? We can have staging/ folder under the v/v and sync content to v/api

we need a robot to sync v/v/staging to v/api.

hwdef avatar Aug 11 '23 02:08 hwdef