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

Automate updating user images in profile list - temporary solution #1590

Closed
2 tasks
GeorgianaElena opened this issue Aug 3, 2022 · 3 comments
Closed
2 tasks
Assignees

Comments

@GeorgianaElena
Copy link
Member

GeorgianaElena commented Aug 3, 2022

Context

Currently, the configurator doesn't support to update user images if multiple profiles are used with multiple images.
So the current workflow in this cases, is for 2i2c engineers to update user images whenever they need changing. Alternatively, someone from the community we're serving can update it themselves by opening up PRs against the infrastructure repo.

But this will probably change in the future -> you can track progress of this feature here #1253 🚀

Proposal

Until a new configurator version is released, another approach could be to use https://github.com/sgibson91/bump-jhub-image-action by creating a new GitHub workflow that tracks the images listed in the hub config and updates them whenever new versions are released

We are piloting this idea for the OceanHackWeek 2022 event -> see #1601

Updates and actions

If the above is considered a valid approach and it turns out useful for the event, then we should:

@GeorgianaElena
Copy link
Member Author

Based on the feedback we got in #1576 (comment), this wasn't find extremely useful, because it lacked the mechanism of testing fast that a image is working and distribute it asap to the users (""It would be really nice to have a faster way to test and update the images (for when I get told about packages just before they are used for a tutorial), that didn't require 2i2c's intervention.")

Some really interesting suggestions that I believe are worth trying to solve the issue above were:

  • provide a test hub without profiles, so that we could use configurator to test images in a live hub ➡️ this could be our demo hub perhaps?

  • a bot or github action that could merge PRs from community representatives to image fields

@sgibson91
Copy link
Member

I personally would focus more on the first option for a temporary workaround. Option 2 would require maintaining a list of the GitHub handles of all Comm. Reps. who might open an image-based PR. (What could go wrong with auto-merging PRs? How would we validate that the PR was only changing images and not any critical infrastructure?) It would involve more dev work that I think should be dedicated towards working on the configurator to be compatible with profile options, which is the long-term solution here.

@GeorgianaElena
Copy link
Member Author

Superseed by 2i2c-org/features#26 and #3095

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done 🎉
Development

No branches or pull requests

2 participants