[BEAM-13218] Re-enable PubSubIntegrationTest.test_streaming_with_attributes

[yeandy]

PubSubIntegrationTest. test_streaming_with_attributes has been flaky, and was previously sickbayed in #16251. We've identified the issue to be related to Dataflow service, so are now reenabling this test.

---

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

• Choose reviewer(s) and mention them in a comment (R: @username).
• Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
• Update CHANGES.md with noteworthy changes.
• If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

ValidatesRunner compliance status (on master branch)

Lang	ULR	Dataflow	Flink	Samza	Spark	Twister2
Go	---					---
Java
Python	---					---
XLang						---

Examples testing status on various runners

Lang	ULR	Dataflow	Flink	Samza	Spark	Twister2
Go	---	---	---	---	---	---	---
Java	---		---	---	---	---	---
Python	---	---	---	---	---	---	---
XLang	---	---	---	---	---	---	---

Post-Commit SDK/Transform Integration Tests Status (on master branch)

Go	Java	Python

Pre-Commit Tests Status (on master branch)

---	Java	Python	Go	Website	Whitespace	Typescript
Non-portable
Portable	---			---	---	---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

[yeandy]

Run Python 3.6 PostCommit

[yeandy]

Run Python 3.7 PostCommit

[yeandy]

Run Python 3.8 PostCommit

[codecov]

Codecov Report

Merging #16497 (3e1096f) into master (9794fb4) will increase coverage by 9.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #16497      +/-   ##
==========================================
+ Coverage   74.60%   83.64%   +9.03%     
==========================================
  Files         654      452     -202     
  Lines       82029    62168   -19861     
==========================================
- Hits        61201    52000    -9201     
+ Misses      19841    10168    -9673     
+ Partials      987        0     -987     

Impacted Files	Coverage Δ
...pache_beam/dataframe/pandas_top_level_functions.py	90.52% <0.00%> (-3.16%)	⬇️
sdks/python/apache_beam/internal/metrics/metric.py	90.00% <0.00%> (-1.00%)	⬇️
...hon/apache_beam/runners/worker/bundle_processor.py	93.14% <0.00%> (-0.50%)	⬇️
...ks/python/apache_beam/runners/worker/sdk_worker.py	88.90% <0.00%> (-0.16%)	⬇️
sdks/go/pkg/beam/util/harnessopts/cache.go
sdks/go/pkg/beam/io/bigqueryio/bigquery.go
sdks/go/pkg/beam/artifact/stage.go
sdks/go/pkg/beam/core/runtime/coderx/string.go
sdks/go/pkg/beam/core/runtime/graphx/user.go
sdks/go/pkg/beam/core/runtime/exec/pcollection.go
... and 199 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9794fb4...3e1096f. Read the comment docs.

[yeandy]

R: @TheNeuralBit
Looks like even with the increased timeout, test_streaming_with_attributes is having the original timestamp attributes error that you originally reported. Looking into it now.

[TheNeuralBit]

Should we revert this PR?

[yeandy]

Let's keep it up for now so we can test timeouts when we resolve the timestamp issue.

[TheNeuralBit]

Sounds good!

[aaltay]

Do we still need this PR?

[yeandy]

I think we can repurpose it to unskip the test_streaming_with_attributes test.

[yeandy]

Run Python 3.8 PostCommit

[yeandy]

Run Python 3.7 PostCommit

[yeandy]

Run Python 3.7 PostCommit

[yeandy]

Run Python 3.7 PostCommit

[yeandy]

Run PythonDocs PreCommit

[yeandy]

Run Python 3.7 PostCommit

[yeandy]

Run Python 3.7 PostCommit

[yeandy]

Run Python 3.7 PostCommit

[yeandy]

Run Python 3.7 PostCommit

[yeandy]

Run Python 3.7 PostCommit

[yeandy]

Run Python 3.7 PostCommit

[yeandy]

Run Python 3.7 PostCommit

[yeandy]

Run Python 3.8 PostCommit

[TheNeuralBit]

Run Python 3.7 PostCommit

[TheNeuralBit]

Run Python 3.8 PostCommit

[TheNeuralBit]

Passed in Python 3.8 PostCommit! 🎉

Python 3.7 PostCommit failure looks unrelated

[yeandy]

Finally! Thanks for the retest and merge @TheNeuralBit!