Skip to content

Conversation

@ritwiksahani
Copy link
Contributor

Draft CL to validate builds.

@ritwiksahani ritwiksahani added the build Triggers github actions build label Jun 5, 2025
@ritwiksahani ritwiksahani marked this pull request as draft June 5, 2025 05:51
Add API definition and request-response for list Upgrades.
* Added a metadata fetcher to fetch mapping from CDAP metadata.

Sample response from metadata API for a user plugin `Trash` and the pipeline `DataFusionQuickstart`
```
{
  "sort": "io.cdap.cdap.data2.metadata.dataset.SortInfo@74a24bc1",
  "offset": 0,
  "limit": 2147483647,
  "numCursors": 0,
  "total": 1,
  "results": [
    {
      "metadataEntity": {
        "details": {
          "namespace": "default",
          "artifact": "trash-plugin",
          "version": "1.2.0",
          "type": "batchsink",
          "plugin": "Trash"
        },
        "type": "plugin"
      },
      "metadata": {
        "SYSTEM": {
          "properties": {
            "default:DataFusionQuickstart": "1"
          },
          "tags": []
        }
      }
    }
  ],
  "cursors": [],
  "showHidden": false,
  "entityScope": [
    "USER"
  ]
}
```
Fix test failure due to a new API being added.
* Upgrade Manager is the class that has the core logic to read from various sources and covert the data to a list of ApplicationUpgradeDetail.
* This PR also has changes to the ApplicationUpgradeDetail class where a new class for plugin upgrade detail is created which extends ArtifactUpgradeDetail class.
Adding fetcher for plugin to pipeline mapping.
Added Upgrade manager interface and implementation.
@ritwiksahani ritwiksahani force-pushed the simplified-upgrades branch from 981084f to 4b65e74 Compare June 20, 2025 07:28
HttpHandler and Module changes for view upgrades API.
* New applications added for upgrades test were not being deleted leading to higher count being returned for list apps.
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
79.2% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Triggers github actions build

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants