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

chore: regenerate recaptchaenterprise client #19018

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

yoshi-code-bot
Copy link
Contributor

@yoshi-code-bot yoshi-code-bot commented Dec 6, 2023

Generated in GitHub action: https://github.com/googleapis/googleapis/google-api-java-client-services/actions/workflows/codegen.yaml

Tracking the compilation issue via b/309667927

@yoshi-code-bot yoshi-code-bot requested a review from a team as a code owner December 6, 2023 02:40
@trusted-contributions-gcf trusted-contributions-gcf bot added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 6, 2023
yoshi-approver
yoshi-approver previously approved these changes Dec 6, 2023
@yoshi-approver yoshi-approver added the automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit label Dec 6, 2023
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 6, 2023
Copy link
Contributor

Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot.

@gcf-merge-on-green gcf-merge-on-green bot removed the automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit label Dec 6, 2023
@lqiu96
Copy link
Contributor

lqiu96 commented Dec 6, 2023

Error:

[INFO] -------------------------------------------------------------
[ERROR] COMPILATION ERROR : 
[INFO] -------------------------------------------------------------
[ERROR] /tmpfs/src/github/google-api-java-client-services/clients/google-api-services-recaptchaenterprise/v1/2.0.0/com/google/api/services/recaptchaenterprise/v1/RecaptchaEnterprise.java:[2461,40] method setKey(java.lang.String) is already defined in class com.google.api.services.recaptchaenterprise.v1.RecaptchaEnterprise.Projects.Keys.RetrieveLegacySecretKey
[INFO] 1 error
[INFO] -------------------------------------------------------------
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  12.180 s
[INFO] Finished at: 2023-12-06T02:42:53Z
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.9.0:compile (default-compile) on project google-api-services-recaptchaenterprise: Compilation failure
[ERROR] /tmpfs/src/github/google-api-java-client-services/clients/google-api-services-recaptchaenterprise/v1/2.0.0/com/google/api/services/recaptchaenterprise/v1/RecaptchaEnterprise.java:[2461,40] method setKey(java.lang.String) is already defined in class com.google.api.services.recaptchaenterprise.v1.RecaptchaEnterprise.Projects.Keys.RetrieveLegacySecretKey

@trusted-contributions-gcf trusted-contributions-gcf bot added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 7, 2023
@yoshi-approver yoshi-approver added the automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit label Dec 7, 2023
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 7, 2023
Copy link
Contributor

Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot.

@gcf-merge-on-green gcf-merge-on-green bot removed the automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit label Dec 7, 2023
@trusted-contributions-gcf trusted-contributions-gcf bot added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 8, 2023
@yoshi-approver yoshi-approver added the automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit label Dec 8, 2023
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 8, 2023
Copy link
Contributor

Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot.

@gcf-merge-on-green gcf-merge-on-green bot removed the automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit label Dec 8, 2023
@trusted-contributions-gcf trusted-contributions-gcf bot added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 9, 2023
@yoshi-approver yoshi-approver added the automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit label Dec 9, 2023
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 9, 2023
Copy link
Contributor

Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot.

@gcf-merge-on-green gcf-merge-on-green bot removed the automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit label Dec 9, 2023
@yoshi-approver yoshi-approver added the automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit label Dec 10, 2023
@trusted-contributions-gcf trusted-contributions-gcf bot added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 10, 2023
@yoshi-approver yoshi-approver added the automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit label May 15, 2024
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 15, 2024
Copy link
Contributor

Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot.

@gcf-merge-on-green gcf-merge-on-green bot removed the automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit label May 15, 2024
@lqiu96
Copy link
Contributor

lqiu96 commented May 15, 2024

From the discovery doc: https://recaptchaenterprise.googleapis.com/$discovery/rest?version=v1. I believe the issue is that the setKey(...) will be generated to set the key parameter for RetrieveLegacySecretKey: image

This conflicts with the setKey(...) that is populated from the default request parameters:
image

The default request parameters are populated for every generated request as per this template:

{% for param in api.parameters %}
{% indent 1 %}{% doc_comment_if param.description %}{% endindent %}
@com.google.api.client.util.Key{% ifnotequal param.wireName param.memberName %}("{{ param.wireName }}"){% endifnotequal %}
private {{ param.codeType }} {{ param.memberName }};
/**{% filter block_comment %}
* {{ param.description }}
* {% if param.default %} [default: {{ param.default }}]{% endif %}
* {% if param.minimum %} [minimum: {{ param.minimum }}]{% endif %}
* {% if param.maximum %} [maximum: {{ param.maximum }}]{% endif %}
* {% endfilter %}
*/
public {{ param.codeType }} {{ param.getterName }}() {
return {{ param.memberName }};
}
{% indent 1 %}{% doc_comment_if param.description %}{% endindent %}
public {{ api.className }}Request<T> {{ param.setterName }}({{ param.codeType }} {{ param.memberName }}) {
this.{{ param.memberName }} = {{ param.memberName }};
return this;
}{% endfor %}

Changing the request template would be a breaking change for all the other libraries generated.

@yoshi-code-bot yoshi-code-bot force-pushed the regenerate-recaptchaenterprise branch from bd90970 to 07b8c86 Compare May 16, 2024 02:39
@trusted-contributions-gcf trusted-contributions-gcf bot added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 16, 2024
@yoshi-approver yoshi-approver added the automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit label May 16, 2024
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 16, 2024
Copy link
Contributor

Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot.

@gcf-merge-on-green gcf-merge-on-green bot removed the automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit label May 16, 2024
@yoshi-code-bot yoshi-code-bot force-pushed the regenerate-recaptchaenterprise branch from 07b8c86 to 9c72ecc Compare May 17, 2024 02:38
@trusted-contributions-gcf trusted-contributions-gcf bot added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 17, 2024
@yoshi-approver yoshi-approver added the automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit label May 17, 2024
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 17, 2024
Copy link
Contributor

Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot.

@gcf-merge-on-green gcf-merge-on-green bot removed the automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit label May 17, 2024
@yoshi-code-bot yoshi-code-bot force-pushed the regenerate-recaptchaenterprise branch from 9c72ecc to 617d9df Compare May 18, 2024 02:39
@yoshi-approver yoshi-approver added the automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit label May 18, 2024
Copy link
Contributor

Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot.

@gcf-merge-on-green gcf-merge-on-green bot removed the automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit label May 18, 2024
@yoshi-code-bot yoshi-code-bot force-pushed the regenerate-recaptchaenterprise branch from 617d9df to 7145a69 Compare May 19, 2024 02:40
@yoshi-approver yoshi-approver added the automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit label May 19, 2024
Copy link
Contributor

Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot.

@gcf-merge-on-green gcf-merge-on-green bot removed the automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit label May 19, 2024
@yoshi-code-bot yoshi-code-bot force-pushed the regenerate-recaptchaenterprise branch from 7145a69 to 6ec4a3c Compare May 20, 2024 02:41
@trusted-contributions-gcf trusted-contributions-gcf bot added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 20, 2024
@yoshi-approver yoshi-approver added the automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit label May 20, 2024
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge: exact Summon MOG for automerging, but approvals need to be against the latest commit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants