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

#274 - Merge release v2.2.0 to v2.3.0 #4216

Merged
merged 7 commits into from
Jan 8, 2025
Merged

Conversation

sh16011993
Copy link
Collaborator

@sh16011993 sh16011993 commented Jan 8, 2025

As a part of this PR, release v2.2.0 is merged to v2.3.0

bidyashish and others added 7 commits December 19, 2024 16:13
**Technical Details**
- [x] Ensure the file name is escaped while downloading to prevent the
issue with the comma and some other potential characters.
- [x] Add an E2E test with some scenarios to test different characters.
- [ ] This should be added to v2.2 branch.


Demo:
Able to Download File with Comma

![image](https://github.com/user-attachments/assets/82bd65d5-ed65-41b2-b252-edad13b07fa8)

![image](https://github.com/user-attachments/assets/dca924d4-1e2a-4727-8b70-deab9b95f060)
**Technical Details**
- [x] Ensure the file name is escaped while downloading to prevent the
issue with the comma and some other potential characters.
- [x] Add an E2E test with some scenarios to test different characters.
- [x] This should be added to v2.2 branch.


Demo:
Able to Download File with Comma


![image](https://github.com/user-attachments/assets/82bd65d5-ed65-41b2-b252-edad13b07fa8)


![image](https://github.com/user-attachments/assets/dca924d4-1e2a-4727-8b70-deab9b95f060)
- Added the
[openshift-tools-installer](https://github.com/redhat-actions/openshift-tools-installer)
to all action steps that require some OC iteration.
- Using the [default
options](https://github.com/redhat-actions/openshift-tools-installer?tab=readme-ov-file#inputs).
- Using the fixed version "4". The current version printed is `OC CLI
Version: Client Version: 4.17.9`.

Please note, for steps printing the `OC version`, the new action was
added before the print step. For steps not printing the `OC version`,
the new step was added right before the "Log in to OpenShift".

Sample workflow execution with the new step working:
https://github.com/bcgov/SIMS/actions/runs/12378896945/job/34551860119
- Added the
[openshift-tools-installer](https://github.com/redhat-actions/openshift-tools-installer)
to all action steps that require some OC iteration.
- Using the [default
options](https://github.com/redhat-actions/openshift-tools-installer?tab=readme-ov-file#inputs).
- Using the fixed version "4". The current version printed is `OC CLI
Version: Client Version: 4.17.9`.

Please note, for steps printing the `OC version`, the new action was
added before the print step. For steps not printing the `OC version`,
the new step was added right before the "Log in to OpenShift".

Sample workflow execution with the new step working:
https://github.com/bcgov/SIMS/actions/runs/12378896945/job/34551860119
### As a part of this PR, the following were completed:

- Ensured that the sequence numbers in PT and FT eCert file header is
incremented as a lifetime value.
- Started the next PROD sequence number at **7 for PT eCerts** and **FT
eCerts are left as is (set to start at 1).** For this, added the
following SQL to the Release 2.2 notes:

```sql
INSERT INTO sims.sequence_controls
(sequence_name, sequence_number)
VALUES ('ECERT_PT_SENT_FILE', 6), ('ECERT_FT_SENT_FILE', 0);
```

**Screenshots:**

<img width="1920" alt="image"
src="https://github.com/user-attachments/assets/d7c854ae-a4f0-4163-9717-acd3e2e3c997"
/>

---------------------------------------------

<img width="1919" alt="image"
src="https://github.com/user-attachments/assets/287fdbd3-e7f7-4a41-8a1b-06b9c20b9c16"
/>
- Resolved conflicts in file e-cert-file-handler.ts
@sh16011993 sh16011993 self-assigned this Jan 8, 2025
@sh16011993 sh16011993 added the Branch Sync-up Branch features move between main/release/hotfix branches. label Jan 8, 2025
Copy link

sonarqubecloud bot commented Jan 8, 2025

Copy link

github-actions bot commented Jan 8, 2025

Backend Unit Tests Coverage Report

Totals Coverage
Statements: 22.22% ( 3784 / 17027 )
Methods: 10.18% ( 219 / 2152 )
Lines: 25.67% ( 3279 / 12775 )
Branches: 13.62% ( 286 / 2100 )

Copy link

github-actions bot commented Jan 8, 2025

E2E Workflow Workers Coverage Report

Totals Coverage
Statements: 65.59% ( 589 / 898 )
Methods: 59.63% ( 65 / 109 )
Lines: 68.72% ( 468 / 681 )
Branches: 51.85% ( 56 / 108 )

Copy link

github-actions bot commented Jan 8, 2025

E2E Queue Consumers Coverage Report

Totals Coverage
Statements: 86.14% ( 1249 / 1450 )
Methods: 82.42% ( 136 / 165 )
Lines: 88.51% ( 1032 / 1166 )
Branches: 68.07% ( 81 / 119 )

@sh16011993 sh16011993 changed the title Merge v2.2.0 to v2.3.0 Merge release v2.2.0 to v2.3.0 Jan 8, 2025
Copy link

github-actions bot commented Jan 8, 2025

E2E SIMS API Coverage Report

Totals Coverage
Statements: 67.4% ( 5922 / 8787 )
Methods: 65.09% ( 729 / 1120 )
Lines: 71.33% ( 4647 / 6515 )
Branches: 47.4% ( 546 / 1152 )

@andrewsignori-aot andrewsignori-aot changed the title Merge release v2.2.0 to v2.3.0 #274 - Merge release v2.2.0 to v2.3.0 Jan 8, 2025
Copy link
Collaborator

@andrewsignori-aot andrewsignori-aot left a comment

Choose a reason for hiding this comment

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

Looks good 👍

Copy link
Collaborator

@dheepak-aot dheepak-aot left a comment

Choose a reason for hiding this comment

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

Looks good 👍

@sh16011993 sh16011993 merged commit a597dee into release/v2.3.0 Jan 8, 2025
21 checks passed
@sh16011993 sh16011993 deleted the merge_2.2_to_2.3 branch January 8, 2025 23:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Branch Sync-up Branch features move between main/release/hotfix branches.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants