Output channel_id in listfunds

[kristapsk]

See #6020 for context. This is useful, makes more easy use of dev-forget-channel to remove channels with no transaction in a blockchain (CHANNELD_AWAITING_LOCKIN in my case), can give channel_id argument.

Changelog-None

[kristapsk]

There is CI error, but not sure how it's related to my changes.

[kristapsk]

There is CI error, but not sure how it's related to my changes.

Ok, got it, needed to also update doc/schemas/listfunds.schema.json.

[kristapsk]

Ok, I cannot figure out how to solve this error:

error[E0063]: missing field `channel_id` in initializer of `cln_rpc::model::ListfundsChannels`
    --> cln-grpc/src/convert.rs:2601:9
     |
2601 |         Self {
     |         ^^^^ missing `channel_id`

Apart from that, this is working, tested on my mainnet node.

[cdecker]

This just requires a run of make check-gen-updated which will re-generate all derived files and fail if there ahve been any changes. If there were changes you can then just add them to git and commit them alongside the commit that added the field to the schema and it should be good to go.

I can do that for you if you'd like, just let me know ^^

[cdecker]

That looks much better, thanks @kristapsk, the CI seems a bit flaky, and the failure seems unrelated, so I started another round :-)

[cdecker]

ACK 5b26a95

[rustyrussell]

Unfortunately needs a single, one-line change! Oh, and a Changelog entry!

I've done it, since it's trivial, though!

[rustyrussell]

Ack 20405b0

[vincenzopalazzo]

ACK 20405b0