confidential-containers / operator

Operator to deploy confidential containers runtime
Apache License 2.0
113 stars 60 forks source link

CI Documentation - Add AMD nodes and labels to jobs table #317

Open ryansavino opened 10 months ago

ryansavino commented 10 months ago

docs: added column for labels to the CI jobs table docs: Added snp and sev job names and labels to CI documentation

Fixes: #316

Signed-Off-By: Ryan Savino ryan.savino@amd.com

wainersm commented 10 months ago

hi @ryansavino , I think that a separated table only to map label -> machine would be better.

portersrc commented 10 months ago

lgtm; wainer's comment is fine, too, if you think it would help

ldoktor commented 10 months ago

Looks good to me as is in the single table (it's not that wide) although I'm having troubles actually finding the pipeline definitions (tried searching for the name as well as the labels). Are they supposed to be already there?

ryansavino commented 9 months ago

I've updated this PR with the machine table as well. @wainersm let me know what you think. I can add other system labels if you guys know what they are. @BbolroC @stevenhorsman @fidencio

Question: default CcRuntime is referring to runtime class? Should I add that column in the job table?

@ldoktor the AMD nodes are already labeled.

fidencio commented 3 months ago

I can add other system labels if you guys know what they are. @BbolroC @stevenhorsman @fidencio

Please, if possible, do.

image

So, for the TDX machine the label is tdx, for the s390x machine the label is S390X.