pendulum icon indicating copy to clipboard operation
pendulum copied to clipboard

Implement `--version` command

Open TorstenStueber opened this issue 1 year ago • 7 comments

Ensure that the node clients return the version number, see this message from a third party collator:

is it possible to have versioning on collators binaries? so when we use ./amplitude-collator --version we get the correct node version and also it will be displayed on telemetry correctly, so you can check for every provider which version is running

TorstenStueber avatar Apr 22 '24 13:04 TorstenStueber

Hey team! Please add your planning poker estimate with Zenhub @b-yap @bogdanS98 @ebma @gianfra-t

TorstenStueber avatar Apr 22 '24 13:04 TorstenStueber

This ticket also entails updating our version numbers in all Cargo.toml files, doesn't it? Is this something we should talk about @TorstenStueber? Like, when do we want to bump to new major, minor, or patch versions? Or do we want to keep this number aligned to something else?

ebma avatar Apr 23 '24 08:04 ebma

Mmhh, I did not think about this. Just thought that the version number could just be the version of the runtime. But that would not really be correct...

Then it probably makes sense to track the version of the client in the Cargo.toml of the client. We would then need to define a scheme for version numbers.

TorstenStueber avatar Apr 23 '24 08:04 TorstenStueber

I think the current node-template already has this --version flag. Not sure what it returns, but it is likely that the person was referring to this. If this is the case we should modify the client to return the same to be consistent with later versions.

gianfra-t avatar Apr 24 '24 12:04 gianfra-t

Yes, that would make sense.

TorstenStueber avatar Apr 24 '24 14:04 TorstenStueber

As far as I know, our node binaries also already support the --version flag but it's currently quite useless as we don't update the version in the Cargo.toml files which is probably the main concern of the external operators.

ebma avatar Apr 25 '24 17:04 ebma

Okay, so the right solution would just be to add the workflow that for every client release we actively bump the version and nothing else would need to be done here ....

TorstenStueber avatar Apr 25 '24 17:04 TorstenStueber