Skip to content

[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

Merged
merged 1 commit into from
Mar 14, 2016

Conversation

sdodson
Copy link
Member

@sdodson sdodson commented Mar 11, 2016

Ship this via a subpackage as it's a 69MiB binary

Ship this via a subpackage as it's a 69MiB binary
@sdodson
Copy link
Member Author

sdodson commented Mar 11, 2016

@timothysc @jayunit100 FYI
@tdawson mind taking a look, this is the most sane location I could come up with for the generically named 'extended.test' binary

@timothysc
Copy link

@sdodson sgtm +1 for libexec & subpackage. I'll poke @ingvagabund for packaging expertise.

@tdawson
Copy link
Member

tdawson commented Mar 11, 2016

LGTM 👍

@timothysc
Copy link

/cc @jeremyeder

@jeremyeder
Copy link
Contributor

@xltian -- need to negotiate getting this run via CI

@jayunit100
Copy link
Contributor

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

@sdodson
Copy link
Member Author

sdodson commented Mar 11, 2016

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.

[root@ose3-master ~]# KUBECONFIG=/etc/origin/master/admin.kubeconfig /usr/libexec/atomic-openshift/extended.test --ginkgo.skip="should becomes" --ginkgo.focus="EmptyDir"
I0311 15:35:44.929296    2940 plugins.go:62] Registered admission plugin "ServiceAccount"
I0311 15:35:44.932840    2940 plugins.go:46] Registered cloud provider "aws"
I0311 15:35:44.933540    2940 plugins.go:46] Registered cloud provider "gce"
Running Suite: Extended Core
============================                                                                                                       
Random Seed: 1457728544       
Will run 18 of 333 specs
...
Ran 18 of 333 Specs in 207.707 seconds
SUCCESS! -- 18 Passed | 0 Failed | 0 Pending | 315 Skipped PASS
[root@ose3-master ~]#                                          

@sdodson
Copy link
Member Author

sdodson commented Mar 11, 2016

@brenton should be good to merge

@brenton
Copy link
Contributor

brenton commented Mar 14, 2016

[merge]

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_requests_origin/5308/) (Image: devenv-rhel7_3717)

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to 0b869f3

@ingvagabund
Copy link
Member

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?

@ingvagabund
Copy link
Member

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

@sdodson
Copy link
Member Author

sdodson commented Mar 14, 2016

@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.

@ingvagabund
Copy link
Member

Spam number 3: From [1]:

/usr/libexec includes internal binaries that are not intended to be executed directly by users
or shell scripts. Applications may use a single subdirectory under /usr/libexec.

Although it says it is for binaries, I can find a lot of Bourne-Again shell script and POSIX shell script files under my /usr/libexec directory. Though, can not find better directory for both the binary and auxiliary shell scripts in [1].

[1] https://www.linuxbase.org/betaspecs/fhs/fhs/ch04s07.html
[2] https://www.linuxbase.org/betaspecs/fhs/fhs/ch04.html

openshift-bot pushed a commit that referenced this pull request Mar 14, 2016
@openshift-bot openshift-bot merged commit 08f4ed2 into openshift:master Mar 14, 2016
@timothysc
Copy link

You have no idea how much time this has saved me.

sudo yum install atomic-openshift-tests

thx @jayunit100 & @sdodson

/cc @mattf

@jayunit100
Copy link
Contributor

Thanks also to @arilivigni for helping bridge this work!

@sdodson sdodson deleted the test-package branch May 11, 2018 19:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants