Bug 731322
Summary: | network test gives 404 when trying fetching files from servers | ||
---|---|---|---|
Product: | [Retired] Red Hat Hardware Certification Program | Reporter: | Caspar Zhang <czhang> |
Component: | Test Suite (tests) | Assignee: | Greg Nichols <gnichols> |
Status: | CLOSED ERRATA | QA Contact: | Guangze Bai <gbai> |
Severity: | high | Docs Contact: | |
Priority: | high | ||
Version: | 1.3 | CC: | qcai, rlandry, ykun, yshao |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: |
An enhancement has been made in v7 1.4 that v7 server stores http transfer file in a new directory "/var/www/v7/transfer" on server side in network testing instead of original place: "/var/v7/store".
Note that this enhancement causes v7 configuration file change, user should remove /etc/v7.xml on both server side and SUT side before executing v7 tests.
|
Story Points: | --- |
Clone Of: | Environment: | ||
Last Closed: | 2011-11-08 15:43:48 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Attachments: |
Description
Caspar Zhang
2011-08-17 10:49:05 UTC
Is selinux on ibm-z10-06.rhts.eng.bos.redhat.com enforcing? permissive Created attachment 518792 [details]
a simple patch to fix 404 error
I find what happened to produce the 404. it was caused due to store directory path error.
A simple fix is attached.
Created attachment 518798 [details]
a more complex version of fix
However, we could think someway to avoid such mistake happen again in codes. I find "store" dir is similar to "export" dir, they were both under /var/v7 and then moved to /var/www/v7 later. In environment.py, export was used as an element for v7ServerDirectory key, but store used as the same level of v7ServerDirectory key. Could we convert it to an element as well?
Here is a new patch to convert v7 store as an element.
That patch would break the "save" command on the SUT, as it currently saves copies of results to /var/v7/store. This issue here is that currently, the server uses /var/www/v7 while the SUT uses /var/v7. The server uses /var/www/v7 because it needs httpd to serve files and run cgi from there. There are a few different options for fixing this bug: 1) make both SUT and server use /var/www/v7. This will make the code simpler and more maintainable, but my cause users some confusion. 2) remove the getStoreDirectory function from environment.py, as the caller (server vs. SUT) needs to use getServerDirectory vs. getDataDirectory depending on whether it's server-side or SUT-side code. 3) It seems arbirary that both the server-side of the network test, and the SUT's save (local) command use the "store" directory. Perhaps the server-side of the network test (v7/server/cgi/networkTest.py), and the URL used by the SUT network test should use a separate area for http transfers. Perhaps /var/www/v7/transfer (httpd configured as /v7/transfer) would be for the network file transfer test. I'd recommend option 3) as it seems to me to be the clearest and most maintainable. Created attachment 518883 [details]
A patch to change the network test to use its own transfer area
Per option 3 of the above comment.
Created attachment 518885 [details]
Patch to change the network test to uses it's own transfer area (corrected)
Committed to R17 Technical note added. If any revisions are required, please edit the "Technical Notes" field accordingly. All revisions will be proofread by the Engineering Content Services team. New Contents: An enhancement has been made in v7 1.4 that v7 server stores http transfer file in a new directory "/var/www/v7/transfer" on server side in network testing instead of original place: "/var/v7/store". Note that this enhancement causes v7 configuration file change, user should remove /etc/v7.xml on both server side and SUT side before executing v7 tests. Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. http://rhn.redhat.com/errata/RHBA-2011-1436.html Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. http://rhn.redhat.com/errata/RHBA-2011-1436.html |