[FOLIO-2930] Add mod-remote-storage module to testing/snapshot envs Created: 30/Dec/20  Updated: 13/Jan/21  Resolved: 13/Jan/21

Status: Closed
Project: FOLIO
Components: None
Affects versions: None
Fix versions: None

Type: Task Priority: P2
Reporter: Siarhei Charniak Assignee: David Crossley
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original estimate: Not Specified

Issue links:
Relates
relates to UIRS-12 Consider require mod-remote-storage i... Closed
Sprint: DevOps: Sprint 105, DevOps: Sprint 104
Development Team: FOLIO DevOps

 Description   

There is a new mod-remote-storage module that we'd like added to the hosted environments for testing and demo purposes.



 Comments   
Comment by David Crossley [ 31/Dec/20 ]

Please note that we are endeavouring to have this process be self-service.

I will elaborate here on that already recently described in Slack #devops channel:

Is ui-remote-storage supposed to require that back-end module? It does not do that yet. When it does, then the back-end module will be automatically included in folio-snapshot.

If that is the case, then you can work with the front-end team for ui-remote-storage. They need to declare the interfaces in the "okapiInterfaces" section of their package.json file.

If the front-end and back-end are not yet ready for each other, then see this guide to explicitly add mod-remote-storage until the UI module is ready for it.
https://dev.folio.org/guides/install-backend-module/

As explained in that guide, adding to folio-testing is not yet self-service. The DevOps team will attend to this part. However it is now holiday season, so that part might be delayed.

If you have questions about the earlier part of the document, then please ask, and we will assist and clarify.

Comment by Siarhei Charniak [ 03/Jan/21 ]

David Crossley
The requirement for the remote-storage back-end module was added to ui-tenant-settings with this PR, but it was not included in folio-snapshot.

Comment by David Crossley [ 04/Jan/21 ]

Siarhei Charniak and Alex Mozolev:

However when that ui-tenant-settings/pull/177 was merged to master, then its build was broken. It failed the stage "Publish Module Descriptor":
Missing dependency: folio_tenant-settings-5.0.1000163 requires remote-storage: 1.0

Investigation shows that mod-remote-storage has two interfaces ("remote-storage-configurations" and "remote-storage-mappings") but not "remote-storage".

It is vital that developers do follow the result of their merge to ensure that all is well.

When that is fixed then the next build of the platforms would include mod-remote-storage, and the subsequent builds of folio-snapshot should then have it.

Comment by David Crossley [ 04/Jan/21 ]

Regarding the separate configuration task for folio-testing: The mod-remote-storage is now included in today's builds.

Comment by Siarhei Charniak [ 04/Jan/21 ]

David Crossley
Everything works fine now, both snapshot and Vagrant box. Thank you for help!

Comment by Marc Johnson [ 06/Jan/21 ]

Craig McNally Jakub Skoczen

Would we consider a new dependency within Platform Core to be a design decision?

Comment by David Crossley [ 07/Jan/21 ]

See UIRS-12 Closed – Consider requiring mod-remote-storage interfaces via ui-remote-storage

Generated at Thu Feb 08 23:24:19 UTC 2024 using Jira 1001.0.0-SNAPSHOT#100246-sha1:7a5c50119eb0633d306e14180817ddef5e80c75d.