-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
pkgx --sync
on main
never finishes if the pantry repo was not updated in the last 24 hours
#1021
Comments
The hanging issue during the Here's an example of how you might add logging to the export default async function({ flags, ...opts }: Args, logger_prefix?: string) {
const logger = make_logger(flags.verbosity, logger_prefix);
if (flags.sync) {
try {
logger.replace('Starting sync...');
await useSync();
logger.replace('Sync completed successfully.');
} catch (err) {
logger.replace(`Sync failed: ${err.message}`);
if (!flags.keepGoing) throw err;
}
}
// ... rest of the code
} Additionally, you can manually check the status of the pantry server by using the async function ensure_pantry() {
if (usePantry().missing()) {
await useSync();
}
} If the pantry is missing and the synchronization fails, it indicates that there might be an issue with the pantry server or related services [1]. If the problem persists, you might want to check the operational status of the external services |
lol. Found out the issue! Since there was no change in the last 24 hours of pantry git, the following line always returns true, causing an infinite loop! |
@jhheider I realized this issue only affects/affected the main branch of pkgx, and not the last released version 1.1.6. I'm very sorry about it. |
In fact, let me keep it open just as a reminder that this has to be fixed before releasing a new version from main. |
pkgx --sync
on main
never finishes if the pantry repo was not updated in the last 24 hours
That's great to know, and explains the lack of outrage. ;) But it'll definitely need fixing. |
We shouldn't need this since we sync for various pkg-not-found situations. Closes #73 Closes pkgxdev/pkgx#1021
We shouldn't need this since we sync for various pkg-not-found situations. Closes #73 Closes pkgxdev/pkgx#1021
We shouldn't need this since we sync for various pkg-not-found situations. Closes #73 Closes pkgxdev/pkgx#1021
I have a lot of pipelines stuck at
pkgx install
. After some troubleshooting, it seems that they aren't being able to sync the pantry.Then I tried in my local machine and can reproduce it:
pkgx --sync
Never finishes, hangs forever.
The text was updated successfully, but these errors were encountered: