-
Notifications
You must be signed in to change notification settings - Fork 2
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
Variety of additional changes #13
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Ryan Gonzalez <[email protected]>
Before, this wouldn't include the url/filename, which made it hard to determine where failures happened. Signed-off-by: Ryan Gonzalez <[email protected]>
aptly wouldn't see any references to the architecture and thus would skip it altogether. Signed-off-by: Ryan Gonzalez <[email protected]>
This slightly speeds up the initial component listing when doing syncs, which is more pronounced when running incrementally & skipping snapshots. Signed-off-by: Ryan Gonzalez <[email protected]>
With a lot of publishes, this starts to get quite slow, taking several seconds for each listing. When resuming an incremental run, this results in taking *minutes* to find the last progress point. Signed-off-by: Ryan Gonzalez <[email protected]>
Signed-off-by: Ryan Gonzalez <[email protected]>
Passing the arch list manually but omitting 'source' results in no source packages being published. Signed-off-by: Ryan Gonzalez <[email protected]>
Signed-off-by: Ryan Gonzalez <[email protected]>
Dropping these repos isn't easily doable because it ends up also nuking the snapshots stored at its subpath. Instead, offer an option to just update in-place, which isn't perfect but works for our purposes. Signed-off-by: Ryan Gonzalez <[email protected]>
This test has worked historically without these files, because obs2aptly doesn't need the versions for packages that are not present in aptly yet. However, future optimizations elsewhere will require this, and that's unpleasant behavior to depend on regardless. Signed-off-by: Ryan Gonzalez <[email protected]>
When running apt2aptly, a large chunk of the runtime can end up taken by repeatedly checking for the same packages. This adds caching so that our known information about packages can be kept across syncs. Signed-off-by: Ryan Gonzalez <[email protected]>
Also move it to a workspace-wide version to make it easier to control. Signed-off-by: Ryan Gonzalez <[email protected]>
Signed-off-by: Ryan Gonzalez <[email protected]>
Signed-off-by: Ryan Gonzalez <[email protected]>
Signed-off-by: Ryan Gonzalez <[email protected]>
Signed-off-by: Ryan Gonzalez <[email protected]>
This is useful for debugging purposes. Signed-off-by: Ryan Gonzalez <[email protected]>
In particular, including the distribution makes the logs significantly clearer. Signed-off-by: Ryan Gonzalez <[email protected]>
Signed-off-by: Ryan Gonzalez <[email protected]>
Signed-off-by: Ryan Gonzalez <[email protected]>
This makes it clearer as to where errors reading the version are coming from. Signed-off-by: Ryan Gonzalez <[email protected]>
If we only do the removal when the newest version hashes don't match, and the sync is interrupted *after* packages are added but *before* deletions take place, then the old debs will never be removed. Signed-off-by: Ryan Gonzalez <[email protected]>
These were accidentally omitted before. Signed-off-by: Ryan Gonzalez <[email protected]>
This is a simple REST API that simply returns the latest snapshot published for a dist. Signed-off-by: Ryan Gonzalez <[email protected]>
Signed-off-by: Ryan Gonzalez <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I don't even know how to summarize all this.