You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/03/23 21:13:49 UTC
[GitHub] [airflow] sunkickr opened a new pull request #14968: Fix Spinx Issues with Docstrings
sunkickr opened a new pull request #14968:
URL: https://github.com/apache/airflow/pull/14968
This PR fix some typos and issues were the doctrings do not adhere to sphinx standards in the following modules:
- AWSDataSyncOperator
- S3KeySizeSensor
- DatadogHook
- DatadogSensor
- ComputeEngineSSHHook
- CloudDataCatalogLookupEntryOperator
- GoogleDisplayVideo360UploadLineItemsOperator
- GoogleDisplayVideo360CreateSDFDownloadTaskOperator
- GoogleDisplayVideo360SDFtoGCSOperator
- AzureBatchOperator
- AzureCosmosDocumentSensor
- SingularityOperator
- SlackAPIOperator
- DataprocCreateMapReduceJobOperator
- DataprocCreateSparkJobOperator
- DataprocCreatePysparkJobOperator
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] github-actions[bot] commented on pull request #14968: Fix Sphinx Issues with Docstrings
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #14968:
URL: https://github.com/apache/airflow/pull/14968#issuecomment-808278149
[The Workflow run](https://github.com/apache/airflow/actions/runs/690179007) is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] kaxil merged pull request #14968: Fix Sphinx Issues with Docstrings
Posted by GitBox <gi...@apache.org>.
kaxil merged pull request #14968:
URL: https://github.com/apache/airflow/pull/14968
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] github-actions[bot] commented on pull request #14968: Fix Sphinx Issues with Docstrings
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #14968:
URL: https://github.com/apache/airflow/pull/14968#issuecomment-806214613
[The Workflow run](https://github.com/apache/airflow/actions/runs/684574320) is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] sunkickr commented on pull request #14968: Fix Sphinx Issues with Docstrings
Posted by GitBox <gi...@apache.org>.
sunkickr commented on pull request #14968:
URL: https://github.com/apache/airflow/pull/14968#issuecomment-806162008
@kaxil is there anything that needs to be done with this PR to merge? Not really sure why the tests are failing. Error message says `ERROR: After October 2020 you may experience errors when installing or updating packages. This is because pip will change the way that it resolves dependency conflicts.`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] kaxil commented on pull request #14968: Fix Sphinx Issues with Docstrings
Posted by GitBox <gi...@apache.org>.
kaxil commented on pull request #14968:
URL: https://github.com/apache/airflow/pull/14968#issuecomment-806260779
@potiuk Any ideas on why the Provider packages job is trying to rebuild image, I checked only jobs on 2.0.1 and old commits -- where it didn't
Example that worked: https://github.com/apache/airflow/runs/2124903826#step:6:87
Example that fails: https://github.com/apache/airflow/runs/2188111255#step:6:76
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] uranusjr commented on pull request #14968: Fix Sphinx Issues with Docstrings
Posted by GitBox <gi...@apache.org>.
uranusjr commented on pull request #14968:
URL: https://github.com/apache/airflow/pull/14968#issuecomment-806219048
The same conflict seems to be failing CI on other PRs as well.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] sunkickr edited a comment on pull request #14968: Fix Sphinx Issues with Docstrings
Posted by GitBox <gi...@apache.org>.
sunkickr edited a comment on pull request #14968:
URL: https://github.com/apache/airflow/pull/14968#issuecomment-806162008
@kaxil is there anything that needs to be done to merge? Not sure why the tests are failing. Error message says `ERROR: After October 2020 you may experience errors when installing or updating packages. This is because pip will change the way that it resolves dependency conflicts.`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] github-actions[bot] commented on pull request #14968: Fix Sphinx Issues with Docstrings
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #14968:
URL: https://github.com/apache/airflow/pull/14968#issuecomment-805411803
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] github-actions[bot] commented on pull request #14968: Fix Sphinx Issues with Docstrings
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #14968:
URL: https://github.com/apache/airflow/pull/14968#issuecomment-806025649
[The Workflow run](https://github.com/apache/airflow/actions/runs/683879198) is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] potiuk commented on pull request #14968: Fix Sphinx Issues with Docstrings
Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #14968:
URL: https://github.com/apache/airflow/pull/14968#issuecomment-806314774
This is super strange. The root cause is that between one step and another looks like the yarn.lock and package.json are changed ! but nothing touches them really between the two (and it always worked fine)
In another PR we see the same:
https://github.com/apache/airflow/pull/14968/checks?check_run_id=2188403270#step:5:125
```
Updated md5sum file /home/runner/actions-runner/_work/airflow/airflow/.build/master/3.6/CI/package.json.md5sum for /home/runner/actions-runner/_work/airflow/airflow/airflow/www/package.json.
Updated md5sum file /home/runner/actions-runner/_work/airflow/airflow/.build/master/3.6/CI/yarn.lock.md5sum for /home/runner/actions-runner/_work/airflow/airflow/airflow/www/yarn.lock.
```
And in the next step (same runner, same job):
https://github.com/apache/airflow/pull/14968/checks?check_run_id=2188403270#step:6:86
```
The md5sum changed for /home/runner/actions-runner/_work/airflow/airflow/airflow/www/package.json
The md5sum changed for /home/runner/actions-runner/_work/airflow/airflow/airflow/www/yarn.lock
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] github-actions[bot] commented on pull request #14968: Fix Sphinx Issues with Docstrings
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #14968:
URL: https://github.com/apache/airflow/pull/14968#issuecomment-805411745
[The Workflow run](https://github.com/apache/airflow/actions/runs/681513848) is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] github-actions[bot] commented on pull request #14968: Fix Sphinx Issues with Docstrings
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #14968:
URL: https://github.com/apache/airflow/pull/14968#issuecomment-806025864
[The Workflow run](https://github.com/apache/airflow/actions/runs/683882306) is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] github-actions[bot] commented on pull request #14968: Fix Spinx Issues with Docstrings
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #14968:
URL: https://github.com/apache/airflow/pull/14968#issuecomment-805328870
The PR is likely OK to be merged with just subset of tests for default Python and Database versions without running the full matrix of tests, because it does not modify the core of Airflow. If the committers decide that the full tests matrix is needed, they will add the label 'full tests needed'. Then you should rebase to the latest master or amend the last commit of the PR, and push it with --force-with-lease.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] kaxil commented on pull request #14968: Fix Sphinx Issues with Docstrings
Posted by GitBox <gi...@apache.org>.
kaxil commented on pull request #14968:
URL: https://github.com/apache/airflow/pull/14968#issuecomment-806209182
Looks like an un-related failure
```
apache-beam 2.28.0 has requirement dill<0.3.2,>=0.3.1.1, but you have dill 0.3.3.
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] github-actions[bot] commented on pull request #14968: Fix Sphinx Issues with Docstrings
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #14968:
URL: https://github.com/apache/airflow/pull/14968#issuecomment-808368817
[The Workflow run](https://github.com/apache/airflow/actions/runs/690652653) is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org