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: set overlay in statestore, more puller logs, remove chainsyncer #4325

Merged
merged 2 commits into from
Sep 19, 2023
Merged

fix: set overlay in statestore, more puller logs, remove chainsyncer #4325

merged 2 commits into from
Sep 19, 2023

Conversation

istae
Copy link
Member

@istae istae commented Sep 18, 2023 •
edited
Loading

Checklist

  • I have read the coding guide.
  • My change requires a documentation update, and I have done it.
  • I have added tests to cover my changes.
  • I have filled out the description and linked the related issues.

Description

Issues discovered during testing.

  1. Set overlay in statestore in case it's missing.
  2. final step in removing chainsyncer.,

Open API Spec Version Changes (if applicable)

Motivation and Context (Optional)

Related Issue (Optional)

Screenshots (if appropriate):

@istae istae requested a review from mrekucci September 18, 2023 14:47
@istae istae changed the title fix: set overlay in statestore and puller logs fix: set overlay in statestore, more puller logs, remove chainsyncer Sep 18, 2023
@istae istae merged commit 84b6eaa into master Sep 19, 2023
13 checks passed
@istae istae deleted the testing-fixes branch September 19, 2023 09:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mrekucci mrekucci mrekucci approved these changes

Assignees
No one assigned
Labels
pull-request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@istae @mrekucci