2021-02-10 00:55:29 UTC - Dominic Kim: Seem quite many things happened during 
the night.
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1612918529146500
----
2021-02-10 00:55:37 UTC - Dominic Kim: @Matt Rutkowski Let me figure it out
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1612918537146800
----
2021-02-10 01:43:06 UTC - Dominic Kim: ok so the main issue was a change in the 
core repo was not applied to other dependent repos.
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1612921386147600?thread_ts=1612921386.147600&cid=C3TPCAQG1
----
2021-02-10 01:44:45 UTC - Dominic Kim: @Matt Rutkowski Thank you for 
investigating the issue on my behalf. It was supposed to be me I think.
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1612921485148400
----
2021-02-10 01:49:54 UTC - Dominic Kim: I believe this can happen again with 
similar changes in the core repo. Currently, the recommended way is to apply 
the changes to dependent repos quickly enough.
Until then, there will be a mismatch and the build will be failed in the 
meantime.
And someone should remember they are dependent and the change should be applied 
too.
But I feel like it's error-prone and easy to forget. We need to automate the 
procedure.
If other repos have such static `hosts` file just to install OW locally, how 
about rather generating it according to the core repo ansible definition.
It seems they are already cloning the core repo during CI build step.
<https://github.com/apache/openwhisk-wskdeploy/blob/master/tools/travis/script.sh#L39>
<https://github.com/apache/openwhisk-runtime-swift/blob/master/tools/travis/setup.sh#L32>
I am curious if there are other reasons for the static `hosts` file.
If not, I am willing to contribute this change to all dependent repos.
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1612921794152100
----
2021-02-10 04:28:10 UTC - Matt Rutkowski: @Dominic Kim  and now the next layer 
of the onion…
```TASK [write test's application conf overrides] 
*********************************

Tuesday 09 February 2021  23:12:45 +0000 (0:00:00.225)       0:00:02.834 ****** 

fatal: [ansible]: FAILED! =&gt; {"changed": false, "msg": 
"AnsibleUndefinedVariable: 'dict object' has no attribute 'etcd'"}

AnsibleUndefinedVariable: 'dict object' has no attribute 'etcd'```
It appears due to PR <https://github.com/apache/openwhisk/pull/5031> we must 
now add etcd as well?  i.e.,
```etcd0            ansible_host=172.17.0.1 ansible_connection=local```
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1612931290153400?thread_ts=1612931290.153400&cid=C3TPCAQG1
----
2021-02-10 04:29:02 UTC - Matt Rutkowski: guessing at this new host entry right 
now (as its 10:30pm here, but will try to add tomorrow and see if this is the 
last layer to peel)
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1612931342154100
----
2021-02-10 04:31:18 UTC - Matt Rutkowski: This is the pattern established with 
all repos. as they came out of IBM research; all new repos. just copied the 
structure of the directories and the working Travis.  Following the old axiom, 
I am sure, “Do something. *If it works*, do more of it.” :wink:
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1612931478154400?thread_ts=1612921386.147600&cid=C3TPCAQG1
----
2021-02-10 05:16:30 UTC - Dominic Kim: Yes that is also newly introduced.
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1612934190154900?thread_ts=1612931290.153400&cid=C3TPCAQG1
----
2021-02-10 05:17:10 UTC - Dominic Kim: How do you think about generating the 
hosts file from the core repo?
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1612934230155100?thread_ts=1612931290.153400&cid=C3TPCAQG1
----
2021-02-10 05:17:32 UTC - Dominic Kim: I could pass the CI tests with this 
change: <https://github.com/apache/openwhisk-wskdeploy/pull/1112>
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1612934252155300?thread_ts=1612931290.153400&cid=C3TPCAQG1
----
2021-02-10 15:21:34 UTC - Matt Rutkowski: Thanks for updating my PR!  I agree 
we should use the hosts file from the core repo. where possible
https://openwhisk-team.slack.com/archives/C3TPCAQG1/p1612970494155500?thread_ts=1612931290.153400&cid=C3TPCAQG1
----

Reply via email to