Skip to content

Kartik ci update#651

Open
kmathur wants to merge 8 commits intobluek8s:masterfrom
kmathur:kartik-ci-update
Open

Kartik ci update#651
kmathur wants to merge 8 commits intobluek8s:masterfrom
kmathur:kartik-ci-update

Conversation

@kmathur
Copy link
Collaborator

@kmathur kmathur commented Mar 10, 2026

No description provided.

kmathur and others added 8 commits March 9, 2026 21:32
- Adds json:"-" tags to IsSet, IsNull, Info fields in SetupPackage
- Fixes operator-sdk generate k8s deepcopy generation failure
- Fields use json:"-" since struct has custom UnmarshalJSON method

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
Replace placeholder comment with proper description explaining
ContainerSpec defines container runtime settings for virtual
cluster members.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
Adds missing MarshalJSON to complement existing UnmarshalJSON method.
This fixes deepcopy generation issues where the generator requires
complete JSON marshaling interface for custom types.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
Provides manual DeepCopyInto and DeepCopy implementations for
SetupPackage to bypass deepcopy generator issues with custom
JSON marshaling. This should resolve the 'invalid type' error
in operator-sdk generate k8s.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
Uses +k8s:deepcopy-gen=false to exclude SetupPackage from automatic
deepcopy generation and provides manual DeepCopy methods instead.
This should resolve the 'invalid type' error while preserving
custom JSON marshaling behavior.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Change cache key to v2 to force fresh cache
- Add make clean step before compilation
- Include go.mod in cache hash for better sensitivity
- Should resolve stale deepcopy generation issues

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
Added comment to workflow to trigger fresh CI run and test
our SetupPackage deepcopy fixes with cache busting.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
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

Successfully merging this pull request may close these issues.

1 participant