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

Did you recently rewrite history? #3241

Closed
antoine-sac opened this issue Aug 28, 2024 · 10 comments
Closed

Did you recently rewrite history? #3241

antoine-sac opened this issue Aug 28, 2024 · 10 comments
Assignees
Labels
bug This issue is a confirmed bug. p1 This is a high priority issue

Comments

@antoine-sac
Copy link

Describe the bug

Hello,

It looks like you may have recently rewritten the commit history and incidentally broken meta-aws because all referenced commits no longer exist on the master branch.

We cannot set up reproducible builds if commits disappear and years of commits have been rewritten :/

For example, this was the commit for botocore v1.34.157 three weeks ago: 894686f

It is no longer attached to any branch so cannot be found.

Related to : aws4embeddedlinux/meta-aws#9442

Expected Behavior

The commits prior to a few weeks ago should still exist in the commit history of a branch.

Current Behavior

Commits have disappeared from the history and broken years of reproducible builds via meta-aws.

For example:
0a2e478
894686f
19ce2f5
da3918a

Reproduction Steps

NA

Possible Solution

No response

Additional Information/Context

No response

SDK version used

NA

Environment details (OS name and version, etc.)

NA

@antoine-sac antoine-sac added bug This issue is a confirmed bug. needs-triage This issue or PR still needs to be triaged. labels Aug 28, 2024
@antoine-sac
Copy link
Author

There have been multiple force-push commits to master in the past 2 weeks according the repository activity https://github.com/boto/botocore/activity.

With any luck, either @nateprewitt or @hssyoo still have the previous master branch and could rebase and force-push it back to its normal state?

I also imagine anyone with the repo checked out prior to the history rewrite will have the previous commits?

image image

@jonathan343
Copy link
Contributor

Hey @antoine-sac,
Thank you for reporting this issue. We’re in the process of restoring the original history in the master branch. We're currently taking steps to make sure we get recover this properly and prevent further issues. We'll post an update once we have more information.

@antoine-sac
Copy link
Author

The good news is that all the version tags are still attached to the correct commits, so people using boto/[email protected] are not impacted. Although those commits are not attached to the master branch.

An issue only arises for yocto builds because we are forced to also provide the branch of the commit, so we can't use a floating commit.

image

@antoine-sac
Copy link
Author

As a temporary fix, would you be able to create a branch for the "old" master, like temp/fixed_master?

This would allow us to keep using the old commits (matching the version tags) simply by switching branch.

@jonathan343
Copy link
Contributor

Hey @antoine-sac,
I just pushed up a temp-master-fix branch. Can you let us know if this resolves your problem or if you run into any other issues?

@RyanFitzSimmonsAK RyanFitzSimmonsAK removed the needs-triage This issue or PR still needs to be triaged. label Aug 29, 2024
@RyanFitzSimmonsAK RyanFitzSimmonsAK self-assigned this Aug 29, 2024
@RyanFitzSimmonsAK RyanFitzSimmonsAK added the p1 This is a high priority issue label Aug 29, 2024
@zach-welch-aquabyte
Copy link

The temp-master-fix fixed my Yocto build that uses meta-aws (after some manual intervention). It is a good short-term workaround that unblocked my development, so thanks for that. Looking forward to seeing the original branch restored and reading the postmortum.

@jonathan343
Copy link
Contributor

We've pushed an update to the master which restores the lost commits and should resolve this issue.
@antoine-sac can you switch off the temp-master-fix and verify the new update to master resolves your issue?

@zach-welch-aquabyte
Copy link

I can verify that my build now works again using the unmodified scarthgap branch of meta-aws.

@jonathan343
Copy link
Contributor

Thanks for the patience and assistance with resolving this issue!
We’ve made updates to our release infrastructure that will prevent us getting in a state where manual pushes to master are required. We encourage anyone using the temp-master-fix branch to migrate off of it ASAP and don’t guarantee its existence forever.

Copy link

github-actions bot commented Sep 5, 2024

This issue is now closed. Comments on closed issues are hard for our team to see.
If you need more assistance, please open a new issue that references this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a confirmed bug. p1 This is a high priority issue
Projects
None yet
Development

No branches or pull requests

4 participants