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

Target Mode docs #4267

Merged
merged 26 commits into from
Jun 5, 2024
Merged

Target Mode docs #4267

merged 26 commits into from
Jun 5, 2024

Conversation

IanMadd
Copy link
Contributor

@IanMadd IanMadd commented Apr 24, 2024

Description

Add documentation about Target Mode.

Definition of Done

Issues Resolved

[List any existing issues this PR resolves, or any Discourse or
StackOverflow discussion that's relevant]

Related PRs

Check List

  • Spell Check
  • Local build
  • Examine the local build
  • All tests pass

Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
Copy link

netlify bot commented Apr 24, 2024

Deploy Preview for chef-web-docs ready!

Name Link
🔨 Latest commit 8b58a01
🔍 Latest deploy log https://app.netlify.com/sites/chef-web-docs/deploys/66607e6a27617b00089f02aa
😎 Deploy Preview https://deploy-preview-4267--chef-web-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

AnkurMundhra and others added 19 commits April 24, 2024 20:35
Signed-off-by: dishanktiwari2501 <dtiwari@progress.com>
Signed-off-by: dishanktiwari2501 <dtiwari@progress.com>
Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
@IanMadd IanMadd marked this pull request as ready for review May 14, 2024 18:24
@IanMadd IanMadd requested a review from a team as a code owner May 14, 2024 18:24
Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
@IanMadd IanMadd changed the title Start target mode docs Target Mode docs May 14, 2024
Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
content/target_mode.md Outdated Show resolved Hide resolved
user = 'root'
password = '123456'

protocol = ssh

Choose a reason for hiding this comment

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

This should be in single quotes. Otherwise it will give syntax error.

user = 'root'
password = '123456'

protocol = ssh
Copy link

@Vasu1105 Vasu1105 May 23, 2024

Choose a reason for hiding this comment

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

Another thing which I want to keep as note here that protocol keyword does not work if using winrm as transport protocol. What works for both ssh and winrm is transport_protocol

Choose a reason for hiding this comment

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

As per Thomas H windows will not be supported for target mode. We will have to make that change later . You can make the change mentioned above though. @tpowell-progress

Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
PATH="/opt/chefdk/bin:/opt/chefdk/embedded/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin"
GEM_ROOT="/opt/chefdk/embedded/lib/ruby/gems/2.6.0"

0,30 * * * * * chef-runner /usr/bin/chef-client -t <TARGET_NAME>
Copy link

@Vasu1105 Vasu1105 May 24, 2024

Choose a reason for hiding this comment

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

As per my knowledge there is missing peace of information here.

  1. Where this cron file should be? On node or on host?
  2. Second when chef-client runs without --local-mode i.e -z option it will check for the client.pem file and client.rb file. That peace of information is missing here. Unless you have client.pem and client.rb file on the host the chef-client run will fail if not running in a local mode.

Choose a reason for hiding this comment

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

@Vasu1105 Given target mode, I would assume the host is the only one with chef-client, and the usual requirements for chef-client setup would apply. It might be worth calling that out specifically still.

Copy link

sonarcloud bot commented Jun 5, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@IanMadd IanMadd merged commit 74bc95b into main Jun 5, 2024
11 of 12 checks passed
@IanMadd IanMadd deleted the target_mode branch June 5, 2024 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants