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

[#IOCOM-504] Enable canary for messages from view. #683

Merged
merged 3 commits into from
Sep 27, 2023
Merged

Conversation

michaeldisaro
Copy link
Contributor

List of changes

Moved FF to canary users, and made a regex that takes 0.4% of fiscal codes in production

Motivation and context

We need to reach the whole user base in taking messages from the view calculated from fn-messages-cqrs.
After a year testing with betatesters we are ready to move to canary users.

Type of changes

  • Add new resources
  • Update configuration to existing resources
  • Remove existing resources

Env to apply

  • DEV
  • UAT
  • PROD

Does this introduce a change to production resources with possible user impact?

Enabling canary users increases the impact of our tests, if something goes wrong will go back to betatester.

  • Yes, users may be impacted applying this change
  • No

Does this introduce an unwanted change on infrastructure? Check terraform plan execution result

  • Yes
  • No

@michaeldisaro michaeldisaro marked this pull request as ready for review September 21, 2023 15:41
@michaeldisaro michaeldisaro requested a review from a team September 21, 2023 15:41
@michaeldisaro michaeldisaro requested review from a team as code owners September 21, 2023 15:41
@sonarcloud
Copy link

sonarcloud bot commented Sep 27, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@michaeldisaro michaeldisaro merged commit e0b5e90 into main Sep 27, 2023
4 checks passed
@michaeldisaro michaeldisaro deleted the iocom-504 branch September 27, 2023 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants