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

Use a new message for roll_forward update #4623

Merged
merged 1 commit into from
Dec 23, 2024

Conversation

yuying-y
Copy link
Contributor

This new messages distinguishes the roll_forward state from the newly installed update state. This is useful for the YTS tests to verify the correct updater states.

b/382561808

Change-Id: I829c490ba22a2fbfe03276864ebdc6e0350ba243

This new messages distinguishes the roll_forward state from the newly
installed update state. This is useful for the YTS tests to verify the
correct updater states.

b/382561808

Change-Id: I829c490ba22a2fbfe03276864ebdc6e0350ba243
Copy link
Contributor

@yjzhang111 yjzhang111 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is for debugging purposes (or internal use) only right?

@yuying-y
Copy link
Contributor Author

This is for debugging purposes (or internal use) only right?

Not only, testers will see this new message too. But it's mostly for the EG tests to be reliably and precisely verifying the two states: update installed vs locally rolled forward.

@yuying-y yuying-y merged commit 6d6386c into youtube:25.lts.1+ Dec 23, 2024
285 of 291 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants