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

判断目的仓是否存在时，应考虑分页场景 #66

Closed
Yikun opened this issue Oct 27, 2020 · 0 comments · Fixed by #71
Closed

判断目的仓是否存在时，应考虑分页场景 #66

Yikun opened this issue Oct 27, 2020 · 0 comments · Fixed by #71
Assignees
@Yikun
Labels
bug Something isn't working priority/high

Comments

@Yikun
Copy link
Owner

Yikun commented Oct 27, 2020

判断目的仓是否存在时，应考虑分页场景，否则会造成没在第一页的话，就认为不在，而调用api创建。

[1] https://github.com/Yikun/hub-mirror-action/blob/master/entrypoint.sh#L129
@Yikun Yikun added bug Something isn't working priority/high labels Oct 27, 2020
@Yikun Yikun self-assigned this Oct 27, 2020
Yikun added a commit that referenced this issue Oct 31, 2020
@Yikun
Add paging support for destiantion repo list
c3790b2 
Close: #66
@Yikun Yikun mentioned this issue Oct 31, 2020
Merged
@Yikun Yikun closed this as completed in #71 Oct 31, 2020
Yikun added a commit that referenced this issue Oct 31, 2020
@Yikun
Add paging support for destination repo list (#71)
ce679fd 
Close: #66
Yikun added a commit that referenced this issue Oct 31, 2020
@Yikun
Add paging support for destination repo list (#71)
dc39b5b 
Close: #66
Yikun added a commit that referenced this issue Oct 31, 2020
@Yikun
Add paging support for destination repo list (#71)
5d45527 
Close: #66
@Yikun Yikun mentioned this issue Nov 2, 2020
Closed
yi-Xu-0100 added a commit to yi-Xu-0100/hub-mirror-action that referenced this issue Jan 27, 2021
@yi-Xu-0100 @Yikun
♻ test merge (#1)
6c9dade 
* Change the get time script (Yikun#68)

Change the get time script to the same version with actions/cache example

[1] https://github.com/actions/cache

* Add paging support for destination repo list (Yikun#71)

Close: Yikun#66

* Cleanup unuse message when create repo (Yikun#72)

Close: Yikun#57

* Skip follow-up step after failed (Yikun#74)

* Add timeout support (Yikun#75)

* Add retry with timeout (Yikun#76)

In this patch, we add the retry function, it retries cmd with timeout after the cmd excuted failed.

Every retries cmd will sleep 2**times, and now we set the retry times to 3.

* Change the default timeout args to `30m`

* Using grep -Fx to match the repo name (Yikun#81)

Co-authored-by: Yikun Jiang <yikunkero@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Yikun Yikun

Labels
bug Something isn't working priority/high
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add paging support for destiantion repo list Yikun/hub-mirror-action
1 participant
@Yikun