-
-
Notifications
You must be signed in to change notification settings - Fork 439
Mark rc builds of arcticdb post 5.1.2 as broken #1468
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
base: main
Are you sure you want to change the base?
Conversation
|
Sames as in #1467 Also: You should not be uploading RC builds to the main label. |
|
RC builds are published on |
|
No, for that the mark broken automation doesn't work AFAIK. |
I can mark them broken manually if you want (that's the only option for non- |
|
It would be helpful. 🙏 All the rc builds in |
|
Oh wow, I did not see the number of builds involved 🤯 This is a non-batch process (needs to be done manually in UI), so I'm not going to click through 200+ packages. If someone has (or creates) a python script using For now my suggestion would be to just rebuild all maintenance branches, and rely on the solver picking up the newest builds. I don't understand though why you need to draw such a wide circle (rc-channel aside) - we tightened the run-exports of all packages on main to |
Yes, this is what we have been and are doing; FYI, see conda-forge/arcticdb-feedstock#426.
OK, I thought this was done for all platforms but it only has been the case for Windows, but users reported problems on Linux as well. Could we also apply this repodata patch to |
Please open two separate PRs that include osx in the |
|
I am not sure I understood. Do you mean to open (up to 10) PRs for elements of:
? |
|
Two PRs. One that extends the subdir condition for the yaml in conda-forge/conda-forge-repodata-patches-feedstock#984 from windows to windows+osx, and then a second one that enables the patch on linux too by deleting the subdir condition. You can also switch the order of linux and osx if you want. Repodata patches are against the current state of the repodata. Windows is already patched (hence why we keep patching it), now we need to add osx and linux in two waves, because there's a practical limit of ~5000 packages being patched in one go (second round a few hours later) |
|
OK, this is much clearer, thank you. Edit: done in conda-forge/conda-forge-repodata-patches-feedstock#988. |
Guidelines for marking packages as broken:
instead of marking packages as broken. This alternative workflow makes environments more reproducible.
not technically broken and should not be marked as such.
but should be patched in the repo data and be marked unbroken later.
the maintainers only, we can allow packages to be marked broken more liberally.
conda-forge/core) try to make a decision on these requests within 24 hours.What will happen when a package is marked broken?
brokenlabel to the package. Themainlabel will remain on the package and this is normal.anaconda.orgCDN picks up the new patches, you will no longer be able to install the package from themainchannel.Checklist:
I want to mark a package as broken (or not broken):
I want to archive a feedstock:
I want to request (or revoke) access to an opt-in CI resource:
I want to copy an artifact following CFEP-3:
I want to add a package output to a feedstock:
Context
See conda-forge/arcticdb-feedstock#426.
Details for rc
These are just release candidate versions of the affected versions, so it is safe to remove them.