Release version 7.6.0

Merge pull request #949 from IrimieBogdan/IAC-1143

(IAC-1143) Custom fact is called twice

(IAC-1143) Remove temporary fix.

Merge branch 'main' into IAC-1143

(IAC-1143) Run Facter.clear before each test.

Merge pull request #948 from sheenaajay/purgeoption

(MODULES-10804) option to force purge source.lists file

(IAC-1143) Adding temporary workaround for facter failures.

(MODULES-10804) option to force purge source.lists file

Merge pull request #947 from david22swan/IAC-978

(IAC-978) - Removal of inappropriate terminology

(IAC-978) - Removal of inappropriate terminology

(IAC-978) - PDK Update

Merge pull request #946 from puppetlabs/release

(maint) - Release version 7.5.0 mergeback

Release version 7.5.0

Merge pull request #942 from gguillotte/clarify-loglevel

(MODULES-10763) loglevel won't affect reports

(MODULES-10763) loglevel won't affect reports

The README suggests that setting a sufficiently high `loglevel` metaparameter
value will squelch the `Exec[apt_update]` resource's corrective changes from
agent reports. This is not true. Correct and clarify this statement to limit
its scope to log output, and link to MODULES-10763 for context on the exec
resource's false-positive reporting of corrective changes as a known issue.

Merge pull request #940 from puppetlabs/pdksync_IAC-973

pdksync - (IAC-973) - Update travis/appveyor to run on new default branch main

(IAC-973) - Update travis/appveyor to run on new default branch main

Merge pull request #938 from mdklapwijk-forks/support-acng-ssl

patch-acng-ssl-support

patch-acng-ssl-support

Support for https repositories via caching proxy apt-cacher-ng using the "tell-me-what-you-need" method as described in the documentation:
- https://www.unix-ag.uni-kl.de/~bloch/acng/html/howtos.html#ssluse

Merge pull request #936 from puppetlabs/pdksync_IAC-746

(IAC-746) - Add ubuntu 20.04 support

(IAC-746) - Add ubuntu 20.04 support

Merge pull request #935 from puppetlabs/pdksync_abs

pdksync - Use abs instead of vmpooler to provision test resources

Switch to default spec_helper_acceptance file

Change provision from vmpooler to abs

Merge pull request #934 from puppetlabs/release

(maint) v7.4.2 Release Merge Back

Adding CI: true to github workflow

Release version 7.4.2

Merge pull request #933 from puppetlabs/pdksync_PDK_UPDATE

pdksync - (maint) - Pdk Update

(maint) - add back gems removed by pdk update

(maint) - Pdk Update

Merge pull request #931 from puppetlabs/pdksync_dependency_gems_update

pdksync - Add dependency gems to development group

Add net-ssh dependency gems to development group

Merge pull request #924 from gimmyxd/facter_4

(maint) Update apt custom facts

Merge pull request #927 from tryfunc/fix_apt_mark

fix apt-mark syntax

fix unless_cmd

apt-mark supports the following syntax:

 * showauto
 * showmanual
 * showhold

Signed-off-by: Vadim Chernyshev <tryfunc@gmail.com>

(maint) Update apt custom facts

This commit updates apt custom facts because the evaluation
of custom facts code has changed between facter 2 and facter 4.
In facter 2, when a custom fact is loaded, the code in `add` block
was not executed.
In facter 4, when a custom fact is loaded, the code outside `setcode`
block is executed(same as Facter 3).

Merge pull request #923 from anarcat/no-mode

Do not specify file modes unless relevant

do not specify file modes unless relevant

MODULES-10583 makes a good point: "why are you messing with my file
permissions"? In my case, the entire reason I made the following
change (in PR #906):

ab2e06b72f2be8dc38d6e3ecec68dc2cdacbce4e MODULES-10548: make files readonly

... is exactly *because* Puppet was changing the file modes from under
me. I was migrating from our own in-house APT module to the forge one,
and our module did *not* intervene in those file modes: it left the
file resources alone. Which means we could have a directive like this:

    File {
        owner   => root,
        group   => root,
        mode    => '444',
        ensure  => file,
    }

... which made all files readonly by default. So when I migrated to
the Puppetlabs APT module, modes were changed to be writable, which I
did not want.

As I reasoned in MODULES-10548, having files readonly provides an
excellent indicator that a file is managed by Puppet, even if some
module does not add a warning header - either because it forgot or
because it's impossible. But I also understand if people do not like
that policy.

I think the proper way of doing this is not specifying a mode at all,
and let local site-specific policies apply. I specifically proppose
this as an alternative to #921 because I believe adding more
parameters to the resources will needlessly complicate the script,
when we have a native, Puppet-DSL supported way of changing those
modes according to the right scope and context.

In a similar way, we might want to reconsider user and group ownership
of the files, but that can be done in a later time.

This reverts commit 316fd8f4dd1eb6595b3be495ccc5f4924da4de1b.

Signed-off-by: Antoine Beaupré <anarcat@debian.org>

Merge pull request #922 from michaeltlombardi/ticket/release/IAC-605

(IAC-605) Prep for 7.4.1 release

(IAC-605) Prep for 7.4.1 release

Merge pull request #920 from carabasdaniel/rollback_permissions

(MODULES-10583) Revert "MODULES-10548: make files readonly"

Revert "MODULES-10548: make files readonly"

This reverts commit ab2e06b72f2be8dc38d6e3ecec68dc2cdacbce4e.

Merge pull request #919 from michaeltlombardi/maint/master/update-docker-image-names

(MAINT) Update docker image names

(MAINT) Update docker image names

Prior to this commit the provision file referenced the old
waffleimage repo for CI docker images to use. This commit
updates the file to point at litmusimage, which is the repo
the latest images are released to.

Merge pull request #918 from puppetlabs/pdksync_IAC-555

(IAC-555) pdksync - Remove distelli-manifest.yml

(IAC-555) - Remove distelli-manifest.yml

Merge pull request #917 from puppetlabs/release

(MODULES-10561) Release Merge back

Merge pull request #916 from david22swan/release_prep

(MODULES-10559) - v7.4.0 Release Preparation

Merge pull request #914 from puppetlabs/pdksync_weekly_run

pdksync - Update weekly scheduled workflows

Merge pull request #915 from puppetlabs/pdksync_PDK_UPDATE

pdksync - (maint) - Pdk Update

(MODULES-10559) - v7.4.0 Release Preparation

(maint) - Pdk Update

Update weekly workflow schedule

Merge pull request #911 from puppetlabs/pdksync_weekly_run

pdksync - Add weekly scheduled workflows

Add weekly workflow on default branch

Merge pull request #910 from paladox/patch-1

Add 'include' param to apt::backports

Add 'include' param to apt::backports

Merge pull request #906 from anarcat/modes

MODULES-10548: make files readonly

MODULES-10548: make files readonly

Files created by the apt module are mode writable by the
owner. Because those files are managed by Puppet, they should really
not be writable by anyone, even root. While root can bypass those
warnings, having files readonly does provide an immediate and reliable
indication that a file should not be edited on site, on top of the
usual top of file warnings.

This also fixes a problem with sources.list.d being non-executable,
which Puppet seems to ignore, but seems better to keep consistent.

Merge pull request #905 from anarcat/no-release

MODULES-10543: only consider lsbdistcodename for apt-transport-https

Merge pull request #909 from puppetlabs/pdksync_IAC-215-3

pdksync - (IAC-215) - Implement use_litmus:true

(IAC-215) - Pdk Update

(IAC-215) - Implement use_litmus:true

Merge pull request #904 from anarcat/sources-list-remove

MODULES-10543: remove sources.list file on purging

only consider lsbdistcodename for apt-transport-https

It does not matter if we have *one* older source from (say) jessie or
stretch, we could still be running buster or later. The latter is more
reliably indicated by the fact than by the provided "release"
parameter, which is really just an arbitrary string that does not
necessarily match a Debian suite, especially for third-party
repositories.

I have had a problem with this setting when deploying a "stretch"
repository on a "buster" system because I needed to keep MongoDB
running (which is gone from buster). The "stretch" line triggered this
line, which conflicted with a `ensure_packages` I had elsewhere in our
code base, to *remove* the `apt-transport-https` package from buster
and later.

An alternative implementation might prefer to remove the package
unconditionnally if we run a newer release, but I figured I would keep
the changes to a minimum.

MODULES-10543: remove sources.list file on purging

The current sources.list purge behavior is inconsistent with the
preferences behavior. The former creates a file with a comment, the
latter removes the file.

This harmonizes this behavior between the two files and simply drops
the file if it is to be purged.

Merge pull request #903 from puppetlabs/pdksync_pdksync-honeycombsetup

(IAC-365) updating tokens and dataset for honeycomb

(IAC-365) updating tokens and dataset for honeycomb

Merge pull request #902 from puppetlabs/pdksync_deb10

pdksync - (FM-8581) - Debian 10 added to travis and provision file refactored

(FM-8581) - Debian 10 added to travis and provision file refactored

Merge pull request #891 from synyx/include_apt

Include apt in apt::backports

Merge pull request #901 from puppetlabs/pdksync_github_actions

pdksync - FM-8834 - Add GitHub action workflow

Add GitHub actions workflow

Merge pull request #899 from puppetlabs/pdksync_travis-updates

pdksync - (maint) travis updates: bundler and slack

(maint) update travis to notify puppet slack

(maint) update travis to not use bundle update --system

Merge pull request #898 from puppetlabs/release

(maint) Release Mergeback v7.3.0

(maint) Releaseprep v7.3.0

Merge pull request #897 from puppetlabs/pdksync_pdksync-addubuntu14

MODULES-10242 Add ubuntu14 support back to the modules

MODULES-10242 Add ubuntu14 support back to the modules

include module apt in backports

Otherwise there can be issues with undef values.
like:

Evaluation Error: Operator '[]' is not applicable to an Undef Value. (file: /etc/puppetlabs/code/environments/production/modules/apt/manifests/backports.pp, line: 73, column: 18) on node …

Merge pull request #894 from atarax/master

MODULES-10063, extend apt::key to support deeplinks, this time with f…

Merge pull request #890 from luckyraul/patch-1

Adding a new parameter for dist

Merge pull request #896 from puppetlabs/pdksync_pdksync-disableforge

pdksync - "MODULES-10236 disable deploy_to_forge for the module"

MODULES-10236 disable deploy_to_forge for the module

MODULES-10063, extend apt::key to support deeplinks, this time with filename (https://some.source/somefile.key)

Merge pull request #893 from puppetlabs/pdksync_simplecov

(MODULES-10120) enable simplecov; update to PDK 1.14.1; minor cleanups

(maint) Update to PDK 1.14.1

(maint) enable simplecov for ruby codecoverage testing

(maint) update gitignore and pdkignore for .project file removal

(maint) remove obsolete gepetto .project files

(maint) mark .gitlab-ci.yml as deleted

(maint) sort and canonicalise .sync.yml

Merge pull request #887 from puppetlabs/pdksync_encrypt-git

pdksync - (FM-8634) ensure encrypted communication for fixtures

Merge pull request #892 from atarax/master

MODULES-10063, extend apt::key to support deeplinks

MODULES-10063, extend apt::key to support deeplinks, current implementation only supports baseurls

Merge pull request #888 from glennsarti/fix-travis

(PDK-1501) Fix acceptance stages in Travis CI

Merge pull request #889 from puppetlabs/pdksync_remove_support_ubuntu14

pdksync - Remove Ubuntu 14 support

Linux Mint Support

Feature for a new possibility to replace LinuxMint with Ubuntu

remove ubuntu 14 support