-
Notifications
You must be signed in to change notification settings - Fork 33
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.
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
feat!: set wdqs concept URI #771
base: main
Are you sure you want to change the base?
Conversation
Allow setting the concept URI of entities in WDQS explicitly and distinct from the hostname used to contact wikibase. This allows the concept URI to be a public URI while the wikibase hostname is the docker network internal hostname used to poll for updates. BREAKING CHANGE: env var WIKIBASE_CONCEPT_URI now mandatory to run image
Sets the WIKIBASE_CONCEPT_URI required by WDQS. #771 BREAKING CHANGE: changes the concept URI used in WDQS
4d7ed10
to
d4a0a96
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks like it should go forward, and would be nice to backport. However, given the major versioning to 1.x and 2.x issue we've been talking about I wonder if there is any sensible way to maintain the current way it functions (even if not optimal or broken) so that it can be a minor release? 🤷🏼♂️
e17ecc8
to
58a3ece
Compare
How to handle this versioning, as discussed today with @lorenjohnson WDQS bumps a major version
Deploy bumps a patch
TODO
|
holding back for #782 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Allow setting the concept URI of entities in WDQS explicitly and distinct from the hostname used to contact wikibase.
This also fixes the problem WDQS not accepting entities when the wikibase host had a DNS name different from
wikibase
(which should be always the case, becausewikibase
is not a FQDN).Note: In deploy, items and properties in WDQS will now have the public wikibase hostname with https protocol as prefix. E.g. https://wikibase.example/
Related:
wikibase.svc
, I want it to belocalhost
#385BREAKING CHANGE: env var WIKIBASE_CONCEPT_URI now mandatory to run wdqs updater