Skip to content

Conversation

abrennan89
Copy link
Contributor

@abrennan89 abrennan89 commented May 17, 2021

Applies for 4.8+

Should be rebased on top of #32453 when it is merged.

@openshift-ci openshift-ci bot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels May 17, 2021
@netlify
Copy link

netlify bot commented May 17, 2021

Deploy preview for osdocs ready!

Built with commit 034c60e028f2c6d84321d5418a824ee438fee621

https://deploy-preview-32585--osdocs.netlify.app

@netlify
Copy link

netlify bot commented May 17, 2021

✔️ Deploy Preview for osdocs ready!

🔨 Explore the source changes: e078a9f

🔍 Inspect the deploy log: https://app.netlify.com/sites/osdocs/deploys/60ae6b33c8f13f0007324c9b

😎 Browse the preview: https://deploy-preview-32585--osdocs.netlify.app

@openshift-ci openshift-ci bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels May 18, 2021
@abrennan89
Copy link
Contributor Author

@invincibleJai
Copy link
Member

Thanks @abrennan89 looks good to me. Have added one comment, PTAL if it make sense

Copy link
Contributor

@Preeticp Preeticp left a comment

Choose a reason for hiding this comment

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

A minor nit, otherwise looks great to me. Thank you @abrennan89

@openshift-ci openshift-ci bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 20, 2021
@openshift-ci openshift-ci bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 20, 2021
@abrennan89 abrennan89 force-pushed the eventsourcefunc branch 2 times, most recently from 3b98ea3 to b66381b Compare May 20, 2021 20:02
Copy link
Contributor

@jrangelramos jrangelramos left a comment

Choose a reason for hiding this comment

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

Hello. Just a note about the procedure to create the sink. It look a little different for me. Better double check this point.

@abrennan89 abrennan89 force-pushed the eventsourcefunc branch 3 times, most recently from 99eef3c to dd1e457 Compare May 24, 2021 17:32
@abrennan89 abrennan89 added this to the Future Release milestone May 25, 2021
@abrennan89 abrennan89 added branch/enterprise-4.8 dev-tools Label for all Odo/Pipelines/Helm/Developer Console/Perspective PRs serverless Label for all Serverless PRs labels May 25, 2021
@jrangelramos
Copy link
Contributor

QE Approved

Copy link
Contributor

@jeana-redhat jeana-redhat left a comment

Choose a reason for hiding this comment

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

Just a couple nits and one suggestion for consideration. LGTM :)

@jeana-redhat jeana-redhat added the peer-review-done Signifies that the peer review team has reviewed this PR label May 26, 2021
@abrennan89 abrennan89 changed the title [WIP][srvls][RHDEVDOCS-2749] Add functions with eventing docs [srvls][RHDEVDOCS-2749] Add functions with eventing docs May 26, 2021
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 26, 2021
@abrennan89 abrennan89 merged commit 2003f25 into openshift:master May 26, 2021
@abrennan89
Copy link
Contributor Author

/cherrypick enterprise-4.8

@openshift-cherrypick-robot

@abrennan89: new pull request created: #32862

In response to this:

/cherrypick enterprise-4.8

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@abrennan89 abrennan89 deleted the eventsourcefunc branch June 3, 2021 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch/enterprise-4.8 dev-tools Label for all Odo/Pipelines/Helm/Developer Console/Perspective PRs peer-review-done Signifies that the peer review team has reviewed this PR serverless Label for all Serverless PRs size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants