Skip to content

Add namespaceId to the activation record as well to enable correlation.#3073

Merged
cbickel merged 1 commit intoapache:masterfrom
markusthoemmes:file-fix
Dec 7, 2017
Merged

Add namespaceId to the activation record as well to enable correlation.#3073
cbickel merged 1 commit intoapache:masterfrom
markusthoemmes:file-fix

Conversation

@markusthoemmes
Copy link
Copy Markdown
Contributor

Correlation should happen based on a unique identifier.

@markusthoemmes markusthoemmes added logging review Review for this PR has been requested and yet needs to be done. labels Dec 7, 2017
Copy link
Copy Markdown
Contributor

@cbickel cbickel left a comment

Choose a reason for hiding this comment

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

LGTM

@markusthoemmes
Copy link
Copy Markdown
Contributor Author

PG3 1456 🔵

@cbickel cbickel merged commit 94200d4 into apache:master Dec 7, 2017
@markusthoemmes markusthoemmes deleted the file-fix branch February 11, 2018 10:48
BillZong pushed a commit to BillZong/openwhisk that referenced this pull request Nov 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

logging review Review for this PR has been requested and yet needs to be done.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants