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

Update CI full sync to use available snapshots #1523

Merged
merged 1 commit into from
Nov 7, 2022
Merged

Update CI full sync to use available snapshots #1523

merged 1 commit into from
Nov 7, 2022

Conversation

Jouzo
Copy link
Collaborator

@Jouzo Jouzo commented Oct 23, 2022 •
edited
Loading

What kind of PR is this?:

/kind feature

What this PR does / why we need it:

Update CI full sync :

  • Generates matrix from available snapshots.
  • Get available snapshots depending on base branch
  • Remove the need for custom image and download the snapshot in default container

@Jouzo Jouzo added the ci/sync label Oct 23, 2022
@Jouzo Jouzo force-pushed the feature/update_fullsync_ci branch 2 times, most recently from 8710a99 to d667077 Compare October 24, 2022 08:57
@Jouzo Jouzo changed the base branch from master to epic/grandcentral October 24, 2022 20:07
@Jouzo Jouzo changed the base branch from epic/grandcentral to master October 31, 2022 09:44
@Jouzo Jouzo force-pushed the feature/update_fullsync_ci branch 3 times, most recently from 920ba0a to a7b539f Compare October 31, 2022 20:50
@Jouzo Jouzo marked this pull request as draft October 31, 2022 21:26
@Jouzo Jouzo force-pushed the feature/update_fullsync_ci branch 6 times, most recently from 23b7752 to efe34d0 Compare November 2, 2022 13:50
@Jouzo Jouzo force-pushed the feature/update_fullsync_ci branch from 9efda97 to 1ba0fe8 Compare November 2, 2022 17:49
@Jouzo Jouzo marked this pull request as ready for review November 4, 2022 11:54
@prasannavl prasannavl merged commit 51cea8f into master Nov 7, 2022
@prasannavl prasannavl deleted the feature/update_fullsync_ci branch November 7, 2022 02:08
@Jouzo Jouzo mentioned this pull request Nov 24, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@prasannavl prasannavl prasannavl approved these changes

@Bushstar Bushstar Awaiting requested review from Bushstar Bushstar is a code owner

@Mixa84 Mixa84 Awaiting requested review from Mixa84

@wafflespeanut wafflespeanut Awaiting requested review from wafflespeanut

@dcorral dcorral Awaiting requested review from dcorral

@mambisi mambisi Awaiting requested review from mambisi

@fuxingloh fuxingloh Awaiting requested review from fuxingloh

Assignees

@Jouzo Jouzo

Labels
ci/sync
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Jouzo @prasannavl @defichain-bot