Actions
Bug #18705
closedcalling file_key_value_present_in_ini_section on a yum repo definition loops
Pull Request:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
To do
Fix check:
To do
Regression:
Description
Calling
set_variable_values_ini( {"/etc/yum.repos.d /rudder-test.repo","test","baseurl","http://mirrorlist.centos.org/?release=$releasever&arch=$basearch&re po=BaseOS&infra=$infra"})
results after many run in
baseurl=http://mirrorlist.centos.org/?release=$releasever&arch =$basearch&repo=BaseOS&infra=$infra=$releasever&arch=$basearch&repo=BaseOS&infra=$infra=$releasever&arch=$ basearch&repo=BaseOS&infra=$infra=$releasever&arch=$basearch&repo=BaseOS&infra=$infra=$releasever&arch=$ba search&repo=BaseOS&infra=$infra=$releasever&arch=$basearch&repo=BaseOS&infra=$infra=$releasever&arch=$base arch&repo=BaseOS&infra=$infra=$releasever&arch=$basearch&repo=BaseOS&infra=$infra=$releasever&arch=$basear ch&repo=BaseOS&infra=$infra=$releasever&arch=$basearch&repo=BaseOS&infra=$infra=$releasever&arch=$basearch &repo=BaseOS&infra=$infra=$releasever&arch=$basearch&repo=BaseOS&infra=$infra=$releasever&arch=$basearch&r epo=BaseOS&infra=$infra=$releasever&arch=$basearch&repo=BaseOS&infra=$infra=$releasever&arch=$basearch&rep o=BaseOS&infra=$infra=$releasever&arch=$basearch&repo=BaseOS&infra=$infra=$releasever&arch=$basearch&repo= BaseOS&infra=$infra=$releasever&arch=$basearch&repo=BaseOS&infra=$infra=$releasever&arch=$basearch&repo=Ba seOS&infra=$infra=$releasever&arch=$basearch&repo=BaseOS&infra=$infra=$releasever&arch=$basearch&repo=Base OS&infra=$infra=$releasever&arch=$basearch&repo=BaseOS&infra=$infra=$releasever&arch=$basearch&repo=BaseOS &infra=$infra=$releasever&arch=$basearch&repo=BaseOS&infra=$infra=$releasever&arch=$basearch&repo=BaseOS&i nfra=$infra=$releasever&arch=$basearch&repo=BaseOS&infra=$infra=$releasever&arch=$basearch&repo=BaseOS&inf ra=$infra=$releasever&arch=$basearch&repo=BaseOS&infra=$infra=$releasever&arch=$basearch&repo=BaseOS&infra =$infra=$releasever&arch=$basearch&repo=BaseOS&infra=$infra=$releasever&arch=$basearch&repo=BaseOS&infra=$ infra=$releasever&arch=$basearch&repo=BaseOS&infra=$infra=$releasever&arch=$base (snip)
verbose output says
ESC[0mESC[34mrudder verbose: P: BEGIN promise 'promise_files_cf_488' of type "field_edits" (pass 1) ESC[0mESC[34mrudder verbose: P: Promiser/affected object: 'baseurl\s:=.:' ESC[0mESC[34mrudder verbose: P: From parameterized bundle: set_variable_values_ini( {"/etc/yum.repos.d /rudder-test.repo","test","baseurl","http://mirrorlist.centos.org/?release=$releasever&arch=$basearch&re po=BaseOS&infra=$infra"}) ESC[0mESC[34mrudder verbose: P: Base context class: any ESC[0mESC[34mrudder verbose: P: "if" class condition: edit_baseurl ESC[0mESC[34mrudder verbose: P: Stack path: /default/rudder_directives/methods/'Global configuration f or all nodes/Package sources (Yum)'/default/run_eeb79970_d0da_4362_9096_e43d1c14ef79/methods/'Global confi guration for all nodes/Package sources (Yum)'/default/yum_repositories_management_1_0_eeb79970_d0da_4362_9 096_e43d1c14ef79/methods/'set repo url 1'/default/file_key_value_present_in_ini_section/files/'/etc/yum.re pos.d/rudder-bobby.repo'/default/set_variable_values_ini/field_edits/'baseurl\s:=.:'[1] ESC[0mESC[34mrudder verbose: Matched line 'baseurl=http://mirrorlist.centos.org/?release=$releasever&arch =$basearch&repo=BaseOS&infra=$infra=$releasever&arch=$basearch&repo=BaseOS&infra=$infra=$releasever&arch=$ basearch&repo=BaseOS&infra=$infra=$releasever&arch=$basearch&repo=BaseOS&infra=$infra=$releasever&arch=$ba search&repo=BaseOS&infra=$infra=$releasever&arch=$basearch&repo=BaseOS&infra=$infra=$releasever&arch=$base arch&repo=BaseOS&infra=$infra=$releasever&arch=$basearch&repo=BaseOS&infra=$infra=$releasever&arch=$basear ch&repo=BaseOS&infra=$infra=$ (snip) ver&arch=$basearch&repo=BaseOS&infra=$infra=$releasever&arch=$basearch&repo=BaseOS&infra=$infra=$releasever&arch=$basearch&repo=BaseOS&infra=$infra' ESC[0mESC[34mrudder verbose: Stopped at field 2 ESC[0mESC[32mrudder info: Setting field sub-value 'http://mirrorlist.centos.org/?release=$releasever&arch=$basearch&repo=BaseOS&infra=$infra' in '/etc/yum.repos.d/rudder-test.repo' ESC[0mESC[32mrudder info: Edited field inside file object /etc/yum.repos.d/rudder-test.repo ESC[0mESC[34mrudder verbose: C: + promise outcome class 'set_variable_values_ini_not_baseurl_reached' ESC[0mESC[34mrudder verbose: C: + promise outcome class 'set_variable_values_ini_not_baseurl_repaired'
Updated by Nicolas CHARLES almost 4 years ago
see https://tracker.mender.io/browse/CFE-413
it suggest to use similar implementation as set_config_values in https://groups.google.com/g/help-cfengine/c/bJNo-udLqaA/m/ZiKqA6FnAQAJ
Updated by Nicolas CHARLES almost 4 years ago
- Status changed from New to In progress
- Assignee set to Nicolas CHARLES
Updated by Nicolas CHARLES almost 4 years ago
Work in progess here: https://github.com/ncharles/ncf/commit/3e2b290b2efe7ac06e1bd26c68717a445b8dcac3
Updated by Nicolas CHARLES almost 4 years ago
- Status changed from In progress to Pending technical review
- Assignee changed from Nicolas CHARLES to Alexis Mousset
- Pull Request set to https://github.com/Normation/ncf/pull/1271
Updated by Nicolas CHARLES almost 4 years ago
- Status changed from Pending technical review to Pending release
Applied in changeset commit:bebb5d474a2bef870d8afbc627f9004ca5718f8e.
Updated by Vincent MEMBRÉ almost 4 years ago
- Status changed from Pending release to Released
This bug has been fixed in Rudder 6.1.7 which was released today.
Updated by Alexis Mousset over 2 years ago
- Project changed from 41 to Rudder
- Category changed from Generic methods - File Management to Generic methods
Actions