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

Fix test-sync CI #2075

Merged
merged 1 commit into from
Jun 16, 2023
Merged

Fix test-sync CI #2075

merged 1 commit into from
Jun 16, 2023

Conversation

sieniven
Copy link
Member

Summary

  • Rsolved CI failing from installing deps_test due to python3 interpreter used on the self hosted runner is the system python.

Implications

  • Storage

    • Database reindex required
    • Database reindex optional
    • Database reindex not required
    • None

  • Consensus

    • Network upgrade required
    • Includes backward compatible changes
    • Includes consensus workarounds
    • Includes consensus refactors
    • None

@sieniven sieniven self-assigned this Jun 15, 2023
@prasannavl prasannavl mentioned this pull request Jun 16, 2023
Closed
@prasannavl prasannavl merged commit 2f2dce7 into master Jun 16, 2023
@prasannavl prasannavl deleted the niven/fix-sync-ci branch June 16, 2023 04:58
canonbrother pushed a commit that referenced this pull request Jun 19, 2023
@sieniven @canonbrother
CI: Elevate privileges to install on self hosted runner (#2075)
9859bec
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

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

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

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

Assignees

@sieniven sieniven

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sieniven @prasannavl