[feature]: Include ParentFee when doing a CPFP using bumpFee RPC

[ziggie1984]

Currently when we try to bump the fee of a transaction via CPFP lnd's sweeper engine it is not accounting for the parent unconfirmed transaction:

https://github.com/lightningnetwork/lnd/blob/master/lnrpc/walletrpc/walletkit_server.go#L926

I was trying to give it a shot while implementing the kweight feature (#7454) but I am not quite sure how I can fetch the inputs of the transaction here:

utxo, err := w.cfg.Wallet.FetchInputInfo(op)
	if err != nil {
		return nil, err
	}

So we have the full transaction saved in the utxo.PrevTx, but to get the fee, I need to basically do Outputs - Inputs, but I do not know how to fetch the Prevout Amounts being in the walletkit_server ? Any ideas how to do that ?

[ziggie1984]

This feature would be useful in high-fee-scenarios, and a user would not have to do all this Effective FeeCalculation but just supply the feerate for the bumpfee, and the right effective fee would be calculated.

[positiveblue]

Agree. This is one of the project ideas for the summer of bitcoin 2023

[ziggie1984]

Ohh I see it says hard, aaargh nice challenge.

[ziggie1984]

Not only should we take the parent into account when sweeping but also when opening channels with zeroconf funds.

[ziggie1984]

I think this problem is pretty easy to solve for the bitcoind backend, before using a zeroconf output, query the mempool for its size via the rpc : bitcoin-cli getmempoolentry there all the important size caluclation is done.

[kaloudis]

Would love to see this. A lot of Zeus users struggle with calculating the effective fee rate on their own

[Chinwendu20]

Hello @ziggie1984 is this issue still active, I can see you made changes to the bumpfee function

[Chinwendu20]

Hello @ziggie1984 is this issue still active, I can see you made changes to the bumpfee function

Nevermind, I can see that it is for a different issue

[yyforyongyu]

I think this will be fixed by the new sweeper?

[kaloudis]

Seems like parents are being accounting for now, at least when bumping via cli

[Roasbeef]

I think this will be fixed by the new sweeper?

Assuming it is aware of the entire input package.

[ziggie1984]

Seems like parents are being accounting for now, at least when bumping via cli

I think you talk about the bumpclosefee rpc, which takes the parent commitment fee into account but for the normal lncli wallet bumpfee we still do not take the parent feerate into account when doing a CPFP. @kaloudis Or which RPC cmd did you try ?

I think this will be fixed by the new sweeper?

So I looked at the final version of #8514, it I don't think this is covered. For the CPFP case we still don't ask our backend to provide us with the detailed ancestor size.

[Roasbeef]

For the CPFP case we still don't ask our backend to provide us with the detailed ancestor size.

Correct, we don't try to walk the transaction graph to get the fee of that unconfirmed component.