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

Fix the documentation following the introduction of SaaS features. #247

Closed
ahsimb opened this issue Jun 26, 2024 · 1 comment · Fixed by #249
Closed

Fix the documentation following the introduction of SaaS features. #247

ahsimb opened this issue Jun 26, 2024 · 1 comment · Fixed by #249
Assignees
Labels
documentation Improvements or additions to documentation

Comments

@ahsimb
Copy link
Collaborator

ahsimb commented Jun 26, 2024

Due are two kinds of changes.

  1. The language container deployment has been moved to the python-extension-common. So has the user guide in this part. This user guide needs to reference the other one.
  2. There is patchy documentation of the new features related to the SaaS deployment. This needs to be sorted out.
@ahsimb ahsimb added the documentation Improvements or additions to documentation label Jun 26, 2024
@ahsimb ahsimb self-assigned this Jun 26, 2024
@ahsimb
Copy link
Collaborator Author

ahsimb commented Jun 26, 2024

From the review of another PR:

+SaaS Database
+
+```buildoutcfg
+python -m exasol_transformers_extension.deploy scripts \

  • --saas-url <SAAS_SERVICE_URL>
    this probably should be optional and default to cloud.exasol.com. Different urls are only necassary for testing.

ahsimb added a commit that referenced this issue Jun 28, 2024
ahsimb added a commit that referenced this issue Jul 1, 2024
* #244 Enabled all tests [CodeBuild]

* #247 Added SaaS deployment documentation

* #244 Addressed the review comments [CodeBuild]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant