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

Added script upgrade from 0.5 to 0.6. #643

Open
wants to merge 64 commits into
base: branch-0.17.0-0.6
Choose a base branch
from

Conversation

lreciomelero
Copy link

Description

Please add the PR description here.

Related Pull Requests

  • PR number: brief description (e.g. "documentation")
  • PR number: brief description (e.g. "tests enhancements")

Pull Request Checklist:

  • [PR title] Include a title referencing a ticket in Jira (e.g. "[CLOUDS-99] Implement a new funcionality").
  • [PR desc] Add a summary of the changes made in simple terms.
  • [PR desc] List any pull-request related to this change (docs, tests, feature, etc.).
  • [PR labels] Add the corresponding labels (release, skips, cherry-pick, AT-eks-smoke, etc).
  • [Docs] Are changes to the documentation required? (if so, please add references to those PRs).
  • [QA] Are new unit tests required according with the changes? (if so, please add references to those PRs).

@lreciomelero lreciomelero requested a review from stg-0 as a code owner December 12, 2024 16:48
@lreciomelero lreciomelero changed the title Added script upgrade from 0.5 to 0.6. OK EKS Added script upgrade from 0.5 to 0.6. Dec 13, 2024
@lreciomelero lreciomelero requested a review from iamjanr as a code owner January 10, 2025 08:53
lreciomelero and others added 15 commits January 20, 2025 17:29
Copy link

@tperez-stratio tperez-stratio left a comment

Choose a reason for hiding this comment

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

@lreciomelero he cogido la PR de vuelta pensando que habías terminado de aplicar cambios pero luego he visto que no y que aún quedan pendientes. No obstante te envío respuesta a los comentarios que habías dejado. Por favor, pide una nueva revisión cuando termines de verlo todo. ¡Gracias!

@@ -0,0 +1,154 @@
= Upgrade Provisioner Script

Choose a reason for hiding this comment

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

Pues te compro lo de aprovisionador por Cloud Provisioner, porque como dices, es el nombre del componente. De hecho, lo correcto sería "Script de actualización de Stratio Cloud Provisioner", pero estando en la documentación de Stratio Cloud Provisioner creo que no es necesario repetir esto, por lo que lo dejaría en "Script de actualización".

Lo que no veo es dejar "upgrade", que es una palabra en inglés que tiene traducción y por ello siempre la utilizamos en la documentación de manera traducida. A veces se traduce por actualización a secas y otra por actualización de versión. ¿Cómo lo ves?

"Script de actualización de versión"


=== Construcción de contenedor de upgrade

Es necesario ejecutar el contenedor Docker utilizando la imagen de upgrade. En él dispondremos del script y del resto de archivos necesarios para ejecutar el proceso:

Choose a reason for hiding this comment

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

Remito al comentario anterior, no sería correcto utilizar la palabra en inglés cuando se puede traducir, a no ser que sea el nombre de algo y vaya en inglés... ¿si ponemos "la imagen de actualización de versión" lo ves mejor?


- Para otros entornos, elimine la sección `images`.

=== Script de Upgrade

Choose a reason for hiding this comment

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

Mismo comentario.

lreciomelero and others added 14 commits January 21, 2025 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.6.1 documentation Improvements or additions to documentation ok-to-merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants