Skip to content

ORC-695: Publish daily snapshot#579

Closed
dongjoon-hyun wants to merge 1 commit intoapache:masterfrom
dongjoon-hyun:ORC-695
Closed

ORC-695: Publish daily snapshot#579
dongjoon-hyun wants to merge 1 commit intoapache:masterfrom
dongjoon-hyun:ORC-695

Conversation

@dongjoon-hyun
Copy link
Copy Markdown
Member

@dongjoon-hyun dongjoon-hyun commented Dec 6, 2020

What changes were proposed in this pull request?

This PR aims to switch to daily snapshot publishing instead of commit-by-commit snapshot publishing.

Why are the changes needed?

This can recover the snapshot repository daily if the snapshot publishing fails due to some flakiness.

How was this patch tested?

N/A.

@dongjoon-hyun
Copy link
Copy Markdown
Member Author

dongjoon-hyun commented Dec 6, 2020

Hi, @pgaref .
Could you review this PR, please?

@dongjoon-hyun
Copy link
Copy Markdown
Member Author

dongjoon-hyun commented Dec 6, 2020

Ur, sorry, but please wait. I'll convert this as Draft. I need to do double-check something.

@dongjoon-hyun dongjoon-hyun marked this pull request as draft December 6, 2020 09:44
@dongjoon-hyun dongjoon-hyun marked this pull request as ready for review December 6, 2020 09:49
@dongjoon-hyun dongjoon-hyun marked this pull request as draft December 6, 2020 09:51
@dongjoon-hyun
Copy link
Copy Markdown
Member Author

This should be used at master branch only.
For the other branch, let's see after that.

@dongjoon-hyun dongjoon-hyun marked this pull request as ready for review December 6, 2020 09:56
@dongjoon-hyun
Copy link
Copy Markdown
Member Author

I'll close this.

@dongjoon-hyun dongjoon-hyun deleted the ORC-695 branch December 6, 2020 10:12
@dongjoon-hyun
Copy link
Copy Markdown
Member Author

Ping @pgaref once more since it seems that you missed here.
Please see the PR content.

- push:
-   branches:
-     - master
+ schedule:
+ - cron:  '0 0 * * *'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant