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

Remove files #629

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Remove files #629

wants to merge 2 commits into from

Conversation

ConnorNelson
Copy link
Member

No description provided.

@ConnorNelson ConnorNelson changed the title Remove windows/challenge image files Remove files Jan 3, 2025
@ConnorNelson ConnorNelson force-pushed the cleanup/workspace-windows branch from 9201a43 to 9f75e8b Compare January 3, 2025 21:09
@ConnorNelson
Copy link
Member Author

This change is blocked by welcome-dojo and example-dojo not depending on challenge-legacy.

@zardus
Copy link
Contributor

zardus commented Jan 3, 2025

Is anything stopping us from making them depend on -legacy?

@ConnorNelson
Copy link
Member Author

We want to make it not require challenge-legacy so that CI doesn't depend on challenge-legacy. Otherwise, it will be slow (pulling a big image), and we may not even have the storage to do it. It shouldn't be too difficult, I think it'll basically just be ubuntu:24.04 + exec-suid and python:latest + exec-suid.

@zardus
Copy link
Contributor

zardus commented Jan 3, 2025

Ah, okay. So we need to:

  • make pwncollege/challenge-exec-suid
  • translate all the challenges to use it
  • test...

@ConnorNelson ConnorNelson force-pushed the cleanup/workspace-windows branch from 9f75e8b to 082eda6 Compare January 7, 2025 17:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants