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

Print latest packet sequences in log or app.yaml #146

Open
orkunkl opened this issue Mar 30, 2021 · 4 comments
Open

Print latest packet sequences in log or app.yaml #146

orkunkl opened this issue Mar 30, 2021 · 4 comments

Comments

@orkunkl
Copy link
Contributor

orkunkl commented Mar 30, 2021

Helpful when debugging and tracking current status of relayer

@bartmacbartek
Copy link
Contributor

There's last-queried-heights.json file at $HOME/.ibc-setup directory (by default):

type LastQueriedHeights = {
  packetHeightA: number,
  packetHeightB: number,
  ackHeightA: number,
  ackHeightB: number
}

Is it any helpful? If not, could you elaborate a bit more on what you would like to see in logs, please?

@orkunkl
Copy link
Contributor Author

orkunkl commented Apr 12, 2021

Not exactly, LastQueriedHeights is actually shown in the logs while the relayer is running, but packet height and ack height indicates the latest heights a packet sent and acknowledged.
Packets has a sequence identifier inside that is sequential and would be helpful to see this in the logs.

interface Packet {
  sequence: uint64
  timeoutHeight: Height
  timeoutTimestamp: uint64
  sourcePort: Identifier
  sourceChannel: Identifier
  destPort: Identifier
  destChannel: Identifier
  data: bytes
}

@ethanfrey
Copy link
Contributor

I cannot add it as info level, as we print one line for all packets:

Relay ${packets.length} packets from ${this.chain(source)} => ${this.otherChain(source)}

We do show it in verbose mode where we show the data of each packet:

Sending packet #${packet.sequence.toNumber()} from ${this.chainId}:${packet.sourceChannel}, presentPacketData(packet.data)

This contains the channel and sequence along with the full packet application data (eg. ics20 source/receive/denom/amount).

You can just run with -v to get this info. Or is there some more info you need?

@orkunkl
Copy link
Contributor Author

orkunkl commented Jun 25, 2021

It has been some time since I created this request. I should have explained the issue better in the description. If packet height provided using verbose mode, all fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants