Skip to content
This repository was archived by the owner on Mar 31, 2023. It is now read-only.

[Deployment] DPM and NCM to use standalone Ignite DB. #732

Merged
xieus merged 53 commits into from
Jan 19, 2022
Merged

[Deployment] DPM and NCM to use standalone Ignite DB. #732

xieus merged 53 commits into from
Jan 19, 2022

Conversation

@pkommoju
Copy link
Contributor

@pkommoju pkommoju commented Jan 13, 2022 •
edited by xieus
Loading

  • DPM and NCM to use their own Ignite DB.
  • Jenkins run passed. Needs to be validated in Kuberentes before merge.
  • After validation, before merge into master the CONFIG_URI needs to be changed to point to ignite config files in master branch of alcor repository.
  • NMM to NCM communication through REST API is broken again. This is preventing node creation in NMM from propagating to NCM.

pkommoju and others added 30 commits May 12, 2021 11:34
@pkommoju
Documentation update
9645772 
Add document about transactional semantics for Alcor Caches in general
and spefically about Ignite Caches.
@pkommoju
Apply changes suggested in review
183c36a
@pkommoju
Merge branch 'master' of https://github.com/futurewei-cloud/alcor
4896515
@pkommoju
Merge branch 'master' of https://github.com/futurewei-cloud/alcor
f170e1e
@pkommoju
Merge branch 'master' of https://github.com/futurewei-cloud/alcor
bcbf7b1
@pkommoju
Merge branch 'master' of https://github.com/futurewei-cloud/alcor
efc5b6e
@pkommoju
Merge branch 'master' of https://github.com/futurewei-cloud/alcor
c785697
Merge branch 'master' of https://github.com/pkommoju/alcor
d213c03
@pkommoju
Merge branch 'master' of https://github.com/futurewei-cloud/alcor
0f56454
@pkommoju
Merge branch 'master' of https://github.com/pkommoju/alcor
4a2a382
@pkommoju
Merge branch 'master' of https://github.com/futurewei-cloud/alcor
7f93fe3
@pkommoju
Merge branch 'master' of https://github.com/futurewei-cloud/alcor
8d6dd0e
@kevin-zhonghao
Merge branch 'master' of https://github.com/futurewei-cloud/alcor
b19d428
@pkommoju
Merge branch 'master' of https://github.com/futurewei-cloud/alcor
539d0c0
@pkommoju
Merge branch 'master' of https://github.com/pkommoju/alcor
6a9a1c6
@pkommoju
Merge branch 'master' of https://github.com/futurewei-cloud/alcor
19430b2
@pkommoju
Merge branch 'master' of https://github.com/futurewei-cloud/alcor
1d7ef7e
@pkommoju
Merge branch 'master' of https://github.com/futurewei-cloud/alcor
cd1fec6
@pkommoju
Merge branch 'master' of https://github.com/futurewei-cloud/alcor
e57abe4
@pkommoju
Add missing file
1b63054
@pkommoju
Merge branch 'master' of https://github.com/futurewei-cloud/alcor
fe81008
@pkommoju
Merge branch 'master' of https://github.com/futurewei-cloud/alcor
118dd53
@pkommoju
Merge branch 'master' of https://github.com/futurewei-cloud/alcor
7d9a2e5
@pkommoju
Merge branch 'master' of https://github.com/futurewei-cloud/alcor
3884a44
@pkommoju
Merge branch 'master' of https://github.com/futurewei-cloud/alcor
0b5cd11
@pkommoju
Merge branch 'master' of https://github.com/futurewei-cloud/alcor
3f094be
@pkommoju
Merge branch 'master' of https://github.com/futurewei-cloud/alcor
bb777ec
@pkommoju
Merge branch 'master' of https://github.com/futurewei-cloud/alcor
c0e8c2d
@pkommoju
Merge branch 'master' of https://github.com/futurewei-cloud/alcor
206abc7
@pkommoju
Merge branch 'master' of https://github.com/futurewei-cloud/alcor
fab8a00
DavidLiu506
DavidLiu506 reviewed Jan 18, 2022
View reviewed changes
cj-chung
cj-chung reviewed Jan 18, 2022
View reviewed changes
@cj-chung cj-chung self-requested a review January 19, 2022 19:20
cj-chung
cj-chung approved these changes Jan 19, 2022
View reviewed changes
Copy link
Contributor

@cj-chung cj-chung left a comment

Choose a reason for hiding this comment

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

LGTM

@xieus xieus changed the title DPM and NCM to use their own Ignite DB. [Deployment] DPM and NCM to use standalone Ignite DB. Jan 19, 2022
@xieus xieus added this to the Version 1.0.2022.01.30 milestone Jan 19, 2022
@xieus
Copy link
Contributor

xieus commented Jan 19, 2022

@pkommoju and @cj-chung There is a file conflict after PR #733 is merged. Please resolve it.

@cj-chung
Copy link
Contributor

cj-chung commented Jan 19, 2022

@pkommoju please pull master to your branch, @yanmo96's branch is already merged to master and the kubernetes/createYaml.sh is conflict with your branch.

yanmo96
yanmo96 reviewed Jan 19, 2022
View reviewed changes
kubernetes/createYaml.sh Outdated

echo "Create yaml files done"

kubectl exec -it ignite-alcor-dpm-0 -n ignite-alcor-dpm -c ignite-alcor-dpm-node -- /opt/ignite/apache-ignite/bin/control.sh --activate
Copy link
Contributor

@yanmo96 yanmo96 Jan 19, 2022

Choose a reason for hiding this comment

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

Hi @pkommoju can you move those two command to line 51 and 52. Thanks

@xieus xieus merged commit 6d727ac into futurewei-cloud:master Jan 19, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

3 more reviewers

@yanmo96 yanmo96 yanmo96 left review comments

@cj-chung cj-chung cj-chung approved these changes

@DavidLiu506 DavidLiu506 DavidLiu506 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@pkommoju pkommoju

Labels

deployment and upgrade k8s

Projects

None yet

Milestone

Version 1.0.2022.01.30

Development

Successfully merging this pull request may close these issues.

6 participants

@pkommoju @xieus @cj-chung @yanmo96 @DavidLiu506 @kevin-zhonghao