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.

Sign up for GitHub

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

Jump to bottom

Fix all CI tests for codec 0.35 #117

Merged
merged 7 commits into from
Apr 21, 2023
Merged

Fix all CI tests for codec 0.35 #117

merged 7 commits into from
Apr 21, 2023

Conversation

noslav
Copy link
Member

@noslav noslav commented Apr 21, 2023

No description provided.

@noslav
fix all tests
611af8f 
Signed-off-by: Pranay Valson <pranay.valson@gmail.com>
@noslav
add codec 0.35 data
d5d5f57 
Signed-off-by: Pranay Valson <pranay.valson@gmail.com>
@noslav
mix format
c8fe3fa 
Signed-off-by: Pranay Valson <pranay.valson@gmail.com>
@noslav noslav mentioned this pull request Apr 21, 2023
Merged
@noslav
update the correct expected blockresult hashes
a523ef0 
Signed-off-by: Pranay Valson <pranay.valson@gmail.com>
@noslav
remove redundant test-data
8c4bc6a 
Signed-off-by: Pranay Valson <pranay.valson@gmail.com>
@noslav noslav changed the base branch from develop to release/public April 21, 2023 05:36
@noslav noslav changed the base branch from release/public to develop April 21, 2023 05:37
@sudeepdino008
Copy link
Contributor

sudeepdino008 commented Apr 21, 2023 •
edited
Loading

finding these in docker-ci-test:


rudder        | connect to [172.18.0.6] from proof-chain.rudder_cqt-net [172.18.0.5] 37503
rudder        | Proof-chain contracts deployed!
rudder        | Uploading test files to local .ipfs...
rudder        | curl: (26) Failed to open/read local data from file/application
rudder        | curl: (26) Failed to open/read local data from file/application
rudder        | Test bsp files uploaded!

also need to fix one test.

EDIT: pushed the fixes.
Approved.

@sudeepdino008
move to plain encode avro formats, as decode is plain by default
412cd5f 
- this fixes the following warning while running tests:
```
Warning:      |   * test Rudder.Avro.BlockSpecimen.encode_file/1 encodes segment json file [L#118]09:36:14.421 [warning] message already contains embeded schema, given schema will be ignored
```
@rogarcia rogarcia merged commit 84a5a22 into develop Apr 21, 2023
3 checks passed
@noslav noslav deleted the fix/ci-tests-codec0.35 branch May 10, 2023 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sudeepdino008 sudeepdino008 sudeepdino008 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@noslav @sudeepdino008 @rogarcia