aws-genai-llm-chatbot icon indicating copy to clipboard operation
aws-genai-llm-chatbot copied to clipboard

Condition disappear for Idefics activation

Open hghandri opened this issue 1 year ago • 5 comments

Hi,

since release version 4.0.3 it seems there is condition disappeared for Idefics Interface activation or not.

You can compare this latest version with previous one like v4.0.3 at the same line :

https://github.com/aws-samples/aws-genai-llm-chatbot/blob/3c710bf28bd74eb3b576b863a4b678ed8ef06e0b/lib/aws-genai-llm-chatbot-stack.ts#L110

is it normal or a mistake ? In my case I cannot not deploy anymore due this new resource not available before and it need a default security group ID for private VPC endpoint. This happened when VPC already exist, it seems the method "ec2.Vpc.fromLookup" does not detect a default security group.

Here the impact :

https://github.com/aws-samples/aws-genai-llm-chatbot/blob/main/lib/model-interfaces/idefics/index.ts#L41

This property "props.shared.vpc.vpcDefaultSecurityGroup" should contains a default security group ID. For an existing VPC this value seems to be blank.

This condition impact also ResourcePath for NagSuppressions.

Best regards,

hghandri avatar Apr 26 '24 14:04 hghandri

Hi,

anyone could check this issue ?

hghandri avatar Apr 30 '24 09:04 hghandri

Hello @hghandri i had the same error when using the existing VPC with Idefics Interface can not find the default security group ID. Do you find the solution about this problem ?

KietNgoTuan avatar May 17 '24 18:05 KietNgoTuan

Hi @KietNgoTuan

I've changed a little bit the code to handle this issue. I pushed a branch on my fork repository : https://github.com/hghandri/aws-genai-llm-chatbot/tree/fix/477-condition-disappear-idefics-activation

That change need to add a configuration in bin/config.json file like this :

You have to set explicitly your default Security Group.

"vpc": {
    "vpcId": "vpc-xxxxx",
    "createVpcEndpoints": false,
    **"vpcDefaultSecurityGroup": "sg-xxxxx"**
  },

++

hghandri avatar Jun 06 '24 08:06 hghandri

I've just opened a PR : #498

hghandri avatar Jun 06 '24 08:06 hghandri

This issue is stale because it has been open for 60 days with no activity.

github-actions[bot] avatar Aug 06 '24 01:08 github-actions[bot]

This issue was closed because it has been inactive for 30 days since being marked as stale.

github-actions[bot] avatar Sep 05 '24 02:09 github-actions[bot]