Skip to content

Conversation

@adskyiproger
Copy link
Contributor

@adskyiproger adskyiproger commented Jun 20, 2025

Description

Related issue: opencrvs/opencrvs-core#6984

Country config field is optional now:

  • If not provided then last commit is used from country-config branch
  • If provided then country-config will use provided commit

Edge cases:

  • Country config doesn't have image build from last commit hash.

Checklist

  • I have linked the correct Github issue under "Development"
  • I have tested the changes locally, and written appropriate tests
  • I have tested beyond the happy path (e.g. edge cases, failure paths)
  • I have updated the changelog with this change (if applicable)
  • I have updated the GitHub issue status accordingly

Test results

Run deployment without defining country config image tag: https://github.com/opencrvs/opencrvs-farajaland/actions/runs/15778974955

Input properties:
image

Skip checkout step:
image

Build summary:
image

Run deployment with specific version of country config image tag:

Input properties:
image

Checkout for both repositories executed:
image

Build summary:

image

@github-actions

This comment has been minimized.

@adskyiproger adskyiproger merged commit e3b73cc into release-v1.8.0 Jun 23, 2025
3 checks passed
@adskyiproger adskyiproger deleted the ocrvs-6984 branch June 23, 2025 10:58
adskyiproger added a commit that referenced this pull request Jul 16, 2025
* fix: Clarify the country config version a country is deployment (#824)

* fix: Clarify the country config version a country is deployment

* fix: n/a -> latest

* chore: add more data columns for 2024, 2025 location statistics

* fix: copy

* fix: Change current_user to ansible_user (#830)

* fix: reverted current_user to ansible_user to avoid creation of runne… (#755)

* fix: reverted current_user to ansible_user to avoid creation of runner user

* Update infrastructure/server-setup/tasks/application.yml

Co-authored-by: Cihan Bebek <[email protected]>

---------

Co-authored-by: Cihan Bebek <[email protected]>

* fix: Change current_user to ansible_user (#756)

---------

Co-authored-by: Cihan Bebek <[email protected]>

* feat: Add multi-arch build support to country-config (#835)

---------

Co-authored-by: adskyiproger <[email protected]>
Co-authored-by: tahmidrahman-dsi <[email protected]>
Co-authored-by: Tahmid Rahman <[email protected]>
Co-authored-by: Cihan Bebek <[email protected]>
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.

4 participants