docs(agents): add cluster testing guidelines to AGENTS.md#1598
docs(agents): add cluster testing guidelines to AGENTS.md#1598akurinnoy wants to merge 1 commit into
Conversation
Document the build pipeline quirks that affect anyone deploying a patched dashboard image to a cluster: - Compile TypeScript before building the Docker image (lib/ is copied, not src/) - Always build multiarch (amd64+arm64) to avoid arch mismatch failures - Use a fresh image tag or imagePullPolicy=Always to avoid stale node cache - Patch the CheCluster CR instead of the Deployment directly, as the operator silently reverts direct Deployment patches - Do not delegate Docker/cluster commands to background agents Assisted-by: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: Oleksii Kurinnyi <okurinny@redhat.com>
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: akurinnoy The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Click here to review and test in web IDE: |
|
Hi! I'm che-ai-assistant — I help with your pull requests. Available commands:
|
|
Docker image build succeeded: quay.io/eclipse/che-dashboard:pr-1598 (linux/amd64, linux/arm64) kubectl patch commandkubectl patch -n eclipse-che "checluster/eclipse-che" --type=json -p="[{"op": "replace", "path": "/spec/components/dashboard/deployment", "value": {containers: [{image: "quay.io/eclipse/che-dashboard:pr-1598", name: che-dashboard}]}}]" |
|
@akurinnoy: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
3 participants
What does this PR do?
Documents common pitfalls when testing dashboard changes on a cluster. Found these during #1597.
Three rules added to
AGENTS.md:lib/, not source, so skipping the build step silently produces a stale imageScreenshot/screencast of this PR
N/A — docs only
What issues does this PR fix or reference?
Related: #1597
Is it tested? How?
Documentation only.
Release Notes
N/A
Docs PR
N/A