Skip to content

Conversation

@nickanderson
Copy link
Member

Prior to the introduction of package promise modules CFEngine would
This bundle was originally introduced to ensure that CFEngine Enterprise had
software inventory reported immediately after initial install using the default
policy set.

With the package_module implementation mature, there is no longer need to have a
packages promise for a fake package to trigger software inventory data to be
reported so it has been removed.

Unsupported platforms that do not have a default package module for inventory
supplied will no longer have software inventory and will not be able to leverage
the packagesmatching() functions unless there is an explicit
package promise made.

Ticket: ENT-13525
Changelog: Commit

Prior to the introduction of package promise modules CFEngine would
This bundle was originally introduced to ensure that CFEngine Enterprise had
software inventory reported immediately after initial install using the default
policy set.

With the package_module implementation mature, there is no longer need to have a
packages promise for a fake package to trigger software inventory data to be
reported so it has been removed.

Unsupported platforms that do not have a default package module for inventory
supplied will no longer have software inventory and will not be able to leverage
the packagesmatching() functions unless there is an explicit
package promise made.

Ticket: ENT-13525
Changelog: Commit
@cf-bottom
Copy link

Thank you for submitting a PR! Maybe @craigcomstock can review this?

Copy link
Contributor

@craigcomstock craigcomstock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Issues I see:

  • the changelog should not be the entire commit I would say. Instead re-word the first sentence so that it expresses the situation, something like "Disabled fake package promise to trigger packages inventory on platforms which no longer need it"
  • should we really remove this feature for platforms that don't have a default package module?
  • some of the commit message needs work, see below:

Prior to the introduction of package promise modules CFEngine would

This incomplete sentence is left hanging, remove it?

With the package_module implementation mature, there is no longer need to have a
packages promise for a fake package to trigger software inventory data to be
reported so it has been removed.

Maybe "Because the package_module implementation has matured, there is no longer a need to have a packages promise for a fake package ..."

Unsupported platforms that do not have a default package module for inventory
supplied will no longer have software inventory and will not be able to leverage
the packagesmatching() functions unless there is an explicit
package promise made.

But why are we removing this for platforms without a default package module? Why not leave those in place?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants