-
Notifications
You must be signed in to change notification settings - Fork 97
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
download-logs commands returning old/inaccurate data #1326
Comments
👋 This is by design as |
Hey @ArinGhazarian - thanks for the response. That makes sense, the migration logs aren't actually tied to the migrated repo (they're stored separately). Is that something that can be changed, so the migration logs are tied to the actual repo? Or is there a way to make an API call (or an extension to the gei tooling) to purge the migration logs? |
Apologies for not being clear. The logs are stored separately but they are tied to the migrated repo. The
As I described above they are tied to a repo but deleting a repo doesn't remove the migration log. If you run a new migration for that repo again and it reaches to the actual migration a new migration log will be uploaded for it and can be downloaded.
No. The logs are read-only and maintained by the GEI's backend. Also as a quick note, the migration log is also available as an issue in the migrated repo but it only reflects the backend migration events in a user-friendly fashion. The gh gei CLI does extra pre migration stuff that won't be reflected in the migration log. Like in case of bbs migrations ( |
Description
I'm debugging our migration automation scripts, and came across an interesting issue. I'm intentionally causing the
gh bbs2gh migrate-repo
script to fail, and I see it coming back as expected:This failure is expected... HOWEVER, as a follow-up step, we also run
gh bbs2gh download-logs
and, well, notice the date?This was run at around 2025-02-17 15:42:10, so I'm not sure why it's giving me logs back from 2025-02-13... AND I thought these logs were only available for 24h? What's even more interesting is that I'm causing a failure on the export side of the script by passing it an invalid set of credentials for the azure storage account, which means it should never hit the import step, which makes the
download-logs
output extra wrong since the API call to the GraphQLstartRepositoryMigration
mutator wouldn't have happened yet.The reason there's migration data here (I suspect) is, since I'm working in a "lab" environment for developing the automation, we are deleting and re-creating the same repos repeatedly, so there's a chance that this is leftover (old) data from the last successful previous attempt before I started intentionally causing the
gh bbs2gh migrate-repo
step to fail.Reproduction Steps
gh bbs2gh migrate-repo
command, followed by agh bbs2gh download-logs
command. Observe logs coming back indicating success.gh bbs2gh migrate-repo
step to fail. In my case, I'm passing a bad set of credentials for theAZURE_STORAGE_ACCOUNT_CONNECTION_STRING
env variable to ensure that the error I'm causing is happening before the import step begins. Observe thegh bbs2gh migrate-repo
failure, and it produces appropriate failure logs. Also observe that the repo to be imported doesn't exist yet.gh bbs2gh download-logs
command. Observe logs are returned indicating a successful import... Old leftover data, ok. Also observe the warning that logs are only available for 24 hours.gh bbs2gh download-logs
command again and observe that the same logs come back indicating success, despite being over 24h past when the migration occurred and the repo not existing in GitHub and the previousgh bbs2gh migrate-repo
attempts failing.The text was updated successfully, but these errors were encountered: