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

Determine process for getting OEP-52 to Accepted #415

Open
rgraber opened this issue Nov 28, 2022 · 4 comments
Open

Determine process for getting OEP-52 to Accepted #415

rgraber opened this issue Nov 28, 2022 · 4 comments
Labels
help wanted Ready to be picked up by anyone in the community

Comments

@rgraber
Copy link
Contributor

rgraber commented Nov 28, 2022

OEP-52 (Event bus architecture https://open-edx-proposals.readthedocs.io/en/latest/architectural-decisions/oep-0052-arch-event-bus-architecture.html) was initially marked as Provisional in line with the definition of Provisional outlined in OEP-1: https://open-edx-proposals.readthedocs.io/en/latest/processes/oep-0001.html#provisional

There is now a working implementation of the event bus being used in production on edx.org, but it is unclear if this is enough to satisfy the requirement for "viable reference examples and platform adoption" to move the OEP back into Under Review with an eye towards getting it accepted. In particular, it's unclear how far we need to be on the roadmap (openedx/platform-roadmap#28). We have completed the Happy- and Sad-Path epics and are fairly near completion on the Production-ready event.

If we think OEP-52 is already ready to be properly reviewed, I think the proper step would be to update this PR: https://github.com/openedx/open-edx-proposals/pull/414/files to make the status Under Review and add a new acceptance period.

Would appreciate any thoughts on this.

@robrap
Copy link
Contributor

robrap commented Nov 28, 2022

I'm wondering if we could announce this issue with a 1-2 week review period, where we decide that OEP-52 is either accepted, or we nail down what is required to bring it to accepted.
@blarghmatey - as the original arbiter - would you mind helping with this process? At a minimum, do you have thoughts on how we can get to acceptance? Thank you!

@rgraber
Copy link
Contributor Author

rgraber commented Dec 2, 2022

Update: https://github.com/openedx/open-edx-proposals/pull/414/files has since been merged with no change to the status. We can have another PR to either update the follow-up work with the path to Acceptance or just change the status to Under Review.

@sarina
Copy link
Contributor

sarina commented May 17, 2024

@rgraber @feanil @robrap OEP-52 is marked as Accepted but has a warning box that it's Provisional. This should be fixed, which one is it?

image

@sarina sarina added the waiting on author PR author needs to resolve review requests, answer questions, fix tests, etc. label May 17, 2024
@robrap
Copy link
Contributor

robrap commented May 17, 2024

The warning is outdated and can be removed.

@sarina sarina added help wanted Ready to be picked up by anyone in the community and removed waiting on author PR author needs to resolve review requests, answer questions, fix tests, etc. labels May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Ready to be picked up by anyone in the community
Projects
None yet
Development

No branches or pull requests

3 participants