Skip to content

support new CDSE dev backend URL in test suite#497

Merged
soxofaan merged 2 commits into
mainfrom
support-new-cdse-dev-backend-url
May 29, 2026
Merged

support new CDSE dev backend URL in test suite#497
soxofaan merged 2 commits into
mainfrom
support-new-cdse-dev-backend-url

Conversation

@VincentVerelst
Copy link
Copy Markdown
Contributor

@VincentVerelst VincentVerelst commented May 7, 2026

I was recently informed by Peter (can't tag him here) that https://openeo.dev.waw3-1.openeo-int.v1.dataspace.copernicus.eu/ is the new go-to URL for CDSE dev. However, the test suite didn't accept this one yet.

@VincentVerelst VincentVerelst requested a review from soxofaan May 7, 2026 15:33
@soxofaan
Copy link
Copy Markdown
Contributor

soxofaan commented May 8, 2026

maybe it's better in this case to write a regex instead of trying to have an exact list ?

@VincentVerelst
Copy link
Copy Markdown
Contributor Author

maybe it's better in this case to write a regex instead of trying to have an exact list ?

@soxofaan, how much freedom should this regex then have? Don't you have the risk of allowing a non-existing backend URL when not using these allow lists?

@soxofaan
Copy link
Copy Markdown
Contributor

@pvbouwel do you have a suggestion for a regex to avoid hardcoding CDSE dev hostnames like

    elif hostname in {
        "openeo.dev.warsaw.openeo.dataspace.copernicus.eu",
        "openeo.dev.waw3-1.openeo-int.v1.dataspace.copernicus.eu",
    }:

@pvbouwel
Copy link
Copy Markdown

pvbouwel commented May 11, 2026

So the only form that should be supported for our cloud environments is

service.env.region.<project>[-int].v1.dataspace.copernicus.eu

For OpenEO endpoint service is "openeo"
environment is "dev|stag|prod"
region is the region name within the cloud provider e.g. "waw3-1|waw4-1|eu-nl|eu-west-1|..."
project is like for what the group of endpoints is setup. So can be OpenEO but if there is a project with its own stack it could be any value (currently always openeo)

So openeo.dev.warsaw.openeo.dataspace.copernicus.eu should anyway be removed as there should not be a use case that cannot be served by openeo.dev.waw3-1.openeo-int.v1.dataspace.copernicus.eu and that one should be preferred.

@soxofaan
Copy link
Copy Markdown
Contributor

@VincentVerelst

so ideally the regex should be something like

elif re.fullmatch(r"openeo\.dev\.([a-z0-9-]+)\.openeo-int\.v1\.dataspace\.copernicus\.eu", hostname):

@JanssenBrm
Copy link
Copy Markdown
Contributor

@soxofaan - Can this PR be merged?

@soxofaan soxofaan force-pushed the support-new-cdse-dev-backend-url branch from 1e8f925 to 2033588 Compare May 29, 2026 14:15
@soxofaan soxofaan merged commit f6c620c into main May 29, 2026
3 checks passed
@soxofaan soxofaan deleted the support-new-cdse-dev-backend-url branch May 29, 2026 14:21
@soxofaan
Copy link
Copy Markdown
Contributor

done

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.

4 participants