Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PE-38219 - Support air gapped installation while using a Windows as Jump host #438

Merged
merged 11 commits into from
Jul 5, 2024

Conversation

cathal41
Copy link
Contributor

@cathal41 cathal41 commented May 10, 2024

Summary

Adding powershell script files for Filesize, Download and the new os Identifier to allow windows users to preform an air gapped installation

Installer download progress support https://github.com/puppetlabs/installer/pull/481

Checklist

  • 🟢 Spec tests.
  • 🟢 Acceptance tests.

Changes include test coverage?

  • Not needed

Have you updated the documentation?

  • Not needed

@cathal41 cathal41 changed the title PE-38219 - Draft PE-38219 - Support air gapped installation while using a Windows as Jump host Jun 13, 2024
@@ -0,0 +1,9 @@
{
Copy link
Collaborator

Choose a reason for hiding this comment

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

is the whole task required? can't you use https://forge.puppet.com/modules/puppetlabs/facts/readme ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Tried to run factor with operatingsystem but because this is an install some people may not have run puppet agent.
Also $facts or $os_family was not available on windows jump host

Copy link
Collaborator

Choose a reason for hiding this comment

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

The task works fine without facter/puppet. They don't need to be installed. That's why I suggested it.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I still think this should be addressed before merging.

@cathal41 cathal41 force-pushed the PE_38219_windowSupportAirGapInstall branch from 754b2fe to 66f40b9 Compare June 26, 2024 08:14
@cathal41 cathal41 marked this pull request as ready for review June 26, 2024 11:36
@cathal41 cathal41 requested review from a team as code owners June 26, 2024 11:36
@cathal41 cathal41 force-pushed the PE_38219_windowSupportAirGapInstall branch from 3e09bd4 to bd80892 Compare July 2, 2024 15:26
Copy link
Contributor

@Jo-Lillie Jo-Lillie left a comment

Choose a reason for hiding this comment

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

tested on windows jumphost successfully ✅

@cathal41 cathal41 force-pushed the PE_38219_windowSupportAirGapInstall branch from 5a7561b to 875a8b3 Compare July 5, 2024 09:45
@@ -84,7 +84,7 @@ jobs:
- name: "Run Static & Syntax Tests"
run: |
bundle exec rake syntax lint metadata_lint check:symlinks check:git_ignore check:dot_underscore check:test_file rubocop
bundle exec dependency-checker metadata.json --override WhatsARanjit/node_manager,0.7.5
# bundle exec dependency-checker metadata.json --override WhatsARanjit/node_manager,0.7.5
Copy link
Collaborator

Choose a reason for hiding this comment

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

why was this commented out?

@cathal41 cathal41 merged commit 6ad0475 into main Jul 5, 2024
48 checks passed
@cathal41 cathal41 deleted the PE_38219_windowSupportAirGapInstall branch July 5, 2024 11:26
@cathal41 cathal41 mentioned this pull request Jul 5, 2024
4 tasks
@bastelfreak
Copy link
Collaborator

@cathal41 there were still open comments. Why was the PR merged? (the same happened in #339)

@cathal41
Copy link
Contributor Author

cathal41 commented Jul 5, 2024

@cathal41 there were still open comments. Why was the PR merged? (the same happened in #339)

This test failure is beyond our control.
A PR has been raised to reinstate this test once the fix is available.
#452

@bastelfreak
Copy link
Collaborator

I don't mind temporary test failures, but https://github.com/puppetlabs/puppetlabs-peadm/pull/438/files#r1639580463 is still an open discussion

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants