tvalentyn commented on a change in pull request #16938:
URL: https://github.com/apache/beam/pull/16938#discussion_r819899475



##########
File path: website/www/site/content/en/documentation/runtime/environments.md
##########
@@ -46,7 +46,7 @@ Beam [SDK container 
images](https://hub.docker.com/search?q=apache%2Fbeam&type=i
 
 1. **[Writing a new](#writing-new-dockerfiles) Dockerfile based on a released 
container image**. This is sufficient for simple additions to the image, such 
as adding artifacts or environment variables.
 2. **[Modifying](#modifying-dockerfiles) a source Dockerfile in 
[Beam](https://github.com/apache/beam)**. This method requires building from 
Beam source but allows for greater customization of the container (including 
replacement of artifacts or base OS/language versions).
-
+3. **[Build](#modify-existing-base-image) an existing container image to make 
it compatible with Apache Beam Runners**. This method is used when users start 
from an existing image, and configure the image to be compatible with Apache 
Beam Runners.

Review comment:
       ```suggestion
   3. **[Modifying](#modify-existing-base-image) an existing container image to 
make it compatible with Apache Beam Runners**. This method is used when users 
start from an existing image, and configure the image to be compatible with 
Apache Beam Runners.
   ```
   
   Also: one of _three_ ways above.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to