Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

build.jpl: Improve reliability of build process - #109

Merged
mgalka merged 1 commit into
kernelci:mainfrom
nuclearcat:IMPROVE-reliability-patches
Oct 12, 2022
Merged

mgalka merged 1 commit into
kernelci:mainfrom
nuclearcat:IMPROVE-reliability-patches

Conversation

@nuclearcat

Copy link
Copy Markdown
Member

Several steps to improve job run reliability:
1)kubectl get nodes are not critical for process and should not cause job termination, ignore exit code. This happen often.
2)Make .kube configuration file mount as read only, as this volume is shared by multiple processed and getting corrupted.
3)Do not run "rm -rf *" as we are supposed to run in new docker image.

Signed-off-by: Denys Fedoryshchenko denys.f@collabora.com

@nuclearcat
nuclearcat force-pushed the IMPROVE-reliability-patches branch 7 times, most recently from e6f1451 to 642d9a6 Compare October 3, 2022 10:49

@mgalka mgalka left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Small comment

Comment thread jobs/build.jpl
sh(script: "az aks list -o table || exit 0")
print("K8S context: ${k8s_context}. Current nodes:")
sh(script: "kubectl --context ${k8s_context} get nodes")
sh(script: "kubectl --context ${k8s_context} get nodes || exit 0")

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

It's worth putting a comment why we're silencing the error

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

done

Comment thread jobs/build.jpl
stage("Init") {
/* Remove any leftover temporary files */
/* Remove any leftover temporary files
sh(script: "rm -rf *")

@aliceinwire aliceinwire Oct 11, 2022 •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Just remove this, leaving it make the comment meaningless. as we are not removing any leftover temporary files with the rm -rf anymore

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

done

@nuclearcat
nuclearcat force-pushed the IMPROVE-reliability-patches branch 2 times, most recently from 0c9282c to d38abac Compare October 11, 2022 12:14
Several steps to improve job run reliability:
* kubectl get nodes are not critical for process and should not
cause job termination, ignore exit code. This happen often.
* Make .kube configuration file mount as read only,
as this volume is shared by multiple processed and getting corrupted.
* Update information about rm -rf *

Signed-off-by: Denys Fedoryshchenko <denys.f@collabora.com>
@nuclearcat
nuclearcat force-pushed the IMPROVE-reliability-patches branch from d38abac to 6930287 Compare October 11, 2022 12:23
@nuclearcat

Copy link
Copy Markdown
Member Author

Latest patch tested on staging jenkins: https://bot.staging.kernelci.org/job/kernel-build/

@mgalka mgalka left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

lgtm

@mgalka
mgalka merged commit e04f78f into kernelci:main Oct 12, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants