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

Gateway-MT and Link Sharing Service should determine if they can access satellites before they start #329

Closed
amwolff opened this issue Apr 11, 2023 · 4 comments

Comments

@amwolff
Copy link
Member

amwolff commented Apr 11, 2023

Goal

On some hosts, Gateway-MT and Link Sharing Service cannot access satellites. Without the ability to access satellites, all traffic ends up in a black hole (probably not the most precise description), timing out.

We should make a startup check that determines whether a particular server can be used.

Acceptance Criteria

  • Gateway-MT and Link Sharing Service check whether they can access satellites before they start
    • If the check fails, they should abort startup and exit to the system immediately

Links

In my opinion, we should create a shared package like startupchecks. It can contain checks themselves and instrumentation to run them. Gateway-MT and Link Sharing Service would call that before starting any HTTP servers.

@storj-gerrit
Copy link

storj-gerrit bot commented Apr 13, 2023

Change https://review.dev.storj.io/c/storj/gateway-mt/+/10185 mentions this issue.

@storj-gerrit
Copy link

storj-gerrit bot commented Apr 14, 2023

Change https://review.dev.storj.io/c/storj/gateway-mt/+/10201 mentions this issue.

@storj-gerrit
Copy link

storj-gerrit bot commented Apr 14, 2023

Change https://review.dev.storj.io/c/storj/gateway-mt/+/10202 mentions this issue.

storjBuildBot pushed a commit that referenced this issue Apr 17, 2023
Updates #329

Change-Id: If66a4eeb4972e5b9db361f90079cd5759e2036e1
@storj-gerrit
Copy link

storj-gerrit bot commented Apr 18, 2023

Change https://review.dev.storj.io/c/storj/gateway-mt/+/10227 mentions this issue.

storjBuildBot pushed a commit that referenced this issue Apr 19, 2023
Updates #329

Change-Id: I517108e4a5da7e793d7cc02aa86c55e5c03866be
storjBuildBot pushed a commit that referenced this issue Apr 19, 2023
Updates #329

Change-Id: I93f0fd0023bb422e4c822b65bd7231d298bab730
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants