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

创建仓库的部分多余信息考虑作为 debug 输出 #57

Closed
yi-Xu-0100 opened this issue Oct 14, 2020 · 4 comments · Fixed by #72
Closed

创建仓库的部分多余信息考虑作为 debug 输出 #57

yi-Xu-0100 opened this issue Oct 14, 2020 · 4 comments · Fixed by #72
Labels
bug Something isn't working priority/high

Comments

@yi-Xu-0100
Copy link
Contributor

我是用 balck_list 去除了这个空仓库后，能够全部完成同步，但是最后没有统计信息，不知道怎么回事。是否可以将统计信息作为输出值，这样让下一个步骤获取后，echo 出来。

链接 :https://github.com/yi-Xu-0100/hub-mirror/actions/runs/306356282

gitee 中未存在的仓库，创建过程会在 log 中输出一大段内容，是否应该作为 debug 输出？

image
@Yikun
Copy link
Owner

Yikun commented Oct 15, 2020

恩，这些信息可以直接干掉。后面对所有的curl加下-s和重定向

curl -s xxx > /dev/null

@Yikun Yikun added the roadmap Something will be completed in future. label Oct 15, 2020
@yi-Xu-0100

This comment has been minimized.

Sign in to view
@Yikun

This comment has been minimized.

Sign in to view
@yi-Xu-0100

This comment has been minimized.

Sign in to view
@yi-Xu-0100 yi-Xu-0100 changed the title 统计信息未输出，创建仓库的部分多余信息考虑作为 debug 输出 创建仓库的部分多余信息考虑作为 debug 输出 Oct 15, 2020
@yi-Xu-0100 yi-Xu-0100 mentioned this issue Oct 15, 2020
Closed
@Yikun Yikun added bug Something isn't working priority/high and removed roadmap Something will be completed in future. labels Oct 16, 2020
Yikun added a commit that referenced this issue Oct 31, 2020
@Yikun
Cleanup unuse message when create repo
faf9a0e 
Close: #57
@Yikun Yikun mentioned this issue Oct 31, 2020
Merged
@Yikun Yikun closed this as completed in #72 Oct 31, 2020
Yikun added a commit that referenced this issue Oct 31, 2020
@Yikun
Cleanup unuse message when create repo (#72)
200747b 
Close: #57
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
No one assigned
Labels
bug Something isn't working priority/high
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Cleanup unuse message when create repo Yikun/hub-mirror-action
2 participants
@Yikun @yi-Xu-0100