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

Add PodAggregator for use by resource awaiters #785

Merged
merged 1 commit into from
Sep 18, 2019

Conversation

lblackstone
Copy link
Member

Proposed changes

The PodAggregator encapsulates the logic required to aggregate
warning/error messages from Pods related to a parent resource
such as a Deployment or Job. This will make it easier to collect
relevant intermediate status in a consistent way.

Related issues (optional)

@lblackstone lblackstone force-pushed the lblackstone/pod-aggregator branch 2 times, most recently from d711a28 to a9cc558 Compare September 9, 2019 16:38
The PodAggregator encapsulates the logic required to aggregate
warning/error messages from Pods related to a parent resource
such as a Deployment or Job. This will make it easier to collect
relevant intermediate status in a consistent way.
@lblackstone lblackstone marked this pull request as ready for review September 9, 2019 20:52
Copy link
Contributor

@metral metral left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. The readability on this code was very easy to follow, kudos!

@lblackstone lblackstone merged commit 6a64359 into master Sep 18, 2019
@pulumi-bot pulumi-bot deleted the lblackstone/pod-aggregator branch September 18, 2019 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants