Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

DNM: experimenting with buildkite #364

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

DNM: experimenting with buildkite #364

wants to merge 4 commits into from

Conversation

chrisseto
Copy link
Contributor

No description provided.

@chrisseto chrisseto force-pushed the chris/p/ci-integration-tests branch 2 times, most recently from 039ab72 to 59564cb Compare December 17, 2024 21:44
RafalKorepta
RafalKorepta reviewed Jan 2, 2025
View reviewed changes
taskfiles/ci.yml Outdated Show resolved Hide resolved
RafalKorepta
RafalKorepta reviewed Jan 2, 2025
View reviewed changes
operator/internal/decommissioning/statefulset_decommissioner_test.go
testutil.SkipIfNotIntegration(t)
testutil.RequireTimeout(t, time.Minute*10)
Copy link
Contributor

Choose a reason for hiding this comment

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

Why this timeout is removed?

RafalKorepta
RafalKorepta approved these changes Jan 2, 2025
View reviewed changes
Copy link
Contributor

@RafalKorepta RafalKorepta left a comment

Choose a reason for hiding this comment

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

LGTM

@RafalKorepta
Copy link
Contributor

https://buildkite.com/redpanda/redpanda-operator/builds/3757#0193d698-6f09-4a5b-a94b-b32534aef418/221-782

✖  github.com/redpanda-data/redpanda-operator/operator/internal/controller/redpanda (10m0.344s)

https://buildkite.com/redpanda/redpanda-operator/builds/3757#0193d698-6f09-4a5b-a94b-b32534aef418/221-815

=== Failed
--
  | === FAIL: github.com/redpanda-data/redpanda-operator/operator/internal/controller/redpanda  (0.00s)
  | panic: test timed out after 10m0s
  | running tests:
  | TestRedpandaController (9m9s)
  | TestRedpandaController/TestObjectsGCed (27s)

I think TestObejctsGCed requires at least 15 minutes to finish.

@RafalKorepta
Copy link
Contributor

https://buildkite.com/redpanda/redpanda-operator/builds/3757#0193d694-a75e-4926-93b9-96df22f8e1d9/351-371

/root/go/pkg/mod/cache/vcs/8ab0686baf2b5d2ea93a2ee284f8c6b17f8880e76a9e136a6e3b2a7bd8240f88: exit status 128:
--
  | fatal: could not read Username for 'https://github.com': terminal prompts disabled

I think

redpanda-operator/.buildkite/pipeline.yml

Line 83 in 59564cb

TAG_NAME=$(ci/scripts/tag-check.sh) ./ci/scripts/run-in-nix-docker.sh task ci:k8s
requires task ci:configure.

This was referenced Jan 3, 2025
Open
Open
@RafalKorepta RafalKorepta force-pushed the chris/p/ci-integration-tests branch from 59564cb to 80a21a2 Compare January 3, 2025 16:36
@RafalKorepta RafalKorepta marked this pull request as ready for review January 3, 2025 16:36
@RafalKorepta RafalKorepta force-pushed the chris/p/ci-integration-tests branch from 80a21a2 to d69be4b Compare January 3, 2025 19:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@RafalKorepta RafalKorepta RafalKorepta approved these changes

@andrewstucki andrewstucki Awaiting requested review from andrewstucki andrewstucki is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@chrisseto @RafalKorepta