Skip to content

Commit

Permalink
all(test): Bump storj-up version
Browse files Browse the repository at this point in the history
Bump the storj-up version to the latest published release.

After the bump we identified that to a certain point a satellite version
started to use a difference nonce for every new access grant creation,
so the test that creates an access grant from the API and encryption key
will never return the same access grant, despite they will allow to
perform the same access level. Hence I remove one assertion.

We also identified that the error that we got with the Edge::Config for
registering a gateway access was not cause because of using an older
storj-up version, so I rephrased the TODO comment.
  • Loading branch information
ifraixedes committed Oct 20, 2023
1 parent 5a83037 commit 7e399ac
Show file tree
Hide file tree
Showing 4 changed files with 367 additions and 145 deletions.
4 changes: 2 additions & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ integration-tests-env-down:
# than localhost and then it doesn't resolve.
# See: https://github.com/storj/up/issues/45#issuecomment-1288808260
@docker compose exec -T satellite-api storj-up credentials --s3 -e \
-a http://authservice:8888 -s satellite-api \
-a http://authservice:8888 -s satellite-api:7777 -c satellite-api:10000 \
| grep -E 'AWS|STORJ_GATEWAY' >> .tmp/env

.tmp/up/storj-up: .tmp/up
Expand All @@ -54,4 +54,4 @@ integration-tests-env-down:
.tmp/up:
mkdir -p .tmp
cd .tmp; git clone https://github.com/storj/up.git
cd .tmp/up; git checkout v1.1.0
cd .tmp/up; git checkout v1.2.3
Loading

0 comments on commit 7e399ac

Please sign in to comment.