feat: add chaos mesh api description and table schema #633
Conversation
| maximum: 200 | ||
| default: 100 | ||
| - in: query | ||
| name: template |
There was a problem hiding this comment.
Isn't it irrelevant? all are templates
There was a problem hiding this comment.
removed - but I closed this MR please see #635
| type: boolean | ||
| example: true | ||
| - in: query | ||
| name: exclude |
There was a problem hiding this comment.
I'd just a common option in the crud sometimes not to show everything
| updated_at: | ||
| type: string | ||
| format: date-time | ||
| description: The date and time that the procesor file was updated. |
There was a problem hiding this comment.
"procesor file" not relevant in this case
| kind: | ||
| type: string | ||
| enum: | ||
| - 'dnschaos' |
There was a problem hiding this comment.
this one is small letters on purpose?
| } | ||
|
|
||
| async function insertChaosExperiment(experimentId, experiment, contextId) { | ||
| const chaosExperimentModel = client.model('chaos-experiment'); |
There was a problem hiding this comment.
shouldnt the function generate the id?
There was a problem hiding this comment.
| return _getChaosExperiment(options); | ||
| } | ||
|
|
||
| async function deleteChaosExperiment(processorId) { |
There was a problem hiding this comment.
should be experiment id and not processor id
| updated_at: { | ||
| type: Sequelize.DataTypes.DATE | ||
| }, | ||
| context_id: { |
There was a problem hiding this comment.
what is context id in this case, and why do we need it here?
There was a problem hiding this comment.
it's an header - I think it's like x-zooz.... saw it is added to some resources
Related issues: #634
Further information: