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

Feature/distributed demo #94

Merged
merged 58 commits into from
Sep 8, 2023
Merged

Feature/distributed demo #94

merged 58 commits into from
Sep 8, 2023

Conversation

astitva1905
Copy link
Collaborator

Porting release v0.2 documentation to feature-distributed.

prasadtalasila and others added 11 commits June 25, 2023 17:17
  - Removes the unused ssl certificate generation code
  - Updates the variable and link names for consistency in
    client code
  - Adds updated presentation of the DTaaS Overview
  - Adds new badges to the STATUS.md page
  - Adds cronjobs for linux and vagrant deployments to run
    website and lib microservice in daemon mode.
  - Adds cronjob to remote the conflicting default route from
    vagrant machines
  - Fixes playwright tests of client
  - Adds new single script install. This script
    can be used to test the installation of
    complete DTaaS application.
  - Restructures the shell scripts in script/
    and deploy/ directories to reduce duplication.
  - Updates project README.md file.
  - Updates the vagrant installation scripts and instructions
  - Deletes unused scripts.
  - Fixes mistakes in mkdocs.yml file
  - Disables docs pdf generation in github actions
@codecov-commenter
Copy link

codecov-commenter commented Sep 3, 2023

Codecov Report

Merging #94 (2f1bc4f) into feature/distributed-demo (9e9d945) will not change coverage.
The diff coverage is n/a.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@                    Coverage Diff                    @@
##           feature/distributed-demo      #94   +/-   ##
=========================================================
  Coverage                     64.37%   64.37%           
=========================================================
  Files                            38       38           
  Lines                           494      494           
  Branches                         28       28           
=========================================================
  Hits                            318      318           
  Misses                          160      160           
  Partials                         16       16           
Components Coverage Δ
Website 62.72% <ø> (ø)
Lib Microservice 69.91% <ø> (ø)

📢 Have feedback on the report? Share it here.

deploy/README.md Outdated Show resolved Hide resolved
deploy/install.sh Outdated Show resolved Hide resolved
deploy/install.sh Outdated Show resolved Hide resolved
deploy/install.sh Outdated Show resolved Hide resolved
deploy/install.sh Outdated Show resolved Hide resolved
docs/user/servers/lib/LIB-MS.md Outdated Show resolved Hide resolved
packages/client/README.md Outdated Show resolved Hide resolved
script/env.sh Show resolved Hide resolved
script/env.sh Outdated Show resolved Hide resolved
ssl/README.md Outdated Show resolved Hide resolved
@prasadtalasila
Copy link
Contributor

prasadtalasila commented Sep 3, 2023

@astitva1905
When you are done please use the docs/PUBLISH.md to publish the docs to your webpage-docs branch. You can use this branch to host project docs. The same technique is used to host the DTaaS documentation.
I can check your docs page to see the final result of changes. Thanks.

Copy link
Contributor

@prasadtalasila prasadtalasila left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@astitva1905 I have checked upto docs/admin/vagrant/base-box.md. Please check the suggested changes. Thanks.

deploy/README.md Outdated Show resolved Hide resolved
deploy/README.md Outdated Show resolved Hide resolved
deploy/README.md Outdated Show resolved Hide resolved
deploy/vagrant/single-machine/README.md Outdated Show resolved Hide resolved
deploy/vagrant/two-machine/README.md Outdated Show resolved Hide resolved
deploy/vagrant/two-machine/Vagrantfile Outdated Show resolved Hide resolved
docs/admin/client/CLIENT.md Outdated Show resolved Hide resolved
docs/admin/host.md Outdated Show resolved Hide resolved
docs/admin/servers/lib/LIB-MS.md Outdated Show resolved Hide resolved
Copy link
Contributor

@prasadtalasila prasadtalasila left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@astitva1905 The review is now complete. Thanks for making the changes.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please revert the change in file permissions.

Copy link
Collaborator Author

@astitva1905 astitva1905 Sep 8, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What should the final file permissions be ?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please revert the change in file permissions. You may have to set the default file permissions in the windows git client. Otherwise, all file permission changes get reflected as a new changes in files. It is not desirable.

servers/lib/README.md Outdated Show resolved Hide resolved
docs/admin/vagrant/single-machine.md Outdated Show resolved Hide resolved
docs/admin/vagrant/two-machines.md Outdated Show resolved Hide resolved
docs/admin/vagrant/two-machines.md Outdated Show resolved Hide resolved
@prasadtalasila prasadtalasila merged commit 9f8d5dd into INTO-CPS-Association:feature/distributed-demo Sep 8, 2023
3 checks passed
prasadtalasila added a commit that referenced this pull request Sep 8, 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.

3 participants