Skip to content
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.

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

[patch] Enable imagescan support in all applications #933

Merged
merged 4 commits into from May 2, 2024
Merged

Conversation

darshan-kansara1
Copy link
Contributor

@darshan-kansara1 darshan-kansara1 commented May 2, 2024

Epic : https://jsw.ibm.com/browse/MASCORE-881
Story : https://jsw.ibm.com/browse/MASCORE-954

Description
Adding a support of imagescan in below applications.

  • Assist
  • IOT
  • Manage
  • Monitor
  • Optimizer
  • Predict
  • Visual inspection

Related Issues
Depends on : https://jsw.ibm.com/browse/MASCORE-953
Depended on by : https://jsw.ibm.com/browse/MASCORE-956

@@ -78,6 +78,18 @@ spec:
- input: "$(params.mas_app_channel_assist)"
operator: notin
values: [""]
- name: imagescan
Copy link
Contributor

Choose a reason for hiding this comment

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

the name needs to be seet to imagescan-{appId}, e.g. imagescan-assist in this case .. same applies to all the other files as well

params:
{{ lookup('template', 'taskdefs/ivt-core/common/params.yml.j2') | indent(8) }}
- name: product_id
value: ibm-mas
Copy link
Contributor

Choose a reason for hiding this comment

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

product_id needs to be updated too ... you can get the product_id value from the entry for ivtcore-xxx

Also need to add entry for product_channel same as we have for ivtcore too (and check it's not missing for the core one while we're doing this) ...

        - name: product_channel
          value: $(params.mas_app_channel_assist)

@@ -26,3 +26,15 @@ spec:
# 1. ADD FVT
# -------------------------------------------------------------------------
{{ lookup('template', 'taskdefs/fvt-data-dictionary/data-dictionary.yml.j2') | indent(4) }}
- name: imagescan
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think we can add this for data dictionary yet .. it's not a real MAS app, would leave it out for now ... get the apps done first. this will need special handling

@@ -32,3 +32,15 @@ spec:
# 1. SLS FVT
# -------------------------------------------------------------------------
{{ lookup('template', 'taskdefs/fvt-sls/sls.yml.j2') | indent(4) }}
- name: imagescan
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think we can add this for sls yet .. it's not a real MAS app, would leave it out for now ... get the apps done first. this will need special handling

@durera durera changed the title Added imagescan support in applications [patch] Enable imagescan support in all applications May 2, 2024
@durera durera merged commit 52f6f58 into master May 2, 2024
3 checks passed
@durera durera deleted the mascore-854-v3 branch May 2, 2024 17:10
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.

None yet

2 participants