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

Status of testing Providers that were prepared on July 27, 2021 (rc2) #17268

Closed
30 of 66 tasks
potiuk opened this issue Jul 27, 2021 · 18 comments
Closed
30 of 66 tasks

Status of testing Providers that were prepared on July 27, 2021 (rc2) #17268

potiuk opened this issue Jul 27, 2021 · 18 comments
Labels
kind:meta High-level information important to the community testing status Status of testing releases

Comments

@potiuk
Copy link
Member

potiuk commented Jul 27, 2021

This is the second attempt (rc2) of releasing July Providers :). I carried over the "checkmarks" from the last testing (#17037) so there is no need to re-test those.

I have a kind request for all the contributors to the latest provider packages release.
Could you help us to test the RC versions of the providers and let us know in the comment, if the issue is addressed there.

Providers that need testing

Those are providers that require testing as there were some substantial changes introduced:

Provider airbyte: 2.1.0rc2

Provider amazon: 2.1.0rc2

Provider apache.druid: 2.0.1rc2

Provider apache.hive: 2.0.1rc2

Provider apache.spark: 2.0.0rc2

Provider apache.sqoop: 2.0.1rc2

Provider cncf.kubernetes: 2.0.1rc2

Provider dingding: 2.0.0rc2

Provider discord: 2.0.0rc2

Provider docker: 2.1.0rc2

Provider google: 5.0.0rc2

Provider hashicorp: 2.0.0rc2

Provider http: 2.0.0rc2

Provider jenkins: 2.0.1rc2

Provider microsoft.azure: 3.1.0rc2

Provider mysql: 2.1.0rc2

Provider odbc: 2.0.0rc2

Provider opsgenie: 2.0.0rc2

Provider oracle: 2.0.0rc2

Provider papermill: 2.0.0rc2

Provider plexus: 2.0.0rc2

Provider postgres: 2.1.0rc2

Provider qubole: 2.0.0rc2

Provider samba: 2.0.0rc2

Provider salesforce: 3.1.0rc2

Provider sftp: 2.1.0rc2

Provider slack: 4.0.0rc2

Provider snowflake: 2.1.0rc2

Provider ssh: 2.1.0rc2

Provider tableau: 2.1.0rc2

New Providers

@potiuk potiuk added the kind:bug This is a clearly a bug label Jul 27, 2021
@raphaelauv
Copy link
Contributor

raphaelauv commented Jul 27, 2021

for docker 2.1.0rc2 it's all good

docker in docker work and also the XCOM


t3 = DockerOperator(
    api_version='auto',
    docker_url="unix://var/run/docker.sock",
    command='/bin/echo toto',
    image='centos:latest',
    network_mode='bridge',
    do_xcom_push=True,
    task_id='docker_op_tester',
    dag=dag,
)

give

[2021-07-27 19:38:44,647] {docker.py:246} INFO - Starting docker container from image centos:latest
[2021-07-27 19:38:44,682] {docker.py:256} WARNING - Using remote engine or docker-in-docker and mounting temporary volume from host is not supported. Falling back to `mount_tmp_dir=False` mode. You can set `mount_tmp_dir` parameter to False to disable mounting and remove the warning
[2021-07-27 19:38:46,975] {docker.py:307} INFO - toto
[2021-07-27 19:38:47,664] {taskinstance.py:1204} INFO - Marking task as SUCCESS. dag_id=docker_sample, task_id=docker_op_tester, execution_date=20210727T150000, start_date=20210727T193839, end_date=20210727T193847
[2021-07-27 19:38:47,770] {taskinstance.py:1265} INFO - 0 downstream tasks scheduled from follow-on schedule check
[2021-07-27 19:38:47,815] {local_task_job.py:149} INFO - Task exited with return code 0

and

t3 = DockerOperator(
    api_version='auto',
    docker_url="unix://var/run/docker.sock",
    command='/bin/echo toto',
    image='centos:latest',
    network_mode='bridge',
    mount_tmp_dir=False,
    do_xcom_push=True,
    task_id='docker_op_tester',
    dag=dag,
)

give

[2021-07-27 19:39:27,176] {docker.py:246} INFO - Starting docker container from image centos:latest
[2021-07-27 19:39:28,091] {docker.py:307} INFO - toto
[2021-07-27 19:39:28,383] {taskinstance.py:1204} INFO - Marking task as SUCCESS. dag_id=docker_sample, task_id=docker_op_tester, execution_date=20210727T180000, start_date=20210727T193926, end_date=20210727T193928
[2021-07-27 19:39:28,424] {taskinstance.py:1265} INFO - 0 downstream tasks scheduled from follow-on schedule check
[2021-07-27 19:39:28,464] {local_task_job.py:149} INFO - Task exited with return code 0

and in all case a XCOM

key : return_value
value : toto

@levyitay
Copy link
Contributor

qubole: 2.0.0rc2 tested and working

@josh-fell
Copy link
Contributor

Contributions for Google, Amazon, and Salesforce providers look good!

@pmalafosse
Copy link
Contributor

amazon: 2.1.0rc2:

I just tried it and ECSOperator is working as expected

@pmalafosse
Copy link
Contributor

This one is fine too:

@Goodkat
Copy link
Contributor

Goodkat commented Jul 28, 2021

Tested again the odbc connection, it works as expected with the boolean parameters:
#15016 (comment)

@potiuk
Copy link
Member Author

potiuk commented Jul 28, 2021

Getting very good progress on testing with this release :) . Thanks a bunch for those! looking forward for more quick tests :)

@pavelhlushchanka
Copy link
Contributor

#16820 is also good to go

@baolsen
Copy link
Contributor

baolsen commented Jul 28, 2021 via email

@ayushchauhan0811
Copy link
Contributor

#17081 is also good to go

@iostreamdoth
Copy link
Contributor

fix: get_cluster call project_id as self.project_id (#17075) looks good.

@infused-kim
Copy link
Contributor

Google ads #17160 is working fine, but I am the one who submitted the PR and it would be good to get it tested by a few more people.

@potiuk
Copy link
Member Author

potiuk commented Jul 29, 2021

Google ads #17160 is working fine, but I am the one who submitted the PR and it would be good to get it tested by a few more people.

@Overbryd and @rv-irosario - since this was addressing your issues #17113 and #17111 - could you please also take a look if https://pypi.org/project/apache-airflow-providers-google/5.0.0rc2 solves the ads compatibility ?

@potiuk
Copy link
Member Author

potiuk commented Jul 30, 2021

@oyarushe
Copy link
Contributor

I'm trying to check #16521, but the fix was provided in DbApiHook, which is a general class and I suppose will be released with Airflow itself.
So I think we need to release the Postgres provider after Airflow.

@potiuk
Copy link
Member Author

potiuk commented Jul 30, 2021

So I think we need to release the Postgres provider after Airflow.

Ah yeah. This is backwards-compatible change, so it's ok to release Postgres now, but indeed testing will be possible only with new Airflow released (and it i's a new feature so it should be only released in 2.2. Good that you raised it!

@ciancolo
Copy link
Contributor

Tested #16365 and #17125 on my local Tableau Server and they works.

@potiuk
Copy link
Member Author

potiuk commented Aug 2, 2021

Thanks everyone who tested! This is really great help for the community. The vote has passed and I am proceeding to release the providers!

@potiuk potiuk closed this as completed Aug 2, 2021
@potiuk potiuk added kind:meta High-level information important to the community and removed kind:bug This is a clearly a bug labels Aug 2, 2021
@potiuk potiuk added the testing status Status of testing releases label Feb 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:meta High-level information important to the community testing status Status of testing releases
Projects
None yet
Development

No branches or pull requests