Skip to content

Conversation

sagikazarmark
Copy link
Member

Overview

Update ent to the latest version.

What this PR does / why we need it

Updates the ent library to the latest version as well as opts-in for the new Atlas migration engine and regenerates generated files.

Special notes for your reviewer

Does this PR introduce a user-facing change?

NONE

@sagikazarmark
Copy link
Member Author

@nabokihms any ideas what's going wrong here?

@nabokihms
Copy link
Member

I have never seen this before. If you want, I can research the actual cause.

@sagikazarmark
Copy link
Member Author

Looking at the error, the returned timestamp is not precise enough. I don't know if that's because of ent or something else.

@sagikazarmark
Copy link
Member Author

Looks like the problem was with Atlas. I disabled it for now.

See the linked issue for details.

@sagikazarmark
Copy link
Member Author

I don't know what's with the Docker build these days.

Signed-off-by: Mark Sagi-Kazar <mark.sagikazar@gmail.com>
Signed-off-by: Mark Sagi-Kazar <mark.sagikazar@gmail.com>
Signed-off-by: Mark Sagi-Kazar <mark.sagikazar@gmail.com>
Signed-off-by: Mark Sagi-Kazar <mark.sagikazar@gmail.com>
Looks like Atlas (the new migration library under Ent) cannot
handle precision properly.

An issue has been reported to Ent: ent/ent#2454

Signed-off-by: Mark Sagi-Kazar <mark.sagikazar@gmail.com>
@sagikazarmark
Copy link
Member Author

@nabokihms can I get a review on this one? I managed to fix the tests. Docker is just...doing it's own thing, I'm gonna ignore that.

Copy link
Member

@nabokihms nabokihms left a comment

Choose a reason for hiding this comment

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

Looks perfect! Thank you.

@sagikazarmark sagikazarmark merged commit a51ed2c into master Apr 7, 2022
@sagikazarmark sagikazarmark deleted the update-ent branch April 7, 2022 15:26
@sagikazarmark sagikazarmark added the release-note/dependency-update Release note: Dependency Updates label Apr 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/storage release-note/dependency-update Release note: Dependency Updates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants