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

Issuer agent, wallet build configuration tweaks. #217

Closed
esune opened this issue Jun 17, 2024 · 2 comments · Fixed by #231
Closed

Issuer agent, wallet build configuration tweaks. #217

esune opened this issue Jun 17, 2024 · 2 comments · Fixed by #231
Assignees

Comments

@esune
Copy link
Member

esune commented Jun 17, 2024

Some issue profiles (idim, lsbc) depend on files hosted in the issuer-kit repository for their builds - see this and this. These are just configuration files, and this is preventing us from being able to archive the issuer-kit repository.

Builds for production issuers should be tweaked to remove this external dependency and be self-contained to this repository.

@esune
Copy link
Member Author

esune commented Aug 23, 2024

Opened #231 to address this - changes still need to be applied to OCP.

Since we are going to want to move issuers to Traction sooner rather than later, I opted for the minimal set of changes to make the builds self-contained to this repository. Once we move to Traction neither build will be required anymore.

I am also wondering if we should deploy the agents directly referencing the image tag, rather than a build with an image tag that is overwritten by the build parameters, potentially... The ACA-Py standalone chart would be very helpful in this scenario even if we decide to keep independent deployments.

c.c.: @WadeBarnes @i5okie

@esune
Copy link
Member Author

esune commented Aug 23, 2024

Also relates to bcgov/DITP-DevOps#18

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant