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

Fix/merge instance #4204

Merged
merged 3 commits into from
Jun 28, 2022

Conversation

tanishqjasoria
Copy link
Contributor

No description provided.

@tanishqjasoria tanishqjasoria changed the base branch from master to test/update-hive-tests-gray-glacier June 24, 2022 08:14
@tanishqjasoria tanishqjasoria marked this pull request as ready for review June 24, 2022 08:16
@MarekM25 MarekM25 self-requested a review June 24, 2022 20:09
@MarekM25
Copy link
Contributor

@tanishqjasoria it seems that tests are still failing

@tanishqjasoria
Copy link
Contributor Author

Yes. There are new hive tests related to access list transactions. They expect that we should return an error whenever there is a transaction with access list before the berlin fork.
But we just ignore the access list and treat the transaction as a normal transaction

@LukaszRozmej LukaszRozmej merged commit 2d0b9ae into test/update-hive-tests-gray-glacier Jun 28, 2022
@LukaszRozmej LukaszRozmej deleted the fix/merge-instance branch June 28, 2022 07:30
LukaszRozmej added a commit that referenced this pull request Jun 28, 2022
* Update hive tests for gray glacier

* Fix/merge instance (#4204)

* London instance as a placeholder for the merge

* fix access list tests

* fix access list check

* Print general tests results

* Revert "Print general tests results"

This reverts commit cf94a8b.

* Add support for post-merge tests

* small refactors

* fix some warnings to run pipeline

Co-authored-by: Tanishq Jasoria <jasoriatanishq@gmail.com>
avalonche pushed a commit that referenced this pull request Jun 29, 2022
* Update hive tests for gray glacier

* Fix/merge instance (#4204)

* London instance as a placeholder for the merge

* fix access list tests

* fix access list check

* Print general tests results

* Revert "Print general tests results"

This reverts commit cf94a8b.

* Add support for post-merge tests

* small refactors

* fix some warnings to run pipeline

Co-authored-by: Tanishq Jasoria <jasoriatanishq@gmail.com>
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.

None yet

3 participants