-
Notifications
You must be signed in to change notification settings - Fork 387
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
internal/version: Change default endpoint to query #3126
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
change default domain name to point to the new version control
stefanbenten
added
Request Code Review
Code review requested
Reviewer Can Merge
If all checks have passed, non-owner can merge PR
labels
Sep 25, 2019
ghost
requested review from
Barterio and
rikysya
and removed request for
a team
September 25, 2019 20:23
mniewrzal
previously approved these changes
Sep 25, 2019
kaloyan-raev
previously approved these changes
Sep 25, 2019
Barterio
previously approved these changes
Sep 25, 2019
stefanbenten
dismissed stale reviews from Barterio, kaloyan-raev, and mniewrzal
via
September 25, 2019 20:45
1aab9d7
mniewrzal
approved these changes
Sep 25, 2019
kaloyan-raev
approved these changes
Sep 25, 2019
littleskunk
approved these changes
Sep 25, 2019
littleskunk
pushed a commit
that referenced
this pull request
Sep 25, 2019
* change default domain name change default domain name to point to the new version control * Update satellite-config.yaml.lock (cherry picked from commit c71f3a3)
bryanchriswhite
added a commit
that referenced
this pull request
Sep 26, 2019
* storj/master: web/satellite: vue style guide applying for UI (#3112) internal/testplanet: ensure monitor chore is finished before contacting satellite (#3124) build: make binaries smaller with -s and -w ldflags (#3125) satellite/satellitedb: update node version columns in UpdateCheckIn (#3129) all: use pkg/rpc instead of pkg/transport internal/version: Change default endpoint to query (#3126) satellite/satellitedb: Initial GE Satellite DB Implementation (#3049) satellite/accounting/rollup: pause in tests (#3122) jenkins: update golangci to v1.19.1 (#3119) satellite/accounting/tally: ensure we have a root piece id (#3123) internal/testplanet: set intervals to 15s by default (#3103) jenkins: disable flaky test-certificates (#3111) storagenode: avoid starting command on service stop (#3105) storagenode/winsvc: avoid the 15s sleep when stopping the service (#3098) fix certificates auth export command (#3110) satellite/repair/checker: set erasure share size in tests (#3101)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
cla-signed
Request Code Review
Code review requested
Reviewer Can Merge
If all checks have passed, non-owner can merge PR
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What: changes the service to ask version.storj.io instead of version.alpha.storj.io
Why: It allows autoupdating
Code Review Checklist (to be filled out by reviewer)