Skip to content

Conversation

@suhothayan
Copy link
Member

Release Notes

Breaking Changes

  • PLACEHOLDER

New Features

Bug Fixes

  • Add support to report error when an Organization is Paused.

Checklist

  • I have successfully built and used a custom CLI binary, without linter issues from this PR.
  • I have clearly specified in the What section below whether this PR applies to Confluent Cloud, Confluent Platform, or both.
  • I have verified this PR in Confluent Cloud pre-prod or production environment, if applicable.
  • I have verified this PR in Confluent Platform on-premises environment, if applicable.
  • I have attached manual CLI verification results or screenshots in the Test & Review section below.
  • I have added appropriate CLI integration or unit tests for any new or updated commands and functionality.
  • I confirm that this PR introduces no breaking changes or backward compatibility issues.
  • I have indicated the potential customer impact if something goes wrong in the Blast Radius section below.
  • I have put checkmarks below confirming that the feature associated with this PR is enabled in:
    • Confluent Cloud prod
    • Confluent Cloud stag
    • Confluent Platform
    • Check this box if the feature is enabled for certain organizations only

What

what:

  • Reporting the appropriate error then an Confluent Cloud Organization is paused
  • Do not block login on a paused Confluent Cloud Organization

why

  • User should get the correct guidance and reasoning why their command is failing when an Organization is paused

Blast Radius

This will only impact new organizations that are created after the Pause Trial Feature is released.

References

https://confluentinc.atlassian.net/wiki/spaces/CPG/pages/4720427042
https://confluentinc.atlassian.net/browse/EXP-21163

Test & Review

@suhothayan suhothayan requested a review from a team as a code owner October 24, 2025 21:51
Copilot AI review requested due to automatic review settings October 24, 2025 21:51
@confluent-cla-assistant
Copy link

🎉 All Contributor License Agreements have been signed. Ready to merge.
Please push an empty commit if you would like to re-run the checks to verify CLA status for all contributors.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for handling paused trial organizations in Confluent Cloud by introducing a new suspension event type PAUSE_TRIAL. The changes ensure users receive appropriate error messages when their organization is paused while allowing login functionality to continue working.

Key changes:

  • Added a new IsOrgPauseTrialSuspended utility function to identify paused trial organizations
  • Updated login blocking logic to exclude pause trial suspensions, allowing users to log in even when their org is paused
  • Introduced specific error messages and suggestions for paused trial organizations

Reviewed Changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
pkg/utils/org_utils.go Added IsOrgPauseTrialSuspended function and updated IsLoginBlockedByOrgSuspension to exclude pause trial events
pkg/utils/org_utils_test.go Added comprehensive test coverage for pause trial detection and login blocking logic
pkg/errors/error_message.go Added PauseTrialSuggestions constant with guidance for paused organizations
pkg/config/config.go Integrated pause trial error handling into cloud login checks
pkg/cmd/run_requirements_test.go Added test case for pause trial suspended organization context
go.mod Updated ccloud-sdk-go-v1-public dependency to support new suspension event types

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@suhothayan suhothayan enabled auto-merge (squash) October 24, 2025 22:11
@sonarqube-confluent
Copy link

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.

2 participants