Merge pull request #968 from Disha-maker/release

(IAC-1436) Release Prep PR version 7.7.1

Release version 7.7.1

Merge pull request #967 from puppetlabs/pdksync_gha-updates

pdksync - Update gha templates

Wrap apply_manifest into RSpec context; initialise litmus earlier

This fixes the following error by exposing the necessary PuppetLitmus context:

```
An error occurred while loading ./spec/acceptance/01_apt_class_spec.rb.
Failure/Error: apply_manifest(lsb_package)
NameError:
  undefined local variable or method `os' for main:Object
# ./vendor/bundle/ruby/2.5.0/gems/puppet_litmus-0.22.0/lib/puppet_litmus/puppet_helpers.rb:86:in `block in apply_manifest'
# ./vendor/bundle/ruby/2.5.0/gems/honeycomb-beeline-2.4.0/lib/honeycomb/client.rb:70:in `start_span'
# ./vendor/bundle/ruby/2.5.0/gems/puppet_litmus-0.22.0/lib/puppet_litmus/puppet_helpers.rb:41:in `apply_manifest'
# ./spec/spec_helper_acceptance_local.rb:17:in `<top (required)>'
# ./spec/spec_helper_acceptance.rb:4:in `require'
# ./spec/spec_helper_acceptance.rb:4:in `<top (required)>'
# ./spec/acceptance/01_apt_class_spec.rb:3:in `require'
# ./spec/acceptance/01_apt_class_spec.rb:3:in `<top (required)>'
```

(maint) Update github actions

Merge pull request #965 from puppetlabs/RE-13436

(RE-13415) Update GPG key in example

(RE-13415) Update GPG key in example

Provide recently changed gpg key update in the example puppet code.

Merge pull request #964 from kenyon/use-modern-os-facts

Use modern os facts

readmes: update note about lsb-release for Puppet 7

spec/classes/apt_backports_spec.rb: fix Mint case

Per
https://github.com/camptocamp/facterdb/blob/master/facts/3.14/linuxmint-19-x86_64.facts#L324
(assuming LinuxMint 17 used the same capitalization as newer
versions).

examples/backports.pp: fix class usage

It's a class, not a defined type.

I missed this in https://github.com/puppetlabs/puppetlabs-apt/pull/962

README: fix link to litmus tutorial

spec_helper_acceptance_local.rb: improve comment about lsb-release

add an acceptance test for apt::backports

manifests: use modern os facts

whitespace cleanup

Merge pull request #963 from puppetlabs/pdksync

Update to puppet-module-gems 1.0, pdk-templates and new rubocop

Fix check for ubuntu version

Apply remaining rubocop fixes

Add all safe auto corrects

Add the FrozenStrings magic comment

Update to pdk-templates and puppet-module-gems 1.0.0

Merge pull request #962 from kenyon/patch-1

backports.pp: correct example

backports.pp: correct example

The example treated this like a defined type, but it's a class.

Merge pull request #961 from puppetlabs/release

(maint) Release Merge Back PR  v7.7.0

Release version 7.7.0

Merge pull request #960 from puppetlabs/pdksync_remove_el6_tests

pdksync - Remove EL6 testing from Travis

Remove EL6 testing from Travis

Merge pull request #957 from carabasdaniel/main

PDK Update for GA workflows

Merge pull request #958 from puppetlabs/pdksync_bump_puppet7

pdksync - (feat) - Add support for Puppet 7

(feat) - Bump Puppet boundary

Merge pull request #956 from puppetlabs/pdksync_maint/main/release_checks_7_config

pdksync - (maint) Add release_checks_7 config to provision.yaml

PDK Update for GA workflows

(maint) Add release_checks_7 config to provision.yaml

Merge pull request #953 from suchpuppet/main

Make auth.conf contents Sensitive

Update unit test to support Sensitive data type

Merge pull request #952 from puppetlabs/pdksync_gitpod

pdksync - Update pdk-templates for gitpod and codespaces support

Make auth.conf contents Sensitive

Update pdk-templates for gitpod and codespaces support

Merge pull request #950 from puppetlabs/release

(maint) Release version v7.6.0

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