openedx / public-engineering

General public issue repository for the Open edX engineering community
3 stars 2 forks source link

[DEPR] Devstack #247

Open kdmccormick opened 2 years ago

kdmccormick commented 2 years ago

Timeline

Replacement

The replacement for Devstack is Tutor. Tutor is "the Docker-based Open edX distribution designed for peace of mind". It is both a deployment tool and a development environment. Tutor has been the only community-support production deployment method since Maple, although the Open edX docs and website do not officially recommend it as a development environment yet.

Rationale

Devstack has several issues:

Tutor shows promise in many of these areas:

For areas where Tutor could be improved, there is an ongoing Tutor Adoption Initiative which aims to support the transition from Devstack to Tutor through a mix of education, plugin development, and core Tutor improvements. Within the initiative roadmap, the "Dev Quality-of-Life" epic contains stories that should ideally close any existing gaps between Tutor and Devstack.

Migration

Many developers can start using Tutor now, and many already are). Particularly, developers whose projects are encompassed by the LMS, CMS, MFEs, and IDAs covered by existing official plugins can generally transition their workflows over to Tutor. On the other hand, for teams that run IDAs with less community adoption (notably, teams at 2U), several new Tutor plugins will be needed in order for Tutor to fully replace Devstack.

There are no plans to help users migrate data from a Devstack instance to a Tutor instance. We recommend that folks start fresh when switching from Devstack to Tutor.

Deprecation & Removal

Deprecation steps

### Removal steps
- [x] Archive https://github.com/openedx/devstack 
- [ ] Update OEP-10, which still mentions "devstack" as a supported distro
- [x] https://github.com/openedx/edx-platform/issues/34430
- [ ] Scour our docs for other references to devstack; update them to Tutor.
- [ ] Delete the various `devstack.py` and `development.py` settings files unless otherwise depended upon.
- [ ] Delete the various `docker-compose.yml` files and Makefile targets that some IDAs (eg enterprise-catalog) use in order to integrate with Devstack.
- [ ] Delete the `docker/` role (incl. Dockerfiles) from github.com/openedx/configuration.
- [ ] Follow up: Decide whether or not to keep the Dockerfiles that exist in various repos, which aren't used in Tutor.
kdmccormick commented 2 years ago

Relates to https://github.com/openedx/build-test-release-wg/issues/138

dianakhuang commented 1 year ago

Since this is primarily a 2U project right now, we are planning on announcing this and gathering feedback.

2U is still planning on using this for the near to medium term until we're confident we can migrate successfully onto Tutor, but we can move it out of the openedx GitHub org to minimize confusion.

kdmccormick commented 1 year ago

In general, sounds great!

Only thing is... MFE development currently sucks in Tutor >.< So nearly all frontend devs still use Devstack when they're trying to change MFEs.

Brian Smith is actively working on rectifying this, with promising results. Would you be willing to delay the deprecation further until Tutor has MFE dev sorted out?

dianakhuang commented 1 year ago

Yup! I think we're still trying to understand the requirements needed here. I'll push out the acceptance date for another few months.

dianakhuang commented 1 year ago

Discuss posting: https://discuss.openedx.org/t/deprecation-devstack/10703

kdmccormick commented 1 year ago

@dianakhuang So, I know I had originally been a proponent of deprecating both configuration and devstack "into the edx org", under the understanding that if only 2U is using some parts of the project, then it's easier for both sides if we transfer those parts over to 2U. Since then, I've talked about this with my team at Axim, and come to understand that the situation is more nuanced than that.

The code in the openedx org is built from a mix of Axim-owned contributions and community-contributor-owned contributions (licensed to Axim under the CLA). We hold the code in this org for the good of all current & former Open edX contributors and users, and as the project's stewards we have a mandate to maintain ownership and ensure availability of all its repositories, even unsupported repositories.

Our procedure of archiving repos by transferring to the openedx-unsupported org lets us mark a repo as "unsupported" while still guaranteeing the repo's availability and keeping its ownership clear. So, if/when devstack and configuration are deprecated, we'll need to move them there.

dianakhuang commented 1 year ago

@kdmccormick that sounds reasonable! 2U/edX.org should be able to then clone the repositories for our own use case after they are put into unsupported, then?

Obviously, this is our first stab and understanding how this process is going to work going forward.

kdmccormick commented 1 year ago

Yes, you could fork from openedx-supported after that full deprecation & archival process.

Even in that case, though, I'd be curious what we could do to help keep 2U involved with community tooling.

Would you be forking devstack & configuration as a temporary stop-gap until you're able to transition off of them, or with the intent of using them indefinitely?

Obviously, this is our first stab and understanding how this process is going to work going forward.

Same here :) learning as we go!

dianakhuang commented 8 months ago

We are going to block acceptance because devstack is still used heavily by people who need to do development on MFEs outside of 2U.

@arbrandes @kdmccormick could you add more context on this?

kdmccormick commented 8 months ago

@arbrandes , I know you mentioned that RaccoonGang are heavy users of Devstack. I wonder if they'd be willing to take on maintenance. If not, and if nobody else is willing to, then we should probably let this DEPR move forward and help MFE developers move towards Tutor.

feanil commented 7 months ago

@dianakhuang given recent changes at 2U, is 2U still willing to maintain this repository? Or should we complete the deprecation and remove this from the openedx org?

dianakhuang commented 7 months ago

@feanil 2U is still willing to maintain this repository for now. We would like to keep in in the openedx org for now if that's okay with everyone else.

feanil commented 7 months ago

I think this is fine for now but since we're trying to move away from devstack for the community, at some point it will make senses to move it out. In the meantime, I'd like to mark this ticket and the repo as deprecated and make sure the relevant messaging exists in the README for this. What do you think?

dianakhuang commented 7 months ago

Yup, let's add some text to the README and move this ticket to the deprecated column.

feanil commented 7 months ago

Cool, is that something you can take on as maintainers?

dianakhuang commented 7 months ago

Yup, I'll try to get on it today and spread the info around 2U so no one freaks out.

feanil commented 6 months ago

We should move this back to Proposed and re-communicate that 2U wants to stop maintaining this as a part of Open edX. Either someone else needs to maintain this for the community or we can archive and 2U and any other interested parties can fork it.

kdmccormick commented 6 months ago

Re-communication happened here: https://discuss.openedx.org/t/deprecation-removal-devstack-update/12384

Aaaand it's archived: https://github.com/openedx-unsupported/devstack

I moved this ticket to public-engineering, since we have a few more clean-up steps before we can close it out.

kdmccormick commented 6 months ago

With the devstack repository archived, the obvious next question is: When will devstack.py, env.devstack, and any other devstack-specific files be removed from upstream repositories? I realize that removing those files will break any forks of devstack, unless the forks adapt by loading the files from somewhere new.

I propose that the files should remain in-place until the Sumac branch cut on October 9th, after which they are fair game for removal. Happy to discuss different dates, a phased removal, or something else. Let me know what you think @dianakhuang @regisb @cmltaWt0 @felipemontoya @robrap @feanil .

kdmccormick commented 6 months ago

^ @brian-smith-tcril @arbrandes @adamstankiewicz , pinging you guys on behalf of the frontend :)

feanil commented 6 months ago

October 9th seems like plenty far int the future, but I think we should put deprecation warnings in those files now with the agreed upon dates so people who touch them have some chance of knowing that they're gonna disappear.

robrap commented 5 months ago

Thanks @kdmccormick. I'm hopeful we can work that into our plan at this point, but tagging @jristau1984 and @spencertiberi, who need to be involved in any deadline agreements. Thanks.

kdmccormick commented 5 months ago

I'd like to settle on a timeline for removal of these auxillary files. If there any objections or alternative suggestions to Oct 9 for the "OK to start removing date", please raise them by the end of next week (Apr 5).

jristau1984 commented 5 months ago

At this time, I have no concerns with having the removal work begin after the Sumac cut, currently scheduled on Oct 9.

Can this date be added to the timeline section of the issue above? Thanks!

robrap commented 5 months ago

@jristau1984: Just to ensure you are aware, this is not just busy work, but requires designing a plan for how we can accomplish this.

kdmccormick commented 5 months ago

Can this date be added to the timeline section of the issue above? Thanks!

Done.

When I get a chance, I'll start a draft PR of these changes for edx-platform, so that devstack users can get a sense of which files will be going away.

kdmccormick commented 4 months ago

Deprecation warnings for devstack settings files in edx-platform: https://github.com/openedx/edx-platform/pull/34795

iamsobanjaved commented 4 days ago

Hi @kdmccormick

I’m working on the 2U side to move Devstack-related settings out of the Open edX repositories and into a different location within the edX GitHub organization. While doing this I was wondering how Tutor has done the development environment setup, I noticed that including the LMS, CMS, and other plugins like discovery and credentials, relies heavily on the devstack.py or development.py modules. These services use devstack.py as a base and then modify a few settings variables.

Is there currently any effort underway for the Tutor to stop depending on these files? Or, as mentioned in the Removal Steps, should we refrain from deleting them if they are still in use?

Delete the various devstack.py and development.py settings files unless otherwise depended upon.

Tutor using devstack files as a base:

regisb commented 4 days ago

I hadn't thought about the fact that getting rid of devstack meant removing devstack.py. Do we all agree that we still need platform-agnostic development settings, though? I would expect that devstack.py would be replaced by development.py settings files that provide good defaults for development. If we do, then we can configure tutor to point to these new development modules instead of devstack.py.

kdmccormick commented 4 days ago

@iamsobanjaved thank you for pointing this out, I had not realized that Tutor was based on those files as well.

@regisb I agree, we do need platform-agnostic development base settings.

I can work to develop a platform-agnostic development.py settings file for both CMS and LMS, which both Tutor and Devstack can use. Once that is merged, I can wait 30 days for Devstack and Tutor to switch over to development.py before we remove devstack.py. Does that sound reasonable?

Also, @regisb , Axim has bought local.openedx.io and we can point it (and its subdomains) at 127.0.0.1. Do you have any objection to me using local.openedx.io as the base URL in the development settings file? Tutor could either use it as well, or it could continue to override the base URL to local.edly.io.

kdmccormick commented 4 days ago

Clarification: development.py would be a suitable base for Devstack settings, but it would not work out-of-the-box for Devstack because it would not include the specific ports and hostnames that Devstack needs, and it may not have the exact same toggles that Devstack users expect. So, Devstack would still need a custom settings file for LMS and CMS. In the meantime, I recommend making sure that Devstack has the capability to define a custom DJANGO_SETTINGS_MODULE.

iamsobanjaved commented 4 days ago

Yeah that's what exactly I wanted, a suitable base for dev settings that can be further extended. Also, we are planning to use YAML to override devstack-related settings and It would be nice to have a development.py that can be extended/overridden by a YAML file.

kdmccormick commented 4 days ago

Great. In order to keep development.py straightforward, I am inclined not to load YAML into it, but I can provide a YAML snippet that you could add to Devstack's custom settings module in order to retain YAML support.

regisb commented 3 days ago

Thank you! I'm looking forward to a future with streamlined development settings.

On loading YAML files: I'm also not too inclined on doing that... but I'm sure you'll be able to do that within devstack.py.

It's awesome that we can now switch to local.openedx.io! As we learned previously, the switch is actually very easy. I'll add that to our roadmap (here).