graphprotocol / graphprotocol/graph-node
[Bug] graphman accepts deployment or namespace misleading to errors
まだ誰も着手していません。
- 主要言語
- Rust
- スター
- 3.2k
- フォーク
- 1.1k
- 平均マージ
- 4日 1時間
- マージ済み PR(30日)
- 1
説明
Bug report
In graphman help, like graphman drop --help it accepts the DEPLOYMENT as an identifier to which SG to remove. The Deployment is unique everywhere in our system and if I try to drop a SG by using the deployment it will get removed from everywhere as usual.
The problem is that the drop command also accepts the namespace, which is unique only to the respective shard where this SG is deployed.
The graphman shouldn't accept the namespace if the help is saying to use the deployment identifier, otherwise it leads to critical mistakes when we want to remove the SG only from a specific shard and not from everywhere.
Also if you might consider this also as a feature request, there should be a possibility to drop the SG only from a specific shard by using it's namespace (if it's a single SG in that table, otherwise throw a warning that's not possible to remove it because there are multiple deployments using the same table)
Relevant log output
No response
IPFS hash
No response
Subgraph name or link to explorer
No response
Some information to help us out
- Tick this box if this bug is caused by a regression found in the latest release.
- Tick this box if this bug is specific to the hosted service.
- I have searched the issue tracker to make sure this issue is not a duplicate.
OS information
macOS
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
graphman の drop --help エントリーポイントから始め、namespace が指定された場合にそのデプロイメント識別子がどのように解決されるかを追跡します。shard 間の現在の動作を確認し、namespace の値がデプロイメント識別子として受け付けられないようにすることを完了条件と定義するとともに、shard ごとの削除がスコープに含まれるかどうかを明確にします。issue ではファイルパスもテストパスも指定されていません。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- rust
- 領域
- cli
- issue の種類
- バグ
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 35/100