-
Notifications
You must be signed in to change notification settings - Fork 3
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
update WCMP2 exercise based on config creation #90
Conversation
documentation/docs/practical-sessions/configuring-wis2-discovery-metadata.md
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe add a comment that they edit the discovery metadata files using WinSCP if they are not familiar with vi ?
+1 to add comment about using WinSCP, I need to do the same in my exercises where I tell the student to edit a file. |
Fixed. |
@@ -102,10 +99,13 @@ Click on your discovery metadata record and inspect the content, noting how it r | |||
Update the title of your discovery metadata, and re-publish: | |||
|
|||
```bash | |||
vi /data/wis2box/my-discovery-metadata.yml | |||
wis2box metadata discovery publish /data/wis2box/my-discovery-metadata.yml | |||
vi /data/metadata/discovery/metadata-synop.yml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
path?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed (edit outside wis2box-management, but run inside).
Now re-publish (from inside the **wis2box-management** container): | ||
|
||
```bash | ||
wis2box metadata discovery publish /data/metadata/discovery/metadata-synop.yml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/data/wis2box/metadata/discovery/metadata-synop.yml?
@@ -84,7 +81,7 @@ python3 wis2box-ctl.py login | |||
Run the following command to publish your discovery metadata: | |||
|
|||
```bash | |||
wis2box metadata discovery publish /data/wis2box/my-discovery-metadata.yml | |||
wis2box metadata discovery publish /data/metadata/discovery/metadata-synop.yml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wis2box metadata discovery publish /data/wis2box/metadata/discovery/metadata-synop.yml
Update the title of your discovery metadata: | ||
|
||
```bash | ||
vi ~/wis2box-data/metadata/discovery/metadata-temp.yml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
At the moment the student is still logged into the management container so the path is wrong. Change to
vi /data/wis2box/metadata/discovery/metadata-synop.yml
@@ -121,7 +126,7 @@ Feel free to update additional values and re-publishing your discovery metadata | |||
Run the below command to add the data to the API: | |||
|
|||
```bash | |||
wis2box data add-collection /data/wis2box/my-discovery-metadata.yml | |||
wis2box data add-collection /data/metadata/discovery/metadata-synop.yml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wis2box data add-collection /data/wis2box/metadata/discovery/metadata-synop.yml
* update WCMP2 exercise based on config creation * add YYYY-MM-DD for BEGIN DATE * add tip for WinSCP usage * add tip for WinSCP usage * update * pathing fixes Co-authored-by: Tom Kralidis <[email protected]>
No description provided.