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

Fix/OID signature in CLI output #2104

Merged
merged 1 commit into from
Nov 29, 2022
Merged

Fix/OID signature in CLI output #2104

merged 1 commit into from
Nov 29, 2022

Conversation

carpawell
Copy link
Member

@carpawell carpawell commented Nov 25, 2022

OID signature should always be present in an object; it does not relate to the object split.

Signed-off-by: Pavel Karpy carpawell@nspcc.ru

It is about the field.

@carpawell carpawell added bug Something isn't working neofs-cli NeoFS CLI application issues 2022Q4 labels Nov 25, 2022
@carpawell carpawell self-assigned this Nov 25, 2022
@sami-nspcc
Copy link

I have recieved a notification of a new pull request. I am starting the build of images and binaries for further testing. Build number is 3716

@sami-nspcc
Copy link

I have recieved a notification of a new pull request. I am starting the build of images and binaries for further testing. Build number is 3717

@carpawell carpawell marked this pull request as ready for review November 25, 2022 11:33
@codecov
Copy link

codecov bot commented Nov 25, 2022

Codecov Report

Merging #2104 (c1a9a89) into master (816c74d) will not change coverage.
The diff coverage is n/a.

❗ Current head c1a9a89 differs from pull request most recent head 126d25a. Consider uploading reports for the commit 126d25a to get more accurate results

@@           Coverage Diff           @@
##           master    #2104   +/-   ##
=======================================
  Coverage   30.61%   30.61%           
=======================================
  Files         380      380           
  Lines       28130    28130           
=======================================
  Hits         8613     8613           
  Misses      18778    18778           
  Partials      739      739           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@sami-nspcc
Copy link

I am running integration tests

@sami-nspcc
Copy link

I have recieved a notification of a new pull request. I am starting the build of images and binaries for further testing. Build number is 3718

Copy link
Contributor

@fyrchik fyrchik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it should be always be present, not presented in the commit message

if signature := obj.Signature(); signature != nil {
cmd.Print("ID signature:\n")

// TODO(@cthulhu-rider): #1387 implement and use another approach to avoid conversion
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's use @carpawell here.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

well, in fact, i have no idea what is it about, i just moved that from one place to another

but ok, changed that

@sami-nspcc
Copy link

Test run is finished. Please download the tarball from link. Untar and use allure open to watch the report

@sami-nspcc
Copy link

I am running integration tests

@sami-nspcc
Copy link

I have recieved a notification of a new pull request. I am starting the build of images and binaries for further testing. Build number is 3719

@sami-nspcc
Copy link

Test run is finished. Please download the tarball from link. Untar and use allure open to watch the report

@sami-nspcc
Copy link

I am running integration tests

OID signature should always be present in an object; it does not relate
to the object split.

Signed-off-by: Pavel Karpy <carpawell@nspcc.ru>
@sami-nspcc
Copy link

I have recieved a notification of a new pull request. I am starting the build of images and binaries for further testing. Build number is 3720

@alexchetaev alexchetaev added U3 Regular and removed 2022Q4 labels Nov 25, 2022
@sami-nspcc
Copy link

Test run is finished. Please download the tarball from link. Untar and use allure open to watch the report

@sami-nspcc
Copy link

I am running integration tests

@sami-nspcc
Copy link

Test run is finished. Please download the tarball from link. Untar and use allure open to watch the report

@fyrchik fyrchik merged commit ca3596d into nspcc-dev:master Nov 29, 2022
@carpawell carpawell deleted the fix/cli-oid-signature-print branch November 29, 2022 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working neofs-cli NeoFS CLI application issues U3 Regular
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants