puppetlabs-inifile
puppetlabs-inifile copied to clipboard
Add support for multiple values per key
This change adds support for ini files in which the same key is used multiple times for lists of values, for example:
[section]
foo=a
foo=b
To set multiple values, an array of values can be passed to the value
property of ini_setting
.
When passing a single value, the module works as before, with one exception: when a single value is passed to value
, but the ini file already contains multiple lines with the same key, any additional lines will now be removed instead of retained.
ini_setting is a type
Breaking changes to this file MAY impact these 151 modules (near match):
- factorit-icingaweb2
- eshamow-pe_httpd
- ajroetker-puppet_server
- mcanevet-freebox
- nullfox-pagerduty
- arusso-puppet
- reidmv-pe_puppet
- jamtur01-pagerduty
- stahnma-avahi
- binford2k-wreckit
- humansky-avahi
- arusso-nfs
- fiddyspence-zabbix
- xdrum-puppet
- fsalum-puppetmaster
- abuxton-puppet_mountpoint
- cnacorrea-puppet
- WhatsARanjit-puppetconfig
- dhgwilliam-cutover
- jgazeley-monyog
- greatboy-bs_sysctl
- eshamow-pe_cert_auth
- joshs-agent_noop
- puppetlabs-data_entitlement
- qroac-isp3node
- johnclarson-security
- encore-service_autorestart
- rfletcher-aws_cli
- daenney-circus
- a2labs-php
- jesse-vra_puppet_plugin_prep
- camptocamp-carbon
- genv-systemd_journal_remote
- simp-dconf
- puppetlabs-vra_puppet_plugin_prep
- puppetlabs-servicenow_cmdb_integration
- praekeltfoundation-certbot
- geoffwilliams-puppet_nonroot
- puppetfinland-gitdaemon
- CERNOps-abrt
- sebastianrakel-gitea
- cjtoolseram-puppetconf
- jdowning-influxdb
- rdelcampog-ec2tagfacts
- camptocamp-networkmanager
- puppetfinland-mcollective
- rauch-kallithea
- icann-quagga
- jantman-archlinux_workstation
- dylanratcliffe-external_data
- juame-ospuppet
- tedivm-hieratic
- cohdjn-cisecurity
- ssm-nifi
- example42-psick
- nodes-php
- vholer-openafs
- tomkrieger-security_baseline
- rplessl-influxdb
- attestra-pgbackrest
- millerjl1701-supervisord
- zpetr-couchdb
- slenky-notifier
- qtechnologies-osbaseline
- opentable-remaster
- petems-cockpit
- binford2k-node_encrypt
- Xebia-xldeploy
- sarus-couchdb
- shinesolutions-amazon_ssm_agent
- ploperations-puppet
- pdsupport-pagerduty
- gabe-ansible
- opstudio-puppet
- nvpuppet-systemenv
- qtechnologies-resolv_conf
- softek-zabbixagent
- danzilio-extended_http_report_processor
- dylanratcliffe-sumologic
- ceritsc-tuned
- jpadams-windows_diff_enable
- nrvale0-site
- pizzaops-cutover
- opentable-nagdash
- brainsware-s3ql
- ceh-freetds
- jebas-add_facts
- opentable-puppetversion
- qtechnologies-central_auth
- spacepants-ds_389
- camptocamp-dell
- hetzner-roundcube
- epostdev-one
- epflsti-epfl_sso
- seteam-role
- cloudscaling-scaleio_openstack
- stephenrjohnson-puppet
- s12v-blackfire
- huit-splunk
- pltraining-classroom_legacy
- pest-blackfire
- hunner-hiera
- MiamiOH-httpproxy
- Lavaburn-cabot
- aharden-puppet_ent_agent
- pltraining-classroom
- jay-console_env
- simp-systemd
- crayfishx-netapp_smo
- cm6051-bitbucket
- appuio-openshift
- squreshi157-bitbucket
- frechetta93-httpproxy
- stackstorm-st2
- abstractit-puppet
- fraenki-ds_389
- halyard-git
- ayohrling-r10k
- camptocamp-systemd
- seteam-splunk
- puppet-jenkins_job_builder
- puppet-cassandra
- puppetfinland-puppetmaster
- ghoneycutt-puppet
- puppet-systemd
- puppet-stash
- puppet-hiera
- puppet-rundeck
- simp-gdm
- treydock-xdmod
- puppet-letsencrypt
- puppet-php
- puppet-splunk
- Azcender-profile
- objectiflibre-puppet
- ncorrare-reportslack
- geoffwilliams-r_profile
- puppet-r10k
- openstack-tripleo
- enterprisemodules-easy_type
- puppetlabs-puppet_agent
- landcareresearch-ckan
- puppetlabs-puppetdb
- jcalles-php
- mayflower-php
- danzilio-letsencrypt
- opentable-rundeck
- zack-r10k
- locp-cassandra
- opentable-jenkins_job_builder
- simp-pupmod
This module is declared in 125 of 579 indexed public Puppetfiles
.
These results were generated with Rangefinder, a tool that helps predict the downstream impact of breaking changes to elements used in Puppet modules. You can run this on the command line to get a full report.
Exact matches are those that we can positively identify via namespace and the declaring modules' metadata. Non-namespaced items, such as Puppet 3.x functions, will always be reported as near matches only.
This would fix #41 and https://tickets.puppetlabs.com/browse/MODULES-5012
Hey @omolenkamp , thank you for your contribution. Looks like we have failures in both spec and acceptance tests for this.
Is it something that you could take a look at? We are here to help if you need anything.
All PR Testing jobs are failing on the step "Install agent", but it doesn't look like that's something I broke.
Hey, I think these issues should be resolved now. I'll re kick the tests 🙂
The test failures look related, probably broke idempotency but not 100% sure.
It looks like there is a single failure occurring across all OSs. Something related to change logging. If this issue is fixed, we should have no problem merging this PR.
Hello! 👋
This pull request has been open for a while and has had no recent activity. We've labelled it with attention-needed
so that we can get a clear view of which PRs need our attention.
If you are waiting on a response from us we will try and address your comments on a future Community Day.
Alternatively, if it is no longer relevant to you please close the PR with a comment.
Please note that if a pull request receives no update for 7 after it has been labelled, it will be closed. We are always happy to re-open pull request if they have been closed in error.
Keep open.
Closing and re-opening to re-kick the tests as the logs of the last run have been discarded.
ini_setting is a type
Breaking changes to this file MAY impact these 154 modules (near match):
- enterprisemodules-easy_type
- jgazeley-monyog
- cm6051-bitbucket
- humansky-avahi
- puppet-splunk
- simp-dconf
- tedivm-hieratic
- opentable-puppetversion
- puppet-hiera
- dylanratcliffe-sumologic
- qtechnologies-resolv_conf
- puppet-php
- eshamow-pe_cert_auth
- joshs-agent_noop
- danzilio-extended_http_report_processor
- sebastianrakel-gitea
- fraenki-ds_389
- jcalles-php
- hunner-hiera
- nodes-php
- crayfishx-netapp_smo
- appuio-openshift
- puppet-stash
- daenney-circus
- stahnma-avahi
- Lavaburn-cabot
- puppetfinland-gitdaemon
- tomkrieger-security_baseline
- ncorrare-reportslack
- jebas-add_facts
- treydock-xdmod
- puppetfinland-local_git_repo
- puppetlabs-servicenow_cmdb_integration
- binford2k-wreckit
- attestra-pgbackrest
- spacepants-ds_389
- jesse-vra_puppet_plugin_prep
- qtechnologies-osbaseline
- opentable-nagdash
- squreshi157-bitbucket
- simp-gdm
- camptocamp-dell
- puppetlabs-vra_puppet_plugin_prep
- puppetfinland-mcollective
- praekeltfoundation-certbot
- greatboy-bs_sysctl
- mayflower-php
- rauch-kallithea
- slenky-notifier
- brainsware-s3ql
- nvpuppet-systemenv
- ssm-nifi
- qroac-isp3node
- dylanratcliffe-external_data
- tomkrieger-cis_security_hardening
- jay-console_env
- openstack-tripleo
- millerjl1701-supervisord
- rdelcampog-ec2tagfacts
- example42-psick_profile
- abuxton-puppet_mountpoint
- nrvale0-site
- jamtur01-pagerduty
- encore-service_autorestart
- puppetfinland-puppetmaster
- ajroetker-puppet_server
- nullfox-pagerduty
- vholer-openafs
- pdsupport-pagerduty
- binford2k-node_encrypt
- Xebia-xldeploy
- zpetr-couchdb
- genv-systemd_journal_remote
- jpadams-windows_diff_enable
- danzilio-letsencrypt
- cnacorrea-puppet
- geoffwilliams-puppet_nonroot
- simp-pupmod
- camptocamp-carbon
- shinesolutions-amazon_ssm_agent
- epostdev-one
- pizzaops-cutover
- puppet-letsencrypt
- halyard-git
- cjtoolseram-puppetconf
- frechetta93-httpproxy
- dhgwilliam-cutover
- zack-r10k
- puppet-r10k
- ceritsc-tuned
- opstudio-puppet
- ayohrling-r10k
- mcanevet-freebox
- ghoneycutt-puppet
- seteam-role
- petems-cockpit
- jantman-archlinux_workstation
- seteam-splunk
- MiamiOH-httpproxy
- johnclarson-security
- puppetlabs-puppet_agent
- rplessl-influxdb
- geoffwilliams-r_profile
- eshamow-pe_httpd
- fiddyspence-zabbix
- opentable-jenkins_job_builder
- opentable-remaster
- puppetlabs-data_entitlement
- icann-quagga
- rfletcher-aws_cli
- qtechnologies-central_auth
- arusso-nfs
- gabe-ansible
- puppet-jenkins_job_builder
- pltraining-classroom
- softek-zabbixagent
- epflsti-epfl_sso
- pltraining-classroom_legacy
- juame-ospuppet
- Azcender-profile
- CERNOps-abrt
- hetzner-roundcube
- cohdjn-cisecurity
- aharden-puppet_ent_agent
- xdrum-puppet
- locp-cassandra
- fsalum-puppetmaster
- puppet-cassandra
- huit-splunk
- simp-systemd
- sarus-couchdb
- ceh-freetds
- example42-psick
- objectiflibre-puppet
- camptocamp-systemd
- puppet-systemd
- pest-blackfire
- arusso-puppet
- camptocamp-networkmanager
- s12v-blackfire
- a2labs-php
- reidmv-pe_puppet
- opentable-rundeck
- abstractit-puppet
- jdowning-influxdb
- cloudscaling-scaleio_openstack
- ploperations-puppet
- puppet-rundeck
- stephenrjohnson-puppet
- factorit-icingaweb2
- stackstorm-st2
- puppetlabs-puppetdb
- landcareresearch-ckan
- WhatsARanjit-puppetconfig
This module is declared in 125 of 579 indexed public Puppetfiles
.
These results were generated with Rangefinder, a tool that helps predict the downstream impact of breaking changes to elements used in Puppet modules. You can run this on the command line to get a full report.
Exact matches are those that we can positively identify via namespace and the declaring modules' metadata. Non-namespaced items, such as Puppet 3.x functions, will always be reported as near matches only.
Hi @omolenkamp, it seems like your PR has been stale for a while. Our team is currently cleaning up our modules and we are looking to address any stale PRs. Since there hasn't been any engagement for a long time, we have decided to put this PR back on our 'attention-needed' list. Please be aware that PRs with this tag are generally closed within 1-2 weeks if there is no response back from the author and/or engagement from the community. Please be aware that closed PRs an be reopened if the author wants to continue working on them!
Since your PR is a potential fix for one of our currently ongoing issues with the module, we can also keep it open if someone else volunteers to 'adopt' this PR and you agree with it.
Thanks for your understanding.
Sorry, haven't looked at this issue for a while. If the tests can be run again to produce new logs, I'll try to fix the remaining bug soon.
ini_setting is a type
Breaking changes to this file MAY impact these 157 modules (near match):
- ajroetker-puppet_server
- nullfox-pagerduty
- mcanevet-freebox
- eshamow-pe_httpd
- arusso-puppet
- factorit-icingaweb2
- jamtur01-pagerduty
- reidmv-pe_puppet
- stahnma-avahi
- binford2k-wreckit
- humansky-avahi
- arusso-nfs
- fiddyspence-zabbix
- xdrum-puppet
- fsalum-puppetmaster
- jcpunk-tuned
- jgazeley-monyog
- greatboy-bs_sysctl
- puppetfinland-local_git_repo
- cnacorrea-puppet
- abuxton-puppet_mountpoint
- example42-psick_profile
- dhgwilliam-cutover
- WhatsARanjit-puppetconfig
- eshamow-pe_cert_auth
- joshs-agent_noop
- qroac-isp3node
- johnclarson-security
- encore-service_autorestart
- rfletcher-aws_cli
- daenney-circus
- a2labs-php
- jesse-vra_puppet_plugin_prep
- camptocamp-carbon
- puppetlabs-servicenow_cmdb_integration
- puppetlabs-vra_puppet_plugin_prep
- puppetlabs-data_entitlement
- praekeltfoundation-certbot
- puppetfinland-gitdaemon
- geoffwilliams-puppet_nonroot
- simp-dconf
- CERNOps-abrt
- cjtoolseram-puppetconf
- jdowning-influxdb
- rdelcampog-ec2tagfacts
- sebastianrakel-gitea
- camptocamp-networkmanager
- puppetfinland-mcollective
- rauch-kallithea
- jantman-archlinux_workstation
- icann-quagga
- dylanratcliffe-external_data
- juame-ospuppet
- tedivm-hieratic
- cohdjn-cisecurity
- nodes-php
- vholer-openafs
- tomkrieger-security_baseline
- rplessl-influxdb
- qtechnologies-osbaseline
- attestra-pgbackrest
- slenky-notifier
- millerjl1701-supervisord
- zpetr-couchdb
- Xebia-xldeploy
- petems-cockpit
- opentable-remaster
- sarus-couchdb
- zend-zendhq
- danzilio-extended_http_report_processor
- nvpuppet-systemenv
- qtechnologies-resolv_conf
- pdsupport-pagerduty
- opstudio-puppet
- gabe-ansible
- softek-zabbixagent
- example42-psick
- ploperations-puppet
- dylanratcliffe-sumologic
- ceritsc-tuned
- jpadams-windows_diff_enable
- nrvale0-site
- pizzaops-cutover
- opentable-nagdash
- opentable-puppetversion
- brainsware-s3ql
- jebas-add_facts
- shinesolutions-amazon_ssm_agent
- genv-systemd_journal_remote
- ceh-freetds
- qtechnologies-central_auth
- spacepants-ds_389
- camptocamp-dell
- epostdev-one
- hetzner-roundcube
- seteam-role
- epflsti-epfl_sso
- cloudscaling-scaleio_openstack
- stephenrjohnson-puppet
- s12v-blackfire
- huit-splunk
- pltraining-classroom_legacy
- binford2k-node_encrypt
- pest-blackfire
- hunner-hiera
- MiamiOH-httpproxy
- Lavaburn-cabot
- pltraining-classroom
- aharden-puppet_ent_agent
- jay-console_env
- simp-systemd
- crayfishx-netapp_smo
- cm6051-bitbucket
- appuio-openshift
- squreshi157-bitbucket
- frechetta93-httpproxy
- abstractit-puppet
- stackstorm-st2
- halyard-git
- ayohrling-r10k
- fraenki-ds_389
- camptocamp-systemd
- seteam-splunk
- puppet-jenkins_job_builder
- puppet-cassandra
- puppetfinland-puppetmaster
- ghoneycutt-puppet
- puppet-stash
- fervid-secure_linux_cis
- puppet-systemd
- puppet-hiera
- landcareresearch-ckan
- puppet-rundeck
- simp-gdm
- treydock-xdmod
- puppet-php
- simp-pupmod
- puppet-letsencrypt
- puppet-splunk
- Azcender-profile
- objectiflibre-puppet
- ssm-nifi
- aursu-lsys
- tomkrieger-cis_security_hardening
- ncorrare-reportslack
- geoffwilliams-r_profile
- puppet-r10k
- enterprisemodules-easy_type
- puppetlabs-puppet_agent
- puppetlabs-puppetdb
- jcalles-php
- mayflower-php
- danzilio-letsencrypt
- zack-r10k
- opentable-jenkins_job_builder
- locp-cassandra
- opentable-rundeck
This module is declared in 127 of 580 indexed public Puppetfiles
.
These results were generated with Rangefinder, a tool that helps predict the downstream impact of breaking changes to elements used in Puppet modules. You can run this on the command line to get a full report.
Exact matches are those that we can positively identify via namespace and the declaring modules' metadata. Non-namespaced items, such as Puppet 3.x functions, will always be reported as near matches only.
@omolenkamp - looks like our github bot wrongly closed this PR. I've re-opened to kick the tests off like you requested. Apologies for the delay in noticing this!
Looks like its one recurring test failure for all OS's, so hoping its nothing to major to fix. Thanks for your work on this one.
Hey @omolenkamp! I'm going to go ahead and close this PR again due to lack of engagement. If you get some time to address the test failures please do re-open this request and we'll take it from here.
reopen @omolenkamp can you please finish that MR? we also need it but I am not able to write in myself :(