Dialogflow generator new#18236
Conversation
Added support for freeFormContext, agentCoachingContext, translationContext, agentFeedbackContext, and customerMessageGenerationContext. Enforced mutual exclusivity using exactly_one_of. Made summarizationContext optional (non-breaking). Added timeouts and tunedModel support.
Added 'tools' property to Generator.yaml and updated samples to demonstrate linkage. Fixed pre-existing errors in Tool connector and datastore samples.
- Update summarizationContext.version candidates in Generator.yaml to 5.0 and 6.0 - Update versions in samples to 6.0 - Update versions in handwritten tests to 5.0 and 6.0 - Add valid path to dialogflow_generator_with_tools OpenAPI spec
cf072d4 to
bdb78ee
Compare
|
Googlers: For automatic test runs see go/terraform-auto-test-runs. @BBBmau, a repository maintainer, has been assigned to review your changes. If you have not received review feedback within 2 business days, please leave a comment on this PR asking them to take a look. You can help make sure that review is quick by doing a self-review and by running impacted tests locally. |
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit bdb78ee: Diff reportYour PR generated the following diffs in downstream repositories:
Breaking Change(s) DetectedThe following breaking change(s) were detected within your pull request.
If you believe this detection to be incorrect please raise the concern with your reviewer. Missing test reportYour PR includes resource fields which are not covered by any test. Resource: resource "google_dialogflow_generator" "primary" {
agent_coaching_context {
instructions {
agent_action = # value needed
condition = # value needed
display_details = # value needed
display_name = # value needed
system_action = # value needed
triggering_event = # value needed
}
output_language_code = # value needed
search_config {
datastores = # value needed
search_engine = # value needed
}
}
agent_feedback_context {
instructions {
agent_action = # value needed
condition = # value needed
display_details = # value needed
display_name = # value needed
system_action = # value needed
triggering_event = # value needed
}
version = # value needed
}
customer_message_generation_context {
customization {
scenario = # value needed
}
version = # value needed
}
free_form_context {
text = # value needed
}
translation_context {
agent_language_code = # value needed
customer_language_code = # value needed
translation_customization {
glossary_terms {
agent_language_glossary = # value needed
customer_language_glossary = # value needed
}
sentence_translations {
agent_language_sentence = # value needed
customer_language_sentence = # value needed
}
}
version = # value needed
}
tuned_model = # value needed
}
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode Action takenFound 5 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
View the replaying VCR build log Step 2: Recording Mode
Caution Issues requiring attention before PR completion 🔴 Initial Recording Failed: Some tests failed during the recording step. See the table above for details. Please address these issues to complete your PR. If you believe these detections are incorrect or unrelated to your change, please raise the concern with your reviewer. View the recording VCR build log or the debug logs folder for detailed results. @matt-degraffenreid, @BBBmau VCR tests complete for bdb78ee! |
Release Note Template for Downstream PRs (will be copied)
See Write release notes for guidance.