-
Notifications
You must be signed in to change notification settings - Fork 4.7k
[RPMS] Add extended.test to /usr/libexec/origin/extended.test #7962
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
Conversation
Ship this via a subpackage as it's a 69MiB binary
@timothysc @jayunit100 FYI |
@sdodson sgtm +1 for libexec & subpackage. I'll poke @ingvagabund for packaging expertise. |
LGTM 👍 |
/cc @jeremyeder |
@xltian -- need to negotiate getting this run via CI |
Looks good to me to thanks. I guess we don't need to use the hack/build-go abstraction . I think I built it this way once and it works the same |
The hack/build-go.sh scripts don't work inside koji/brew right now so unfortunately we do some odd stuff. Looks like the binary runs as expected.
|
@brenton should be good to merge |
[merge] |
continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_requests_origin/5308/) (Image: devenv-rhel7_3717) |
Evaluated for origin merge up to 0b869f3 |
Looks fine, %{_libexecdir}/%{name} is owned only by one subpackage. What is intention of the binary? Is it self-standing binary? Does not it need any other auxiliary test files? What about some short md file describing simple example how to use it? |
Ok, it is a self-standing binary for running unit tests. IIUC, you could ship core.sh, all.sh and other *.sh files from [1] as well. [1] https://github.com/openshift/origin/tree/master/test/extended |
@jayunit100 whatya say about other files we may need to ship? I actually know nothing of the test suite so need guidance from someone who does. |
Spam number 3: From [1]:
Although it says it is for binaries, I can find a lot of [1] https://www.linuxbase.org/betaspecs/fhs/fhs/ch04s07.html |
Merged by openshift-bot
You have no idea how much time this has saved me.
thx @jayunit100 & @sdodson /cc @mattf |
Thanks also to @arilivigni for helping bridge this work! |
Ship this via a subpackage as it's a 69MiB binary