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

Flights #1

Merged
merged 35 commits into from
Jul 8, 2016
Merged

Flights #1

merged 35 commits into from
Jul 8, 2016

Conversation

lakshmanok
Copy link
Contributor

Chapters 1-8

gstripling pushed a commit that referenced this pull request Jun 11, 2020
gstripling pushed a commit that referenced this pull request Jun 22, 2020
gstripling pushed a commit that referenced this pull request Jul 14, 2020
gstripling pushed a commit that referenced this pull request Jul 14, 2020
gstripling pushed a commit that referenced this pull request Jul 15, 2020
gstripling pushed a commit that referenced this pull request Jul 21, 2020
gstripling pushed a commit that referenced this pull request Jul 28, 2020
gstripling pushed a commit that referenced this pull request Sep 15, 2020
gstripling pushed a commit that referenced this pull request Sep 21, 2020
priumoraes pushed a commit that referenced this pull request Sep 30, 2020
gstripling pushed a commit that referenced this pull request Oct 15, 2020
dylanmpeck pushed a commit that referenced this pull request Jan 18, 2021
gstripling pushed a commit that referenced this pull request Mar 2, 2021
wkharold pushed a commit that referenced this pull request May 10, 2021
gstripling pushed a commit that referenced this pull request Jun 18, 2021
ghost pushed a commit that referenced this pull request Sep 3, 2021
philmaier pushed a commit that referenced this pull request Feb 2, 2022
GSP354 - vertex-challenge-lab.ipynb
hrishikesh-kadam added a commit to hrishikesh-kadam/training-data-analyst that referenced this pull request Feb 28, 2022
Lab - https://www.cloudskillsboost.google/course_sessions/812316/labs/190664
Video - https://www.cloudskillsboost.google/course_sessions/812316/video/190665
Course - https://www.cloudskillsboost.google/course_templates/2

In the Lab, Task 4 needs the correction.
Currently, the Task 4 is expected to fail but as below.

Expected -
```
Finished Step GoogleCloudPlatform#1
ERROR
ERROR: build step 1 "gcr.io/qwiklabs-gcp-04-9d788c36b0e6/quickstart-image" failed: step exited with non-zero status: 1
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------

BUILD FAILURE: Build step failure: build step 1 "gcr.io/qwiklabs-gcp-04-9d788c36b0e6/quickstart-image" failed: step exited with non-zero status: 1
```

The Task 4 is incorrectly failing due to the incorrect argument passed
to the cloudbuild.yaml
The arguments only contains the ['fail'] instead of the
['./quickstart.sh', 'fail']
As there is no mention of ENTRYPOINT in Dockerfile as well as in
cloudbuild.yaml, the passed arguments overrides the CMD in the
Dockerfile[1].

Also, I have updated the executable bit of all the quickstart.sh

References -

[1]: 4th Point of https://docs.docker.com/engine/reference/builder/#understand-how-cmd-and-entrypoint-interact
hrishikesh-kadam added a commit to hrishikesh-kadam/training-data-analyst that referenced this pull request Feb 28, 2022
Lab - https://www.cloudskillsboost.google/course_sessions/812316/labs/190664
Video - https://www.cloudskillsboost.google/course_sessions/812316/video/190665
Course - https://www.cloudskillsboost.google/course_templates/2

In the Lab, Task 4 needs the correction.
Currently, the Task 4 is expected to fail but as below.

Expected -
```
Finished Step GoogleCloudPlatform#1
ERROR
ERROR: build step 1 "gcr.io/qwiklabs-gcp-04-9d788c36b0e6/quickstart-image" failed: step exited with non-zero status: 1
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------

BUILD FAILURE: Build step failure: build step 1 "gcr.io/qwiklabs-gcp-04-9d788c36b0e6/quickstart-image" failed: step exited with non-zero status: 1
```

The Task 4 is incorrectly failing due to the incorrect argument passed
to the cloudbuild.yaml
The arguments only contains the ['fail'] instead of the
['./quickstart.sh', 'fail']
As there is no mention of ENTRYPOINT in Dockerfile as well as in
cloudbuild.yaml, the passed arguments overrides the CMD in the
Dockerfile[1].

I have also updated the executable bit of all the quickstart.sh

As a suggestion in the Lab, I think it would have been more appropriate
if after building and pushing the images to gcr.io, the Lab also teach
about how to pull and run the images using docker command.

References -

[1]: 4th Point of https://docs.docker.com/engine/reference/builder/#understand-how-cmd-and-entrypoint-interact
hrishikesh-kadam added a commit to hrishikesh-kadam/training-data-analyst that referenced this pull request Feb 28, 2022
Lab - https://www.cloudskillsboost.google/course_sessions/812316/labs/190664
Video - https://www.cloudskillsboost.google/course_sessions/812316/video/190665
Course - https://www.cloudskillsboost.google/course_templates/2

In the Lab, Task 4 needs the correction.
Currently, the Task 4 is expected to fail but as below.

Expected -
```
Finished Step GoogleCloudPlatform#1
ERROR
ERROR: build step 1 "gcr.io/qwiklabs-gcp-04-9d788c36b0e6/quickstart-image" failed: step exited with non-zero status: 1
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------

BUILD FAILURE: Build step failure: build step 1 "gcr.io/qwiklabs-gcp-04-9d788c36b0e6/quickstart-image" failed: step exited with non-zero status: 1
```

The Task 4 is incorrectly failing due to the incorrect argument passed
to the cloudbuild.yaml
The arguments only contains the ['fail'] instead of the
['/quickstart.sh', 'fail']
As there is no mention of ENTRYPOINT in Dockerfile as well as in
cloudbuild.yaml, the passed arguments overrides the CMD in the
Dockerfile[1].

I have also updated the executable bit of all the quickstart.sh

As a suggestion in the Lab, I think it would have been more appropriate
if after building and pushing the images to gcr.io, the Lab also teach
about how to pull and run the images using docker command.

References -

[1]: 4th Point of https://docs.docker.com/engine/reference/builder/#understand-how-cmd-and-entrypoint-interact
clupone-g pushed a commit that referenced this pull request Aug 11, 2022
gstripling pushed a commit that referenced this pull request Aug 30, 2022
gstripling pushed a commit that referenced this pull request Sep 6, 2022
gstripling pushed a commit that referenced this pull request Sep 20, 2022
rosera pushed a commit that referenced this pull request Jan 13, 2023
aiplatform package installation, kernel restart code added
0yster pushed a commit that referenced this pull request Apr 18, 2023
Notebook Update: GSP013 - train_deploy.ipynb
Yoanna-Long pushed a commit that referenced this pull request Aug 1, 2023
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