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

[paymentservice] [checkoutservice] introduce new featureflags #1449

Conversation

EislM0203
Copy link
Contributor

@EislM0203 EislM0203 commented Mar 12, 2024

Changes

This PR is an updated version of #1383 to conform to the updated feature flags. Also a comment regarding the paymentServiceUnreachable feature flag was addressed by moving the feature flag to the checkout-service.

Merge Requirements

For new features contributions please make sure you have completed the following
essential items:

  • CHANGELOG.md updated to document new feature additions
  • Appropriate documentation updates in the docs
  • Appropriate Helm chart updates in the helm-charts

Maintainers will not merge until the above have been completed. If you're unsure
which docs need to be changed ping the
@open-telemetry/demo-approvers.

@EislM0203
Copy link
Contributor Author

Hello, as stated above this PR is the updated version of #1383. Sadly it seems like the helm chart is still outdated and I do not see any open PRs addressing that. Can you tell me what the status is there?

@EislM0203 EislM0203 marked this pull request as ready for review March 12, 2024 13:52
@EislM0203 EislM0203 requested a review from a team March 12, 2024 13:52
@julianocosta89
Copy link
Member

Hello, as stated above this PR is the updated version of #1383. Sadly it seems like the helm chart is still outdated and I do not see any open PRs addressing that. Can you tell me what the status is there?

Excelent question!
I'll take a look into it.

@julianocosta89
Copy link
Member

Hello, as stated above this PR is the updated version of #1383. Sadly it seems like the helm chart is still outdated and I do not see any open PRs addressing that. Can you tell me what the status is there?

There you go: open-telemetry/opentelemetry-helm-charts#1082

@EislM0203
Copy link
Contributor Author

There you go: open-telemetry/opentelemetry-helm-charts#1082

@julianocosta89, thanks a lot for the info, as soon as this PR is merged I will make the changes to the helm chart.

Also, do you perchance know why the tracetesting check is failing? I am asking because I made sure to check that it passes locally (see image). Would appreciate your insights.
Screenshot 2024-03-14 at 08 58 25

@julianocosta89
Copy link
Member

Also, do you perchance know why the tracetesting check is failing? I am asking because I made sure to check that it passes locally (see image). Would appreciate your insights.

Testing out your PR, I did see the failing test.
It seems that the folks from TraceTest changed the name of the trigger and the test wasn't found.
Not related to your PR, but since the update of traceTest you are the first one to touch checkoutService.

I added the fix to your PR, hope you don't mind.

@julianocosta89 julianocosta89 merged commit 8de7041 into open-telemetry:main Mar 14, 2024
25 checks passed
@EislM0203 EislM0203 deleted the paymentservice-checkoutservice-introduce-new-featureflags branch March 15, 2024 06:17
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.

3 participants