Skip to content

ENT-11325: Added hostswithgroup policy function#5741

Merged
olehermanse merged 1 commit intocfengine:masterfrom
victormlg:hostswithgroups
Apr 11, 2025
Merged

ENT-11325: Added hostswithgroup policy function#5741
olehermanse merged 1 commit intocfengine:masterfrom
victormlg:hostswithgroups

Conversation

@victormlg
Copy link
Contributor

@victormlg victormlg commented Mar 27, 2025 •
edited
Loading

@victormlg victormlg changed the title ENT-11325: Added new hostswithgroup policy function ENT-11325: Added hostswithgroup policy function Mar 27, 2025
github-advanced-security[bot]
github-advanced-security bot found potential problems Mar 27, 2025
View reviewed changes
@victormlg victormlg force-pushed the hostswithgroups branch 2 times, most recently from 503d732 to 7a12a4c Compare March 27, 2025 12:40
@victormlg victormlg requested a review from olehermanse March 27, 2025 12:43
@victormlg victormlg mentioned this pull request Mar 27, 2025
Merged
@olehermanse olehermanse requested a review from larsewi March 27, 2025 13:45
@olehermanse
Copy link
Member

olehermanse commented Mar 27, 2025

@cf-bottom jenkins, please

@cf-bottom
Copy link

cf-bottom commented Mar 27, 2025

Alright, I triggered a build:

Build Status

Jenkins: https://ci.cfengine.com/job/build-and-deploy-docs-master/13/

Packages: http://buildcache.cfengine.com/packages/testing-pr/jenkins-build-and-deploy-docs-master-13/

Documentation: http://buildcache.cfengine.com/packages/build-documentation-pr/jenkins-build-and-deploy-docs-master-13/output/_site/

craigcomstock
craigcomstock previously approved these changes Mar 27, 2025
View reviewed changes
larsewi
larsewi reviewed Mar 28, 2025
View reviewed changes
Copy link
Contributor

@larsewi larsewi left a comment

Choose a reason for hiding this comment

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

Elegant solution 🚀

@olehermanse
Copy link
Member

olehermanse commented Mar 28, 2025

@cf-bottom jenkins, please

@cf-bottom
Copy link

cf-bottom commented Mar 28, 2025

Alright, I triggered a build:

Build Status

Jenkins: https://ci.cfengine.com/job/build-and-deploy-docs-master/15/

Packages: http://buildcache.cfengine.com/packages/testing-pr/jenkins-build-and-deploy-docs-master-15/

Documentation: http://buildcache.cfengine.com/packages/build-documentation-pr/jenkins-build-and-deploy-docs-master-15/output/_site/

@victormlg
Added new hostswithgroup policy function
ef7da8b 
Changelog: Added new hostswithgroup function to get hosts from a group, selecting a specific field. This function is enterprise only.
Ticket: ENT-11325
Signed-off-by: Victor Moene <victor.moene@northern.tech>
@larsewi
Copy link
Contributor

larsewi commented Apr 2, 2025

Failing tests above are unrelated

@olehermanse
Copy link
Member

olehermanse commented Apr 4, 2025

@cf-bottom test this in Jenkins, please :)

@cf-bottom
Copy link

cf-bottom commented Apr 4, 2025

Alright, I triggered a build:

Build Status

Jenkins: https://ci.cfengine.com/job/build-and-deploy-docs-master/19/

Packages: http://buildcache.cfengine.com/packages/testing-pr/jenkins-build-and-deploy-docs-master-19/

Documentation: http://buildcache.cfengine.com/packages/build-documentation-pr/jenkins-build-and-deploy-docs-master-19/output/_site/

@olehermanse
Copy link
Member

olehermanse commented Apr 10, 2025

@cf-bottom please run this again in jenkins, please

@cf-bottom
Copy link

cf-bottom commented Apr 10, 2025

Alright, I triggered a build:

Build Status

Jenkins: https://ci.cfengine.com/job/build-and-deploy-docs-master/21/

Packages: http://buildcache.cfengine.com/packages/testing-pr/jenkins-build-and-deploy-docs-master-21/

Documentation: http://buildcache.cfengine.com/packages/build-documentation-pr/jenkins-build-and-deploy-docs-master-21/output/_site/

@craigcomstock
Copy link
Contributor

craigcomstock commented Apr 11, 2025

@olehermanse olehermanse merged commit 373d92b into cfengine:master Apr 11, 2025
37 checks passed
@victormlg victormlg deleted the hostswithgroups branch April 23, 2025 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@craigcomstock craigcomstock craigcomstock approved these changes

@larsewi larsewi larsewi approved these changes

@olehermanse olehermanse Awaiting requested review from olehermanse

@nickanderson nickanderson Awaiting requested review from nickanderson

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@victormlg @olehermanse @cf-bottom @larsewi @craigcomstock @nickanderson