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

make test-e2e fails #3010

Open
rootulp opened this issue Jan 12, 2024 · 5 comments
Open

make test-e2e fails #3010

rootulp opened this issue Jan 12, 2024 · 5 comments
Labels
testing items that are strictly related to adding or extending test coverage WS: Maintenance 🔧 includes bugs, refactors, flakes, and tech debt etc

Comments

@rootulp
Copy link
Collaborator

rootulp commented Jan 12, 2024

Context

The nightly workflow is failing and has been for a while https://github.com/celestiaorg/celestia-app/actions/runs/7498578218/job/20413921302

Problem

=== RUN   TestE2ESimple
    simple_test.go:44: Running simple e2e test version 3[8](https://github.com/celestiaorg/celestia-app/actions/runs/7498578218/job/20413921302#step:5:9)02ea[9](https://github.com/celestiaorg/celestia-app/actions/runs/7498578218/job/20413921302#step:5:10)
    simple_test.go:72: 
        	Error Trace:	/home/runner/work/celestia-app/celestia-app/test/e2e/simple_test.go:72
        	Error:      	Not equal: 
        	            	expected: 0x1
        	            	actual  : 0x0
        	Test:       	TestE2ESimple
--- FAIL: TestE2ESimple (302.66s)
=== RUN   TestMinorVersionCompatibility
    upgrade_test.go:31: skipping e2e test: E2E_VERSIONS not set
--- SKIP: TestMinorVersionCompatibility (0.00s)
=== RUN   TestMajorUpgradeToV2
    upgrade_test.go:[19](https://github.com/celestiaorg/celestia-app/actions/runs/7498578218/job/20413921302#step:5:20)2: 
        	Error Trace:	/home/runner/work/celestia-app/celestia-app/test/e2e/upgrade_test.go:192
        	Error:      	Not equal: 
        	            	expected: 0x2
        	            	actual  : 0x1
        	Test:       	TestMajorUpgradeToV2
        	Messages:   	version mismatch after upgrade
@rootulp rootulp added the testing items that are strictly related to adding or extending test coverage label Jan 12, 2024
@rootulp rootulp self-assigned this Jan 12, 2024
@rootulp
Copy link
Collaborator Author

rootulp commented Jan 21, 2024

This repros locally but it takes 12+ mins to complete so slow to iterate on.

@cmwaters
Copy link
Contributor

I think this might come about because of: #3094

i.e. it only started failing last night

@rootulp
Copy link
Collaborator Author

rootulp commented Mar 27, 2024

@cmwaters is this issue close-able after #3211 merged?

@staheri14
Copy link
Collaborator

Unassigning myself as I’m not working on this. I’ll leave it to others to decide whether it can be closed.

@staheri14 staheri14 removed their assignment Dec 10, 2024
@rootulp
Copy link
Collaborator Author

rootulp commented Dec 10, 2024

@rootulp rootulp added the WS: Maintenance 🔧 includes bugs, refactors, flakes, and tech debt etc label Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing items that are strictly related to adding or extending test coverage WS: Maintenance 🔧 includes bugs, refactors, flakes, and tech debt etc
Projects
None yet
Development

No branches or pull requests

3 participants