Bug #314

The Interpreter should only replace the parser on a successful parse

Added by Luke Kanies over 5 years ago. Updated almost 4 years ago.

Status:Closed Start date:
Priority:Normal Due date:
Assignee:Luke Kanies % Done:

0%

Category:language
Target version:0.23.1
Affected Puppet version: Branch:
Keywords:
Votes: 0

Description

When the interpreter is reparsing the manifests, it replaces the default parser before the parser has successfully parsed the current files. It makes more sense to parse the files and only then replace the parser; this way, if there’s a parse failure, clients can continue getting their configurations until the manifests are fixed.

History

Updated by Luke Kanies over 5 years ago

I’m not sure this is actually possible — I might need to replace the interpreter rather than the parser when the files change, because parsing modifies too much internal interpreter state. At the least, this won’t be fixed in the next release.

Updated by alfred karma about 5 years ago

http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/1/index.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/2/index1.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/3/index2.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/¾/index3.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/5/index4.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/6/index5.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/7/index6.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/8/index7.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/9/index8.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/10/index9.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/11/index10.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/12/index11.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/13/index12.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/14/index13.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/15/index14.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/16/index15.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/17/index16.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/18/index17.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/19/index18.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/20/index19.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/21/index20.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/22/index21.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/23/index22.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/24/index23.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/25/index24.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/26/index25.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/27/index26.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/28/index27.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/29/index28.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/30/index29.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/31/index30.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/32/index31.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/33/index32.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/34/index33.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/35/index34.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/36/index35.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/37/index36.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/38/index37.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/39/index38.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/40/index39.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/41/index40.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/42/index41.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/43/index42.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/44/index43.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/45/index44.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/46/index45.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/47/index46.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/48/index47.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/49/index48.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/50/index49.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/51/index50.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/52/index51.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/53/index52.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/54/index53.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/55/index54.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/56/index55.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/57/index56.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/58/index57.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/59/index58.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/60/index59.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/61/index60.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/62/index61.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/63/index62.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/64/index63.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/65/index64.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/66/index65.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/67/index66.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/68/index67.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/69/index68.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/70/index69.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/71/index70.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/72/index71.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/73/index72.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/74/index73.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/75/index74.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/76/index75.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/77/index76.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/78/index77.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/79/index78.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/80/index79.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/81/index80.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/82/index81.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/83/index82.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/84/index83.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/85/index84.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/86/index85.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/87/index86.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/88/index87.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/89/index88.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/90/index89.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/91/index90.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/92/index91.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/93/index92.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/94/index93.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/95/index94.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/96/index95.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/97/index96.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/98/index97.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/99/index98.html http://gforge.info.ucl.ac.be/tracker/download.php/13/135/3/100/index99.html

Updated by Luke Kanies about 5 years ago

  • Status changed from 1 to 2

Updated by Anonymous about 5 years ago

Milestone unplanned deleted

Updated by Luke Kanies almost 5 years ago

  • Status changed from 2 to Closed
  • 7 set to fixed

Fixed in r2742.

Updated by James Turnbull almost 4 years ago

  • Target version changed from 4 to 0.23.1

Also available in: Atom PDF