Skip to content
Snippets Groups Projects

CI/CD changes | Adding helm charts

Merged Komal Makkar requested to merge adding_helm_charts into master

Type of change

  • Bug Fix
  • Feature

Please provide link to gitlab issue or ADR(Architecture Decision Record)

Does this introduce a change in the core logic?

  • [YES/NO] Yes

Does this introduce a change in the cloud provider implementation, if so which cloud?

  • AWS
  • Azure
  • GCP
  • IBM

Does this introduce a breaking change?

  • [YES/NO] No

What is the current behavior?

The exceptions from Entitlement are propagated as it is. However, ITs expects 401, so changing the core to accommodate for the same.

What is the new/expected behavior?

  • Adding helm charts
  • Updated gitlab-ci.yml
  • Added README
  • updated the ITs
  • removed logger from the request-scoped objects

Have you added/updated Unit Tests and Integration Tests?

Yes

Any other useful information

Edited by Komal Makkar

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Hema Vishnu Pola [Microsoft]
  • Komal Makkar added 6 commits

    added 6 commits

    Compare with previous version

  • Komal Makkar added 1 commit

    added 1 commit

    Compare with previous version

  • Komal Makkar added 60 commits

    added 60 commits

    Compare with previous version

  • mentioned in issue #3 (closed)

  • Komal Makkar marked as a Work In Progress

    marked as a Work In Progress

  • Komal Makkar added 4 commits

    added 4 commits

    Compare with previous version

  • Komal Makkar changed the description

    changed the description

  • Komal Makkar resolved all threads

    resolved all threads

  • Komal Makkar unmarked as a Work In Progress

    unmarked as a Work In Progress

  • Komal Makkar added 1 commit

    added 1 commit

    Compare with previous version

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading