Project

General

Profile

Actions

Feature #14447

closed

Add serialised cache of parsed manifests

Added by Dominic Cleal almost 8 years ago. Updated almost 8 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

This tickets covers the idea of a third JSON-based parser mentioned at https://groups.google.com/d/msg/foreman-dev/Ijvodu1hJkE/Ssl83t35IAAJ and in https://github.com/theforeman/kafo_parsers/pull/13. However, instead of a JSON-based parser in kafo_parsers, adding a cache from a serialised file around the parser in Kafo itself would have a few benefits:

  • Kafo already had kafo-export-params, which I thought would be the perfect tool to extend to generate the YAML files. It already handles loading the configuration, output formats etc, so adding a new output format for the cache is trivial.
  • If extending kafo-export-params to write the cache, it made sense to keep the cache reader and writer in the same project so it's easy to keep in sync.
  • The cache location can be set in the Kafo configuration, which is hard to access from kafo_parsers right now.

This helps with support for Puppet AIO packages as when the cache is in use, there's no need to load the Puppet dependency in kafo_parsers.

This does delay the requirement for a puppet-strings parser, as we can generate the parser cache on the current Puppet 3 installation at build time and use it with a Puppet 4 AIO installation. It also means users won't have to install puppet-strings. We will still want a puppet-strings parser so we're not limited to building on Puppet 3.


Related issues 6 (0 open6 closed)

Related to Kafo - Bug #14683: Parser cache is configured by default, causing warning when it's missingClosedDominic Cleal04/18/2016Actions
Related to Kafo - Feature #14946: Add command line option to force use of parser cacheClosedDominic Cleal05/06/2016Actions
Related to Kafo - Feature #16230: Read multiple parser caches for a single configurationClosedAdam Price08/22/2016Actions
Has duplicate Kafo - Bug #7530: Speed up katello-installer --helpResolved09/18/2014Actions
Blocks Foreman - Tracker #8447: Puppet 4 supportClosed11/19/2014

Actions
Blocks Installer - Feature #14453: Generate Kafo parser cache during buildClosedDominic Cleal04/04/2016Actions
Actions #1

Updated by Dominic Cleal almost 8 years ago

Actions #2

Updated by Dominic Cleal almost 8 years ago

  • Blocks Feature #14453: Generate Kafo parser cache during build added
Actions #3

Updated by Dominic Cleal almost 8 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions #4

Updated by Dominic Cleal almost 8 years ago

  • Related to Bug #14683: Parser cache is configured by default, causing warning when it's missing added
Actions #5

Updated by Dominic Cleal almost 8 years ago

  • Related to Feature #14946: Add command line option to force use of parser cache added
Actions #6

Updated by Dominic Cleal almost 8 years ago

  • Has duplicate Bug #7530: Speed up katello-installer --help added
Actions #7

Updated by Adam Price over 7 years ago

  • Related to Feature #16230: Read multiple parser caches for a single configuration added
Actions

Also available in: Atom PDF