have GcsComputeLogManager implement CapturedLogManager interface
Needs ReviewPublic
Actions

Authored by • prha on Jul 12 2021, 9:58 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Aug 9, 3:53 AM

	Unknown Object (File)
	Sat, Aug 6, 7:56 PM

	Unknown Object (File)
	Fri, Jul 15, 3:21 AM

	Unknown Object (File)
	Fri, Jul 15, 3:21 AM

	Unknown Object (File)
	Fri, Jul 15, 3:21 AM

	Unknown Object (File)
	Jul 10 2022, 7:01 PM

	Unknown Object (File)
	Jul 7 2022, 3:19 PM

	Unknown Object (File)
	Jun 24 2022, 12:54 PM

View All 18 Files

Subscribers

None

Details

Reviewers

alangenfeld
• max

Summary

Depends on D8385, D8840

Test Plan

Diff Detail

Repository

R1 dagster

Lint

Lint Passed

Unit

No Test Coverage

Event Timeline

• prha created this revision.Jul 12 2021, 9:58 PM

Harbormaster completed remote builds in B33597: Diff 41474.Jul 12 2021, 10:42 PM

• prha requested review of this revision.Jul 12 2021, 10:42 PM

rebase

Harbormaster completed remote builds in B33749: Diff 41672.Jul 15 2021, 5:39 PM

• max resigned from this revision.Jul 21 2021, 11:44 PM

Revision Contents
Changeset List

Path

Size

python_modules/

libraries/

dagster-gcp/

dagster_gcp/

gcs/

compute_log_manager.py

85 lines

dagster_gcp_tests/

gcs_tests/

test_compute_log_manager.py

13 lines

Diff 41672

View Options

python_modules/libraries/dagster-gcp/dagster_gcp/gcs/compute_log_manager.py