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.

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

Explicit push to the gateway #1561

Merged
merged 11 commits into from
Jan 11, 2024
Merged

Explicit push to the gateway #1561

merged 11 commits into from
Jan 11, 2024

Conversation

juligasa
Copy link
Collaborator

@juligasa juligasa commented Jan 8, 2024

  1. Creation of a new flag syncing.allow-push that allows any mintter daemon to be pushed content. If that flag is false, then only group members can push content. regular p2p daemon will get content by regular periodic syncing.
  2. New documents API endpoint PushPublication that once called, blocks until it either succeeds in pushing content to the gateway or fails. It pushes the document version requested + supported materials. It is a remote call on the site which will look if it already has the blobs before engaging in any syncing operation, therefore It's safe to "abuse" it.

@juligasa juligasa added type: enhancement New feature or request scope: daemon Daemon and p2p networking priority: high Critical to have languages::Go scope:infrastructure The terraform managed infrastructure labels Jan 8, 2024
@juligasa juligasa self-assigned this Jan 8, 2024
@burdiyan
Copy link
Collaborator

LGTM. Could merge, as long as we understand that the version field does nothing in the PushPublication request, and we're OK with that.

@juligasa juligasa merged commit 15e8a36 into main Jan 11, 2024
2 checks passed
@juligasa juligasa deleted the feat/gw-direct-push branch January 11, 2024 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: high Critical to have scope: daemon Daemon and p2p networking scope:infrastructure The terraform managed infrastructure type: enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants