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

test_byterange.sh failure with URL3 thredds link #2500

Open
abhibaruah opened this issue Sep 7, 2022 · 3 comments
Open

test_byterange.sh failure with URL3 thredds link #2500

abhibaruah opened this issue Sep 7, 2022 · 3 comments
Assignees
@DennisHeimbigner
Milestone
4.9.3

Comments

@abhibaruah
Copy link

NetCDF version: v4.8.1
OS: Linux Debian 11

While trying to build netcdf v4.8.1, we see failures in the test test_byterange.sh.
The log file netcdf/Testing/Temporary/LastTest.log has the following log:

73/197 Testing: nc_test_test_byterange
73/197 Test: nc_test_test_byterange
Command: "/usr/local/bin/bash" "-c" "export srcdir=/mathworks/devel/sandbox/abaruah/3p-tmw/3p/derived/glnxa64/netcdf/nc_test;export TOPSRCDIR=/mathworks/devel/sandbox/abaruah/3p-tmw/3p/derived/glnxa64/netcdf;/mathworks/devel/sandbox/abaruah/3p-tmw/3p/derived/glnxa64/netcdf/nc_test/test_byterange.sh "
Directory: /mathworks/devel/sandbox/abaruah/3p-tmw/3p/derived/glnxa64/netcdf/nc_test
"nc_test_test_byterange" start time: Sep 07 12:49 EDT
Output:
----------------------------------------------------------

*** Testing reading NetCDF-3 file with http
***Test remote classic file
/mathworks/devel/sandbox/abaruah/3p-tmw/3p/derived/glnxa64/netcdf/ncdump/ncdump: https://thredds-test.unidata.ucar.edu/thredds/fileServer/pointData/cf_dsg/example/point.nc#mode=bytes: NetCDF: Unknown file format
test_http: -k flag mismatch: expected=classic have=
<end of output>
Test time =   0.35 sec
----------------------------------------------------------
Test Failed.
"nc_test_test_byterange" end time: Sep 07 12:49 EDT
"nc_test_test_byterange" time elapsed: 00:00:00
----------------------------------------------------------

Looks like the failure is only with URL3="https://thredds-test.unidata.ucar.edu/thredds/fileServer/pointData/cf_dsg/example/point.nc#mode=bytes"

When I comment out this URL, the test passes. Also, pasting this link in the browser returns an HTTP Status 404-Not Found.
Can someone confirm if this failure is related to #2407 ?
Or if it is an issue with the thredds server where the file is hosted?
@DennisHeimbigner
Copy link
Collaborator

Sorry about that. Recently, thredds-test was de-commissioned
so those URLS are no longer supported. I put up a PR to fix this.
If you need byterange, then you will need to kill those URLS.
Otherwise use option --disable-byterange

@abhibaruah
Copy link
Author

Thanks for the update. For now, we can remove those URLs.
Will the test dap4_test/test_thredds.sh be removed/updated as well?

@DennisHeimbigner
Copy link
Collaborator

Yes, I am eliding all thredds-test URLs. I will probably re-introduce
the equivalents when I can find an alternate server.

@WardF WardF assigned WardF and DennisHeimbigner and unassigned WardF Sep 7, 2022
@WardF WardF moved this to Todo in netCDF-C v4.9.1 Sep 7, 2022
@WardF WardF added this to the 4.9.1 milestone Sep 7, 2022
Avimitin added a commit to Avimitin/archriscv-packages that referenced this issue Oct 25, 2022
@Avimitin
addpkg: netcdf-openmpi-bootstrap
fa2c8f5 
This package provides `netcdf-openmpi` for package `pnetcdf-openmpi`.
And the `pnetcdf-openmpi` package can be used for building the
`netcdf-openmpi` itself.

Modifications:
==============

1. Becasue of the bug mentioned in [netcdf-c felixonmars#2500], BYTERANGE feature is
closed.
2. Close the pnetcdf feature and remove the pnetcdf makedepends.

[netcdf-c felixonmars#2500]: Unidata/netcdf-c#2500

Build Sequence:
===============

1. netcdf-openmpi-bootstrap
2. pnetcdf-openmpi
3. netcdf-openmpi

Additional Info:
================

All the three packages mention above needs `--nocheck`. This is because
there are always some bug in shared memory from openmpi. All tests
except openmpi parallel tests passed successfully. hdf5-openmpi [ignore] the
failure too. So no check to bypass those parallel test is acceptable.

[ignore]: https://github.com/archlinux/svntogit-community/blob/packages/hdf5-openmpi/trunk/PKGBUILD#L88

Signed-off-by: Avimitin <avimitin@gmail.com>
@Avimitin Avimitin mentioned this issue Oct 25, 2022
Merged
felixonmars pushed a commit to felixonmars/archriscv-packages that referenced this issue Nov 17, 2022
@Avimitin @felixonmars
addpkg: netcdf-openmpi-bootstrap
4fb1182 
This package provides `netcdf-openmpi` for package `pnetcdf-openmpi`.
And the `pnetcdf-openmpi` package can be used for building the
`netcdf-openmpi` itself.

Modifications:
==============

1. Becasue of the bug mentioned in [netcdf-c #2500], BYTERANGE feature is
closed.
2. Close the pnetcdf feature and remove the pnetcdf makedepends.

[netcdf-c #2500]: Unidata/netcdf-c#2500

Build Sequence:
===============

1. netcdf-openmpi-bootstrap
2. pnetcdf-openmpi
3. netcdf-openmpi

Additional Info:
================

All the three packages mention above needs `--nocheck`. This is because
there are always some bug in shared memory from openmpi. All tests
except openmpi parallel tests passed successfully. hdf5-openmpi [ignore] the
failure too. So no check to bypass those parallel test is acceptable.

[ignore]: https://github.com/archlinux/svntogit-community/blob/packages/hdf5-openmpi/trunk/PKGBUILD#L88

Signed-off-by: Avimitin <avimitin@gmail.com>
@djhoese djhoese mentioned this issue Jan 5, 2023
Closed
5 tasks
@WardF WardF modified the milestones: 4.9.1, 4.9.2 Feb 13, 2023
@WardF WardF modified the milestones: 4.9.2, 4.9.3 May 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DennisHeimbigner DennisHeimbigner

Labels
None yet
Projects
No open projects
netCDF-C v4.9.1
Status: Todo
Milestone
4.9.3
Development

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