Maintenance: Planio will be observing a scheduled maintenance window this Sunday, November 10, 2024 from 20:00 UTC until 21:00 UTC to perform important network maintenance in our primary data center. Your Planio account will be unavailable for a few minutes during this maintenance window.
Issue #3532
closed
/api/v3/ and /content/ could collide with a Live API
Status:
CLOSED - CURRENTRELEASE
Description
Problem¶
Live APIs can live anywhere and have hard-coded requirements. For example, pulp_ansible expects to server various urls also rooted at /api/v3/ because the Galaxy API works like that and it's also at version 3. I don't think any of the core provided urls formally collide with Galaxy urls.
The same problem can occur with the /content/ app.
Solution¶
Pulp could minimize these types of issues by rooting everything under /pulp/
so the API would move to /pulp/api/v3/
and content to /pulp/content/
.
Pulp 2 uses pulp/api/v2/, so my first instinct would be to use pulp/api/v3/.
There is another compelling reason to go that direction. Id bet that /content/ will be even more overloaded that api/v3/.
So we could move our content app to pulp/content/ and our api to pulp/api/v3/.
- Subject changed from /api/v3/ could collide with a Live API to /api/v3/ and /content/ could collide with a Live API
- Description updated (diff)
@asmacdo, that would be much better overall. I've rewritten it to reflect this idea.
- Triaged changed from No to Yes
- Sprint set to Sprint 35
I could not grep any occurrences of "api/v3" in the pulp/devel repository, so I did not file an issue.
- Related to Issue #3547: Fix documentation to reflect pulpcore's new API roots added
- Groomed changed from No to Yes
- Tags Pulp 3 MVP added
- Sprint changed from Sprint 35 to Sprint 36
- Status changed from NEW to POST
- Assignee set to amacdona@redhat.com
- Status changed from POST to MODIFIED
- Sprint/Milestone set to 3.0.0
- Tags deleted (
Pulp 3, Pulp 3 MVP)
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Also available in: Atom
PDF
move api and content roots
fixes #3532 https://pulp.plan.io/issues/3532
Required PR: https://github.com/PulpQE/pulp-smash/pull/969