-
Notifications
You must be signed in to change notification settings - Fork 181
ECH and ECE ccs template migration updated #3312
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
Conversation
🔍 Preview links for changed docs |
|
I THINK the ece doc is ok to delete. I think the most common entrypoint for this doc is a link from the UI and it comes with a message:
so I expect the flow would be
argument against this would be that maybe this is exactly the type of thing that would be good to expose in the 7.17 migration instructions? given that you've been thinking a little about what those would look like I'll let you make the final call. |
shainaraskas
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
approved so you can make a choice to keep or get rid of this doc, considering the 7.17 sunset
note: if you do delete it, we'll need to ask the web team to redirect this url to 3.8 (instead of these docs), and remove the listing from https://github.com/elastic/docs/blob/master/resources/asciidoctor/lib/chunker/v3-mapping.json
|
@kunisen , I'd like to get your thoughts here (for the ECE doc): Do you think it's beneficial to keep this doc in the new system? https://docs-v3-preview.elastic.dev/elastic/docs-content/pull/3312/deploy-manage/remote-clusters/ece-migrate-ccs Or would it be enough to leave the 3.8 doc as the reference? https://www.elastic.co/guide/en/cloud-enterprise/3.8/ece-migrate-ccs.html The reason is that the doc is not applicable to ECE 4.x, as before upgrading to ECE 4.x all deployments should have been upgraded to 8.x, and that would imply to get rid of the ccs deployment template. I don't have a clear opinion here, I'm ok removing the doc because it's probably not needed, but I'm ok leaving it also. |
|
@eedugon sorry for the late reply. I think we could keep it there so that people still have the visibility and understand it's not applicable to ECE 4. Also, what's the most important, thanks for highlight the fact that it doesn't apply to ECE 4. This makes the whole story crystal clear. That said, we probably should consider removing this page when we release ECE 5 :) |
I've updated the ECE and ECH docs to migrate from the legacy / removed CCS template to a standard template, as the dedicated template is no longer needed anymore.
I've updated the ECE and ECH docs to migrate from the legacy / removed CCS template to a standard template, as the dedicated template is no longer needed anymore.
I've updated the ECE and ECH docs to migrate from the legacy / removed CCS template to a standard template, as the dedicated template is no longer needed anymore.
@shainaraskas : For the ECE doc we have to take a decision, as the doc is NOT applicable to ECE 4.x at all.
I've added a note explaining that the doc is only valid for ECE 3.x installations running deployments in versions earlier than 8.0 with the mentioned template.
The reason of not being applicable is that before upgrading to ECE 4.x all deployments must be on 8.x, and to upgrade to 8.x this template must be not used by any deployment.
Our options are:
The doc for ECH is valid because in ECH we are still supporting 7.17 deployments and "maybe" this removed template, not sure about that.
PS - I have removed the
deprecatedstatement as if the template isremovedfrom the system that implies a previous deprecation.