Skip to content

Fix make run-in-kind by upgrading KRM function versions in Dockerfile - #1166

Open
liamfallon wants to merge 3 commits into
kptdev:mainfrom
Nordix:fix-make-run-in-kind
Open

Fix make run-in-kind by upgrading KRM function versions in Dockerfile#1166
liamfallon wants to merge 3 commits into
kptdev:mainfrom
Nordix:fix-make-run-in-kind

Conversation

@liamfallon

@liamfallon liamfallon commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Fix make run-in-kind by upgrading KRM function versions in Dockerfile


Description

  • What changed: The krm function versions are upgraded in the function Dockerfile
  • Why it’s needed: The make run-in-kind target was failing
  • How it works: The correct KRM function versions are specified

Related Issue(s)

  • Closes/Fixes #

Type of Change

  • Bug fix
  • New feature
  • Enhancement
  • Refactor
  • Documentation
  • Tests
  • Other: ________

Checklist

  • Code follows project style guidelines
  • Self-reviewed changes
  • Tests added/updated
  • Documentation added/updated
  • All tests and gating checks pass

AI Disclosure

  • I have used AI in the creation of this PR.

@netlify

netlify Bot commented Sep 1, 2026

Copy link
Copy Markdown

Deploy Preview for kpt-porch ready!

Name Link
🔨 Latest commit ef5794b
🔍 Latest deploy log https://app.netlify.com/projects/kpt-porch/deploys/6a983ee2223ab10008c6d92b
😎 Deploy Preview https://deploy-preview-1166--kpt-porch.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the function-runner build image set to use a newer ensure-name-substring KRM function image so make run-in-kind can successfully build and deploy Porch in a kind cluster.

Changes:

  • Bump ensure-name-substring function image from v0.2.1 to v0.3.0 in func/Dockerfile.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread func/Dockerfile Outdated
Comment thread func/Dockerfile

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The implemented version bumps expand beyond what the PR description/title claims, increasing scope/risk without documented rationale.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Lite

Comment thread func/Dockerfile

@kushnaidu kushnaidu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm, Just need to address copilot's comment to update pr desc.

@liamfallon liamfallon changed the title Fix make run-in-kind by upgrading ensure-name-substring Fix make run-in-kind by upgrading KRM function versions in Dockerfile Sep 2, 2026
Signed-off-by: liamfallon <liam.fallon@est.tech>
Signed-off-by: liamfallon <liam.fallon@est.tech>
Signed-off-by: liamfallon <liam.fallon@est.tech>
@sonarqubecloud

sonarqubecloud Bot commented Sep 2, 2026

Copy link
Copy Markdown

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.

5 participants