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

delete file status validation for 204. #396

Merged
merged 3 commits into from
Feb 28, 2022

Conversation

princeparmar
Copy link
Contributor

@princeparmar princeparmar commented Feb 25, 2022

Signed-off-by: Pradip Parmar [email protected]

Tasks to complete before merging PR:

  • Ensure system tests are passing. If not Run them manually to check for any regressions 📋
  • Do any new system tests need added to test this change? do any existing system tests need updated? If so create a PR at 0chain/system_test
  • Merge your system tests PR to master BEFORE merging this PR

Issue 0chain/zboxcli#99
Associated PRs (Link as appropriate):
...

@service-0chain
Copy link
Contributor

Manual system tests [failure] with the following config

config value
tests skipped FALSE
system-tests master
gosdk feature/99-statuscode-204-for-file-not-found
zboxcli staging
zwalletcli staging
0chain staging
blobber staging
authorizer staging
0box staging
0block staging
0dns staging
explorer staging
0proxy staging
0search staging

@cnlangzi cnlangzi force-pushed the feature/99-statuscode-204-for-file-not-found branch from 3700e4b to 5a87376 Compare February 25, 2022 11:38
@service-0chain
Copy link
Contributor

Manual system tests [failure] with the following config

config value
tests skipped FALSE
system-tests master
gosdk feature/99-statuscode-204-for-file-not-found
zboxcli feature/99-statuscode-204-for-file-not-found
zwalletcli staging
0chain staging
blobber feature/99-statuscode-204-for-file-not-found
authorizer staging
0box staging
0block staging
0dns staging
explorer staging
0proxy staging
0search staging

@cnlangzi cnlangzi force-pushed the feature/99-statuscode-204-for-file-not-found branch from 5a87376 to 602f409 Compare February 27, 2022 17:00
….com:0chain/gosdk into feature/99-statuscode-204-for-file-not-found
@service-0chain
Copy link
Contributor

Manual system tests [success] with the following config

config value
tests skipped FALSE
system-tests master
gosdk feature/99-statuscode-204-for-file-not-found
zboxcli feature/99-statuscode-204-for-file-not-found
zwalletcli staging
0chain staging
blobber feature/99-statuscode-204-for-file-not-found
authorizer staging
0box staging
0block staging
0dns staging
explorer staging
0proxy staging
0search staging

@cnlangzi cnlangzi merged commit e9c56ab into staging Feb 28, 2022
@cnlangzi cnlangzi deleted the feature/99-statuscode-204-for-file-not-found branch February 28, 2022 00:40
yash10019coder pushed a commit that referenced this pull request Jan 5, 2025
* Move min lock demand (#458)

* min_lock_demand

* min_lock_demand

* remove dead code

* go mod

* go mod

* updated gosdk (#466)

* Rename to not_available (#457)

* remove unstake

* not_available

* go mod

* go mod

* repoint to not_available gosdk

---------

Co-authored-by: Kishan-Dhakan <[email protected]>
Co-authored-by: Kishan Dhakan <[email protected]>

* point gosdk to staging

* fix RepairCompleted (#470)

* multi upload in cli (#468)

* multi upload in cli

* workdir

* make remotepath optional

* multi download

* update go version

* update gosdkk

* update gosdk (#472)

* Update gosdk (#450)

* use gosdk of fix thumb hash

* fix gosdk version

* update gosdk

* update gosdk

* update gosdk

* updated gosdk

* updated gosdk

---------

Co-authored-by: dabasov <[email protected]>

* add flag blobber_id to download file from single blobber (#437)

* add flag blobber_id to download file from single blobber

* Updated gosdk

* update gosdk

* Delete a.txt

* chore(gomod): upgraded bls-go-binary with bls v1.35 (#396)

* added go setup inside runner for build-zbox.yaml (#446)

* added go setup inside runner for build-zbox.yaml

* fixed build-zboxcli.yml

* go mod tidy

* change exit status to 0

---------

Co-authored-by: Jayash <[email protected]>
Co-authored-by: Lz <[email protected]>
Co-authored-by: shahnawaz-creator <[email protected]>
Co-authored-by: Jayash Satolia <[email protected]>
Co-authored-by: Yury <[email protected]>

* update gosdk to latest commit (#476)

* Update gosdk for repair allocation (#461)

* update gosdk for repair alloc

* update gosdk

* update gosdk

* update gosdk

---------

Co-authored-by: Kishan-Dhakan <[email protected]>
Co-authored-by: Kishan Dhakan <[email protected]>

* point to feat gosdk (#478)

* Update gosdk - fix file callback (#477)

* update gosdk

* update gosdk

* update gosdk

* update gosdk

* add preferred_blobbers flag to cli (#482)

Co-authored-by: Yury <[email protected]>

* Fix exclude path (#479)

* Updated gosdk

* Updated gosdk

* Updated gosdk

* Updated gosdk

* Updated gosdk

* Show actual num blocks, show size for directories (#456)

* show size for dir, add actual numblocks

* update gosdk

* update gosdk

* update gosdk

* update gosdk

* update gosdk

* update gosdk

* update gosdk

* update gosdk

* updated gosdk

* updated gosdk

* Fix (#487)

* Fix (#488)

* docs: command line cli for update commmand (#481)

* docs: command line cli for update commmand

* update description

* fix multi-download in cli (#484)

* fix multidownload: add wait group

* cleanup variable names

* use multidownload gosdk commit

* upload as encrypted

* fix actual_size error

* Update the logic for update blobber and validator calls as per new models (#480)

* updating the logic for valiator and blobber udpate calls.

* fixing npe in tests

* updating the logic to populate terms and stakepoolsettings

* update the gosdk version

* update the gosdk version

* Refactoring to take the provided id as input for validator and blobber udpate setting calls.

* Updating gosdk version

* update gosdk (#489)

* update gosdk to staging

---------

Co-authored-by: Piers Shepperson <[email protected]>
Co-authored-by: Yury <[email protected]>
Co-authored-by: boddumanohar <[email protected]>
Co-authored-by: Dinmukhammed <[email protected]>
Co-authored-by: Hitenjain14 <[email protected]>
Co-authored-by: Jayash <[email protected]>
Co-authored-by: Lz <[email protected]>
Co-authored-by: shahnawaz-creator <[email protected]>
Co-authored-by: Jayash Satolia <[email protected]>
Co-authored-by: stewartie4 <[email protected]>
Co-authored-by: Sunil Kumar <[email protected]>
yash10019coder pushed a commit that referenced this pull request Jan 5, 2025
* Move min lock demand (#458)

* min_lock_demand

* min_lock_demand

* remove dead code

* go mod

* go mod

* updated gosdk (#466)

* Rename to not_available (#457)

* remove unstake

* not_available

* go mod

* go mod

* repoint to not_available gosdk

---------

Co-authored-by: Kishan-Dhakan <[email protected]>
Co-authored-by: Kishan Dhakan <[email protected]>

* point gosdk to staging

* fix RepairCompleted (#470)

* multi upload in cli (#468)

* multi upload in cli

* workdir

* make remotepath optional

* multi download

* update go version

* update gosdkk

* update gosdk (#472)

* Update gosdk (#450)

* use gosdk of fix thumb hash

* fix gosdk version

* update gosdk

* update gosdk

* update gosdk

* updated gosdk

* updated gosdk

---------

Co-authored-by: dabasov <[email protected]>

* add flag blobber_id to download file from single blobber (#437)

* add flag blobber_id to download file from single blobber

* Updated gosdk

* update gosdk

* Delete a.txt

* chore(gomod): upgraded bls-go-binary with bls v1.35 (#396)

* added go setup inside runner for build-zbox.yaml (#446)

* added go setup inside runner for build-zbox.yaml

* fixed build-zboxcli.yml

* go mod tidy

* change exit status to 0

---------

Co-authored-by: Jayash <[email protected]>
Co-authored-by: Lz <[email protected]>
Co-authored-by: shahnawaz-creator <[email protected]>
Co-authored-by: Jayash Satolia <[email protected]>
Co-authored-by: Yury <[email protected]>

* update gosdk to latest commit (#476)

* Update gosdk for repair allocation (#461)

* update gosdk for repair alloc

* update gosdk

* update gosdk

* update gosdk

---------

Co-authored-by: Kishan-Dhakan <[email protected]>
Co-authored-by: Kishan Dhakan <[email protected]>

* point to feat gosdk (#478)

* Update gosdk - fix file callback (#477)

* update gosdk

* update gosdk

* update gosdk

* update gosdk

* add preferred_blobbers flag to cli (#482)

Co-authored-by: Yury <[email protected]>

* Fix exclude path (#479)

* Updated gosdk

* Updated gosdk

* Updated gosdk

* Updated gosdk

* Updated gosdk

* Show actual num blocks, show size for directories (#456)

* show size for dir, add actual numblocks

* update gosdk

* update gosdk

* update gosdk

* update gosdk

* update gosdk

* update gosdk

* update gosdk

* update gosdk

* updated gosdk

* updated gosdk

* Fix

* Fix

* Fix (#487)

* Fix (#488)

* docs: command line cli for update commmand (#481)

* docs: command line cli for update commmand

* update description

* fix multi-download in cli (#484)

* fix multidownload: add wait group

* cleanup variable names

* use multidownload gosdk commit

* upload as encrypted

* fix actual_size error

* Update the logic for update blobber and validator calls as per new models (#480)

* updating the logic for valiator and blobber udpate calls.

* fixing npe in tests

* updating the logic to populate terms and stakepoolsettings

* update the gosdk version

* update the gosdk version

* Refactoring to take the provided id as input for validator and blobber udpate setting calls.

* Updating gosdk version

* update gosdk (#489)

* Updated gosdk

* Updated gosdk

* Updated gosdk

---------

Co-authored-by: Piers Shepperson <[email protected]>
Co-authored-by: Yury <[email protected]>
Co-authored-by: Kishan-Dhakan <[email protected]>
Co-authored-by: Kishan Dhakan <[email protected]>
Co-authored-by: boddumanohar <[email protected]>
Co-authored-by: Dinmukhammed <[email protected]>
Co-authored-by: Hitenjain14 <[email protected]>
Co-authored-by: Lz <[email protected]>
Co-authored-by: shahnawaz-creator <[email protected]>
Co-authored-by: stewartie4 <[email protected]>
Co-authored-by: Sunil Kumar <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants