[PRDP-15] ci: Updated check and assign on pr workflows #2
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
This pull request does not contain a valid label. Please add one of the following labels: |
|
Comment this PR with update_code to update |
svariant
pushed a commit
that referenced
this pull request
Jun 16, 2023
Co-authored-by: Alessio Cialini <acialini@external.nexigroup.com>
pasqualespica
added a commit
that referenced
this pull request
Jun 19, 2023
* [PRDP-17][feat-add-event-to-receipt-function] Implemented fn-bizevent-to-receipt function * [PRDP-17][feat-add-event-to-receipt-function] Fixed missing dependency - function test template * [PRDP-17][feat-add-event-to-receipt-function] Defined Receipt object - managed receipt status during function stream * [PRDP-17] Refactor classes in correct package * [PRDP-17] Changed QueueClient to annotation @QueueOutput * [PRDP-17] Completed unit test for successful run * [PRDP-17] Defined skeleton for GenerateReceipt function with todos - added reason error code for PDF generation * [PRDP-17] Completed unit tests for BizEventToReceipt function - defined QueueService class for sending messages to queue * [PRDP-17] Changed packages' name and renamed Client classes * [PRDP-17] Defined structure to handle the generation of the PDF for both payer and debtor * [PRDP-17] Fixed bizEvent-to-receipt function error test * [PRDP-17] Improved generate receipt function structure * [PRDP-17] Changed "Client" classes to implementations of interfaces * [PRDP-17] Added Feign dependencies - defined PDFEngine client and client provider implementing Feign * [PRDP-17] Added temporary HTML templates to resources * [PRDP-17] Added HttpTrigger function for health check * [PRDP-17] Changed request body to PDF engine * [PRDP-17] Defined method to build request to PDF engine from Biz event * [PRDP-17] Fixed package name Info file * [PRDP-17] Changed PDF engine client from Feign to Apache HttpClient - defined client success test - removed unused imports * [PRDP-17] Re-added local.setting.json.example * [PRDP-17] Completed PDFEngine client - completed success test - added PdfEngineResponse class * [PRDP-17] fix: changed reference to Cosmos in queue env configs * [PRDP-17] Deleted template folders * [PRDP-17] Fixed queue config and sent message's encoding - added object properties' condition - added HttpStatus status code * [PRDP-17] Implemented Blob Storage client * [PRDP-17] Fixed receipt CosmosDB query - diminished to 1 second queue wait time * [PRDP-17] Added method to map String to Class in ObjectMapperUtils * [PRDP-17] Changed pdf in PdfEngineRequest from File to byte[] * [PRDP-17] Added AzureBlobStorage dependency - added queue config to host.json * [PRDP-17] Completed receipt to pdf function - fixed binding config * [PRDP-17] Minor tests' fix * [PRDP-17] Cleaned clients' code * [PRDP-17] Fixed debtor's partial template * [PRDP-17] Added user info to pdf data map * [PRDP-17] Managed error response from PDF engine * [PRDP-17] Managed clients' instances for mocking * [PRDP-17] Fixed event-to-receipt function's tests * [PRDP-17][feat-add-event-to-receipt-function] Implemented fn-bizevent-to-receipt function * [PRDP-17][feat-add-event-to-receipt-function] Fixed missing dependency - function test template * [PRDP-17][feat-add-event-to-receipt-function] Defined Receipt object - managed receipt status during function stream * [PRDP-17] Refactor classes in correct package * [PRDP-17] Changed QueueClient to annotation @QueueOutput * [PRDP-17] Completed unit test for successful run * [PRDP-17] Defined skeleton for GenerateReceipt function with todos - added reason error code for PDF generation * [PRDP-17] Completed unit tests for BizEventToReceipt function - defined QueueService class for sending messages to queue * [PRDP-17] Changed packages' name and renamed Client classes * [PRDP-17] Defined structure to handle the generation of the PDF for both payer and debtor * [PRDP-17] Fixed bizEvent-to-receipt function error test * [PRDP-17] fix: changed reference to Cosmos in queue env configs * [PRDP-17] Modified pom version * fix * Bump to version 0.0.1-1-PRDP-17-feat-add-event-to-receipt-function [skip ci] * [PRDP-17] Completed unit tests for receipt-to-pdf function - added Info function test * [PRDP-17] Added template zip files * [PRDP-17] Changed class name * [PRDP-15] ci: Updated check and assign on pr workflows (#2) Co-authored-by: Alessio Cialini <acialini@external.nexigroup.com> * [PRDP-15] ci: Updated deploy workflow to avoid using azure pipelines (#4) Co-authored-by: Alessio Cialini <acialini@external.nexigroup.com> * [PRDP-15] ci: Updated code review worklows using only github actions (#3) Co-authored-by: Alessio Cialini <acialini@external.nexigroup.com> * feat: [PRDP-41] ci: Introducing .identity folder with terraform files for G… (#6) * [PRDP-41] ci: Introducing .identity folder with terraform files for Github Runner * fix --------- Co-authored-by: Alessio Cialini <acialini@external.nexigroup.com> Co-authored-by: pasqualespica <36746022+pasqualespica@users.noreply.github.com> * feat: Prdp 41 ci update helm chart (#7) * [PRDP-41] ci: Updating helm chart values * [PRDP-41] ci: Updating helm chart values with canary and affinity configs * [PRDP-41] ci: updated helm charts in order to introduce secret and config valus * [PRDP-41] ci: updated helm charts in order to introduce secret and config values * minor fix --------- Co-authored-by: Alessio Cialini <acialini@external.nexigroup.com> * [PRDP-17] Modified pom version * fix * Bump to version 0.0.1-1-PRDP-17-feat-add-event-to-receipt-function [skip ci] * [PRDP-17] Deleted GenerateReceipt (changed name - merge artifact) * [PRDP-17] Fixed receipt cosmos update - removed property mdAttach BlobStorageResponse * [PRDP-17] Fixed code smells and code complexity * [PRDP-17] Added PdfEngineClient unit test * [PRDP-17] Improved GenerateReceiptPdf tests' teardown instances * [PRDP-17] Changed Mockito version to latest - renamed pdf engine client test class * [PRDP-17] Added ReceiptBlobClientImpl unit tests * [PRDP-17] Added ReceiptCosmosClientImpl unit tests * [PRDP-17] Added ReceiptQueueClientImpl unit tests * [PRDP-17] Added pre-commit hooks * [PRDP-17] Fixed code smells * [PRDP-17] Renamed project in pom.xml * fix * fix coverage excl * Bump to version 0.0.2-PRDP-17-feat-add-event-to-receipt-function [skip ci] --------- Co-authored-by: Alessio Cialini <acialini@external.nexigroup.com> Co-authored-by: pasqualespica <36746022+pasqualespica@users.noreply.github.com> Co-authored-by: pagopa-github-bot <github-bot@pagopa.it> Co-authored-by: Alessio Cialini <63233981+alessio-cialini@users.noreply.github.com>
pasqualespica
added a commit
that referenced
this pull request
Jun 20, 2023
* [PRDP-17][feat-add-event-to-receipt-function] Implemented fn-bizevent-to-receipt function * [PRDP-17][feat-add-event-to-receipt-function] Fixed missing dependency - function test template * [PRDP-17][feat-add-event-to-receipt-function] Defined Receipt object - managed receipt status during function stream * [PRDP-17] Refactor classes in correct package * [PRDP-17] Changed QueueClient to annotation @QueueOutput * [PRDP-17] Completed unit test for successful run * [PRDP-17] Defined skeleton for GenerateReceipt function with todos - added reason error code for PDF generation * [PRDP-17] Completed unit tests for BizEventToReceipt function - defined QueueService class for sending messages to queue * [PRDP-17] Changed packages' name and renamed Client classes * [PRDP-17] Defined structure to handle the generation of the PDF for both payer and debtor * [PRDP-17] Fixed bizEvent-to-receipt function error test * [PRDP-17] Improved generate receipt function structure * [PRDP-17] Changed "Client" classes to implementations of interfaces * [PRDP-17] Added Feign dependencies - defined PDFEngine client and client provider implementing Feign * [PRDP-17] Added temporary HTML templates to resources * [PRDP-17] Added HttpTrigger function for health check * [PRDP-17] Changed request body to PDF engine * [PRDP-17] Defined method to build request to PDF engine from Biz event * [PRDP-17] Fixed package name Info file * [PRDP-17] Changed PDF engine client from Feign to Apache HttpClient - defined client success test - removed unused imports * [PRDP-17] Re-added local.setting.json.example * [PRDP-17] Completed PDFEngine client - completed success test - added PdfEngineResponse class * [PRDP-17] fix: changed reference to Cosmos in queue env configs * [PRDP-17] Deleted template folders * [PRDP-17] Fixed queue config and sent message's encoding - added object properties' condition - added HttpStatus status code * [PRDP-17] Implemented Blob Storage client * [PRDP-17] Fixed receipt CosmosDB query - diminished to 1 second queue wait time * [PRDP-17] Added method to map String to Class in ObjectMapperUtils * [PRDP-17] Changed pdf in PdfEngineRequest from File to byte[] * [PRDP-17] Added AzureBlobStorage dependency - added queue config to host.json * [PRDP-17] Completed receipt to pdf function - fixed binding config * [PRDP-17] Minor tests' fix * [PRDP-17] Cleaned clients' code * [PRDP-17] Fixed debtor's partial template * [PRDP-17] Added user info to pdf data map * [PRDP-17] Managed error response from PDF engine * [PRDP-17] Managed clients' instances for mocking * [PRDP-17] Fixed event-to-receipt function's tests * [PRDP-17][feat-add-event-to-receipt-function] Implemented fn-bizevent-to-receipt function * [PRDP-17][feat-add-event-to-receipt-function] Fixed missing dependency - function test template * [PRDP-17][feat-add-event-to-receipt-function] Defined Receipt object - managed receipt status during function stream * [PRDP-17] Refactor classes in correct package * [PRDP-17] Changed QueueClient to annotation @QueueOutput * [PRDP-17] Completed unit test for successful run * [PRDP-17] Defined skeleton for GenerateReceipt function with todos - added reason error code for PDF generation * [PRDP-17] Completed unit tests for BizEventToReceipt function - defined QueueService class for sending messages to queue * [PRDP-17] Changed packages' name and renamed Client classes * [PRDP-17] Defined structure to handle the generation of the PDF for both payer and debtor * [PRDP-17] Fixed bizEvent-to-receipt function error test * [PRDP-17] fix: changed reference to Cosmos in queue env configs * [PRDP-17] Modified pom version * fix * Bump to version 0.0.1-1-PRDP-17-feat-add-event-to-receipt-function [skip ci] * [PRDP-17] Completed unit tests for receipt-to-pdf function - added Info function test * [PRDP-17] Added template zip files * [PRDP-17] Changed class name * [PRDP-15] ci: Updated check and assign on pr workflows (#2) Co-authored-by: Alessio Cialini <acialini@external.nexigroup.com> * [PRDP-15] ci: Updated deploy workflow to avoid using azure pipelines (#4) Co-authored-by: Alessio Cialini <acialini@external.nexigroup.com> * [PRDP-15] ci: Updated code review worklows using only github actions (#3) Co-authored-by: Alessio Cialini <acialini@external.nexigroup.com> * feat: [PRDP-41] ci: Introducing .identity folder with terraform files for G… (#6) * [PRDP-41] ci: Introducing .identity folder with terraform files for Github Runner * fix --------- Co-authored-by: Alessio Cialini <acialini@external.nexigroup.com> Co-authored-by: pasqualespica <36746022+pasqualespica@users.noreply.github.com> * feat: Prdp 41 ci update helm chart (#7) * [PRDP-41] ci: Updating helm chart values * [PRDP-41] ci: Updating helm chart values with canary and affinity configs * [PRDP-41] ci: updated helm charts in order to introduce secret and config valus * [PRDP-41] ci: updated helm charts in order to introduce secret and config values * minor fix --------- Co-authored-by: Alessio Cialini <acialini@external.nexigroup.com> * [PRDP-17] Modified pom version * fix * Bump to version 0.0.1-1-PRDP-17-feat-add-event-to-receipt-function [skip ci] * [PRDP-17] Deleted GenerateReceipt (changed name - merge artifact) * [PRDP-17] Fixed receipt cosmos update - removed property mdAttach BlobStorageResponse * [PRDP-17] Fixed code smells and code complexity * [PRDP-17] Added PdfEngineClient unit test * [PRDP-17] Improved GenerateReceiptPdf tests' teardown instances * [PRDP-17] Changed Mockito version to latest - renamed pdf engine client test class * [PRDP-17] Added ReceiptBlobClientImpl unit tests * [PRDP-17] Added ReceiptCosmosClientImpl unit tests * [PRDP-17] Added ReceiptQueueClientImpl unit tests * [PRDP-17] Added pre-commit hooks * [PRDP-17] Fixed code smells * [PRDP-17] Renamed project in pom.xml * fix * fix coverage excl * Bump to version 0.0.2-PRDP-17-feat-add-event-to-receipt-function [skip ci] * fix * [PRDP-17] Deleted unused utils methods * [PRDP-17] Cleaned code - added unit tests for template not found exception * [PRDP-17] Added unit test for ObjectMapperUtils class * [PRDP-17] Fixed exceptions from queue & blob storage clients - added tests * [PRDP-17] Defined transaction creation date for event data * [PRDP-17] Extracted methods from GenerateReceipt function to utility class * [PRDP-17] Documented functions, class and methods * [PRDP-17] Written README * [PRDP-17] Changed test info * [PRDP-17] Improved info logging in generate-receipt function --------- Co-authored-by: Alessio Cialini <acialini@external.nexigroup.com> Co-authored-by: pasqualespica <36746022+pasqualespica@users.noreply.github.com> Co-authored-by: pagopa-github-bot <github-bot@pagopa.it> Co-authored-by: Alessio Cialini <63233981+alessio-cialini@users.noreply.github.com>
pasqualespica
added a commit
that referenced
this pull request
Jun 28, 2023
pasqualespica
added a commit
that referenced
this pull request
Jun 30, 2023
* PRDP-21: setup integration test project * PRDP-21: added test implementation * PRDP-21: added secrets for connection string, added env var for integration test and updated integration test * PRDP-21: enable smoke test * PRDP-21: added docker-compose and script sh * fix identity * PRDP-21: fix typo in run_docker.sh and added env var sample * PRDP-21: increase delay in integration test and update assertions * PRDP-21: export secrets for integration tests * PRDP-21: added log * PRDP-21: added log * PRDP-21: update environment in integration test action * PRDP-21: added log * try it * try it #2 * try it #3 * try it #4 * add base64 secrets encoded * PRDP-21: update variaiblles * PRDP-21: fix base63 decode * try #4 * PRDP-21: add log * fix run int-test #1 * fix run int-test #2 * fix run int-test #3 * fix * fix pipe --------- Co-authored-by: giomella <gioele.mella.st@everis.nttdata.com> Co-authored-by: pasqualespica <36746022+pasqualespica@users.noreply.github.com>
alessio-cialini
pushed a commit
that referenced
this pull request
Oct 17, 2023
* PRDP-21: setup integration test project * PRDP-21: added test implementation * PRDP-21: added secrets for connection string, added env var for integration test and updated integration test * PRDP-21: enable smoke test * PRDP-21: added docker-compose and script sh * fix identity * PRDP-21: fix typo in run_docker.sh and added env var sample * PRDP-21: increase delay in integration test and update assertions * PRDP-21: export secrets for integration tests * PRDP-21: added log * PRDP-21: added log * PRDP-21: update environment in integration test action * PRDP-21: added log * try it * try it #2 * try it #3 * try it #4 * add base64 secrets encoded * PRDP-21: update variaiblles * PRDP-21: fix base63 decode * try #4 * PRDP-21: add log * fix run int-test #1 * fix run int-test #2 * fix run int-test #3 * fix * fix pipe --------- Co-authored-by: giomella <gioele.mella.st@everis.nttdata.com> Co-authored-by: pasqualespica <36746022+pasqualespica@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
List of Changes
Motivation and Context
Updated workflows to match latest template changes
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: