Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Toggle to allow warnings to block patching #143 #150

Merged
merged 57 commits into from
Aug 21, 2019

Conversation

albatrossflavour
Copy link
Owner

If there are warnings in the facts, should patching be blocked? New parameter (block_patching_on_warnings) defaults to false.

@albatrossflavour albatrossflavour merged commit a6e9594 into development Aug 21, 2019
@albatrossflavour albatrossflavour deleted the feature/vsbnlp branch August 21, 2019 02:33
@albatrossflavour albatrossflavour self-assigned this Aug 21, 2019
@albatrossflavour albatrossflavour added the enhancement New feature or request label Aug 21, 2019
albatrossflavour pushed a commit that referenced this pull request Aug 21, 2019
* The declared ISO format does not exist, had one extra `dd` (#141)

* Force usage of the 'C' locale (#142)

When parsing command output, we should ensure the utility will emit
messages in the language we are using for matching patterns.  Force the
locale to be 'C' by setting the LC_ALL environment variable.

* link to the wiki

* #128 additional queries as examples

* Add missing dependency (#144)

The `Exec[os_patching::exec::fact]` requires the
`${cache_dir}/reboot_override` file.

* Updates to facter and bug fixes (#148)

* #143 try a new way of doing matching for pinned packages

* Fixed typos

* #147 update the fact with the new data

* #147 fix ordering

* #147 only create the mismatch array if the file has data in it

* #147 cross linux distro compatability

* #147 sles compatability

* #145 add a list of KB updates to the fact

* Fix handling of KBArticleIDs (#146)

* Fix handling of KBArticleIDs

- Ensures all KBArticleIDs are processed, not just the first one in the array
- Ensures the ID's start with "KB", which aids reusing the fact info later on.

* fix typo in variable

typo in variable (kbslist --> kblist) on line 57

* #143 try a new way of doing matching for pinned packages

* Fixed typos

* #147 update the fact with the new data

* #147 fix ordering

* #147 only create the mismatch array if the file has data in it

* #147 cross linux distro compatability

* #147 sles compatability

* #145 add a list of KB updates to the fact

* Fix handling of KBArticleIDs (#146)

* Fix handling of KBArticleIDs

- Ensures all KBArticleIDs are processed, not just the first one in the array
- Ensures the ID's start with "KB", which aids reusing the fact info later on.

* fix typo in variable

typo in variable (kbslist --> kblist) on line 57

* Fix missing slash (#147)

* #143 Fix formatting and use puppet_vardir rather than puppet_client_datadir

* #143 change to clientcert rather than fqdn for the catalog name

* rename element to version_specified_but_not_locked_packages

* Toggle to allow warnings to block patching #143 (#150)

* #143 first stab at a new parameter to control VSBNLP behaviour

* Try the new format for the locked files

* Abort flag

* update fact

* #149 allow Debian to run `apt-get autoremove` at reboot (#151)

* Add an option to the task to allow the running of `apt-get autoremove` after patching

* #149 try both ways of doing the autoremove

* #149 fix default

* #149 fix default

* #149 allow cron job to be removed

* #149 revert changes to the task now that we are doing the autoremove via cron

* Add an option to the task to allow the running of `apt-get autoremove` after patching

* #149 try both ways of doing the autoremove

* #149 fix default

* #149 fix default

* #149 allow cron job to be removed

* #149 revert changes to the task now that we are doing the autoremove via cron

* Pre-v0.12.0 release
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant