wasmCloud icon indicating copy to clipboard operation
wasmCloud copied to clipboard

[BUG] wash uses model.list which gives a deprecation warning on wadm

Open thomas9911 opened this issue 11 months ago • 3 comments

Affected project(s)

  • [ ] documentation
  • [ ] examples
  • [ ] wasmCloud host
  • [x] wasmCloud CLI (wash)
  • [ ] wasmCloud dashboard UI (washboard)
  • [ ] capability providers
  • [ ] provider bindgen
  • [ ] control interface client
  • [ ] other / not sure

Describe the bug

I get this warning when calling wash app list

wadm::server: Received deprecated subject: model.list. Please use model.get instead

Steps to reproduce

  1. run wash app list
  2. you get the warning seen above in wadm

Expected behavior

no deprecation warning

Environment

latest wadm and wash cli (dont know the exact version, should be less than 2 weeks ago)

Screenshots / Logs / Additional context

wadm::server: Received deprecated subject: model.list. Please use model.get instead

thomas9911 avatar May 16 '25 07:05 thomas9911

Hey Folks!

I would love to work on this.

sp6370 avatar May 21 '25 17:05 sp6370

Hey @sp6370 it's yours! Please let us know if you need any help!

vados-cosmonic avatar May 21 '25 17:05 vados-cosmonic

Hey Folks!

Just started working on this. I will keep you posted. Apologies for the delayed response. 😃

sp6370 avatar Jun 09 '25 01:06 sp6370

@sp6370 Hi! Are you still working on this?

arujjval avatar Jul 04 '25 17:07 arujjval

@sp6370 Hi! Are you still working on this?

Sadly no! Feel free to pick this up @arujjval

sp6370 avatar Jul 04 '25 17:07 sp6370

@sp6370 Sure! Thanks for notifying. @vados-cosmonic I would like to work on this.

arujjval avatar Jul 05 '25 05:07 arujjval

Thanks @arijjval, let us know if you have any questions!

lachieh avatar Jul 07 '25 11:07 lachieh

@lachieh @thomas9911 I tried reproducing it, but it works in my case.

Image

arujjval avatar Jul 13 '25 15:07 arujjval

@arujjval it does work indeed, the warning is shown in wadm (the server)

thomas9911 avatar Jul 13 '25 15:07 thomas9911

Yes, to be clearer (apologies for not detailing earlier), the fix here is to make the wash app list command use model.get instead of model.list.

lachieh avatar Jul 15 '25 14:07 lachieh

@lachieh I am struggling to reproduce it. As I am new here, I don't know where wadm::server: Received deprecated subject: model.list. Please use model.get instead this warning is showing up.

arujjval avatar Jul 16 '25 17:07 arujjval

RUST_LOG=info wash app list

@arujjval This should help show the log

brooksmtownsend avatar Jul 16 '25 17:07 brooksmtownsend

@arujjval Hi! I'm interested in helping with this issue. Are you still working on this?

jtakakura avatar Aug 09 '25 01:08 jtakakura

@jtakakura sure, you can take it.

arujjval avatar Aug 09 '25 04:08 arujjval

@arujjval Thanks a lot! I appreciate you letting me take this. I'll get started right away. @vados-cosmonic Could you please assign this issue to me?

jtakakura avatar Aug 09 '25 07:08 jtakakura