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 999999-SNAPSHOT version of Remoting from Artifactory #4173

Closed
basil opened this issue Jul 10, 2024 · 10 comments
Closed

Remove 999999-SNAPSHOT version of Remoting from Artifactory #4173

basil opened this issue Jul 10, 2024 · 10 comments
Assignees

Comments

@basil
Copy link
Collaborator

basil commented Jul 10, 2024

Service(s)

Artifactory

Summary

Core isn't getting Dependabot PRs for Remoting anymore, forcing me to manually update it as in jenkinsci/jenkins#9412, or is getting nonsense PRs like jenkinsci/jenkins#9448. I suspect this is yet another manifestation of #3919. Looks like the current content in https://repo.jenkins-ci.org/artifactory/snapshots/org/jenkins-ci/main/remoting/999999-SNAPSHOT/ is a set of snapshots uploaded by @jtnord on May 15, presumably to test jenkinsci/remoting#741, jenkinsci/remoting#742, / jenkinsci/remoting#743, all of which have now been delivered. Assuming the snapshots in https://repo.jenkins-ci.org/artifactory/snapshots/org/jenkins-ci/main/remoting/999999-SNAPSHOT/ are no longer needed for testing, can we delete them from the server as a workaround for #3919? Ideally the root cause of the problem would be addressed, but I believe the above action would at least restore the status quo and re-enable automated Dependabot bumps to core.

Reproduction steps

No response

@basil basil added the triage Incoming issues that need review label Jul 10, 2024
@jtnord
Copy link

jtnord commented Jul 11, 2024

Ideally the root cause of the problem would be addressed, but I believe the above action would at least restore the status quo and re-enable automated Dependabot bumps to core.

dependabot/dependabot-core#8514 (fixed) - perhaps this one is due to our setup that is using a single mirror for both snapshots and releases?

FWIW, I naively expected a snapshot clean-up policy to be in force, as such I would think that any SNAPSHOT (and certainly CD/incrementals based snapshot 999999-SNAPSHOT version) older than 2 weeks could be safely culled.

@timja
Copy link
Member

timja commented Jul 11, 2024

incrementals are sometimes used for a lot longer than 2 weeks, but yes SNAPSHOTS generally aren't needed for long

@timja
Copy link
Member

timja commented Jul 11, 2024

or we could swap to renovate to avoid this I expect

@timja
Copy link
Member

timja commented Jul 13, 2024

Verified renovate handles this just fine: timja-org/jenkins#6

@timja
Copy link
Member

timja commented Jul 13, 2024

Raised a renovate switch PR for discussion: jenkinsci/jenkins#9459

@dduportal dduportal added this to the infra-team-sync-2024-07-23 milestone Jul 15, 2024
@dduportal dduportal removed the triage Incoming issues that need review label Jul 15, 2024
@dduportal dduportal removed this from the infra-team-sync-2024-07-30 milestone Jul 30, 2024
@basil
Copy link
Collaborator Author

basil commented Jul 31, 2024

Assuming the snapshots in https://repo.jenkins-ci.org/artifactory/snapshots/org/jenkins-ci/main/remoting/999999-SNAPSHOT/ are no longer needed for testing, can we delete them from the server as a workaround for #3919?

Any update on this ticket? I do not see any reason for us not to implement the workaround I described above, especially given that we have been unable to achieve a migration to Renovate.

@timja
Copy link
Member

timja commented Aug 2, 2024

(FWIW renovate issues are resolved and that can proceed not that it blocks removing this: jenkinsci/jenkins#9459)

@dduportal dduportal self-assigned this Aug 5, 2024
@dduportal dduportal added this to the infra-team-sync-2024-08-13 milestone Aug 5, 2024
@dduportal
Copy link
Contributor

Assuming the snapshots in https://repo.jenkins-ci.org/artifactory/snapshots/org/jenkins-ci/main/remoting/999999-SNAPSHOT/ are no longer needed for testing, can we delete them from the server as a workaround for #3919?

Any update on this ticket? I do not see any reason for us not to implement the workaround I described above, especially given that we have been unable to achieve a migration to Renovate.

Starting deletion of the artifact. I'll keep a copy of them in case we need to rollback.

@dduportal
Copy link
Contributor

Assuming the snapshots in https://repo.jenkins-ci.org/artifactory/snapshots/org/jenkins-ci/main/remoting/999999-SNAPSHOT/ are no longer needed for testing, can we delete them from the server as a workaround for #3919?

Any update on this ticket? I do not see any reason for us not to implement the workaround I described above, especially given that we have been unable to achieve a migration to Renovate.

Starting deletion of the artifact. I'll keep a copy of them in case we need to rollback.

@basil artifact deleted: can you confirm it is ok on your side?

@basil
Copy link
Collaborator Author

basil commented Aug 5, 2024

jenkinsci/jenkins#9448 was closed as expected. Thanks!

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

No branches or pull requests

4 participants