[FOLIO-3574] Remove MODS record loading from all reference environment build playbooks Created: 07/Sep/22 Updated: 30/Sep/22 Resolved: 30/Sep/22 |
|
| Status: | Closed |
| Project: | FOLIO |
| Components: | None |
| Affects versions: | None |
| Fix versions: | None |
| Type: | Task | Priority: | TBD |
| Reporter: | Wayne Schneider | Assignee: | Wayne Schneider |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original estimate: | Not Specified | ||
| Issue links: |
|
||||||||
| Sprint: | DevOps Sprint 146, DevOps Sprint 149 | ||||||||
| Development Team: | FOLIO DevOps | ||||||||
| RCA Group: | TBD | ||||||||
| Description |
|
The API will be removed from a future snapshot version of mod-inventory, and it's actually not working anyway!
|
| Comments |
| Comment by Marc Johnson [ 29/Sep/22 ] |
|
Wayne Schneider I've raised a pull request for removing this work from the ansible builds. I would appreciate a review (and merge if it's ok). I'm conscious that we are close to the Nolana feature freeze threshold. The motivation for making these changes now is that a breaking compatibility change had already been made in mod-inventory and so combining this into that work* reduces the overhead of removing code like this. (*yes, I rejected this idea for a long time, however the overhead is too high for me to keep that stance, and thus I now support one major interface change per module release, even though this somewhat undermines the interface vs. implementation split) |
| Comment by Wayne Schneider [ 30/Sep/22 ] |
|
Thank you, Marc Johnson. The PR has been tested and merged (and a companion PR raised, tested, and merged in the folio-infrastructure repository). Thank you for doing most of my work for me |
| Comment by Marc Johnson [ 30/Sep/22 ] |
|
Wayne Schneider Thanks for making the changes to the other repository (I don't have access to it and I couldn't fork it due to it being private) |