Skip to content

The example was noop (30megs -> 30 megs)#117

Open
jkremser wants to merge 1 commit into
kedify:mainfrom
jkremser:example-was-noop
Open

The example was noop (30megs -> 30 megs)#117
jkremser wants to merge 1 commit into
kedify:mainfrom
jkremser:example-was-noop

Conversation

@jkremser

@jkremser jkremser commented Jul 2, 2026

Copy link
Copy Markdown
Member

w/ this it's 50 megs by default and after 30 seconds the requests for memory are lowered to 30 megs

Signed-off-by: Jirka Kremser <jiri.kremser@gmail.com>
@jkremser jkremser requested a review from a team as a code owner July 2, 2026 09:55
Copilot AI review requested due to automatic review settings July 2, 2026 09:55

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

Adjusts the example PodResourceProfile test data so the deployment starts with a higher memory request and then reduces it after a short delay, matching the PR description (50M initially, 30M after 30s).

Changes:

  • Increase the Deployment container memory request from 30M to 50M.
  • Keep the PodResourceProfile’s newResources.requests.memory at 30M (applied after delay: 30s).
Comments suppressed due to low confidence (1)

test-data/resources/prp.yaml:43

  • The file now ends with an extra blank line (and trailing whitespace) after newResources.requests.memory. This adds noise to diffs and can trip whitespace/lint checks; please remove the trailing empty/whitespace-only lines.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@zroubalik zroubalik 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.

good catch

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.

3 participants