Skip to content

Ignore uwsgi.ini file (created by Tutor)#32268

Closed
bradenmacdonald wants to merge 1 commit intoopenedx:masterfrom
open-craft:ignore-uwsgi-ini
Closed

Ignore uwsgi.ini file (created by Tutor)#32268
bradenmacdonald wants to merge 1 commit intoopenedx:masterfrom
open-craft:ignore-uwsgi-ini

Conversation

@bradenmacdonald
Copy link
Contributor

Description

When running Tutor in dev mode (or other modes) with a bind-mounted edx-platform, I keep seeing a uwsgi.ini file show up as a modified file in the git status. I believe it is created by Tutor and gets re-created if deleted, so seems like it should be ignored.

Supporting information

I couldn't find prior discussions of this - is it just me who notices it?

Testing instructions

Run a Tutor devstack with bind-mounted edx-platform, make sure git shows no changes to edx-platform.

Deadline

None

@bradenmacdonald bradenmacdonald requested a review from regisb May 18, 2023 23:54
@openedx-webhooks openedx-webhooks added open-source-contribution PR author is not from Axim or 2U core committer labels May 18, 2023
@openedx-webhooks
Copy link

Thanks for the pull request, @bradenmacdonald!

As a core committer in this repo, you can merge this once the pull request is approved per the core committer reviewer requirements and according to the agreement with your edX Champion.


# Visual Studio Code
.vscode

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I moved this into the existing "Editor and IDE artifacts" section above.

@regisb
Copy link
Contributor

regisb commented May 30, 2023

Yes, I just faced the same issue. I'm wondering whether we should put uwsgi.ini in a different folder inside the openedx container? The /openedx/ folder is a bit of a mess in that container, and it would be great if we could put some order in there.

My opinion is that is an issue that should be fixed in tutor, not edx-platform, so I opened this issue: overhangio/tutor#846

@bradenmacdonald
Copy link
Contributor Author

Makes sense! I'll close this.

@openedx-webhooks
Copy link

@bradenmacdonald Even though your pull request wasn’t merged, please take a moment to answer a two question survey so we can improve your experience in the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core committer open-source-contribution PR author is not from Axim or 2U

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants