Skip to content

misc: fixing swagger openapi.yaml #6751

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 12 commits into from
Jul 23, 2025
Merged

misc: fixing swagger openapi.yaml #6751

merged 12 commits into from
Jul 23, 2025
+1,955 −299

Conversation

badal773
Copy link
Contributor

@badal773 badal773 commented Jul 23, 2025 •
edited by bito-code-review bot
Loading

Summary by Bito

This pull request enhances the OpenAPI specification with new endpoints for user management, role groups, authentication flows, and cluster operations. It introduces detailed endpoint summaries, refined error responses, and improved security schemes to create a more comprehensive API reference.

@github-actions GitHub Actions
Copy link

Some linked issues are invalid. Please update the issue links:\nIssue # in is not found or invalid (HTTP 404).\n

@bito-code-review Bito Code Review
Copy link

bito-code-review bot commented Jul 23, 2025 •
edited
Loading

Code Review Agent Run #94c570

Actionable Suggestions - 0
Review Details
  • Files reviewed - 2 · Commit Range: 54a8871..54a8871
    • specs/application.yaml
    • specs/openapi.yaml
  • Files skipped - 0
  • Tools
    • Whispers (Secret Scanner) - ✔︎ Successful
    • Detect-secrets (Secret Scanner) - ✔︎ Successful
Bito Usage Guide

Commands

Type the following command in the pull request comment and save the comment.

  • /review - Manually triggers a full AI review.

  • /pause - Pauses automatic reviews on this pull request.

  • /resume - Resumes automatic reviews.

  • /resolve - Marks all Bito-posted review comments as resolved.

  • /abort - Cancels all in-progress reviews.

Refer to the documentation for additional commands.

Configuration

This repository uses Default Agent You can customize the agent settings here or contact your Bito workspace admin at [email protected].

Documentation & Help

AI Code Review powered by Bito Logo

@bito-code-review Bito Code Review
Copy link

Changelist by Bito

This pull request implements the following key changes.

Key Change Files Impacted
Documentation - API Documentation Update

- application.yaml - Enhanced API documentation by adding license details, new security schemes, refined endpoint summaries, and standardized error response definitions.

- openapi.yaml - Expanded and refined API schemas with comprehensive definitions covering resource management, pod rotation, workflow DTOs, enriched request bodies, responses, and query parameters.
New Feature - Workflow Endpoint Addition

- openapi.yaml - Introduced new orchestrator endpoints offering workflow creation and deletion capabilities along with extended schema definitions to support workflow management.

@badal773 badal773 changed the title feat: fixing swagger.yaml misc: fixing swagger.yaml Jul 23, 2025
prakarsh-dt
prakarsh-dt previously approved these changes Jul 23, 2025
View reviewed changes
@bito-code-review Bito Code Review
Copy link

bito-code-review bot commented Jul 23, 2025 •
edited
Loading

Bito Automatic Review Skipped - Large PR

Bito didn't auto-review this change because the pull request exceeded the line limit. No action is needed if you didn't intend for the agent to review it. Otherwise, to manually trigger a review, type /review in a comment and save.

@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@badal773 badal773 changed the title misc: fixing swagger.yaml misc: fixing swagger openapi.yaml Jul 23, 2025
@badal773 badal773 requested a review from prakarsh-dt July 23, 2025 12:02
@badal773 badal773 merged commit 6503b20 into main Jul 23, 2025
13 checks passed
@badal773 badal773 deleted the swagger-spec-fix branch July 23, 2025 12:17
@bito-code-review Bito Code Review
Copy link

bito-code-review bot commented Jul 23, 2025 •
edited
Loading

Bito Review Skipped - Source Branch Not Found

Bito didn’t review this change because the pull request is no longer valid. It may have been merged, or the source/target branch may no longer exist.

Ash-exp added a commit that referenced this pull request Jul 31, 2025
@Ash-exp @prkhrkat
@iamayushm @vikramdevtron @akshatsinha007 @kamal-devtron @badal773 @pawan-59
chore: Merged origin/main branch into develop (#6770)
839e287 
* ephemeral error fix

* ephemeral error fix (#6737)

* panic fix

* NotificationRule removed

* vendor update

* misc: Added Github Actions for linting and building api specs (#6720)

* Added action to check for linting errors in openapi.yaml

* Create build-docs.yaml

* added read-only permission in api-linter

* updated action to add permission

* misc: open api spec addition (#6750)

* added version yml

* changes in openapi.yaml

* Updated build-docs.yaml (#6752)

switch the approach from rebasing to creating a temp folder and copying the file

* misc: fixing swagger openapi.yaml (#6751)

* resolved application.yaml

* resolved application.yaml

* resolved applisting.yaml

* resolved charts.yaml helm-apps list

* resolved jobs.yaml jobs list

* removed unrequired server

* renamed the x-displayName

* resolved app-labels.yaml

* resolved merge-conflict

* resolved merge-conflict - 02

* added Get Deployment History (ENT) spec

* misc: adding codeowner for swagger api spec (#6754)

* adding codeowner for swagger api spec

* adding codeowner for swagger api spec

* gitops validation spec added (#6755)

* misc: Update openapi spec (#6758)

* Updated openapi-spec with notificats api

* updated openapi-spec to add historical-get-api-change

* updated the title of openapi specs (#6759)

* removed timeout notification (#6760)

* misc: Timeout main notification (#6761)

* removed timeout notification

* removed timeout notification

* misc: changes in description of api spec (#6767)

* changes in description of api spec

* refactor: rename bulk edit config SQL files for consistency

* chore: update dependencies to origin/main

---------

Co-authored-by: prakhar katiyar <[email protected]>
Co-authored-by: prakhar katiyar <[email protected]>
Co-authored-by: ayushmaheshwari <[email protected]>
Co-authored-by: iamayushm <[email protected]>
Co-authored-by: Vikram <[email protected]>
Co-authored-by: Vikram Singh <[email protected]>
Co-authored-by: akshatsinha007 <[email protected]>
Co-authored-by: kamal-devtron <[email protected]>
Co-authored-by: Badal Kumar <[email protected]>
Co-authored-by: Pawan Kumar <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vikramdevtron vikramdevtron vikramdevtron approved these changes

@pawan-59 pawan-59 pawan-59 approved these changes

@nishant-d nishant-d Awaiting requested review from nishant-d

@vivek-devtron vivek-devtron Awaiting requested review from vivek-devtron

@prakarsh-dt prakarsh-dt Awaiting requested review from prakarsh-dt prakarsh-dt is a code owner

Assignees

No one assigned

Labels

PR:Ready-to-Review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@badal773 @prakarsh-dt @vikramdevtron @pawan-59