[FOLIO-2617] Request to Update ebsco-rmapi-config settings for Hosted Reference Environments Created: 28/May/20 Updated: 22/Jun/20 Resolved: 22/Jun/20 |
|
| Status: | Closed |
| Project: | FOLIO |
| Components: | None |
| Affects versions: | None |
| Fix versions: | None |
| Type: | Story | Priority: | P2 |
| Reporter: | Carole Godfrey | Assignee: | David Crossley |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original estimate: | Not Specified | ||
| Attachments: |
|
||||||||||||||||
| Issue links: |
|
||||||||||||||||
| Sprint: | DevOps: sprint 90 | ||||||||||||||||
| Development Team: | FOLIO DevOps | ||||||||||||||||
| Description |
|
The location where eHoldings application stores its configuration for rm api credentials is changing in Q2-2020. These changes have been merged into master branch Without configuration changes, hosted reference environments which are built from master will not have kb credentials configured for the eHoldings application Prior to this change - configuration for rm api credentials are gotten from mod-configuration (believe this is the reference in ansible project) We'd like request that the 3 configuration api calls be replaced with a single POST request to a new /eholdings/kb-credentials endpoint POST /eholdings/kb-credentials
{
"data": {
"type": "kbCredentials",
"attributes": {
"name": "OpenTown Libraries",
"apiKey": "XXXX",
"url": "YYYY",
"customerId": "ZZZZ"
}
}
}
|
| Comments |
| Comment by David Crossley [ 11/Jun/20 ] |
|
Carole Godfrey The configuration is done and tested with Jenkins build folio-snapshot-test #158 The nightly builds of reference environments will utilise this on their next run. |
| Comment by Carole Godfrey [ 11/Jun/20 ] |
|
Thank you David Crossley |
| Comment by David Crossley [ 11/Jun/20 ] |
|
I needed to temporarily disable the "ebsco-rmapi-config" for the Jenkins "folio-release" build. Its ansible playbook also uses the master of folio-ansible (as does snapshot and testing), hence fails because the module is not yet released with that new endpoint. |
| Comment by David Crossley [ 11/Jun/20 ] |
|
Today's "folio-snapshot" Jenkins builds are okay. |
| Comment by Carole Godfrey [ 11/Jun/20 ] |
|
The latest code in master uses a newer endpoint for those credentials. Older reference environments would not have this newer endpoint available from the API Older environments would continue to need to be configured using the 3 configuration endpoints Also David Crossley we would like the name to say either "OpenTown Libraries" or "Knowledge Base" instead of the tenant diku (it seems to be showing tenant id instead of a tenant display name) - could this be updated? |
| Comment by David Crossley [ 15/Jun/20 ] |
|
The name "OpenTown Libraries" is set by the front-end Stripes configuration. So it will need to be that other hard-coded alternative "Knowledge Base". I will get that adjusted for Tuesday's reference environment builds. |
| Comment by Carole Godfrey [ 15/Jun/20 ] |
|
Sounds good – thanks David Crossley |
| Comment by Carole Godfrey [ 18/Jun/20 ] |
|
David Crossley – just to mention – I added a separate related ticket in this area – it is due to migration script changes which added a Dummy Credentials |
| Comment by David Crossley [ 22/Jun/20 ] |
|
All done now. |