http://localhost/ticket/1134 <p> <strong>Proposer</strong>: Adrià Mercader </p> <h2 id="Abstract">Abstract</h2> <p> The new harvester interface allows to create harvesters for different sources, but right now harvesters don't have many ways to describe and configure themselves. We need a way of allowing them to: </p> <ul><li>Expose their type and other details so they can be used internally and on the UI. </li><li>Define configuration settings for particular harvester instances. </li></ul><h2 id="TheProblem">The Problem</h2> <h3 id="Harvesterdescription">Harvester description</h3> <p> The current UI for adding and editing harvest sources is the same used in ckanext-dgu, and thus the 3 harvester types used in DGU to harvest various GEMINI realted sources are hardcoded in the form. The form will be migrated to a DGU-independent one, so we need the harvesters to provide all the necessary data. There is a current <tt>get_type</tt> method that returns the harvester type, but for make it compatible with the DGU forms, it returns a machine-readable string (e.g. "CSW Server"), making it error prone. </p> <h3 id="Arbitraryconfiguration">Arbitrary configuration</h3> <p> In the current implementation, when the harvest process is started, ckanext-harvest looks for all the available plugins that implement the <tt>IHarvester</tt> interface and calls the appropiate methods for the current stage (<tt>gather_stage</tt>,<tt>fetch_stage</tt>,<tt>import_stage</tt>). At these stages, harvesters have no way of applying arbitrary configuration options, so all harvesters of the same type behave on the same way. For instance, the CKAN harvester needs a way to define the API version to use when harvesting remote instances (Right now, the version 2 is hardcoded on the code). </p> <h2 id="Specification">Specification</h2> <h3 id="Harvesterdescription1">Harvester description</h3> <p> Harvesters will need to provide the following information so the UI form can be built: </p> <ul><li>name: machine-readable name (e.g. "waf"). This will be the value stored in the database, and the one used by ckanext-harvest to call the appropiate harvester. </li><li>title: human-readable name (e.g. "Web Accessible Folder (WAF)"). This will appear in the form's select box. </li><li>description: a description of what the harvester does (e.g. "A Web Accessible Folder (WAF) displaying a list of GEMINI 2.1 documents"). This will appear on the form as a guidance to the user. </li></ul><p> The way to provide it will be an <tt>info</tt> method that all harvesters must implement, which will return a dictionary with the previous elements: </p> <pre class="wiki"> { 'name': 'csw', 'title': 'CSW Server', 'description': 'A server that implements OGC's Catalog Service for the Web (CSW) standard' } </pre><h3 id="Arbitraryconfiguration1">Arbitrary configuration</h3> <p> As different harvesters will have very different needs, we need to provide a way to persist arbitrary configuration flags for each harvest source. The more flexible way given the current architecture in my opinion would be to store the configuration options as a JSON encoded object as a property of the harvest source (There already is an unused DB field called <tt>config</tt> in the database) (Maybe using <a class="missing wiki">JsonType?</a>?). </p> <p> This will mean adding an extra field in the harvest source form to allow entering the configuration. This could be just a simple text field where users enter the JSON encoded object or a more clever mechanism (i.e an "Add a configuration flag" link that adds two new text fields for the key and value for each flag, and a mechanism to later build the JSON object). In any case, this should probably be hidden in an "Advance options" section. </p> <h2 id="Whydoitthisway">Why do it this way</h2> <h3 id="Harvesterdescription2">Harvester description</h3> <p> The <tt>info</tt> method would provide a single point to get all the information related to the harvester, and future properties could be added to the dictionary returned without having to modify the interface. </p> <h3 id="Arbitraryconfiguration2">Arbitrary configuration</h3> <p> There is an already existing <tt>config</tt> field in the database, so we won't need to change the model. Harvesters could access the config object at any of the stages. Of course they could provide default values in their implementations so users don't need to enter them everytime. </p> <h2 id="Implementationplan">Implementation plan</h2> <h3 id="Deliverables">Deliverables</h3> <h3 id="Risksandmitigations">Risks and mitigations</h3> <p> The highest risk on the harvesters <tt>info</tt> method side is that harvester implementation don't offer one of the necessary properties (namely name and title). This could fire a warning when showing the UI form or using the CLI. </p> <h3 id="Participants">Participants</h3> <p> Adrià Mercader to do it. </p> <h3 id="Progress">Progress</h3> <p> None yet. </p> en-us http://assets.okfn.org/p/ckan/img/ckan_logo_shortname.png http://localhost/ticket/1134 Trac 0.12.3 thejimmyg Wed, 11 May 2011 14:47:08 GMT http://localhost/ticket/1134#comment:1 http://localhost/ticket/1134#comment:1 <ul> <li><strong>cc</strong> <em>david.raznick@…</em> added </li> <li><strong>priority</strong> changed from <em>awaiting triage</em> to <em>major</em> </li> <li><strong>state</strong> changed from <em>draft</em> to <em>accepted</em> </li> </ul> <p> Great stuff. </p> <p> Yes, I agree with all this, but as initial thoughts I suggest that for phase 1, the info() method returns one more key called "form_config_interface" which takes a string representing a the type of interface for the config field on the form. If the key is missing it is treated as "None". </p> <p> The two possible values are: </p> <p> None - the field is not present and must always be stored as NULL Text - a single text field will be provided on the interface </p> <p> Whatever the interface, the value built will always be stored in the config column of the table. </p> <p> We then also provide a "get_schema()" method that returns a schema cablable of parsing the data submitted by the form and storing it as a single key named "config". ckanext-inspire will then add other functions to the schema for URL etc so that regardless of what the harvest plugin does, the key fields ckanext-harvest are always there. </p> <p> This then allows: </p> <ul><li>Customiseable config interfaces </li><li>Customisable valiation </li><li>A consistent place to store config in the database </li></ul><p> Sound good? </p> Ticket sebbacon Tue, 17 May 2011 09:12:11 GMT http://localhost/ticket/1134#comment:2 http://localhost/ticket/1134#comment:2 <p> Agreed with James that we should consider *from the start* how to provide for a nice UI -- my worry is if we start out with people editing raw JSON we'll end up keeping that for years :) </p> Ticket seanh Fri, 15 Jun 2012 15:35:54 GMT http://localhost/ticket/1134#comment:3 http://localhost/ticket/1134#comment:3 <ul> <li><strong>milestone</strong> set to <em>ckan-backlog</em> </li> </ul> Ticket