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

Run integration tests with Kubernetes Executor too #1906

Open
9 tasks
pankajkoti opened this issue Apr 24, 2023 · 0 comments
Open
9 tasks

Run integration tests with Kubernetes Executor too #1906

pankajkoti opened this issue Apr 24, 2023 · 0 comments
Labels
feature New feature or request

Comments

@pankajkoti
Copy link
Contributor

Please describe the feature you'd like to see
Currently, we run integration tests only on Celery Executor in our deployment. Customer is facing an issue with K8s executor on Runtime 7.3, details in Zendesk ticket.

We should also run integration tests (Example DAGs) on Kubernetes Executor to catch these issues early on.

Acceptance Criteria

  • All checks and tests in the CI should pass
  • Unit tests (90% code coverage or more, once available)
  • Integration tests (if the feature relates to a new database or external service)
  • Example DAG
  • Docstrings in reStructuredText for each of methods, classes, functions and module-level attributes (including Example DAG on how it should be used)
  • Exception handling in case of errors
  • Logging (are we exposing useful information to the user? e.g. source and destination)
  • Improve the documentation (README, Sphinx, and any other relevant)
  • How to use Guide for the feature (example)
@pankajkoti pankajkoti added the feature New feature or request label Apr 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant