Skip to content
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.

Sign up for GitHub

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

Jump to bottom

current master does not pass make check due to ncdap_test/tst_zero_len_var.sh #1449

Closed
edwardhartnett opened this issue Aug 1, 2019 · 8 comments
Closed

current master does not pass make check due to ncdap_test/tst_zero_len_var.sh #1449

edwardhartnett opened this issue Aug 1, 2019 · 8 comments
Assignees
@WardF
Labels
area/DAP2 area/DAP4
Milestone
4.7.2

Comments

@edwardhartnett
Copy link
Contributor

The current master is failing due to this test.

Not sure what is going on here. I will comment this test out of my branch while you guys take a look. This is also the test that is failing for PR #1448.

ed@mikado:~/noaa/netcdf-c/ncdap_test$ bash -x ./tst_zero_len_var.sh 
+ test x '!=' x
+ set -e
+ test x = x
++ pwd
+ srcdir=/home/ed/noaa/netcdf-c/ncdap_test
+ . ../test_common.sh
++ TOPSRCDIR=/home/ed/noaa/netcdf-c
++ TOPBUILDDIR=/home/ed/noaa/netcdf-c
++ set -e
++ test x = x1
++ top_srcdir=/home/ed/noaa/netcdf-c
++ top_builddir=/home/ed/noaa/netcdf-c
++ test x/home/ed/noaa/netcdf-c/ncdap_test = x
+++ pwd
++ builddir=/home/ed/noaa/netcdf-c/ncdap_test
++ execdir=/home/ed/noaa/netcdf-c/ncdap_test
+++ basename /home/ed/noaa/netcdf-c/ncdap_test
++ thisdir=ncdap_test
+++ pwd
++ WD=/home/ed/noaa/netcdf-c/ncdap_test
++ cd /home/ed/noaa/netcdf-c/ncdap_test
+++ pwd
++ srcdir=/home/ed/noaa/netcdf-c/ncdap_test
++ cd /home/ed/noaa/netcdf-c/ncdap_test
++ cd /home/ed/noaa/netcdf-c
+++ pwd
++ top_srcdir=/home/ed/noaa/netcdf-c
++ cd /home/ed/noaa/netcdf-c/ncdap_test
++ cd /home/ed/noaa/netcdf-c/ncdap_test
+++ pwd
++ builddir=/home/ed/noaa/netcdf-c/ncdap_test
++ cd /home/ed/noaa/netcdf-c/ncdap_test
++ cd /home/ed/noaa/netcdf-c
+++ pwd
++ top_builddir=/home/ed/noaa/netcdf-c
++ cd /home/ed/noaa/netcdf-c/ncdap_test
++ cd /home/ed/noaa/netcdf-c/ncdap_test
+++ pwd
++ execdir=/home/ed/noaa/netcdf-c/ncdap_test
++ cd /home/ed/noaa/netcdf-c/ncdap_test
++ export srcdir top_srcdir builddir top_builddir execdir
++ test -e /home/ed/noaa/netcdf-c/ncdump/ncdump.exe
++ ext=
++ export NCDUMP=/home/ed/noaa/netcdf-c/ncdump/ncdump
++ NCDUMP=/home/ed/noaa/netcdf-c/ncdump/ncdump
++ export NCCOPY=/home/ed/noaa/netcdf-c/ncdump/nccopy
++ NCCOPY=/home/ed/noaa/netcdf-c/ncdump/nccopy
++ export NCGEN=/home/ed/noaa/netcdf-c/ncgen/ncgen
++ NCGEN=/home/ed/noaa/netcdf-c/ncgen/ncgen
++ export NCGEN3=/home/ed/noaa/netcdf-c/ncgen3/ncgen3
++ NCGEN3=/home/ed/noaa/netcdf-c/ncgen3/ncgen3
++ ncgen3c0=/home/ed/noaa/netcdf-c/ncgen3/c0.cdl
++ ncgenc0=/home/ed/noaa/netcdf-c/ncgen/c0.cdl
++ ncgenc04=/home/ed/noaa/netcdf-c/ncgen/c0_4.cdl
++ cd /home/ed/noaa/netcdf-c/ncdap_test
+ /home/ed/noaa/netcdf-c/ncdump/ncdump http://test.opendap.org/opendap/data/nc/zero_length_array.nc
syntax error, unexpected WORD_WORD, expecting SCAN_ATTR or SCAN_DATASET or SCAN_ERROR
context: <!DOCTYPE^ HTML PUBLIC "-//IETF//DTD HTML 2.0//EN"><html><head><title>503 Service Temporarily Unavailable</title></head><body><h1>Service Temporarily Unavailable</h1><p>The server is temporarily unable to service yourrequest due to maintenance downtime or capacityproblems. Please try again later.</p><hr><address>Apache/2.2.15 (CentOS) Server at test.opendap.org Port 80</address></body></html>
/home/ed/noaa/netcdf-c/ncdump/ncdump: http://test.opendap.org/opendap/data/nc/zero_length_array.nc: http://test.opendap.org/opendap/data/nc/zero_length_array.nc: NetCDF: DAP server error
@WardF
Copy link
Member

WardF commented Aug 1, 2019

Interesting, it appears to be a DAP test so could be something related to the opendap.org server. I'll take a look, thanks. Do you have any immediate thoughts on it, @DennisHeimbigner ?

@WardF WardF self-assigned this Aug 1, 2019
@WardF WardF modified the milestones: future, 4.7.1, 4.7.2 Aug 1, 2019
@WardF
Copy link
Member

WardF commented Aug 1, 2019

Well, embedded in the response is The server is temporarily unable to service yourrequest due to maintenance downtime or capacityproblems. Please try again later.. So, we will try a bit and if it persists I'll see if it's their side, or something we need to change on our end. @DennisHeimbigner is this a test we could shift to our test server, do you think?

@edwardhartnett
Copy link
Contributor Author

Thanks for the quick response! I will continue without this test, but I'll be careful not to commit my commenting-out of the test.

@DennisHeimbigner
Copy link
Collaborator

Looking at it.

@DennisHeimbigner
Copy link
Collaborator

Right at this moment, it appears that test.opendap.org/ is down except
for the landing page. Will try again later.

In the longer term, the right thing to do is to change
this test (and any others) to not use test.opendap.org.

@DennisHeimbigner
Copy link
Collaborator

I think I can add a test to our test server that can replace the opendap.org
one. Will take a while, though. I sent a message to opendap support
to ask about the status of test.opendap.org.

@edwardhartnett edwardhartnett mentioned this issue Aug 1, 2019
Merged
@DennisHeimbigner
Copy link
Collaborator

The server is back up; See if the test works now.

@edwardhartnett
Copy link
Contributor Author

Indeed it seems to be working fine now. ;-)

I have turned the test back on in my PR. I will close this issue.

@DennisHeimbigner DennisHeimbigner mentioned this issue Aug 2, 2019
Closed
@xylar xylar mentioned this issue Dec 23, 2019
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@WardF WardF

Labels
area/DAP2 area/DAP4
Projects
None yet
Milestone
4.7.2
Development

No branches or pull requests
3 participants
@DennisHeimbigner @WardF @edwardhartnett