close
Skip to content

Add a new pipeline to test samples on Fabric v3 and FabricX#46

Merged
pasquale95 merged 14 commits intomainfrom
feature/samples_pipeline
Feb 5, 2026
Merged

Add a new pipeline to test samples on Fabric v3 and FabricX#46
pasquale95 merged 14 commits intomainfrom
feature/samples_pipeline

Conversation

@pasquale95
Copy link
Copy Markdown
Contributor

@pasquale95 pasquale95 commented Feb 4, 2026

Type of change

  • New feature

Description

This PR adds a new pipeline called Sample Tests in GitHub Actions whose goal is to run and test the tokens samples against Fabric v3 and Fabric-X.
In this way we automate the checks and make sure that the samples work as expected on a Linux environment.

Additional details (Optional)

The PR also updates to hyperledger.fabricx:0.5.5 Ansible collection being used, due to a recent bug that has been discovered and addressed with LF-Decentralized-Trust-labs/fabric-x-ansible-collection#124.

Signed-off-by: pco <pasquale.convertini@ibm.com>
Signed-off-by: pco <pasquale.convertini@ibm.com>
Signed-off-by: pco <pasquale.convertini@ibm.com>
Signed-off-by: pco <pasquale.convertini@ibm.com>
Signed-off-by: pco <pasquale.convertini@ibm.com>
Signed-off-by: pco <pasquale.convertini@ibm.com>
@pasquale95 pasquale95 linked an issue Feb 4, 2026 that may be closed by this pull request
@pasquale95
Copy link
Copy Markdown
Contributor Author

@arner can you look at this PR? It updates the pipeline to run the token samples in the pipeline and assess if they're running as expected.

Signed-off-by: pco <pasquale.convertini@ibm.com>
Copy link
Copy Markdown
Contributor

@arner arner left a comment

Choose a reason for hiding this comment

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

Great stuff, thanks!

Comment thread samples/tokens/fabricx_ansible.mk Outdated
Comment thread samples/tokens/fabricx_ansible.mk Outdated
Copy link
Copy Markdown
Contributor

@mbrandenburger mbrandenburger left a comment

Choose a reason for hiding this comment

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

I think this is a great PR! Thanks @pasquale95

Comment thread .github/workflows/samples-tests.yml Outdated
Comment thread samples/tokens/fabricx_ansible.mk Outdated
Comment thread .github/workflows/samples-tests.yml Outdated
pasquale95 and others added 7 commits February 4, 2026 16:52
Signed-off-by: pasquale95 <pasqualeconvertini95@gmail.com>
Signed-off-by: pco <pasquale.convertini@ibm.com>
Signed-off-by: pco <pasquale.convertini@ibm.com>
Signed-off-by: pco <pasquale.convertini@ibm.com>
Signed-off-by: pco <pasquale.convertini@ibm.com>
Signed-off-by: pco <pasquale.convertini@ibm.com>
Signed-off-by: pco <pasquale.convertini@ibm.com>
@pasquale95 pasquale95 force-pushed the feature/samples_pipeline branch from 498fd73 to 05e07ac Compare February 5, 2026 11:02
@tock-ibm
Copy link
Copy Markdown
Contributor

tock-ibm commented Feb 5, 2026

Really good idea to have tests working across Fabric V3 and Fabric-X!

Copy link
Copy Markdown
Contributor

@mbrandenburger mbrandenburger left a comment

Choose a reason for hiding this comment

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

LGTM

@pasquale95 pasquale95 merged commit af2477e into main Feb 5, 2026
8 checks passed
@pasquale95 pasquale95 deleted the feature/samples_pipeline branch February 5, 2026 15:04
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.

Add sample application integration tests

4 participants