fix: Increase test-cluster TTL to 6h #123
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@Techassi, FYI for the new integration tests. I am thinking the infrastructure.yaml since only one operator needs it longer for now. What do you suggest? |
|
I think we should keep the default as low as possible (where most operators still succeed). If individual operators need longer TTLs, we can set it in the integration test config file. |
NickLarsenNZ
added a commit
to stackabletech/hdfs-operator
that referenced
this pull request
Nov 12, 2024
|
Sounds good! Was thinking of something similar :) |
github-merge-queue bot
pushed a commit
to stackabletech/hdfs-operator
that referenced
this pull request
Dec 13, 2024
* chore: Generated commit to update templated files since the last template run up to stackabletech/operator-templating@52c2678 Reference-to: stackabletech/operator-templating@52c2678 (Add integration test workflow) * chore: Add integration test config file * chore(ci): Set test parameters for scheduled runs * chore(test): bump the cluster-ttl as per stackabletech/ci#123 * bump action version * update infrastructure * drop the nodes * another infrastructure tweak --------- Co-authored-by: Techassi <[email protected]> Co-authored-by: Nick Larsen <[email protected]> Co-authored-by: Razvan-Daniel Mihai <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As HDFS tests can take longer than 4 hours, currently this deletes the k8s before the test finished, resulting in
Get "https://cluster-dns-1nvkkrsm.hcp.eastus.azmk8s.io:443/apis/apps/v1/namespaces/kuttl-test-fond-chipmunk/statefulsets/test-hdfs-automatic-log-namenode-default": dial tcp: lookup cluster-dns-1nvkkrsm.hcp.eastus.azmk8s.io on 8.8.8.8:53: no such host.Discussed in https://github.com/replicated-collab/stackable-replicated/issues/7