Add unit tests for InstanceMetadata
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 65/100
- issue の種類
- 機能追加
- 明瞭さ
- 明確に書かれている
- 活発さ
- 静か
- 技術スタック
- go
- 領域
- cloud, testing-qa
調査の方向性
Read InstanceMetadata in internal/provider/instances_v2.go and inspect the mock rework's parameter-asserting fake style first. Add coverage for each NIC and external-IP shape, hostname, provider ID, instance type, and both InstanceNetworkInterfaceList and InstanceExternalIpList error paths, asserting errors.Is; done means every listed shape and both errors are tested.
索引モデルが issue の本文から書いたものです。
説明
Context
InstanceMetadata (internal/provider/instances_v2.go) is the most complex function in InstancesV2 — NIC dual-stack handling, SNAT filtering, external IP mapping, provider ID formatting — and it has zero test coverage today, even though the mock already has fields to fake its inputs.
Scope
Add unit tests covering:
- v4-only, v6-only, and dual-stack NICs mapping to
NodeInternalIPaddresses. - SNAT external IPs being skipped.
- Ephemeral and floating external IPs mapping to
NodeExternalIP. - Hostname address always present.
- Provider ID formatting (
oxide://<uuid>). - Instance-type string (
<ncpus>-<memory GiB>). - Error paths for
InstanceNetworkInterfaceListandInstanceExternalIpList, asserted witherrors.Is.
Done when
Each NIC/external-IP shape has a test and both error paths are asserted.
Blocked by the mock rework (parameter-asserting fakes) — build these tests on the new fake style.
- 主要言語
- Go
- スター
- 6
- フォーク
- 2
- 平均マージ
- 2時間 50分
- マージ済み PR（30日）
- 7
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
oxidecomputer/oxide-cloud-controller-manager のほかの issue
-
Documentation Kubernetes Cloud Controller Manager (CCM)
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
-
Documentation Kubernetes Cloud Controller Manager (CCM)
難易度 2/5 1〜3時間 初心者へのやさしさ 90/100
-
Bug Kubernetes Cloud Controller Manager (CCM)
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
-
Bug Kubernetes Cloud Controller Manager (CCM)
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
oxidecomputer/oxide-cloud-controller-manager#275 · コメント 1 件 ·
-
Enhancement Kubernetes Cloud Controller Manager (CCM)
難易度 2/5 1〜3時間 初心者へのやさしさ 86/100
oxidecomputer/oxide-cloud-controller-manager の issue をすべて見る
似ている issue
-
agent-review-finding bug
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
jordansmall/spindrift#5281 ·
メンテナーはふだん 1 日以内に返信
-
unbounded value for n in `/v1/logs` can panic/OOM対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープンsmall item
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
canonical/pebble#1127 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
[schema-coverage] elasticstack_elasticsearch_synonym_set (data source)対応中かも @github-actions が今日担当しました。 オープンacceptance-tests phase-coding schema-coverage testing triaged
難易度 2/5 1〜3時間 初心者へのやさしさ 81/100
elastic/terraform-provider-elasticstack#5211 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 89/100
open-policy-agent/opa#9366 · リアクション 1 件 ·
メンテナーはふだん 1 日以内に返信
-
defect
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
nats-io/nats-server#8752 ·
メンテナーはふだん 1 日以内に返信