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

Two jars in a bundle to support both versions of Apigee runtime #199

Merged
merged 8 commits into from
Apr 29, 2021

Conversation

rockspore
Copy link
Contributor

@rockspore rockspore commented Apr 28, 2021

Close #198

If the new bundle failed to be deployed as revision n, the fallback will not force an old bundle deployment as revision n + 1, which avoids another round of waiting for the old revision being undeployed and deleted.

There will be only one bundle containing two jars. Hopefull the support of 1.4 and 1.5 runtime will be transparent.

@codecov
Copy link

codecov bot commented Apr 28, 2021

Codecov Report

Merging #199 (c32d747) into master (28f291e) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #199   +/-   ##
=======================================
  Coverage   81.06%   81.06%           
=======================================
  Files          17       17           
  Lines        1748     1748           
=======================================
  Hits         1417     1417           
  Misses        175      175           
  Partials      156      156           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 28f291e...c32d747. Read the comment docs.

@rockspore rockspore changed the title Issue 198 Two jars in a bundle to support both versions of Apigee runtime Apr 28, 2021
Copy link
Member

@theganyo theganyo left a comment

Choose a reason for hiding this comment

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

approved pending completion of Apigee X testing

@rockspore rockspore marked this pull request as ready for review April 29, 2021 00:26
@rockspore rockspore merged commit ce42600 into master Apr 29, 2021
@rockspore rockspore deleted the issue-198 branch April 29, 2021 00:26
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.

Proxy on Apigee 1.5 is failing to query products
2 participants