-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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.
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
Add a presubmit job for kubernetes-client/haskell #15331
Conversation
Welcome @akshaymankar! |
Hi @akshaymankar. Thanks for your PR. I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions 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/test-infra repository. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/test all
config/jobs/kubernetes-client/haskell/client-haskell-presubmits.yml
Outdated
Show resolved
Hide resolved
- mbohlool | ||
- jkachmar | ||
- jonschoning | ||
- akshaymankar |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for including an owners file!
Please add some of these people to the review by typing
/assign @brendandburns
or
/cc @mbohlool
config/jobs/kubernetes-client/haskell/client-haskell-presubmits.yml
Outdated
Show resolved
Hide resolved
args: | ||
- -c | ||
- | | ||
set -euo pipefail |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You may have an easier time if you put this code in a bash file in this repository, and call that script. decorate: true
will pull down a repository for you.
Also, I see that this script is just calling the script at https://get.haskellstack.org/
. Is that script stored in git somewhere?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see that some projects keep their presubmits.sh
in the project repo, like repo-infra. Do you recommend we keep a script like this, or should I put a script somewhere in this (test-infra) repository? If its the later, can you please tell me where I should keep the script?
About the script at https://get.haskellstack.org/
, it is in a git repository, but I guess it would be better to have stack
(the build tool) in the base image for running tests.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A script for another repo's presubmits should be with the code it's testing, in that repository. Otherwise, you'll have a harder time maintaining those tests.
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: akshaymankar The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest |
@akshaymankar: Cannot trigger testing until a trusted user reviews the PR and leaves an In response to this:
Instructions 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/test-infra repository. |
@chases2 @mbohlool @brendanburns Would any two of you please be sponsors of my membership of this org? |
/ok-to-test |
Once this is passing tests, it looks fine to me. I think you might have less flakiness if you pull a repository with the pod utilities instead of curling for the script, but that's just a hunch. If you're looking for sponsors, you'll need more than just one. I'd recommend trying to organize that on Slack |
The following users are mentioned in OWNERS file(s) but are not members of the kubernetes org. Once all users have been added as members of the org, you can trigger verification by writing
|
@akshaymankar: The following test failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. Instructions 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/test-infra repository. I understand the commands that are listed here. |
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
Thanks for your pull request. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please follow instructions at https://git.k8s.io/community/CLA.md#the-contributor-license-agreement to sign the CLA. It may take a couple minutes for the CLA signature to be fully registered; after that, please reply here with a new comment and we'll verify. Thanks.
Instructions 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/test-infra repository. I understand the commands that are listed here. |
Stale issues rot after 30d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
Rotten issues close after 30d of inactivity. Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
@fejta-bot: Closed this PR. In response to this:
Instructions 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/test-infra repository. |
/reopen |
@akshaymankar: Failed to re-open PR: state cannot be changed. The add-haskell-client branch was force-pushed or recreated. In response to this:
Instructions 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/test-infra repository. |
I have never used prow before, so I am totally looking for suggestions.
It would be great if a couple of directories could be cached across builds, but I couldn't find much about how it can be done. So I would appreciate some help in that. 🙏