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

Add warning about running Manage with Server and A2 #3159

Merged
merged 4 commits into from
Mar 25, 2020

Conversation

IanMadd
Copy link
Contributor

@IanMadd IanMadd commented Mar 19, 2020

Signed-off-by: IanMadd imaddaus@chef.io

🔩 Description: What code changed, and why?

This adds a warning about deploying Infra Server with Chef Manage and A2.

⛓️ Related Resources

👍 Definition of Done

👟 How to Build and Test the Change

✅ Checklist

📷 Screenshots, if applicable

@IanMadd IanMadd added the documentation Anything related to the Automate docs. label Mar 19, 2020
@IanMadd IanMadd requested a review from a team March 19, 2020 21:02
@@ -12,6 +12,12 @@ toc = true
weight = 60
+++

{{% warning %}}

Chef Automate does not deploy Chef Manage alongside Chef Infra Server.
Copy link
Contributor

Choose a reason for hiding this comment

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

Chef Automate does not support installing any Chef Infra Server add-ons, such as Chef Manage or Push Jobs Server.

Copy link
Contributor

Choose a reason for hiding this comment

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

do we want to mention that supermarket is not supported as well?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

rewrite:

Chef Automate will not deploy the following Chef Infra Server add-ons:

  • Chef Manage
  • Push Jobs Server
  • Supermarket

Copy link
Contributor

Choose a reason for hiding this comment

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

technically supermarket isn't an "addon" as the addons were packages that could be installed with chef-server-ctl install NAME. I was happy to confirm this morning that the chef-server-ctl command shipped with automate is already well pruned and doesn't have the install command at all.

https://docs.chef.io/ctl_chef_server/#install

https://github.com/chef/chef-server/blob/master/src/chef-server-ctl/plugins/install.rb#L4-L8

(reporting is already EOL a while ago).

It's more technically correct that Supermarket cannot integrate with Automate Infra Server because we don't ship the oc-id service which is what supermarket uses to authenticate. We haven't ruled out doing something with Automate on-premises and Supermarket but nothing is currently planned either.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

So maybe two separate warnings.

Chef Automate will not deploy the Chef Infra Server add-ons Chef Manage
and Push Jobs Server.

and

Supermarket cannot authenticate users on Chef Infra Server that is deployed 
with Chef Automate.

Copy link
Contributor

Choose a reason for hiding this comment

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

sounds good to me!

@IanMadd IanMadd force-pushed the im/warning_manage_automate_server branch from e159938 to 2acf725 Compare March 25, 2020 17:02
IanMadd added 4 commits March 25, 2020 12:16
Signed-off-by: IanMadd <imaddaus@chef.io>
Signed-off-by: IanMadd <imaddaus@chef.io>
Signed-off-by: IanMadd <imaddaus@chef.io>
Signed-off-by: IanMadd <imaddaus@chef.io>
@IanMadd IanMadd force-pushed the im/warning_manage_automate_server branch from 2acf725 to 90819b6 Compare March 25, 2020 19:18
@IanMadd IanMadd merged commit ce74f7c into master Mar 25, 2020
@chef-expeditor chef-expeditor bot deleted the im/warning_manage_automate_server branch March 25, 2020 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Anything related to the Automate docs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants