Skip to content
This repository has been archived by the owner on Jan 15, 2025. It is now read-only.

(CAT-643) - Add puppet_forge to pdk-runtime #919

Merged
merged 1 commit into from
Oct 14, 2024
Merged

(CAT-643) - Add puppet_forge to pdk-runtime #919

merged 1 commit into from
Oct 14, 2024

Conversation

danadoherty639
Copy link
Contributor

@danadoherty639 danadoherty639 commented Oct 4, 2024

This PR adds 5.0.4 puppet forge gem as a component and a dependency to the PDK project

dependencies of puppet-forge gem:
faraday ~> 2.0 -> v2.12.0 added as component
faraday-follow_redirects ~> 0.3.0 -> added as component
minitar < 1.0.0 -> v0.9.0 already present in runtime
semantic_puppet ~> 1.0 -> semantic puppet version 1.1.0 already present in runtime (default version)

https://jenkins-platform.delivery.puppetlabs.net/view/vanagon-generic-builder/job/platform_vanagon-generic-builder_vanagon-packaging_generic-builder/3293/

@danadoherty639 danadoherty639 force-pushed the cat_643 branch 2 times, most recently from 4f79de5 to e2e4979 Compare October 4, 2024 11:06
Copy link
Contributor

@david22swan david22swan left a comment

Choose a reason for hiding this comment

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

LGTM
But since this is touching a used component would prefer another team signing of as well

@jordanbreen28 jordanbreen28 marked this pull request as draft October 4, 2024 13:01
@jordanbreen28 jordanbreen28 marked this pull request as ready for review October 8, 2024 10:09
@danadoherty639 danadoherty639 force-pushed the cat_643 branch 7 times, most recently from f0795d4 to b4e139f Compare October 8, 2024 15:43
@jordanbreen28 jordanbreen28 changed the title (CAT-643) - Add puppet_forge (CAT-643) - Add puppet_forge to pdk-runtime Oct 9, 2024
@joshcooper
Copy link
Contributor

joshcooper commented Oct 10, 2024

Changes look good, only the pdk-runtime is affected (at least on RHEL 7)

$ bundle exec rake vanagon:component_diff -- --from dd4b02c --to HEAD --project all --platform el-7-x86_64
...
Here is what your code changes would affect:

Project pe-bolt-server-runtime-2023.8.x
Nothing is affected 😊
Project pe-installer-runtime-main
Nothing is affected 😊
Project pe-bolt-server-runtime-main
Nothing is affected 😊
Project agent-runtime-7.x
Nothing is affected 😊
Project pe-bolt-server-runtime-2021.7.x
Nothing is affected 😊
Project pe-installer-runtime-2021.7.x
Nothing is affected 😊
Project bolt-runtime
Nothing is affected 😊
Project pdk-runtime

Platform name: el-7-x86_64
    Component 'rubygem-faraday' was newly added, not showing diff for it

    Component 'rubygem-faraday-follow_redirects' was newly added, not showing diff for it

    Component 'rubygem-faraday-net_http' was newly added, not showing diff for it

    Component 'rubygem-puppet_forge' was newly added, not showing diff for it

    Component 'rubygem-semantic_puppet' was newly added, not showing diff for it

Project client-tools-runtime-2023.8.x
Nothing is affected 😊
Project client-tools-runtime-main
Nothing is affected 😊
Project pe-installer-runtime-2023.8.x
Nothing is affected 😊
Project client-tools-runtime-2021.7.x
Nothing is affected 😊
Project agent-runtime-main
Nothing is affected 😊

One nit, could you update your commit message to describe why the puppet_forge gem is being added and why the other gems were bumped?

Also could you attempt to build the pdk runtime in the vanagon-generic-builder job https://jenkins-platform.delivery.puppetlabs.net/view/vanagon-generic-builder/job/platform_vanagon-generic-builder_vanagon-generic-builder-init_generic-builder/? You'll need to point it to your github username, repo, branch, vanagon project and vanagon build targets. It's a good idea to try to build on at least one platform of each supported OS (RHEL, Ubuntu, Windows, macOS)

@danadoherty639
Copy link
Contributor Author

Changes look good, only the pdk-runtime is affected (at least on RHEL 7)

$ bundle exec rake vanagon:component_diff -- --from dd4b02c --to HEAD --project all --platform el-7-x86_64
...
Here is what your code changes would affect:

Project pe-bolt-server-runtime-2023.8.x
Nothing is affected 😊
Project pe-installer-runtime-main
Nothing is affected 😊
Project pe-bolt-server-runtime-main
Nothing is affected 😊
Project agent-runtime-7.x
Nothing is affected 😊
Project pe-bolt-server-runtime-2021.7.x
Nothing is affected 😊
Project pe-installer-runtime-2021.7.x
Nothing is affected 😊
Project bolt-runtime
Nothing is affected 😊
Project pdk-runtime

Platform name: el-7-x86_64
    Component 'rubygem-faraday' was newly added, not showing diff for it

    Component 'rubygem-faraday-follow_redirects' was newly added, not showing diff for it

    Component 'rubygem-faraday-net_http' was newly added, not showing diff for it

    Component 'rubygem-puppet_forge' was newly added, not showing diff for it

    Component 'rubygem-semantic_puppet' was newly added, not showing diff for it

Project client-tools-runtime-2023.8.x
Nothing is affected 😊
Project client-tools-runtime-main
Nothing is affected 😊
Project pe-installer-runtime-2023.8.x
Nothing is affected 😊
Project client-tools-runtime-2021.7.x
Nothing is affected 😊
Project agent-runtime-main
Nothing is affected 😊

One nit, could you update your commit message to describe why the puppet_forge gem is being added and why the other gems were bumped?

Also could you attempt to build the pdk runtime in the vanagon-generic-builder job https://jenkins-platform.delivery.puppetlabs.net/view/vanagon-generic-builder/job/platform_vanagon-generic-builder_vanagon-generic-builder-init_generic-builder/? You'll need to point it to your github username, repo, branch, vanagon project and vanagon build targets. It's a good idea to try to build on at least one platform of each supported OS (RHEL, Ubuntu, Windows, macOS)

Hey Josh I thought the vanagon-builder init jenkins step has already been done, I attached the link in the description?

(CAT-643) - This commit adds 5.0.4 puppet forge gem as a component and a dependency to the PDK project

dependencies of puppet-forge gem:
faraday ~> 2.0 -> v2.12.0 added as component
faraday-follow_redirects ~> 0.3.0 -> added as component
minitar < 1.0.0 -> v0.9.0 already present in runtime
semantic_puppet ~> 1.0 -> semantic puppet version 1.1.0 already present in runtime (default version)
@joshcooper
Copy link
Contributor

I attached the link in the description

Oh sorry, I missed that. Looks good.

@joshcooper joshcooper merged commit 903be86 into master Oct 14, 2024
3 checks passed
@joshcooper joshcooper deleted the cat_643 branch October 14, 2024 16:28
@david22swan david22swan added the pdk label Nov 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants