Skip to content

Conversation

@Ash-exp
Copy link
Contributor

@Ash-exp Ash-exp commented Nov 27, 2023

Description

After enabling artifacts V2 flag, get material list API is breaking for external CI and job CI (Throwing error: pg: no rows in result set)

Fixes #4290

How Has This Been Tested?

  • Job CI
    • Approval list
    • Pre CD list
    • CD list
    • Post CD list
  • External CI
    • Approval list
    • Pre CD list
    • CD list
    • Post CD list
  • Build CI
    • Approval list
    • Pre CD list
    • CD list
    • Post CD list
  • Link CI
    • Approval list
    • Pre CD list
    • CD list
    • Post CD list

Checklist:

  • The title of the PR states what changed and the related issues number (used for the release note).
  • Does this PR requires documentation updates?
  • I've updated documentation as required by this PR.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have tested it for all user roles.
  • I have added all the required unit/api test cases.

Does this PR introduce a user-facing change?


@Ash-exp Ash-exp added the enhancement New feature or request label Nov 27, 2023
@Ash-exp Ash-exp requested a review from iamayushm November 27, 2023 10:11
@Ash-exp Ash-exp self-assigned this Nov 27, 2023
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@Ash-exp Ash-exp merged commit 4bb931d into main Nov 27, 2023
@Ash-exp Ash-exp deleted the fix-artifacts-list branch November 27, 2023 12:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request PR:Ready-to-Review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: Not able to view images in case of External CI

4 participants