First version

First version is expected to have at least following:

React-table
Look and feel of Chakra UI table should follow Chakra Pro https://pro.chakra-ui.com/components/application/tables.
Source code below:
TableWithSearch.zip
Using i18next on all table headers and labels:

Accept a prop apiUrl, if it's missing show an error message saying "apiUrl is missing or malformed".  If it's available and is valid load data from the URL using GET request and then display.
API response you can have a sample from the demo server I provided to you earlier http://<IP_ADDRESS>:8081/api/g/subarea/all.  This is also the sample API you can use against for development.
You should also expect a prop "columns" to be provided, sample you can check from old version's Storybook https://github.com/bsol-oss/react-datatable/tree/main/src/components/DataTableSever/stories
The major change against old version, is we want the component to be easily composable, instead of 1 big components with tons of props.  What it means is we want individual components to be exported, so it can be used like this:

import {DataTable, Controls, SearchBar, DisplayModeButtons, TableHeaders, TableData, DataLoadingSkeleton, Footer, PaginationControl, DataStats} from '@bsol-oss/react-datatable2'

<DataTable>
  <Controls>
    <SearchBar />
    <DisplayModeButtons enabledView={['ROW', 'ROWCONDENSED']} />
  </Controls>
  <TableHeaders columns={columns} />
  <TableData
    apiUrl="xxx"
    onLoad={DataLoadingSkeleton}
  />
  <Footer>
    <PaginationControl />
    <DataStats />
  </Footer>
</DataTable>

Note: Above is JUST AN EXAMPLE.  But it highlights we want to revamp the component to be Chakra UI style composable and flexible for all the different use cases.
You should refer to "DataTableServer" https://github.com/bsol-oss/react-datatable/tree/main/src/components/DataTableSever, not "DataTable"

Hi @Risk-taker0501 please refer to above for a summary of the first version we are looking for.  I estimate it may take a few days to absorb and re-write into the above mode.  I recommend re-write to avoid bringing old issues to new code.
But the old code is having 95%+ features we wants and it's 70% there except being bulky.
In the next few days I will add other tasks to highlight the changes we need to put in after the first version,  thanks!

Okay, I got it. I will follow your idea.
would you use Slack for our communication?
It will be better.
On Fri, 9 Jun 2023 at 00:03, Ross Chiu @.***> wrote:

Hi @Risk-taker0501 https://github.com/Risk-taker0501 please refer to
above for a summary of the first version we are looking for. I estimate it
may take a few days to absorb and re-write into the above mode. I recommend
re-write to avoid bringing old issues to new code.
But the old code is having 95%+ features we wants and it's 70% there
except being bulky.
In the next few days I will add other tasks to highlight the changes we
need to put in after the first version, thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/bsol-oss/react-datatable2/issues/1#issuecomment-1582760310,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A63FEAVDCDSE4CNKYKKSGMTXKHSSLANCNFSM6AAAAAAY7NR6V4
.
You are receiving this because you were mentioned.Message ID:
@.***>

I created Chakra DataTable that you sent me.
And divided 3 individual components(searchWrapper, MainTable, PageWapper) and use like this:


Fallback (WIP)

To add a little more chaos, this PR adds a check to ensure that there has been a new PR merged within the last 24 hours. If this is not the case, the most recent merge will be reverted.
Two reasons for this:

Encourages continued improvement, and adds a fun new time pressure to democracy
Potentially protects against PRs that have stopped new ones being picked up

TODO:
Would be interested in peoples thoughts on the best way to revert the last commit via api (going to bed now...), or any thoughts on this PR in general. 👋

I mean it fits the name chaos. But the thought of adding a commit revert wind up clock makes me uncomfortable.
However I am interested in the idea of being able to revert a commit via api

@MINIMAN10000 I can't say I'd ever want it to happen myself, but I like the idea of the threat being there :stuck_out_tongue

(updated description & title)


i18n.php changed location

do we need a composer update too?

do we need a composer update too?

IMHO no

:+1:

meant composer install /-:
:+1:


SSO Realm Openshift Config (more)

openshift sso updates for bceid
some more realm conversion

Overview
This PR includes the following proposed change(s):

{List all the changes, if possible add the linked issue/ticket #}

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Refactoring / Documentation
[ ] Version change

if your change is a breaking change, please add breaking change label to this PR
How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Screenshots
Please add any relevant UI screenshots if applicable.
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] New and existing unit tests pass locally with my changes

/rebase


Extended bar is out when returning to Search Results

When you go back to the search results (press back or home), the extended bar is out. This is correct when you're at the top of the results, but not when it returns you to the middle of them.
Possible solutions are to scroll to the top or change how extended bar handles back/home

Could possible be fixed with #9


Source a single file via a command in Solution Explorer

This basically amounts to executing "source(...)" in the REPL.
Note that with https://rtvs.visualstudio.com/DefaultCollection/RTVS/_workitems/edit/174, there's no need to have two separate commands to run with or without debugging

This is implemented already.


Fix "Add a network" option in menu

When I select "Add a network", the menu simply disappears and I don't see any error message in the console:

@markmhx Am in V2.16 Chrome and is able to view the  add network popup,may I know which version and browser did you find the issue with?Thx

Test passed and the network modal is looking good esp the ability to delete the new network modal which was custom
https://user-images.githubusercontent.com/38872890/134065487-07f679a2-1990-4d58-a1e0-75185be9469d.mov
added
Quick finding:

When user adds new network and view the network menu,the new network is not added until you have a network toast popup on the explorer.
Can we hard code https:// for the url section as if the user tends to copy paste the url,they are thrown with error of the missing https ://

When user adds new network and view the network menu,the new network is not added until you have a network toast popup on the explorer.

I think this looks ok ...maybe you were just removing/adding networks real quickly. I don't think this will be an issue for most users.

Can we hard code https:// for the url section as if the user tends to copy paste the url,they are thrown with error of the missing https ://

This could definitely be an enhancement if you want to create another issue for it? I don't think I will add it to this work bc there are already a lot of code changes and I'd like to get it live asap.
Thanks for the feedback!


YJDH-69 | Eauthorizations OIDC flow

Description :sparkles:
Run local django server with SSL sertificate to get https callback urls that are required by suomi.fi eauthorizations. Also running the localhost as https also helps to bring the local environment closer to staging / production.
Add functionality for the suomi.fi eauthorizations flow. Upon successful helsinki profiili authentication, redirect the user straight to the suomi.fi valtuudet authorization init view that first handles the user registration for the valtuudet API and then redirects the user to the valtuudet authentication flow to choose the company on whose behalf they are doing business.
After receiving the "code" in EauthAuthenticationCallbackView, switch that to the auth tokens from the suomi.fi token endpoint. Save the tokens to EAuthorizationProfile for later use and to fetch the organization roles (and the business id) with oidc.utils.get_organization_roles.
Refs YJDH-69
Issues :bug:
YJDH-69
Testing :alembic:
Screenshots :camera_flash:
Additional notes :spiral_notepad:
TODO:
Use oidc.utils.get_organization_roles function to fetch the company business id to be used with the YTJ integration.

Codecov Report

Merging #74 (2663063) into main (ce1a927) will increase coverage by 34.04%.
The diff coverage is 92.09%.

@@             Coverage Diff             @@
##             main      #74       +/-   ##
===========================================
+ Coverage   61.38%   95.43%   +34.04%
===========================================
  Files         121       43       -78
  Lines        1753     1116      -637
  Branches      108        0      -108
===========================================
- Hits         1076     1065       -11
+ Misses        649       51      -598
+ Partials       28        0       -28

Impacted Files
Coverage Δ

backend/kesaseteli/oidc/tests/test_auth_backend.py
100.00% <ø> (ø)

backend/kesaseteli/oidc/views/hki_views.py
90.19% <71.42%> (-3.29%)
:arrow_down:

backend/kesaseteli/oidc/views/eauth_views.py
84.28% <84.28%> (ø)

backend/kesaseteli/oidc/services.py
97.36% <88.88%> (-2.64%)
:arrow_down:

backend/kesaseteli/kesaseteli/settings.py
91.78% <100.00%> (+0.60%)
:arrow_up:

backend/kesaseteli/oidc/tests/test_eauth_views.py
100.00% <100.00%> (ø)

backend/kesaseteli/oidc/tests/test_oidc_views.py
100.00% <100.00%> (ø)

backend/kesaseteli/oidc/urls.py
88.88% <100.00%> (+1.38%)
:arrow_up:

backend/kesaseteli/oidc/utils.py
93.54% <100.00%> (+18.54%)
:arrow_up:

frontend/shared/src/components/Layout.tsx

... and 80 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ce1a927...2663063. Read the comment docs.


Use raw string literals when \ is present

What does this PR do?
fix flake8 compliance
Motivation
bump the flake8 version we use to 3.6.0
Review checklist

[ ] PR has a meaningful title or PR has the no-changelog label attached
[ ] Feature or bugfix has tests
[ ] Git history is clean
[ ] If PR impacts documentation, docs team has been notified or an issue has been opened on the documentation repo

Codecov Report

Merging #2465 into master will increase coverage by 1.29%.
The diff coverage is 88.23%.

@@            Coverage Diff             @@
##           master    #2465      +/-   ##
==========================================
+ Coverage   82.87%   84.16%   +1.29%
==========================================
  Files         440      581     +141
  Lines       21346    34386   +13040
  Branches     2823     4187    +1364
==========================================
+ Hits        17690    28942   +11252
- Misses       2834     4175    +1341
- Partials      822     1269     +447


Doesn't work

Tried on 3 different phones, nothing appears when I receive a phone call.

Thanks for the feedback, I'm sorry it doesn't work for you.  I haven't touched this project in a while and I expect recent OS updates have caused the older implementation to stop working.
Testing on a Nexus 5 API 21 (OS 5.1) emulator and checking the adb logs (below) show it failing to launch with the old code.
FYI, this talk on the WindowManager is where I would start if I was going to refresh the project:
https://www.youtube.com/watch?v=ujfe_IF1O4A
For anybody else interested, here's how you can test this using an emulator:
http://stackoverflow.com/a/14518646/112705
I haven't got plans to update this any time soon...
05-13 22:48:31.538: I/Flashback(2739): launchFlashback attempt: 1
05-13 22:48:32.107: I/Flashback(2739): launchFlashback attempt: 1
05-13 22:48:32.390: I/Flashback(2739): launchFlashback attempt: 2
05-13 22:48:32.397: I/Flashback(2739): launchFlashback attempt: 2
05-13 22:48:32.691: I/Flashback(2739): launchFlashback attempt: 3
05-13 22:48:32.695: I/Flashback(2739): launchFlashback attempt: 3
05-13 22:48:32.991: I/Flashback(2739): launchFlashback attempt: 4
05-13 22:48:32.997: I/Flashback(2739): launchFlashback attempt: 4
05-13 22:48:33.291: I/Flashback(2739): launchFlashback attempt: 5
05-13 22:48:33.295: I/Flashback(2739): launchFlashback attempt: 5
05-13 22:48:33.591: I/Flashback(2739): launchFlashback attempt: 6
05-13 22:48:33.595: I/Flashback(2739): launchFlashback attempt: 6
05-13 22:48:33.892: I/Flashback(2739): launchFlashback attempt: 7
05-13 22:48:33.893: I/Flashback(2739): launchFlashback attempt: 7
05-13 22:48:34.191: I/Flashback(2739): launchFlashback attempt: 8
05-13 22:48:34.192: I/Flashback(2739): launchFlashback attempt: 8
05-13 22:48:34.491: I/Flashback(2739): launchFlashback attempt: 9
05-13 22:48:34.492: I/Flashback(2739): launchFlashback attempt: 9
05-13 22:48:34.791: I/Flashback(2739): launchFlashback attempt: 10
05-13 22:48:34.791: I/Flashback(2739): Giving up on trying to launch Flashback


🐛 [ui] macOS Sign In

Reported on macOS, Sign In is causing problems:
Checklist

[ ] Google Sign In produces a blank screen when signing in:

[ ] Signing in with email produces a network error - this was fixed by enabling entitlements - this should be documented or a better error thown.

You must enable "Outgoing connections" in xcode

Thanks, will get it added to the docs.

@bazookon's suggestion would work, however it is recommended to edit entitlements manually.
To enable outgoing requests, add the following lines to the DebugProfile.entitlements and Release.entitlements:
<key>com.apple.security.network.client</key>
<true/>

Thank you @darshankawar for referencing me to this issue. May I recommend in the meantime that we add a line to the documentation referencing that the flag may need to be changed? (as mentioned in #8863). Otherwise I do not think the documentation is complete.
I'll just re-affirm that I struggled for like a week on this issue and almost gave up. Would hate for a less experienced dev to just abandon ship.

This is now reflected in documentation in this section.
The error message is now also displayed and hints about network failure, however the error coming from the underlying firebase sdk is too generic to be able to recognize that this is related to entitlements.


Add horizon-cmp command

Adds a simple command to compare responses of two horizon servers. Can be used to check differences between the latest stable version and release candidate.
EDIT: forgot about "Work in Progress" option in GH, can't change the status once PR is added.
TODO:

[x] Add more paths to the test corpus (can also add code that generates relevant paths).
[x] Print diff instead of full body when responses differ.
[x] ~Setting params via CLI.~ #1183
[x] ~Test streaming.~ #1184
[x] ~Run in CI and add diffs to PRs.~ #1185

@howardtw can you take a look again?

Not sure why I can't reply to the first two comments (no text box). Here they are:

https://github.com/stellar/go/pull/977#discussion_r278693673

We need to remove result_meta_xdrbecause it can be different between stellar-core instances, unfortunately. It's using unordered_map so the order can be different.

https://github.com/stellar/go/pull/977#discussion_r278694593

This is the code that generates file name from the path. We can't remove it.


MS Teams Adaptive Cards Support [Question]

Hello Everyone,
I need to know when Adaptive Cards is supported for MS Teams (NodeJS).
Now the state is "Coming Soon".
Does anyone know when the state is "coming soon"?
Thanks 👍

It's now supporting officially. You can find out here for more info.
Hope this helps!

Please give Teams a go and let us know if you run into any issues!


Definitive plan for cruft

Our cruft plan has hit a snag due to Microsoft org security settings, preventing GH Actions to create PRs.
FMI - https://docs.github.com/en/actions/security-guides/automatic-token-authentication#modifying-the-permissions-for-the-github_token.
Option 1: Manually run cruft update as a pre-commit hook?
Pros

Would check for updates and raise alarm when things aren't in sync

Cons

Adds pre-commit which would call for some other configuration changes.
Doesn't auto-update

Option 2: Create Repo, Update, and commit
Pros

Does EVERYTHING but submit the PR

Cons

Ran into some issues trying to deploy
Many of the checks are on PR and not push

This could be changed

Option 3: Manually Update each one
Pros

Works Every time

Cons

Potential drift if repos are missed

Update
Option 2 Wins, but there are some minor exceptions.
We aren't able to run cruft via workflow when modifying a GitHub Action so we can create a script that lives in cookiecutter-relelcloud/tools that does the following.
Iterates through combinations:

Creates a Placeholder Folder
Attempts to pull the combination url
Runs Cruft to update based on the path's cruft.json
Creates pull request for the updates


Update Dark Mage pouch decay checks (#3)

Auto-verify non-broken pouches (closes #3)

Add support for already repaired pouches via the repair dialog

Add support for Dark Mage -> Repair MenuOption Dialog
Add basic support for quickly repair and not finishing the dialog

Not consistent if the dialog finishes or exists faster than a tick

TODO:

Update version to 1.3.4
Implement a better solution to update decay state when a player quickly repairs a pouch before the next part of the dialog starts

Race conditions yet again

Take another look at CS2 scripts

2153 is any keyboard input at all and not specific to continue

Maybe we can verify here if there is a Widget with "Please wait..."

Move texts to be a constant variable instead of defined in the check scope

Handle repair pouch race conditions via Dark Mage via fbe6fd5

Repair pouches when the appropriate MenuAction.WIDGET_CONTINUE is received

When a player clicks on "Click to continue" to accept the repair

Repair pouches when the appropriate CS2 Script is received

When a player uses the spacebar to accept the repair ("Click to continue" via Space)


ECONNREFUSED

When using grunt serve, the first load is ok, but on reload I get the following spaming message :
Unable to update live reload: { [Error: connect ECONNREFUSED]
  code: 'ECONNREFUSED',
  errno: 'ECONNREFUSED',
  syscall: 'connect' }

I suspect it is when I add a plugin including some jquery, like angular-clndr, or ui-calendar, yes i'd like to use a calendar inside my ionic app

i second this issue


Breaking Windows Irrlicht port change on 7/20?

Describe the bug
When building minetest on windows using VCPKG for the dependencies, the build succeeds but the app fails due to character encoding issues.  I see on 7/20 a change was made to character handling defines using a patch, I'm guessing that broke the build.  If different code is generated that would break existing projects that is a problem...
Environment

OS: Windows 10
Compiler:

To Reproduce
Steps to reproduce the behavior:

Follow minetest build instructions in the readme
cmake . -G"Visual Studio 15 2017 Win64" -DCMAKE_TOOLCHAIN_FILE=D:/vcpkg/scripts/buildsystems/vcpkg.cmake -DCMAKE_BUILD_TYPE=Release -DENABLE_GETTEXT=0 -DENABLE_CURSES=0
cmake --build . --config Release

In the call to CFileSystem::createAndOpenFile() an ansi string is passed, but debugging into the Irrlicht method it treats it as unicode.
Expected behavior
The game should compile and run
Failure logs
none
Additional context
PR

edit CMakeLists.txt
replace
set(IRR_ROOT_DIR "")
to
set(IRR_ROOT_DIR .)

ver 1.9
customized version of Irrlicht 1.9.0 and runs on Windows machines only
http://irrlicht.sourceforge.net/?p=1596
https://github.com/zaki/irrlicht

Close this issue via #7726 merged.


Consider some type of "warning" for incorrect or out of date information.

When parsing and displaying information from a SARIF log, there are a number of things that the extension code is doing that "skips" information in the log that "violates" the spec. One example of this is the end line for a location being less than the start line.
Another example, which probably has more impact to the end user, is the source file (physical location URI) being referenced by the SARIF log has been edited before the log is parsed. This can cause any location (line, column) information in the SARIF log to be incorrect.
So there are two aspects to this.. As a SARIF generator (tool developer), you'd like to know about issues in the SARIF your tool is generating. As a SARIF consumer (a developer running static analysis), you would like to be aware that the analysis results you are looking at may not be accurate.

Jeff and Henok believe we should drop invalid Sarif and not notify customer. However should send signal of Sarif tool owner.

Extension should validate SARIF beforehand


weird _adjustContentOffsetIfNecessary changed my contentOffset

I have two tabs, each contains one UICollectionView. I embed SVPullToRefresh in each UICollectionView as well. I set up the snapping by 0.5 threshold for BLK height bar.
tab 1 has 6 cells and tab 2 has 20 cells, so tab 2 could snap to progress 1, while tab 1 cannot.
My behaviour is copied from Facebook style behaviour:
- (void)scrollViewDidScroll:(UIScrollView *)scrollView
{
    if(!self.isCurrentlySnapping)
    {
        CGFloat deltaYOffset = scrollView.contentOffset.y - self.previousYOffset;
        CGFloat deltaProgress = deltaYOffset / (self.flexibleHeightBar.maximumBarHeight-self.flexibleHeightBar.minimumBarHeight);

        self.flexibleHeightBar.progress = self.previousProgress + deltaProgress;
        NSLog(@"contentOffset Y %f, previousYOffset:%f, progress %f",scrollView.contentOffset.y,self.previousYOffset, self.previousProgress + deltaProgress);

        [self.flexibleHeightBar setNeedsLayout];
    }
}

everytime when switching tab, I will update the UICollectionView like this:
targetCollection.delegate = (id<UICollectionViewDelegate>)self.delegateSplitter;
targetCollection.contentOffset = CGPointMake(targetCollection.contentOffset.x, -CGRectGetHeight(self.BLKBar.frame));

the second line is to keep the BLKBar height consistent while switching between tabs. e.g. When the BLK height bar is at minimum height, I should keep it, so I need to update the offset when switching to target tab.
Steps:
First, I switch from tab 1 to tab 2; BLKBar is at progress 0;
when I am on tab 2, I scroll the UICollectionView, snap BLKBar to progress 1;
Then I switch to tab 1 by paning horizontally, ideally the BLK Bar should keep the snapping at progress 1, aka minimum height, however, I found it is at progress 0!
In order to debug, I subclass a HackUICollectionView to see what is happenning.
I setup my BLKBar minimum height is 64, max height is 108.
so while debugging, after
 targetCollection.contentOffset = CGPointMake(targetCollection.contentOffset.x, -CGRectGetHeight(self.BLKBar.frame));
is set to (0,-64),
there is one API called _adjustContentOffsetIfNecessary changed my contentOffset to -108:
...
    frame #8: 0x000000010a008bda UIKit`-[UIScrollView setContentOffset:] + 645
    frame #9: 0x000000010969e3ca MCompass`-[HackUICollectionView setContentOffset:](self=0x00007f8279854e00, _cmd=0x000000010a765895, contentOffset=(x = 0, y = -108)) + 74 at HackUICollectionView.m:14
    frame #10: 0x0000000109b5bbf6 Foundation`_NSSetPointValueAndNotify + 139
    frame #11: 0x000000010a01d535 UIKit`-[UIScrollView(UIScrollViewInternal) _adjustContentOffsetIfNecessary] + 1445
    frame #12: 0x000000010a01a727 UIKit`-[UIScrollView(UIScrollViewInternal) _stopScrollingNotify:pin:tramplingDragFlags:] + 417
...

If I override _adjustContentOffsetIfNecessary in sub class, it will not change it to -108 obviously, but I CANNOT fix my problem by this way because it is a internal API, I cannot simply pass through it.
Any idea why this happened? How can I keep the progress while switching UICollectionViews?

Thanks for the detailed explanation. I'll look into this soon.


Only check bundle version if the bundle is embedded

This check is useful for released version of crc. For development, we need to be able to launch any bundle.

Wish we could rename constants

/retest


Doesn't work in conjunction with ClassVar

Reproduction
from __future__ import annotations

import abc
from typing import ClassVar

from abcattrs import Abstract
from abcattrs import abstractattrs

@abstractattrs
class Base(abc.ABC):
    attr: ClassVar[Abstract[str]]

class Sub(Base):
    ...

Fixed in #12


Create rule "module-parent-folder"

Check if module is inside specific parent folder paths. Defaults to "modules"
.monolinter.json:

Enable rule

{
  "rules": {
    "module-parent-folder": true
  }
}

Enable rule with custom parent names

{
  "rules": {
    "module-parent-regex": ["services", "web", "libs", "services/group1", "services/group2"]
  }
}

Hi! I'm taking this for my Open Source Friday...

🎉🎉

I think it would make sense to have the module-parent-folder being either a boolean or the array of regex instead of having two different attributes, right?
Like:
Boolean:
{
  "rules": {
    "module-parent-folder": true
  }
}

Regex:
{
  "rules": {
    "module-parent-folder": ["services", "web", "libs", "services/group1", "services/group2"]
  }
}

Right?

Also, I'm not sure if it in this case having a default value makes sense... because this type of structure is very wide IMO. Maybe it would be better to have it just as an array of glob patterns.
wdyt?

An array of glob patterns is perfect and very powerful! Maybe the default could be “modules”, what do you think?
About glob processing, we use it on discovermodules already using “fast-glob”. Take a look while implementing it.

Nice, thanks for the fast-glob tip, I'll check that out! Very handy.
About the default modules folder, still not sure if there is enough common sense that a project would have modules as the parent folder of the created modules. For instance, in https://github.com/vsavkin/large-monorepo we don't have it and in some of the projects at https://github.com/korfuri/awesome-monorepo#notable-public-monorepos I couldn't find a common sense. I still think that this lack of common sense/practice leads to not having a default approach so far.
My suggestion is to start without the default as it simplifies a little bit the code, and then we discuss this in another issue. Sounds reasonable?

Ok! Let's do that.


Downloading OneFlow service twice with the same name fails with 500

Description
When an app is downloaded from one markerplace (for example Service Kubernetes 1.23) and some resources already exist Sunstone fails and this criptic error is logged into the /var/log/one/sunstone.log file:
Mon May 09 16:31:21 2022 [I]: ::1 - - [09/May/2022:16:31:21 +0000] "POST /marketplaceapp/589/action HTTP/1.1" 500 - 0.0248

To Reproduce

It is enough to download a service once and then retry keeping the same value in the Name box.
The other way would be to instantiate a service from templates, then delete templates using Delete Images and VM Templates. Because VMs are running, disk images are not deleted and downloading the service again will cause the same conflict(s) and error.

Expected behavior

User should be notified about the conflict with a human readable explanation, the error message should not be a raw 500..
User should be informed what needs to be done to workaround the problem (it's either resource cleanup or service name change).

Details

Affected Component: Sunstone
Hypervisor: NA
Version: ALL

Additional context
NO

Progress Status

[ ] Code commited
[ ] Testing - QA
[ ] Documentation (Release notes - resolved issues, compatibility, known issues)

PRs associated with this issue:
https://github.com/OpenNebula/development/pull/1975
https://github.com/OpenNebula/docs/pull/2355
https://github.com/OpenNebula/one-ee/pull/2215
https://github.com/OpenNebula/one-ee/pull/2227


currently not working

Thanks for creating this.  It worked at first, but now, in Firefox at least, when switching streams, the screen goes black and the loading icon appears.  It seems to last as long as the ad would have, then the stream starts.

To debug it would be helpful to have information about the HLS m3u8 requests that Twitch makes. An easy way to provide these would be:

Go to the twitch directory (no stream playing)
Open the Developer Tools: Ctrl+Shift+I
Select "Network"
Put m3u8 into the "Filter URLs" top-left field
Open a stream
If the stream goes black wait until the stream starts
Now click top-right on "HAR" and select "Save All as HAR"
Attach the saved file here as comment

I see no "HAR" option..

@SaiyanOfDarkness The steps are for OP since he mentioned using Firefox. For Chromium the steps are:

Go to the twitch directory (no stream playing)
Open the Developer Tools: Ctrl+Shift+I
Select "Network"
Put m3u8 into the "Filter" top-left field
Open a stream
If the stream goes black wait until the stream starts
Now right-click on any request shown in the network table and select "Save all as HAR with content"
Attach the saved file here as comment

Saving HAR produces a 0kb file with nothing inside of it.
Not sure what I'm doing wrong.

That's odd, sounds like a browser bug. Maybe upgrade your version (if outdated) or try Chrome?

Yep. Simply just not working in Firefox.
Black screen when loading a stream but said stream never comes up.

Stopped working for me about 3-5 days ago on Firefox


Installing from yum repo fails with HTTP 403 error

Description
Attempting to follow Installation using the Apache CouchDB convenience binary packages on a RedHat/CentOS system fails to install the package with a HTTP 403 Forbidden error from the package repository.
Steps to Reproduce
yum install -y yum-utils
yum-config-manager --add-repo https://couchdb.apache.org/repo/couchdb.repo
yum install -y couchdb

Expected Behaviour
CouchDB is installed from an RPM downloaded from the repository.
Your Environment

Operating system and version: CentOS 7.9

Additional Context
Full output from commands:
[root@45ae2b263cc5 /]# yum install -y yum-utils
Loaded plugins: ovl
There are no enabled repos.
 Run "yum repolist all" to see the repos you have.
 To enable Red Hat Subscription Management repositories:
     subscription-manager repos --enable <repo>
 To enable custom repositories:
     yum-config-manager --enable <repo>
[root@45ae2b263cc5 /]# yum-config-manager --add-repo https://couchdb.apache.org/repo/couchdb.repo
Loaded plugins: ovl
adding repo from: https://couchdb.apache.org/repo/couchdb.repo
grabbing file https://couchdb.apache.org/repo/couchdb.repo to /etc/yum.repos.d/couchdb.repo
repo saved to /etc/yum.repos.d/couchdb.repo
[root@45ae2b263cc5 /]# cat /etc/yum.repos.d/couchdb.repo
[couchdb]
name=couchdb
baseurl=https://apache.jfrog.io/artifactory/couchdb-rpm/el$releasever/$basearch/
gpgkey=https://couchdb.apache.org/repo/keys.asc https://couchdb.apache.org/repo/rpm-package-key.asc
gpgcheck=1
repo_gpgcheck=1
enabled=1
[root@45ae2b263cc5 /]# yum install -y couchdb
Loaded plugins: ovl
https://apache.jfrog.io/artifactory/couchdb-rpm/el7/x86_64/repodata/repomd.xml: [Errno 14] HTTPS Error 403 - Forbidden
Trying other mirror.
To address this issue please refer to the below wiki article

https://wiki.centos.org/yum-errors

If above article doesn't help to resolve this issue please use https://bugs.centos.org/.

 One of the configured repositories failed (couchdb),
 and yum doesn't have enough cached data to continue. At this point the only
 safe thing yum can do is fail. There are a few ways to work "fix" this:

     1. Contact the upstream for the repository and get them to fix the problem.

     2. Reconfigure the baseurl/etc. for the repository, to point to a working
        upstream. This is most often useful if you are using a newer
        distribution release than is supported by the repository (and the
        packages for the previous distribution release still work).

     3. Run the command with the repository temporarily disabled
            yum --disablerepo=couchdb ...

     4. Disable the repository permanently, so yum won't use it by default. Yum
        will then just ignore the repository until you permanently enable it
        again or use --enablerepo for temporary usage:

            yum-config-manager --disable couchdb
        or
            subscription-manager repos --disable=couchdb

     5. Configure the failing repository to be skipped, if it is unavailable.
        Note that yum will try to contact the repo. when it runs most commands,
        so will have to try and fail each time (and thus. yum will be be much
        slower). If it is a very temporary problem though, this is often a nice
        compromise:

            yum-config-manager --save --setopt=couchdb.skip_if_unavailable=true

failure: repodata/repomd.xml from couchdb: [Errno 256] No more mirrors to try.
https://apache.jfrog.io/artifactory/couchdb-rpm/el7/x86_64/repodata/repomd.xml: [Errno 14] HTTPS Error 403 - Forbidden

I'm having the same issue on Debian Buster:
#  apt update
Hit:1 http://security.debian.org/debian-security buster/updates InRelease
Hit:2 http://cdn-fastly.deb.debian.org/debian buster InRelease
Err:3 https://apache.jfrog.io/artifactory/couchdb-deb buster InRelease
  403  Forbidden [IP: <IP_ADDRESS> 443]
Reading package lists... Done
E: Failed to fetch https://apache.jfrog.io/artifactory/couchdb-deb/dists/buster/InRelease  403  Forbidden [IP: <IP_ADDRESS> 443]
E: The repository 'https://apache.jfrog.io/artifactory/couchdb-deb buster InRelease' is no longer signed.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
N: See apt-secure(8) manpage for repository creation and user configuration details.

and Debian Stretch:
# apt update
Hit:1 http://security.debian.org/debian-security stretch/updates InRelease
Ign:2 http://cdn-fastly.deb.debian.org/debian stretch InRelease
Hit:4 https://deb.nodesource.com/node_8.x stretch InRelease
Hit:3 http://cdn-fastly.deb.debian.org/debian stretch Release
Ign:6 https://apache.jfrog.io/artifactory/couchdb-deb stretch InRelease
Err:7 https://apache.jfrog.io/artifactory/couchdb-deb stretch Release
  403  Forbidden
Reading package lists... Done
E: The repository 'https://apache.jfrog.io/artifactory/couchdb-deb stretch Release' does not have a Release file.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
N: See apt-secure(8) manpage for repository creation and user configuration details.

This seems to be resolved now, and it's possible again to install from the package repository on CentOS.
@zamb3zi I'm not sure if the Debian package repository is still having issues?
If possible, it would be good to know the underlying issue and if it's likely to happen again.

@ncraike looks good to me now. Working in both debian stretch and buster.

Not resolved on ubuntu 21.10
$ sudo apt update
...
Ign:6 https://apache.jfrog.io/artifactory/couchdb-deb impish InRelease
Get:7 http://us.archive.ubuntu.com/ubuntu impish-security InRelease [110 kB]
Err:8 https://apache.jfrog.io/artifactory/couchdb-deb impish Release
  404   [IP: <IP_ADDRESS> 443]
Reading package lists... Done
E: The repository 'https://apache.jfrog.io/artifactory/couchdb-deb impish Release' does not have a Release file.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
N: See apt-secure(8) manpage for repository creation and user configuration details.

@handerpeder It's a different issue. We don't support impish Ubuntu 21.10. Try using the Ubuntu 20.04 package
https://apache.jfrog.io/ui/native/couchdb-deb/dists/

A similar issue has started happening today:
Get:1 http://security.debian.org/debian-security buster/updates InRelease [34.8 kB]
Hit:2 http://deb.debian.org/debian buster InRelease
Get:3 http://security.debian.org/debian-security buster/updates/contrib Sources [1,084 B]
Get:5 http://security.debian.org/debian-security buster/updates/main Sources [373 kB]
Get:6 http://security.debian.org/debian-security buster/updates/main amd64 Packages [589 kB]
Get:7 http://security.debian.org/debian-security buster/updates/main Translation-en [319 kB]
Get:8 http://security.debian.org/debian-security buster/updates/contrib amd64 Packages [3,460 B]
Get:9 http://security.debian.org/debian-security buster/updates/contrib Translation-en [1,488 B]
Get:4 https://apache.jfrog.io/artifactory/couchdb-deb buster InRelease [11.5 kB]
Err:4 https://apache.jfrog.io/artifactory/couchdb-deb buster InRelease
  Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)
Reading package lists... Done
E: Failed to fetch https://apache.jfrog.io/artifactory/couchdb-deb/dists/buster/InRelease  Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)
E: The repository 'https://apache.jfrog.io/artifactory/couchdb-deb buster InRelease' is no longer signed.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
N: See apt-secure(8) manpage for repository creation and user configuration details.

Note: this exactly 2 years to the day to the original issue. Certificate expiry?

Thank you for the report. We are aware of the problem. The ASF Infra team is already working on it, unfortunately there is nothing we can do from our side.

i am facing this issue couchdb is failing due to below error
etrieving key from https://couchdb.apache.org/repo/keys.asc
Importing GPG key 0x7A00258D:
Userid     : "The Apache Software Foundation (Package repository signing key<EMAIL_ADDRESS>Fingerprint: 390e f70b b1ea 12b2 7739 6295 0ee6 2fb3 7a00 258d
From       : https://couchdb.apache.org/repo/keys.asc
Is this ok [y/N]: y
Retrieving key from https://couchdb.apache.org/repo/rpm-package-key.asc
couchdb/7/x86_64/signature                                                                                |  11 kB  00:00:03
https://apache.jfrog.io/artifactory/couchdb-rpm/el7/x86_64/repodata/repomd.xml: [Errno -1] repomd.xml signature could not be verified for couchdb
Trying other mirror.
One of the configured repositories failed (couchdb),
and yum doesn't have enough cached data to continue. At this point the only
safe thing yum can do is fail. There are a few ways to work "fix" this:
 1. Contact the upstream for the repository and get them to fix the problem.

 2. Reconfigure the baseurl/etc. for the repository, to point to a working
    upstream. This is most often useful if you are using a newer
    distribution release than is supported by the repository (and the
    packages for the previous distribution release still work).

 3. Run the command with the repository temporarily disabled
        yum --disablerepo=couchdb ...

 4. Disable the repository permanently, so yum won't use it by default. Yum
    will then just ignore the repository until you permanently enable it
    again or use --enablerepo for temporary usage:

        yum-config-manager --disable couchdb
    or
        subscription-manager repos --disable=couchdb

 5. Configure the failing repository to be skipped, if it is unavailable.
    Note that yum will try to contact the repo. when it runs most commands,
    so will have to try and fail each time (and thus. yum will be be much
    slower). If it is a very temporary problem though, this is often a nice
    compromise:

        yum-config-manager --save --setopt=couchdb.skip_if_unavailable=true

failure: repodata/repomd.xml from couchdb: [Errno 256] No more mirrors to try.
https://apache.jfrog.io/artifactory/couchdb-rpm/el7/x86_64/repodata/repomd.xml: [Errno -1] repomd.xml signature could not be verified for couchdb


Build and test

This GitHub Action

builds a nanobot binary,
uploads this binary as an artifact,
and runs a test by comparing the output of nanobot init with the contents of tests/init.txt.

Setting up tests in this way doesn't seem too painful.

Do you think it make sense to fold this into the same GitHub Action as #10?

Yes. We'd only need to add the cargo fmt action here because cargo check is part of cargo build.

As discussed on the call, this is what I asked for, but now I'd like to make better use of make. I'll update the issue with my new thoughts when I get a minute.

After thinking about it for a bit, I would like to replace most of the GitHub Action steps with make tasks that can also be run locally:
* `make deps` check for Ubuntu, then `apt-get install rust cargo` any required dependencies

* `make format` run `cargo` format checks

* `make build` build binaries

* `make test` run integration tests on binaries

Specialized GitHub Action steps will still be required for checkout and uploading binaries, but everything that is not specific to GitHub Actions should run through make.

I don't know if this is helpful (maybe not at all), but there are Makefile targets to do some of the things that James is asking for here in the valve.py's Makefile


HOWTO: manage general purpose channels

While waiting for subscriptions support both here and in absinthe I though of building a custom pubsub mechanism to inform the clients when they should update. To do so i should be able to handle the websockets events in my application (for example join-room, new-chat-message, ...) I was wondering if it's possible to do so while using this library and how would you suggest doing it (or why not)

Hey @G3z,
We are almost done with GQL subscription, I just need a bit a time to fix the remaining merge conflicts on that branch, and would certainly be thankful if you'd want to help me test that branch with either the absinthe subscriptions example or with your own use case. I'll do my best to try giving this a couple of hours today and ping you back with some instructions/example on how to test.
Otherwise, (in case that neither you nor I have time for advancing the PR) I guess you could use Apollo.watchQuery with the pollInterval option. That would not be as ideal as an actual GQL subscription, but I guess in the mean time you might consider having a query polling to the server.

uh I didn't know you were so close, sorry.
I missed the "absinthe subscriptions example" you are talking about 🤔
I'd be happy to help!

Maybe i should open an appropriate branch but...
I tried the subscription branch
I made a subscription which I tested in graphiql (I cloned a query)
In my app nothing appears (not even an error
but when i unmount the component i receive this
Unhandled JS Exception: f.apply(...).then is not a function

generated from
function unsubscribe (subID) {
    const processReponse = subscriptions[subID]
    const doUnsubscribe = connect.bind(
      null, sockets, performUnsubscribe.bind(null, processReponse))
    pipeP(requestMiddleware, doUnsubscribe)({subID}) // here
  }

module.exports = function _pipeP(f, g) {
  return function() {
    var ctx = this;
    return f.apply(ctx, arguments).then(function(x) { // this throws
      return g.call(ctx, x);
    });
  };
};

Lets close this and keep progress on #9


Edited Modal resposivness, added Web Ilustration

Fixed a bug in the modal responsiveness.
Added web illustration to modal.
Solution to the android problem, keyboard hides the input field:
Changed android modal css position to sticky - makes the modal jump up when trying to write to the input field.
made the image disappear when clicking the 'enter location manually' button( on android)

Thanks @guyesh! I'll review it soon.
Meanwhile, some notes on the illustration:

The filesize is 44kb, which is a lot. Try to use SVGOMG to reduce the file size, but if it doesn't make it go under <10kb~ - use jpg & webp to display it.
Change the file name to something descriptive, such as welcome-illustration.svg
If Lena haven't talked to you yet, please send her a message on discord (@Lena)

This is how it looks initially (page load) on iPhone 7 Plus:

Maybe placing the modal slightly above the header can help?

sure! will be done.

made the modal scrollable when the screen is shorter than the modal
this solved the android problem( that the keyboard hides the input field).
also on some short landscape screens, the modal was cut off by the bottom of the screen. the scrollable modal solved the problem.
Added a low-resolution image (6KB) to appear when the high-resolution one(28KB) is being loaded, thus the user doesn't need to wait for the image but also doesn't lose quality.

IPHONE 11 MAX Image:

samsung note 9 Images:


Minor fix: FunctionsToExport

You are missing a few Functions in your Export.
I have found these missing:
GUI
Write-PodeValueToResponse
Write-PodeValueToResponseFromFile

Hey @LLIT
The GUI function is a bug, it should be there and is missing.
As for the other two (Write-PodeValueToResponse and Write-PodeValueToResponseFromFile), I removed them in #209, as they were grouped with functions that aren't documented, and not needed to be exported as they were internal - which is why they also got renamed.
Is there a flow you're using where this throws errors if they're not exported; or are you using the functions directly?
I can add them back in, just curious if there's cause to give the functions an alias/rename - like route or listen 🤔

The functions Write-PodeValue.... was just discoverd by reading the code, and detecting they were missing.
It was the Gui function i was after :-)

Awesome, I'll get that added for you! :)

Gui exported in v0.28.1


LZ4 Dependency missing

github.com/pierrec/lz4/v4
Package is no longer provided, break the archiver.

Resolved already, discussion in https://github.com/mholt/archiver/issues/248


ETL - apply deltas in the situation: input-change(web)-output - spikelike probe - phase 2

This is a follow-on to issue #1250 - continuing that spike, apply the computed deltas to an update of the input_spreadsheet to produce an output spreadsheet with the changes added

Note this is completed, except for files. Separate issue: #1253.


[Feature] Allow spellcheck to be disabled from editor

Initial checklist

[X] I agree to follow the code of conduct
[X] I searched issues and discussions and couldn’t find anything (or linked relevant results below)

Problem
Option to disable spellcheck using the HTML attribute spellCheck on the Editor component
Solution
Option to disable spellcheck using the HTML attribute spellCheck on the Editor component
Alternatives

You can set the attribute through this Prosemirror API
https://prosemirror.net/docs/ref/#view.EditorProps.attributes


Update alerts-activity-log-service-notifications.md

Fixing issue that I requested on #32889 that was not fixed properly.

@dcbrown16 : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Condense test data generation for spatial_analysis_test and spatial_analysis_utils_test

Describe the solution you'd like
The process to generate synthetic data for testing purposes in spatial_analysis_test and spatial_analysis_utils_test should be combined into one function for each separate dataframe/xarray/etc. needed (ex. dist_mats, expression_mats, neighborhood_mats should be in separate functions). We're relying on entirely different test matrices for both functions and it's overcomplicating the testing process. By placing these data-generating functions in test_utils and combining shared logic, we should have a much more modular testing process.

We can update the tests when we reintroduce the pairwise distance notebook


can't seem to switch subscriptions

I am attempting to use a different quota/subscription id using --sid, but I get the following error after submitting orders:
"Quota check failed - Insufficient quota: /subscriptions do not match all of the events"

Hi @AndrewAnnex
Guessing you a  E & R user, you might want to check if you have quota left. Note you have two subscriptions one for PlanetScope and RapidEye data and the other for Landsat.Hence the insufficient quota message because you might be trying to order PlanetScope images using the sid for Landsat and your quota is maxed out.

Hi @AndrewAnnex
Guessing you a E & R user, you might want to check if you have quota left. Note you have two subscriptions one for PlanetScope and RapidEye data and the other for Landsat.Hence the insufficient quota message because you might be trying to order PlanetScope images using the sid for Landsat and your quota is maxed out.


Add "getwalletheight" RPC method

What current issue(s) does this address, or what feature is it adding?
This PR addresses issue #189 and adds new getwalletheight RPC method
Adds similar functionality to https://github.com/neo-project/neo-cli/pull/203
How did you solve this problem?
Trial and Error
How did you make sure your solution works?
Added two tests
Are there any special changes in the code that we should be aware of?
No
Please check the following, if applicable:

[x] Did you add any tests?
[x] Did you run make lint?
[x] Did you run make test?
[x] Are you making a PR to a feature branch or development rather than master?
[x] Did you add an entry to CHANGELOG.rst? (if not, please do)

Coverage increased (+0.02%) to 80.283% when pulling ef119c885b2c7b40a2e5868be66de2ec6c28060d on jseagrave21:patch-1 into 0b47cfdeb87dfe107d7ab3b188b66842fdb78de4 on CityOfZion:development.

thanks 👍


Receiving multiple webhook new message data for single question

Thanks for building this wonderful wrapper.
My problem is -
I am receiving the same new message webhook data multiple times even if it's asked once on WhatsApp.
I visited the issues sections to check comments but this one looks different to me.
I do not have more than one app tied to the same account which was the case for a few.
My app uses langchain and openai internally to generate responses to incoming message. And at times it takes some seconds or even a minute to generate a response. And probably this could be one of the reasons the 200 status ack is delayed and WhatsApp pings the same question to webhook again and again until it receives 200 ack from one of the previous requests.
I am not 100% sure if the above case is true. If it is, I would like to know in the WhatsApp cloud API account -

how can we extend the time window? or
how can we reduce the retries to 1?

If not, how can we handle the delay in the code itself?
Any help will be appreciated, Thanks!
@Kalebu @filipporomani

Well, tbh this sounds strange.
You coud try to return code 200 immediately and run the other code with a post-request app middleware (this is possible if i'm not wrong).
I don't know if there's a way to extend the time window, but I don't actually think so.
You shouldn't reduce the retries from WhatsApp, you should try to slove the issue in your code.
Try the thing I said above and let me know!

Great solution too! Thanks a lot for sharing with me!


Can not connect and download via recorder_console.py of StreamRecorderConverter

I can access my Holoelens 2 via WiFi and the Windows Device Portal in the browser. This way I can download the recordings of StreamRecorder. However, the provided script for automatic data download does not work.
This is the command I use, with the ip address (<IP_ADDRESS>), username and password which works with Windows Device Portal via the browser. For the port number (9000) I tried also with 10080
python recorder_console.py --dev_portal_address '<IP_ADDRESS>:9000' --dev_portal_username 'username' --dev_portal_password 'password' --workspace_path 'C:\example'
The error message is:
`Connecting to HoloLens Device Portal...
[...]
TimeoutError: [WinError 10060] A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond
[...]
urllib.error.URLError: <urlopen error [WinError 10060] A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond>
`
I also tested with a turned of requirement for SSL (Device Portal - > System -> Preferences) but I get the same error message.

Try without the port number, It worked for me.

Removing the port number worked.


initial commit for ZAP Chat 18

Heyho - here are the unfortunately delayed but revised subtitles for the ZAP Chat 18 video.
I hope I spelled Antero(?) correctly. Otherwise I had problems with the other open source project name KIX(?) - so the name may be misspelled here.
Otherwise I hope it fits.
videourl: https://www.youtube.com/watch?v=7kqg7x7d6os

For whatever reason the subtitles for episode 17 are in there now - I only used sqash and signoff  :-(

Antero is correct: https://www.zaproxy.org/docs/team/antero/ It's KICS.


Under specific circumstances one step of the simulator can take 800 seconds

We experienced at least twice that one single tick of the simulator took 800 seconds.
An example of this is can be found at 12:38:224 of K-RR-A3, CIT Brains vs RoboFEI where nothing special seems to be appearing.
[2944.414|0757.320] Info: STATUS: Avg speed factor: 0.267 (over last 20.04 seconds)
[2944.414|0757.320] Info: STATUS: state: STATE_READY, remaining time: 600
[3751.897|0758.224] Info: STATUS: Avg speed factor: 0.001 (over last 807.48 seconds)
[3751.897|0758.224] Info: STATUS: state: STATE_READY, remaining time: 600
[3771.945|0763.264] Info: STATUS: Avg speed factor: 0.251 (over last 20.05 seconds)
[3771.945|0763.264] Info: STATUS: state: STATE_READY, remaining time: 600

The same error occurred at 12:38:360 in K-RR-E1, MRL-HSL vs RoboFEI in another state of the game.
[3233.029|0755.728] Info: STATUS: Avg speed factor: 0.234 (over last 20.01 seconds)
[3233.029|0755.728] Info: STATUS: state: STATE_PLAYING, remaining time: 572
[3240.819|0757.536] Info: Ball touched again by red player 3.
[4031.769|0758.360] Info: STATUS: Avg speed factor: 0.003 (over last 798.74 seconds)
[4031.769|0758.360] Info: STATUS: state: STATE_PLAYING, remaining time: 570
[4051.827|0762.656] Info: STATUS: Avg speed factor: 0.214 (over last 20.06 seconds)
[4051.827|0762.656] Info: STATUS: state: STATE_PLAYING, remaining time: 565

I'm wondering, whether it's possible that it's related with an error related to cache that we obtain systematically:
[30230:30255:0625/023527.458433:ERROR:disk_cache.cc(184)] Unable to create cache
[30230:30255:0625/023527.458488:ERROR:shader_disk_cache.cc(606)] Shader Cache Creation failed: -2
[30266:30274:0625/023527.489241:ERROR:simple_backend_impl.cc(81)] Failed to create directory: /home/ubuntu/.cache/Cyberbotics/Webots/QtWebEngine/Default/Cache
[30266:30274:0625/023527.489299:ERROR:simple_backend_impl.cc(81)] Failed to create directory: /home/ubuntu/.cache/Cyberbotics/Webots/QtWebEngine/Default/Cache
[30266:30274:0625/023527.489313:ERROR:simple_backend_impl.cc(757)] Simple Cache Backend: wrong file structure on disk: 1 path: /home/ubuntu/.cache/Cyberbotics/Webots/QtWebEngine/Default/Cache
[30266:30274:0625/023527.489384:ERROR:simple_backend_impl.cc(81)] Failed to create directory: /home/ubuntu/.cache/Cyberbotics/Webots/QtWebEngine/Default/Cache
[30266:30274:0625/023527.489416:ERROR:simple_backend_impl.cc(81)] Failed to create directory: /home/ubuntu/.cache/Cyberbotics/Webots/QtWebEngine/Default/Cache
[30266:30274:0625/023527.489426:ERROR:simple_backend_impl.cc(757)] Simple Cache Backend: wrong file structure on disk: 1 path: /home/ubuntu/.cache/Cyberbotics/Webots/QtWebEngine/Default/Cache
[30266:30270:0625/023527.489448:ERROR:disk_cache.cc(184)] Unable to create cache

I have never seen this error before... But that could indeed explain the problem. Are you sure you have sufficient space on the disk?

Yes I am certain we have sufficient space on the disk, three things I can already mention is that:

We also had this error on other computers, the common factor was just running the game over ssh (when we run webots locally, we don't have this issue)
Looking at old logs, we have been having this message from the beginning
It's not crashing and the 800 seconds appeared only with games involving the RoboFEI model until know and only once, so it's not gamebreaking for us, but still quite curious.

Yes, what is really strange is that it happens only with one model of robot... Can we download this model somewhere to try to reproduce the problem?

Here is a link to the RoboFEI robot model.
Note that they tried moving to an 'embedded' (i.e not regenerating) version of the proto, but that led to crash of the referee. So they reverted in order to at least be able to participate.


Implements shadow jar and backfills README

This corrects shadow configuration and backfills documentation on how to
use it.
The shadow jar was not configured correctly, mostly due to the "spark"
scope, which was not necessary to call out. This removes that scope and
defers to "compile". The impact is that if a user expects to exclude
spark, or handle it differently, they should just exclude the transitive
dep.
This also renames the env variable ZIPKIN_KEYSPACE to
CASSANDRA_KEYSPACE. This properly aligns the namespace to upstream and
other cassandra-specific properties.

incidentally, I fixed this to help @prat0318, who was formerly using gradle to run the job in production. When errors occurrred, they were obfuscated by gradle console layering. By supporting a fat jar, we are closer to publishing a release and also make mean-time usage far more straight-forward.

nothing notable on build failures, and doesn't fail locally. kicking

lgtm

build fixed via https://github.com/openzipkin/zipkin-dependencies-spark/pull/12


iPad - Login - App freezes after login in landscape orientation

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Launch the app in landscape orientation
Login with valid credentials

Expected Result:
User should be taken to the home page
Actual Result:
The chat is displayed for a second and then the app freezes
Workaround:
Unknown
Platform:
Where is this issue occurring?
Web
iOS ✔
Android
Desktop App
Mobile Web
Version Number: 1.0.70-0
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
https://user-images.githubusercontent.com/43996225/122312711-cea85080-cee2-11eb-90e7-18c7bb81ec5e.mp4
Expensify/Expensify Issue URL:
View all open jobs on Upwork

Triggered auto assignment to @Luke9389 (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

I can't take this right now. Super busy today/ this week.

Triggered auto assignment to @bfitzexpensify (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

Verified that this is not occuring on iOS in general.

Looks like a regression caused by https://github.com/Expensify/Expensify.cash/pull/3515
iPad (and android tablets) will be enter this conditional and crash because document.body is undefined on native.
Maybe we should make a index.native.js for the Popover here? cc @parasharrajat  @roryabraham

Somebody deleted the native file which got merged while pulling the latest changes. Fix is already up to #3635.

Thanks for the quick PR @parasharrajat! Upwork job has been posted and I've sent a hire request to you for the job.

Triggered auto assignment to @roryabraham (Exported), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

Thanks, @bfitzexpensify.


API cleanup for level validation

Description of changes
API cleanup for level validation
Issue #, if available
Checklist for requesting a review
The change in this PR is (choose one, and delete the other options):

[x] A change (breaking or otherwise) that only impacts unreleased or experimental code.

I confirm that this PR (choose one, and delete the other options):

[x] Updates the "Unreleased" section of the CHANGELOG with a description of my change (required for major/minor version bumps).

I confirm that cedar-spec (choose one, and delete the other options):

[x] Requires updates, but I do not plan to make them in the near future. (Make sure that your changes are hidden behind a feature flag to mark them as experimental.)

I confirm that docs.cedarpolicy.com (choose one, and delete the other options):

[x] Does not require updates because my change does not impact the Cedar language specification.

SemVer change is expected.


Snap 1828

Changes proposed in this pull request
As a fix for SNAP-1828, removing part of changes of commit 758ffdda21cb97a011be8dbbd07399556924f43 . sqlEviction.bt was failing due to this.
Patch testing
sqlEviction.bt. No failures observed after this change.
precheckin
ReleaseNotes changes
NA
Other PRs
NA

For now this is ok but as discussed, its still not clear why this change is required because the buffer returned here should always be a duplicate one and changing the limit/position of that buffer should not have mattered. If that assumption is getting broken in some path (i.e. buffer returned is internal one and not a duplicate/temporary one) then that needs to be fixed instead because it can cause other issues.

I have filed task https://jira.snappydata.io/browse/SNAP-1921 to track this.


Extensible Native variant for CosmosMsg

There was some discussion about how to make CosmosMsg (and query types) easily extensible by custom cosmos-sdk apps, without rebuilding the entire cosmwasm stack.
This demonstrates one approach, making CosmosMsg generic over the NativeMsg type, eg.
CosmosMsg::Native(T). I have a bit more investigation to validate this works, but would like a review on the approach (is it powerful enough without being too much of a burden for those who do not care?). Another possbile approach is adding RawJson support to our serialization lib and using that, but then we lose all type-safety.
I demonstrate this by updating the reflect contract to return a custom NativeMsg, only by defining it in the contract. This passes through nicely, while other implementations (eg. hackatom) default to NativeMsg without needing to explicitly define it.
However, when running integration tests, we now need to specify the type of CosmosMsg::Native() as a generic bound. This is fine for test code, as we want to parse the struct and compare the content, but this should not be the case for the production path. That is, if I call handle via go-cosmwasm, neither go-cosmwasm nor cosmwasm-vm should be aware (or care) about what concrete type we use with CosmosMsg::Native(), allowing this to be passed all the way through to the CosmosSDK app - thus, only the contract and the SDK app would need to agree, which is the proper place to allow quick iteration for a project importing this library.

[x] Make all types in std generic
[x] Exports in std handle generic
[x] Demonstrate custom NativeMsg in reflect
[ ] Update cosmwasm-vm::testing to handle custom types
[ ] Ensure cosmwasm-vm doesn't care about types outside of testing
[ ] Fix all other contracts as needed

Also, open question is how to do some checks on this (at contract upload or instantiation), but we have that issue with other feature flags (like staking) as well.

Researching VM type checks.
call_init and call_handle are used by testing::{init, handle} and DO require proper types, as they deserialize the result from the API and CANNOT be used in go-cosmwasm
call_init_raw and call_handle_raw just pass the raw binary response from the contract up the stack. These are SAFE for use in go-cosmwasm.
go-cosmwasm DOES indeed use the raw variants, just passing the literal response from the wasm contract into the go code to be parsed: https://github.com/CosmWasm/go-cosmwasm/blob/master/src/lib.rs#L160
We need to modify the Go code in go-cosmwasm to pass this unparsed (which is easy with json.RawMessage and then let the sdk app logic handle this.

Want to take a stab at #110 ?
I'm quite sure all unit tests pass if we swap out the libs (as we use serde json in the vm). I'm also sure it will be rejected by the deterministic middleware.
It would be interesting to see what could be done in a day on a fork or serde json... maybe looking at the functions in debug build, maybe counting floating point ops in wasm to have some metric of progress.
I would be happier with a serde json fork without floats than the current one


Modify ExampleRulesetExporter to export all registered rules.

Changes:

Removes the old ExampleRulesetExporter , which created and exported a single ruleset.
Releases dependency on HR Essentials mod.

As a result of no longer creating an example ruleset in the configuration mod, it no longer has a dependency on the mod where the rules/rulesets are defined.

Introduce a new ExampleRulesetExporter that exports all registered rulesets as JSON to the path UserData\HouseRules\ExampleRulesets.

In the process, exported rulesets are prepended with the string (Custom) .

Add filename sanitization usable in general export procedures.

Note:
There are two lines in ExampleRulesetExporter.cs:
// Uncomment to export registered rulesets.
// ExportRegisteredRuleset();

These lines may be uncommented to view the effects.  This will be part of the release cycle, so that all registered rulesets are written out, but not during regular use.  This may be made easier with the use of #pragma and the DEBUG (or some other) constant, but is not part of this PR.

Merged, but a question around the settings for exports. Are there any options for NewtonSoft to make it give a more condensed output for easier editing? NewtonSoft seems to really like putting each thing on a separate line.
e.g. this..
      "Rule": "StartCardsModified",
      "Config": {
        "HeroGuardian": [
          { "Card": "HealingPotion", "IsReplenishable": false },
          { "Card": "ReplenishArmor", "IsReplenishable": true },
          { "Card": "Bone", "IsReplenishable": true },

versus...
      "Rule": "StartCardsModified",
      "Config": {
        "HeroBard": [
          {
            "Card": "CallCompanion",
            "IsReplenishable": false
          },
          {
            "Card": "SummonElemental",
            "IsReplenishable": false
          },
          {
            "Card": "HuntersMark",
            "IsReplenishable": true
          }
        ],

Not by default.  Only None and Indented formatting options
The ability to do so is there, but would require writing custom converters.


New version: monkeyWie.Gopeed.Web version 1.6.2

Created by 🥟 Dumplings in workflow run #9561.

Log
Updated: 1.6.1 → 1.6.2
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Installation failed with exit code -1978335187
2024-11-14 23:19:51.834 [FAIL] Installer failed security check. Url: https://github.com/GopeedLab/gopeed/releases/download/v1.6.2/gopeed-web-v1.6.2-windows-386.zip Result: 0x80004005

Automated error analysis suggests -1978335187 may mean Installer failed security check. Another script should retry this every 18 hours.
Automated error analysis suggests 80004005 may mean Resource missing
(Automated response - build 897.)


Refactor base.pp

Make the way we define conditional dependencies consistent and remove
the, (IMO dubious), use of merging multiple parameters into the splat
attribute.

Replaces https://github.com/voxpupuli/puppet-rsyslog/pull/178

Replaces https://github.com/voxpupuli/puppet-rsyslog/pull/178


Miniflare options transform rename

Explicitly name the transformMiniflareOptions function to transformMf2OptionsToMf3Options
resolves #2203

No changeset, technical implementation detail.


Fix l2 block older than l1 origin error

We were encountering the following error when running op-node with a migrated datadir:
ERROR[06-25|16:53:24.779] Derivation pipeline not ready, failed to reset engine err="reset: cannot reset block derivation to start at L2 block 0xdf1dfd6566270260a33f4a33406358e93b8402d4bb6829bb35d685259cc1c3c2:23912612 with time<PHONE_NUMBER> older than its L1 origin 0xbbed3612407993e67f8ca7a423b181837ae164a531941e78f5ee48e766d39cad:1729797 with time<PHONE_NUMBER>, time invariant is broken"

This pr fixes that by using time.Now for the time of the l2 transition block.
We really do need a deterministic time for the migration block so that
all parties that run the migration arrive at the same migration block
but the problem is that op-geth requires that the L2 migration block
(aka l2 origin) occurs after the l1 origin (I guess the point where you
deploy the bridge contracts to the l1). When we migrate a partially
synced datadir the block before the transition block will be very old,
up to 4 years old! So of course it occurs before the l1 origin. So a fix
just to get things working is to use time.Now(), but probably we should
make this a configurable parameter.

Hey @alecps, I was thinking that we might want to use the flag for the actual migration, it would give us flexibility, for example if we defined the celo-blockchain end block but then had some problems deploying the contracts to L1 and ended up with the L1 contract deployment happening after the end block. The thing is you don't want to deploy the l1 contracts too far in advance since op-node will try to "catch up" to ensure that the time between l1 origin and now is filled with the appropriate number of l2 blocks.


Adding RAI attributes to the editor

Following our discussion during the Croissant task force meeting, this pull request attempts to add the Croissant RAI Extension attributes to the editor. I've added a new Tab clearly marked “Extension: Responsible AI.” Where users can fill in the RAI attributes, this kind of "Tab extension" could serve as a guide for more extensions. For instance, a domain-specific one (biology)
Here is a picture of the editor. Any layout and styling change are welcome!

Regarding the pull request:
I've created a new view and an event file for the RAI extension on the editor codebase. I've added the new attributes to the metadata data class in core/state.py. Few more changes have been made to it.
On the other hand, the most critical part has been touching the Python library mlcommons to generate proper JSON to download. I basically touched the structure_graph/nodes/metadata.py, adding the attributes into the class declaration and the to_json() function. To properly use the export function, a new release of the ml-commons library needs to be released as a pip package. If not, I suspect it will generate problems.
Let's talk about it!

recheck

recheck

recheck

Finally, I've managed to properly deserialize the JSON-LD into the editor for RAI attributes. Now, they can be created and loaded using the widget. We plan to add/modify some existing example datasets, adding proper RAI information before the launch.
I've synced the fork because there were conflicting changes in the constant.py, and the PR has become a bit messy. For clarity, we can close this one, reopen a new one, remaking the changes. What do you think?

Closing PR, new PR opened more easy to follow and review at https://github.com/mlcommons/croissant/pull/487


RUN command with multiple lines is not copied

Describe the bug
When using a custom dockerfile with run including multiple lines seperated by , only the first line is being added to the final dockerfile.
To Reproduce
For example using this dockerfile:

RUN  yum -y install oracle-release-el7 oracle-nodejs-release-el7 && \
     yum-config-manager --disable ol7_developer_EPEL && \
     yum -y install oracle-instantclient19.3-basiclite nodejs && \
     rm -rf /var/cache/yum

WORKDIR /myapp
ADD package.json /myapp/
ADD index.js /myapp/
RUN npm install

CMD exec node index.js

Expected behavior
Copying the entire RUN command
Screenshots

Desktop (please complete the following information):
Running on macos

Hi,
can you test if the issue is resolved in the version 0.8.2?
You can install an current build with npm i --save-dev<EMAIL_ADDRESS>


get IUrlHelper inside the Html Helper's method

hi,
is there an easy way to get the IUrlHelper inside the extension method of an IHtmlHelper, besides passing it as a parameter from the view
I tried this:
html.ViewContext.HttpContext.RequestServices.GetService(typeof(IUrlHelper));

but no luck

Get the IUrlHelperFactory and use that to create an IUrlHelper.

@dougbu  urlHelperFactory.GetUrlHelper asks for ActionContext as a parameter, can I get from IHtmlHelper ?

can I get from IHtmlHelper?

Yes, ViewContext is an ActionContext subclass and that's exposed in IHtmlHelper.

Thank You, it works


XCode 8 Ruins Everything

Wait for Qt 5.7.2 to fix the broken build process before continuing development.

Qt's 5.8 (beta) fixes compilation. It is still necessary to launch the application from XCode.


[contracts] Create Domain->ChainId Registry

Need to have onchain reference for this.

IMO this can be non-MVP. It's a utility, so we can launch with simply listing domains in docs and then build this after.

cleaning up old stuff


Hardhat testing calldata execution on another contract failed.

  it("should create a new proposal for setting a token address and execute it after voting", async function () {
    this.timeout(90000);
    const { finCubeDAO, owner } = await loadFixture(deployFinCubeDAOFixture);

    const token = "0x1234567890123456789012345678901234567890"; // Replace with actual token address

    // Encode the function call data
    const calldata = tokenAddressContract.interface.encodeFunctionData("set", [token]);
    // const calldata = '0x2801617e0000000000000000000000001234567890123456789012345678901234567890'; // taken from remix and doesnt work either

    // Create a new proposal for setting a token address
    await finCubeDAO.connect(owner).newProposal(calldata, tokenAddressContract.runner.address);
   // console.log(tokenAddressContract);
    // Vote on the proposal
    await new Promise(resolve => setTimeout(resolve, 6000));
    // Cast a 'yes' vote on the proposal
    await finCubeDAO.connect(owner).castVote(0, true);

    // Execute the proposal
    await new Promise(resolve => setTimeout(resolve, 65000));
    console.log("amfdusdgdsgsdgsdofd");
    await fincubeDAO.connect(owner).executeProposal(0);
    console.log(calldata);
    // Check that the token address was set correctly
    const tokenAddress = await (tokenAddressContract.getToken());
    console.log(tokenAddress, token);

    expect(tokenAddress).to.equal(token);
  });


app.json "test" script for Heroku CI

This allows tests to automatically run in the (currently in beta) Heroku CI service.

poke :)

Any chance to get this merged before SymfonyCon begins @javiereguiluz? :)

I know nothing about Heroku ... so I cannot review or test this myself. I was waiting to receive any approval/reject comment from anyone.
Luckily you are trustworthy, so I guess I'll have to merge this blindly.

Has CI already worked on Heroku for users? I triggered an error in unit tests here, but Heroku still successfully build the app without any errors.
Should we wait until this feature will be stable and available for all Heroku users? I see you mentioned it's currently in beta.

@dzuelke is there any doc about Heroku CI anywhere ? I haven't found any

It's in private beta, @bocharsky-bw and @stof. I'm also happy to wait with this PR until it's GA, @javiereguiluz :)

This is now in public beta, so merge away, @stof or @bocharsky-bw :)

Thank you @dzuelke.


Is it possible, to send gif animation file through Bluetooth to ESP32 ?

Greetings,
Is it possible, to send gif animation file through Bluetooth to ESP32, then save it to SD card, then show it in RGB Panel as usual...
Best regards

Idk about Bluetooth but I did the same with WIFI, by using an ASYNC webserver to send gif files to the spiff and play them.
here it is if you are intrested.
https://github.com/mzashh/HUB75-Pixel-Art-Display

Very kind from You for this advice... I'll check it out.
But I'm still want to do this  using only Bluetooth, I think WiFi link need a lot of additional functions to be accomplished, or I'm wrong? Thanks again, and thanks for those who give me advice like this

Very kind from You for this advice... I'll check it out.
But I'm still want to do this using only Bluetooth, I think WiFi link need a lot of additional functions to be accomplished, or I'm wrong? Thanks again, and thanks for those who give me advice like this

imo WIFI on the ESP32 is more simpler than Bluetooth, as you need a dedicated app or something to send files to the ESP32, WIFI is much simpler, my code above has a simple WEB-UI to upload GIFs which can be accessed via any WIFI capable device on your network. . Why don't you give it a try and see if it suits your needs.

What about the power consumption, is WiFi less or more energy demand than Bluetooth? Thanks in advance.

BLE should have considerably low power consumption than WIFI, on avg I had 200mA going to the ESP32 but considering that you are using a LED matrix I don't think it should be significant, also you can disable the WIFI in code when not in use

Enabling Bluetooth on the ESP32 will also chew a lot of memory that might be of better use.
If the sketch already uses WiFi it would probably easier and more reliable to to use that + mDNS for discovery.


ps: Fix the time code

Except:
❯ ps
    PID TTY          TIME CMD
  16643 pts/1    00:00:03 zsh
  16660 pts/1    00:00:00 zsh
  16733 pts/1    00:00:00 zsh
  16734 pts/1    00:00:00 zsh
  16736 pts/1    00:00:00 gitstatusd-linu
 117274 pts/1    00:00:00 ps

But:
❯ cargo run ps
    Finished `dev` profile [unoptimized + debuginfo] target(s) in 1.85s
     Running `target/debug/procps ps`
 PID     TTY    TIME      CMD
 16643   pts/1  00:00:00  zsh
 16660   pts/1  00:00:00  zsh
 16733   pts/1  00:00:00  zsh
 16734   pts/1  00:00:00  zsh
 16736   pts/1  00:00:00  gitstatusd-linu
 116825  pts/1  00:00:00  procps

By the way, I noticed that the GNU's implementation keep 3 space for [DD:] display.


Traceflow failed when using tcp protocol

Describe the bug
I have two pod: antrea-octant and nginx. I would like to traceflow packet from antrea-octant to nginx using tcp protocol.
root@slave1:antrea# kubectl get nodes
NAME     STATUS   ROLES           AGE    VERSION
master   Ready    control-plane   4d1h   v1.27.1
slave1   Ready    <none>          4d1h   v1.27.1

root@slave1:antrea# kubectl get po -A -owide|grep -Ei "nginx|octant"
kube-system   antrea-octant-d446dfb7f-sxs69      1/1     Running   0               58m    <IP_ADDRESS>       master   <none>           <none>
ns-test       nginx-deployment-f6dc544c7-szxv7   1/1     Running   0               29m    <IP_ADDRESS>      slave1   <none>           <none>

To Reproduce

create traceflow crd yaml

apiVersion: crd.antrea.io/v1alpha1
kind: Traceflow
metadata:
  name: tcp-tf1
spec:
  #liveTraffic: true
  destination:
    namespace: ns-test
    pod: nginx-deployment-f6dc544c7-szxv7
  source:
    namespace: kube-system
    pod: antrea-octant-d446dfb7f-sxs69
    # destination can also be an IP address ('ip' field) or a Service name ('service' field); the 3 choices are mutually exclusive.
  packet:
    ipHeader: # If ipHeader/ipv6Header is not set, the default value is IPv4+ICMP.
      protocol: 6 # Protocol here can be 6 (TCP), 17 (UDP) or 1 (ICMP), default value is 1 (ICMP)
    transportHeader:
      tcp:
        srcPort: 50012 # Source port needs to be set when Protocol is TCP/UDP.
        dstPort: 80 # Destination port needs to be set when Protocol is TCP/UDP.

Check the result

root@master:antrea_yaml# kubectl get traceflow  -oyaml
apiVersion: v1
items:
- apiVersion: crd.antrea.io/v1alpha1
  kind: Traceflow
  metadata:
    annotations:
      kubectl.kubernetes.io/last-applied-configuration: |
        {"apiVersion":"crd.antrea.io/v1alpha1","kind":"Traceflow","metadata":{"annotations":{},"name":"tcp-tf1"},"spec":{"destination":{"namespace":"ns-test","pod":"nginx-deployment-f6dc544c7-szxv7"},"packet":{"ipHeader":{"protocol":6},"transportHeader":{"tcp":{"dstPort":80,"srcPort":50012}}},"source":{"namespace":"kube-system","pod":"antrea-octant-d446dfb7f-sxs69"}}}
    creationTimestamp: "2023-04-25T09:25:25Z"
    generation: 1
    name: tcp-tf1
    resourceVersion: "510069"
    uid: b2968d1a-bc83-4c65-9be3-1d8488ee5ebc
  spec:
    destination:
      namespace: ns-test
      pod: nginx-deployment-f6dc544c7-szxv7
    packet:
      ipHeader:
        protocol: 6
      transportHeader:
        tcp:
          dstPort: 80
          srcPort: 50012
    source:
      namespace: kube-system
      pod: antrea-octant-d446dfb7f-sxs69
  status:
    phase: Failed
    reason: Traceflow timeout
    startTime: "2023-04-25T09:25:25Z"
kind: List
metadata:
  resourceVersion: ""

The result is failed.
I use antctl command is the same result.
root@master:/# antctl traceflow -S kube-system/antrea-octant-d446dfb7f-sxs69 -D ns-test/nginx-deployment-f6dc544c7-szxv7 -f tcp,tcp_dst=80
name: kube-system-antrea-octant-d446dfb7f-sxs69-to-ns-test-nginx-deployment-f6dc544c7-szxv7-7c6lfq22
phase: Failed
reason: Traceflow timeout
source: kube-system/antrea-octant-d446dfb7f-sxs69
destination: ns-test/nginx-deployment-f6dc544c7-szxv7

Expected
Traceflow is succeed when using tcp protocol from a pod to anther.
Actual behavior
Traceflow is failed.
Versions:

kubernetes version

root@master:openvswitch# kubectl version
WARNING: This version information is deprecated and will be replaced with the output from kubectl version --short.  Use --output=yaml|json to get the full version.
Client Version: version.Info{Major:"1", Minor:"27", GitVersion:"v1.27.1", GitCommit:"4c9411232e10168d7b050c49a1b59f6df9d7ea4b", GitTreeState:"clean", BuildDate:"2023-04-14T13:21:19Z", GoVersion:"go1.20.3", Compiler:"gc", Platform:"linux/amd64"}
Kustomize Version: v5.0.1
Server Version: version.Info{Major:"1", Minor:"27", GitVersion:"v1.27.1", GitCommit:"4c9411232e10168d7b050c49a1b59f6df9d7ea4b", GitTreeState:"clean", BuildDate:"2023-04-14T13:14:42Z", GoVersion:"go1.20.3", Compiler:"gc", Platform:"linux/amd64"}

antrea version

root@master:~# kubectl exec -itn kube-system   antrea-agent-4p2pq    bash -- antctl version
agentVersion: 1.12.0-dev-1a4e331.clean
antctlVersion: v1.12.0-dev-1a4e331

openvswitch version

root@master:~# kubectl exec -itn kube-system   antrea-agent-4p2pq -c antrea-ovs    -- ovs-vsctl --version
ovs-vsctl (Open vSwitch) 2.17.5
DB Schema 8.3.0

openvswitch debug information:

create openflow

2023-04-25T09:25:25.174Z|745011|vconn|DBG|unix#0: received: OFPT_BUNDLE_CONTROL (OF1.5) (xid=0x12249):
 bundle_id=0xf7 type=OPEN_REQUEST flags=atomic
2023-04-25T09:25:25.174Z|745012|vconn|DBG|unix#0: sent (Success): OFPT_BUNDLE_CONTROL (OF1.5) (xid=0x12249):
 bundle_id=0xf7 type=OPEN_REPLY flags=0
2023-04-25T09:25:25.174Z|745013|poll_loop|DBG|wakeup due to [POLLIN] on fd 53 (/var/run/openvswitch/br-int.mgmt<->) at lib/stream-fd.c:157 (0% CPU usage)
2023-04-25T09:25:25.175Z|745014|vconn|DBG|unix#0: received: OFPT_BUNDLE_ADD_MESSAGE (OF1.5) (xid=0x1224c):
 bundle_id=0xf7 flags=atomic
OFPT_FLOW_MOD (OF1.5) (xid=0x1224c): ADD table:7 priority=191,ip,nw_tos=28 cookie:0x2070000000000 hard:20 actions=goto_table:8
2023-04-25T09:25:25.175Z|745015|vconn|DBG|unix#0: received: OFPT_BUNDLE_ADD_MESSAGE (OF1.5) (xid=0x1224e):
 bundle_id=0xf7 flags=atomic
OFPT_FLOW_MOD (OF1.5) (xid=0x1224e): ADD table:7 priority=192,ct_state=+rpl+trk,ip,nw_tos=28 cookie:0x2070000000000 hard:20 actions=drop
2023-04-25T09:25:25.175Z|745016|vconn|DBG|unix#0: received: OFPT_BUNDLE_ADD_MESSAGE (OF1.5) (xid=0x12250):
 bundle_id=0xf7 flags=atomic
OFPT_FLOW_MOD (OF1.5) (xid=0x12250): ADD table:28 priority=203,ip,reg0=0x100/0x100,reg1=0x1,nw_tos=28 cookie:0x2070000000000 hard:20 actions=output:NXM_NX_REG1[],meter:2,controller(max_len=128,id=8339)
2023-04-25T09:25:25.175Z|745017|vconn|DBG|unix#0: received: OFPT_BUNDLE_ADD_MESSAGE (OF1.5) (xid=0x12252):
 bundle_id=0xf7 flags=atomic
OFPT_FLOW_MOD (OF1.5) (xid=0x12252): ADD table:28 priority=202,ip,reg0=0x100/0x100,reg1=0x2,nw_tos=28 cookie:0x2070000000000 hard:20 actions=meter:2,controller(max_len=128,id=8339)
2023-04-25T09:25:25.175Z|745018|vconn|DBG|unix#0: received: OFPT_BUNDLE_ADD_MESSAGE (OF1.5) (xid=0x12254):
 bundle_id=0xf7 flags=atomic
OFPT_FLOW_MOD (OF1.5) (xid=0x12254): ADD table:28 priority=203,ip,reg0=0x100/0x100,reg1=0x2,nw_dst=<IP_ADDRESS>,nw_tos=28 cookie:0x2070000000000 hard:20 actions=meter:2,controller(max_len=128,id=8339)
2023-04-25T09:25:25.175Z|745019|vconn|DBG|unix#0: received: OFPT_BUNDLE_ADD_MESSAGE (OF1.5) (xid=0x12256):
 bundle_id=0xf7 flags=atomic
OFPT_FLOW_MOD (OF1.5) (xid=0x12256): ADD table:28 priority=202,ip,reg0=0x100/0x100,nw_tos=28 cookie:0x2070000000000 hard:20 actions=meter:2,controller(max_len=128,id=8339)
2023-04-25T09:25:25.175Z|745020|vconn|DBG|unix#0: received: OFPT_BUNDLE_ADD_MESSAGE (OF1.5) (xid=0x12258):
 bundle_id=0xf7 flags=atomic
OFPT_FLOW_MOD (OF1.5) (xid=0x12258): ADD table:28 priority=212,ct_mark=0x40/0x40,ip,nw_tos=28 cookie:0x2070000000000 hard:20 actions=meter:2,controller(max_len=128,id=8339)
2023-04-25T09:25:25.175Z|745021|vconn|DBG|unix#0: received: OFPT_BUNDLE_CONTROL (OF1.5) (xid=0x12259):
 bundle_id=0xf7 type=CLOSE_REQUEST flags=atomic
2023-04-25T09:25:25.175Z|745022|vconn|DBG|unix#0: sent (Success): OFPT_BUNDLE_CONTROL (OF1.5) (xid=0x12259):
 bundle_id=0xf7 type=CLOSE_REPLY flags=0
2023-04-25T09:25:25.175Z|745023|poll_loop|DBG|wakeup due to [POLLIN] on fd 53 (/var/run/openvswitch/br-int.mgmt<->) at lib/stream-fd.c:157 (0% CPU usage)
2023-04-25T09:25:25.175Z|745024|vconn|DBG|unix#0: received: OFPT_BUNDLE_CONTROL (OF1.5) (xid=0x1225b):
 bundle_id=0xf7 type=COMMIT_REQUEST flags=atomic
2023-04-25T09:25:25.175Z|745025|vconn|DBG|unix#0: sent (Success): OFPT_BUNDLE_CONTROL (OF1.5) (xid=0x1225b):
 bundle_id=0xf7 type=COMMIT_REPLY flags=0

packet_out, miss upcall

2023-04-25T09:25:27.176Z|745040|vconn|DBG|unix#0: received: OFPT_PACKET_OUT (OF1.5) (xid=0x1225d): in_port=4 actions=set_field:28/0xfc->nw_tos,TABLE data_len=54
tcp,vlan_tci=0x0000,dl_src=b2:1c:bd:da:c4:15,dl_dst=f6:29:3c:59:1d:6f,nw_src=<IP_ADDRESS>,nw_dst=<IP_ADDRESS>,nw_tos=0,nw_ecn=0,nw_ttl=64,nw_frag=no,tp_src=50012,tp_dst=80,tcp_flags=0 tcp_csum:a691
2023-04-25T09:25:27.177Z|745041|dpif|DBG|Dropped 5 log messages in last 0 seconds (most recently, 0 seconds ago) due to excessive rate
2023-04-25T09:25:27.177Z|13211|poll_loop(handler2)|DBG|wakeup due to [POLLIN] on fd 28 (unknown anon_inode:[eventpoll]) at lib/dpif-netlink.c:3183
2023-04-25T09:25:27.177Z|11008|poll_loop(handler6)|DBG|wakeup due to [POLLIN] on fd 31 (unknown anon_inode:[eventpoll]) at lib/dpif-netlink.c:3183
2023-04-25T09:25:27.177Z|11567|poll_loop(handler4)|DBG|wakeup due to [POLLIN] on fd 30 (unknown anon_inode:[eventpoll]) at lib/dpif-netlink.c:3183 (0% CPU usage)
2023-04-25T09:25:27.177Z|14340|poll_loop(handler1)|DBG|wakeup due to [POLLIN] on fd 27 (unknown anon_inode:[eventpoll]) at lib/dpif-netlink.c:3183
2023-04-25T09:25:27.177Z|12390|poll_loop(handler3)|DBG|wakeup due to [POLLIN] on fd 29 (unknown anon_inode:[eventpoll]) at lib/dpif-netlink.c:3183
2023-04-25T09:25:27.177Z|745042|dpif|DBG|system@ovs-system: execute set(ipv4(tos=0x1c/0xfc)),ct(zone=65520,nat),recirc(0x8b) on packet tcp,vlan_tci=0x0000,dl_src=b2:1c:bd:da:c4:15,dl_dst=f6:29:3c:59:1d:6f,nw_src=<IP_ADDRESS>,nw_dst=<IP_ADDRESS>,nw_tos=0,nw_ecn=0,nw_ttl=64,nw_frag=no,tp_src=50012,tp_dst=80,tcp_flags=0 tcp_csum:a691
 with metadata skb_priority(0),skb_mark(0),in_port(4) mtu 0
2023-04-25T09:25:27.177Z|13212|dpif(handler2)|DBG|system@ovs-system: miss upcall:
recirc_id(0x8b),dp_hash(0),skb_priority(0),in_port(4),skb_mark(0),ct_state(0x30),ct_zone(0xfff0),ct_mark(0),ct_label(0),eth(src=b2:1c:bd:da:c4:15,dst=f6:29:3c:59:1d:6f),eth_type(0x0800),ipv4(src=<IP_ADDRESS>,dst=<IP_ADDRESS>,proto=6,tos=0x1c,ttl=64,frag=no),tcp(src=50012,dst=80),tcp_flags(0)
tcp,vlan_tci=0x0000,dl_src=b2:1c:bd:da:c4:15,dl_dst=f6:29:3c:59:1d:6f,nw_src=<IP_ADDRESS>,nw_dst=<IP_ADDRESS>,nw_tos=28,nw_ecn=0,nw_ttl=64,nw_frag=no,tp_src=50012,tp_dst=80,tcp_flags=0 tcp_csum:a691
2023-04-25T09:25:27.177Z|13213|ofproto_dpif_upcall(handler2)|DBG|upcall: no reference for recirc flow

I found have no packet_in information from openvswitch to antrea-agent controller.

@gujun4990 thanks for the report. It's perhaps because the tcp flags was not set, causing the packet to be dropped by connection state check (a TCP packet with neither of SYN, ACK, ... flags is considered invalid). You may add "tcp_flags=2" to construct a SYN packet.
That being said, I'm not sure why the example in antctl traceflow  doesn't set tcp_flags. Perhaps there was a default flags set on server side but removed sometime ago or the example never worked. Regardless, this is indeed an issue which should be fixed, as the other two prococols, udp and icmp can handle the defaulting correctly.

Thanks for your reply, I add "tcp_flags=2" options and traceflow is succeed. The shouldn't a bug, but maybe need to optimize the documents about traceflow.
BTW, I found only a request packet from orig to dest, not a reply packet. If the reply packet is dropped, the connection shouldn't failed among pods. But the traceflow is succeed actually.

Thanks for your reply, I add "tcp_flags=2" options and traceflow is succeed. The shouldn't a bug, but maybe need to optimize the documents about traceflow. BTW, I found only a request packet from orig to dest, not a reply packet. If the reply packet is dropped, the connection should be failed among pods. But the traceflow is succeed actually. So I wonder whether the openflow table do something that I miss.

The injected Traceflow packets are discarded intentionally (even if the trace result is ALLOW) before being forwarded to Pod interface to avoid affecting applications.


Include proxy in ENV on converge

This patch adds the http_proxy and https_proxy
to the creation of the Dockerfile so that
RUN apt-get update works behind the proxy.
A better approach would probably be to postpone the
RUN commands and execute them interactively after the
inital converge.
NOTE: This is not recommended for environments were the proxy settings change. Because this will mess up the container cache. So switching to a non proxy environment uses the cached proxy settings from the layer before/below
Fixes: #72

This should be working https://github.com/portertech/kitchen-docker/blob/03b2a30d66a7a35ad9f2c5b16511651de6390deb/lib/kitchen/driver/docker.rb#L226-L227

@portertech it appears to run all the proxy env configuration after it attempts to set everything else up (ie; yum/apt package installs) Making every build behind a proxy fail on Step 2 of the created Dockerfile.

Got added a while ago, sorry for not updating this ticket.


Support propagation determined when task is called

The current implementation requires the entire app propagate parent spans. I have a use case where when a task is queued, we would like to make that determination by sending a custom header.

@AgDude, is this header used for other purposes or just for determining whether to propagate?

This header would be used only to determine if spans should propagate.


Dependency updates

Rails
IRB
Nokogiri
Public Suffix
Rack
Reline
Rubocop
Sentry

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.429%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
348

Relevant Lines:
350

💛  - Coveralls

The tests pass and the review app looks like it is functioning correctly - so merging this without code review as usual for dep updates.